codex-overleaf-link 2.2.1 → 2.3.1

package/extension/styles/panel.css

@@ -2686,13 +2686,23 @@
 }
 
 #codex-overleaf-panel .run-math--fallback {
-  border-radius: 4px;
-  background: var(--tl-surface-2);
-  color: var(--tl-review);
-  padding: 1px 4px;
+  border-bottom: 1px dotted var(--tl-fg-4);
+  border-radius: 0;
+  background: transparent;
+  color: inherit;
+  padding: 0 2px;
   white-space: pre-wrap;
   overflow-wrap: anywhere;
-  font: 0.96em/1.4 ui-monospace, SFMono-Regular, Menlo, Consolas, monospace;
+  font: inherit;
+}
+
+#codex-overleaf-panel .run-math--display.run-math--fallback {
+  border: 1px solid var(--tl-border);
+  border-left: 3px solid var(--tl-fg-4);
+  border-radius: 6px;
+  background: var(--tl-surface-2);
+  padding: 8px 10px;
+  text-align: left;
 }
 
 #codex-overleaf-panel .run-stream[data-stream-role="reasoning"] .run-stream-text {
@@ -3173,12 +3183,12 @@
   white-space: nowrap;
 }
 
-/* Mode switch (Ask/Suggest/Auto) — a real segmented control: a bordered track
-   holding three choices. Inactive choices are muted; hover is a light fill
+/* Mode switch (Ask/Auto) — a real segmented control: a bordered track
+   holding two choices. Inactive choices are muted; hover is a light fill
    (distinct from the active accent-wash fill) so hover never reads as active. */
 #codex-overleaf-panel .codex-mode-switch {
   display: grid;
-  grid-template-columns: repeat(3, minmax(0, 1fr));
+  grid-template-columns: repeat(2, minmax(0, 1fr));
   gap: 2px;
   min-width: 0;
   padding: 2px;

package/native-host/src/codexPromptAssembly.js

@@ -64,7 +64,7 @@ function buildCodexTurnPrompt(options = {}) {
     'Mode for this turn:',
     `- ${context.mode}`,
     '- ask: inspect and explain only; do not edit files.',
-    '- confirm/auto: edit the local workspace directly when the request calls for changes. The browser bridge handles review, confirmation, deletion approval, and syncing back to Overleaf.',
+    '- auto: edit the local workspace directly when the request calls for changes. The browser bridge handles authorization, deletion approval, and syncing back to Overleaf.',
     '',
     'Write expectation for this turn:',
     formatWriteExpectation({

package/native-host/src/codexRuntimeIdentity.js

@@ -7,7 +7,7 @@ const { spawnSync } = require('node:child_process');
 
 const RUNTIME_ENV_KEY = 'CODEX_OVERLEAF_CODEX_RUNTIME_JSON';
 const MAX_CODEX_CANDIDATES = 8;
-const VERSION_TIMEOUT_MS = 700;
+const VERSION_TIMEOUT_MS = 3000;
 
 function buildCodexRuntimeIdentity(options = {}) {
   const env = options.env || process.env;

package/native-host/src/codexSessionRunner.js

@@ -10,7 +10,10 @@ const { computeTextPatches } = require('./textPatch');
 const { buildCodexHomeEnv } = require('./codexHome');
 const { buildCodexSpeedArgs } = require('./codexArgs');
 const { truncateText } = require('./debugLog');
-const { enforceNativeOkResponseBudget } = require('./nativeResponseBudget');
+const {
+  estimateBase64DecodedBytes,
+  reduceTaskResult
+} = require('./nativeTransportEnvelope');
 const { buildCodexTurnPrompt: buildCodexPromptParts } = require('./codexPromptAssembly');
 const { evaluateSkillCommand } = require('./commandApproval');
 const {
@@ -186,7 +189,7 @@ async function runCodexSession({ params = {}, env = process.env, emit = () => {}
   throwIfAborted(signal);
 
   if (skillInstallTurn) {
-    return enforceNativeOkResponseBudget({
+    return reduceTaskResult({
       status: 'completed',
       projectId: mirror.projectKey,
       workspacePath: mirror.workspacePath,
@@ -245,7 +248,7 @@ async function runCodexSession({ params = {}, env = process.env, emit = () => {}
       ignoredChangedCount: rawSyncChanges.length,
       ignoredUnsupportedCount: unsupportedChanges.length
     }, rawSyncChanges.length || unsupportedChanges.length ? 'warning' : 'completed');
-    return enforceNativeOkResponseBudget({
+    return reduceTaskResult({
       status: 'completed',
       projectId: mirror.projectKey,
       workspacePath: mirror.workspacePath,
@@ -266,7 +269,7 @@ async function runCodexSession({ params = {}, env = process.env, emit = () => {}
     }
     return change;
   });
-  const response = enforceNativeOkResponseBudget({
+  const response = reduceTaskResult({
     status: 'completed',
     projectId: mirror.projectKey,
     workspacePath: mirror.workspacePath,
@@ -370,15 +373,6 @@ function normalizeTurnAttachments(value) {
   return result;
 }
 
-function estimateBase64DecodedBytes(value) {
-  const clean = String(value || '').replace(/\s+/g, '');
-  if (!clean) {
-    return 0;
-  }
-  const padding = clean.endsWith('==') ? 2 : clean.endsWith('=') ? 1 : 0;
-  return Math.max(0, Math.floor(clean.length * 3 / 4) - padding);
-}
-
 function sanitizeAttachmentFileName(value) {
   const basename = String(value || '')
     .replace(/\0/g, '')

package/native-host/src/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 'use strict';
 
-const { decodeFrames, encodeMessage } = require('./nativeMessaging');
+const { decodeInputFrames, encodeOutputFrame } = require('./nativeTransportEnvelope');
 const { logDebug } = require('./debugLog');
 const { handleRequest } = require('./taskRunner');
 const { abortAllActiveOperations, getActiveNativeWorkState } = require('./taskRunnerRuntime');
@@ -21,7 +21,7 @@ process.stdin.on('data', chunk => {
 
   let decoded;
   try {
-    decoded = decodeFrames(buffered);
+    decoded = decodeInputFrames(buffered);
   } catch (error) {
     writeResponse({
       ok: false,
@@ -111,24 +111,7 @@ function writeResponse(response) {
     logDebug('stdout.write_skipped', { ok: response?.ok, code: response?.error?.code });
     return false;
   }
-  let frame;
-  try {
-    frame = encodeMessage(response);
-  } catch (error) {
-    const fallback = buildOversizeResponseFallback(response, error);
-    try {
-      frame = encodeMessage(fallback);
-    } catch (fallbackError) {
-      frame = encodeMessage({
-        id: response?.id,
-        ok: false,
-        error: {
-          code: 'native_response_too_large',
-          message: truncateForNativeFrame(fallbackError.message || error.message || 'Native response exceeded the browser frame limit.', 800)
-        }
-      });
-    }
-  }
+  const frame = encodeOutputFrame(response);
   logDebug('stdout.write', { bytes: frame.length, ok: response?.ok, code: response?.error?.code });
   try {
     process.stdout.write(frame, error => {
@@ -168,55 +151,6 @@ function handleNativeDisconnect(source, error) {
   exitTimer.unref?.();
 }
 
-function buildOversizeResponseFallback(response, error) {
-  if (response?.event) {
-    const event = response.event || {};
-    return {
-      id: response.id,
-      ok: true,
-      event: {
-        type: event.type || 'native.event.truncated',
-        title: truncateForNativeFrame(event.title || 'Native event was truncated', 500),
-        status: event.status || 'warning',
-        detail: {
-          code: 'native_event_truncated',
-          reason: truncateForNativeFrame(error?.message || 'Native event exceeded the browser frame limit.', 800),
-          originalType: truncateForNativeFrame(event.type || '', 160),
-          originalTitle: truncateForNativeFrame(event.title || '', 500),
-          originalDetailBytes: stringByteLength(safeJsonStringify(event.detail))
-        },
-        timestamp: event.timestamp || new Date().toISOString()
-      }
-    };
-  }
-
-  return {
-    id: response?.id,
-    ok: false,
-    error: {
-      code: 'native_response_too_large',
-      message: truncateForNativeFrame(error?.message || 'Native response exceeded the browser frame limit.', 800),
-      originalOk: response?.ok === true
-    }
-  };
-}
-
-function truncateForNativeFrame(value, limit) {
-  const text = String(value || '');
-  if (text.length <= limit) {
-    return text;
-  }
-  return `${text.slice(0, Math.max(0, limit - 24))}... [truncated]`;
-}
-
-function safeJsonStringify(value) {
-  try {
-    return JSON.stringify(value);
-  } catch (_error) {
-    return '';
-  }
-}
-
 function summarizeRequest(message) {
   const params = message?.params || {};
   return {

package/native-host/src/managedInstall.js

@@ -54,14 +54,19 @@ function installManagedDistribution(options = {}) {
   const unique = process.pid + '-' + Date.now() + '-' + Math.random().toString(16).slice(2);
   const extensionStage = path.join(parent, '.extension.staging-' + unique);
   const extensionRollback = path.join(parent, '.extension.rollback-' + unique);
-  buildManagedExtensionTree({
-    packageRoot,
-    targetRoot: extensionStage,
-    version,
-    releaseRef,
-    releaseChannel,
-    allowedFiles: options.allowedFiles
-  });
+  try {
+    buildManagedExtensionTree({
+      packageRoot,
+      targetRoot: extensionStage,
+      version,
+      releaseRef,
+      releaseChannel,
+      allowedFiles: options.allowedFiles
+    });
+  } catch (error) {
+    safeRemove(extensionStage);
+    throw error;
+  }
 
   const previousExtensionMarker = readManagedMarker(extensionRoot, EXTENSION_MARKER);
   if (fs.existsSync(extensionRoot) && !isManagedMarker(previousExtensionMarker, 'extension')) {
@@ -145,6 +150,7 @@ function buildManagedExtensionTree(options = {}) {
   copyPackageTree(packageRoot, 'extension/vendor', targetRoot, 'runtime/vendor', { allowedFiles });
   copyPackageFile(packageRoot, 'extension/runtime-manifest.json', targetRoot, 'runtime/runtime-manifest.json', allowedFiles);
   rewriteManagedRuntimeAssetPaths(targetRoot);
+  assertManagedRuntimeManifestFiles(targetRoot);
   fs.writeFileSync(path.join(targetRoot, EXTENSION_MARKER), JSON.stringify({
     managedBy: MANAGED_BY,
     kind: 'extension',
@@ -413,6 +419,41 @@ function resolveForPlatform(value, platformPath) {
   return platformPath.isAbsolute(value) ? value : platformPath.resolve(value);
 }
 
+function assertManagedRuntimeManifestFiles(targetRoot) {
+  const runtimeRoot = path.join(targetRoot, 'runtime');
+  const runtimeManifest = readJson(path.join(runtimeRoot, 'runtime-manifest.json'));
+  const runtimeFiles = [
+    ...(Array.isArray(runtimeManifest.js) ? runtimeManifest.js : []),
+    ...(Array.isArray(runtimeManifest.css) ? runtimeManifest.css : [])
+  ];
+  const missing = [];
+
+  for (const relativePath of runtimeFiles) {
+    if (
+      typeof relativePath !== 'string'
+      || relativePath.length === 0
+      || path.isAbsolute(relativePath)
+      || relativePath.split(/[\\/]+/).includes('..')
+    ) {
+      throw new Error('Managed runtime manifest contains an unsafe path: ' + String(relativePath));
+    }
+    const filePath = path.resolve(runtimeRoot, relativePath);
+    const relativeToRuntime = path.relative(runtimeRoot, filePath);
+    if (
+      relativeToRuntime.startsWith('..')
+      || path.isAbsolute(relativeToRuntime)
+      || !fs.existsSync(filePath)
+      || !fs.statSync(filePath).isFile()
+    ) {
+      missing.push(relativePath);
+    }
+  }
+
+  if (missing.length > 0) {
+    throw new Error('Managed runtime is incomplete; missing file(s): ' + missing.join(', '));
+  }
+}
+
 function safeRemove(target) {
   fs.rmSync(target, { recursive: true, force: true });
 }
@@ -425,6 +466,7 @@ module.exports = {
   EXTENSION_MARKER,
   MANAGED_BY,
   NATIVE_MARKER,
+  assertManagedRuntimeManifestFiles,
   assertSafeManagedRoot,
   buildManagedExtensionTree,
   installManagedDistribution,

package/native-host/src/nativeTransportEnvelope.js

@@ -0,0 +1,148 @@
+'use strict';
+
+const {
+  MAX_NATIVE_OUTPUT_MESSAGE_BYTES,
+  decodeFrames,
+  encodeMessage
+} = require('./nativeMessaging');
+const nativeQuotas = require('./nativeQuotas');
+const {
+  enforceNativeOkResponseBudget,
+  truncateUtf8Text
+} = require('./nativeResponseBudget');
+
+function decodeInputFrames(buffer) {
+  return decodeFrames(buffer);
+}
+
+function encodeOutputFrame(response) {
+  try {
+    return encodeMessage(response);
+  } catch (error) {
+    const fallback = buildOversizeResponseFallback(response, error);
+    try {
+      return encodeMessage(fallback);
+    } catch (fallbackError) {
+      return encodeMessage(buildFinalOversizeError(response, fallbackError, error));
+    }
+  }
+}
+
+  function getRequestQuotaViolation(request) {
+    if (typeof request?.id === "string" && request.id.length > 160) {
+      return {
+        field: "id",
+        limit: 160,
+        actual: request.id.length,
+        reason: "request id is too large"
+      };
+    }
+    return nativeQuotas.validateNativeRequestQuotas(request);
+  }
+
+function reduceTaskResult(result) {
+  return enforceNativeOkResponseBudget(result);
+}
+
+function buildOversizeResponseFallback(response, error) {
+  if (response?.event) {
+    const event = response.event || {};
+    return {
+      id: boundedResponseId(response.id),
+      ok: true,
+      event: {
+        type: event.type || 'native.event.truncated',
+        title: truncateForNativeFrame(event.title || 'Native event was truncated', 500),
+        status: event.status || 'warning',
+        detail: {
+          code: 'native_event_truncated',
+          reason: truncateForNativeFrame(
+            error?.message || 'Native event exceeded the browser frame limit.',
+            800
+          ),
+          originalType: truncateForNativeFrame(event.type || '', 160),
+          originalTitle: truncateForNativeFrame(event.title || '', 500),
+          originalDetailBytes: measureJsonBytes(event.detail)
+        },
+        timestamp: event.timestamp || new Date().toISOString()
+      }
+    };
+  }
+
+  return {
+    id: boundedResponseId(response?.id),
+    ok: false,
+    error: {
+      code: 'native_response_too_large',
+      message: truncateForNativeFrame(
+        error?.message || 'Native response exceeded the browser frame limit.',
+        800
+      ),
+      originalOk: response?.ok === true
+    }
+  };
+}
+
+function buildFinalOversizeError(response, fallbackError, originalError) {
+  return {
+    id: boundedResponseId(response?.id),
+    ok: false,
+    error: {
+      code: 'native_response_too_large',
+      message: truncateForNativeFrame(
+        fallbackError?.message
+          || originalError?.message
+          || 'Native response exceeded the browser frame limit.',
+        800
+      )
+    }
+  };
+}
+
+function boundedResponseId(value) {
+  if (typeof value === 'number' && Number.isFinite(value)) {
+    return value;
+  }
+  return truncateForNativeFrame(value, 160);
+}
+
+function measureJsonBytes(value) {
+  try {
+    return Buffer.byteLength(JSON.stringify(value), 'utf8');
+  } catch (_error) {
+    return 0;
+  }
+}
+
+function truncateForNativeFrame(value, maxChars) {
+  const text = String(value || '');
+  if (text.length <= maxChars) {
+    return text;
+  }
+  return `${text.slice(0, Math.max(0, maxChars - 24))}... [truncated]`;
+}
+
+module.exports = {
+  MAX_NATIVE_OUTPUT_MESSAGE_BYTES,
+  NATIVE_REQUEST_QUOTAS: nativeQuotas.NATIVE_REQUEST_QUOTAS,
+  buildOversizeResponseFallback,
+  decodeInputFrames,
+  encodeOutputFrame,
+  estimateBase64DecodedBytes: nativeQuotas.estimateBase64DecodedBytes,
+  firstQuotaViolation: nativeQuotas.firstQuotaViolation,
+  getRequestQuotaViolation,
+  measureJsonBytes,
+  measureOperationPayloads: nativeQuotas.measureOperationPayloads,
+  quotaViolation: nativeQuotas.quotaViolation,
+  reduceTaskResult,
+  truncateForNativeFrame,
+  truncateUtf8Text,
+  validateAttachmentQuota: nativeQuotas.validateAttachmentQuota,
+  validateCompileLogQuota: nativeQuotas.validateCompileLogQuota,
+  validateFilePayloadQuota: nativeQuotas.validateFilePayloadQuota,
+  validateOperationListQuota: nativeQuotas.validateOperationListQuota,
+  validateOperationPayloadQuota: nativeQuotas.validateOperationPayloadQuota,
+  validatePatchFileTextQuota: nativeQuotas.validatePatchFileTextQuota,
+  validateProjectSnapshotQuota: nativeQuotas.validateProjectSnapshotQuota,
+  validateSkillContentQuota: nativeQuotas.validateSkillContentQuota
+};

package/native-host/src/taskRunnerRuntime.js

@@ -3,7 +3,6 @@
 const { buildCodexRuntimeEvent } = require('./codexRuntimeIdentity');
 
 const { spawn } = require('node:child_process');
-const crypto = require('node:crypto');
 const { buildOperationSummary, splitDeletePlan } = require('../../extension/src/shared/summary');
 const {
   MIN_COMPATIBLE_EXTENSION_VERSION,
@@ -26,10 +25,10 @@ const { getNativeRuntimePlatform, summarizeNativeEnvironment } = require('./nati
 const {
   NATIVE_REQUEST_QUOTAS,
   firstQuotaViolation,
-  validateNativeRequestQuotas,
+  getRequestQuotaViolation,
   validateOperationListQuota,
   validateOperationPayloadQuota
-} = require('./nativeQuotas');
+} = require('./nativeTransportEnvelope');
 const { version: PACKAGE_VERSION } = require('../../package.json');
 
 const activeProjectLocks = new Map();
@@ -40,8 +39,6 @@ const activeRunEntries = new Map();
 // Overleaf tab was reloaded — the requestId lived in content-side JS state
 // and is gone, but the native-host-side controller is still running).
 const activeRunByProject = new Map();
-const pendingPlans = new Map();
-const PENDING_PLAN_TTL_MS = 30 * 60 * 1000;
 const CODEX_RUN_PASSTHROUGH_ERROR_CODES = new Set(['thread_resume_failed', 'codex_no_usable_result']);
 
 async function handleRequest(request, env = process.env, emit = () => {}) {
@@ -49,7 +46,7 @@ async function handleRequest(request, env = process.env, emit = () => {}) {
     return errorResponse(undefined, 'invalid_request', 'Request must be an object');
   }
 
-  const quotaError = validateNativeRequestQuotas(request);
+  const quotaError = getRequestQuotaViolation(request);
   if (quotaError) {
     return quotaErrorResponse(request.id, quotaError);
   }
@@ -136,7 +133,7 @@ async function handleRequest(request, env = process.env, emit = () => {}) {
   }
 
   if (request.method === 'task.confirm') {
-    return handleTaskConfirm(request);
+    return errorResponse(request.id, 'suggest_mode_removed', 'Suggest mode has been removed. Reload the extension and choose Ask or Auto.');
   }
 
   return errorResponse(request.id, 'method_not_found', `Unknown method: ${request.method}`);
@@ -716,8 +713,11 @@ async function handleTaskRun(request, env, emit) {
   const params = request.params || {};
   const mode = params.mode;
 
-  if (!['ask', 'confirm', 'auto'].includes(mode)) {
-    return errorResponse(request.id, 'invalid_mode', 'Mode must be "ask", "confirm", or "auto"');
+  if (mode === 'confirm') {
+    return errorResponse(request.id, 'suggest_mode_removed', 'Suggest mode has been removed. Choose Ask or Auto.');
+  }
+  if (!['ask', 'auto'].includes(mode)) {
+    return errorResponse(request.id, 'invalid_mode', 'Mode must be "ask" or "auto"');
   }
 
   if (mode === 'auto' && !params.checkpoint?.ok && !isVerifiedReviewing(params.reviewing)) {
@@ -805,24 +805,6 @@ function isVerifiedReviewing(reviewing) {
   return reviewing?.ok === true && reviewing.status !== 'manual-override';
 }
 
-async function handleTaskConfirm(request) {
-  const planId = request.params?.planId;
-  purgeExpiredPendingPlans();
-  if (!planId || !pendingPlans.has(planId)) {
-    return errorResponse(request.id, 'plan_not_found', 'No pending task plan matched the supplied planId');
-  }
-
-  const plan = pendingPlans.get(planId);
-  pendingPlans.delete(planId);
-
-  return okResponse(request.id, {
-    status: 'confirmed',
-    notes: plan.notes || '',
-    userReport: plan.userReport,
-    operations: plan.operations
-  });
-}
-
 function buildDefaultTaskResult(mode, operations, params = {}) {
   if (mode === 'ask') {
     return {
@@ -836,16 +818,6 @@ function buildDefaultTaskResult(mode, operations, params = {}) {
 
   const summary = buildOperationSummary(operations);
 
-  if (mode === 'confirm') {
-    return {
-      status: 'requires_task_confirmation',
-      summary,
-      notes: '',
-      userReport: buildDefaultUserReport(mode, operations, params),
-      operations
-    };
-  }
-
   const split = splitDeletePlan(operations);
   if (split.needsConfirmation.length > 0) {
     return {
@@ -889,7 +861,7 @@ function normalizeAgentResult(mode, result, params = {}) {
   };
 
   if (!normalized.status) {
-    normalized.status = mode === 'confirm' ? 'requires_task_confirmation' : 'completed';
+    normalized.status = 'completed';
   }
 
   return normalized;
@@ -911,40 +883,7 @@ function prepareResultForResponse(mode, result) {
     }
   }
 
-  const confirmOperations = mode === 'confirm' ? collectResultOperations(result) : [];
-  if (confirmOperations.length > 0) {
-    result = {
-      ...result,
-      status: 'requires_task_confirmation',
-      summary: buildOperationSummary(confirmOperations),
-      operations: confirmOperations
-    };
-  }
-
-  if (mode !== 'confirm' || result.status !== 'requires_task_confirmation') {
-    return result;
-  }
-
-  purgeExpiredPendingPlans();
-  const planId = `plan_${crypto.randomUUID()}`;
-  pendingPlans.set(planId, {
-    createdAt: Date.now(),
-    expiresAt: Date.now() + PENDING_PLAN_TTL_MS,
-    notes: result.notes || '',
-    userReport: result.userReport,
-    operations: Array.isArray(result.operations) ? result.operations : []
-  });
-
-  const {
-    operations: _operations,
-    pendingOperations: _pendingOperations,
-    deletePlan: _deletePlan,
-    ...redacted
-  } = result;
-  return {
-    ...redacted,
-    planId
-  };
+  return result;
 }
 
 function validateTaskResultOperationQuotas(result = {}) {
@@ -965,7 +904,7 @@ function buildDefaultUserReport(mode, operations = [], params = {}) {
 
   return {
     conclusion: hasOperations
-      ? reportText(locale, 'Codex prepared proposed changes and is waiting for confirmation or writeback.', 'Codex 已准备好建议修改，等待确认或写入。')
+      ? reportText(locale, 'Codex prepared changes for writeback.', 'Codex 已准备好写回修改。')
       : reportText(locale, 'This task completed without writing Overleaf files.', '这轮任务已完成，没有写入 Overleaf 文件。'),
     checked,
     findings: [],
@@ -975,7 +914,7 @@ function buildDefaultUserReport(mode, operations = [], params = {}) {
       ? ''
       : (mode === 'ask' ? reportText(locale, 'This run was Ask mode.', '这轮是只问不改。') : ''),
     nextStep: hasOperations
-      ? reportText(locale, 'Review and confirm the proposed changes before writing them to Overleaf.', '请确认修改方案后写入 Overleaf。')
+      ? reportText(locale, 'Review the written changes in Overleaf.', '请在 Overleaf 中检查写入的修改。')
       : reportText(locale, 'Continue the conversation, or add more @context and run another check.', '可以继续追问，或加入更多 @context 后再检查。')
   };
 }
@@ -1193,14 +1132,6 @@ function runExternalAgent(agentSpec, params, emit = () => {}, options = {}) {
   });
 }
 
-function purgeExpiredPendingPlans(now = Date.now()) {
-  for (const [planId, plan] of pendingPlans.entries()) {
-    if (Number.isFinite(plan?.expiresAt) && plan.expiresAt <= now) {
-      pendingPlans.delete(planId);
-    }
-  }
-}
-
 function parsePositiveInteger(value, fallback) {
   const parsed = Number.parseInt(value, 10);
   return Number.isFinite(parsed) && parsed > 0 ? parsed : fallback;
@@ -1295,6 +1226,5 @@ module.exports = {
   buildDefaultTaskResult,
   getActiveNativeWorkState,
   handleRequest,
-  parseAgentEventLines,
-  purgeExpiredPendingPlans
+  parseAgentEventLines
 };