Skip to content

docs(ci): the bump-level rule — an additive widening of a published surface is at least minor (ruling C, #15294) - #15380

Merged
baozhoutao merged 1 commit into
mainfrom
claude/issue-15294-additive-widening-bump-rule
Sep 4, 2026
Merged

docs(ci): the bump-level rule — an additive widening of a published surface is at least minor (ruling C, #15294)#15380
baozhoutao merged 1 commit into
mainfrom
claude/issue-15294-additive-widening-bump-rule

Conversation

@baozhoutao

Copy link
Copy Markdown
Contributor

Fixes #15294

Records maintainer ruling C as prose, in the two places the issue named. No new gate, no behaviour change, nothing retro-fixed.

Provenance

Ruling C, recorded on #15294 as comment 5540571415 by the Director seat: "Provenance: maintainer, live PM chat, 2026-09-04, decision batch #35, verbatim 「同意」 on the presented recommendation 1A · 2(Q1 A · Q2 A · Q3 B→A) (this card is item 3, C)." Ruling authority: maintainer 2026-09-04 (batch #35).

Where the paragraph now appears

  1. .github/workflows/pr-automation.yml — inside the Require a changeset (or the skip-changeset label) step of the Check Changeset job, in the heredoc the step prints to the job log when a PR adds no changeset. It sits under route 1 ("It releases something -> run 'pnpm changeset' and name the packages it releases"), because route 1 is exactly the moment an author has to pick a level. It opens WHICH LEVEL: and carries the ruled sentence, then its provenance, then the two things the ruling explicitly rules out (the 64 historical patch precedents are pre-rule; this is prose, not a gate).
  2. scripts/check-changeset-no-major.mjs — a cross-reference in the file header, placed immediately after the paragraph that explains why the bump level is not the carrier of breaking-ness during the launch window, since that is the sentence the rule's last clause depends on. It states the floor in one line and points at the workflow prose and at finding(changeset): two independent contract reviews read the repo's own history to opposite bumps for "add an exported symbol to a published index" #15294 for the full text.

The ruled sentence is reproduced word for word; only the markdown bold markers around its first sentence were dropped, because the destination is a plain-text job log rather than rendered markdown.

Verification

Every run below was made on the branch head fad832b7e (git rev-parse --short HEAD), the final commit of this PR.

  • Behaviour unchanged, proven rather than asserted. The verdict step's run script was extracted with a YAML parser from HEAD~1 and from HEAD: with the heredoc body excluded the two shell scripts are byte-identical, and executing both under ADDED=0, ADDED=2 and ADDED=n/a gives the same exit codes on both (1, 0, 0).
  • Comment-only on the script. git diff -U0 -- scripts/check-changeset-no-major.mjs filtered to lines that are not block-comment continuations is empty (0 lines), so the floor, the roster and the --self-test fixtures are untouched; check-changeset-no-major.mjs --self-test is green.
  • The derived gate family for these two paths (42 commands from scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack) is green, plus the always-runs tail and the roster families whose lists live under the two touched directories. Whole-repo pnpm lint (eslint . --no-inline-config) is green with zero findings, run in the foreground under the shared verify lock (os-verify-lock: VERDICT command-exit 0). Full accounting in the report comment on finding(changeset): two independent contract reviews read the repo's own history to opposite bumps for "add an exported symbol to a published index" #15294.

skip-changeset: this PR releases nothing — a workflow comment/prose change plus a comment line in a repo script.

🤖 Generated with Claude Code

https://claude.ai/code/session_012zGPuVVX3deAx9LdjK8jCk


Generated by Claude Code

…urface is at least minor

Record the maintainer's ruling (2026-09-04, decision batch #35) where authors
already read the changeset rules: the `Check Changeset` step's route-1 prose in
`.github/workflows/pr-automation.yml`, plus a one-line cross-reference from the
header of `scripts/check-changeset-no-major.mjs`, which already carries the
launch-window reasoning the rule's last sentence depends on.

Prose only: no new gate, no behaviour change, nothing retro-fixed.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012zGPuVVX3deAx9LdjK8jCk
@claude claude Bot added the skip-changeset PR has no user-facing published change; bypasses the changeset gate label Sep 4, 2026
@baozhoutao
baozhoutao marked this pull request as ready for review September 4, 2026 13:36
@baozhoutao
baozhoutao enabled auto-merge September 4, 2026 13:36
@baozhoutao
baozhoutao added this pull request to the merge queue Sep 4, 2026
Merged via the queue into main with commit b337a13 Sep 4, 2026
36 checks passed
@baozhoutao
baozhoutao deleted the claude/issue-15294-additive-widening-bump-rule branch September 4, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd size/s skip-changeset PR has no user-facing published change; bypasses the changeset gate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

finding(changeset): two independent contract reviews read the repo's own history to opposite bumps for "add an exported symbol to a published index"

2 participants