Skip to content

fix(registry): opis server.json poniżej limitu 100 znaków + retry publikacji do rejestru - #21

Merged
woladi merged 1 commit into
masterfrom
woladi/fix-registry-description
Aug 24, 2026
Merged

woladi merged 1 commit into
masterfrom
woladi/fix-registry-description

Conversation

@woladi

@woladi woladi commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Naprawa 422 z rejestru MCP. Publikacja v0.7.5 na npm poszła (OIDC Trusted Publishing potwierdzone), ale job rejestru padł: description w server.json miało 105 znaków przy limicie 100. Zastany błąd w manifeście, nie regresja migracji. Opis skrócony do 93 znaków.
  • Test zamiast niespodzianki. tests/server-manifest.test.ts sprawdza w CI ograniczenia rejestru (długość opisu, zgodność name z package.json#mcpName, identyfikator npm) — rejestr waliduje dopiero przy publikacji, czyli już PO wypuszczeniu na npm, więc taki błąd zostawia rejestr wersję w tyle.
  • Ścieżka retry. workflow_dispatch z publish_registry=true powtarza samą połowę rejestrową względem bieżącego server.json, z weryfikacją, że dana wersja faktycznie istnieje na npm.

Test plan

  • npm run lint, npm run test:run (256 testów) — zielone
  • Nowy test wychwytuje stary, 105-znakowy opis
  • Po merge: gh workflow run release.yml -f publish_registry=true aktualizuje wpis w rejestrze na 0.7.5

🤖 Generated with Claude Code

The MCP registry rejected the v0.7.5 publish with a 422: the description was
105 characters against a limit of 100. It only validates at publish time, i.e.
after npm has already gone out, so the registry sat a version behind while npm
was current.

- trim the description to 93 characters
- add tests/server-manifest.test.ts so CI catches the registry's constraints
  (description length, server name vs package.json#mcpName, npm identifier)
  before a release instead of halfway through one
- allow `workflow_dispatch` with publish_registry=true to re-run just the
  registry half against the current server.json, and verify the target version
  actually exists on npm before pointing the registry at it

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@woladi
woladi merged commit 20da3ba into master Aug 24, 2026
1 check passed
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.

1 participant