Add thin toolkit-routing agent adapters - #3
Conversation
Add scripts/build-agent-adapters.js and check-agent-adapters.js that generate thin Codex/Vibe/GitHub Copilot/Cursor adapters routing to the architecture-knowledge-toolkit (no local skills). Wire adapters/check-adapters/test into build.sh and the CI validate job, turn .github/copilot-instructions.md into a thin entry point, and document the reference-don't-copy principle in AGENTS.md and general-semantic-contracts.md. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Do not assume the architecture-knowledge-toolkit is a direct sibling. The local lookup order now searches ../architecture-knowledge-toolkit and then each parent directory up to the filesystem root, so projects outside the toolkit's parent folder still resolve it before the public fallback. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
dieterbaier
left a comment
dieterbaier
left a comment
There was a problem hiding this comment.
Die Änderungen sind technisch schlüssig, die CI läuft erfolgreich und der Adapter-Check ist sauber eingebunden. Ich habe nur eine kleine Zukunfts-Inkonsistenz im Generator kommentiert; für den aktuellen Zustand ohne lokale Skills ist das nicht blockierend.
| const body = `# ${target.title} | ||
|
|
||
| This is a thin ${target.agent}-specific wrapper for the ${project} | ||
| repository. ${project} keeps no local agent skills of its own; architecture and |
There was a problem hiding this comment.
Kleine Zukunfts-Inkonsistenz: Sobald skills.length > 0 ist, hängt der Generator zwar einen Abschnitt „Local Skills“ an, behauptet direkt davor aber weiterhin, dass docs-toolbox keine lokalen Skills besitzt. Für den heutigen Zustand ist das korrekt; falls die angekündigte Erweiterbarkeit wirklich gewollt ist, sollte dieser Satz ebenfalls abhängig von skills.length gerendert werden.
Review nit: the generator appended a Local Skills section once skills existed but still claimed docs-toolbox has no local skills. Make the intro and skill-selection step depend on skills.length so the wording stays correct if local skills are ever added. No change to current output (no local skills today). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Nit adressiert ( |
dieterbaier
left a comment
dieterbaier
left a comment
There was a problem hiding this comment.
Nachprüfung nach 96051d1: Der kommentierte Widerspruch ist sauber behoben. Intro und Skill-Auswahlschritt wechseln jetzt konsistent mit skills.length; die aktuelle Ausgabe ohne lokale Skills bleibt unverändert. Von meiner Seite keine weiteren Findings.
1 participant
Summary
Bring docs-toolbox in line with the architecture-knowledge-toolkit's generated agent-adapter approach. docs-toolbox keeps no local skills, so the adapters route agents to the toolkit.
scripts/build-agent-adapters.jsandscripts/check-agent-adapters.jsgenerating thin Codex/Vibe/GitHub Copilot/Cursor adapters underadapters/that route to the toolkit.adapters/check-adapters/testintobuild.shand add adapter check + generator test to the CI validate job..github/copilot-instructions.mdinto a thin entry point pointing toadapters/github-copilot/.AGENTS.mdandgeneral-semantic-contracts.md, and record the toolkit lookup order with an upward directory search (no direct-sibling assumption). Note Node.js/npm in the image tool list.test/build-agent-adapters.test.mjs(4 scenarios).Verification
node scripts/check-agent-adapters.js: adapters current.node --test test/build-agent-adapters.test.mjs: 4/4 pass.sh -n build.shand localadapters/check-adapters/testcommands verified.🤖 Generated with Claude Code