refactor(triggers): decouple triggers from PR body work item parsing

[zbigniewsobiecki]

Summary

• Add pr_work_items DB table to persist PR-to-work-item links at PR creation time, replacing fragile PR body URL parsing
• Gate check-suite triggers on implementer persona authorship instead of Trello URL presence — fixes JIRA projects where PRs were silently skipped
• Add resolveWorkItemId() utility with DB-first lookup and PR body extraction fallback for backward compatibility
• All triggers now fire without a linked work item (graceful degradation) — agents run, PM updates are simply skipped

Motivation

PR #77 on car-dealership (JIRA project) didn't trigger a review because CheckSuiteSuccessTrigger gated on finding a Trello card URL in the PR body. JIRA uses issue keys like BTS-54, so the Trello-only regex silently skipped all JIRA PRs.

Changes

Database

• New migration 0014_pr_work_items.sql with unique constraint on (project_id, pr_number)
• Drizzle schema + repository with linkPRToWorkItem (upsert) and lookupWorkItemForPR

GitHub client

• Add user: { login } to PRDetails for persona matching in check-suite triggers

Session state / CreatePR gadget

• Thread projectId and cardId through session state → builder factory → agents
• CreatePR gadget persists the PR↔work-item link after successful PR creation (best-effort)

Triggers (8 files updated)

Trigger	Before	After
check-suite-success	Gate on hasTrelloCardUrl()	Gate on implementer persona authorship
check-suite-failure	Gate on hasTrelloCardUrl()	Gate on implementer persona authorship
pr-review-submitted	requireTrelloCardId() → return null	resolveWorkItemId() → proceed
pr-comment-mention	requireTrelloCardId() → return null	resolveWorkItemId() → proceed
pr-opened	hasTrelloCardUrl() → return null	resolveWorkItemId() → proceed
review-requested	extractWorkItemId() → return null	resolveWorkItemId() → proceed
pr-ready-to-merge	requireWorkItemId() → return null	resolveWorkItemId() → skip if none
pr-merged	requireWorkItemId() → return null	resolveWorkItemId() → skip if none

Backward compatibility

resolveWorkItemId() falls back to PR body regex extraction when the DB lookup returns null, so PRs created before this migration continue to work.

Test plan

• New unit tests for prWorkItemsRepository (upsert, lookup)
• New unit tests for resolveWorkItemId (DB hit, PR body fallback, JIRA fallback, DB error fallback, null body)
• Updated check-suite-success tests: implementer persona gate replaces Trello URL gate
• Updated check-suite-failure tests: same pattern
• Updated all other trigger tests: mock lookupWorkItemForPR, verify triggers fire without work item
• Updated getPR tests: user field in response
• Updated CreatePR gadget tests: repoFullName in result
• All 2506 tests passing, lint clean, typecheck clean

🤖 Generated with Claude Code