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
scripts/graph/ is a SCIP code-graph indexer built for the maintainer's own agent tooling. It is not part of @ipv9/tokentracker-cli and ships to nobody.
It costs the product repo three things:
12 of the repo's 118 test files — roughly a tenth of the suite, testing something users never run. That distorts every coverage judgement: the repo looks well tested while four product parsers (hermes, codebuddy, craft, kimi) have no test at all (test: parser conformance ratchet — cover the code that actually carries risk #104).
Time in every ci:local run, on every PR, for all contributors.
Review surface — anyone auditing this repo has to work out that it is unrelated before they can ignore it.
Proposal
Move it to its own repository. If it must stay for now, at minimum split it out of the default ci:local chain into a separate script so the product gate reflects the product.
Before removing
Check nothing in the product path imports it — grep -rn "scripts/graph" --include='*.js' --include='*.cjs' --include='*.json' . — and confirm no workflow or skill outside this repo depends on the path. If something does, note it in the destination repo's README rather than leaving the dependency implicit.
Priority
Lowest of the set. It is hygiene, not a defect, and worth doing on a quiet day rather than ahead of #100.
scripts/graph/is a SCIP code-graph indexer built for the maintainer's own agent tooling. It is not part of@ipv9/tokentracker-cliand ships to nobody.It costs the product repo three things:
hermes,codebuddy,craft,kimi) have no test at all (test: parser conformance ratchet — cover the code that actually carries risk #104).ci:localrun, on every PR, for all contributors.Proposal
Move it to its own repository. If it must stay for now, at minimum split it out of the default
ci:localchain into a separate script so the product gate reflects the product.Before removing
Check nothing in the product path imports it —
grep -rn "scripts/graph" --include='*.js' --include='*.cjs' --include='*.json' .— and confirm no workflow or skill outside this repo depends on the path. If something does, note it in the destination repo's README rather than leaving the dependency implicit.Priority
Lowest of the set. It is hygiene, not a defect, and worth doing on a quiet day rather than ahead of #100.