codex-overleaf-link 2.4.1 → 2.5.1

package/extension/src/content/sessionForkController.js

@@ -7,15 +7,32 @@
 
   function create(deps = {}) {
     const {
-      SessionState, getState, setState, sendBackgroundNative, saveState,
+      SessionState, getState, setState, getScope, sendBackgroundNative, saveState,
       applyStateToPanel, showPluginToast, tr
     } = deps;
 
     async function forkRunFromNode(runId) {
       const previousState = getState();
+      const skillPreferences = {
+        loadCodexLocalSkills: previousState.loadCodexLocalSkills !== false,
+        loadCodexOverleafSkills: previousState.loadCodexOverleafSkills !== false,
+        codexLocalSkillEnabled: { ...previousState.codexLocalSkillEnabled }
+      };
       const source = findSource(previousState, runId);
       if (!source) return failFork('session_fork_source_missing', tr('forkRunUnavailable'));
-      const { session, run, runIndex } = source;
+      const scope = getScope ? { ...getScope() } : null;
+      const requestedActiveId = previousState.activeSessionId;
+      // The Native request and copied history must describe the same origin,
+      // even when the source session is edited while the request is pending.
+      const { session, run, runIndex } = JSON.parse(JSON.stringify(source));
+      const sameScope = () => {
+        const current = getScope?.();
+        return !scope || ['accountScopeId', 'projectId', 'generation'].every(key => current?.[key] === scope[key]);
+      };
+      const currentSource = () => {
+        const current = findSource(getState(), runId);
+        return current?.session.id === session.id && current.run.codexTurnId === run.codexTurnId;
+      };
       const draftThreadId = run.codexTurnId && run.interruptedDraft?.turnId === run.codexTurnId
         ? run.interruptedDraft.threadId : '';
       // Provider switches replace the session thread, not the selected node's origin.
@@ -25,25 +42,39 @@
       }
       let response;
       try {
+        const resolveLocalSkills = deps.captureLocalSkillIdsForFork || deps.getEnabledCodexLocalSkillIds;
+        const enabledCodexLocalSkillIds = skillPreferences.loadCodexLocalSkills
+          ? captureLocalSkillIds(await resolveLocalSkills?.(skillPreferences)) : [];
+        if (!sameScope() || !currentSource()) return null;
+        if (resolveLocalSkills && !Array.isArray(enabledCodexLocalSkillIds)) {
+          throw codedError('codex_local_skill_unavailable', tr('forkRunFailed'));
+        }
         response = await sendBackgroundNative({
           method: 'codex.thread.fork',
           params: {
             threadId: sourceThreadId,
             lastTurnId: run.codexTurnId,
             speedTier: session.speedTier,
-            loadCodexLocalSkills: previousState.loadCodexLocalSkills !== false,
-            loadCodexOverleafSkills: previousState.loadCodexOverleafSkills !== false
+            loadCodexLocalSkills: skillPreferences.loadCodexLocalSkills,
+            loadCodexOverleafSkills: skillPreferences.loadCodexOverleafSkills,
+            enabledCodexLocalSkillIds
           }
         });
       } catch (error) {
+        if (!sameScope() || !currentSource()) return null;
         return failFork(error?.code || 'session_fork_failed', error?.message || tr('forkRunFailed'));
       }
+      if (!sameScope() || !currentSource()) {
+        if (response?.ok && response.result?.threadId) await clearFork(response.result.threadId);
+        return null;
+      }
       if (!response?.ok || !response.result?.threadId) {
         return failFork(response?.error?.code || 'session_fork_failed', response?.error?.message || tr('forkRunFailed'));
       }
       const copiedRuns = session.runs.slice(0, runIndex + 1).map(copyRunSnapshot);
       const titleBase = String(session.title || copiedRuns[0]?.task || tr('newSessionFallback')).trim();
-      const forkTitle = buildUniqueForkTitle(previousState.sessions, titleBase);
+      const mergeBase = getState();
+      const forkTitle = buildUniqueForkTitle(mergeBase.sessions, titleBase);
       const forked = SessionState.createSession({
         title: forkTitle,
         titleSource: 'manual',
@@ -62,27 +93,55 @@
         forkedFromSessionId: session.id,
         forkedFromRunId: run.id
       });
-      setState(SessionState.normalizePanelState({
-        ...previousState,
-        sessions: [...(previousState.sessions || []), forked],
-        activeSessionId: forked.id
-      }));
+      const activeId = mergeBase.activeSessionId === requestedActiveId
+        ? forked.id : mergeBase.activeSessionId;
+      const installed = SessionState.setActiveSession({
+        ...mergeBase, sessions: [...(mergeBase.sessions || []), forked]
+      }, activeId);
+      const installedSnapshot = JSON.stringify(installed);
+      const forkSnapshot = JSON.stringify(forked);
+      setState(installed);
       try {
+        // Keep the visible composer and active-session owner aligned during save.
+        applyStateToPanel();
         await saveState();
       } catch (error) {
-        setState(previousState);
-        await sendBackgroundNative({
-          method: 'codex.history.clearPlugin',
-          params: { sessionId: forked.id, threadId: response.result.threadId }
-        }).catch(() => null);
-        showPluginToast(error.message || tr('forkRunFailed'), { status: 'failed', sticky: true });
+        if (sameScope()) {
+          const current = getState();
+          const currentFork = current.sessions?.find(item => item.id === forked.id);
+          // Never erase work entered into the fork after its optimistic insert.
+          if (!currentFork || JSON.stringify(currentFork) === forkSnapshot) {
+            if (currentFork) {
+              if (current === installed && JSON.stringify(current) === installedSnapshot) {
+                setState(mergeBase);
+              } else {
+                const remaining = current.sessions.filter(item => item.id !== forked.id);
+                const fallbackId = remaining.some(item => item.id === mergeBase.activeSessionId)
+                  ? mergeBase.activeSessionId : remaining[remaining.length - 1]?.id;
+                setState(SessionState.setActiveSession({ ...current, sessions: remaining },
+                  current.activeSessionId === forked.id ? fallbackId : current.activeSessionId));
+              }
+              try { applyStateToPanel(); } catch (_renderError) { /* Preserve the original failure and finish cleanup. */ }
+            }
+            await clearFork(response.result.threadId, forked.id);
+          }
+          if (sameScope()) showPluginToast(error.message || tr('forkRunFailed'), { status: 'failed', sticky: true });
+        }
         throw error;
       }
-      applyStateToPanel();
+      // Persistence owns its captured scope. A late result cannot render a new
+      // view or delete a thread that may already have been saved in the old one.
+      if (!sameScope() || !getState().sessions?.some(item => item.id === forked.id)) return null;
       showPluginToast(tr('forkRunDone'), { status: 'completed' });
       return forked;
     }
 
+    async function clearFork(threadId, sessionId) {
+      try {
+        await sendBackgroundNative({ method: 'codex.history.clearPlugin', params: { sessionId, threadId } });
+      } catch (_error) { /* Best-effort cleanup must not restore a stale view. */ }
+    }
+
     function failFork(code, message) {
       showPluginToast(message, { status: 'failed', sticky: true });
       throw codedError(code, message);
@@ -91,6 +150,10 @@
     return { forkRunFromNode };
   }
 
+  function captureLocalSkillIds(value) {
+    return Array.isArray(value) ? [...value] : undefined;
+  }
+
   function buildUniqueForkTitle(sessions = [], sourceTitle = '') {
     const fallback = String(sourceTitle || 'New Session').trim() || 'New Session';
     const baseTitle = fallback

package/extension/src/content/sessionManager.js

@@ -201,7 +201,7 @@
         }
       });
       if (!response?.ok) {
-        showPluginToast(tr('deleteSessionHistoryFailedToast', { message: response?.error?.message || 'native host did not return success' }), { status: 'warning', sticky: true });
+        showPluginToast(tr('deleteSessionHistoryFailedToast', { message: response?.error?.message || tr('deleteSessionNativeFailureDetail') }), { status: 'warning', sticky: true });
       } else if (response.result?.skipped) {
         showPluginToast(tr('deleteSessionNoThreadToast'), { status: 'info' });
       } else {

package/extension/src/content/sessionPersistence.js

@@ -85,7 +85,12 @@
       record !== rawExistingSessions[index] && !writable.some(item => item.id === record.id)
     );
     if (cleanedExisting.length || writable.length) {
-      await StorageDb.putRecords('sessions', [...cleanedExisting, ...writable]);
+      const written = await StorageDb.putRecords('sessions', [...cleanedExisting, ...writable], {
+        reviewRunIds: options.reviewRunIds
+      });
+      // Check the actual durable result, including a decision already saved
+      // from another origin, instead of acknowledging only the proposed view.
+      return written.filter(record => writable.some(candidate => candidate.id === record.id));
     }
     return writable;
   }

package/extension/src/content/settingsPanel.js

@@ -1,4 +1,10 @@
-(function initCodexOverleafSettingsPanel() {
+(function (root, factory) {
+  if (typeof module === 'object' && module.exports) {
+    module.exports = factory(require('./settingsWorkbench'));
+  } else {
+    root.CodexOverleafModuleRegistry.define('SettingsPanel', ['SettingsWorkbench'], factory);
+  }
+})(typeof globalThis !== 'undefined' ? globalThis : this, function settingsPanelFactory(SettingsWorkbench) {
   'use strict';
 
   function codexIcon(name) {
@@ -45,10 +51,16 @@
         <div class="codex-project-settings-status" data-project-settings-status></div>
         <div class="codex-project-settings-scope">
           <div class="codex-project-settings-scope-title codex-set-eyebrow" data-i18n="settingsScopeProjectTitle">This project</div>
+          <details class="codex-set-group" data-set-group="writing-style" open>
+            <summary class="codex-set-group-head">
+              <span class="codex-set-group-title">${codexSetIcon('pen')}<span data-writing-style-heading>Writing style</span></span>
+            </summary>
+            <div class="codex-set-card" data-writing-style-settings></div>
+          </details>
           <details class="codex-set-group" data-set-group="personalization" open>
             <summary class="codex-set-group-head">
               <span class="codex-set-group-title">${codexSetIcon('pen')}<span data-i18n="personalizationConfig">Personalization</span></span>
-              <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+              <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
             </summary>
             <div class="codex-set-card">
               <p class="codex-set-row-help" data-i18n="personalizationHelp">Style, terminology, and LaTeX conventions Codex should follow in this project.</p>
@@ -58,7 +70,7 @@
           <details class="codex-set-group" data-set-group="protection" open>
             <summary class="codex-set-group-head">
               <span class="codex-set-group-title">${codexSetIcon('shield')}<span data-i18n="fileProtectionTitle">File protection</span></span>
-              <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+              <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
             </summary>
             <div class="codex-set-card">
               <div class="codex-set-row">
@@ -78,7 +90,7 @@
           <details class="codex-set-group" data-set-group="privacy" open>
             <summary class="codex-set-group-head">
               <span class="codex-set-group-title">${codexSetIcon('lock')}<span data-i18n="privacyTitle">Privacy</span></span>
-              <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+              <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
             </summary>
             <div class="codex-set-card">
               <label class="codex-project-settings-row codex-project-settings-row--switch">
@@ -96,7 +108,7 @@
           <details class="codex-set-group" data-set-group="experimental">
             <summary class="codex-set-group-head">
               <span class="codex-set-group-title">${codexSetIcon('database')}<span data-i18n="otSettingsTitle">Project cache</span></span>
-              <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+              <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
             </summary>
             <div class="codex-set-card">
               <label class="codex-project-settings-row codex-project-settings-row--switch">
@@ -113,19 +125,37 @@
         <div class="codex-project-settings-scope codex-project-settings-scope--global">
           <details class="codex-set-group" data-set-group="history" data-history-card>
             <summary class="codex-set-group-head">
-              <span class="codex-set-group-title">${codexSetIcon('history')}<span data-i18n="historyTitle">Change history</span></span>
+              <span class="codex-set-group-title">${codexSetIcon('history')}<span data-i18n="historyTitle">File changes</span><span class="codex-history-count" data-history-count aria-live="polite"></span></span>
             </summary>
             <div class="codex-set-card">
-              <p class="codex-set-row-help" data-i18n="historyHelp">What Codex wrote to this project, per run. Filter by file name or task text.</p>
-              <input type="text" class="codex-set-select codex-history-filter" data-history-filter>
+              <p class="codex-set-row-help" data-i18n="historyHelp">Review recent file changes and open the matching conversation.</p>
+              <div class="codex-history-toolbar">
+                <label class="codex-history-search">
+                  <span class="codex-history-search-mark" aria-hidden="true"></span>
+                  <input type="search" class="codex-set-select codex-history-filter" data-history-filter autocomplete="off">
+                </label>
+                <select class="codex-set-select codex-history-status-filter" data-history-status data-i18n-aria-label="historyStatusLabel">
+                  <option value="all" data-i18n="historyStatusAll">All statuses</option>
+                  <option value="pending" data-i18n="historyStatusPending">Pending review</option>
+                  <option value="accepted" data-i18n="historyStatusAccepted">Accepted</option>
+                  <option value="undone" data-i18n="historyStatusUndone">Undone</option>
+                  <option value="attention" data-i18n="historyStatusAttention">Needs attention</option>
+                  <option value="written" data-i18n="historyStatusWritten">Written</option>
+                </select>
+              </div>
               <div class="codex-history-list" data-history-list></div>
+              <button type="button" class="codex-history-jump codex-history-more" data-history-more hidden></button>
+              <label class="codex-history-include" hidden>
+                <input type="checkbox" data-history-include-empty>
+                <span data-history-include-label></span>
+              </label>
             </div>
           </details>
           <div class="codex-project-settings-scope-title codex-set-eyebrow" data-i18n="settingsScopeGlobalTitle">All projects</div>
           <details class="codex-set-group" data-set-group="appearance" open>
             <summary class="codex-set-group-head">
               <span class="codex-set-group-title">${codexSetIcon('appearance')}<span data-i18n="appearanceTitle">Appearance</span></span>
-              <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+              <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
             </summary>
             <div class="codex-set-card">
               <label class="codex-project-settings-row codex-project-settings-row--select">
@@ -153,7 +183,7 @@
             </summary>
             <div class="codex-set-card codex-provider-settings-entry">
               <div class="codex-provider-settings-entry-copy">
-                <p class="codex-set-row-help" data-i18n="providerSettingsHelp">Configure experimental third-party model APIs. Compatibility varies by provider and gateway.</p>
+                <p class="codex-set-row-help" data-i18n="providerSettingsHelp">Manage model API connections.</p>
                 <p class="codex-provider-settings-summary" data-provider-settings-summary data-tone="loading">${t(instance, 'providerSettingsLoading')}</p>
               </div>
               <button type="button" class="codex-set-btn" data-provider-settings-open data-i18n="providerSettingsConfigure">Configure</button>
@@ -162,7 +192,7 @@
           <details class="codex-set-group" data-set-group="context-loading" open>
             <summary class="codex-set-group-head">
               <span class="codex-set-group-title">${codexSetIcon('database')}<span data-i18n="contextLoadingTitle">Context loading</span></span>
-              <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+              <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
             </summary>
             <div class="codex-set-card">
               <label class="codex-project-settings-row codex-project-settings-row--switch">
@@ -186,18 +216,24 @@
           </details>
           <details class="codex-set-group" data-set-group="storage" data-storage-card>
             <summary class="codex-set-group-head">
-              <span class="codex-set-group-title">${codexSetIcon('database')}<span data-i18n="storageTitle">History &amp; storage</span></span>
+              <span class="codex-set-group-title">${codexSetIcon('database')}<span data-i18n="storageTitle">Local data</span></span>
             </summary>
             <div class="codex-set-card">
-              <p class="codex-set-row-help" data-storage-usage data-i18n="storageUsageLoading">Calculating usage…</p>
-              <button type="button" class="codex-set-btn codex-set-btn--danger" data-clear-all-history data-i18n="storageClearAll">Clear all history…</button>
-              <p class="codex-set-row-help" data-i18n="storageClearAllHelp">Removes every stored session, run and audit record for all projects. Project settings and rules are kept.</p>
+              <div class="codex-storage-data-row">
+                <div class="codex-storage-data-copy">
+                  <div class="codex-storage-data-label" data-i18n="storageSavedData">Saved conversations &amp; records</div>
+                  <p class="codex-set-row-help" data-storage-usage data-i18n="storageUsageLoading">Loading saved history…</p>
+                </div>
+                <button type="button" class="codex-set-btn codex-set-btn--danger" data-clear-all-history data-i18n="storageClearAll">Clear local history…</button>
+              </div>
+              <p class="codex-set-row-help codex-storage-clear-help" data-i18n="storageClearAllHelp">Clearing removes local history and undo information. Overleaf project files stay unchanged.</p>
+              <p class="codex-storage-estimate" data-storage-estimate aria-live="polite"></p>
             </div>
           </details>
           <details class="codex-set-group" data-set-group="skills" open>
             <summary class="codex-set-group-head">
               <span class="codex-set-group-title">${codexSetIcon('bolt')}<span data-i18n="localSkillsTitle">Skills</span></span>
-              <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+              <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
             </summary>
             <div class="codex-set-card">
               <label class="codex-project-settings-row codex-project-settings-row--switch">
@@ -206,7 +242,7 @@
               </label>
               <p class="codex-set-row-help" data-i18n="loadCodexLocalSkillsHelp">Pull skills from your local Codex installation.</p>
               <button type="button" class="codex-skills-entry" data-skills-entry>
-                <span class="codex-skills-entry-label" data-i18n="codexOverleafSkillsEntry">Codex Overleaf skills</span>
+                <span class="codex-skills-entry-label" data-i18n="skillsManageEntry">Manage skills</span>
                 <span class="codex-skills-entry-summary" data-skills-entry-summary></span>
                 <span class="codex-skills-entry-chevron" aria-hidden="true">›</span>
               </button>
@@ -216,11 +252,11 @@
       </section>
       <section class="codex-custom-instructions-panel codex-skills-panel" data-skills-screen>
         <div class="codex-custom-instructions-head">
-          <button type="button" data-skills-back title="Back" aria-label="Back">‹</button>
+          <button type="button" data-skills-back data-i18n-title="settingsBack" data-i18n-aria-label="settingsBack" title="Back" aria-label="Back">‹</button>
           <div>
-            <div class="codex-custom-instructions-title" data-i18n="codexOverleafSkillsTitle">Codex Overleaf skills</div>
+            <div class="codex-custom-instructions-title" data-i18n="localSkillsTitle">Skills</div>
           </div>
-          <span class="codex-set-saved" data-set-saved hidden>✓ <span data-i18n="settingsSaved">Saved</span></span>
+          <span class="codex-set-saved" data-set-saved hidden><svg class="codex-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="m3.5 8 3 3 6-6"/></svg><span data-i18n="settingsSaved">Saved</span></span>
         </div>
         <div class="codex-set-card">
           <label class="codex-project-settings-row codex-project-settings-row--switch">
@@ -230,6 +266,9 @@
         </div>
         <div class="codex-skills-divider"></div>
         <div class="codex-local-skill-list" data-local-skill-list></div>
+        <div class="codex-skills-divider"></div>
+        <div class="codex-wb-skills-list-title codex-wb-skills-caption" data-i18n="codexLocalSkillsTitle">Local Codex skills</div>
+        <div class="codex-local-skill-list" data-codex-local-skill-list></div>
       </section>
     `;
 
@@ -247,6 +286,8 @@
       if (event.target.open) instance.callbacks.onStorageOpen?.();
     });
     container.querySelector('[data-history-filter]')?.addEventListener('input', () => instance.callbacks.onHistoryFilter?.());
+    container.querySelector('[data-history-status]')?.addEventListener('change', () => instance.callbacks.onHistoryFilter?.());
+    container.querySelector('[data-history-include-empty]')?.addEventListener('change', () => instance.callbacks.onHistoryFilter?.());
     // OT mirror: a single visible switch; legacy storage/DOM keys are retained. The click is
     // intercepted (the runtime's handler preventDefaults, runs the enable
     // confirmation, then sets checked + drives the change flow itself) so the
@@ -265,8 +306,9 @@
         Promise.resolve(instance.callbacks.onInputChange?.(event))
           .then(() => {
             flashSaved(instance, event);
-            if (selector === '[data-language-select]' && container.querySelector('[data-storage-card]')?.open) {
-              instance.callbacks.onStorageOpen?.();
+            if (selector === '[data-language-select]') {
+              if (container.querySelector('[data-storage-card]')?.open) instance.callbacks.onStorageOpen?.();
+              instance.callbacks.onHistoryFilter?.();
             }
           }).catch(() => {});
       });
@@ -280,12 +322,18 @@
       element?.addEventListener?.('input', () => updateGovernanceNotes(instance));
       element?.addEventListener?.('change', () => updateGovernanceNotes(instance));
     }
-    setupGroupPersistence(instance);
+    instance.workbench = SettingsWorkbench.mount(instance, { icon: codexIcon });
 
     return {
       get container() { return instance.container; },
       show: () => show(instance),
       hide: () => hide(instance),
+      openProviderPage: () => {
+        const host = instance.workbench?.revealProviders();
+        if (host) show(instance);
+        return host || null;
+      },
+      setProviderCloseGuard: guard => instance.workbench?.setProviderCloseGuard(guard),
       loadState: state => loadState(instance, state),
       readState: () => readState(instance),
       setStatus: (text, status) => setStatus(instance, text, status),
@@ -293,6 +341,7 @@
       setSkillsSummary: text => setSkillsSummary(instance, text),
       setProviderSummary: summary => setProviderSummary(instance, summary),
       refreshNotes: () => updateGovernanceNotes(instance),
+      refreshLocale: () => instance.workbench?.refresh(),
       destroy: () => destroy(instance),
       _instance: instance
     };
@@ -310,10 +359,12 @@
       button.dataset.active = 'true';
       button.setAttribute('aria-expanded', 'true');
     }
-    root.querySelector('[data-custom-instructions-input]')?.focus?.();
+    if (instance.workbench) instance.workbench.show();
+    else root.querySelector('[data-custom-instructions-input]')?.focus?.();
   }
 
   function refreshSettingsScope(instance, root) {
+    if (instance.workbench) { instance.workbench.refresh(); return; }
     const accountScope = instance.container?.closest?.('[data-settings-scope]')?.dataset?.settingsScope === 'account';
     const prefix = accountScope ? 'recentProjects_settings' : 'projectSettings';
     for (const [selector, suffix] of [['[data-settings-title]', 'Title'], ['[data-settings-subtitle]', 'Subtitle']]) {
@@ -337,6 +388,7 @@
       button.setAttribute('aria-expanded', 'false');
     }
     clearStatus(instance);
+    instance.workbench?.hide();
   }
 
   function isVisible(target) {
@@ -386,6 +438,7 @@
       setValue(scope, '[data-theme-select]', state.theme || 'dark');
       setValue(scope, '[data-language-select]', state.language || 'en');
     }
+    instance.workbench?.refresh();
   }
 
   function readState(target) {
@@ -413,7 +466,7 @@
 
   function setStatus(target, text, status = 'info') {
     const instance = target?._instance || target;
-    const element = getRoot(instance)?.querySelector('[data-project-settings-status]');
+    const element = instance?.container?.querySelector('[data-project-settings-status]');
     if (!element) {
       return;
     }
@@ -423,7 +476,7 @@
 
   function clearStatus(target) {
     const instance = target?._instance || target;
-    const element = getRoot(instance)?.querySelector('[data-project-settings-status]');
+    const element = instance?.container?.querySelector('[data-project-settings-status]');
     if (!element) {
       return;
     }
@@ -440,11 +493,11 @@
     element.textContent = text || '';
   }
 
-  // Flash the "✓ Saved" indicator after a field auto-saves, then fade it out.
+  // Flash the check-mark "Saved" indicator after a field auto-saves, then fade it out.
   // The fields persist immediately on change, so this is purely a confirmation
   // cue — it never gates the save itself.
   function flashSaved(instance, event) {
-    // Card-level feedback: the ✓ appears on the card that was actually
+    // Card-level feedback: the check appears on the card that was actually
     // changed (falls back to the screen's first badge, e.g. the skills page).
     const origin = event?.target?.closest?.('details.codex-set-group, section');
     const badge = origin?.querySelector?.('[data-set-saved]')
@@ -530,31 +583,6 @@
     element.dataset.tone = typeof value === 'object' && value.tone ? value.tone : 'ok';
   }
 
-  // Collapse memory: card open/closed state persists per browser profile.
-  const GROUP_STORE_KEY = 'codexOverleafSettingsGroups';
-
-  function setupGroupPersistence(instance) {
-    let stored = {};
-    try {
-      stored = JSON.parse(globalThis.localStorage?.getItem(GROUP_STORE_KEY) || '{}') || {};
-    } catch (_error) {
-      stored = {};
-    }
-    for (const details of instance.container.querySelectorAll('details[data-set-group]')) {
-      const key = details.dataset.setGroup;
-      if (key in stored) {
-        details.open = stored[key] !== false;
-      }
-      details.addEventListener('toggle', () => {
-        try {
-          const current = JSON.parse(globalThis.localStorage?.getItem(GROUP_STORE_KEY) || '{}') || {};
-          current[key] = details.open;
-          globalThis.localStorage?.setItem(GROUP_STORE_KEY, JSON.stringify(current));
-        } catch (_error) { /* private mode — collapse state just doesn't persist */ }
-      });
-    }
-  }
-
   function getRoot(instance) {
     return instance?.container?.querySelector('[data-project-settings-panel]') || null;
   }
@@ -582,6 +610,7 @@
   }
 
   function destroy(instance) {
+    instance.workbench?.destroy();
     if (instance._savedFlashTimer) {
       clearTimeout(instance._savedFlashTimer);
       instance._savedFlashTimer = null;
@@ -589,7 +618,7 @@
     instance.container.textContent = '';
   }
 
-  window.CodexOverleafSettingsPanel = {
+  return {
     create,
     loadState,
     readState,
@@ -601,4 +630,4 @@
     setSkillsSummary,
     setProviderSummary
   };
-})();
+});