@jsonstudio/llms 0.6.1449 → 0.6.1643

package/dist/tools/apply-patch/execution-capturer.js

@@ -5,9 +5,13 @@ import { createHash } from 'crypto';
 /**
  * Captures apply_patch execution failures (tool role message contents) into errorsamples.
  *
- * These failures are not schema/shape failures; they happen when the tool is executed
- * (e.g. file missing, context mismatch). We capture for debugging and compatibility
- * tracking, but do NOT attempt semantic repair.
+ * IMPORTANT (RouteCodex policy):
+ * - Only keep apply_patch "shape" failures in errorsamples.
+ * - Context/execution failures (file missing, context mismatch, expected lines mismatch)
+ *   are noisy and not actionable, so they are NOT captured by default.
+ *
+ * Explicit opt-in for debugging:
+ *   ROUTECODEX_ERRORSAMPLES_APPLY_PATCH_EXEC=1
  *
  * Default destination:
  *   ~/.routecodex/errorsamples/apply_patch_exec/<error-type>/
@@ -18,6 +22,7 @@ import { createHash } from 'crypto';
  * Note: if ROUTECODEX_ERRORSAMPLES_DIR is set, it is used as the base root.
  */
 const MAX_SAMPLES_PER_TYPE = 250;
+const EXEC_ERRORSAMPLES_ENABLED = String(process?.env?.ROUTECODEX_ERRORSAMPLES_APPLY_PATCH_EXEC || '').trim() === '1';
 function resolveSamplesRoot() {
     const explicit = String(process?.env?.ROUTECODEX_APPLY_PATCH_EXEC_DIR || '').trim();
     if (explicit)
@@ -47,6 +52,13 @@ function classifyExecutionFailure(content) {
         return { errorType: 'parse_failed', message: msg };
     return { errorType: 'unknown', message: msg };
 }
+function isContextExecutionFailureType(errorType) {
+    const t = String(errorType || '').trim().toLowerCase();
+    return (t === 'read_file_failed' ||
+        t === 'file_not_found' ||
+        t === 'context_not_found' ||
+        t === 'expected_lines_not_found');
+}
 function stableSampleId(fields) {
     const key = `${String(fields.errorType || 'unknown')}:${String(fields.errorMessage || '')}:` +
         `${String(fields.toolCallId || '')}:${String(fields.toolCallArgs || '')}:${String(fields.requestId || '')}:${String(fields.mode || '')}`;
@@ -54,7 +66,14 @@ function stableSampleId(fields) {
 }
 export function captureApplyPatchExecutionFailure(content, context) {
     try {
+        if (!EXEC_ERRORSAMPLES_ENABLED)
+            return;
         const { errorType, message } = classifyExecutionFailure(content);
+        // Keep only non-context types (i.e. parse/format issues). Context mismatches are not captured.
+        if (isContextExecutionFailureType(errorType))
+            return;
+        if (String(errorType || '').trim().toLowerCase() === 'unknown')
+            return;
         const root = resolveSamplesRoot();
         const safeType = String(errorType || 'unknown').replace(/[^a-z0-9-]/gi, '_');
         const typeDir = path.join(root, safeType);
@@ -103,6 +122,8 @@ export function captureApplyPatchExecutionFailure(content, context) {
 }
 export function captureApplyPatchExecutionFailuresFromProcessedRequest(processedRequest) {
     try {
+        if (!EXEC_ERRORSAMPLES_ENABLED)
+            return;
         const anyReq = processedRequest;
         const messages = Array.isArray(anyReq?.messages) ? anyReq.messages : [];
         if (!messages.length)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsonstudio/llms",
-  "version": "0.6.1449",
+  "version": "0.6.1643",
   "type": "module",
   "main": "dist/index.js",
   "module": "dist/index.js",
@@ -27,7 +27,8 @@
     "test:looprt:gemini": "npm run build && node scripts/tests/loop-rt-gemini.mjs",
     "replay:responses:chat-sse": "node scripts/exp3-responses-sse-to-chat-sse.mjs",
     "replay:responses:loop": "node scripts/exp4-responses-sse-loop.mjs",
-    "test:virtual-router": "npm run build:dev && node scripts/tests/virtual-router-prefer-autoclear.mjs && node scripts/tests/virtual-router-dry-run.mjs && node scripts/tests/virtual-router-context.mjs && node scripts/tests/virtual-router-antigravity-alias-pin.mjs && node scripts/tests/virtual-router-antigravity-auth-verify.mjs",
+    "test:quota-manager": "npm run build:dev && node scripts/tests/quota-manager-402-reset.mjs",
+    "test:virtual-router": "npm run build:dev && node scripts/tests/virtual-router-prefer-autoclear.mjs && node scripts/tests/virtual-router-dry-run.mjs && node scripts/tests/virtual-router-context.mjs && node scripts/tests/virtual-router-antigravity-alias-pin.mjs && node scripts/tests/virtual-router-antigravity-session-binding.mjs && node scripts/tests/virtual-router-antigravity-auth-verify.mjs",
     "test:virtual-router-health": "npm run build:dev && node scripts/tests/virtual-router-health.mjs",
     "test:golden": "npm run build:ci && node scripts/tests/chat-golden-roundtrip.mjs && node scripts/tests/anthropic-golden-roundtrip.mjs",
     "test:coverage": "node scripts/run-ci-coverage.mjs"