docs(spec): stop claiming install targets register MCP servers - #1223
Merged
Conversation
The spec described MCP registration as part of the Tier 1 install pipeline and, per target, as something the claude, zed and continue targets do. None of them do: every '--target <tool>' installs skills and rules only, and MCP registration belongs to 'egc init' and the shell installers. This is the same false claim #1206 corrected in the installation guide; it survived here. The runtime map row also pointed at tests/test_orchestrator.py as its validator, a file removed in #307. It is now listed among the honest gaps the spec already tracks, alongside the schema's real consumers. Signed-off-by: Felipe Marzochi <fmarzochi@gmail.com>
|
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Fmarzochi
added a commit
that referenced
this pull request
Aug 6, 2026
The spec's own semver rule assigns PATCH to documentation and non-contract changes. #1223 corrected what the spec said about MCP registration and about a validator that no longer exists, without touching any contract, so the spec version moves one patch. Signed-off-by: Felipe Marzochi <fmarzochi@gmail.com>
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
1 participant
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.
What was wrong
The executable spec under
docs/spec/had drifted from the code in two ways:1. MCP registration attributed to install targets (three places). The Tier 1 row listed "MCP" among what the unified pipeline ships, and the per-tool table said the
claude,zedandcontinuetargets register MCP servers. Verified against the code:claude-home.js,zed-home.jsandcontinue-home.jscontain zero MCP registration; the whole registry lives inscripts/lib/mcp-register.js, driven byegc initand the shell installers. This is exactly the claim #1206 corrected indocs/installation.md("--target <tool>only ever installs skills and rules"), which survived untouched in the spec.2. A validator that no longer exists. The runtime-map row pointed at
tests/test_orchestrator.py, deleted in #307. The schema's real consumers (scripts/runtime/discovery.js,scripts/orchestration/router.py) have no dedicated validator, so the row now sits with the other honest gaps the spec already tracks rather than claiming coverage it does not have.What was checked and found correct
SUPPORTED_INSTALL_TARGETSidentifiers in the compatibility commitments matchinstall-manifests.jsexactly..kiro/install.sh,.trae/install.sh) exist and are executable.resolve-egc-root.js.Documentation only, no contract change, so
SPEC_VERSIONstays at 0.1.0.Verification
tests/spec/integration-tiers.test.js5/5,tests/scripts/bootstrap-cognitive.test.js50/50,tests/hooks/doc-file-warning.test.js60/60,tests/ci/agent-yaml-surface.test.js4/4.Summary by cubic
Fixed the executable spec to stop claiming Tier 1 install targets register MCP servers and to reflect that the runtime map validator no longer exists. Targets install skills and rules only; MCP registration is handled by
egc initand the shell installers.claude,zed, andcontinuetargets don’t register MCP servers.scripts/runtime/discovery.jsandscripts/orchestration/router.py.Written for commit 7dc651d. Summary will update on new commits.