test(triggers): characterize agent execution pipeline

[aaight]

Summary

Adds facade-level characterization coverage for runAgentExecutionPipeline before production helper extraction.

Card: https://trello.com/c/69fdedb3941d8fafe8cf4bb1

Changes

• Pins the missing-agentType guard so the lifecycle, validation, and agent runner are not invoked for non-dispatchable trigger results.
• Covers integration-validation failure behavior, including PM failure notification when PM validation is usable, PM-notification skip when PM validation failed, and onFailure callback invocation.
• Adds best-effort persistence coverage showing work-item row failures do not prevent agent execution.
• Pins callback ordering for onSuccess and onFailure after agent execution and post-run lifecycle handling.
• Adds facade coverage for lifecycle skip flags around prepare/cleanup/failure/success handling.

Tests

• npx vitest run --project unit-triggers tests/unit/triggers/shared/agent-execution.test.ts tests/unit/triggers/agent-execution.test.ts
• npm run lint:fix
• npm run lint
• npm run typecheck
• npm test

🕵️ codex · gpt-5.5 · run details

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[nhopeatall]

LGTM - the PR is test-only, keeps production code unchanged, and adds facade-level characterization coverage for the requested agent execution pipeline branches. Targeted trigger tests pass locally, and CI is green.

🕵️ codex · gpt-5.5 · run details