Align public repo with Runtime, Protocol, and Proof boundary#44
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis PR updates public-facing copy and generated docs to present ZERO as an open Runtime, Protocol, and Proof substrate and clarifies that hosted product surfaces (Studio, Control, Registry, etc.) are external to the repository; it also updates LLM context generation, readiness checks, and some package metadata. ChangesZERO Public Boundary & Product Positioning
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
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: 2
🧹 Nitpick comments (1)
README.md (1)
88-89: ⚡ Quick winClarify "legacy CLI language" reference.
The text describes
/cockpit-modeas "legacy CLI language" but this is the current documented command with no alternative mentioned. This phrasing may confuse users into thinking the feature is deprecated when it appears to be the primary interface for accessing the Runtime control packet.Consider either:
- Removing the "legacy" qualifier if the command naming is current
- Explaining what makes it "legacy" and what the preferred terminology is
- Clarifying that only the naming is legacy while the feature remains primary
🤖 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 88 - 89, The phrasing that calls "/cockpit-mode" "legacy CLI language" is ambiguous; update the README text around "Runtime control packet" and "/cockpit-mode" to either (a) remove the word "legacy" if the command is current, (b) explicitly state what makes the naming legacy and provide the preferred term, or (c) clarify that only the naming is legacy while the feature remains primary—also mention "paper/default mode" behavior (refuse live risk and show the reason) remains unchanged so readers understand the command status versus functionality.
🤖 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 `@docs/positioning.md`:
- Around line 57-74: Choose whether the repo defines four surfaces (ZERO
Runtime, ZERO Protocol, ZERO Proof, Developers with ZERO CLI treated as Runtime
tooling) or five distinct surfaces (adding ZERO CLI as its own surface and
possibly Evolution); then update the surface enumeration and wording
consistently in the docs by: adjusting the list items and headings for "ZERO
Runtime", "ZERO Protocol", "ZERO Proof", "ZERO CLI", and
"Developers"/"Evolution" wherever they appear; make the README table rows match
the chosen set and rephrase the sentence in the architecture doc that currently
says "four first-class open surfaces" so it matches the chosen count; ensure any
references to "CLI as tooling" or "Evolution as Runtime subsystem" are aligned
with the final decision.
---
Nitpick comments:
In `@README.md`:
- Around line 88-89: The phrasing that calls "/cockpit-mode" "legacy CLI
language" is ambiguous; update the README text around "Runtime control packet"
and "/cockpit-mode" to either (a) remove the word "legacy" if the command is
current, (b) explicitly state what makes the naming legacy and provide the
preferred term, or (c) clarify that only the naming is legacy while the feature
remains primary—also mention "paper/default mode" behavior (refuse live risk and
show the reason) remains unchanged so readers understand the command status
versus functionality.
🪄 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: 3fde8cc2-4ab1-4537-a5c1-d4b4cd734911
📒 Files selected for processing (13)
README.mdcli/Cargo.tomldocs/architecture.mddocs/autonomous-os-plan.mddocs/llms-full.txtdocs/llms.txtdocs/open-core-boundary.mddocs/positioning.mddocs/private-engine-capability-gap-audit.mddocs/public-upgrade.mdengine/pyproject.tomlllms.txtscripts/public_readiness_gate.sh
3fb620b to
2c13046
Compare
Summary
Align the public
zero-intel/zerorepository with the v95 Runtime / Protocol / Proof boundary.This keeps the public repo focused on the open substrate while leaving hosted product surfaces outside this repository:
Safety
Verification
scripts/generate_llms_full.pyjust stale-artifact-checkjust docs-checkjust public-readinessjust hardening-gatejust public-proofgit diff --checkcd cli && cargo metadata --no-deps --format-version 1 >/tmp/zero-public-cargo-metadata.json && python3 -m json.tool /tmp/zero-public-cargo-metadata.json >/dev/nullengine/pyproject.tomlandcli/Cargo.tomlSummary by CodeRabbit
Documentation
Chores