Skip to content

ci: per-PR plugin version bumps force repeated base merges on every PR at the current merge rate - part 2 of 7 #10858

Closed

Description

@melodic-automation-lanes

Part 2 of 7 of #6006, split at intake (owner Q93). The done-when below is copied from #6006 as written.

Done when

  • 1. Tooling, with no plugin opted in
  • check-changelog-parity.sh --check-bump (lines 93-96): count a consumed fragment as the change. Today a release PR touches only CHANGELOG, the version field and .changes/, so it fails BUMP WITHOUT CHANGE. That would keep the pilot's ci-status red.
  • Reject a hand version bump or CHANGELOG heading on a fragment-mode plugin in any PR other than the release PR. --check-required currently exempts CHANGELOG and version-only plugin.json edits for every PR. In a scratch test, a non-release PR that bumped a listed plugin by hand and added no fragment passed.

Activity

  1. added
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Oct 29, 2026
  2. github-actions commented on Oct 29, 2026

    @github-actions
    Contributor

    Proposed work class: work-class: scoped
    Basis: automated intake triage; the lanes App applied this class.

    Size: Small
    Complexity: Medium
    Uncertainty: Known
    Flags: none
    Blocked by: none

    Understood: this is part 2 of 7 of #6006, split at intake. It asks for two changes to the changelog parity check, with no plugin opted in yet. First, --check-bump should count a consumed .changes/ fragment as the change, so a release PR no longer fails BUMP WITHOUT CHANGE. Second, a hand version bump or CHANGELOG heading on a fragment-mode plugin should be rejected in any PR other than the release PR. The done-when names a scratch test that currently passes when it should fail. Nothing is missing. The sibling parts (#10857, #10859 to #10863) cover different slices of #6006, so none is a duplicate.

    Automated triage, applied by the lanes App.

  3. added and removed
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Oct 29, 2026
  4. added
    status: readyTriaged, unblocked, and fully specified; eligible to pick up.
    admission-deferredAdmitted but waiting for WIP room; the hourly work-unblock sweep labels it agent-ready.
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    and removed
    admission-deferredAdmitted but waiting for WIP room; the hourly work-unblock sweep labels it agent-ready.
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    on Oct 29, 2026
  5. melodic-automation-lanes commented on Oct 29, 2026

    @melodic-automation-lanes
    ContributorAuthor

    The work-resolve-issue lane stopped partway through this issue and needs a person.

    Why, in the run's own words:

    already-done: both done-when items of #10858 are already on main at abb359b, so there is nothing to change. (1) scripts/check-changelog-parity.sh --check-bump counts a fragment the change set deletes under .changes/<name>/ as the change (consumed_fragment, lines 624-635 and 925), so a release PR no longer fails BUMP WITHOUT CHANGE. (2) On any PR other than the release PR, --check-bump fails a version move or an added CHANGELOG heading on a fragment-mode plugin (FRAGMENT-MODE RELEASE, lines 829-848). check-changelog-fragments.sh --check-required now exempts the CHANGELOG.md edit and the version-only plugin.json edit only on the release PR (lines 145-156). A person should check this and close the issue as already done.

    What the run says it did not do:

    • scripts/check-changelog-parity.test.sh and scripts/check-changelog-fragments.test.sh were not run: this lane can run only the named wrappers. I read the tests but did not execute them.
    • tidy, review, verify and the PR prep were skipped: the run made no change, so there is nothing to tidy or review.

    Run: https://github.com/melodic-software/claude-code-plugins/actions/runs/40844966342

    To retry, remove needs-human and add agent-ready again.

  6. kyle-sexton commented on Oct 29, 2026

    @kyle-sexton
    Member

    Already done on main, verified: scripts/check-changelog-parity.sh counts a consumed fragment as the change (:536, :633) and fails a fragment-mode release on a non-release PR (:804-840); check-changelog-fragments.sh exempts the CHANGELOG/version edit only on the release PR (:145-156). check-changelog-parity.test.sh PASS=127 FAIL=0, check-changelog-fragments.test.sh PASS=75 FAIL=0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.status: readyTriaged, unblocked, and fully specified; eligible to pick up.

    Type

    Fields

    No fields configured for Task.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions