Align public repo with Runtime, Protocol, and Proof surfaces#41
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✅ Files skipped from review due to trivial changes (4)
📝 WalkthroughWalkthroughThree documentation files—README.md, docs/llms-full.txt, and docs/public-upgrade.md—are updated to adopt consistent "Runtime, Protocol, and Proof" terminology. Product positioning, surfaces, operator guidance, and contributor resources are revised throughout to frame ZERO's open offering as inspectable, self-custodial runtime and proof artifacts without hosted dependencies. ChangesRuntime, Protocol, and Proof terminology alignment
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
README.md (1)
180-181: ⚡ Quick winAlign the Mermaid surface label with the new terminology.
The updated flow points to Proof/Public verification, but Line 178 still uses
ZERO Terminal, which conflicts with the Runtime/Protocol/Proof framing in this PR. Consider renaming that node toZERO CLI(orZERO Control) for consistency.🤖 Prompt for 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. In `@README.md` around lines 180 - 181, The Mermaid diagram currently labels the node referenced by symbol F as "ZERO Terminal" which conflicts with the PR's Runtime/Protocol/Proof terminology; update F's label to "ZERO CLI" (or "ZERO Control") and ensure any edges or references (e.g., F --> G and subsequent G --> H) remain unchanged so the flow from F to G["ZERO Proof"] to H["Public verification"] reads consistently.docs/llms-full.txt (1)
244-245: ⚡ Quick winAlign the mermaid node label with the new surface naming.
The flow now routes through Proof/Public verification, but the upstream node still says
ZERO Terminal, which conflicts with the updatedZERO CLIsurface naming in the same section. Renaming that node keeps the taxonomy consistent.🤖 Prompt for 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. In `@docs/llms-full.txt` around lines 244 - 245, The mermaid flow uses node labels F -> G -> H but the upstream node label still references "ZERO Terminal"; update the node label for F (the node currently named "ZERO Terminal") to match the new surface naming "ZERO CLI" (or otherwise rename it to "ZERO CLI") so the sequence F --> G["ZERO Proof"] --> H["Public verification"] is consistent with the updated surface taxonomy.
🤖 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 679-680: Remove the duplicate documentation links in the README
docs list by deleting the repeated entries "- [ZERO
Proof](docs/proof/README.md)" and "- [MCP Server](docs/mcp.md)" (they already
appear earlier), leaving only the original links to avoid redundancy in
navigation.
---
Nitpick comments:
In `@docs/llms-full.txt`:
- Around line 244-245: The mermaid flow uses node labels F -> G -> H but the
upstream node label still references "ZERO Terminal"; update the node label for
F (the node currently named "ZERO Terminal") to match the new surface naming
"ZERO CLI" (or otherwise rename it to "ZERO CLI") so the sequence F --> G["ZERO
Proof"] --> H["Public verification"] is consistent with the updated surface
taxonomy.
In `@README.md`:
- Around line 180-181: The Mermaid diagram currently labels the node referenced
by symbol F as "ZERO Terminal" which conflicts with the PR's
Runtime/Protocol/Proof terminology; update F's label to "ZERO CLI" (or "ZERO
Control") and ensure any edges or references (e.g., F --> G and subsequent G -->
H) remain unchanged so the flow from F to G["ZERO Proof"] to H["Public
verification"] reads consistently.
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: 024e8ac8-26f1-482a-928f-f1ae2d46f62b
📒 Files selected for processing (3)
README.mddocs/llms-full.txtdocs/public-upgrade.md
Summary
Safety
Validation
Summary by CodeRabbit