You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
fix(skill-gen): eliminate manual maps to prevent undocumented commands (#670)
## Summary
- Replaces the three manually-maintained maps (`ROUTE_TO_REFERENCE`,
`REFERENCE_TITLES`, `REFERENCE_DESCRIPTIONS`) in `generate-skill.ts`
with 1:1 route-to-file mapping
- Titles and descriptions are now derived from route metadata (`brief`),
matching the strategy used by `generate-command-docs.ts`
- Adding a new route to `app.ts` now automatically produces a
correctly-titled reference file with zero manual steps
## Motivation
PR #662 identified that `release` and `sourcemap` routes had degraded
skill documentation despite the fully automated generation pipeline. The
root cause was that `generate-skill.ts` required manual map updates that
`generate-command-docs.ts` did not — and the silent fallback (`??
route.name`) masked the problem by producing subtly wrong output that
CI's staleness check couldn't distinguish from correct output.
Rather than adding validation on top of the manual maps, this PR
eliminates them entirely so the two generation pipelines share the same
self-healing strategy: one file per visible route, metadata derived from
the route tree.
## Trade-off
Reference files go from 14 (grouped) to 18 (1:1). The 3 former groupings
were: `trace`+`span` → `traces.md`, `team`+`repo` → `teams.md`,
`cli`+`init`+`schema` → `setup.md`. Each route now gets its own file.
Closes#662
0 commit comments