codex-overleaf-link 2.2.0 → 2.3.0

package/extension/src/content/contextTray.js

@@ -1,10 +1,10 @@
 (function initCodexOverleafContextTray(root, factory) {
   if (typeof module === 'object' && module.exports) {
-    module.exports = factory();
+    module.exports = factory(require('../shared/projectFiles'));
   } else {
-    root.CodexOverleafContextTray = factory();
+    root.CodexOverleafModuleRegistry.define('ContextTray', ['ProjectFiles'], factory);
   }
-})(typeof window !== 'undefined' ? window : globalThis, function contextTrayFactory() {
+})(typeof window !== 'undefined' ? window : globalThis, function contextTrayFactory(ProjectFiles) {
   'use strict';
 
   const DEFAULT_CONTEXT_EXACT_FILE_LIST_ZIP_TIMEOUT_MS = 30000;
@@ -38,9 +38,7 @@
       || root.document
       || (typeof globalThis !== 'undefined' ? globalThis.document : null);
     const getLocationHref = getLocationHrefDep || (() => root.location?.href || '');
-    const projectFiles = projectFilesDep
-      || root.CodexOverleafProjectFiles
-      || (typeof window !== 'undefined' ? window.CodexOverleafProjectFiles : null);
+    const projectFiles = projectFilesDep || ProjectFiles;
     const exactFileListZipTimeoutMs = Number(exactFileListZipTimeoutMsDep) ||
       DEFAULT_CONTEXT_EXACT_FILE_LIST_ZIP_TIMEOUT_MS;
 

package/extension/src/content/diagnosticsController.js

@@ -1,8 +1,6 @@
 (function initCodexOverleafDiagnosticsController() {
   'use strict';
 
-  const CodexOverleafCompatibility = window.CodexOverleafCompatibility;
-
   // Diagnostics controller — the check runners (inspect*) and result formatters
   // carved out of contentRuntime.js (v1.4.5 structural-debt phase 1). The code
   // below moved verbatim (original indentation kept); runtime collaborators are
@@ -10,6 +8,7 @@
   // state, OT status) is read through injected getters.
   function create(deps = {}) {
     const {
+      Compatibility: CodexOverleafCompatibility,
       tr,
       tx,
       getExtensionCompatibilityMetadata,

package/extension/src/content/diffReviewPanel.js

@@ -1,10 +1,10 @@
 (function initCodexOverleafDiffReviewPanel(root, factory) {
   if (typeof module === 'object' && module.exports) {
-    module.exports = factory();
+    module.exports = factory(require('./reviewHunks'));
   } else {
-    root.CodexOverleafDiffReviewPanel = factory();
+    root.CodexOverleafModuleRegistry.define('DiffReviewPanel', ['ReviewHunks'], factory);
   }
-})(typeof window !== 'undefined' ? window : globalThis, function diffReviewPanelFactory() {
+})(typeof window !== 'undefined' ? window : globalThis, function diffReviewPanelFactory(ReviewHunks) {
   'use strict';
 
   const MAX_INITIAL_REVIEW_HUNKS = 20;
@@ -17,15 +17,14 @@
     const callPageBridge = typeof deps.callPageBridge === 'function'
       ? deps.callPageBridge
       : () => Promise.resolve({ ok: false, reason: tr('diffHunkJumpFailed') });
+    const getCurrentProjectId = typeof deps.getCurrentProjectId === 'function'
+      ? deps.getCurrentProjectId
+      : () => '';
     const getRunEvents = typeof deps.getRunEvents === 'function' ? deps.getRunEvents : () => null;
     const appendRunEvent = typeof deps.appendRunEvent === 'function' ? deps.appendRunEvent : () => {};
     const scrollLogToBottom = typeof deps.scrollLogToBottom === 'function' ? deps.scrollLogToBottom : () => {};
 
-    function getReviewHunks() {
-      return deps.reviewHunks ||
-        root.CodexOverleafReviewHunks ||
-        (typeof window !== 'undefined' ? window.CodexOverleafReviewHunks : null);
-    }
+    const getReviewHunks = () => deps.reviewHunks || ReviewHunks;
 
     function createDiffReviewElement(syncChanges, options = {}) {
       const readonly = Boolean(options.readonly);
@@ -37,10 +36,7 @@
       if (readonly) {
         container.dataset.readonly = 'true';
       }
-      const reviewHunks = options.reviewHunks ||
-        deps.reviewHunks ||
-        root.CodexOverleafReviewHunks ||
-        (typeof window !== 'undefined' ? window.CodexOverleafReviewHunks : null);
+      const reviewHunks = options.reviewHunks || getReviewHunks();
       const reviewModel = reviewHunks?.buildReviewModel?.(syncChanges) || { files: [], hunks: [] };
       const reviewFilesByIndex = new Map(reviewModel.files.map(file => [file.index, file]));
       const fileStates = new Map();
@@ -249,7 +245,12 @@
         const view = hunkViews.get(reviewHunk.decisionKey);
         setHunkJumpStatus(view, 'pending', '');
         try {
-          const result = await callPageBridge('jumpToPosition', { path, from, to });
+          const result = await callPageBridge('jumpToPosition', {
+            path,
+            from,
+            to,
+            runProjectId: getCurrentProjectId()
+          });
           if (result?.ok === false) {
             setHunkJumpStatus(view, 'failed', result.reason || tr('diffHunkJumpFailed'));
           } else {