Skip to content

fix(pr-management-triage): dedupe paginated PRs - #1071

Open
awishere wants to merge 1 commit into
apache:mainfrom
awishere:codex/fix-triage-pagination-dedup
Open

fix(pr-management-triage): dedupe paginated PRs#1071
awishere wants to merge 1 commit into
apache:mainfrom
awishere:codex/fix-triage-pagination-dedup

Conversation

@awishere

@awishere awishere commented Aug 5, 2026

Copy link
Copy Markdown

Summary

  • Deduplicate PRs by number after the full pagination loop.
  • Keep the final occurrence so an updated PR uses its freshest snapshot and correct position.
  • Add regression coverage for a PR moving between pages and for an already-unique result set.

Type of change

  • Skill change (skills/<name>/) — eval fixtures updated below
  • Tool / bridge contract (tools/<system>/*.md)
  • Python package
  • Documentation
  • CI / dev loop

Test plan

  • prek run --all-files passes
  • All 35 pr-management-triage eval fixtures assemble successfully
  • Both pagination-dedup cases pass in exact automated mode
  • All 150 skill-eval framework tests pass
  • Regression fixtures are included

RFC-AI-0004 compliance

  • Vendor neutrality — no provider-specific behavior added

Linked issues

Closes #77

Generative AI disclosure

Codex was used while preparing this change. The contributor reviewed the diff and test results before submission.

Deduplicate the fully fetched list by PR number while retaining each PR’s freshest occurrence and its position in the fetched ordering.

Generated-by: Codex (GPT-5)
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.

Silent in-session de-dup of already-seen / already-skipped PRs

1 participant