Strip release-process narration from per-repo docs - #15
Conversation
The release process is owned by rainix's rainix-tag-release reusable and restating it per-repo drifts: rainlanguage/rainix#338 made tag-release push-free, so the 'commits the snapshot back to main' narration here was already stale. Docs-only; repo-local invariants kept, mechanics point to rainix. Ruling: rainlanguage/claude-audit-skills#91 Philosophy: rainlanguage/rainix#298 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe workflow comment and repository documentation now describe manual ChangesRelease documentation alignment
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This docs-only change removes stale release-process narration without changing product or deployment behavior; no actionable merge-blocking risk remains. Possibly related issues
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Most of the file was discoverable content the cap gate targets (build/test command listings, CI job breakdown, file-by-file architecture, dep list). Kept only the non-discoverable invariants: interfaces-live-in-rain.factory, pragma pins, deterministic-address hazards, frozen/append-only snapshots, the last-released [package].version, and the manual-deploy-before-tag shape. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Docs-only. Strip release-process narration that this per-repo doc restates but does not own.
Human ruling (2026-08-20): "you really shouldn't put release processes in the per repo docs because it's super fragile." The release process is owned by rainix's
rainix-tag-release.yamlreusable; per-repo restatements drift the moment the reusable changes — which just happened: rainlanguage/rainix#338 / rainlanguage/rainix#339 made tag-release push-free, so this repo's "commits the frozen snapshot back to main" was already factually wrong.Removed
.github/workflows/package-release.yaml— collapsed the header comment (rainix-tag-release verifies prod at regenerated pins / publishes to Soldeer / commits the frozen snapshot back to main) to one line of deploy-repo identity.CLAUDE.md"Releases and versioning" — removed the step-by-steprainix-tag-releasemechanics (writes version into foundry.toml, regenerates snapshot, verifies chains, publishes to Soldeer, commits snapshot back to main — now stale) and the "never broadcasts, only attests" narration.README.md"Releases" — removed the samerainix-tag-releasemechanics sentence (regenerate / verify / publish / commit-back — now stale).Kept (repo-local invariants an agent would get wrong from the repo alone)
[package].versionis the last-released version not a next-version slot (a normal PR does not bump it); a manualsol-v*tag is the sole release trigger; the on-chain deploy is a manual dispatch run BEFORE tagging and tagging never broadcasts;src/generated/<tag>/snapshots are frozen/append-only.QA
Summary by CodeRabbit