Repository navigation
Refresh the MCP catalog snapshot so validate includes mu - #72
Merged
Merged
Conversation
The directory entry landed without regenerating mcp/catalog.snapshot.json, so the committed snapshot comparison failed on main. Rewrote the snapshot with site export-catalog. Co-authored-by: Kieran <bubbleptr@users.noreply.github.com>
export-catalog rewrote every package's stars and downloads. Restored those fields from the previous snapshot, matching earlier directory updates. mu keeps the stats from the current snapshot. Co-authored-by: Kieran <bubbleptr@users.noreply.github.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
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.
Why
Production deploy of
mainat8fcacea(chore: 更新资源统计数据) failedcd site && bun run validate. The failing test wasthe committed MCP snapshot matches the catalog aside from the stats snapshot.muwas added to Alternative Distributions infc6eba9without regeneratingmcp/catalog.snapshot.json. That comparison ignores stars, downloads, trend, repo URL, andstatsGeneratedAt, so the later stats commit did not create the mismatch. It deployed a tree that was already missingmu, and the distributions count was still 1.What
Regenerated the committed snapshot with the documented exporter:
Then restored stars, weekly downloads, and trend on packages that were already in the snapshot, which is how earlier directory updates kept weekly stats out of this file. The net diff adds
mu(mu-a1e735dd, with its current stats) and sets the distributions / alternative-distributions counts to 2.Verification
cd site && bun run validatepassed locally after the final snapshot: 85 unit tests (including the snapshot comparison),astro check, the static build, and 42 rendered tests.cd mcp && npm testpassed (12 tests).