test: add coverage for 10 highest-risk untested modules

[aaight]

Summary

Adds 10 new test files covering the highest-risk untested modules identified in the coverage analysis:

• tests/unit/triggers/config-resolver.test.ts — 20 tests covering resolveTriggerConfigs, isTriggerEnabled, getTriggerParameters, getResolvedTriggerConfig with DB override merging and definition fallback logic
• tests/unit/triggers/shared/trigger-check.test.ts — 10 tests for checkTriggerEnabled and checkTriggerEnabledWithParams including logging behavior
• tests/unit/gadgets/sessionState.test.ts — 16 tests for initSessionState, getters, record functions, deleteInitialComment with error recovery, and getSessionState copy semantics
• tests/unit/config/agentMessages.test.ts — 10 tests for initAgentMessages, proxy guard enforcement (pre-init throws), getAgentLabel fallback, and _resetAgentMessages
• tests/unit/gadgets/github/core/getPRChecks.test.ts — 14 tests for formatCheckStatus (all icon types, edge cases) and getPRChecks async + error paths
• tests/unit/gadgets/github/core/getPRComments.test.ts — 8 tests for getPRComments covering empty/populated/reply/line-omission cases
• tests/unit/server/webhookParsers.test.ts — 13 tests for parseTrelloPayload, parseGitHubPayload, parseJiraPayload including header extraction and parse failures
• tests/unit/server/webhookReactionSender.test.ts — 16 tests for buildReactionSender for trello, github, jira, and unknown sources
• tests/unit/backends/progressMonitor.test.ts — 29 tests covering constructor, start(), stop(), tick lifecycle, concurrent tick skip, PM/GitHub poster delegation, fallback on model failure
• tests/unit/agents/definitions/profiles.test.ts — 17 tests for getAgentProfile (sdkTools, needsGitHubToken, filterTools, finishHooks, fetchContext, buildTaskPrompt, getLlmistGadgets) and hasFinishValidation
• tests/unit/triggers/jira-comment-mention.test.ts — 17 tests for JiraCommentMentionTrigger.matches() and handle() including wiki markup and ADF mention detection, self-authored skip, trigger disabled path

Total: 170 new tests across 10 files

Test plan

• All 234 unit test files pass (3958 tests total)
• Lint passes (biome check with no errors)
• TypeScript type check passes
• All new tests follow existing project conventions: vi.mock(), vi.hoisted(), describe/it blocks
• Module-level state modules (sessionState.ts, agentMessages.ts) properly reset between tests

Card: https://trello.com/c/nHlrCbXy/174-find-top-area-that-needs-test-coverage-and-plan-new-tests

🤖 Generated with Claude Code

[nhopeatall]

Summary

LGTM — Solid test coverage addition for 11 high-risk untested modules. All 170 tests pass, CI is green, and the implementations correctly mirror the source code behavior.

Verified:

• Cross-referenced test assertions against source implementations (sessionState, agentMessages, trigger-check, getPRChecks, getPRComments, webhookReactionSender)
• Tests follow existing project conventions (vi.mock(), vi.hoisted(), describe/it, beforeEach cleanup)
• Module-level state modules (sessionState, agentMessages) properly reset between tests
• Error paths, edge cases, and boundary conditions are well-covered (e.g., self-authored comment skip in Jira trigger, proxy guard enforcement, concurrent tick skip in ProgressMonitor)

Minor note: The PR description says "10 new test files" but actually includes 11 — not a code issue, just a description mismatch.