fix: reset execution state and variables on file switch

- Fixed Run All button stuck in Stop mode when switching .md files
- Fixed document variables leaking across files on switch
- Added resetFileSessionState() in workspace.js (abort + button + vars + progress cleanup)
- Added forceReset() in exec-controller.js for immediate _running hard-reset
- Wired into both wsOpenFile() and openDiskFile()

Author: ijbo

CHANGELOG-file-switch-reset.md

@@ -0,0 +1,33 @@
+# File-Switch State Reset — Bug Fixes
+
+- Fixed: Run All button stays in "Stop" mode when switching to another .md file while execution is running
+- Fixed: Document variables from one file leak into another file's variable panel on file switch
+- Added `resetFileSessionState()` helper in `workspace.js` — aborts running execution, force-resets `_running` flag, resets Run All button to default, removes progress bar, clears runtime + manual variables, clears `__API_VARS`, clears SQLite exec context
+- Added `forceReset()` to exec-controller for immediate hard-reset of internal state (`_running`, `_currentRun`, abort flags)
+- Wired `resetFileSessionState()` into both `wsOpenFile()` and `openDiskFile()` file-switch paths
+
+---
+
+## Summary
+When switching between .md files, the Run All execution state and document variables were not reset, causing stale UI (Stop button persisting) and stale data (previous file's variables showing in the new file). Both `wsOpenFile()` and `openDiskFile()` now perform a full session cleanup on file switch.
+
+---
+
+## 1. File-Switch Session Reset
+**Files:** `js/workspace.js`
+**What:** New `resetFileSessionState()` function (L37-70) called from both `wsOpenFile()` (L636) and `openDiskFile()` (L389). It: (1) calls `abort()` + `forceReset()` on the exec-controller, (2) resets the `#run-all-btn` DOM (removes `fmt-run-active` class, restores innerHTML and title, nulls `onclick`), (3) removes the `.exec-progress-bar` element, (4) calls `clearRuntime()` + `clearManual()` on `M._vars`, (5) clears `window.__API_VARS`, (6) clears the SQLite exec context.
+**Impact:** Run All button always shows "▶ Run All" on a fresh file; variable panel starts empty for new files; progress bar is removed on switch.
+
+## 2. Exec-Controller Force Reset
+**Files:** `js/exec-controller.js`
+**What:** New `forceReset()` function (L1026-1031) immediately sets `_running = false`, `_currentRun = null`, and abort flags. Exposed via `M._execController.forceReset`.
+**Impact:** Unlike `abort()` (which only sets `_abortRequested` and waits for the loop to exit), `forceReset()` provides an immediate hard-reset needed when switching files where the execution loop may be blocked awaiting a Promise.
+
+---
+
+## Files Changed (2 total)
+
+| File | Lines Changed | Type |
+|------|:---:|------|
+| `js/workspace.js` | +44 | New `resetFileSessionState()` helper + wired into `wsOpenFile()` and `openDiskFile()` |
+| `js/exec-controller.js` | +12 | New `forceReset()` function + exposed in `M._execController` API |

README.md

@@ -459,6 +459,7 @@ TextAgent has undergone significant evolution since its inception. What started
 
 | Date | Commits | Feature / Update |
 |------|---------|-----------------|
+| **2026-03-16** | | 🐛 **File-Switch State Reset** — fixed Run All button staying in "Stop" mode when switching .md files; fixed document variables leaking across files; new `resetFileSessionState()` in `workspace.js` aborts execution, force-resets `_running` flag, clears Run All button/progress bar/variables/exec context on every file switch; new `forceReset()` in `exec-controller.js` for immediate hard-reset of internal state |
 | **2026-03-15** | | 🚀 **Run All Engine & TTS UX** — pre-execution model readiness check auto-loads all required models (AI + Kokoro TTS) before block execution starts; detailed `[RunAll]` console logging with `console.table` block summary, per-block timing, variable resolution status (✅/⚠), and completion summary; Stop button now works during model loading via `M._execAborted` cross-module flag; `ensureModelReadyAsync()` rewritten with fail-fast on missing consent/API key; compact preflight dialog (960px, smaller fonts, all 8 columns visible); `waitForModelReady()` handles Kokoro TTS via `M.tts.isKokoroReady()`; TTS card split into 3 buttons: ▶ Run (generate audio only), ▷ Play (replay stored audio), 💾 Save (download WAV); new `M.tts.generate()`, `playLastAudio()`, `isKokoroReady()`, `initKokoro()` APIs; AI model fallback in `run-requirements.js` correctly defaults to text models |
 | **2026-03-14** | | 🔗 **AI Variable Controls** — new unified 🔗 Vars button on AI and Agent cards opens combined dropdown with 📤 Output Variable (text input to name the block's result) and 📥 Input Variables (checkbox picker listing declared `@var:` names from other blocks + runtime vars); variable chaining enables multi-block AI pipelines (`@var: research` → `@input: research`); declared variables appear before execution with "declared" badge; Doc Variables Panel (`{•} Vars` toolbar button) now shows ⏳ Pending Vars section for declared-but-unexecuted variables; `@var:` and `@input:` directives stripped from displayed prompt text |
 | **2026-03-14** | | 🧠 **Think Mode Refinement & Multi-Select Search** — Think mode (`@think: Yes` / 🧠 toggle) now uses two-pass generation: first generates with thinking enabled, then passes the draft back to the model to add important details, examples, and missing information; removed complex ReAct pattern in favor of simple refinement; multi-select search provider dropdown on AI Generate and Agent Flow cards (checkbox pills, activate multiple engines simultaneously); search results fetched in parallel and merged |

js/exec-controller.js

@@ -1019,6 +1019,17 @@
         if (M.showToast) M.showToast('⏹ Stopping execution...', 'info');
     }
 
+    /**
+     * Hard-reset all execution state. Used when switching files so
+     * the stale running state from the previous file doesn't persist.
+     */
+    function forceReset() {
+        _abortRequested = true;
+        M._execAborted = true;
+        _running = false;
+        _currentRun = null;
+    }
+
     // ========================================
     // Progress UI + Log Panel
     // ========================================
@@ -1176,6 +1187,7 @@
         runAll: runAll,
         runSingle: runSingle,
         abort: abort,
+        forceReset: forceReset,
         isRunning: function () { return _running; },
         on: on,
         off: off

js/workspace.js

@@ -29,6 +29,46 @@
         return Date.now().toString(36) + Math.random().toString(36).substring(2, 8);
     }
 
+    /**
+     * Reset execution and variable state when switching files.
+     * Fixes: Run All button staying in "Stop" mode across files,
+     *        and variables from previous file leaking into new file.
+     */
+    function resetFileSessionState() {
+        // Abort any running execution and force-reset internal state
+        if (M._execController) {
+            if (M._execController.isRunning()) M._execController.abort();
+            if (M._execController.forceReset) M._execController.forceReset();
+        }
+
+        // Reset the Run All button to default state
+        var runAllBtn = document.getElementById('run-all-btn');
+        if (runAllBtn) {
+            runAllBtn.classList.remove('fmt-run-active');
+            runAllBtn.innerHTML = '<i class="bi bi-play-circle"></i> <span>Run All</span>';
+            runAllBtn.title = 'Run all executable blocks in document order';
+            runAllBtn.onclick = null;
+        }
+
+        // Remove progress bar if visible
+        var progressEl = document.querySelector('.exec-progress-bar');
+        if (progressEl) {
+            progressEl.remove();
+        }
+
+        // Clear document variables (runtime + manual)
+        if (M._vars) {
+            if (M._vars.clearRuntime) M._vars.clearRuntime();
+            if (M._vars.clearManual) M._vars.clearManual();
+        }
+
+        // Clear legacy __API_VARS
+        if (window.__API_VARS) window.__API_VARS = {};
+
+        // Clear SQLite exec context
+        if (M._execContext && M._execContext.clear) M._execContext.clear();
+    }
+
     function saveWorkspace() {
         try {
             localStorage.setItem(WORKSPACE_KEY, JSON.stringify(workspace));
@@ -345,6 +385,8 @@
         else if (M.resetCloudForFileSwitch) M.resetCloudForFileSwitch();
         // Save current file
         M.wsSaveCurrent();
+        // Reset execution state & variables for new file
+        resetFileSessionState();
 
         // Check if this file is already in the workspace manifest
         var wsFile = findFileByName(entry.path) || findFileByName(entry.name);
@@ -590,6 +632,8 @@
         else if (M.resetCloudForFileSwitch) M.resetCloudForFileSwitch();
         // Save current
         M.wsSaveCurrent();
+        // Reset execution state & variables for new file
+        resetFileSessionState();
         // Switch
         workspace.activeFileId = id;
         M.wsActiveFileId = id;