Skip to content

Commit 192514e

Browse files
rafaelscostaclaude
andcommitted
docs: address CodeRabbit nitpick on release-procedure.md lockstep wording
PR #751 CodeRabbit review flagged that the "Bump all five version sites in lockstep" wording was ambiguous — the table below explicitly treats `packages/installer/package.json` as conditional ("patch bump if installer changed; otherwise leave"). Operators reading only line 36 could over-bump the installer. Reworded to distinguish: - Root + internal manifest + compat: MUST stay in lockstep at same version - packages/installer: bumped ONLY when installer source changed (not every release) Also explicit on the two failure-mode detection layers: - Smoke tests (catch silent publish mismatches) - validate-aiox-core-namespace.js wired into validate:publish (rule 4 enforces lockstep) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 6c0ad2b commit 192514e

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

docs/guides/release-procedure.md

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,12 @@ Block the release if any of these are red.
3333

3434
## Version bump (T-5 minutes)
3535

36-
Bump **all five** version sites in lockstep. Mismatches cause silent publish
37-
failures that only show up in the smoke tests OR are caught by
38-
`validate-aiox-core-namespace.js` (which is wired into `validate:publish`).
36+
Review **all five** version sites during release prep. The root surface, the
37+
internal manifest, and the compat wrapper MUST stay in lockstep at the same
38+
version. `packages/installer/package.json` is bumped **only** when installer
39+
source changed — not on every release. Mismatches cause silent publish failures
40+
that show up in the smoke tests OR are caught by
41+
`validate-aiox-core-namespace.js` (wired into `validate:publish`).
3942

4043
| File | What to bump |
4144
|---|---|

0 commit comments

Comments
 (0)