fix(triggers): address code review findings for integration-driven architecture

src/agents/definitions/debug.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Analyzes session logs to identify what went wrong
   initialMessage: "**\U0001F41B Analyzing session logs** — Reviewing what happened and identifying issues..."
 
+# Explicit integration requirements.
+integrations:
+  required: [pm]
+  optional: []
+
 # Read-only FS access for log analysis, full PM access for creating debug cards.
 capabilities:
   required:
@@ -15,6 +20,10 @@ capabilities:
     - pm:checklist
   optional: []
 
+# Debug agent is triggered manually or via internal attachment upload detection.
+# No external event-based triggers are configured.
+triggers: []
+
 strategies:
   contextPipeline: [directoryListing, contextFiles, squint, workItem]
 

src/agents/definitions/email-joke.yaml

@@ -4,6 +4,12 @@ identity:
   roleHint: Reads emails from a specific sender and responds with jokes
   initialMessage: "**\U0001F602 Checking for emails to respond to with jokes**"
 
+# Explicit integration requirements.
+# Email-joke agent demonstrates an agent that ONLY needs email integration.
+integrations:
+  required: [email]
+  optional: []
+
 # Minimal agent with email access only.
 # No file editing, no GitHub, no PM tools.
 capabilities:
@@ -13,6 +19,19 @@ capabilities:
     - email:write
   optional: []
 
+# Supported triggers for this agent
+triggers:
+  - event: email:received
+    label: Email Received
+    description: Trigger when new emails are received (polled periodically)
+    defaultEnabled: true
+    parameters:
+      - name: senderEmail
+        type: email
+        label: Sender Email Filter
+        description: Only process emails from this sender (leave empty for all)
+        required: false
+
 strategies:
   contextPipeline: [prefetchedEmails]
 

src/agents/definitions/implementation.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Writes code, runs tests, and prepares a pull request
   initialMessage: "**\U0001F680 Implementing changes** — Writing code, running tests, and preparing a PR..."
 
+# Explicit integration requirements.
+integrations:
+  required: [pm, scm]
+  optional: []
+
 # Capabilities define what the agent can do.
 # Integrations and tools are DERIVED from these capabilities.
 capabilities:
@@ -18,6 +23,41 @@ capabilities:
     - scm:pr
   optional: []
 
+# Supported triggers for this agent
+triggers:
+  - event: pm:card-moved
+    label: Card Moved to Todo
+    description: Trigger when card moved to Todo list
+    defaultEnabled: true
+    providers: [trello]
+    parameters:
+      - name: targetList
+        type: select
+        label: Target List
+        options: [todo]
+        defaultValue: todo
+  - event: pm:issue-transitioned
+    label: Issue Transitioned
+    description: Trigger when issue transitions to Todo status
+    defaultEnabled: true
+    providers: [jira]
+    parameters:
+      - name: targetStatus
+        type: select
+        label: Target Status
+        options: [todo]
+        defaultValue: todo
+  - event: pm:label-added
+    label: Ready to Process Label
+    description: Trigger when Ready to Process label added to a card in the Todo list
+    defaultEnabled: true
+    parameters:
+      - name: listKey
+        type: select
+        label: Target List
+        options: [todo]
+        defaultValue: todo
+
 strategies:
   contextPipeline: [directoryListing, contextFiles, squint, workItem]
 

src/agents/definitions/planning.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Studies the codebase and designs a step-by-step implementation plan
   initialMessage: "**\U0001F5FA\uFE0F Planning implementation** — Studying the codebase and designing a step-by-step plan..."
 
+# Explicit integration requirements.
+integrations:
+  required: [pm]
+  optional: []
+
 # Read-only agent that explores code and writes plans to PM system.
 # No file editing, no PR creation, no checklist updates.
 capabilities:
@@ -15,6 +20,45 @@ capabilities:
     - pm:write
   optional: []
 
+# Supported triggers for this agent
+triggers:
+  - event: pm:card-moved
+    label: Card Moved to Planning
+    description: Trigger when card moved to Planning list
+    defaultEnabled: true
+    providers: [trello]
+    parameters:
+      - name: targetList
+        type: select
+        label: Target List
+        options: [planning]
+        defaultValue: planning
+  - event: pm:issue-transitioned
+    label: Issue Transitioned
+    description: Trigger when issue transitions to Planning status
+    defaultEnabled: true
+    providers: [jira]
+    parameters:
+      - name: targetStatus
+        type: select
+        label: Target Status
+        options: [planning]
+        defaultValue: planning
+  - event: pm:label-added
+    label: Ready to Process Label
+    description: Trigger when Ready to Process label added to a card in Planning list
+    defaultEnabled: true
+    parameters:
+      - name: listKey
+        type: select
+        label: Target List
+        options: [planning]
+        defaultValue: planning
+  - event: pm:comment-mention
+    label: Comment @mention
+    description: Trigger when bot is @mentioned in a card/issue comment
+    defaultEnabled: true
+
 strategies:
   contextPipeline: [directoryListing, contextFiles, squint, workItem]
 

src/agents/definitions/respond-to-ci.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Analyzes failed CI checks and works on a fix
   initialMessage: "**\U0001F527 Fixing CI failures** — Analyzing the failed checks and working on a fix..."
 
+# Explicit integration requirements.
+integrations:
+  required: [scm]
+  optional: [pm]
+
 # Can edit files and read SCM, optional PM for status updates.
 capabilities:
   required:
@@ -18,6 +23,14 @@ capabilities:
     - pm:write
     - pm:checklist
 
+# Supported triggers for this agent
+triggers:
+  - event: scm:check-suite-failure
+    label: Check Suite Failure
+    description: Trigger when CI checks fail
+    defaultEnabled: true
+    providers: [github]
+
 strategies:
   contextPipeline: [prContext, directoryListing, contextFiles, squint, workItem]
 

src/agents/definitions/respond-to-planning-comment.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Reads user feedback and updates the plan accordingly
   initialMessage: "**\U0001F4AC Responding to feedback** — Reading your comment and updating the plan accordingly..."
 
+# Explicit integration requirements.
+integrations:
+  required: [pm]
+  optional: []
+
 # Can update PM checklists to respond to feedback, but no file editing.
 capabilities:
   required:
@@ -15,6 +20,14 @@ capabilities:
     - pm:checklist
   optional: []
 
+# Supported triggers for this agent
+# Note: This agent is triggered via pm:comment-mention shared with planning agent
+triggers:
+  - event: pm:comment-mention
+    label: Comment @mention
+    description: Trigger when bot is @mentioned in a card/issue comment
+    defaultEnabled: true
+
 strategies:
   contextPipeline: [directoryListing, contextFiles, squint, workItem]
 

src/agents/definitions/respond-to-pr-comment.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Reads a PR comment and takes action
   initialMessage: "**\U0001F4AC Responding to PR comment** — Reading your comment and taking action..."
 
+# Explicit integration requirements.
+integrations:
+  required: [scm]
+  optional: [pm]
+
 # Can edit files and interact with PR comments, optional PM for status updates.
 capabilities:
   required:
@@ -17,6 +22,14 @@ capabilities:
     - pm:read
     - pm:write
 
+# Supported triggers for this agent
+triggers:
+  - event: scm:pr-comment-mention
+    label: PR Comment @mention
+    description: Trigger when the implementer bot is @mentioned in a PR comment
+    defaultEnabled: true
+    providers: [github]
+
 strategies:
   contextPipeline: [prContext, prConversation, directoryListing, contextFiles, squint]
   gadgetOptions:

src/agents/definitions/respond-to-review.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Addresses code review feedback by making requested changes
   initialMessage: "**\U0001F527 Addressing review feedback** — Making the requested changes from the code review..."
 
+# Explicit integration requirements.
+integrations:
+  required: [scm]
+  optional: [pm]
+
 # Can edit files to address review feedback, interact with PR comments.
 # Optional PM for status updates if configured.
 capabilities:
@@ -18,6 +23,14 @@ capabilities:
     - pm:read
     - pm:write
 
+# Supported triggers for this agent
+triggers:
+  - event: scm:pr-review-submitted
+    label: PR Review Submitted
+    description: Trigger when a review with changes requested is submitted
+    defaultEnabled: true
+    providers: [github]
+
 strategies:
   contextPipeline: [prContext, prConversation, directoryListing, contextFiles, squint]
   gadgetOptions:

src/agents/definitions/review.yaml

@@ -4,6 +4,11 @@ identity:
   roleHint: Reviews pull request changes for quality and correctness
   initialMessage: "**\U0001F50D Reviewing code** — Examining the PR changes for quality and correctness..."
 
+# Explicit integration requirements.
+integrations:
+  required: [scm]
+  optional: [pm]
+
 # Read-only agent that reviews PRs. Can submit reviews and update comments.
 # Optional PM for reading linked work item context.
 capabilities:
@@ -17,6 +22,31 @@ capabilities:
   optional:
     - pm:read
 
+# Supported triggers for this agent
+triggers:
+  - event: scm:check-suite-success
+    label: CI Passed
+    description: Trigger review when CI checks pass on a PR
+    defaultEnabled: false
+    providers: [github]
+    parameters:
+      - name: authorMode
+        type: select
+        label: Author Filter
+        description: Filter PRs by author type
+        options: [own, external, all]
+        defaultValue: own
+  - event: scm:review-requested
+    label: On Review Requested
+    description: Trigger review when a CASCADE persona is explicitly requested as reviewer
+    defaultEnabled: false
+    providers: [github]
+  - event: scm:pr-opened
+    label: PR Opened
+    description: Trigger review when a new PR is opened (without waiting for CI)
+    defaultEnabled: false
+    providers: [github]
+
 strategies:
   contextPipeline: [prContext, contextFiles, squint]