Skip to content

fix: remove unverified Smithery manifest - #6

Closed
lukisch wants to merge 3 commits into
masterfrom
codex/remove-unverified-smithery
Closed

fix: remove unverified Smithery manifest#6
lukisch wants to merge 3 commits into
masterfrom
codex/remove-unverified-smithery

Conversation

@lukisch

@lukisch lukisch commented Jul 29, 2026

Copy link
Copy Markdown
Member

Summary

  • remove the legacy Smithery manifest that no longer matches the current local-stdio publishing contract
  • remove the package whitelist entry so the invalid manifest is not shipped
  • document the correction in the changelog

Verification

  • npm audit --audit-level=moderate (0 vulnerabilities)
  • npm run build
  • npm test (175 passed)
  • npm pack --dry-run (no smithery.yaml)

@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

Welcome! 👋 Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what and why

Thanks for contributing!

lukisch pushed a commit that referenced this pull request Aug 6, 2026
Lukas Geiger @claude
fix: Node.js-Untergrenze von 18 auf 20 anheben
a883264 
Die CI-Matrix testet 20/22/24 und @hono/node-server 2.x verlangt Node 20 —
die oeffentlich deklarierte Untergrenze ">=18" war damit nachweislich
falsch. Angehoben in package.json (engines), package-lock.json, beiden
READMEs (Badge + Voraussetzungen) und llms.txt.

Inhalt stammt aus PR #6, der nicht direkt gemergt werden konnte: er zweigt
von 1.3.18 ab und haette das Lockfile vom aktuellen Stand 1.3.22 zurueck-
gedreht. Der zweite Teil jenes PRs (Entfernen von smithery.yaml) ist hier
bewusst NICHT enthalten — die Datei wurde am 2026-07-27 absichtlich fuer
die Registry-Discovery angelegt; ob sie weg soll, entscheidet der Nutzer.

Verifiziert: npm run build und 175/175 Vitest-Tests gruen.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017LJfKmX9pPmq32ApLWervk
@lukisch

lukisch commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

Teilweise übernommen — dieser PR bündelt zwei Anliegen, die unterschiedlich zu bewerten sind.

Übernommen (a883264): Node.js-Untergrenze 18 → 20 in package.json (engines), Lockfile, beiden READMEs und llms.txt. Faktisch belegt: die CI-Matrix testet 20/22/24, @hono/node-server 2.x verlangt 20. Der PR ließ sich nicht direkt mergen, weil er von 1.3.18 abzweigt und das Lockfile vom aktuellen Stand 1.3.22 zurückgedreht hätte.

Offen — bitte entscheiden: Das Entfernen von smithery.yaml. Die Datei wurde am 2026-07-27 mit ab41d9e absichtlich für die Registry-Discovery angelegt ("triple-registry discovery"); dieser PR will sie zwei Tage später als unverifiziert wieder entfernen. Zwei Automationen widersprechen sich hier über die Distributionsfläche — das ist eine Sichtbarkeitsentscheidung, keine technische, deshalb habe ich sie nicht im Alleingang getroffen.

Der PR bleibt für diesen Rest offen.

lukisch pushed a commit that referenced this pull request Aug 6, 2026
Lukas Geiger @claude
chore: unwirksames Smithery-Manifest entfernen
d1630a8 
Empirisch geprueft: smithery.ai/server/@ellmos-ai/ellmos-codecommander-mcp
liefert 404 — identisch zum Ergebnis fuer einen frei erfundenen Namen. Das
Manifest liegt seit 2026-07-27 im Repo und hat keine Listung bewirkt; eine
smithery.yaml registriert fuer sich genommen nichts, dafuer braucht es eine
Publikation ueber die Smithery-CLI. Zudem entspricht das verwendete Format
(startCommand.config mit command/args) nicht dem dokumentierten
stdio-Schema, das eine commandFunction erwartet.

Damit ist der zweite Teil von PR #6 erledigt. Faellt die Entscheidung
spaeter fuer Smithery, gehoert dazu eine echte Publikation plus Badge —
nicht nur eine Datei im Repo.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017LJfKmX9pPmq32ApLWervk
@lukisch

lukisch commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

Both parts of this PR are now on master, applied directly rather than merged — the branch was cut from 1.3.18 while master had already moved to 1.3.22, so merging it would have reverted the lockfile.

  • Node floor 18 → 20 — a883264. The CI matrix has only ever tested 20/22/24 and @hono/node-server 2.x requires 20, so the advertised >=18 was wrong. Raised across package.json (engines), the lockfile, both READMEs and llms.txt.
  • Smithery manifest removed — d1630a8. smithery.yaml and its files entry are gone. The listing it advertised does not exist: smithery.ai/server/@ellmos-ai/ellmos-codecommander-mcp returns HTTP 404. The same check and the same removal were applied to ellmos-clatcher-mcp (see Reconcile secure 1.0.12 metadata into main ellmos-clatcher-mcp#1).

Found and fixed alongside: glama.json had drifted to 1.3.18 while everything else moved to 1.3.22 (cb239dd). The metadata test now covers glama.json as well, verified by reverting the version and watching the new assertion fail.

176/176 Vitest tests green. Closing as applied.

@lukisch lukisch closed this Aug 6, 2026
@lukisch lukisch mentioned this pull request Aug 6, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@lukisch