Skip to content

fix: enforce GitHub author commit threshold#270

Open
thejonali wants to merge 1 commit into
interviewstreet:mainfrom
thejonali:fix/github-author-commit-threshold
Open

fix: enforce GitHub author commit threshold#270
thejonali wants to merge 1 commit into
interviewstreet:mainfrom
thejonali:fix/github-author-commit-threshold

Conversation

@thejonali

@thejonali thejonali commented Jun 25, 2026

Copy link
Copy Markdown

Summary

Enforces the existing GitHub project selection rule that selected repositories must have at least 4 author commits.

Previously, the prompt required author_commit_count >= 4, but the code only filtered out repositories with exactly 0 author commits. As a result, repositories with 1-3 author commits could still be passed to the LLM and returned in the final selected GitHub projects.

Changes

  • Filters candidate GitHub projects before LLM selection when author_commit_count < 4.
  • Filters LLM-selected projects again before returning them, so invalid selections cannot pass through even if the model returns them.

Validation

  • Ran the resume workflow against a GitHub account
  • Ran python -m py_compile github.py
  • Confirmed repos < 4 were not included

Closes #269

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

GitHub project selector includes repositories below author commit threshold

1 participant