Skip to content

Support non-default base branches for agent PRs (evaluation lanes) - #83

Merged
iamkayleb merged 1 commit into
mainfrom
eval/autonomous-agent-pipeline
Aug 25, 2026
Merged

iamkayleb merged 1 commit into
mainfrom
eval/autonomous-agent-pipeline

Conversation

@iamkayleb

Copy link
Copy Markdown
Owner

Agent bootstrap PRs always targeted the repository default branch: the issue bridge hardcoded base = data.default_branch with no override. That makes per-agent evaluation lanes (eval/claude, eval/codex) impossible — every agent PR lands on main.

  • reusable-agents-issue-bridge: resolve the PR base as base_branch input > <!-- base-branch: X --> issue marker > repo default, with an existence guard that falls back to the default when a marker names a missing branch (a typo can no longer wedge dispatch). The input is passed via env, not inline interpolation.
  • agents-verify-to-new-pr: carry a non-default PR base into the generated follow-up issue as a base-branch marker (both the LLM and fallback paths), so remediation PRs stay in the lane instead of escaping to main.

Resolution order verified against all four cases; workflow YAML validator passes. Two pre-existing long-line lint failures in the bridge are upstream and untouched.

Workflow Source

Started from:

  • GitHub issue: #
  • Direct PR / remote GitHub work
  • Local Codex/user request
  • Automation run
  • Review follow-up from PR #
  • Sync / maintenance campaign
  • Dependabot or dependency update
  • Do not automate

Automation intent:

  • Verifier should review this
  • Keepalive may manage this PR
  • Human-only unless checks fail

Notes:

Summary

Testing

Agent bootstrap PRs always targeted the repository default branch: the
issue bridge hardcoded `base = data.default_branch` with no override. That
makes per-agent evaluation lanes (eval/claude, eval/codex) impossible —
every agent PR lands on main.

- reusable-agents-issue-bridge: resolve the PR base as
  base_branch input > `<!-- base-branch: X -->` issue marker > repo
  default, with an existence guard that falls back to the default when a
  marker names a missing branch (a typo can no longer wedge dispatch).
  The input is passed via env, not inline interpolation.
- agents-verify-to-new-pr: carry a non-default PR base into the generated
  follow-up issue as a base-branch marker (both the LLM and fallback
  paths), so remediation PRs stay in the lane instead of escaping to main.

Resolution order verified against all four cases; workflow YAML validator
passes. Two pre-existing long-line lint failures in the bridge are
upstream and untouched.
@iamkayleb iamkayleb closed this Aug 25, 2026
@iamkayleb iamkayleb reopened this Aug 25, 2026
@iamkayleb
iamkayleb merged commit 16dc53b into main Aug 25, 2026
26 of 36 checks passed
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.

2 participants