refactor: trigger config migration cleanup

src/agents/definitions/review.yaml

@@ -48,7 +48,26 @@ triggers:
     description: Trigger review when a new PR is opened (without waiting for CI)
     defaultEnabled: false
     providers: [github]
+    parameters:
+      - name: authorMode
+        type: select
+        label: Author Filter
+        description: Filter PRs by author type
+        options: [own, external, all]
+        defaultValue: own
     contextPipeline: [prContext, contextFiles, squint]
+  - event: scm:pr-ready-to-merge
+    label: PR Ready to Merge
+    description: Move work item to DONE when PR is approved and all checks pass
+    defaultEnabled: true
+    providers: [github]
+    contextPipeline: []
+  - event: scm:pr-merged
+    label: PR Merged
+    description: Move work item to MERGED status when PR is merged
+    defaultEnabled: true
+    providers: [github]
+    contextPipeline: []
 
 strategies: {}
 

src/config/schema.ts

@@ -1,9 +1,4 @@
 import { z } from 'zod';
-import {
-	GitHubTriggerConfigSchema,
-	JiraTriggerConfigSchema,
-	TrelloTriggerConfigSchema,
-} from './triggerConfig.js';
 
 const AgentBackendConfigSchema = z.object({
 	default: z.string().default('llmist'),
@@ -29,7 +24,6 @@ const JiraConfigSchema = z.object({
 			readyToProcess: z.string().default('cascade-ready'),
 		})
 		.optional(),
-	triggers: JiraTriggerConfigSchema.partial().optional(),
 });
 
 export const ProjectConfigSchema = z.object({
@@ -59,22 +53,11 @@ export const ProjectConfigSchema = z.object({
 					cost: z.string().optional(),
 				})
 				.optional(),
-			triggers: TrelloTriggerConfigSchema.partial().optional(),
 		})
 		.optional(),
 
 	jira: JiraConfigSchema.optional(),
 
-	/**
-	 * GitHub-specific configuration, including trigger toggles.
-	 * Separate from trello/jira because GitHub integration is always present for code operations.
-	 */
-	github: z
-		.object({
-			triggers: GitHubTriggerConfigSchema.partial().optional(),
-		})
-		.optional(),
-
 	model: z.string().optional(),
 	agentModels: z.record(z.string()).optional(),
 	cardBudgetUsd: z.number().positive().optional(),