codex-overleaf-link 2.2.1 → 2.3.1

package/extension/src/content/writebackOrchestrator.js

@@ -25,7 +25,6 @@
   let appendOperationsPreview;
   let appendPartialWritebackWarning;
   let appendApplyResult;
-  let renderDiffReview;
   let renderReadOnlyDiffReview;
   let showPluginConfirm;
   let callPageBridge;
@@ -53,11 +52,21 @@
   let buildReviewingBlockedApplyResult;
   let ensureReviewingBeforeWrite;
   let confirmBinaryOperations;
+  let compileAdapter;
   let filterSyncChangesByOperations;
   let writebackController;
   let RUN_SNAPSHOT_ZIP_TIMEOUT_MS;
   let getState;
   let getCurrentRunView;
+  let onMirrorRefreshSettled;
+  let writebackSettlement;
+  let injectedWritebackSettlement;
+
+  function buildSettlement(input = {}) {
+    return writebackSettlement?.settle instanceof Function
+      ? writebackSettlement.settle(input)
+      : null;
+  }
 
   async function applySyncChangesToOverleaf(syncChanges = [], project = {}, options = {}) {
     const runMode = options.mode || getState().mode;
@@ -86,13 +95,17 @@
         applyResults: [],
         mode: runMode,
         nextStep: tx(
-          'Switch to Suggest or Auto only when you want Codex to edit files.',
-          '只有希望 Codex 修改文件时，才需要切换到建议修改或自动写入。'
+          'Switch to Auto only when you want Codex to edit files.',
+          '只有希望 Codex 修改文件时，才需要切换到自动写入。'
         )
       });
       return {
         summaryLine: assistantMessage || tx('Ask mode completed without Overleaf changes', 'Ask 模式已完成，未修改 Overleaf'),
         hasSkippedOperations: false,
+        settlement: buildSettlement({
+          operations: [],
+          applyResult: { ok: true, applied: [], skipped: [] }
+        }),
         audit: buildAuditSummaryFromApply({
           operations: [],
           resultStatus: 'ask_ignored_local_changes',
@@ -138,16 +151,23 @@
         conclusion: assistantMessage || tx('Codex finished locally. There are no changes to sync back to Overleaf.', 'Codex 已完成本地处理，没有需要同步回 Overleaf 的改动。'),
         status: runMode === 'ask' ? tr('modeAsk') : 'completed',
         operations: [],
-        applyResults: [],
-        unchangedReason: assistantMessage
-          ? tx('No file changes need to sync back to Overleaf.', '没有产生需要同步回 Overleaf 的文件改动。')
-          : formatUnsupportedLocalChangeSummary(unsupportedChanges),
+        applyResults: additionalSkippedEntries.length ? [{ ok: false, applied: [], skipped: additionalSkippedEntries }] : [],
+        unchangedReason: additionalSkippedEntries.find(item => item?.result?.reason)?.result.reason
+          || (assistantMessage ? tx('No file changes need to sync back to Overleaf.', '没有产生需要同步回 Overleaf 的文件改动。') : formatUnsupportedLocalChangeSummary(unsupportedChanges)),
         mode: runMode,
-        nextStep: tx('You can continue the conversation, or adjust @context and run again.', '可以继续追问，或调整 @context 后重新运行。')
+        nextStep: additionalSkippedEntries.length ? formatWritebackSkippedNextStep({ ok: false, applied: [], skipped: additionalSkippedEntries }) : tx('You can continue the conversation, or adjust @context and run again.', '可以继续追问，或调整 @context 后重新运行。')
       });
       return {
         summaryLine: assistantMessage || tx('No changes to sync', '没有需要同步的改动'),
         hasSkippedOperations: additionalSkippedEntries.length > 0,
+        settlement: buildSettlement({
+          operations: buildSyncApplyOperations(syncChanges, project),
+          applyResult: {
+            ok: additionalSkippedEntries.length === 0,
+            applied: [],
+            skipped: additionalSkippedEntries
+          }
+        }),
         audit: buildAuditSummaryFromApply({
           operations: buildSyncApplyOperations(syncChanges, project),
           applyResults: additionalSkippedEntries.length ? [{ ok: false, applied: [], skipped: additionalSkippedEntries }] : [],
@@ -157,47 +177,6 @@
       };
     }
 
-    if (runMode === 'confirm') {
-      appendRunEvent({
-        title: tx(
-          `Local Codex produced ${operations.length} change(s). Review the diff before applying.`,
-          `本地 Codex 产生了 ${operations.length} 项改动，请查看差异后确认。`
-        ),
-        status: 'running'
-      });
-      const accepted = await renderDiffReview(visibleSyncChanges);
-      if (!accepted.length) {
-        appendRunEvent({
-          title: tx('Sync cancelled: Overleaf was not modified.', '已取消同步：Overleaf 没有被修改。'),
-          status: 'completed'
-        });
-        appendCompletionReport({
-          conclusion: tx('You cancelled syncing. Local Codex changes were not written back to Overleaf.', '你取消了同步，本地 Codex 改动没有写回 Overleaf。'),
-          status: 'rejected',
-          operations: [],
-          applyResults: [],
-          mode: runMode,
-          nextStep: tx('Run the task again, or switch to Auto if you want changes synced directly.', '可以重新运行任务，或切换到自动写入后再同步。')
-        });
-        return {
-          summaryLine: tx('Sync cancelled', '已取消同步'),
-          hasSkippedOperations: additionalSkippedEntries.length > 0,
-          audit: buildAuditSummaryFromApply({
-            operations,
-            applyResults: additionalSkippedEntries.length ? [{ ok: false, applied: [], skipped: additionalSkippedEntries }] : [],
-            blockedFiles: additionalSkippedEntries.map(item => summarizeOperationForAudit(item.operation, item.result, 'blocked')),
-            resultStatus: 'rejected'
-          })
-        };
-      }
-      operations = buildSyncApplyOperations(accepted, project);
-      const acceptedPathSafety = partitionUnsafeProjectPathOperations(operations);
-      if (acceptedPathSafety.skipped.length) {
-        additionalSkippedEntries.push(...acceptedPathSafety.skipped);
-        operations = acceptedPathSafety.safe;
-      }
-    }
-
     const deleteOperations = operations.filter(operation => operation.type === 'delete');
     if (deleteOperations.length) {
       const approved = await showPluginConfirm({
@@ -239,14 +218,23 @@
         conclusion: assistantMessage || tx('No approved changes were written back to Overleaf.', '没有已确认的改动写回 Overleaf。'),
         status: runMode === 'ask' ? tr('modeAsk') : 'completed',
         operations: [],
-        applyResults: [],
-        unchangedReason: tx('No approved file changes remain to sync back to Overleaf.', '没有剩余已确认的文件改动需要同步回 Overleaf。'),
+        applyResults: additionalSkippedEntries.length ? [{ ok: false, applied: [], skipped: additionalSkippedEntries }] : [],
+        unchangedReason: additionalSkippedEntries.find(item => item?.result?.reason)?.result.reason
+          || tx('No approved file changes remain to sync back to Overleaf.', '没有剩余已确认的文件改动需要同步回 Overleaf。'),
         mode: runMode,
-        nextStep: tx('You can continue the conversation, or adjust @context and run again.', '可以继续追问，或调整 @context 后重新运行。')
+        nextStep: additionalSkippedEntries.length ? formatWritebackSkippedNextStep({ ok: false, applied: [], skipped: additionalSkippedEntries }) : tx('You can continue the conversation, or adjust @context and run again.', '可以继续追问，或调整 @context 后重新运行。')
       });
       return {
         summaryLine: tx('No changes applied', '没有应用改动'),
         hasSkippedOperations: additionalSkippedEntries.length > 0,
+        settlement: buildSettlement({
+          operations: buildSyncApplyOperations(syncChanges, project),
+          applyResult: {
+            ok: additionalSkippedEntries.length === 0,
+            applied: [],
+            skipped: additionalSkippedEntries
+          }
+        }),
         audit: buildAuditSummaryFromApply({
           operations: buildSyncApplyOperations(syncChanges, project),
           applyResults: additionalSkippedEntries.length ? [{ ok: false, applied: [], skipped: additionalSkippedEntries }] : [],
@@ -283,6 +271,7 @@
           '已阻止写入：未确认 Overleaf Reviewing/Track Changes'
         ),
         hasSkippedOperations: true,
+        settlement: buildSettlement({ operations, applyResult: blocked }),
         audit: buildAuditSummaryFromApply({
           operations,
           applyResults: [blocked],
@@ -342,6 +331,7 @@
     // pending promise is exposed so undoRun and the next runTask barrier on
     // it — a mirror.sync landing AFTER an undo would otherwise push the
     // pre-undo snapshot as the local baseline.
+    const mirrorSettlementTarget = getCurrentRunView?.();
     pendingMirrorRefresh = refreshProjectMirrorAfterWriteback(project, applied, saveVerification)
       .catch(error => {
         appendRunEvent({
@@ -351,12 +341,25 @@
           ),
           status: 'failed'
         });
+        return 'failed';
+      })
+      .then(state => {
+        if (typeof onMirrorRefreshSettled !== 'function') return;
+        return onMirrorRefreshSettled({
+          recordId: mirrorSettlementTarget?.recordId || '',
+          sessionId: mirrorSettlementTarget?.sessionId || '',
+          runProjectId: mirrorSettlementTarget?.runProjectId || '',
+          state: state || 'not-attempted'
+        });
       })
       .finally(() => {
         pendingMirrorRefresh = null;
       });
     const compileSummary = appliedPaths.length
-      ? await autoRecompileAfterWriteback(appliedPaths, saveVerification).catch(error => {
+      ? await autoRecompileAfterWriteback(appliedPaths, saveVerification, {
+        autoRecompile: options.autoRecompile,
+        mode: runMode
+      }).catch(error => {
         appendRunEvent({
           title: tx(`Post-write compile failed: ${error.message}`, `写后编译出错：${error.message}`),
           status: 'failed'
@@ -419,6 +422,17 @@
       // Existing call sites that only read `summaryLine` / `hasSkippedOperations`
       // / `audit` are unaffected.
       applied,
+      settlement: buildSettlement({
+        operations,
+        applyResult: applied,
+        saveVerification,
+        mirror: {
+          state: appliedPaths.length && saveVerification?.state === 'verified_saved'
+            ? 'pending'
+            : 'not_started'
+        },
+        compile: compileSummary
+      }),
       audit: buildAuditSummaryFromApply({
         operations,
         applyResults: [applied],
@@ -447,8 +461,8 @@
       if (result?.ok === true && !result?.state) {
         return {
           ...result,
-          ok: true,
-          state: 'verified_saved'
+          ok: false,
+          state: 'verified_quiet'
         };
       }
       if (result?.state === 'unknown_timeout' || result?.state === 'unavailable') {
@@ -522,10 +536,10 @@
 
   async function refreshProjectMirrorAfterWriteback(project = {}, applied = {}, saveVerification = {}) {
     if (saveVerification?.state !== 'verified_saved') {
-      return;
+      return 'not-attempted';
     }
     if (!Array.isArray(applied?.applied) || !applied.applied.length) {
-      return;
+      return 'not-attempted';
     }
 
     if (await tryConfirmMirrorWriteback(applied)) {
@@ -536,7 +550,7 @@
         ),
         status: 'completed'
       });
-      return;
+      return 'complete';
     }
 
     appendRunEvent({
@@ -567,7 +581,7 @@
         ),
         status: 'failed'
       });
-      return;
+      return 'failed';
     }
 
     const syncedProject = mergeVerifiedAppliedFiles(freshProject, project, applied);
@@ -584,7 +598,7 @@
         title: tx('Local Codex workspace refreshed. The next run will start from the latest Overleaf content.', '已刷新本地 Codex workspace，下一轮会从最新 Overleaf 内容开始。'),
         status: 'completed'
       });
-      return;
+      return 'complete';
     }
 
     const message = response?.error?.message || tx('native host did not respond', 'native host 没有响应');
@@ -595,6 +609,7 @@
       ),
       status: 'failed'
     });
+    return 'failed';
   }
 
   function mergeVerifiedAppliedFiles(freshProject = {}, originalProject = {}, applied = {}) {
@@ -615,11 +630,12 @@
     return writebackController.formatUnsupportedLocalChangeSummary(changes, getLocale());
   }
 
-  async function autoRecompileAfterWriteback(writtenPaths = [], saveVerification = {}) {
-    if (getState().autoRecompile === false) return null;
-    if (getState().mode === 'ask') return null;
+  async function autoRecompileAfterWriteback(writtenPaths = [], saveVerification = {}, options = {}) {
+    if (options.autoRecompile === false
+      || (options.autoRecompile === undefined && getState().autoRecompile === false)) return null;
+    if ((options.mode || getState().mode) === 'ask') return null;
 
-    const CompileAdapter = window.CodexOverleafCompileAdapter;
+    const CompileAdapter = compileAdapter;
     if (!CompileAdapter) return null;
 
     const hasCompilableFile = writtenPaths.some(filePath => CompileAdapter.isCompilableFile(filePath));
@@ -642,7 +658,8 @@
       const result = await callPageBridge('triggerCompile', {
         preferUiClick: true,
         waitForSaveMs: 5000,
-        requireVerifiedSave: saveVerification?.state === 'verified_saved'
+        requireVerifiedSave: saveVerification?.state === 'verified_saved',
+        runProjectId: getCurrentRunView()?.runProjectId || getCurrentProjectId()
       });
       if (result?.ok) {
         let logResult = null;
@@ -650,7 +667,8 @@
           logResult = await callPageBridge('getCompileLog', {
             triggerIfStale: false,
             maxAgeMs: 30000,
-            waitForSaveMs: 0
+            waitForSaveMs: 0,
+            runProjectId: getCurrentRunView()?.runProjectId || getCurrentProjectId()
           });
         } catch (_error) {
           logResult = null;
@@ -775,7 +793,8 @@
       const result = await callPageBridge('getCompileLog', {
         triggerIfStale: true,
         maxAgeMs: 30000,
-        waitForSaveMs: 5000
+        waitForSaveMs: 5000,
+        runProjectId: getCurrentRunView()?.runProjectId || getCurrentProjectId()
       });
 
       if (!result?.ok) {
@@ -807,7 +826,6 @@
       appendOperationsPreview,
       appendPartialWritebackWarning,
       appendApplyResult,
-      renderDiffReview,
       renderReadOnlyDiffReview,
       showPluginConfirm,
       callPageBridge,
@@ -841,6 +859,9 @@
       getState,
       getCurrentRunView,
     } = deps);
+    onMirrorRefreshSettled = deps.onMirrorRefreshSettled;
+    writebackSettlement = deps.writebackSettlement;
+    compileAdapter = deps.compileAdapter;
     return {
       applySyncChangesToOverleaf,
       resolveCompileLogContext,

package/extension/src/contentScript.js

@@ -4,5 +4,9 @@
   if (!window.CodexOverleafContentRuntime || typeof window.CodexOverleafContentRuntime.init !== 'function') {
     throw new Error('Codex Overleaf content runtime did not load.');
   }
-  window.CodexOverleafContentRuntime.init();
+  const registry = window.CodexOverleafLegacyGlobalRegistry;
+  if (!registry || typeof registry.create !== 'function') {
+    throw new Error('Codex Overleaf dependency registry did not load.');
+  }
+  window.CodexOverleafContentRuntime.init(registry.create(window));
 })();

package/extension/src/page/trackedChangesLifecycle.js

@@ -971,8 +971,15 @@
     }
 
     for (const path of paths) {
-      if (path && getActiveFilePath() !== path) {
-        const opened = await openFileByPath(path);
+      const postContent = postByPath.get(path);
+      const activePathMatches = getActiveFilePath() === path;
+      const activeContentMatches = readActiveEditorText() === postContent;
+      // Overleaf can update the selected tree path before CodeMirror swaps its
+      // document. In that state getActiveFilePath() already names `path`, while
+      // readActiveEditorText() still belongs to the previously viewed file.
+      // Force-reopen the target whenever either identity signal disagrees.
+      if (path && (!activePathMatches || !activeContentMatches)) {
+        const opened = await openFileByPath(path, { force: true });
         if (!opened.ok) {
           return {
             ok: false,
@@ -983,8 +990,7 @@
         }
       }
 
-      const postContent = postByPath.get(path);
-      const postReady = await waitForActiveEditorExpectedText(path, postContent, 2500);
+      const postReady = await waitForActiveEditorExpectedText(path, postContent, 9000);
       if (!postReady.ok) {
         return {
           ok: false,
@@ -1008,7 +1014,12 @@
         result: {
           ok: true,
           method: 'overleaf-editor-undo',
-          undoClicks: result.clicks
+          undoClicks: result.clicks,
+          // undoEditorHistoryUntilContent returns ok only after the active
+          // editor exactly matches this run's pre-write checkpoint. Preserve
+          // that proof for the content-side lifecycle settlement.
+          verified: true,
+          verifiedContent: expectedByPath.get(path)
         }
       });
     }

package/extension/src/page/writebackRouter.js

@@ -1667,13 +1667,17 @@
         continue;
       }
       const key = typeof ref.key === 'string' ? ref.key : '';
-      if (!key || seen.has(key)) {
+      if (!key) {
         continue;
       }
-      seen.add(key);
       const hasPath = typeof ref.path === 'string';
       const hasExplicitPath = hasPath && ref.path.trim() !== '';
       const path = hasPath ? normalizeSafeProjectPath(ref.path) : '';
+      const identity = `${path}\u0000${key}`;
+      if (seen.has(identity)) {
+        continue;
+      }
+      seen.add(identity);
       normalized.push({
         key,
         id: typeof ref.id === 'string' ? ref.id : '',

package/extension/src/pageBridge.js

@@ -2,7 +2,8 @@
   'use strict';
 
   const PAGE_BRIDGE_INSTALL_VERSION = window.CodexOverleafCompatibility?.BUILD_TARGET_VERSION || 'dev';
-  const PAGE_BRIDGE_INSTALL_REVISION = '2026-05-21-editor-readiness-v11';
+  const pageRpcContract = window.CodexOverleafPageRpcContract;
+  const PAGE_BRIDGE_INSTALL_REVISION = pageRpcContract.REVISION;
   if (window.__codexOverleafPageBridgeInstalledVersion === PAGE_BRIDGE_INSTALL_VERSION
     && window.__codexOverleafPageBridgeInstalledRevision === PAGE_BRIDGE_INSTALL_REVISION) {
     return;
@@ -152,10 +153,7 @@
         result = await dispatch(method, params || {});
       }
     } catch (error) {
-      result = {
-        ok: false,
-        error: error.message
-      };
+      result = pageRpcContract.normalizeFailure(method, error);
     }
 
     window.postMessage({
@@ -187,111 +185,62 @@
     return writeCancellationSequence;
   }
 
-  async function dispatch(method, params) {
-    if (method === 'probe') {
-      return probe(params);
-    }
-    if (method === 'cancelActiveWrite') {
+  const pageRpcHandlers = {
+    probe,
+    cancelActiveWrite() {
       const sequence = bumpWriteCancellationSequence();
       return { ok: true, cancelled: true, sequence };
-    }
-    if (method === 'getProjectSnapshot') {
-      return projectSnapshotBridge.getProjectSnapshot(withSnapshotCacheIdentity(params));
-    }
-    if (method === 'getProjectFileList') {
-      return projectSnapshotBridge.getProjectFileList(params);
-    }
-    if (method === 'invalidateProjectSnapshot') {
+    },
+    getProjectSnapshot: params => projectSnapshotBridge.getProjectSnapshot(withSnapshotCacheIdentity(params)),
+    getProjectFileList: params => projectSnapshotBridge.getProjectFileList(params),
+    invalidateProjectSnapshot(params) {
       projectSnapshotBridge.invalidateProjectSnapshot(params);
       return { ok: true };
-    }
-    if (method === 'createCheckpoint') {
-      return createCheckpoint(params.label);
-    }
-    if (method === 'ensureReviewing') {
-      // Gate the Reviewing-mode toggle with the same writeGuard the per-write
-      // path uses. Without this, a mid-flight SPA navigation could leave the
-      // pre-flight toggle flipping Track Changes in a different Overleaf
-      // project than the run was bound to. params.runProjectId is set by
-      // the content-side caller (preflightWriteSafety / ensureReviewingBeforeWrite);
-      // when absent (defensive), the guard short-circuits via its own
-      // missing-runProjectId branch.
-      if (typeof params?.runProjectId === 'string' && params.runProjectId) {
-        const blocked = await writeGuard.runWriteGuard(params);
-        if (blocked) return blocked;
-      }
-      return ensureReviewing(params);
-    }
-    if (method === 'ensureEditing') {
-      if (typeof params?.runProjectId === 'string' && params.runProjectId) {
-        const blocked = await writeGuard.runWriteGuard(params);
-        if (blocked) return blocked;
-      }
-      return ensureEditing(params);
-    }
-    if (method === 'applyOperations') {
-      // Welcome-panel + write-guard:
-      // First gate: runProjectId vs page-side editorProjectId. Runs before
-      // any reviewing / save-state / open-file readiness check so a
-      // mid-run navigation cannot write into a different project. runWriteGuard
-      // is async because it retries the page-side editorProjectId reader with
-      // backoff to ride out Overleaf's hydration window.
-      const blocked = await writeGuard.runWriteGuard(params);
-      if (blocked) return blocked;
-      return applyOperations(params.operations || [], {
+    },
+    createCheckpoint: params => createCheckpoint(params.label),
+    ensureReviewing,
+    ensureEditing,
+    applyOperations: params => applyOperations(params.operations || [], {
         baseFiles: params.baseFiles || null,
         reviewingPolicy: params.reviewingPolicy || '',
         requireReviewing: params.requireReviewing === true,
         requireEditing: params.requireEditing === true,
-        // Propagate runProjectId so the writeback router's defense-in-depth
-        // missing-runProjectId check sees it on this in-bridge dispatch path.
         runProjectId: typeof params.runProjectId === 'string' ? params.runProjectId : ''
-      });
-    }
-    if (method === 'jumpToPosition') {
-      return jumpToPosition(params);
-    }
-    if (method === 'rejectTrackedChanges') {
-      const blocked = await writeGuard.runWriteGuard(params);
-      if (blocked) return blocked;
-      return rejectTrackedChanges(params);
+      }),
+    jumpToPosition,
+    rejectTrackedChanges,
+    acceptTrackedChanges,
+    triggerCompile: params => compileBridge.triggerCompile(params),
+    getCompileLog: params => compileBridge.getCompileLog(params),
+    getCompileState: () => compileBridge.getCompileState(),
+    waitForSaveState,
+    startOtObserver: params => otObserver.start(params),
+    stopOtObserver: () => otObserver.stop(),
+    getOtStatus: () => otObserver.getStatus(),
+    drainOtEvents() {
+      return { ok: true, events: otObserver.drainEvents().map(sanitizeOtEventForContent) };
     }
-    if (method === 'acceptTrackedChanges') {
+  };
+
+  async function dispatch(method, params) {
+    const contract = pageRpcContract.getMethod(method);
+    if (!contract || method === PAGE_BRIDGE_CAPABILITY_METHOD) {
+      return { ok: false, error: `Unknown page bridge method: ${method}` };
+    }
+    const handler = pageRpcHandlers[method];
+    if (typeof handler !== 'function') {
+      return { ok: false, error: `Page bridge handler unavailable: ${method}` };
+    }
+    const hasRunProjectId = typeof params?.runProjectId === 'string'
+      && params.runProjectId.trim().length > 0;
+    if (
+      contract.projectIdentity === 'required'
+      || (contract.projectIdentity === 'optional' && hasRunProjectId)
+    ) {
       const blocked = await writeGuard.runWriteGuard(params);
       if (blocked) return blocked;
-      return acceptTrackedChanges(params);
-    }
-    if (method === 'triggerCompile') {
-      return compileBridge.triggerCompile(params);
-    }
-    if (method === 'getCompileLog') {
-      return compileBridge.getCompileLog(params);
-    }
-    if (method === 'getCompileState') {
-      return compileBridge.getCompileState();
-    }
-    if (method === 'waitForSaveState') {
-      return waitForSaveState(params);
     }
-    if (method === 'startOtObserver') {
-      return otObserver.start(params);
-    }
-    if (method === 'stopOtObserver') {
-      return otObserver.stop();
-    }
-    if (method === 'getOtStatus') {
-      return otObserver.getStatus();
-    }
-    if (method === 'drainOtEvents') {
-      return {
-        ok: true,
-        events: otObserver.drainEvents().map(sanitizeOtEventForContent)
-      };
-    }
-    return {
-      ok: false,
-      error: `Unknown page bridge method: ${method}`
-    };
+    return handler(params);
   }
 
   function sanitizeOtEventForContent(event = {}) {
@@ -468,7 +417,7 @@
       reviewing: window.CodexOverleafReviewing.detectReviewingFromSignals(reviewingSignals),
       reviewingDiagnostics: reviewingSignals.diagnostics,
       editor,
-      capabilities: collectPageCapabilities(editor),
+      capabilities: pageRpcContract.withCapabilityReport(collectPageCapabilities(editor)),
       editorDiagnostics: buildEditorDiagnostics(editor),
       projectDiagnostics: {
         internalRootKeys: collectInternalRootKeys(),

package/extension/src/shared/agentTranscript.js

@@ -5,11 +5,14 @@
       getI18n: function getI18nCjs() { return require('./i18n.js'); }
     });
   } else {
-    // Browser script-tag load order is determined by manifest.json content_scripts;
-    // resolve dependencies lazily so this module can be listed before its deps.
-    root.CodexOverleafAgentTranscript = factory({
-      getFailureReasons: function getFailureReasonsWindow() { return root.CodexOverleafFailureReasons || null; },
-      getI18n: function getI18nWindow() { return root.CodexOverleafI18n || null; }
+    root.CodexOverleafModuleRegistry.define('AgentTranscript', [
+      'FailureReasons',
+      'I18n'
+    ], function createAgentTranscript(FailureReasons, I18n) {
+      return factory({
+        getFailureReasons: function getFailureReasonsWindow() { return FailureReasons; },
+        getI18n: function getI18nWindow() { return I18n; }
+      });
     });
   }
 })(typeof globalThis !== 'undefined' ? globalThis : window, function agentTranscriptFactory(deps) {
@@ -724,10 +727,17 @@
     if (providerError) {
       return providerError;
     }
-    if (/Mode must be "confirm" or "auto"/i.test(text)) {
+    if (/suggest_mode_removed|Suggest mode has been removed/i.test(text)) {
+      return {
+        conclusion: textFor(locale, '建议修改模式已经移除。', 'Suggest mode has been removed.'),
+        nextStep: textFor(locale, '请刷新扩展，然后选择“只问不改”或“自动写入”。', 'Reload the extension, then choose Ask or Auto.'),
+        failureCode: 'suggest_mode_removed'
+      };
+    }
+    if (/Mode must be "(?:confirm|ask)" or "auto"/i.test(text)) {
       return {
         conclusion: textFor(locale, '这轮没有写入：当前是“只问不改”，但这个任务需要写入权限。', 'No files were written: this task needs write access, but the current mode is Ask.'),
-        nextStep: textFor(locale, '请切换到“建议修改”或“自动写入”后重新运行。', 'Switch to Suggest or Auto and run the task again.'),
+        nextStep: textFor(locale, '请切换到“自动写入”后重新运行。', 'Switch to Auto and run the task again.'),
         failureCode: null  // no direct catalog match — preflight / governance
       };
     }
@@ -794,7 +804,7 @@
     if (/checkpoint/i.test(text)) {
       return {
         conclusion: textFor(locale, '这轮没有自动写入：Codex 没有拿到可恢复版本。', 'This run did not auto-write: Codex did not get a recoverable version.'),
-        nextStep: textFor(locale, '请切换到“建议修改”，或确认 Overleaf Reviewing 已开启后再用“自动写入”。', 'Switch to Suggest, or confirm Overleaf Reviewing is enabled before using Auto.'),
+        nextStep: textFor(locale, '请确认 Overleaf Reviewing 已开启后再用“自动写入”。', 'Confirm that Overleaf Reviewing is enabled before using Auto.'),
         failureCode: null  // no current catalog code; preflight reviewing issue
       };
     }

package/extension/src/shared/compatibility.js

@@ -12,7 +12,7 @@
   const MIN_NATIVE_VERSION = '1.0.0';
   const MIN_COMPATIBLE_NATIVE_VERSION = '1.0.0';
   const MIN_COMPATIBLE_EXTENSION_VERSION = '1.0.0';
-  const BUILD_TARGET_VERSION = '2.2.1';
+  const BUILD_TARGET_VERSION = '2.3.1';
   const DEFAULT_CHROME_EXTENSION_ID = 'illdpneeeopfffmiepaejglgmhpmdhdc';
   const REQUIRED_CAPABILITIES = Object.freeze([
     'bridgePing',

package/extension/src/shared/governanceRules.js

@@ -8,7 +8,7 @@
     }
     module.exports = factory(projectFiles);
   } else {
-    root.CodexOverleafGovernanceRules = factory(root.CodexOverleafProjectFiles);
+    root.CodexOverleafModuleRegistry.define('GovernanceRules', ['ProjectFiles'], factory);
   }
 })(typeof globalThis !== 'undefined' ? globalThis : window, function governanceRulesFactory(projectFiles) {
   'use strict';