claude-ide-bridge 2.25.10 → 2.25.14

package/dist/automation.d.ts

@@ -18,6 +18,10 @@ export interface PromptSource {
     promptName?: string;
     promptArgs?: Record<string, string>;
     condition?: string;
+    /** Per-hook model override. Falls back to policy.defaultModel (Haiku). */
+    model?: string;
+    /** Per-hook effort override. Falls back to policy.defaultEffort ("low"). */
+    effort?: "low" | "medium" | "high" | "max";
 }
 export interface OnDiagnosticsErrorPolicy extends PromptSource {
     enabled: boolean;
@@ -238,6 +242,30 @@ export interface TaskSuccessResult {
     output: string;
 }
 export interface AutomationPolicy {
+    /**
+     * Default model for all automation tasks.
+     * Defaults to "claude-haiku-4-5-20251001" to minimise cost.
+     * Override with "claude-sonnet-4-6" etc. for hooks that need more reasoning.
+     */
+    defaultModel?: string;
+    /**
+     * Hard cap on automation tasks spawned per hour (rolling 60-min window).
+     * Defaults to 20. Set to 0 to disable.
+     */
+    maxTasksPerHour?: number;
+    /**
+     * Custom system prompt passed via --system-prompt to every automation subprocess.
+     * Replaces the default Claude Code system prompt, preventing CLAUDE.md from being
+     * loaded as workspace instructions. Keep it short — this is the biggest token lever.
+     * Default: a lean "be brief" prompt. Max 4096 chars.
+     */
+    automationSystemPrompt?: string;
+    /**
+     * Default effort level for all automation tasks (low/medium/high/max).
+     * Defaults to "low" — automation tasks rarely need deep reasoning.
+     * Override per-hook via the hook's own effort field.
+     */
+    defaultEffort?: "low" | "medium" | "high" | "max";
     onDiagnosticsError?: OnDiagnosticsErrorPolicy;
     onFileSave?: OnFileSavePolicy;
     /** Fired by Claude Code 2.1.83+ FileChanged hook — reacts to any file edit, not just explicit saves. */
@@ -330,7 +358,19 @@ export declare class AutomationHooks {
     private prevDiagnosticErrors;
     /** Active task ID for the task-success handler (workspace-global). */
     private activeTaskSuccessTaskId;
+    /**
+     * Rolling window of task enqueue timestamps for maxTasksPerHour enforcement.
+     * Entries older than 60 minutes are pruned on each enqueue.
+     */
+    private taskTimestamps;
     constructor(policy: AutomationPolicy, orchestrator: ClaudeOrchestrator, log: (msg: string) => void, extensionClient?: ExtensionClient | undefined, workspace?: string | undefined);
+    /**
+     * Central enqueue for all automation-triggered tasks.
+     * Applies defaultModel (Haiku by default) and enforces maxTasksPerHour.
+     * Throws with the same "Task queue is full" message on rate-limit breach so
+     * callers can handle it identically.
+     */
+    private _enqueueAutomationTask;
     /**
      * Resolve a named prompt, substituting any `{{placeholder}}` tokens in
      * `promptArgs` values with sanitized event data before calling `getPrompt()`.
@@ -472,5 +512,10 @@ export declare class AutomationHooks {
             cooldownMs: number;
         } | null;
         unwiredEnabledHooks: string[];
+        defaultModel: string;
+        maxTasksPerHour: number;
+        tasksThisHour: number;
+        defaultEffort: string;
+        automationSystemPrompt: string;
     };
 }

package/dist/automation.js

@@ -50,6 +50,10 @@ function truncatePrompt(prompt) {
 }
 /** Prune lastTrigger entries older than this to prevent unbounded Map growth */
 const LAST_TRIGGER_MAX_AGE_MS = 7 * 24 * 60 * 60 * 1000; // 1 week
+/** Default system prompt for automation subprocesses when none is set in policy. */
+const DEFAULT_AUTOMATION_SYSTEM_PROMPT = "You are a concise automation assistant. " +
+    "Respond in \u22645 lines. No preamble. No markdown headers. " +
+    "Call the tools listed in the task prompt, then report results only.";
 /**
  * Deterministic signature over a diagnostic list — used for content-aware
  * dedupe in onDiagnosticsError. Sorting by a stable key makes the signature
@@ -104,6 +108,16 @@ function validatePromptSource(hookName, cfg) {
             throw new Error(`"${hookName}.condition" must be a string ≤ 1024 characters`);
         }
     }
+    if (cfg.model !== undefined) {
+        if (typeof cfg.model !== "string" || cfg.model.trim() === "") {
+            throw new Error(`"${hookName}.model" must be a non-empty string`);
+        }
+    }
+    if (cfg.effort !== undefined) {
+        if (!["low", "medium", "high", "max"].includes(cfg.effort)) {
+            throw new Error(`"${hookName}.effort" must be one of "low", "medium", "high", "max"`);
+        }
+    }
 }
 /** Load and validate a JSON automation policy file. Throws on any failure. */
 export function loadPolicy(filePath) {
@@ -125,6 +139,31 @@ export function loadPolicy(filePath) {
         throw new Error(`Automation policy must be a JSON object in "${filePath}"`);
     }
     const policy = parsed;
+    // Validate top-level fields
+    if (policy.defaultModel !== undefined &&
+        typeof policy.defaultModel !== "string") {
+        throw new Error(`"defaultModel" must be a string`);
+    }
+    if (policy.maxTasksPerHour !== undefined) {
+        if (typeof policy.maxTasksPerHour !== "number" ||
+            !Number.isInteger(policy.maxTasksPerHour) ||
+            policy.maxTasksPerHour < 0) {
+            throw new Error(`"maxTasksPerHour" must be a non-negative integer`);
+        }
+    }
+    if (policy.automationSystemPrompt !== undefined) {
+        if (typeof policy.automationSystemPrompt !== "string") {
+            throw new Error(`"automationSystemPrompt" must be a string`);
+        }
+        if (policy.automationSystemPrompt.length > 4096) {
+            throw new Error(`"automationSystemPrompt" must be ≤ 4096 characters`);
+        }
+    }
+    if (policy.defaultEffort !== undefined) {
+        if (!["low", "medium", "high", "max"].includes(policy.defaultEffort)) {
+            throw new Error(`"defaultEffort" must be one of "low", "medium", "high", "max"`);
+        }
+    }
     // Validate onDiagnosticsError
     if (policy.onDiagnosticsError !== undefined) {
         const d = policy.onDiagnosticsError;
@@ -538,6 +577,11 @@ export class AutomationHooks {
     prevDiagnosticErrors = new Map();
     /** Active task ID for the task-success handler (workspace-global). */
     activeTaskSuccessTaskId = null;
+    /**
+     * Rolling window of task enqueue timestamps for maxTasksPerHour enforcement.
+     * Entries older than 60 minutes are pruned on each enqueue.
+     */
+    taskTimestamps = [];
     constructor(policy, orchestrator, log, extensionClient, workspace) {
         this.policy = policy;
         this.orchestrator = orchestrator;
@@ -545,6 +589,43 @@ export class AutomationHooks {
         this.extensionClient = extensionClient;
         this.workspace = workspace;
     }
+    /**
+     * Central enqueue for all automation-triggered tasks.
+     * Applies defaultModel (Haiku by default) and enforces maxTasksPerHour.
+     * Throws with the same "Task queue is full" message on rate-limit breach so
+     * callers can handle it identically.
+     */
+    _enqueueAutomationTask(opts) {
+        const maxPerHour = this.policy.maxTasksPerHour ?? 20;
+        if (maxPerHour > 0) {
+            const now = Date.now();
+            const cutoff = now - 60 * 60 * 1_000;
+            // Prune old timestamps
+            let i = 0;
+            while (i < this.taskTimestamps.length && this.taskTimestamps[i] < cutoff)
+                i++;
+            if (i > 0)
+                this.taskTimestamps.splice(0, i);
+            if (this.taskTimestamps.length >= maxPerHour) {
+                throw new Error(`Automation rate limit reached (max ${maxPerHour} tasks/hour)`);
+            }
+            this.taskTimestamps.push(now);
+        }
+        const model = opts.hookCfg?.model ??
+            this.policy.defaultModel ??
+            "claude-haiku-4-5-20251001";
+        const effort = opts.hookCfg?.effort ?? this.policy.defaultEffort ?? "low";
+        const systemPrompt = this.policy.automationSystemPrompt ?? DEFAULT_AUTOMATION_SYSTEM_PROMPT;
+        return this.orchestrator.enqueue({
+            prompt: opts.prompt,
+            sessionId: "",
+            isAutomationTask: true,
+            triggerSource: opts.triggerSource,
+            model,
+            effort,
+            systemPrompt,
+        });
+    }
     /**
      * Resolve a named prompt, substituting any `{{placeholder}}` tokens in
      * `promptArgs` values with sanitized event data before calling `getPrompt()`.
@@ -694,10 +775,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onDiagnosticsError", normalizedFile) + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onDiagnosticsError",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeDiagnosticsTasks.set(normalizedFile, taskId);
@@ -752,10 +833,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onCwdChanged") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onCwdChanged",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.log(`[automation] triggered cwd-changed task ${taskId.slice(0, 8)} for ${newCwd}`);
@@ -792,10 +873,10 @@ export class AutomationHooks {
         }
         postCompactPrompt = truncatePrompt(buildHookMetadata("onPostCompact") + postCompactPrompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt: postCompactPrompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onPostCompact",
+                hookCfg: cfg,
             });
             // Set lastTrigger AFTER successful enqueue so a failed enqueue does not
             // impose a spurious cooldown on the next trigger attempt.
@@ -835,10 +916,10 @@ export class AutomationHooks {
         }
         instrPrompt = truncatePrompt(buildHookMetadata("onInstructionsLoaded") + instrPrompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt: instrPrompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onInstructionsLoaded",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.log(`[automation] triggered InstructionsLoaded task ${taskId.slice(0, 8)}`);
@@ -904,10 +985,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onFileSave", normalizedFile) + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onFileSave",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeSaveTasks.set(normalizedFile, taskId);
@@ -977,10 +1058,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onFileChanged", normalizedFile) + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onFileChanged",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeFileChangedTasks.set(normalizedFile, taskId);
@@ -1069,10 +1150,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onTestRun") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onTestRun",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeTestRunTaskId = taskId;
@@ -1168,10 +1249,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onGitCommit") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onGitCommit",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeGitCommitTaskId = taskId;
@@ -1230,10 +1311,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onGitPush") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onGitPush",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeGitPushTaskId = taskId;
@@ -1289,10 +1370,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onGitPull") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onGitPull",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeGitPullTaskId = taskId;
@@ -1354,10 +1435,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onBranchCheckout") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onBranchCheckout",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeBranchCheckoutTaskId = taskId;
@@ -1421,10 +1502,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onPullRequest") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onPullRequest",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activePullRequestTaskId = taskId;
@@ -1474,7 +1555,11 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onTaskCreated") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({ prompt, sessionId: "" });
+            const taskId = this._enqueueAutomationTask({
+                prompt,
+                triggerSource: "onTaskCreated",
+                hookCfg: cfg,
+            });
             this.lastTrigger.set(key, now);
             this.activeTaskCreatedTaskId = taskId;
             this.log(`[automation] triggered task-created task ${taskId.slice(0, 8)} (spawned task: ${result.taskId.slice(0, 8)})`);
@@ -1524,7 +1609,11 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onPermissionDenied") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({ prompt, sessionId: "" });
+            const taskId = this._enqueueAutomationTask({
+                prompt,
+                triggerSource: "onPermissionDenied",
+                hookCfg: cfg,
+            });
             this.lastTrigger.set(key, now);
             this.activePermissionDeniedTaskId = taskId;
             this.log(`[automation] triggered permission-denied task ${taskId.slice(0, 8)} (tool: ${result.tool})`);
@@ -1577,7 +1666,11 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onDiagnosticsCleared", normalizedFile) + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({ prompt, sessionId: "" });
+            const taskId = this._enqueueAutomationTask({
+                prompt,
+                triggerSource: "onDiagnosticsCleared",
+                hookCfg: cfg,
+            });
             this.lastTrigger.set(key, now);
             this.activeDiagnosticsClearedTasks.set(normalizedFile, taskId);
             this.log(`[automation] triggered diagnostics-cleared task ${taskId.slice(0, 8)} for ${normalizedFile}`);
@@ -1631,10 +1724,10 @@ export class AutomationHooks {
         }
         prompt = truncatePrompt(buildHookMetadata("onTaskSuccess") + prompt);
         try {
-            const taskId = this.orchestrator.enqueue({
+            const taskId = this._enqueueAutomationTask({
                 prompt,
-                sessionId: "",
-                isAutomationTask: true,
+                triggerSource: "onTaskSuccess",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeTaskSuccessTaskId = taskId;
@@ -1750,6 +1843,11 @@ export class AutomationHooks {
                 }
                 : null,
             unwiredEnabledHooks,
+            defaultModel: p.defaultModel ?? "claude-haiku-4-5-20251001",
+            maxTasksPerHour: p.maxTasksPerHour ?? 20,
+            tasksThisHour: this.taskTimestamps.filter((t) => t >= Date.now() - 60 * 60 * 1_000).length,
+            defaultEffort: p.defaultEffort ?? "low",
+            automationSystemPrompt: (p.automationSystemPrompt ?? DEFAULT_AUTOMATION_SYSTEM_PROMPT).slice(0, 80),
         };
     }
 }