Skip to content

feat: make GitHub reviews incremental and idempotent - #57

Merged
EmersonBraun merged 1 commit into
mainfrom
codex/issue-46-github-review-state
Aug 26, 2026
Merged

feat: make GitHub reviews incremental and idempotent#57
EmersonBraun merged 1 commit into
mainfrom
codex/issue-46-github-review-state

Conversation

@EmersonBraun

Copy link
Copy Markdown
Member

Summary

Closes #46.

  • Adds bounded, timeout-controlled GETs with one idempotent retry and no automatic POST/PATCH retry.
  • Adds SHA/policy fingerprints and marker reconciliation to skip duplicate provider work and update one summary comment.
  • Uses GitHub compare scope only when the previous marked SHA is an ancestor; missing/changed policy or force-push falls back to the full PR file list.
  • Detects fork PRs on the safe pull_request boundary and exits SKIPPED/2 before provider execution.
  • Adds offline tests for markers, retry, fork detection, and incremental scope; updates operations/changelog docs.

Verification

  • npm run check
  • npm pack --dry-run
  • 53 tests passing locally.

@EmersonBraun
EmersonBraun merged commit e9eba01 into main Aug 26, 2026
5 checks passed
@EmersonBraun
EmersonBraun deleted the codex/issue-46-github-review-state branch August 26, 2026 02:05
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.

feat: make GitHub reporting incremental and fork-safe

1 participant