refactor(runtime): consolidate ECC runtime and Studio integration - #225
Merged
Yell-walkalone merged 119 commits intoSep 16, 2026
Merged
Conversation
Keep Studio Dashboard and Project Comparison on committed Snapshot QoR extension fields, and point ECC at interactive run snapshot commits.
…time-consolidation-studio # Conflicts: # ecos/gui/apps/desktop-electron/electron/services/qorAnalysis.ts # ecos/gui/apps/renderer/src/composables/useHomeSnapshots.test.ts # ecos/gui/apps/renderer/src/composables/useHomeSnapshots.ts # ecos/gui/apps/renderer/src/utils/projectManagement.test.ts # ecos/gui/apps/renderer/src/utils/projectManagement.ts # ecos/gui/apps/renderer/src/utils/projectQorTrend.test.ts # ecos/gui/apps/renderer/src/views/project-management/ProjectAnalysisPanel.vue # ecos/gui/apps/renderer/src/views/project-management/projectDashboard.fixture.ts # ecos/gui/apps/renderer/src/views/project-management/projectDashboard.test.ts # ecos/gui/apps/renderer/src/views/project-management/projectDashboard.ts # ecos/gui/apps/renderer/src/views/project-management/projectWorkspaceAnalysisData.test.ts # ecos/gui/apps/renderer/src/views/project-management/projectWorkspaceAnalysisData.ts # ecos/gui/packages/shared/src/utils/projectManagementSummary.test.ts # ecos/gui/packages/shared/src/utils/projectManagementSummary.ts
…zard The filelist picker filtered the dialog to extension matches only, so the extensionless ECC convention (origin/filelist) was greyed out and the path validator rejected it as well. Add an All Files filter entry and accept filelist basenames without an extension.
Move process specs, ADRs, and research notes out of the published ecos/docs tree. The root docs/ directory is gitignored, so stop force-tracking those local working documents as well.
Move CONTEXT.md, CONTEXT-MAP.md, and the backend-architecture diagram artifacts into the gitignored root docs/ directory so they stay local working notes instead of published repository files.
Drop the brief product brief until it can be rewritten with enough substance. Inline the one Agent permission-model citation that pointed at it.
Default workspace names like ws_0001 were indistinguishable across designs.
KoEkko
changed the title
Yell-walkalone
approved these changes
Sep 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR removes Studio's duplicate runtime adapter and makes ECC the single owner of workspace execution.
ecc rpc serve; workspace creation/opening, Flow execution, cancellation, recovery, snapshots, and QoR all use ECC's consolidated runtime contracts.eccgitlink is pinned to83fe68d15bc859143ce04dbbf4f4dd3bdad6c85a3, the latest publishedecc/maincommit after ECC #284, so Studio and ECC use the same runtime contract.PATH, and the workspace Agent panel can be collapsed.Closes #220
Closes #222
Scope
Select the areas touched by this PR:
ecos/gui, including renderer, Electron, and shared packages.The documentation changes are part of the runtime consolidation, not a docs-only change.
Validation
List the commands you ran. Mark checks that are not applicable as N/A.
cd ecos/gui && pnpm run typecheckcd ecos/gui && pnpm run testcd ecos/gui && pnpm run buildmake buildmake demo-gcdmake demo-retrosoccd ecos/gui && pnpm run devcd ecos/gui && pnpm run checkpython3 .github/scripts/check-version.pygit diff --checkpnpm install --frozen-lockfilealso completed successfully.Skipped checks and reason:
pnpm run build: not run; this update changes only the ECC gitlink and PR metadata, with no packaging configuration changes.make build/ demos: no AppImage, version, or packaging inputs changed. Full Linux release packaging was not run locally.Screenshots or Recordings
Required for visible GUI changes.
Not attached in this session. The visible GUI changes are covered by the existing PR scope; an Electron visual check remains before merge.
Release, Packaging, and Runtime Impact
Notes:
83fe68d15bc859143ce04dbbf4f4dd3bdad6c85a3(ecc/main). This is the latest published ECC main commit and contains the merged ECC #284 runtime consolidation.Checklist