Repository navigation
Add MAINTAINING.md - #33
Conversation
What lives only here: the entry points, and which surface the unit tests actually cover against which is demo-verified. The facts true of more than one Scolta repo stay in scolta-core's shared MAINTAINING.md, RELEASING.md, REGRESSION.md and ASSETS.md; this file links there rather than repeating them.
SummaryAdds a Type: docs Walkthrough
Related Issues & PRsNo related issues identified. AI Review Summary — generated by ai-pr-review |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
AI Review: Approved
No findings above the confidence threshold. The changes look good.
Token usage by agent
| Agent | Model | Input | Output | Cache Write | Cache Read | Total | Est. Cost |
|---|---|---|---|---|---|---|---|
| silent-failure-hunter | Sonnet 5 | 5394 | 146 / 16384 | 1095 | 0 | 6635 | $0.0224 |
| code-reviewer | Sonnet 5 | 6177 | 180 / 16384 | 1095 | 0 | 7452 | $0.0253 |
| Total | 11571 | 326 | 2190 | 0 | 14087 | $0.0477 |
AI Review — generated by ai-pr-review
There was a problem hiding this comment.
AI Review: Approved
No findings above the confidence threshold. The changes look good.
Token usage by agent
| Agent | Model | Input | Output | Total | Est. Cost |
|---|---|---|---|---|---|
| code-reviewer | Sonnet 5 | 6335 | 60 / 16384 | 6395 | $0.0199 |
| Total | 6335 | 60 | 6395 | $0.0199 |
AI Review — generated by ai-pr-review
This package's own maintenance facts: the entry points, and which surface the unit tests actually cover against which is demo-verified.
One fact, one place: anything true of more than one Scolta repo lives in scolta-core/MAINTAINING.md (tag1consulting/scolta-core#63), and this file links there instead of repeating it.
Where the repo contradicted the notes this was drafted from
src/runtime/are not exercised by any test here (runtime-memoization.test.tscoverssrc/runtime/util.tsonly), but this repo'sCLAUDE.mdrecords that the module and its Nitro routes are verified end to end by the demo. Stated as a bare coverage gap it reads as untested; stated accurately it tells a maintainer that a change to those four files is demo-verified rather than CI-verified, which is the thing worth knowing.tscand tests. There is onetestjob across Node 20 and 22 running build, test, typecheck, lint,check:publishandcheck:pack.CLAUDE.md, because it costs an afternoon: never run plainnpm installwhile the localscoltasymlink exists, since npm's tree repair drops esbuild's platform binary.scolta@^X.Y.0; the doc does not pin a number, per the shared guide's rule about floors in prose.This repo has no pull-request template, so no template line was added.