test(triggers): add integration tests for trigger config resolver

[aaight]

Summary

• Adds tests/integration/trigger-config-resolver.test.ts with 26 integration tests covering the full resolution chain for trigger configurations
• Tests verify definition defaults → DB overrides → merged output for all four exported resolver functions
• Tests the isTriggerEnabled, getTriggerParameters, resolveTriggerConfigs, and getResolvedTriggerConfig functions end-to-end against a real test database

What was implemented

• isTriggerEnabled() — 7 tests: returns definition default (true/false), DB override wins (true/false), agent-not-enabled gate, unknown agent type, unknown trigger event
• getTriggerParameters() — 7 tests: definition defaults, empty params, DB override wins, parameter merging (DB wins on conflicts + adds new keys), agent-not-enabled gate, unknown agent/trigger
• resolveTriggerConfigs() — 6 tests: all triggers with defaults, isCustomized flag, agent-not-enabled, unknown agent, no-trigger definition, multi-trigger DB override merging
• getResolvedTriggerConfig() — 6 tests: known event (no DB), known event (with DB override), null for unknown event, null when agent not enabled, null for unknown agent, description/providers from definition

Test approach

• Seeds a custom test-resolver-agent definition into the DB for each test, avoiding dependency on built-in YAML definitions
• Calls clearDefinitionCache() in beforeEach to prevent stale cache state between tests
• Uses existing seedAgentDefinition, seedAgentConfig, seedTriggerConfig helpers from tests/integration/helpers/seed.ts
• Uses truncateAll() in beforeEach for test isolation

Test plan

• All 26 new integration tests pass
• All 5422 existing unit tests pass
• Lint clean (biome check --write . — no fixes applied)
• TypeScript type check clean (tsc --noEmit — no errors)

Trello: https://trello.com/c/69b8008fa6ab955c5b3f814b

🤖 Generated with Claude Code

🕵️ claude-code · claude-sonnet-4-6 · run details

[nhopeatall]

LGTM — well-structured integration tests that thoroughly cover the trigger config resolution chain (definition defaults → DB overrides → merged output) across all four exported functions. Test isolation is solid (truncateAll + clearDefinitionCache in beforeEach), fixtures are self-contained with a custom agent type to avoid coupling to built-in YAMLs, and edge cases (unknown agent/trigger, agent-not-enabled gate) are all covered. All CI checks pass.

🕵️ claude-code · claude-opus-4-6 · run details