@spotpatch/agent 1.2.4 → 1.3.0

package/dist/index.js

@@ -1848,7 +1848,7 @@ var AGENT_TOOL_DEFINITIONS = Object.freeze([
   }),
   Object.freeze({
     name: AGENT_TOOL_NAMES.readFile,
-    description: "Read a bounded inclusive line range from one allowed UTF-8 text file. Choose paths returned by list_files or search_text. A retryable TOOL_PATH_DENIED result means no file was read or changed: do not retry that path; discover an allowed path instead.",
+    description: "Read a bounded inclusive line range from one allowed UTF-8 text file. Choose an exact path supplied by trusted SpotPatch context or returned by list_files or search_text. A retryable TOOL_PATH_DENIED result means no file was read or changed: do not retry that path; discover an allowed path instead.",
     parameters: Object.freeze({
       type: "object",
       properties: Object.freeze({
@@ -1902,6 +1902,11 @@ var AGENT_TOOL_DEFINITIONS = Object.freeze([
     })
   })
 ]);
+var AGENT_TOOL_DEFINITIONS_WITHOUT_CHECKS = Object.freeze(
+  AGENT_TOOL_DEFINITIONS.filter(
+    (definition) => definition.name !== AGENT_TOOL_NAMES.runCheck
+  )
+);
 
 // src/tools/tool-executor.ts
 var SEARCH_READ_CONCURRENCY = 8;
@@ -2855,6 +2860,7 @@ import {
 var MAX_PROJECT_CONVENTION_CHARACTERS = 3500;
 var MAX_VALIDATION_CHECK_CHARACTERS = 1200;
 var MINIMUM_SELECTION_CONTEXT_CHARACTERS = 1024;
+var VALIDATION_SYSTEM_INSTRUCTION = "- Run each relevant configured check after the final write so failures can be corrected. Do not rerun an unchanged check, and do not claim a check passed unless run_check returned a passed status.\n";
 var AGENT_SYSTEM_INSTRUCTIONS = `You are editing code only inside a disposable, isolated Git worktree.
 
 Follow these rules exactly:
@@ -2871,8 +2877,11 @@ Follow these rules exactly:
 - If any tool returns a retryable TOOL_ARGUMENTS_INVALID result, no file changed. Retry once with a new tool call ID using only the declared fields and value types.
 - If read_file returns a retryable TOOL_PATH_DENIED result, no file was read or changed. Do not retry that path. Use list_files or search_text and choose an allowed path returned by the tool; never probe protected, external, generated, credential, environment, or lock files.
 - Never modify credentials, environment files, lockfiles, generated output, Git metadata, or dependencies.
-- Run each relevant configured check after the final write so failures can be corrected. Do not rerun an unchanged check, and do not claim a check passed unless run_check returned a passed status.
+${VALIDATION_SYSTEM_INSTRUCTION.trimEnd()}
 - Finish with a concise factual summary after all needed tool calls. Do not include secrets or absolute paths.`;
+function resolveAgentSystemInstructions(trustedFast) {
+  return trustedFast ? AGENT_SYSTEM_INSTRUCTIONS.replace(VALIDATION_SYSTEM_INSTRUCTION, "") : AGENT_SYSTEM_INSTRUCTIONS;
+}
 function redactedJson(value) {
   return JSON.stringify(
     value,
@@ -3064,7 +3073,8 @@ function composeAgentUserPrompt(annotation, maximumCharacters, context = {}) {
     (target, index) => `Target ${String(index + 1)}:
 ${redactSensitiveText2(target.instruction.trim())}`
   ).join("\n\n");
-  const requestBlock = `${requestPrefix}${request}`;
+  const trustedFastBlock = context.trustedFast ? "\n\nTrusted direct execution is enabled. Start from each target's supplied code.relativePath or source.relativePath. When an exact path is present, do not call list_files first. Read each affected file once unless a write is rejected, make the smallest exact replacement that satisfies the request, and finish immediately after the successful write. No project validation check is available in this mode." : "";
+  const requestBlock = `${requestPrefix}${request}${trustedFastBlock}`;
   const checksPrefix = "\n\nConfigured validation checks (IDs and labels only):\n<validation_checks>\n";
   const checksSuffix = "\n</validation_checks>";
   if (requestBlock.length + contextPrefix.length + suffix.length + MINIMUM_SELECTION_CONTEXT_CHARACTERS > maximumCharacters) {
@@ -3167,6 +3177,8 @@ async function executeToolCalls(calls, turn, executor, callbacks, signal) {
   return Object.freeze(results);
 }
 async function executeAgentChange(options) {
+  const trustedFast = options.execution.applyMode === "trusted-auto";
+  const activeChecks = trustedFast ? Object.freeze({}) : options.execution.checks;
   const controller = new AbortController();
   const unlink = linkSignal(options.signal, controller);
   let jobTimedOut = false;
@@ -3198,7 +3210,7 @@ async function executeAgentChange(options) {
       })
     );
     const executor = createAgentToolExecutor({
-      checks: options.execution.checks,
+      checks: activeChecks,
       limits: options.execution.limits,
       worktreeRoot: worktree.root,
       onCheck(result2) {
@@ -3214,16 +3226,17 @@ async function executeAgentChange(options) {
       provider: options.provider,
       model: options.model,
       credential: options.credential,
-      instructions: AGENT_SYSTEM_INSTRUCTIONS,
+      instructions: resolveAgentSystemInstructions(trustedFast),
       userPrompt: composeAgentUserPrompt(
         options.annotation,
         options.promptMaxCharacters ?? 16e3,
         Object.freeze({
-          checks: options.execution.checks,
-          projectConventions
+          checks: activeChecks,
+          projectConventions,
+          trustedFast
         })
       ),
-      tools: AGENT_TOOL_DEFINITIONS,
+      tools: trustedFast ? AGENT_TOOL_DEFINITIONS_WITHOUT_CHECKS : AGENT_TOOL_DEFINITIONS,
       limits: options.execution.limits,
       ...options.fetch === void 0 ? {} : { fetch: options.fetch }
     });
@@ -3260,7 +3273,7 @@ async function executeAgentChange(options) {
     options.callbacks?.onPhase?.(
       Object.freeze({
         phase: "validating",
-        message: "Validating proposed changes."
+        message: trustedFast ? "Preparing trusted change for direct apply." : "Validating proposed changes."
       })
     );
     const initialChangeSet = await collectAgentChangeSet(
@@ -3269,7 +3282,7 @@ async function executeAgentChange(options) {
       options.execution.limits,
       controller.signal
     );
-    const requiredChecks = initialChangeSet.diff.length === 0 ? [] : Object.values(options.execution.checks).filter((check) => check.required);
+    const requiredChecks = initialChangeSet.diff.length === 0 ? [] : Object.values(activeChecks).filter((check) => check.required);
     const finalChecks = [];
     let ranFinalCheck = false;
     for (const check of requiredChecks) {