Skip to content

Drop the MCP Registry workflow from the release skill - #269

Open
keypair34 wants to merge 1 commit into
developmentfrom
fix/release-skill-registry-verification
Open

keypair34 wants to merge 1 commit into
developmentfrom
fix/release-skill-registry-verification

Conversation

@keypair34

Copy link
Copy Markdown
Collaborator

The MCP Registry listing moved to getsigit/si (commit 5aa0be3), so this repo no longer has a server.json or a release-mcp-registry.yml. The release skill still described both as part of the flow, which sends whoever is cutting a release looking for files that aren't there.

This replaces that with what the flow actually does now, and fills two gaps in the checklist that cost time during 1.7.6:

  • winget goes live late. A green release-winget.yml run only means a manifest PR was opened against microsoft/winget-pkgs. Until a human merges it, winget install still serves the previous version. The checklist now says to report it as "PR opened, awaiting review" with the link, not as done and not as a failure.
  • Verify each registry by its real published name. PyPI is sigit-code, not sigit — that name belongs to an unrelated package stuck at 0.0.58, so checking the wrong one looks like the release failed. Same for the rest: crates.io sigit, npm under both @getsigit and @smbcloud, NuGet SiGit.Code, plus the tap paths for Homebrew and Scoop. Also notes that crates.io 403s a bare curl and needs a User-Agent.

Docs only — no code changes.

The listing moved to getsigit/si, so this repo has no
release-mcp-registry.yml and a release note that points at one sends the
reader looking for a file that is not there. Replace it with what the
flow actually does now, and add the two post-release checks that were
missing: winget only goes live when its manifest PR merges, and each
registry has to be verified by the name it publishes under (PyPI is
`sigit-code`, not `sigit`).

Co-Authored-By: siGit Code v1.7.5-acp <noreply@sigit.si>

@sigit-code-review sigit-code-review Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change updates the release skill documentation to reflect the removal of the MCP Registry workflow and server.json from this repository, as the MCP Registry listing is now maintained in getsigit/si. It clarifies that the MCP server listing is not part of the siGit Code release flow, and corrects the workflow and checklist to match the current process. The checklist now explicitly notes that winget releases are not live until the PR is merged, and instructs to verify each registry by its actual published name, addressing past confusion with PyPI and other registries. No code changes are involved—this is a documentation-only update.


Automated review by siGit Code Review · commit 60996b8 · see the review dashboard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants