Make onboarding readable and document the real trust boundary - #16
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThe README is substantially condensed and reorganized around onboarding, CLI usage, delivery, installation, adapters, safety, and references. A new ChangesDocumentation and Trust Boundaries
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai review |
✅ Action performedReview finished.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Around line 84-95: Update the Mermaid delivery flow around “Project tests and
checks” and “Independent review” so independent review is conditional on project
policy rather than mandatory for every project. Preserve the existing
valid-finding loop and route projects that do not require review directly to the
merge-ready result.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 7a8a8132-dda9-4a6e-b809-fa49b2957b73
📒 Files selected for processing (2)
README.mddocs/TRUST.md
|
@coderabbitai review |
✅ Action performedReview finished.
|
6b31fed to
baaab9e
Compare
|
@coderabbitai review |
✅ Action performedReview finished.
|
What changed
docs/SOURCES_AND_TRADEOFFS.mddocs/TRUST.mdwith a side-by-side map of CLI-enforced controls versus agent instructionsWhy
The previous README was a strong engineering audit but a difficult product entrance. The package names non-coders as a primary audience, yet most of the front page required senior-engineer vocabulary. The old structure also mixed provenance, onboarding, architecture, and trust into one long page.
User impact
A new user reaches the working prompt and commands immediately, understands what the agent will ask, and can see the honest safety boundary without reading an audit ledger. Engineers retain direct links to the full architecture, trust model, source history, review loop, adapters, and release process.
The README falls from 438 lines and roughly 2,108 words to 198 lines and roughly 1,136 total words. Prose-only estimation is about 693 words, Flesch reading ease 53.3, and grade level 9.0.
Validation
npm run release:checkdocs/TRUST.mdnpx --yes markdownlint-cli2@0.20.0 '**/*.md': 49 files, 0 errorsgit diff --checkSummary by CodeRabbit