Prepare v0.11.0-alpha.1 release - #110
Conversation
|
Warning Review limit reached
Next review available in: 40 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughPrepares the v0.11.0-alpha.1 release: bumps Changesv0.11.0-alpha.1 Release Preparation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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
🤖 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/topics/release-process/policy.toml`:
- Around line 454-460: The structured v0.11 non-goal list is missing several
exclusions that are already described by the release notes contract. Update the
non_goals table in policy.toml to include the missing markers for
participant-policy logic, Echo/git-warp implementation claims, additional target
profiles, and extra source-to-target fixtures, then make the same marker
additions in the release-process test-plan and xtask coverage paths so all three
locations stay in sync.
In `@ROADMAP.md`:
- Around line 476-483: Add the missing non-goal to the structured source of
truth by updating policy.toml’s non_goals list to match the ROADMAP entry for
git-warp commit creation/CRDT reducer verification, using the existing non_goals
entries as the pattern; if that scope is not meant to be tracked, remove the
ROADMAP-only item instead so ROADMAP and policy.toml stay aligned. Reference the
non_goals list in policy.toml and the ROADMAP non-goals section to keep the
single source of truth consistent.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: b3746cc3-d345-473e-9fca-afa7549b925e
⛔ Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
📒 Files selected for processing (11)
CHANGELOG.mdREADME.mdROADMAP.mdcrates/edict-cli/Cargo.tomlcrates/edict-syntax/Cargo.tomldocs/README.mddocs/releases/v0.11.0-alpha.1.mddocs/topics/release-process/README.mddocs/topics/release-process/policy.tomldocs/topics/release-process/test-plan.mdxtask/src/main.rs
📜 Review details
🧰 Additional context used
📓 Path-based instructions (3)
**
⚙️ CodeRabbit configuration file
**: # AGENTSGit Rules
NEVER amend git commits. Make a new commit instead.
NEVER use
git rebaseunless the user explicitly approves a rare exception.
Use regular merge commits.NEVER force any git operation. If a force operation appears necessary, stop and
explain what happened and what options remain.NEVER create draft pull requests.
NEVER use a
codexprefix in branch names, PR titles, or commit messages.Pull request bodies for issue work MUST include GitHub auto-close text such as
Closes#123`` for every issue the PR is intended to close.Think
Think is durable memory for cross-session coordination.
- Use
codex-think --remember --jsonwhen starting a new session, changing into
this repository, or regaining context after a context shift.- Use
codex-think "..." --jsonwhen a cycle closes or a significant event
should survive across turns.- Treat Think as memory, not repo truth. Anchor strong claims back to files,
commits, commands, issues, or pull requests.- Claude memories are read-only. Use
claude-think --remember --jsononly for
additional context.Topic Shelves
docs/topics/contains the living contract graph for landed behavior. Topic
shelves are not proposals, retrospectives, or design archaeology.Each shelf may contain:
README.md: what is true in HEAD.test-plan.md: how those truths are verified, including requirements, cases,
fixtures, oracles, implemented evidence, planned cases, and known gaps.architecture.md: optional structure or dataflow notes when the machinery
earns a separate page.rationale.md: optional still-relevant tradeoffs and rejected approaches.When To Update Topic Shelves
For every nontrivial behavior, contract, workflow, release, schema, validation,
or public-surface change:
- Identify the owning topic shelf before editing code.
- If no shelf owns durable behavior, create one.
- Update
test-plan.mdbefore or alongside tests with requirement IDs, case
IDs,...
Files:
docs/README.mdcrates/edict-syntax/Cargo.tomlcrates/edict-cli/Cargo.tomlCHANGELOG.mddocs/releases/v0.11.0-alpha.1.mddocs/topics/release-process/policy.tomlROADMAP.mddocs/topics/release-process/test-plan.mdxtask/src/main.rsdocs/topics/release-process/README.mdREADME.md
docs/topics/**/test-plan.md
📄 CodeRabbit inference engine (AGENTS.md)
docs/topics/**/test-plan.md: Update a topic shelftest-plan.mdbefore or alongside tests with requirement IDs, case IDs, fixtures, oracles, implemented evidence, planned cases, and known gaps.
Mark planned cases as implemented only when executable evidence exists.
Do not usepolicyrows in topic-shelf test plans to avoid writing behavior tests for software behavior.
Negative tests in topic-shelf test plans should assert stable error kinds or structured artifacts, not merelyis_err()or diagnostic text.
For nontrivial behavior, contract, workflow, release, schema, validation, or public-surface changes, update the owning topictest-plan.mdwith planned requirement and case rows before or alongside the first test.
Mark planned rows as implemented only after executable evidence exists.
Files:
docs/topics/release-process/test-plan.md
docs/topics/**/README.md
📄 CodeRabbit inference engine (AGENTS.md)
docs/topics/**/README.md: Topic shelfREADME.mdfiles must describe only what is true in HEAD.
Update the topic shelfREADME.mdonly after the behavior exists in the branch; do not describe intended future behavior there.
Do not churn topic-shelf README files for purely mechanical edits that do not change a contract.
Files:
docs/topics/release-process/README.md
🪛 LanguageTool
ROADMAP.md
[style] ~480-~480: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ecific admission execution or policy. - No Echo verifier completeness. - No git-wa...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~481-~481: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...icy. - No Echo verifier completeness. - No git-warp commit creation or CRDT reduce...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~482-~482: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...reation or CRDT reducer verification. - No general target plugin dispatch. - No ca...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~483-~483: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ... - No general target plugin dispatch. - No canonical ContractBundleManifest byte...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
🔇 Additional comments (15)
crates/edict-cli/Cargo.toml (1)
3-3: LGTM!crates/edict-syntax/Cargo.toml (1)
3-3: LGTM!docs/releases/v0.11.0-alpha.1.md (4)
1-136: Comprehensive release notes; one structural concern on verification timing.The document is thorough and well-structured. The verification section (Lines 101-112) mixes commands that can run now (
cargo xtask contract-check,cargo xtask verify) with test selectors that depend on later layers. Ensure the release notes are re-verified after all stacked changes land, or add a note that verification steps are cumulative across the release-prep stack.No other blocking issues in this new file.
41-45: No change needed here.
114-116: No change needed.docs/topics/contains 21 topic shelves, and the release notes already carry the 21/21 audit claim.
105-106: No change needed for these release-test selectors.release_policy_tracks_v0_11_boundaryalready exists inxtask/src/main.rs, andrelease_is an intentional broad filter for the release test suite.CHANGELOG.md (2)
9-14: LGTM!
15-66: LGTM!ROADMAP.md (1)
692-693: LGTM!docs/README.md (1)
23-26: LGTM!docs/topics/release-process/README.md (1)
99-108: LGTM!README.md (4)
17-23: LGTM!
605-608: LGTM!
640-640: LGTM!
656-660: LGTM!
Activity Summary
RED: GREEN: |
Summary
Prepare
v0.11.0-alpha.1as the contract-bundle assembly and canonical Target IR artifact freeze alpha.This release packages:
0.11.0-alpha.1foredict-syntaxandedict-cli;docs/releases/v0.11.0-alpha.1.mdrelease-prep notes;Closes #109
Release Thesis
v0.11.0-alpha.1freezes the v0.11 contract-bundle assembly and canonical Target IR artifact boundaries. It covers participant-neutral bundle assembly from real Core modules, semantic/release bundle digest preimage and golden checks, canonical Target IR artifact bytes/digests for Echo and git-warp review artifacts, and bundle assembly from a realTargetIrArtifactwith computedtargetIrDigest.Explicit Non-Goals
This release prep does not claim runtime execution, admission execution, verifier completeness, general target plugin dispatch, canonical
ContractBundleManifestbytes, or crates.io publication.Previous-Tag Diff Reconciliation
Checked with:
Reconciled scope includes post-v0.10 release-process evidence, CLI check-facade/help/input-schema fixes, contract-bundle assembly and bundle goldens, canonical Target IR artifact bytes/digests and goldens, bundle integration from real Target IR, and release-policy/version/signpost updates.
Topic-Shelf Audit
Audit evidence is recorded on #109: #109 (comment)
RED/GREEN
RED:
cargo test -p xtask release_policy_tracks_v0_11_boundaryFailed on the missing
[release_notes.v0_11_0_alpha_1]section.GREEN:
Release Automation Notes
After this PR merges and
mainCI passes,auto-release-tag.ymlshould derive:The Release workflow should publish a GitHub prerelease from
docs/releases/v0.11.0-alpha.1.md, close milestone #12 only after confirming zero open issues, and publish no crates or package assets.