docs: correct stale test count, pnpm pin, and repo layout in the public docs - #45
Merged
Merged
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes seven factual inaccuracies in
README.mdandRUNBOOK.mdfound by a documentation-accuracy audit that checked every claim against the code.pnpm test→ 407 (216 frontend + 191 backend)package.jsonpins 10.33.3 — the RUNBOOK validation checkpoint (pnpm -vreports10.33.0) failed on a fresh clonetests/smoke/smoke/dir exists, andtests/is gitignored — it was never part of the published treeschema.prisma"(Location, Project, TariffConfig, Quota)"Quotamodel; the auth models (User/Session/Account) were missing/chatbackend/src/routes/chat.tsand the wholeservices/chat/subtree existruntime: nodeis inrender.yamlDockerfileotherwise. The trap was documented only in arender.yamlcomment, not in the runbookLast verified: 03/05/26 (against main @ <replace-with-current-sha>)The layout tree also now surfaces
Dockerfileanddocker-compose.yml, which shipped in #18 and were invisible in the tree.Verified alongside: all 15 RUNBOOK TOC anchors resolve, every
pnpmcommand in both docs exists inpackage.json, RUNBOOK §4 + Appendix A match the Zod schema inbackend/src/config/env.tsexactly (no extra, no missing), all repo-relative asset links resolve, and both external links (milestone 1, thev1.1-walkthroughrelease asset) are live.https://claude.ai/code/session_01URDpaxhXyPLWnUQtvowNyM