Address brad's feedback

Author: clean6378-max-it

DEPLOYMENT.md

@@ -12,7 +12,9 @@ pip install gunicorn                     # Linux / macOS
 # pip install waitress                   # Windows-friendly alternative (see below)
 
 # Multi-process (recommended): one thread per worker avoids any per-worker state surprises.
-gunicorn --factory --bind 127.0.0.1:3000 --workers 2 --threads 1 app:create_app
+# WEB_CONCURRENCY (or CURSOR_BROWSER_MULTI_WORKER=1) lets the app detect multi-worker mode so
+# POST /api/set-workspace returns 409 instead of a misleading 200 on a single worker.
+WEB_CONCURRENCY=2 gunicorn --factory --bind 127.0.0.1:3000 --workers 2 --threads 1 app:create_app
 
 # Single-process, multi-threaded: safe after the #43 lock; useful for lighter deployments.
 gunicorn --factory --bind 127.0.0.1:3000 --workers 1 --threads 4 app:create_app

templates/config.html

@@ -18,6 +18,35 @@ <h1>Configuration</h1>
 </div>
 
 <script>
+async function postSetWorkspace(path) {
+  const res = await fetch('/api/set-workspace', {
+    method: 'POST',
+    headers: { 'Content-Type': 'application/json' },
+    body: JSON.stringify({ path })
+  });
+  let body = {};
+  try {
+    body = await res.json();
+  } catch (e) {
+    body = {};
+  }
+  if (res.status === 409) {
+    return {
+      ok: false,
+      message: body.error || 'Workspace path cannot be changed in this deployment.',
+      code: body.code
+    };
+  }
+  if (!res.ok) {
+    return {
+      ok: false,
+      message: body.error || 'Failed to save workspace path.',
+      code: body.code
+    };
+  }
+  return { ok: true, body };
+}
+
 document.addEventListener('DOMContentLoaded', async () => {
   const stored = localStorage.getItem('workspacePath');
   if (stored) {
@@ -53,14 +82,12 @@ <h1>Configuration</h1>
       });
       const valData = await valRes.json();
       if (valData.valid) {
-        localStorage.setItem('workspacePath', detected);
-        await fetch('/api/set-workspace', {
-          method: 'POST',
-          headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify({ path: detected })
-        });
-        window.location.href = '/';
-        return;
+        const saved = await postSetWorkspace(detected);
+        if (saved.ok) {
+          localStorage.setItem('workspacePath', detected);
+          window.location.href = '/';
+          return;
+        }
       }
     }
     document.getElementById('workspace-path').value = detected;
@@ -84,12 +111,14 @@ <h1>Configuration</h1>
     const data = await res.json();
 
     if (data.valid) {
+      const saved = await postSetWorkspace(path);
+      if (!saved.ok) {
+        statusEl.className = 'alert alert-danger';
+        statusEl.textContent = saved.message;
+        statusEl.style.display = 'block';
+        return;
+      }
       localStorage.setItem('workspacePath', path);
-      await fetch('/api/set-workspace', {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ path })
-      });
       statusEl.className = 'alert alert-success';
       statusEl.textContent = `Found ${data.workspaceCount} workspaces in the specified location`;
       statusEl.style.display = 'block';

tests/test_set_workspace_multiworker.py

@@ -42,6 +42,10 @@ def test_multi_worker_returns_409_with_stable_code(self):
         self.assertEqual(body["code"], "set_workspace_multi_worker_unsupported")
         self.assertIn("WORKSPACE_PATH", body["error"])
 
+        from utils.workspace_path import get_workspace_path_override
+
+        self.assertIsNone(get_workspace_path_override())
+
     def test_single_process_still_succeeds_when_not_multi_worker(self):
         with patch(
             "api.config_api.is_multi_worker_process_deployment",