Repository navigation
fix(orch): prefer active PR over branch history - #684
Open
rudycelekli wants to merge 1 commit into
Open
rudycelekli wants to merge 1 commit into
rudycelekli wants to merge 1 commit into
Conversation
Signed-off-by: Rudy Celekli <rudycelekli@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
After a stacker closes an obsolete PR and creates a replacement from the same branch,
orch frontier setrejects that branch because its all-state forge query returns both PRs. The suggested recovery, closing stale PRs, cannot clear the error because closed rows are still included.Prefer the sole OPEN PR when selecting a branch anchor. Retain closed and merged anchors when no active PR exists, and continue rejecting multiple active PRs. Both GitHub and Origin use the same policy. Regenerate the Claude Code harness.
Verification: real local Git stacks with fixture forge executables reproduce two baseline failures (GitHub and Origin), with 19 controls passing. Fixed source passes all 72 native script tests using Bun 1.4.3 with a 30-second per-test timeout. Coverage includes closed and merged branch history, active ambiguity, historical chain anchors, and optional frontier pins. Strict typecheck, plugin validator, generated harness check, and diff whitespace check pass. No hosted mutations or credentials are used by tests.
AI assistance: Codex implementation and local verification.
Note
Medium Risk
Changes how
orch frontier setresolves branch anchors in the PR base chain; incorrect selection could mis-pin the frontier, but behavior is narrow and covered by new tests.Overview
Frontier resolution no longer treats every PR returned by an all-states
--headquery as competing anchors on the same branch.branchPullRequestnow picks the sole OPEN PR when one exists (e.g. after closing an obsolete PR and opening a replacement on the same branch), and only falls back to closed/merged rows when there is no active PR—preserving historical chain anchors after stack surgery.The multiple PR error applies to that candidate set (multiple open PRs still fail). Tests cover GitHub and Origin for replacement PRs alongside closed/merged history and for rejecting two open PRs on one branch.
Reviewed by Cursor Bugbot for commit 0dc8625. Bugbot is set up for automated code reviews on this repo. Configure here.