@pellux/goodvibes-sdk 0.30.3 → 0.33.0

package/dist/platform/tools/exec/file-ops.d.ts

@@ -2,14 +2,16 @@ import type { ExecFileOp } from './schema.js';
 export interface FileOpResult {
     op: string;
     source: string;
-    destination?: string;
-    dry_run?: boolean;
-    would_delete?: string[];
-    updated_imports?: string[];
+    destination?: string | undefined;
+    dry_run?: boolean | undefined;
+    would_delete?: string[] | undefined;
+    updated_imports?: string[] | undefined;
+    warnings?: string[] | undefined;
 }
 export declare function executeFileOp(op: ExecFileOp, projectRoot: string): FileOpResult;
 export declare function executeFileOperations(fileOps: ExecFileOp[] | undefined, projectRoot: string): Promise<{
     fileOpResults: FileOpResult[];
     fileOpError?: string;
+    fileOpWarnings?: string[];
 }>;
 //# sourceMappingURL=file-ops.d.ts.map

package/dist/platform/tools/exec/file-ops.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"file-ops.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/exec/file-ops.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAG9C,MAAM,WAAW,YAAY;IAC3B,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,eAAe,CAAC,EAAE,MAAM,EAAE,CAAC;CAC5B;AA+GD,wBAAgB,aAAa,CAAC,EAAE,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,GAAG,YAAY,CAoD/E;AAED,wBAAsB,qBAAqB,CACzC,OAAO,EAAE,UAAU,EAAE,GAAG,SAAS,EACjC,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC;IAAE,aAAa,EAAE,YAAY,EAAE,CAAC;IAAC,WAAW,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAoClE"}
+{"version":3,"file":"file-ops.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/exec/file-ops.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AAG9C,MAAM,WAAW,YAAY;IAC3B,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACjC,OAAO,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAC9B,YAAY,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAC;IACpC,eAAe,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAC;IACvC,QAAQ,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAC;CACjC;AA6HD,wBAAgB,aAAa,CAAC,EAAE,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,GAAG,YAAY,CA4D/E;AAED,wBAAsB,qBAAqB,CACzC,OAAO,EAAE,UAAU,EAAE,GAAG,SAAS,EACjC,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC;IAAE,aAAa,EAAE,YAAY,EAAE,CAAC;IAAC,WAAW,CAAC,EAAE,MAAM,CAAC;IAAC,cAAc,CAAC,EAAE,MAAM,EAAE,CAAA;CAAE,CAAC,CAwD7F"}

package/dist/platform/tools/exec/file-ops.js

@@ -3,25 +3,29 @@ import { resolve, isAbsolute, join, relative, dirname } from 'node:path';
 import { logger } from '../../utils/logger.js';
 import { resolveAndValidatePath } from '../../utils/path-safety.js';
 import { summarizeError } from '../../utils/error-display.js';
+function appendFileOpWarning(result, warning) {
+    result.warnings = [...(result.warnings ?? []), warning];
+}
 function resolveFileOpPath(p, op, projectRoot) {
     if (op === 'delete' || !isAbsolute(p)) {
         return resolveAndValidatePath(p, projectRoot);
     }
     return resolve(p);
 }
-function collectPaths(p, acc = []) {
+function collectPaths(p, acc = [], warnings = []) {
     try {
         const st = statSync(p);
         if (st.isDirectory()) {
             for (const entry of readdirSync(p)) {
-                collectPaths(join(p, entry), acc);
+                collectPaths(join(p, entry), acc, warnings);
             }
         }
         else {
             acc.push(p);
         }
     }
-    catch {
+    catch (err) {
+        warnings.push(`Could not inspect '${p}' while collecting delete preview: ${summarizeError(err)}`);
         acc.push(p);
     }
     return acc;
@@ -40,13 +44,15 @@ async function updateImportsAfterMove(oldSrc, newDst, projectRoot) {
     const TS_EXTS = new Set(['.ts', '.tsx', '.js', '.jsx', '.mjs', '.cjs']);
     const SKIP_DIRS = new Set(['.git', 'node_modules', 'dist', '.next', '.nuxt', '.cache', '__pycache__']);
     const allFiles = [];
+    const warnings = [];
     function walkDir(dir) {
         let entries;
         try {
             entries = readdirSync(dir);
         }
         catch (error) {
-            logger.debug('file move import update: failed to read directory', {
+            warnings.push(`Import update could not read directory '${dir}': ${summarizeError(error)}`);
+            logger.warn('file move import update: failed to read directory', {
                 dir,
                 error: summarizeError(error),
             });
@@ -67,7 +73,8 @@ async function updateImportsAfterMove(oldSrc, newDst, projectRoot) {
                     allFiles.push(full);
             }
             catch (error) {
-                logger.debug('file move import update: failed to stat path', {
+                warnings.push(`Import update could not stat '${full}': ${summarizeError(error)}`);
+                logger.warn('file move import update: failed to stat path', {
                     path: full,
                     error: summarizeError(error),
                 });
@@ -84,7 +91,8 @@ async function updateImportsAfterMove(oldSrc, newDst, projectRoot) {
             content = readFileSync(file, 'utf-8');
         }
         catch (error) {
-            logger.debug('file move import update: failed to read file', {
+            warnings.push(`Import update could not read '${file}': ${summarizeError(error)}`);
+            logger.warn('file move import update: failed to read file', {
                 file,
                 error: summarizeError(error),
             });
@@ -106,22 +114,27 @@ async function updateImportsAfterMove(oldSrc, newDst, projectRoot) {
                 updated.push(file);
             }
             catch (err) {
-                logger.debug('exec file_ops update_imports: write failed (non-fatal)', {
+                warnings.push(`Import update could not write '${file}': ${summarizeError(err)}`);
+                logger.warn('exec file_ops update_imports: write failed', {
                     file,
                     error: summarizeError(err),
                 });
             }
         }
     }
-    return updated;
+    return { updated, warnings };
 }
 export function executeFileOp(op, projectRoot) {
     const src = resolveFileOpPath(op.source, op.op, projectRoot);
     const result = { op: op.op, source: src };
     if (op.op === 'delete') {
         if (op.dry_run) {
+            const warnings = [];
             result.dry_run = true;
-            result.would_delete = collectPaths(src);
+            result.would_delete = collectPaths(src, [], warnings);
+            if (warnings.length > 0) {
+                result.warnings = warnings;
+            }
             return result;
         }
         if (op.recursive) {
@@ -153,7 +166,8 @@ export function executeFileOp(op, projectRoot) {
         try {
             renameSync(src, dst);
         }
-        catch {
+        catch (err) {
+            appendFileOpWarning(result, `file_ops move used copy/delete fallback after rename failed from '${src}' to '${dst}': ${summarizeError(err)}`);
             if (op.recursive) {
                 cpSync(src, dst, { recursive: true });
                 rmSync(src, { recursive: true, force: true });
@@ -169,6 +183,7 @@ export function executeFileOp(op, projectRoot) {
 export async function executeFileOperations(fileOps, projectRoot) {
     const fileOpResults = [];
     const pendingImportUpdates = [];
+    const fileOpWarnings = [];
     if (!fileOps || fileOps.length === 0) {
         return { fileOpResults };
     }
@@ -176,29 +191,47 @@ export async function executeFileOperations(fileOps, projectRoot) {
         try {
             const opResult = executeFileOp(op, projectRoot);
             fileOpResults.push(opResult);
+            fileOpWarnings.push(...(opResult.warnings ?? []));
             if (op.op === 'move' && op.update_imports && opResult.destination) {
                 pendingImportUpdates.push({ src: opResult.source, dst: opResult.destination });
             }
         }
         catch (err) {
             const msg = summarizeError(err);
-            return { fileOpResults, fileOpError: `file_ops failed: ${msg}` };
+            return {
+                fileOpResults,
+                fileOpError: `file_ops failed: ${msg}`,
+                ...(fileOpWarnings.length > 0 ? { fileOpWarnings } : {}),
+            };
         }
     }
     for (const { src, dst } of pendingImportUpdates) {
+        const matchingResult = fileOpResults.find((r) => r.source === src && r.destination === dst);
         try {
-            const updated = await updateImportsAfterMove(src, dst, projectRoot);
-            const matchingResult = fileOpResults.find((r) => r.source === src && r.destination === dst);
-            if (matchingResult)
-                matchingResult.updated_imports = updated;
+            const updateResult = await updateImportsAfterMove(src, dst, projectRoot);
+            if (matchingResult) {
+                matchingResult.updated_imports = updateResult.updated;
+                for (const warning of updateResult.warnings) {
+                    appendFileOpWarning(matchingResult, warning);
+                }
+            }
+            fileOpWarnings.push(...updateResult.warnings);
         }
         catch (err) {
-            logger.debug('exec file_ops: update_imports failed (non-fatal)', {
+            const warning = `file_ops update_imports failed after moving '${src}' to '${dst}': ${summarizeError(err)}`;
+            if (matchingResult) {
+                appendFileOpWarning(matchingResult, warning);
+            }
+            fileOpWarnings.push(warning);
+            logger.warn('exec file_ops: update_imports failed', {
                 src,
                 dst,
                 error: summarizeError(err),
             });
         }
     }
-    return { fileOpResults };
+    return {
+        fileOpResults,
+        ...(fileOpWarnings.length > 0 ? { fileOpWarnings } : {}),
+    };
 }

package/dist/platform/tools/exec/runtime.d.ts

@@ -16,7 +16,7 @@ import type { FeatureFlagManager } from '../../runtime/feature-flags/index.js';
  */
 export declare function isRetryableExecResult(result: ExecCommandResult, allowed?: ReadonlyArray<'network' | 'lock' | 'busy' | 'oom'>): boolean;
 export declare function createExecTool(processManager: ProcessManager, options?: {
-    readonly featureFlags?: Pick<FeatureFlagManager, 'isEnabled'> | null;
-    readonly overflowHandler?: OverflowHandler;
+    readonly featureFlags?: Pick<FeatureFlagManager, 'isEnabled'> | null | undefined;
+    readonly overflowHandler?: OverflowHandler | undefined;
 }): Tool;
 //# sourceMappingURL=runtime.d.ts.map

package/dist/platform/tools/exec/runtime.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"runtime.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/exec/runtime.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,sBAAsB,CAAC;AAGjD,OAAO,EAAqB,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAC3E,OAAO,KAAK,EAA+B,iBAAiB,EAAiB,MAAM,aAAa,CAAC;AACjG,OAAO,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAG9D,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,sCAAsC,CAAC;AAsc/E;;;;;;;;;;GAUG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,iBAAiB,EACzB,OAAO,CAAC,EAAE,aAAa,CAAC,SAAS,GAAG,MAAM,GAAG,MAAM,GAAG,KAAK,CAAC,GAC3D,OAAO,CA2CT;AA4ID,wBAAgB,cAAc,CAC5B,cAAc,EAAE,cAAc,EAC9B,OAAO,GAAE;IACP,QAAQ,CAAC,YAAY,CAAC,EAAE,IAAI,CAAC,kBAAkB,EAAE,WAAW,CAAC,GAAG,IAAI,CAAC;IACrE,QAAQ,CAAC,eAAe,CAAC,EAAE,eAAe,CAAC;CACvC,GACL,IAAI,CAwEN"}
+{"version":3,"file":"runtime.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/exec/runtime.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,sBAAsB,CAAC;AAGjD,OAAO,EAAqB,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAC3E,OAAO,KAAK,EAA+B,iBAAiB,EAAiB,MAAM,aAAa,CAAC;AACjG,OAAO,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAG9D,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,sCAAsC,CAAC;AAkd/E;;;;;;;;;;GAUG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,iBAAiB,EACzB,OAAO,CAAC,EAAE,aAAa,CAAC,SAAS,GAAG,MAAM,GAAG,MAAM,GAAG,KAAK,CAAC,GAC3D,OAAO,CA2CT;AA8ID,wBAAgB,cAAc,CAC5B,cAAc,EAAE,cAAc,EAC9B,OAAO,GAAE;IACP,QAAQ,CAAC,YAAY,CAAC,EAAE,IAAI,CAAC,kBAAkB,EAAE,WAAW,CAAC,GAAG,IAAI,GAAG,SAAS,CAAC;IACjF,QAAQ,CAAC,eAAe,CAAC,EAAE,eAAe,GAAG,SAAS,CAAC;CACnD,GACL,IAAI,CAmFN"}

package/dist/platform/tools/exec/runtime.js

@@ -1,5 +1,5 @@
 import { join, resolve, isAbsolute } from 'node:path';
-import { randomUUID } from 'node:crypto';
+import { randomInt, randomUUID } from 'node:crypto';
 import { logger } from '../../utils/logger.js';
 import { EXEC_TOOL_SCHEMA } from './schema.js';
 import { DEFAULT_MAX_CHARS, OverflowHandler } from '../shared/overflow.js';
@@ -8,6 +8,7 @@ import { guardExecCommand, formatDenialResponse } from './ast-guard.js';
 import { executeFileOperations } from './file-ops.js';
 import { DEFAULT_ALLOWED_CLASSES } from '../../runtime/permissions/normalization/verdict.js';
 import { mapWithConcurrency } from '../../utils/concurrency.js';
+import { compileSafeRegExp, safeRegExpTest } from '../../utils/safe-regex.js';
 const DEFAULT_TIMEOUT_MS = 120_000;
 const PROGRESS_AUTO_THRESHOLD_MS = 30_000;
 const OVERFLOW_SUBDIR = ['.goodvibes', '.overflow'];
@@ -64,7 +65,7 @@ function computeRetryDelay(attempt, delayMs, backoff, maxDelayMs = 30_000) {
         return delayMs;
     // Full jitter: random in [0, min(base * 2^attempt, maxDelay)] — avoids thundering herd
     const cap = Math.min(delayMs * Math.pow(2, attempt), maxDelayMs);
-    return Math.random() * cap;
+    return randomInt(0, Math.max(1, Math.floor(cap) + 1));
 }
 function buildCleanEnv() {
     return Object.fromEntries(Object.entries(process.env).filter(([, v]) => v !== undefined));
@@ -100,7 +101,7 @@ function initProgressFile(cmdStr, workingDirectory) {
         mkdirSync(progressDirectory, { recursive: true });
     }
     catch (err) {
-        logger.debug('initProgressFile: mkdirSync failed (dir may already exist)', { error: summarizeError(err) });
+        logger.warn('initProgressFile: mkdirSync failed', { error: summarizeError(err) });
     }
     const id = `${Date.now()}-${randomUUID().slice(0, 8)}`;
     const filePath = getProgressFilePath(workingDirectory, id);
@@ -112,7 +113,7 @@ function initProgressFile(cmdStr, workingDirectory) {
                 appendFileSync(filePath, chunk);
             }
             catch (err) {
-                logger.debug('initProgressFile: appendFileSync failed', { path: filePath, error: summarizeError(err) });
+                logger.warn('initProgressFile: appendFileSync failed', { path: filePath, error: summarizeError(err) });
             }
         },
     };
@@ -151,7 +152,7 @@ async function runCommand(processManager, overflowHandler, featureFlags, cmdStr,
     if (useProgress) {
         return runCommandWithProgress(processManager, overflowHandler, cmdStr, cmdInput, workingDirectory, cwd, mergedEnv, timeoutMs, startTime);
     }
-    const proc = Bun.spawn(['/bin/sh', '-c', cmdStr], { cwd, env: mergedEnv, stdout: 'pipe', stderr: 'pipe' });
+    const proc = Bun.spawn(['/bin/sh', '-c', cmdStr], { ...(cwd !== undefined ? { cwd } : {}), env: mergedEnv, stdout: 'pipe', stderr: 'pipe' });
     let timedOut = false;
     let killTimer;
     let timeoutResolve;
@@ -164,7 +165,7 @@ async function runCommand(processManager, overflowHandler, featureFlags, cmdStr,
             proc.kill('SIGKILL');
         }
         catch (err) {
-            // OBS-11: Non-fatal — process may have already exited before kill
+            // The process may have already exited before kill.
             logger.debug('[ExecRuntime] kill on timeout failed (process may have exited)', { error: String(err) });
         }
         timeoutResolve();
@@ -219,7 +220,7 @@ async function runCommand(processManager, overflowHandler, featureFlags, cmdStr,
 }
 async function runCommandWithProgress(_processManager, overflowHandler, cmdStr, cmdInput, workingDirectory, cwd, mergedEnv, timeoutMs, startTime) {
     const progressFile = initProgressFile(cmdStr, workingDirectory);
-    const proc = Bun.spawn(['/bin/sh', '-c', cmdStr], { cwd, env: mergedEnv, stdout: 'pipe', stderr: 'pipe' });
+    const proc = Bun.spawn(['/bin/sh', '-c', cmdStr], { ...(cwd !== undefined ? { cwd } : {}), env: mergedEnv, stdout: 'pipe', stderr: 'pipe' });
     let timedOut = false;
     let killTimer;
     let timeoutResolve;
@@ -240,6 +241,7 @@ async function runCommandWithProgress(_processManager, overflowHandler, cmdStr,
     killTimer.unref?.();
     let stdoutBuf = '';
     let stderrBuf = '';
+    const warnings = [];
     const readStdout = async () => {
         const decoder = new TextDecoder();
         const reader = proc.stdout.getReader();
@@ -254,7 +256,9 @@ async function runCommandWithProgress(_processManager, overflowHandler, cmdStr,
             }
         }
         catch (err) {
-            logger.debug('[ExecRuntime] stdout stream read ended with error', { error: String(err) });
+            const warning = `stdout stream read failed: ${summarizeError(err)}`;
+            warnings.push(warning);
+            logger.warn('[ExecRuntime] stdout stream read ended with error', { error: summarizeError(err) });
         }
         finally {
             reader.releaseLock();
@@ -273,7 +277,9 @@ async function runCommandWithProgress(_processManager, overflowHandler, cmdStr,
             }
         }
         catch (err) {
-            logger.debug('[ExecRuntime] stderr stream read ended with error', { error: String(err) });
+            const warning = `stderr stream read failed: ${summarizeError(err)}`;
+            warnings.push(warning);
+            logger.warn('[ExecRuntime] stderr stream read ended with error', { error: summarizeError(err) });
         }
         finally {
             reader.releaseLock();
@@ -295,10 +301,11 @@ async function runCommandWithProgress(_processManager, overflowHandler, cmdStr,
         return { ...buildTimedOutResult(cmdStr, cwd, Date.now() - startTime, progressFile.path), stdout: stdoutBuf, stderr: stderrBuf };
     }
     const ioResult = await ioPromise.catch((error) => {
-        logger.debug('exec progress command IO collection failed', {
+        logger.warn('exec progress command IO collection failed', {
             command: cmdStr,
             error: summarizeError(error),
         });
+        warnings.push(`progress command IO collection failed: ${summarizeError(error)}`);
         return [undefined, undefined, undefined];
     });
     const actualExitCode = ioResult[2] ?? await proc.exited;
@@ -311,11 +318,12 @@ async function runCommandWithProgress(_processManager, overflowHandler, cmdStr,
         exit_code: actualExitCode,
         stdout: stdoutResult.text,
         stderr: stderrResult.text,
-        success: actualExitCode === 0,
+        success: actualExitCode === 0 && warnings.length === 0,
         duration_ms: duration,
         cwd,
         env: cmdInput.env,
         progress_file: progressFile.path,
+        ...(warnings.length > 0 && { warnings }),
         ...(stdoutResult.truncated && { stdout_truncated: true }),
         ...(stderrResult.truncated && { stderr_truncated: true }),
     };
@@ -323,13 +331,14 @@ async function runCommandWithProgress(_processManager, overflowHandler, cmdStr,
 }
 async function runUntil(_processManager, overflowHandler, cmdStr, cmdInput, cwd, env, timeoutMs, startTime) {
     const until = cmdInput.until;
-    const pattern = new RegExp(until.pattern);
+    const pattern = compileSafeRegExp(until.pattern, '', { operation: 'exec until pattern' });
     const untilTimeout = until.timeout_ms ?? timeoutMs;
     const killAfter = until.kill_after ?? false;
-    const proc = Bun.spawn(['/bin/sh', '-c', cmdStr], { cwd, env, stdout: 'pipe', stderr: 'pipe' });
+    const proc = Bun.spawn(['/bin/sh', '-c', cmdStr], { ...(cwd !== undefined ? { cwd } : {}), env, stdout: 'pipe', stderr: 'pipe' });
     let stdoutBuf = '';
     let stderrBuf = '';
     let matched = false;
+    const warnings = [];
     const readStream = async (stream, isStderr) => {
         const decoder = new TextDecoder();
         const reader = stream.getReader();
@@ -343,7 +352,7 @@ async function runUntil(_processManager, overflowHandler, cmdStr, cmdInput, cwd,
                     stderrBuf += chunk;
                 else
                     stdoutBuf += chunk;
-                if (!matched && pattern.test(stdoutBuf + stderrBuf)) {
+                if (!matched && safeRegExpTest(pattern, stdoutBuf + stderrBuf, { operation: 'exec until pattern', maxInputChars: 500_000 })) {
                     matched = true;
                     if (killAfter) {
                         killExecProcess(proc, cmdStr, 'match');
@@ -354,7 +363,9 @@ async function runUntil(_processManager, overflowHandler, cmdStr, cmdInput, cwd,
             }
         }
         catch (error) {
-            logger.debug('exec run-until stream read failed', {
+            const warning = `${isStderr ? 'stderr' : 'stdout'} stream read failed: ${summarizeError(error)}`;
+            warnings.push(warning);
+            logger.warn('exec run-until stream read failed', {
                 command: cmdStr,
                 stream: isStderr ? 'stderr' : 'stdout',
                 error: summarizeError(error),
@@ -376,9 +387,10 @@ async function runUntil(_processManager, overflowHandler, cmdStr, cmdInput, cwd,
         exit_code: exitCode,
         stdout: stdoutResult.text,
         stderr: stderrResult.text,
-        success: matched,
+        success: matched && warnings.length === 0,
         duration_ms: duration,
         cwd,
+        ...(warnings.length > 0 && { warnings }),
         ...(stdoutResult.truncated && { stdout_truncated: true }),
         ...(stderrResult.truncated && { stderr_truncated: true }),
     };
@@ -522,6 +534,7 @@ function formatResult(result, verbosity) {
                 ...(result.timed_out && { timed_out: true }),
                 ...(result.process_id && { process_id: result.process_id, pid: result.pid }),
                 ...(result.progress_file && { progress_file: result.progress_file }),
+                ...(result.warnings && { warnings: result.warnings }),
             };
         }
         case 'verbose':
@@ -541,6 +554,7 @@ function formatResult(result, verbosity) {
                 ...(result.stderr_truncated && { stderr_truncated: true }),
                 ...(result.retries !== undefined && { retries: result.retries }),
                 ...(result.progress_file && { progress_file: result.progress_file }),
+                ...(result.warnings && { warnings: result.warnings }),
             };
     }
 }
@@ -575,9 +589,14 @@ export function createExecTool(processManager, options = {}) {
                 const globalTimeout = input.timeout_ms ?? DEFAULT_TIMEOUT_MS;
                 const failFast = input.fail_fast === true || input.stop_on_error === true;
                 const projectRoot = resolve(workingDirectory);
-                const { fileOpResults, fileOpError } = await executeFileOperations(input.file_ops, projectRoot);
-                if (fileOpError)
-                    return { success: false, error: fileOpError };
+                const { fileOpResults, fileOpError, fileOpWarnings } = await executeFileOperations(input.file_ops, projectRoot);
+                if (fileOpError) {
+                    return {
+                        success: false,
+                        error: fileOpError,
+                        ...(fileOpWarnings && fileOpWarnings.length > 0 ? { warnings: fileOpWarnings } : {}),
+                    };
+                }
                 const resolvedCmds = [];
                 for (const cmdInput of input.commands) {
                     let cmdStr;
@@ -596,7 +615,13 @@ export function createExecTool(processManager, options = {}) {
                 const responseData = formatted.length === 1 ? { ...formatted[0] } : { commands: formatted, total: formatted.length };
                 if (fileOpResults.length > 0)
                     responseData.file_ops = fileOpResults;
-                return { success: allSuccess, output: JSON.stringify(responseData) };
+                if (fileOpWarnings && fileOpWarnings.length > 0)
+                    responseData.warnings = fileOpWarnings;
+                return {
+                    success: allSuccess,
+                    output: JSON.stringify(responseData),
+                    ...(fileOpWarnings && fileOpWarnings.length > 0 ? { warnings: fileOpWarnings } : {}),
+                };
             }
             catch (err) {
                 const message = summarizeError(err);

package/dist/platform/tools/exec/schema.d.ts

@@ -196,63 +196,63 @@ export declare const EXEC_TOOL_SCHEMA: {
 };
 export type ExecVerbosity = 'count_only' | 'minimal' | 'standard' | 'verbose';
 export interface ExecExpect {
-    exit_code?: number;
-    stdout_contains?: string;
-    stderr_contains?: string;
+    exit_code?: number | undefined;
+    stdout_contains?: string | undefined;
+    stderr_contains?: string | undefined;
 }
 export interface ExecRetry {
-    max?: number;
-    delay_ms?: number;
+    max?: number | undefined;
+    delay_ms?: number | undefined;
     /** Max jitter cap for exponential backoff. Default: 30000. */
-    max_delay_ms?: number;
-    backoff?: 'fixed' | 'exponential';
+    max_delay_ms?: number | undefined;
+    backoff?: 'fixed' | 'exponential' | undefined;
     /** Error categories to retry on. Default: ['network', 'lock', 'busy']. */
-    on?: ReadonlyArray<'network' | 'lock' | 'busy' | 'oom'>;
+    on?: ReadonlyArray<'network' | 'lock' | 'busy' | 'oom'> | undefined;
 }
 export interface ExecUntil {
     pattern: string;
-    timeout_ms?: number;
-    kill_after?: boolean;
+    timeout_ms?: number | undefined;
+    kill_after?: boolean | undefined;
 }
 export interface ExecFileOp {
     op: 'copy' | 'move' | 'delete';
     source: string;
-    destination?: string;
-    recursive?: boolean;
+    destination?: string | undefined;
+    recursive?: boolean | undefined;
     /** Overwrite destination if it exists (copy/move only). Default: false. */
-    overwrite?: boolean;
+    overwrite?: boolean | undefined;
     /** Preview what would be deleted without deleting (delete only). Default: false. */
-    dry_run?: boolean;
+    dry_run?: boolean | undefined;
     /** Rewrite TS/JS import paths after move (move only). Default: false. */
-    update_imports?: boolean;
+    update_imports?: boolean | undefined;
 }
 export interface ExecCommandInput {
-    cmd?: string;
-    cmd_base64?: string;
-    cwd?: string;
-    timeout_ms?: number;
-    env?: Record<string, string>;
-    expect?: ExecExpect;
-    background?: boolean;
-    retry?: ExecRetry;
-    until?: ExecUntil;
+    cmd?: string | undefined;
+    cmd_base64?: string | undefined;
+    cwd?: string | undefined;
+    timeout_ms?: number | undefined;
+    env?: Record<string, string> | undefined;
+    expect?: ExecExpect | undefined;
+    background?: boolean | undefined;
+    retry?: ExecRetry | undefined;
+    until?: ExecUntil | undefined;
     /** Stream stdout to a pollable progress file. Auto-enabled when timeout_ms > 30000. */
-    progress?: boolean;
+    progress?: boolean | undefined;
 }
 export interface ExecInput {
     commands: ExecCommandInput[];
-    parallel?: boolean;
-    working_dir?: string;
-    timeout_ms?: number;
-    verbosity?: ExecVerbosity;
-    file_ops?: ExecFileOp[];
+    parallel?: boolean | undefined;
+    working_dir?: string | undefined;
+    timeout_ms?: number | undefined;
+    verbosity?: ExecVerbosity | undefined;
+    file_ops?: ExecFileOp[] | undefined;
     /**
      * Stop sequential execution on first failed command.
      * Unexecuted commands appear as {skipped: true} entries. Default: false.
      */
-    fail_fast?: boolean;
+    fail_fast?: boolean | undefined;
     /** Alias for fail_fast. */
-    stop_on_error?: boolean;
+    stop_on_error?: boolean | undefined;
 }
 export interface ExecCommandResult {
     cmd: string;
@@ -261,25 +261,27 @@ export interface ExecCommandResult {
     stderr: string;
     success: boolean;
     /** Set when expectations are violated. */
-    expectation_error?: string;
+    expectation_error?: string | undefined;
     /** Set when command exceeded timeout. */
-    timed_out?: boolean;
+    timed_out?: boolean | undefined;
     /** Set when command ran in background. */
-    process_id?: string;
-    pid?: number;
+    process_id?: string | undefined;
+    pid?: number | undefined;
     /** Timing info (verbose only). */
-    duration_ms?: number;
-    cwd?: string;
-    env?: Record<string, string>;
+    duration_ms?: number | undefined;
+    cwd?: string | undefined;
+    env?: Record<string, string> | undefined;
     /** Truncation note. */
-    stdout_truncated?: boolean;
-    stderr_truncated?: boolean;
+    stdout_truncated?: boolean | undefined;
+    stderr_truncated?: boolean | undefined;
     /** Number of retry attempts used. */
-    retries?: number;
+    retries?: number | undefined;
     /** Set when this command was not executed due to fail_fast/stop_on_error. */
-    skipped?: boolean;
+    skipped?: boolean | undefined;
     /** Path to the pollable progress file when progress tracking is enabled. */
-    progress_file?: string;
+    progress_file?: string | undefined;
+    /** Tool-level warnings for degraded command collection or side effects. */
+    warnings?: string[] | undefined;
 }
 export type { BackgroundProcess } from '../shared/process-manager.js';
 //# sourceMappingURL=schema.d.ts.map

package/dist/platform/tools/exec/schema.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"schema.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/exec/schema.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,eAAO,MAAM,gBAAgB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA6MnB,CAAC;AAIX,MAAM,MAAM,aAAa,GAAG,YAAY,GAAG,SAAS,GAAG,UAAU,GAAG,SAAS,CAAC;AAE9E,MAAM,WAAW,UAAU;IACzB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,SAAS;IACxB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,8DAA8D;IAC9D,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,OAAO,CAAC,EAAE,OAAO,GAAG,aAAa,CAAC;IAClC,0EAA0E;IAC1E,EAAE,CAAC,EAAE,aAAa,CAAC,SAAS,GAAG,MAAM,GAAG,MAAM,GAAG,KAAK,CAAC,CAAC;CACzD;AAED,MAAM,WAAW,SAAS;IACxB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB;AAED,MAAM,WAAW,UAAU;IACzB,EAAE,EAAE,MAAM,GAAG,MAAM,GAAG,QAAQ,CAAC;IAC/B,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,2EAA2E;IAC3E,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,oFAAoF;IACpF,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,yEAAyE;IACzE,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAED,MAAM,WAAW,gBAAgB;IAC/B,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7B,MAAM,CAAC,EAAE,UAAU,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,KAAK,CAAC,EAAE,SAAS,CAAC;IAClB,KAAK,CAAC,EAAE,SAAS,CAAC;IAClB,uFAAuF;IACvF,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAED,MAAM,WAAW,SAAS;IACxB,QAAQ,EAAE,gBAAgB,EAAE,CAAC;IAC7B,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,aAAa,CAAC;IAC1B,QAAQ,CAAC,EAAE,UAAU,EAAE,CAAC;IACxB;;;OAGG;IACH,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,2BAA2B;IAC3B,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAID,MAAM,WAAW,iBAAiB;IAChC,GAAG,EAAE,MAAM,CAAC;IACZ,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,OAAO,CAAC;IACjB,0CAA0C;IAC1C,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,yCAAyC;IACzC,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,0CAA0C;IAC1C,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,kCAAkC;IAClC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7B,uBAAuB;IACvB,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,qCAAqC;IACrC,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,6EAA6E;IAC7E,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,4EAA4E;IAC5E,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAGD,YAAY,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAC"}
+{"version":3,"file":"schema.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/exec/schema.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,eAAO,MAAM,gBAAgB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA6MnB,CAAC;AAIX,MAAM,MAAM,aAAa,GAAG,YAAY,GAAG,SAAS,GAAG,UAAU,GAAG,SAAS,CAAC;AAE9E,MAAM,WAAW,UAAU;IACzB,SAAS,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC/B,eAAe,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACrC,eAAe,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CACtC;AAED,MAAM,WAAW,SAAS;IACxB,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,QAAQ,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC9B,8DAA8D;IAC9D,YAAY,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAClC,OAAO,CAAC,EAAE,OAAO,GAAG,aAAa,GAAG,SAAS,CAAC;IAC9C,0EAA0E;IAC1E,EAAE,CAAC,EAAE,aAAa,CAAC,SAAS,GAAG,MAAM,GAAG,MAAM,GAAG,KAAK,CAAC,GAAG,SAAS,CAAC;CACrE;AAED,MAAM,WAAW,SAAS;IACxB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,UAAU,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CAClC;AAED,MAAM,WAAW,UAAU;IACzB,EAAE,EAAE,MAAM,GAAG,MAAM,GAAG,QAAQ,CAAC;IAC/B,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACjC,SAAS,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAChC,2EAA2E;IAC3E,SAAS,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAChC,oFAAoF;IACpF,OAAO,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAC9B,yEAAyE;IACzE,cAAc,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CACtC;AAED,MAAM,WAAW,gBAAgB;IAC/B,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,UAAU,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,UAAU,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,CAAC;IACzC,MAAM,CAAC,EAAE,UAAU,GAAG,SAAS,CAAC;IAChC,UAAU,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACjC,KAAK,CAAC,EAAE,SAAS,GAAG,SAAS,CAAC;IAC9B,KAAK,CAAC,EAAE,SAAS,GAAG,SAAS,CAAC;IAC9B,uFAAuF;IACvF,QAAQ,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CAChC;AAED,MAAM,WAAW,SAAS;IACxB,QAAQ,EAAE,gBAAgB,EAAE,CAAC;IAC7B,QAAQ,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAC/B,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACjC,UAAU,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,SAAS,CAAC,EAAE,aAAa,GAAG,SAAS,CAAC;IACtC,QAAQ,CAAC,EAAE,UAAU,EAAE,GAAG,SAAS,CAAC;IACpC;;;OAGG;IACH,SAAS,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAChC,2BAA2B;IAC3B,aAAa,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CACrC;AAID,MAAM,WAAW,iBAAiB;IAChC,GAAG,EAAE,MAAM,CAAC;IACZ,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,OAAO,CAAC;IACjB,0CAA0C;IAC1C,iBAAiB,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACvC,yCAAyC;IACzC,SAAS,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAChC,0CAA0C;IAC1C,UAAU,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,kCAAkC;IAClC,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACjC,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,CAAC;IACzC,uBAAuB;IACvB,gBAAgB,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACvC,gBAAgB,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACvC,qCAAqC;IACrC,OAAO,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC7B,6EAA6E;IAC7E,OAAO,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAC9B,4EAA4E;IAC5E,aAAa,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,2EAA2E;IAC3E,QAAQ,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAC;CACjC;AAGD,YAAY,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAC"}

package/dist/platform/tools/exec/schema.js

@@ -189,8 +189,8 @@ export const EXEC_TOOL_SCHEMA = {
                     },
                     update_imports: {
                         type: 'boolean',
-                        description: 'After a move, find all TypeScript/JavaScript files that import from the old path '
-                            + 'and rewrite their import statements to use the new path (move only). Default: false.',
+                        description: 'After a move, find all TypeScript/JavaScript files that import from the previous path '
+                            + 'and rewrite their import statements to use the new path for this move. Default: false.',
                         default: false,
                     },
                 },

package/dist/platform/tools/fetch/extract.d.ts

@@ -1,6 +1,6 @@
 import type { FetchExtractMode } from './schema.js';
 export declare function sniffContentType(contentType: string, body: string): string;
 export declare function applyExtract(body: string, contentType: string, mode: FetchExtractMode, opts?: {
-    selectors?: string[];
-}): string;
+    selectors?: string[] | undefined;
+} | undefined): string;
 //# sourceMappingURL=extract.d.ts.map

package/dist/platform/tools/fetch/extract.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"extract.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/fetch/extract.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAuQpD,wBAAgB,gBAAgB,CAAC,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAe1E;AAED,wBAAgB,YAAY,CAC1B,IAAI,EAAE,MAAM,EACZ,WAAW,EAAE,MAAM,EACnB,IAAI,EAAE,gBAAgB,EACtB,IAAI,CAAC,EAAE;IAAE,SAAS,CAAC,EAAE,MAAM,EAAE,CAAA;CAAE,GAC9B,MAAM,CA2CR"}
+{"version":3,"file":"extract.d.ts","sourceRoot":"","sources":["../../../../src/platform/tools/fetch/extract.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAkRpD,wBAAgB,gBAAgB,CAAC,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAe1E;AAED,wBAAgB,YAAY,CAC1B,IAAI,EAAE,MAAM,EACZ,WAAW,EAAE,MAAM,EACnB,IAAI,EAAE,gBAAgB,EACtB,IAAI,CAAC,EAAE;IAAE,SAAS,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAA;CAAE,GAAG,SAAS,GACtD,MAAM,CA2CR"}