claude-ide-bridge 2.25.11 → 2.25.16

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;
@@ -249,6 +253,19 @@ export interface AutomationPolicy {
      * 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. */
@@ -341,6 +358,10 @@ export declare class AutomationHooks {
     private prevDiagnosticErrors;
     /** Active task ID for the task-success handler (workspace-global). */
     private activeTaskSuccessTaskId;
+    /** Active task ID for the post-compact handler (workspace-global). */
+    private activePostCompactTaskId;
+    /** Active task ID for the instructions-loaded handler (workspace-global). */
+    private activeInstructionsLoadedTaskId;
     /**
      * Rolling window of task enqueue timestamps for maxTasksPerHour enforcement.
      * Entries older than 60 minutes are pruned on each enqueue.
@@ -498,5 +519,7 @@ export declare class AutomationHooks {
         defaultModel: string;
         maxTasksPerHour: number;
         tasksThisHour: number;
+        defaultEffort: string;
+        automationSystemPrompt: string;
     };
 }

package/dist/automation.js

@@ -5,6 +5,7 @@ import { minimatch } from "minimatch";
 import { getPrompt } from "./prompts.js";
 /** Maximum length (chars) of a single diagnostic message before truncation */
 const MAX_DIAGNOSTIC_MSG_CHARS = 500;
+const MAX_DIAGNOSTICS_IN_PROMPT = 20;
 /**
  * Wrap an untrusted user-controlled value in delimiters that include a
  * per-trigger nonce so a crafted value cannot forge a closing delimiter.
@@ -50,6 +51,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 +109,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) {
@@ -137,6 +152,19 @@ export function loadPolicy(filePath) {
             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;
@@ -550,6 +578,10 @@ export class AutomationHooks {
     prevDiagnosticErrors = new Map();
     /** Active task ID for the task-success handler (workspace-global). */
     activeTaskSuccessTaskId = null;
+    /** Active task ID for the post-compact handler (workspace-global). */
+    activePostCompactTaskId = null;
+    /** Active task ID for the instructions-loaded handler (workspace-global). */
+    activeInstructionsLoadedTaskId = null;
     /**
      * Rolling window of task enqueue timestamps for maxTasksPerHour enforcement.
      * Entries older than 60 minutes are pruned on each enqueue.
@@ -584,13 +616,19 @@ export class AutomationHooks {
             }
             this.taskTimestamps.push(now);
         }
-        const model = this.policy.defaultModel ?? "claude-haiku-4-5-20251001";
+        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,
         });
     }
     /**
@@ -731,9 +769,12 @@ export class AutomationHooks {
             prompt = resolved;
         }
         else {
-            const diagnosticsText = matching
+            const displayMatching = matching.slice(0, MAX_DIAGNOSTICS_IN_PROMPT);
+            const omittedCount = matching.length - displayMatching.length;
+            const diagnosticsText = displayMatching
                 .map((d) => `[${d.severity}] ${d.message.slice(0, MAX_DIAGNOSTIC_MSG_CHARS)}`)
-                .join("\n");
+                .join("\n") +
+                (omittedCount > 0 ? `\n… and ${omittedCount} more` : "");
             const nonce = crypto.randomBytes(6).toString("hex");
             prompt =
                 (cfg.prompt ?? "")
@@ -745,6 +786,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onDiagnosticsError",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeDiagnosticsTasks.set(normalizedFile, taskId);
@@ -802,6 +844,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onCwdChanged",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.log(`[automation] triggered cwd-changed task ${taskId.slice(0, 8)} for ${newCwd}`);
@@ -818,6 +861,15 @@ export class AutomationHooks {
         const cfg = this.policy.onPostCompact;
         if (!cfg?.enabled)
             return;
+        if (this.activePostCompactTaskId) {
+            const existing = this.orchestrator.getTask(this.activePostCompactTaskId);
+            if (existing &&
+                (existing.status === "pending" || existing.status === "running")) {
+                this.log(`[automation] skipping post-compact trigger — task ${this.activePostCompactTaskId.slice(0, 8)} still active`);
+                return;
+            }
+            this.activePostCompactTaskId = null;
+        }
         const key = "post-compact";
         const now = Date.now();
         const last = this.lastTrigger.get(key) ?? 0;
@@ -841,10 +893,12 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt: postCompactPrompt,
                 triggerSource: "onPostCompact",
+                hookCfg: cfg,
             });
             // Set lastTrigger AFTER successful enqueue so a failed enqueue does not
             // impose a spurious cooldown on the next trigger attempt.
             this.lastTrigger.set(key, now);
+            this.activePostCompactTaskId = taskId;
             this.log(`[automation] triggered PostCompact task ${taskId.slice(0, 8)}`);
         }
         catch (err) {
@@ -859,6 +913,15 @@ export class AutomationHooks {
         const cfg = this.policy.onInstructionsLoaded;
         if (!cfg?.enabled)
             return;
+        if (this.activeInstructionsLoadedTaskId) {
+            const existing = this.orchestrator.getTask(this.activeInstructionsLoadedTaskId);
+            if (existing &&
+                (existing.status === "pending" || existing.status === "running")) {
+                this.log(`[automation] skipping instructions-loaded trigger — task ${this.activeInstructionsLoadedTaskId.slice(0, 8)} still active`);
+                return;
+            }
+            this.activeInstructionsLoadedTaskId = null;
+        }
         const cooldownMs = cfg.cooldownMs ?? 60_000;
         const key = "instructions-loaded";
         const now = Date.now();
@@ -883,8 +946,10 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt: instrPrompt,
                 triggerSource: "onInstructionsLoaded",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
+            this.activeInstructionsLoadedTaskId = taskId;
             this.log(`[automation] triggered InstructionsLoaded task ${taskId.slice(0, 8)}`);
         }
         catch (err) {
@@ -951,6 +1016,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onFileSave",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeSaveTasks.set(normalizedFile, taskId);
@@ -1023,6 +1089,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onFileChanged",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeFileChangedTasks.set(normalizedFile, taskId);
@@ -1114,6 +1181,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onTestRun",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeTestRunTaskId = taskId;
@@ -1201,7 +1269,7 @@ export class AutomationHooks {
                     .replace(/\{\{hash\}\}/g, untrustedBlock("COMMIT HASH", safeHash, nonce))
                     .replace(/\{\{branch\}\}/g, untrustedBlock("BRANCH", safeBranchCommit, nonce))
                     .replace(/\{\{message\}\}/g, untrustedBlock("COMMIT MESSAGE", safeMessage, nonce))
-                    .replace(/\{\{count\}\}/g, String(result.count))
+                    .replace(/\{\{count\}\}/g, untrustedBlock("COMMIT COUNT", String(result.count), nonce))
                     .replace(/\{\{files\}\}/g, untrustedBlock("COMMITTED FILES", filesText, nonce))
                     .replace(/\{\{changeImpact\}\}/g, changeImpact
                     ? untrustedBlock("CHANGE IMPACT", changeImpact, nonce)
@@ -1212,6 +1280,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onGitCommit",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeGitCommitTaskId = taskId;
@@ -1273,6 +1342,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onGitPush",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeGitPushTaskId = taskId;
@@ -1331,6 +1401,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onGitPull",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeGitPullTaskId = taskId;
@@ -1395,6 +1466,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onBranchCheckout",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeBranchCheckoutTaskId = taskId;
@@ -1461,6 +1533,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onPullRequest",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activePullRequestTaskId = taskId;
@@ -1513,6 +1586,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onTaskCreated",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeTaskCreatedTaskId = taskId;
@@ -1566,6 +1640,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onPermissionDenied",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activePermissionDeniedTaskId = taskId;
@@ -1622,6 +1697,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onDiagnosticsCleared",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeDiagnosticsClearedTasks.set(normalizedFile, taskId);
@@ -1679,6 +1755,7 @@ export class AutomationHooks {
             const taskId = this._enqueueAutomationTask({
                 prompt,
                 triggerSource: "onTaskSuccess",
+                hookCfg: cfg,
             });
             this.lastTrigger.set(key, now);
             this.activeTaskSuccessTaskId = taskId;
@@ -1797,6 +1874,8 @@ export class AutomationHooks {
             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),
         };
     }
 }