Skip to content

fix: deduplicate issues across paginated responses - #12

Open
wangedmund77-cmyk wants to merge 1 commit into
alisteuber4ee1:mainfrom
wangedmund77-cmyk:codex/deduplicate-paginated-issues
Open

fix: deduplicate issues across paginated responses#12
wangedmund77-cmyk wants to merge 1 commit into
alisteuber4ee1:mainfrom
wangedmund77-cmyk:codex/deduplicate-paginated-issues

Conversation

@wangedmund77-cmyk

Copy link
Copy Markdown

/claim #1

Summary

  • Deduplicates issues across paginated GitHub responses using the issue API URL, falling back to ID when needed.
  • Keeps pagination driven by resp.NextPage while preserving the since query across pages.
  • Adds a regression test for duplicate issues returned on different pages and fixes test server setup so handlers can safely build Link headers.

Verification

  • Not run locally: Go toolchain is not installed in this environment (go: command not found).
  • Static review confirms the new test covers two paginated requests, since preservation, and three unique issues from four returned rows.

@opirebot

opirebot Bot commented Jul 11, 2026

Copy link
Copy Markdown

👀 We've notified the reward creators here.
Make sure your payment account is ready to receive the payment for your hard work 💪

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