@pellux/goodvibes-sdk 0.21.20 → 0.21.23

package/dist/_internal/platform/workspace/workspace-swap-manager.js

@@ -27,6 +27,7 @@ import { createEventEnvelope } from '../runtime/events/index.js';
 export class WorkspaceSwapManager {
     deps;
     currentWorkingDir;
+    swapInProgress = null;
     constructor(initialWorkingDir, deps) {
         this.deps = deps;
         this.currentWorkingDir = initialWorkingDir;
@@ -35,6 +36,18 @@ export class WorkspaceSwapManager {
         return this.currentWorkingDir;
     }
     async requestSwap(newWorkingDir) {
+        if (this.swapInProgress) {
+            return { ok: false, code: 'WORKSPACE_BUSY', reason: 'Another workspace swap is already in progress.', retryAfter: 1 };
+        }
+        this.swapInProgress = this._requestSwapInner(newWorkingDir);
+        try {
+            return await this.swapInProgress;
+        }
+        finally {
+            this.swapInProgress = null;
+        }
+    }
+    async _requestSwapInner(newWorkingDir) {
         const raw = newWorkingDir.trim();
         if (!raw) {
             return { ok: false, code: 'INVALID_PATH', reason: 'Working directory path must not be empty.' };
@@ -42,21 +55,15 @@ export class WorkspaceSwapManager {
         const resolved = isAbsolute(raw) ? resolve(raw) : resolve(this.currentWorkingDir, raw);
         const from = this.currentWorkingDir;
         const to = resolved;
-        // Emit started
-        this._emit('WORKSPACE_SWAP_STARTED', { type: 'WORKSPACE_SWAP_STARTED', from, to });
-        // Check busy sessions
+        // Check busy sessions BEFORE emitting STARTED — don't emit STARTED if we'll immediately refuse.
         const busyCount = this.deps.getBusySessionCount();
         if (busyCount > 0) {
             const reason = `${busyCount} session(s) have pending input. Retry after active inputs complete.`;
-            this._emit('WORKSPACE_SWAP_REFUSED', {
-                type: 'WORKSPACE_SWAP_REFUSED',
-                from,
-                to,
-                reason,
-                retryAfter: 5,
-            });
+            this._emit({ type: 'WORKSPACE_SWAP_REFUSED', from, to, reason, retryAfter: 5 });
             return { ok: false, code: 'WORKSPACE_BUSY', reason, retryAfter: 5 };
         }
+        // Swap is proceeding — emit STARTED now that busy check passed.
+        this._emit({ type: 'WORKSPACE_SWAP_STARTED', from, to });
         // Validate and create directory
         try {
             mkdirSync(join(resolved, '.goodvibes', 'sessions'), { recursive: true });
@@ -86,29 +93,22 @@ export class WorkspaceSwapManager {
         catch {
             // Non-fatal — swap succeeded but persistence failed
         }
-        this._emit('WORKSPACE_SWAP_COMPLETED', {
-            type: 'WORKSPACE_SWAP_COMPLETED',
-            from,
-            to: resolved,
-            persistedInDaemonSettings,
-        });
+        this._emit({ type: 'WORKSPACE_SWAP_COMPLETED', from, to: resolved, persistedInDaemonSettings });
         return { ok: true, previous: from, current: resolved };
     }
     // ---------------------------------------------------------------------------
     // Internal helpers
     // ---------------------------------------------------------------------------
-    _emit(type, payload) {
+    _emit(payload) {
         if (!this.deps.runtimeBus)
             return;
         try {
-            const envelope = createEventEnvelope(type, payload, {
-                sessionId: 'system',
-                source: 'workspace-swap-manager',
-            });
-            // Cast: workspace events are not in AnyRuntimeEvent's narrow union but are
-            // registered under the 'workspace' domain. See analogous cast in
-            // runtime/health/effect-handlers.ts for CASCADE_APPLIED.
-            this.deps.runtimeBus.emit('workspace', envelope);
+            const envelope = createEventEnvelope(payload.type, payload, { sessionId: '', source: 'workspace-swap-manager' });
+            this.deps.runtimeBus.emit('workspace', 
+            // WorkspaceEvent is the DomainEventMap['workspace'] union; createEventEnvelope
+            // infers the payload type as the concrete member rather than the full union.
+            // A single widening cast (not through unknown) is safe here.
+            envelope);
         }
         catch {
             // Never throw from event emission

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pellux/goodvibes-sdk",
-  "version": "0.21.20",
+  "version": "0.21.23",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/mgd34msu/goodvibes-sdk.git"