Skip to content

fix: filter cross-repo closing references in PR timeline query (#1041) - #1147

Closed
alpurkan17 wants to merge 19 commits into
entrius:testfrom
alpurkan17:fix/1041-cross-repo-timeline-closing
Closed

fix: filter cross-repo closing references in PR timeline query (#1041)#1147
alpurkan17 wants to merge 19 commits into
entrius:testfrom
alpurkan17:fix/1041-cross-repo-timeline-closing

Conversation

@alpurkan17

Copy link
Copy Markdown
Contributor

Summary

_PR_TIMELINE_QUERY's closingIssuesReferences fragment emits only nodes { number } without repository qualification. When a PR closes a same-numbered issue in a different repository, find_solver_from_cross_references matches by issue number alone and can credit the wrong solver.

Fix mirrors the approach from PR #1020 on the timeline query:

  • Add repository { nameWithOwner } to the closingIssuesReferences fragment in _PR_TIMELINE_QUERY
  • Filter closing numbers by matching repository name in the parser

Validation

  • uv run ruff check — clean
  • uv run ruff format --check — clean
  • uv run pyright — 0 errors

Closes #1041

alpurkan17 and others added 19 commits May 11, 2026 04:05
- time_decay_tracker.py: monitor PR age & decay multiplier
- pr_quality_check.py: validate 7 scoring factors before PR submission
- GITTENSOR_STATUS.md: full miner status, 47 rules, compliance table
- Analysis of last 20 merged PRs (labels, prefix, contributors)
- fix: prefix has 60% merge rate - safest choice
- Updated checklist based on real merged PR patterns
- pr_body_builder.py: generate body PR sesuai gaya anderdc (## Summary + bullet + ## Test plan)
- Update strategi: adopsi pola anderdc (tanpa formal checklist, fokus ke technical summary)
- Body template baru: ## Summary, ## Related Issues (Fixes #N), ## Test plan
…ommit conventions

- issue_checker.py: cek author issue (maintainer=1.66x vs standard=1.33x)
  - --suggest: cari open issues dari maintainer
- pre_submit.py: validasi branch, commit, ruff, pyright, pytest sebelum push
- pr_body_builder.py: generate body gaya anderdc (## Summary + Fixes #N + ## Test plan)
- Update strategi: branch naming, commit message convention, Review Quality optimasi
- pr_body_builder.py v2: struktur ala anderdc dengan Root Cause + Impact + Why
- Update semua 10 PR body dengan gaya enhanced via REST API
- Setiap PR sekarang punya: Summary, Root Cause, Impact, Fixes #N, Test plan
@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label May 11, 2026
@alpurkan17

Copy link
Copy Markdown
Contributor Author

Closing — will recreate with clean branch

@alpurkan17 alpurkan17 closed this May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

1 participant