deepseek-coder-agent-cli 1.0.0

package/dist/core/toolPreconditions.js

@@ -0,0 +1,242 @@
+import { existsSync, realpathSync } from 'node:fs';
+import { basename, dirname, isAbsolute, join, normalize as normalizePath, resolve as resolvePath } from 'node:path';
+import { analyzeBashFlow } from './bashCommandGuidance.js';
+export const EDIT_WITHOUT_READ = 'EDIT_WITHOUT_READ';
+/**
+ * Validate tool preconditions before execution to prevent common AI flow failures.
+ *
+ * This function implements the critical AI flow design principle: validate before execute.
+ * It catches common patterns that lead to tool failures and provides actionable guidance.
+ *
+ * @param toolName - Name of the tool being called
+ * @param args - Tool arguments
+ * @returns Array of pre-flight warnings (empty if all validations pass)
+ */
+export function validateToolPreconditions(toolName, args) {
+    const warnings = [];
+    const toolLower = toolName.toLowerCase();
+    // -------------------------------------------------------------------------
+    // Edit Tool Validation - Critical for AI software engineering flow
+    // -------------------------------------------------------------------------
+    if (toolLower === 'edit') {
+        const oldString = args['old_string'];
+        const newString = args['new_string'];
+        const filePath = args['file_path'];
+        if (!oldString) {
+            // Empty old_string = new file creation - validate path structure
+            if (filePath && !filePath.startsWith('/')) {
+                warnings.push({
+                    code: 'EDIT_RELATIVE_PATH',
+                    message: 'Edit file_path should be absolute path for new file creation',
+                    severity: 'warning',
+                    suggestion: 'Use absolute path starting with / for file creation',
+                });
+            }
+        }
+        else {
+            // PATTERN 1: Placeholder patterns - AI is guessing content (but NOT "TODO" which is legitimate code)
+            const placeholderPatterns = ['[content]', '[code]', '/* ... */', '// ...'];
+            // Only flag "..." if it's standalone, not as part of spread operator or ellipsis in comments
+            const hasStandalonePlaceholder = placeholderPatterns.some((p) => oldString.includes(p)) ||
+                /(?<!\.)\.\.\.(?!\.)/.test(oldString); // Matches ... but not .... or .....
+            if (hasStandalonePlaceholder) {
+                warnings.push({
+                    code: 'EDIT_PLACEHOLDER',
+                    message: 'Placeholder patterns detected - HALLUCINATION ALERT',
+                    severity: 'critical',
+                    suggestion: 'AI is guessing content instead of reading actual file. ALWAYS use Read tool to get exact file content including whitespace.',
+                });
+            }
+            // PATTERN 2: Tab vs space mismatch (legitimate concern)
+            if (oldString && newString) {
+                const oldHasTabs = oldString.includes('\t');
+                const newHasTabs = newString.includes('\t');
+                if (oldHasTabs && !newHasTabs) {
+                    warnings.push({
+                        code: 'EDIT_WHITESPACE_MISMATCH',
+                        message: 'old_string uses tabs, new_string uses spaces',
+                        severity: 'warning',
+                        suggestion: 'Whitespace type mismatch may cause Edit to fail.',
+                    });
+                }
+            }
+        }
+    }
+    // -------------------------------------------------------------------------
+    // Search Operations Validation - Only flag truly problematic patterns
+    // -------------------------------------------------------------------------
+    if (toolLower === 'grep' || toolLower === 'glob' || toolLower.includes('search')) {
+        const patternArg = args['pattern'];
+        const pattern = typeof patternArg === 'string' ? patternArg.trim() : '';
+        const isGrepTool = toolLower.includes('grep');
+        if (pattern) {
+            // Only warn on truly broad patterns that would match everything
+            if (isBroadBasePattern(pattern)) {
+                warnings.push({
+                    code: 'SEARCH_BROAD_PATTERN',
+                    message: `Pattern '${pattern}' is too broad and may return thousands of files`,
+                    severity: 'critical',
+                    suggestion: "Use targeted patterns like '**/*.ts' or 'src/**/*.js'.",
+                });
+            }
+            // For grep: only warn on patterns that literally match everything
+            if (isGrepTool && isVeryBroadRegex(pattern)) {
+                const hasFileFilters = typeof args['glob'] === 'string' || typeof args['type'] === 'string';
+                if (!hasFileFilters) {
+                    warnings.push({
+                        code: 'SEARCH_CONTEXT_OVERFLOW_RISK',
+                        message: `Regex pattern '${pattern}' matches everything`,
+                        severity: 'warning',
+                        suggestion: 'Use a more specific pattern or add a glob/type filter.',
+                    });
+                }
+            }
+        }
+    }
+    // -------------------------------------------------------------------------
+    // Bash Command Efficiency
+    // -------------------------------------------------------------------------
+    if (toolLower === 'bash' || toolLower === 'execute_bash') {
+        const command = String(args['command'] || '');
+        const flowWarnings = analyzeBashFlow(command);
+        for (const flow of flowWarnings) {
+            warnings.push({
+                code: flow.code,
+                message: flow.message,
+                severity: flow.severity === 'critical' ? 'critical' : flow.severity === 'info' ? 'info' : 'warning',
+                suggestion: flow.suggestion ?? flow.message,
+            });
+        }
+    }
+    return warnings;
+}
+/**
+ * Enhanced AI flow validation for TypeScript software engineering
+ * Provides comprehensive validation of AI tool usage patterns
+ */
+export function validateAIFlowPatterns(toolName, args, toolHistory) {
+    const warnings = [];
+    const toolLower = toolName.toLowerCase();
+    // Pattern: Edit without prior Read - Critical for hallucination reduction
+    if (toolLower === 'edit') {
+        const targetPath = normalizeFilePath(args['file_path']);
+        const isNewFileCreation = isNewFileEdit(args['old_string']);
+        const hasRecentRead = hasMatchingRead(toolHistory, targetPath);
+        if (!isNewFileCreation && !hasRecentRead) {
+            warnings.push({
+                code: EDIT_WITHOUT_READ,
+                message: 'Edit attempted without prior file read - HIGH HALLUCINATION RISK',
+                severity: 'critical',
+                suggestion: 'ALWAYS use Read tool first to get exact text including whitespace and indentation. Never guess file content.',
+            });
+        }
+    }
+    // Pattern: Multiple validation runs
+    if (toolLower === 'run_repo_checks' || toolLower === 'run_tests' || toolLower === 'run_build') {
+        const now = Date.now();
+        const recencyWindowMs = 15 * 60 * 1000; // Only warn when validations are clustered
+        const recentValidations = toolHistory
+            .filter((call) => call.toolName.toLowerCase().includes('run_') ||
+            call.toolName.toLowerCase().includes('test') ||
+            call.toolName.toLowerCase().includes('build'))
+            .filter((call) => {
+            if (typeof call.timestamp !== 'number') {
+                return true;
+            }
+            return now - call.timestamp <= recencyWindowMs;
+        });
+        if (recentValidations.length > 0) {
+            warnings.push({
+                code: 'DEFERRED_VALIDATION_REMINDER',
+                message: 'Validation already ran recently; batch edits and run one final pass at the end.',
+                severity: 'info',
+                suggestion: 'Finish implementation work first, then run a single build/test/check step to validate.',
+            });
+        }
+    }
+    // Pattern: Excessive git status calls (actually wasteful)
+    // Note: Multiple git operations in sequence (add, commit, push) is fine - don't warn on that
+    if (toolLower.includes('git')) {
+        const command = String(args['command'] || '');
+        if (command.includes('git status')) {
+            const recentStatusCalls = toolHistory.filter((call) => {
+                const cmd = String(call.args['command'] || '');
+                return cmd.includes('git status');
+            });
+            // Only warn if there are 3+ status calls - 2 is normal (before and after operations)
+            if (recentStatusCalls.length >= 3) {
+                warnings.push({
+                    code: 'GIT_REDUNDANT_STATUS',
+                    message: 'Multiple git status calls detected',
+                    severity: 'info',
+                    suggestion: 'One git status call is usually sufficient.',
+                });
+            }
+        }
+    }
+    return warnings;
+}
+function isBroadBasePattern(pattern) {
+    // Only the truly universal patterns that match everything
+    return pattern === '.' || pattern === '*' || pattern === '**' || pattern === '**/*';
+}
+function isVeryBroadRegex(pattern) {
+    const normalized = pattern.trim();
+    return normalized === '.*' || normalized === '.+' || normalized === '.';
+}
+function hasMatchingRead(toolHistory, targetPath) {
+    if (!targetPath)
+        return false;
+    return toolHistory.some((call) => {
+        const callLower = call.toolName.toLowerCase();
+        if (!callLower.includes('read')) {
+            return false;
+        }
+        const readPath = normalizeFilePath(call.args['path'] ?? call.args['file_path']);
+        return Boolean(readPath && readPath === targetPath);
+    });
+}
+function isNewFileEdit(oldString) {
+    return typeof oldString === 'string' && oldString.length === 0;
+}
+function normalizeFilePath(pathValue) {
+    if (typeof pathValue !== 'string') {
+        return null;
+    }
+    const trimmed = pathValue.trim();
+    if (!trimmed) {
+        return null;
+    }
+    try {
+        const normalized = normalizePath(trimmed);
+        const absolutePath = isAbsolute(normalized) ? normalized : resolvePath(process.cwd(), normalized);
+        // Resolve symlinks on the deepest existing directory to avoid /var vs /private/var mismatches
+        let currentDir = dirname(absolutePath);
+        let suffix = basename(absolutePath);
+        // Loop until we find an existing ancestor or hit the filesystem root.
+        // eslint-disable-next-line no-constant-condition
+        while (true) {
+            if (existsSync(currentDir)) {
+                try {
+                    const realDir = realpathSync(currentDir);
+                    return normalizePath(join(realDir, suffix));
+                }
+                catch {
+                    // If realpath fails, fall through to the default normalization
+                    break;
+                }
+            }
+            const parentDir = dirname(currentDir);
+            if (parentDir === currentDir) {
+                break;
+            }
+            suffix = join(basename(currentDir), suffix);
+            currentDir = parentDir;
+        }
+        return normalizePath(absolutePath);
+    }
+    catch {
+        return trimmed;
+    }
+}
+//# sourceMappingURL=toolPreconditions.js.map

package/dist/core/toolPreconditions.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"toolPreconditions.js","sourceRoot":"","sources":["../../src/core/toolPreconditions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACnD,OAAO,EAAE,QAAQ,EAAE,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,SAAS,IAAI,aAAa,EAAE,OAAO,IAAI,WAAW,EAAE,MAAM,WAAW,CAAC;AACpH,OAAO,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAC;AAa3D,MAAM,CAAC,MAAM,iBAAiB,GAAG,mBAAmB,CAAC;AAQrD;;;;;;;;;GASG;AACH,MAAM,UAAU,yBAAyB,CACvC,QAAgB,EAChB,IAA6B;IAE7B,MAAM,QAAQ,GAAuB,EAAE,CAAC;IACxC,MAAM,SAAS,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC;IAEzC,4EAA4E;IAC5E,mEAAmE;IACnE,4EAA4E;IAC5E,IAAI,SAAS,KAAK,MAAM,EAAE,CAAC;QACzB,MAAM,SAAS,GAAG,IAAI,CAAC,YAAY,CAAuB,CAAC;QAC3D,MAAM,SAAS,GAAG,IAAI,CAAC,YAAY,CAAuB,CAAC;QAC3D,MAAM,QAAQ,GAAG,IAAI,CAAC,WAAW,CAAuB,CAAC;QAEzD,IAAI,CAAC,SAAS,EAAE,CAAC;YACf,iEAAiE;YACjE,IAAI,QAAQ,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;gBAC1C,QAAQ,CAAC,IAAI,CAAC;oBACZ,IAAI,EAAE,oBAAoB;oBAC1B,OAAO,EAAE,8DAA8D;oBACvE,QAAQ,EAAE,SAAS;oBACnB,UAAU,EAAE,qDAAqD;iBAClE,CAAC,CAAC;YACL,CAAC;QACH,CAAC;aAAM,CAAC;YACN,qGAAqG;YACrG,MAAM,mBAAmB,GAAG,CAAC,WAAW,EAAE,QAAQ,EAAE,WAAW,EAAE,QAAQ,CAAC,CAAC;YAC3E,6FAA6F;YAC7F,MAAM,wBAAwB,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC;gBACrF,qBAAqB,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,oCAAoC;YAC7E,IAAI,wBAAwB,EAAE,CAAC;gBAC7B,QAAQ,CAAC,IAAI,CAAC;oBACZ,IAAI,EAAE,kBAAkB;oBACxB,OAAO,EAAE,qDAAqD;oBAC9D,QAAQ,EAAE,UAAU;oBACpB,UAAU,EACR,6HAA6H;iBAChI,CAAC,CAAC;YACL,CAAC;YAED,wDAAwD;YACxD,IAAI,SAAS,IAAI,SAAS,EAAE,CAAC;gBAC3B,MAAM,UAAU,GAAG,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;gBAC5C,MAAM,UAAU,GAAG,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;gBAC5C,IAAI,UAAU,IAAI,CAAC,UAAU,EAAE,CAAC;oBAC9B,QAAQ,CAAC,IAAI,CAAC;wBACZ,IAAI,EAAE,0BAA0B;wBAChC,OAAO,EAAE,8CAA8C;wBACvD,QAAQ,EAAE,SAAS;wBACnB,UAAU,EAAE,kDAAkD;qBAC/D,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAED,4EAA4E;IAC5E,sEAAsE;IACtE,4EAA4E;IAC5E,IAAI,SAAS,KAAK,MAAM,IAAI,SAAS,KAAK,MAAM,IAAI,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;QACjF,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC;QACnC,MAAM,OAAO,GAAG,OAAO,UAAU,KAAK,QAAQ,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACxE,MAAM,UAAU,GAAG,SAAS,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;QAE9C,IAAI,OAAO,EAAE,CAAC;YACZ,gEAAgE;YAChE,IAAI,kBAAkB,CAAC,OAAO,CAAC,EAAE,CAAC;gBAChC,QAAQ,CAAC,IAAI,CAAC;oBACZ,IAAI,EAAE,sBAAsB;oBAC5B,OAAO,EAAE,YAAY,OAAO,kDAAkD;oBAC9E,QAAQ,EAAE,UAAU;oBACpB,UAAU,EAAE,wDAAwD;iBACrE,CAAC,CAAC;YACL,CAAC;YAED,kEAAkE;YAClE,IAAI,UAAU,IAAI,gBAAgB,CAAC,OAAO,CAAC,EAAE,CAAC;gBAC5C,MAAM,cAAc,GAAG,OAAO,IAAI,CAAC,MAAM,CAAC,KAAK,QAAQ,IAAI,OAAO,IAAI,CAAC,MAAM,CAAC,KAAK,QAAQ,CAAC;gBAC5F,IAAI,CAAC,cAAc,EAAE,CAAC;oBACpB,QAAQ,CAAC,IAAI,CAAC;wBACZ,IAAI,EAAE,8BAA8B;wBACpC,OAAO,EAAE,kBAAkB,OAAO,sBAAsB;wBACxD,QAAQ,EAAE,SAAS;wBACnB,UAAU,EAAE,wDAAwD;qBACrE,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAED,4EAA4E;IAC5E,0BAA0B;IAC1B,4EAA4E;IAC5E,IAAI,SAAS,KAAK,MAAM,IAAI,SAAS,KAAK,cAAc,EAAE,CAAC;QACzD,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC;QAC9C,MAAM,YAAY,GAAG,eAAe,CAAC,OAAO,CAAC,CAAC;QAE9C,KAAK,MAAM,IAAI,IAAI,YAAY,EAAE,CAAC;YAChC,QAAQ,CAAC,IAAI,CAAC;gBACZ,IAAI,EAAE,IAAI,CAAC,IAAI;gBACf,OAAO,EAAE,IAAI,CAAC,OAAO;gBACrB,QAAQ,EAAE,IAAI,CAAC,QAAQ,KAAK,UAAU,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,SAAS;gBACnG,UAAU,EAAE,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,OAAO;aAC5C,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,sBAAsB,CACpC,QAAgB,EAChB,IAA6B,EAC7B,WAAuC;IAEvC,MAAM,QAAQ,GAAuB,EAAE,CAAC;IACxC,MAAM,SAAS,GAAG,QAAQ,CAAC,WAAW,EAAE,CAAC;IAEzC,0EAA0E;IAC1E,IAAI,SAAS,KAAK,MAAM,EAAE,CAAC;QACzB,MAAM,UAAU,GAAG,iBAAiB,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,CAAC;QACxD,MAAM,iBAAiB,GAAG,aAAa,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC;QAC5D,MAAM,aAAa,GAAG,eAAe,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC;QAE/D,IAAI,CAAC,iBAAiB,IAAI,CAAC,aAAa,EAAE,CAAC;YACzC,QAAQ,CAAC,IAAI,CAAC;gBACZ,IAAI,EAAE,iBAAiB;gBACvB,OAAO,EAAE,kEAAkE;gBAC3E,QAAQ,EAAE,UAAU;gBACpB,UAAU,EAAE,8GAA8G;aAC3H,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,oCAAoC;IACpC,IAAI,SAAS,KAAK,iBAAiB,IAAI,SAAS,KAAK,WAAW,IAAI,SAAS,KAAK,WAAW,EAAE,CAAC;QAC9F,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;QACvB,MAAM,eAAe,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC,2CAA2C;QACnF,MAAM,iBAAiB,GAAG,WAAW;aAClC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CACf,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,MAAM,CAAC;YAC5C,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,MAAM,CAAC;YAC5C,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,CAC9C;aACA,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE;YACf,IAAI,OAAO,IAAI,CAAC,SAAS,KAAK,QAAQ,EAAE,CAAC;gBACvC,OAAO,IAAI,CAAC;YACd,CAAC;YACD,OAAO,GAAG,GAAG,IAAI,CAAC,SAAS,IAAI,eAAe,CAAC;QACjD,CAAC,CAAC,CAAC;QAEL,IAAI,iBAAiB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACjC,QAAQ,CAAC,IAAI,CAAC;gBACZ,IAAI,EAAE,8BAA8B;gBACpC,OAAO,EAAE,iFAAiF;gBAC1F,QAAQ,EAAE,MAAM;gBAChB,UAAU,EAAE,wFAAwF;aACrG,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,0DAA0D;IAC1D,6FAA6F;IAC7F,IAAI,SAAS,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QAC9B,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC;QAC9C,IAAI,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAC,EAAE,CAAC;YACnC,MAAM,iBAAiB,GAAG,WAAW,CAAC,MAAM,CAC1C,CAAC,IAAI,EAAE,EAAE;gBACP,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC;gBAC/C,OAAO,GAAG,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC;YACpC,CAAC,CACF,CAAC;YACF,qFAAqF;YACrF,IAAI,iBAAiB,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;gBAClC,QAAQ,CAAC,IAAI,CAAC;oBACZ,IAAI,EAAE,sBAAsB;oBAC5B,OAAO,EAAE,oCAAoC;oBAC7C,QAAQ,EAAE,MAAM;oBAChB,UAAU,EAAE,4CAA4C;iBACzD,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,SAAS,kBAAkB,CAAC,OAAe;IACzC,0DAA0D;IAC1D,OAAO,OAAO,KAAK,GAAG,IAAI,OAAO,KAAK,GAAG,IAAI,OAAO,KAAK,IAAI,IAAI,OAAO,KAAK,MAAM,CAAC;AACtF,CAAC;AAED,SAAS,gBAAgB,CAAC,OAAe;IACvC,MAAM,UAAU,GAAG,OAAO,CAAC,IAAI,EAAE,CAAC;IAClC,OAAO,UAAU,KAAK,IAAI,IAAI,UAAU,KAAK,IAAI,IAAI,UAAU,KAAK,GAAG,CAAC;AAC1E,CAAC;AAED,SAAS,eAAe,CACtB,WAAuC,EACvC,UAAyB;IAEzB,IAAI,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IAE9B,OAAO,WAAW,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE;QAC/B,MAAM,SAAS,GAAG,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC;QAC9C,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;YAChC,OAAO,KAAK,CAAC;QACf,CAAC;QAED,MAAM,QAAQ,GAAG,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,CAAC;QAChF,OAAO,OAAO,CAAC,QAAQ,IAAI,QAAQ,KAAK,UAAU,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;AACL,CAAC;AAED,SAAS,aAAa,CAAC,SAAkB;IACvC,OAAO,OAAO,SAAS,KAAK,QAAQ,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,CAAC;AACjE,CAAC;AAED,SAAS,iBAAiB,CAAC,SAAkB;IAC3C,IAAI,OAAO,SAAS,KAAK,QAAQ,EAAE,CAAC;QAClC,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,OAAO,GAAG,SAAS,CAAC,IAAI,EAAE,CAAC;IACjC,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC;QAC1C,MAAM,YAAY,GAAG,UAAU,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,WAAW,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,UAAU,CAAC,CAAC;QAElG,8FAA8F;QAC9F,IAAI,UAAU,GAAG,OAAO,CAAC,YAAY,CAAC,CAAC;QACvC,IAAI,MAAM,GAAG,QAAQ,CAAC,YAAY,CAAC,CAAC;QAEpC,sEAAsE;QACtE,iDAAiD;QACjD,OAAO,IAAI,EAAE,CAAC;YACZ,IAAI,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;gBAC3B,IAAI,CAAC;oBACH,MAAM,OAAO,GAAG,YAAY,CAAC,UAAU,CAAC,CAAC;oBACzC,OAAO,aAAa,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC;gBAC9C,CAAC;gBAAC,MAAM,CAAC;oBACP,+DAA+D;oBAC/D,MAAM;gBACR,CAAC;YACH,CAAC;YAED,MAAM,SAAS,GAAG,OAAO,CAAC,UAAU,CAAC,CAAC;YACtC,IAAI,SAAS,KAAK,UAAU,EAAE,CAAC;gBAC7B,MAAM;YACR,CAAC;YAED,MAAM,GAAG,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC,CAAC;YAC5C,UAAU,GAAG,SAAS,CAAC;QACzB,CAAC;QAED,OAAO,aAAa,CAAC,YAAY,CAAC,CAAC;IACrC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,OAAO,CAAC;IACjB,CAAC;AACH,CAAC"}

package/dist/core/toolRuntime.d.ts

@@ -0,0 +1,185 @@
+import { type JSONSchemaObject, type ProviderId, type ProviderToolDefinition, type ToolCallRequest } from './types.js';
+import { ContextManager } from './contextManager.js';
+import { type PreflightWarning } from './toolPreconditions.js';
+/**
+ * Execution context for tool operations with strict TypeScript typing
+ */
+export interface ToolExecutionContext {
+    readonly profileName: string;
+    readonly provider: ProviderId;
+    readonly model: string;
+    readonly workspaceContext?: string | null;
+}
+/**
+ * Type-safe tool observer with generic parameter inference for AI flow monitoring
+ */
+export interface ToolRuntimeObserver<T extends Record<string, unknown> = Record<string, unknown>> {
+    /** Called when tool execution begins */
+    onToolStart?(call: ToolCallRequest & {
+        args: T;
+    }): void;
+    /** Called when tool execution completes successfully */
+    onToolResult?(call: ToolCallRequest & {
+        args: T;
+    }, output: string): void;
+    /** Called when tool execution fails */
+    onToolError?(call: ToolCallRequest & {
+        args: T;
+    }, error: string): void;
+    /** Called when cached result is used instead of execution */
+    onCacheHit?(call: ToolCallRequest & {
+        args: T;
+    }): void;
+    /** Called for progress updates during long-running operations */
+    onToolProgress?(call: ToolCallRequest & {
+        args: T;
+    }, progress: ToolProgressUpdate): void;
+    /** Called for pre-flight warnings before tool execution */
+    onToolWarning?(call: ToolCallRequest & {
+        args: T;
+    }, warning: PreflightWarning | string): void;
+}
+interface ToolRuntimeOptions {
+    readonly observer?: ToolRuntimeObserver;
+    readonly contextManager?: ContextManager;
+    readonly enableCache?: boolean;
+    readonly cacheTTLMs?: number;
+}
+/**
+ * Generic tool handler with parameter type inference for AI flow execution
+ */
+type ToolHandler<T extends Record<string, unknown> = Record<string, unknown>> = (args: T) => Promise<string> | string;
+/**
+ * Enhanced tool definition with parameter type safety for AI software engineering
+ */
+export interface ToolDefinition<T extends Record<string, unknown> = Record<string, unknown>> {
+    /** Unique identifier for the tool */
+    readonly name: string;
+    /** Human-readable description for AI understanding */
+    readonly description: string;
+    /** JSON Schema defining the tool's parameter structure */
+    readonly parameters?: JSONSchemaObject;
+    /** Function that implements the tool's behavior */
+    readonly handler: ToolHandler<T>;
+    /** Whether results can be cached for performance optimization */
+    readonly cacheable?: boolean;
+    /** Optional per-tool cache TTL in milliseconds (falls back to runtime default) */
+    readonly cacheTtlMs?: number;
+}
+/**
+ * Collection of related tools grouped by functionality
+ */
+export interface ToolSuite {
+    /** Unique identifier for the tool suite */
+    readonly id: string;
+    /** Human-readable description of the suite's purpose */
+    readonly description?: string;
+    /** Array of tool definitions in this suite */
+    readonly tools: readonly ToolDefinition[];
+}
+export interface ToolHistoryEntry {
+    toolName: string;
+    args: Record<string, unknown>;
+    timestamp: number;
+    success: boolean;
+    hasOutput: boolean;
+    error?: string;
+}
+export interface DiffSnapshotRecord {
+    command: string;
+    output: string;
+    timestamp: number;
+}
+export interface ToolProgressUpdate {
+    current: number;
+    total?: number;
+    message?: string;
+}
+/**
+ * Report incremental progress for the currently executing tool.
+ * Tools can call this to surface live status updates (e.g., indexing files).
+ */
+export declare function reportToolProgress(progress: ToolProgressUpdate): void;
+/**
+ * Type-safe utility functions for tool runtime operations
+ */
+export declare namespace ToolRuntimeUtils {
+    /**
+     * Creates a type-safe tool definition with inferred parameter types
+     */
+    function createToolDefinition<T extends Record<string, unknown>>(definition: ToolDefinition<T>): ToolDefinition<T>;
+    /**
+     * Creates a type-safe tool suite with inferred tool types
+     */
+    function createToolSuite(suite: ToolSuite): ToolSuite;
+    /**
+     * Type guard to check if a tool definition matches expected parameter schema
+     */
+    function isToolDefinition(__tool: ToolDefinition, expectedSchema?: JSONSchemaObject): boolean;
+}
+/**
+ * Interface describing the public API of ToolRuntime.
+ * Used by wrapper implementations like RestrictedToolRuntime.
+ */
+export interface IToolRuntime {
+    listProviderTools(): ProviderToolDefinition[];
+    execute(call: ToolCallRequest, context?: {
+        profileName?: string;
+        provider?: string;
+        model?: string;
+    }): Promise<string>;
+    registerSuite(suite: ToolSuite): void;
+    unregisterSuite(id: string): void;
+    clearCache(): void;
+    getCacheStats(): {
+        size: number;
+        entries: number;
+    };
+    clearToolHistory(): void;
+    getToolHistory(): readonly ToolHistoryEntry[];
+    clearDiffSnapshots(): void;
+    getDiffSnapshots(): readonly DiffSnapshotRecord[];
+}
+export declare class ToolRuntime implements IToolRuntime {
+    private readonly registry;
+    private readonly registrationOrder;
+    private readonly observer;
+    private readonly contextManager;
+    private readonly cache;
+    private readonly enableCache;
+    private readonly cacheTTLMs;
+    private readonly toolHistory;
+    private readonly maxHistorySize;
+    private readonly diffSnapshots;
+    private readonly maxDiffSnapshots;
+    private readonly maxDiffSnapshotLength;
+    constructor(baseTools?: ToolDefinition[], options?: ToolRuntimeOptions);
+    registerSuite(suite: ToolSuite): void;
+    unregisterSuite(id: string): void;
+    listProviderTools(): ProviderToolDefinition[];
+    execute(call: ToolCallRequest, _context?: {
+        profileName?: string;
+        provider?: string;
+        model?: string;
+    }): Promise<string>;
+    private getCacheKey;
+    clearCache(): void;
+    getCacheStats(): {
+        size: number;
+        entries: number;
+    };
+    clearToolHistory(): void;
+    getToolHistory(): readonly ToolHistoryEntry[];
+    private recordToolHistory;
+    clearDiffSnapshots(): void;
+    getDiffSnapshots(): readonly DiffSnapshotRecord[];
+    private addTool;
+    private removeFromOrder;
+    private recordDiffSnapshot;
+    private findGitDiffCommand;
+    private extractCommands;
+    private isGitDiffCommand;
+}
+export declare function createDefaultToolRuntime(context: ToolExecutionContext, toolSuites?: ToolSuite[], options?: ToolRuntimeOptions): ToolRuntime;
+export {};
+//# sourceMappingURL=toolRuntime.d.ts.map

package/dist/core/toolRuntime.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"toolRuntime.d.ts","sourceRoot":"","sources":["../../src/core/toolRuntime.ts"],"names":[],"mappings":"AAEA,OAAO,EACL,KAAK,gBAAgB,EACrB,KAAK,UAAU,EACf,KAAK,sBAAsB,EAC3B,KAAK,eAAe,EACrB,MAAM,YAAY,CAAC;AAMpB,OAAO,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAErD,OAAO,EAAwE,KAAK,gBAAgB,EAAE,MAAM,wBAAwB,CAAC;AAIrI;;GAEG;AACH,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,QAAQ,EAAE,UAAU,CAAC;IAC9B,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,gBAAgB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3C;AAED;;GAEG;AACH,MAAM,WAAW,mBAAmB,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC;IAC9F,wCAAwC;IACxC,WAAW,CAAC,CAAC,IAAI,EAAE,eAAe,GAAG;QAAE,IAAI,EAAE,CAAC,CAAA;KAAE,GAAG,IAAI,CAAC;IAExD,wDAAwD;IACxD,YAAY,CAAC,CAAC,IAAI,EAAE,eAAe,GAAG;QAAE,IAAI,EAAE,CAAC,CAAA;KAAE,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IAEzE,uCAAuC;IACvC,WAAW,CAAC,CAAC,IAAI,EAAE,eAAe,GAAG;QAAE,IAAI,EAAE,CAAC,CAAA;KAAE,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IAEvE,6DAA6D;IAC7D,UAAU,CAAC,CAAC,IAAI,EAAE,eAAe,GAAG;QAAE,IAAI,EAAE,CAAC,CAAA;KAAE,GAAG,IAAI,CAAC;IAEvD,iEAAiE;IACjE,cAAc,CAAC,CAAC,IAAI,EAAE,eAAe,GAAG;QAAE,IAAI,EAAE,CAAC,CAAA;KAAE,EAAE,QAAQ,EAAE,kBAAkB,GAAG,IAAI,CAAC;IAEzF,2DAA2D;IAC3D,aAAa,CAAC,CAAC,IAAI,EAAE,eAAe,GAAG;QAAE,IAAI,EAAE,CAAC,CAAA;KAAE,EAAE,OAAO,EAAE,gBAAgB,GAAG,MAAM,GAAG,IAAI,CAAC;CAC/F;AAED,UAAU,kBAAkB;IAC1B,QAAQ,CAAC,QAAQ,CAAC,EAAE,mBAAmB,CAAC;IACxC,QAAQ,CAAC,cAAc,CAAC,EAAE,cAAc,CAAC;IACzC,QAAQ,CAAC,WAAW,CAAC,EAAE,OAAO,CAAC;IAC/B,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC;CAC9B;AAED;;GAEG;AACH,KAAK,WAAW,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,IAAI,CAC9E,IAAI,EAAE,CAAC,KACJ,OAAO,CAAC,MAAM,CAAC,GAAG,MAAM,CAAC;AAE9B;;GAEG;AACH,MAAM,WAAW,cAAc,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC;IACzF,qCAAqC;IACrC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IAEtB,sDAAsD;IACtD,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAE7B,0DAA0D;IAC1D,QAAQ,CAAC,UAAU,CAAC,EAAE,gBAAgB,CAAC;IAEvC,mDAAmD;IACnD,QAAQ,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC;IAEjC,iEAAiE;IACjE,QAAQ,CAAC,SAAS,CAAC,EAAE,OAAO,CAAC;IAE7B,kFAAkF;IAClF,QAAQ,CAAC,UAAU,CAAC,EAAE,MAAM,CAAC;CAC9B;AAED;;GAEG;AACH,MAAM,WAAW,SAAS;IACxB,2CAA2C;IAC3C,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IAEpB,wDAAwD;IACxD,QAAQ,CAAC,WAAW,CAAC,EAAE,MAAM,CAAC;IAE9B,8CAA8C;IAC9C,QAAQ,CAAC,KAAK,EAAE,SAAS,cAAc,EAAE,CAAC;CAC3C;AAYD,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,OAAO,CAAC;IACjB,SAAS,EAAE,OAAO,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,kBAAkB;IACjC,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,kBAAkB;IACjC,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AASD;;;GAGG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,kBAAkB,GAAG,IAAI,CAkBrE;AA+CD;;GAEG;AACH,yBAAiB,gBAAgB,CAAC;IAChC;;OAEG;IACH,SAAgB,oBAAoB,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EACpE,UAAU,EAAE,cAAc,CAAC,CAAC,CAAC,GAC5B,cAAc,CAAC,CAAC,CAAC,CAEnB;IAED;;OAEG;IACH,SAAgB,eAAe,CAC7B,KAAK,EAAE,SAAS,GACf,SAAS,CAEX;IAED;;OAEG;IACH,SAAgB,gBAAgB,CAC9B,MAAM,EAAE,cAAc,EACtB,cAAc,CAAC,EAAE,gBAAgB,GAChC,OAAO,CAIT;CACF;AAED;;;GAGG;AACH,MAAM,WAAW,YAAY;IAC3B,iBAAiB,IAAI,sBAAsB,EAAE,CAAC;IAC9C,OAAO,CAAC,IAAI,EAAE,eAAe,EAAE,OAAO,CAAC,EAAE;QAAE,WAAW,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC;IACvH,aAAa,CAAC,KAAK,EAAE,SAAS,GAAG,IAAI,CAAC;IACtC,eAAe,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,UAAU,IAAI,IAAI,CAAC;IACnB,aAAa,IAAI;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IACnD,gBAAgB,IAAI,IAAI,CAAC;IACzB,cAAc,IAAI,SAAS,gBAAgB,EAAE,CAAC;IAC9C,kBAAkB,IAAI,IAAI,CAAC;IAC3B,gBAAgB,IAAI,SAAS,kBAAkB,EAAE,CAAC;CACnD;AAED,qBAAa,WAAY,YAAW,YAAY;IAC9C,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAiC;IAC1D,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAgB;IAClD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAA6B;IACtD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAwB;IACvD,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAiC;IACvD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAU;IACtC,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,WAAW,CAA0B;IACtD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAM;IACrC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA4B;IAC1D,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAK;IACtC,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAQ;gBAElC,SAAS,GAAE,cAAc,EAAO,EAAE,OAAO,GAAE,kBAAuB;IAc9E,aAAa,CAAC,KAAK,EAAE,SAAS,GAAG,IAAI;IAUrC,eAAe,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAYjC,iBAAiB,IAAI,sBAAsB,EAAE;IAWvC,OAAO,CAAC,IAAI,EAAE,eAAe,EAAE,QAAQ,CAAC,EAAE;QAAE,WAAW,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,GAAG,OAAO,CAAC,MAAM,CAAC;IAwJ7H,OAAO,CAAC,WAAW;IAInB,UAAU,IAAI,IAAI;IAIlB,aAAa,IAAI;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE;IAWlD,gBAAgB,IAAI,IAAI;IAIxB,cAAc,IAAI,SAAS,gBAAgB,EAAE;IAI7C,OAAO,CAAC,iBAAiB;IAOzB,kBAAkB,IAAI,IAAI;IAI1B,gBAAgB,IAAI,SAAS,kBAAkB,EAAE;IAIjD,OAAO,CAAC,OAAO;IAef,OAAO,CAAC,eAAe;IAOvB,OAAO,CAAC,kBAAkB;IAiB1B,OAAO,CAAC,kBAAkB;IAS1B,OAAO,CAAC,eAAe;IAoBvB,OAAO,CAAC,gBAAgB;CAGzB;AAED,wBAAgB,wBAAwB,CACtC,OAAO,EAAE,oBAAoB,EAC7B,UAAU,GAAE,SAAS,EAAO,EAC5B,OAAO,GAAE,kBAAuB,GAC/B,WAAW,CAUb"}