Skip to content

fix(page): scope contributor PR links to program repos - #103

Merged
anurag629 merged 1 commit into
codercops:developfrom
sank8-2:fix/contributor-pr-search-repos
Oct 26, 2026
Merged

anurag629 merged 1 commit into
codercops:developfrom
sank8-2:fix/contributor-pr-search-repos

Conversation

@sank8-2

@sank8-2 sank8-2 commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

What does this change?

The contributor page's "See 's merged pull requests on GitHub" link searched org:codercops, so it could list PRs the card never counted. It now builds its query from program.repos, the same way searchMergedPrs does, via a new shared mergedPrQuery() helper in src/github.ts. Author, label, and merge-date filters are unchanged.

Closes #101

How I tested this

  • npm run typecheck — passes.
  • npm test — 378 tests pass.
  • Updated the expected search string in test/card.test.ts and added a case with a program that sets its own repos list. That new case fails against the old org-wide link and passes with this change.

AI assistance

AI assistance was used: an AI coding agent (OpenCode) drafted the mergedPrQuery extraction, the test update, and this description. I reviewed the diff and ran the typecheck and test suite locally.

Checklist

  • I was assigned the linked issue
  • npm run typecheck and npm test pass
  • New behavior has a test in test/
  • For a bug fix: the new test fails without my change
  • No tokens or secrets in the diff

Build the contributor page's GitHub search from program.repos, the same way searchMergedPrs does, instead of searching the whole organization.

Closes codercops#101

@anurag629 anurag629 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @sank8-2, this works. Typecheck and all 378 tests pass on top of develop, both link tests fail with develop's index.ts and github.ts, and searchMergedPrs still builds the same query as before.

@anurag629 anurag629 added the hacktoberfest-accepted PR counts for Hacktoberfest label Oct 26, 2026
@anurag629
anurag629 merged commit 416cca6 into codercops:develop Oct 26, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted PR counts for Hacktoberfest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The contributor page links to an org-wide PR search instead of the program's repos

2 participants