codex-overleaf-link 2.5.1 → 2.6.0

package/extension/src/content/writebackOrchestrator.js

@@ -130,7 +130,7 @@
       const skipped = withheldOnly.map(buildWithheldEntry);
       const hasSelectionViolation = withheldOnly.some(change => change.reason === 'selection_scope_violation');
       const applied = { ok: false, applied: [], skipped };
-      appendCompletionReport({ conclusion: assistantMessage || tx('Some generated changes were blocked.', '部分生成的改动已被阻止。'),
+      appendCompletionReport({ conclusionKey: options.assistantMessageKey || (!assistantMessage ? 'withheld_changes' : undefined), conclusion: assistantMessage || tx('Some generated changes were blocked.', '部分生成的改动已被阻止。'),
         status: 'failed', operations: [], applyResults: [applied], mode: runMode,
         nextStep: hasSelectionViolation
           ? selectionViolationNextStep()
@@ -151,7 +151,7 @@
         ),
         status: 'warning'
       });
-      appendCompletionReport({
+      appendCompletionReport({ conclusionKey: options.assistantMessageKey || (!assistantMessage ? 'ask_ignored_changes' : undefined),
         conclusion: assistantMessage || tx(
           'Codex finished in Ask mode. Any local file changes were ignored and were not synced to Overleaf.',
           'Codex 已在 Ask 模式完成；本地文件改动已忽略，未同步到 Overleaf。'
@@ -228,8 +228,15 @@
         title: tx('Codex did not produce file changes that need to sync back to Overleaf.', 'Codex 没有产生需要同步回 Overleaf 的文件改动。'),
         status: 'completed'
       });
-      appendCompletionReport({
-        conclusion: assistantMessage || tx('Codex finished locally. There are no changes to sync back to Overleaf.', 'Codex 已完成本地处理，没有需要同步回 Overleaf 的改动。'),
+      const blockedConclusion = assistantMessage
+        ? `${assistantMessage}\n\n${tx('Sync note: no changes were written to Overleaf; every change was blocked before writing.', '同步提示：没有向 Overleaf 写入任何改动；所有改动都在写入前被拦下。')}`
+        : tx('No changes were written to Overleaf; every change was blocked before writing.', '没有向 Overleaf 写入任何改动；所有改动都在写入前被拦下。');
+      appendCompletionReport({ conclusionKey: additionalSkippedEntries.length ? undefined
+        : options.assistantMessageKey || (!assistantMessage ? 'no_sync_changes' : undefined),
+        conclusion: additionalSkippedEntries.length ? blockedConclusion
+          : assistantMessage || tx('Codex finished locally. There are no changes to sync back to Overleaf.', 'Codex 已完成本地处理，没有需要同步回 Overleaf 的改动。'),
+        conclusionSource: additionalSkippedEntries.length ? { origin: 'product', key: 'writeback',
+          outcome: 'blocked_sync', prefixLength: assistantMessage.length } : undefined,
         status: additionalSkippedEntries.length ? 'failed' : runMode === 'ask' ? tr('modeAsk') : 'completed',
         failure: blockedFailure,
         operations: [],
@@ -295,7 +302,7 @@
         title: tx('No approved file changes remain to sync back to Overleaf.', '没有剩余已确认的文件改动需要同步回 Overleaf。'),
         status: 'completed'
       });
-      appendCompletionReport({
+      appendCompletionReport({ conclusionKey: options.assistantMessageKey || (!assistantMessage ? 'no_approved_changes' : undefined),
         conclusion: assistantMessage || tx('No approved changes were written back to Overleaf.', '没有已确认的改动写回 Overleaf。'),
         status: additionalSkippedEntries.length ? 'failed' : runMode === 'ask' ? tr('modeAsk') : 'completed',
         failure: approvalFailure,
@@ -326,8 +333,8 @@
     if (!reviewing.ok) {
       const blocked = buildReviewingBlockedApplyResult(operations, reviewing);
       appendApplyResult(blocked);
-      appendCompletionReport({
-        conclusion: tx(
+      appendCompletionReport({ conclusionKey: 'reviewing_unavailable',
+        conclusion: reviewing.code === 'reviewing_unavailable' ? tr('failureReason_reviewing_unavailable_user') : tx(
           'No files were written: Track is enabled, but Codex could not verify Overleaf Reviewing/Track Changes.',
           '这轮没有写入：已开启“留痕”要求，但 Codex 没能确认 Overleaf 正在用 Reviewing/Track Changes。'
         ),
@@ -335,7 +342,7 @@
         operations,
         applyResults: [blocked],
         mode: runMode,
-        nextStep: tx(
+        nextStep: reviewing.code === 'reviewing_unavailable' ? tr('failureReason_reviewing_unavailable_next') : tx(
           'Switch Overleaf to Reviewing/Track Changes manually and rerun, or turn off Track before writing.',
           '请在 Overleaf 手动切到 Reviewing/Track Changes 后重新运行，或关闭“留痕”再写入。'
         )
@@ -415,7 +422,7 @@
       ? await verifyPostWriteSaveState(applied, project, saveTarget)
       : { verification: { ok: false, state: 'not_checked', reason: 'No applied writeback operations were returned.' } };
     const saveVerification = confirmation.verification;
-    await settleWritebackRecovery?.(applied, project, saveVerification, saveTarget);
+    await settleWritebackRecovery?.(applied, project, saveVerification, saveTarget, runRequireReviewing);
     if (appliedPaths.length) {
       appendPostWriteSaveVerificationWarning(saveVerification, saveTarget);
       appendRunEvent({ kind: 'technical', title: 'Post-write save confirmation', status: 'info', detail: saveVerification });
@@ -453,7 +460,7 @@
           title: tx(`Post-write compile failed: ${error.message}`, `写后编译出错：${error.message}`),
           status: 'failed'
         });
-        return buildPostWriteCompileSummary({ error });
+        return compileAdapter.buildPostWriteCompileSummary({ error });
       })
       : null;
     const hasSkippedApplyResult = skippedEntries.length > 0;
@@ -485,6 +492,10 @@
         : tx('Review the synced file in Overleaf.', '请在 Overleaf 中查看同步后的文件。');
     appendCompletionReport({
       conclusion: appendCompileSummaryToConclusion(writebackConclusion, compileSummary),
+      conclusionSource: { origin: 'product', key: 'writeback',
+        outcome: !hasConfirmedApplyResult ? 'unconfirmed_sync' : hasSkippedApplyResult ? 'partial_sync' : 'synced',
+        prefixLength: assistantMessage.length, prefixKey: options.assistantMessageKey, hasCompile: Boolean(compileSummary), compileStatus: compileSummary?.status || '',
+        compileReason: compileAdapter.localizeCompileReason(compileSummary?.reason || '', getLocale()), errors: compileSummary?.errors?.slice(0, 3) || [], errorCount: compileSummary?.errors?.length || 0, warningCount: compileSummary?.warnings?.length || 0 },
       status: writebackIncomplete ? 'failed' : 'completed',
       operations,
       applyResults: [applied],
@@ -747,40 +758,40 @@
         let logResult = null;
         try {
           logResult = await callPageBridge('getCompileLog', {
+            captureId: result.compile?.captureId,
             triggerIfStale: false,
             maxAgeMs: 30000,
             waitForSaveMs: 0,
             runProjectId: options.runProjectId || getCurrentRunView()?.runProjectId || getCurrentProjectId()
           });
-        } catch (_error) {
-          logResult = null;
+        } catch (error) {
+          logResult = { ok: false, reason: error.message || 'compile_log_fetch_failed' };
         }
-        const compile = result.compile;
+        const compile = compileAdapter.buildPostWriteCompileSummary({ result, logResult });
+        compile.errors = compile.errors.map(value => CompileAdapter.localizeCompileReason(value, getLocale()));
         if (compile?.status === 'success') {
           appendRunEvent({ title: tx('Compile succeeded.', '编译成功。'), status: 'completed',
             activity: activity('completed', [...(logResult?.errors || []), ...(logResult?.warnings || [])].slice(0, 5).map(CompileAdapter.formatCompileDiagnosticForSummary).join('\n')) });
         } else if (compile?.status === 'triggered') {
-          appendRunEvent({ title: tx('Overleaf compile was triggered. The page will continue showing progress.', '已触发 Overleaf 编译；页面会继续显示编译进度。'), status: 'completed', activity: activity('triggered') });
+          appendRunEvent({ title: tx('Overleaf compile was triggered. The page will continue showing progress.', '已触发 Overleaf 编译；页面会继续显示编译进度。'), status: 'completed', activity: activity('triggered', compile.reason) });
         } else {
-          appendRunEvent({ title: tx(`Compile finished with status: ${compile?.status || 'unknown'}`, `编译完成，状态：${compile?.status || '未知'}`), status: 'completed', activity: activity(compile?.status === 'failed' ? 'failed' : 'unknown') });
+          appendRunEvent({ title: compile.status === 'unconfirmed' ? tr('runCompileUnconfirmed')
+            : tx(`Compile finished with status: ${compile.status}`, `编译完成，状态：${compile.status}`),
+            status: 'completed', activity: activity(compile.status === 'failed' ? 'failed' : 'unknown', compile.errors.join('\n') || compile.reason) });
         }
-        return buildPostWriteCompileSummary({ result, logResult });
+        return compile;
       } else {
         const reason = CompileAdapter.localizeCompileReason(result?.reason, getLocale()) || tx('unknown reason', '未知原因');
         appendRunEvent({ title: tx(`Post-write compile did not succeed: ${reason}`, `写后编译未成功：${reason}`), status: 'failed', activity: activity('failed', reason) });
-        return buildPostWriteCompileSummary({ result });
+        return compileAdapter.buildPostWriteCompileSummary({ result });
       }
     } catch (error) {
       const reason = CompileAdapter.localizeCompileReason(error.message, getLocale());
       appendRunEvent({ title: tx(`Post-write compile failed: ${reason}`, `写后编译出错：${reason}`), status: 'failed', activity: activity('failed', reason) });
-      return buildPostWriteCompileSummary({ error });
+      return compileAdapter.buildPostWriteCompileSummary({ error });
     }
   }
 
-  function buildPostWriteCompileSummary(input = {}) {
-    return compileAdapter.buildPostWriteCompileSummary(input);
-  }
-
   // Use the same normalized failure selection as settlement and its UI, while
   // preserving the selected producer's full evidence on the historical report.
   function promoteWritebackSkippedFailure(applied) {
@@ -814,6 +825,8 @@
           `已触发写后编译；最新日志显示 ${warnings.length} 个警告。`
         )
         : tx('Post-write compile was triggered; Overleaf may still be updating the result.', '已触发写后编译；Overleaf 可能仍在更新结果。');
+    } else if (compileSummary.status === 'unconfirmed') {
+      summary = tr('runCompileUnconfirmed');
     } else if (compileSummary.status === 'failed') {
       const reason = compileAdapter.localizeCompileReason(compileSummary.reason, getLocale());
       summary = tx(