Skip to content

fix(github): scope PR reviewer wakes by task key (BLO-7457) - #167

Merged
kkroo merged 2 commits into
masterfrom
omar/blo-7457-github-webhook-context
May 25, 2026
Merged

fix(github): scope PR reviewer wakes by task key (BLO-7457)#167
kkroo merged 2 commits into
masterfrom
omar/blo-7457-github-webhook-context

Conversation

@kkroo

@kkroo kkroo commented May 25, 2026

Copy link
Copy Markdown

Summary

  • add a stable pr_review:<repo>:<number> task key to GitHub reviewer wakes
  • preserve that task key in both wake payload and heartbeat contextSnapshot so reviewer wakes no longer share the null task scope
  • add a BLO-7457 regression covering a thin active automation run plus a new reviewer PR wake
  • let webhook route tests inject heartbeat test options so they do not dispatch background runs or hit ccrotate during assertions

Verification

  • pnpm exec vitest run server/src/__tests__/github-webhook.test.ts
  • pnpm --filter @paperclipai/server typecheck

Refs BLO-7457.

@kkroo

kkroo commented May 25, 2026

Copy link
Copy Markdown
Author

@ally please review BLO-7457. Focus area: GitHub reviewer wakes now carry a stable pr_review:<repo>:<number> taskKey through payload/contextSnapshot, and the regression verifies a rich reviewer wake is not coalesced into a thin null-scope automation run.

@kkroo
kkroo merged commit 56f8185 into master May 25, 2026
8 checks passed
@kkroo
kkroo deleted the omar/blo-7457-github-webhook-context branch May 25, 2026 03:25
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