Skip to content

fix(backfill): create issue rows before PR metadata jobs#95

Open
JSONbored wants to merge 1 commit into
entrius:testfrom
JSONbored:codex/das-backfill-issues-before-metadata
Open

fix(backfill): create issue rows before PR metadata jobs#95
JSONbored wants to merge 1 commit into
entrius:testfrom
JSONbored:codex/das-backfill-issues-before-metadata

Conversation

@JSONbored
Copy link
Copy Markdown

Summary

  • Move issue backfill before PR metadata/file job enqueueing during repository backfill.
  • Keep the change scoped to the existing backfill ordering path.

What changed

  • handleBackfill() now calls backfillIssues() immediately after PR backfill completes.
  • PR metadata and file follow-up jobs are enqueued only after issue rows have been fetched/upserted.

Why

PR metadata jobs can only link solved_by_pr when the referenced issue rows already exist. Enqueueing metadata jobs before issue backfill can leave already-seen PRs without issue attribution during backfill.

Fixes #28

Validation

  • npm ci
  • npm run build
  • npm run lint
  • npm run format:check
  • git diff --check

Move issue backfill ahead of PR metadata and file job enqueueing so metadata jobs can resolve solved_by_pr links during backfill.

Fixes entrius#28
@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label May 14, 2026
@JSONbored JSONbored marked this pull request as ready for review May 14, 2026 12:37
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

Development

Successfully merging this pull request may close these issues.

Backfill can miss solved_by_pr links when PR metadata runs before issues exist

1 participant