Skip to content

fix(resolve_base): a main push whose HEAD is behind the tip is a push shape too — main red on 6157d1924 - #3201

Merged
noahgift merged 2 commits into
mainfrom
PMAT-1098-resolve-base-push-behind-tip
Sep 13, 2026
Merged

fix(resolve_base): a main push whose HEAD is behind the tip is a push shape too — main red on 6157d1924#3201
noahgift merged 2 commits into
mainfrom
PMAT-1098-resolve-base-push-behind-tip

Conversation

@noahgift

Copy link
Copy Markdown
Contributor

Main went red on 6157d19 (run 34747052599): check_hardcoded_paths.sh refused with "the resolver named HEAD itself as the base (merge-base(origin/main, HEAD)); never the tree against itself". The run's guard-tree started at 08:16Z, after the next squash (daa07e6) had landed on main, so HEAD was on main but no longer its tip. resolve_base.sh recognised the push shape only when HEAD is the tip; one commit behind, merge-base(origin/main, HEAD) is HEAD and the run is refused. Any main push whose run starts after the next merge lands hits this.

Fix: push shape = HEAD is on origin/main's first-parent line (tip or behind it), base = HEAD's first parent. "Ancestor of the tip" alone is wrong: a feature commit merged into main is an ancestor too and its base is the merge-base (row 15 catches that). New case-table row 19 (HEAD behind the tip in an isolated clone) is RED on main's resolver and GREEN here; rows 1–25 all pass; check_hardcoded_paths.sh --full-if-capable passes on this tree.

no-close: the red run is a guard defect on a merged commit; no issue was filed for it (T-5: fix, do not file).

🤖 Generated with Claude Code

…med HEAD as its own base and was refused — main went red on 6157d19 while a later merge had landed; the push shape is 'on main's first-parent line', not 'is the tip' (row 19)

Pmat-Ticket: PMAT-1098

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@noahgift
noahgift enabled auto-merge September 13, 2026 09:03
@github-actions

github-actions Bot commented Sep 13, 2026

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=3201 head=75bad36f7683d6d6056acbaa085d8773c243e883 verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

check_roadmap_diff_additive.sh reported base id(s) missing at head: this branch forked before they landed on main, and a plain update takes one side of the file whole. Resolved as main's roadmap plus this branch's own new entries, each at its sorted slot among same-prefix peers. The other three roadmap guards pass on the result.

Pmat-Ticket: PMAT-1098

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@noahgift
noahgift added this pull request to the merge queue Sep 13, 2026
Merged via the queue into main with commit 966ec97 Sep 13, 2026
18 of 19 checks passed
@noahgift
noahgift deleted the PMAT-1098-resolve-base-push-behind-tip branch September 13, 2026 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant