Repository navigation
chore: remove MCP registry publishing - #55
Merged
Merged
Conversation
The fork publishes to npm only. Upstream's registry workflow triggered on `chrome-devtools-mcp-v*` tags, so it never fired for an Opera release, but `workflow_dispatch` left it manually publishable. Deletes the workflow along with `server.json` and the `verify-server-json-version` script that existed only to feed it. A release now bumps `package.json` and `package-lock.json`.
The merge driver does not apply to a delete. Git reports a modify/delete conflict and leaves upstream's copy in the tree, so `git add -A` during an intake silently restores the file, and registering the path made the seam check accept it in any state. Every path in the rename-or-delete table must now be absent from the working tree. `server.json` makes this worth enforcing: upstream bumps it in every release commit, so it conflicts on each intake that spans a release.
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
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.
The fork publishes to npm only, never to the MCP registry — the registry has no
opera-devtools-mcpentry.publish-to-mcp-registry-on-tag.ymlcame from upstream and triggers onchrome-devtools-mcp-v*tags, so it has never fired for an Opera release. It also hasworkflow_dispatch, so it could still be published by hand.Removed:
.github/workflows/publish-to-mcp-registry-on-tag.ymlserver.json— the registry manifest, its only consumerscripts/verify-server-json-version.tsand theverify-server-json-versionnpm scriptpre-release.ymlserver.json merge=opera-oursrow in.gitattributesA release now bumps
package.jsonandpackage-lock.jsoninstead of three files.Intake safety
Deleting
server.jsonis not free: upstream bumps it in every release commit, so it will conflict on each intake that spans an upstream release. Two things make that worse than it looks:merge=opera-oursdoes not apply to a delete. Git reportsCONFLICT (modify/delete)and leaves upstream's copy in the tree, so a reflexivegit add -Arestores the file.docs/UPSTREAM.mdmadeverify-upstream-seamaccept it in any state, deleted or restored — so nothing caught the resurrection.Second commit closes that: every path in the rename-or-delete table must now be absent from the working tree.
Verified by restoring
server.jsonand a renamed e2e test in turn — the check fails on each and passes on a clean tree. All nine paths already in that table satisfy it.npm run genproduces no drift; format, typecheck andverify-upstream-seampass locally.