Skip to content

Refresh stale pull request bases - #35

Merged
dapi merged 1 commit into
masterfrom
agent/refresh-stale-pr-base
Jul 26, 2026
Merged

Refresh stale pull request bases#35
dapi merged 1 commit into
masterfrom
agent/refresh-stale-pr-base

Conversation

@dapi

@dapi dapi commented Jul 26, 2026

Copy link
Copy Markdown
Owner

What changed

When the uniquely selected remote-tracking base of an open PR is stale, review discovery now fetches only that advertised base branch into the existing tracking ref and revalidates its SHA before starting Codex.

Safety

  • Never runs git pull and does not modify the current branch, worktree, real index, or PR.
  • Never fetches for explicit --review-base, local-only, ambiguous, or non-PR bases.
  • A failed fetch or post-fetch SHA mismatch remains an operational failure.

Routing and validation

  • Flow: Feature follow-up within FT-016, which owns review-base discovery and its public contract.
  • Validation profile: high-risk; this adds a bounded network/ref mutation at the provider trust boundary.
  • go test ./...
  • go vet ./...
  • make docs-lint
  • git diff --check

@dapi
dapi marked this pull request as ready for review July 26, 2026 21:02
@dapi
dapi merged commit 48f9dc7 into master Jul 26, 2026
1 check 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.

1 participant