Skip to content

Recognize active putting-up-PR claims - #155

Merged
laclance merged 2 commits into
devfrom
fix/putting-up-pr-claim
Oct 8, 2026
Merged

laclance merged 2 commits into
devfrom
fix/putting-up-pr-claim

Conversation

@laclance

@laclance laclance commented Oct 7, 2026

Copy link
Copy Markdown
Owner

Summary

Fix a production competition-detection gap exposed after the historical-PR attribution fix.

In grpc/grpc-go#9470, a contributor wrote before private run #13:

Putting up a PR: route all setters through an update() helper ...

That is explicit active implementation ownership, but the strategic claim parser recognizes nearby forms such as “I'll open a PR” and “I'm implementing” while missing “Putting up a PR”.

This change adds that narrow active-claim phrase family.

Safety boundary

  • active forms like Putting up a PR: or Putting up a PR to ... count
  • passive/hypothetical wording like Putting up a PR would help ... does not count
  • no change to claim age, ownership precedence, linked PR handling, readiness, scoring, or paid verification

Regression coverage

  • production-shaped Putting up a PR: ... is recognized
  • passive hypothetical wording remains a non-claim

Scope

Only:

  • opportunity_scout/strategic/claims.py
  • tests/strategic/test_claims.py

This intentionally does not touch draft PR #154's shared-tracker competition logic.

@laclance
laclance marked this pull request as ready for review October 8, 2026 18:17
@laclance
laclance merged commit c571c19 into dev Oct 8, 2026
4 checks passed
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