@shepai/cli 1.119.0-pr381.415b252 → 1.119.0-pr382.f844b0c

package/dist/packages/core/src/infrastructure/services/agents/common/agent-executor-factory.service.js

@@ -45,7 +45,7 @@ export class AgentExecutorFactory {
                 executor = new ClaudeCodeExecutorService(this.spawn);
                 break;
             case 'cursor':
-                executor = new CursorExecutorService(this.spawn, _authConfig);
+                executor = new CursorExecutorService(this.spawn);
                 break;
             case 'dev':
                 executor = new DevAgentExecutorService();

package/dist/packages/core/src/infrastructure/services/agents/common/executors/cursor-executor.service.d.ts

@@ -8,7 +8,7 @@
  * Uses constructor dependency injection for the spawn function
  * to enable testability without mocking node:child_process directly.
  */
-import type { AgentType, AgentFeature, AgentConfig } from '../../../../../domain/generated/output.js';
+import type { AgentType, AgentFeature } from '../../../../../domain/generated/output.js';
 import type { IAgentExecutor, AgentExecutionOptions, AgentExecutionResult, AgentExecutionStreamEvent } from '../../../../../application/ports/output/agents/agent-executor.interface.js';
 import type { SpawnFunction } from '../types.js';
 /**
@@ -17,11 +17,10 @@ import type { SpawnFunction } from '../types.js';
  */
 export declare class CursorExecutorService implements IAgentExecutor {
     private readonly spawn;
-    private readonly authConfig?;
     readonly agentType: AgentType;
     /** When true, suppresses debug logging (set per-call via options.silent) */
     private silent;
-    constructor(spawn: SpawnFunction, authConfig?: AgentConfig | undefined);
+    constructor(spawn: SpawnFunction);
     /** Debug logging — writes to stdout so it appears in the worker log file */
     private log;
     supportsFeature(feature: AgentFeature): boolean;
@@ -34,35 +33,16 @@ export declare class CursorExecutorService implements IAgentExecutor {
     private buildArgs;
     private buildStreamArgs;
     /**
-     * Resolve the cursor CLI's bundled node.exe and index.js paths on Windows.
+     * Spawn the agent process, handling Windows specially via PowerShell.
      *
-     * The cursor CLI installs to `%LOCALAPPDATA%\cursor-agent\` with:
-     *   - `agent.cmd` / `cursor-agent.ps1` (shell wrappers)
-     *   - `versions/<YYYY.MM.DD-commit>/node.exe` + `index.js`
-     *
-     * Spawning via agent.cmd creates a problematic nesting chain:
-     *   PowerShell → agent.cmd → PowerShell → cursor-agent.ps1 → node.exe
-     * This chain hangs on Windows CI runners. Direct invocation bypasses it.
-     *
-     * @returns `{ nodePath, indexPath }` or `undefined` if not found
-     */
-    private resolveCursorBinary;
-    /**
-     * Spawn the agent process, handling Windows specially.
-     *
-     * On Windows, the cursor CLI ships as `agent.cmd` which spawns PowerShell
-     * which runs `cursor-agent.ps1` which runs `node.exe index.js`. This triple
-     * nesting (PowerShell → cmd.exe → PowerShell → node.exe) hangs on Windows
-     * CI runners. Solution: resolve the cursor CLI's bundled `node.exe` and
-     * `index.js` directly, bypassing all shell wrappers.
+     * On Windows, cursor CLI ships as `agent.cmd` which requires `shell: true`,
+     * but cmd.exe mangles long `-p` arguments (8191-char limit + special chars).
+     * Solution: write prompt to a temp file, invoke agent via PowerShell which
+     * reads the file and passes the content as `-p`. PowerShell handles long
+     * strings natively (32K limit) and doesn't mangle arguments.
      *
      * On Linux/macOS, spawn `agent` directly — no shell needed.
      */
-    /**
-     * Build the subprocess environment: strip CLAUDECODE, inject CURSOR_API_KEY
-     * from authConfig when token auth is configured.
-     */
-    private buildEnv;
     private spawnAgent;
     private parseStreamLine;
 }

package/dist/packages/core/src/infrastructure/services/agents/common/executors/cursor-executor.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cursor-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/cursor-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EACV,SAAS,EACT,YAAY,EACZ,WAAW,EACZ,MAAM,2CAA2C,CAAC;AACnD,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EACpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAuBjD;;;GAGG;AACH,qBAAa,qBAAsB,YAAW,cAAc;IAOxD,OAAO,CAAC,QAAQ,CAAC,KAAK;IACtB,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC;IAP9B,QAAQ,CAAC,SAAS,EAAE,SAAS,CAAyB;IAEtD,4EAA4E;IAC5E,OAAO,CAAC,MAAM,CAAS;gBAGJ,KAAK,EAAE,aAAa,EACpB,UAAU,CAAC,EAAE,WAAW,YAAA;IAG3C,4EAA4E;IAC5E,OAAO,CAAC,GAAG;IAMX,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAIzC,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IA2KtF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAkF3C;;OAEG;IACH,OAAO,CAAC,cAAc;IAkCtB,OAAO,CAAC,SAAS;IASjB,OAAO,CAAC,eAAe;IAOvB;;;;;;;;;;;;OAYG;IACH,OAAO,CAAC,mBAAmB;IAyB3B;;;;;;;;;;OAUG;IACH;;;OAGG;IACH,OAAO,CAAC,QAAQ;IAuBhB,OAAO,CAAC,UAAU;IA6ElB,OAAO,CAAC,eAAe;CAgDxB"}
+{"version":3,"file":"cursor-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/cursor-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACzF,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EACpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAuBjD;;;GAGG;AACH,qBAAa,qBAAsB,YAAW,cAAc;IAM9C,OAAO,CAAC,QAAQ,CAAC,KAAK;IALlC,QAAQ,CAAC,SAAS,EAAE,SAAS,CAAyB;IAEtD,4EAA4E;IAC5E,OAAO,CAAC,MAAM,CAAS;gBAEM,KAAK,EAAE,aAAa;IAEjD,4EAA4E;IAC5E,OAAO,CAAC,GAAG;IAMX,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAIzC,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAoItF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAkF3C;;OAEG;IACH,OAAO,CAAC,cAAc;IAkCtB,OAAO,CAAC,SAAS;IASjB,OAAO,CAAC,eAAe;IAOvB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,UAAU;IA2DlB,OAAO,CAAC,eAAe;CAgDxB"}

package/dist/packages/core/src/infrastructure/services/agents/common/executors/cursor-executor.service.js

@@ -8,7 +8,7 @@
  * Uses constructor dependency injection for the spawn function
  * to enable testability without mocking node:child_process directly.
  */
-import { writeFileSync, unlinkSync, readdirSync, existsSync } from 'node:fs';
+import { writeFileSync, unlinkSync } from 'node:fs';
 import { execFileSync } from 'node:child_process';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
@@ -36,13 +36,11 @@ const SUPPORTED_FEATURES = new Set(['session-resume', 'streaming']);
  */
 export class CursorExecutorService {
     spawn;
-    authConfig;
     agentType = 'cursor';
     /** When true, suppresses debug logging (set per-call via options.silent) */
     silent = false;
-    constructor(spawn, authConfig) {
+    constructor(spawn) {
         this.spawn = spawn;
-        this.authConfig = authConfig;
     }
     /** Debug logging — writes to stdout so it appears in the worker log file */
     log(message) {
@@ -60,34 +58,19 @@ export class CursorExecutorService {
         // stream-json is unreliable on Windows where shell: true can mangle args.
         const args = this.buildArgs(prompt, options);
         const { proc, tmpFile } = this.spawnAgent(prompt, args, options);
-        // Diagnostic: verify streams are connected and process is alive
-        this.log(`[diag] proc.pid=${proc.pid}, killed=${proc.killed}`);
-        this.log(`[diag] stdout=${!!proc.stdout}, stderr=${!!proc.stderr}, stdin=${!!proc.stdin}`);
-        this.log(`[diag] CURSOR_API_KEY set=${!!process.env.CURSOR_API_KEY}, len=${process.env.CURSOR_API_KEY?.length ?? 0}`);
-        this.log(`[diag] authConfig.token set=${!!this.authConfig?.token}, method=${this.authConfig?.authMethod ?? 'none'}`);
-        this.log(`[diag] platform=${process.platform}, cwd=${options?.cwd ?? '(inherited)'}`);
         return new Promise((resolve, reject) => {
             let lineBuffer = '';
             let stderr = '';
             let timedOut = false;
             let timeoutId;
-            let stdoutChunks = 0;
-            let stderrChunks = 0;
-            const startTime = Date.now();
             // Accumulated from JSON events or raw text fallback
             let resultText = '';
             let rawText = '';
             let sessionId;
             let metadata;
-            // Periodic heartbeat to prove the executor is alive and waiting
-            const heartbeatId = setInterval(() => {
-                const elapsed = Math.round((Date.now() - startTime) / 1000);
-                this.log(`[diag] heartbeat ${elapsed}s: stdout_chunks=${stdoutChunks}, stderr_chunks=${stderrChunks}, pid_killed=${proc.killed}`);
-            }, 15_000);
             if (options?.timeout) {
                 timeoutId = setTimeout(() => {
                     timedOut = true;
-                    this.log(`[diag] TIMEOUT after ${options.timeout}ms — killing process tree`);
                     // On Windows, proc.kill() may not kill the entire process tree
                     // (PowerShell + child agent process). Use taskkill /T for tree kill.
                     if (process.platform === 'win32' && proc.pid) {
@@ -95,11 +78,9 @@ export class CursorExecutorService {
                             execFileSync('taskkill', ['/F', '/T', '/PID', String(proc.pid)], {
                                 stdio: 'ignore',
                             });
-                            this.log(`[diag] taskkill /T /PID ${proc.pid} succeeded`);
                         }
                         catch {
                             proc.kill();
-                            this.log(`[diag] taskkill failed, fell back to proc.kill()`);
                         }
                     }
                     else {
@@ -136,12 +117,7 @@ export class CursorExecutorService {
                 }
             };
             proc.stdout?.on('data', (chunk) => {
-                stdoutChunks++;
-                const data = chunk.toString();
-                if (stdoutChunks <= 3) {
-                    this.log(`[diag] stdout chunk #${stdoutChunks}: ${data.length} bytes, first 200: ${data.slice(0, 200).replace(/\n/g, '\\n')}`);
-                }
-                lineBuffer += data;
+                lineBuffer += chunk.toString();
                 const lines = lineBuffer.split('\n');
                 lineBuffer = lines.pop() ?? '';
                 for (const line of lines) {
@@ -151,22 +127,19 @@ export class CursorExecutorService {
                 }
             });
             proc.stderr?.on('data', (chunk) => {
-                stderrChunks++;
                 const data = chunk.toString();
                 stderr += data;
-                this.log(`[diag] stderr chunk #${stderrChunks}: ${data.trimEnd()}`);
+                this.log(`stderr: ${data.trimEnd()}`);
                 // Detect fatal errors early so callers don't waste time retrying
                 if (data.includes('Cannot use this model')) {
                     if (timeoutId)
                         clearTimeout(timeoutId);
-                    clearInterval(heartbeatId);
                     proc.kill();
                     reject(new Error(data.trim()));
                 }
             });
             proc.on('error', (error) => {
-                this.log(`[diag] Process ERROR event: ${error.message} (code=${error.code})`);
-                clearInterval(heartbeatId);
+                this.log(`Process error event: ${error.message}`);
                 if (timeoutId)
                     clearTimeout(timeoutId);
                 if (error.code === 'ENOENT') {
@@ -177,8 +150,6 @@ export class CursorExecutorService {
                 }
             });
             proc.on('close', (code) => {
-                const elapsed = Math.round((Date.now() - startTime) / 1000);
-                clearInterval(heartbeatId);
                 // Clean up temp file on Windows
                 if (tmpFile) {
                     try {
@@ -192,7 +163,7 @@ export class CursorExecutorService {
                     processLine(lineBuffer.trim());
                 // Use raw text as fallback when no JSON result was captured
                 const finalText = resultText || rawText.trim();
-                this.log(`[diag] Process CLOSE: code=${code}, elapsed=${elapsed}s, stdout_chunks=${stdoutChunks}, stderr_chunks=${stderrChunks}, result=${finalText.length} chars, raw=${rawText.length} chars`);
+                this.log(`Process closed with code ${code}, result=${finalText.length} chars`);
                 if (timeoutId)
                     clearTimeout(timeoutId);
                 if (timedOut) {
@@ -339,104 +310,36 @@ export class CursorExecutorService {
         return args;
     }
     /**
-     * Resolve the cursor CLI's bundled node.exe and index.js paths on Windows.
-     *
-     * The cursor CLI installs to `%LOCALAPPDATA%\cursor-agent\` with:
-     *   - `agent.cmd` / `cursor-agent.ps1` (shell wrappers)
-     *   - `versions/<YYYY.MM.DD-commit>/node.exe` + `index.js`
-     *
-     * Spawning via agent.cmd creates a problematic nesting chain:
-     *   PowerShell → agent.cmd → PowerShell → cursor-agent.ps1 → node.exe
-     * This chain hangs on Windows CI runners. Direct invocation bypasses it.
-     *
-     * @returns `{ nodePath, indexPath }` or `undefined` if not found
-     */
-    resolveCursorBinary() {
-        const localAppData = process.env.LOCALAPPDATA;
-        if (!localAppData)
-            return undefined;
-        const versionsDir = join(localAppData, 'cursor-agent', 'versions');
-        if (!existsSync(versionsDir))
-            return undefined;
-        // Find the latest version directory (format: YYYY.MM.DD-commit)
-        const versionPattern = /^\d{4}\.\d{1,2}\.\d{1,2}-[a-f0-9]+$/;
-        const versions = readdirSync(versionsDir)
-            .filter((d) => versionPattern.test(d))
-            .sort()
-            .reverse();
-        if (versions.length === 0)
-            return undefined;
-        const versionDir = join(versionsDir, versions[0]);
-        const nodePath = join(versionDir, 'node.exe');
-        const indexPath = join(versionDir, 'index.js');
-        if (!existsSync(nodePath) || !existsSync(indexPath))
-            return undefined;
-        return { nodePath, indexPath };
-    }
-    /**
-     * Spawn the agent process, handling Windows specially.
+     * Spawn the agent process, handling Windows specially via PowerShell.
      *
-     * On Windows, the cursor CLI ships as `agent.cmd` which spawns PowerShell
-     * which runs `cursor-agent.ps1` which runs `node.exe index.js`. This triple
-     * nesting (PowerShell → cmd.exe → PowerShell → node.exe) hangs on Windows
-     * CI runners. Solution: resolve the cursor CLI's bundled `node.exe` and
-     * `index.js` directly, bypassing all shell wrappers.
+     * On Windows, cursor CLI ships as `agent.cmd` which requires `shell: true`,
+     * but cmd.exe mangles long `-p` arguments (8191-char limit + special chars).
+     * Solution: write prompt to a temp file, invoke agent via PowerShell which
+     * reads the file and passes the content as `-p`. PowerShell handles long
+     * strings natively (32K limit) and doesn't mangle arguments.
      *
      * On Linux/macOS, spawn `agent` directly — no shell needed.
      */
-    /**
-     * Build the subprocess environment: strip CLAUDECODE, inject CURSOR_API_KEY
-     * from authConfig when token auth is configured.
-     */
-    buildEnv(extra) {
-        const { CLAUDECODE: _, ...cleanEnv } = process.env;
-        // Inject CURSOR_API_KEY from stored settings so the cursor CLI authenticates
-        // even when the env var isn't inherited (e.g. direct node.exe invocation on Windows).
-        if (this.authConfig?.authMethod === 'token' && this.authConfig.token) {
-            cleanEnv.CURSOR_API_KEY = this.authConfig.token;
-            this.log(`[diag] buildEnv: injected CURSOR_API_KEY from authConfig (${this.authConfig.token.length} chars)`);
-        }
-        else {
-            this.log(`[diag] buildEnv: NO token injection (authMethod=${this.authConfig?.authMethod ?? 'none'}, hasToken=${!!this.authConfig?.token}, envKey=${!!cleanEnv.CURSOR_API_KEY})`);
-        }
-        const env = extra ? { ...cleanEnv, ...extra } : cleanEnv;
-        this.log(`[diag] buildEnv: final CURSOR_API_KEY in env=${!!env.CURSOR_API_KEY}, len=${env.CURSOR_API_KEY?.length ?? 0}`);
-        return env;
-    }
     spawnAgent(prompt, args, options) {
+        const { CLAUDECODE: _, ...cleanEnv } = process.env;
         const cwd = options?.cwd;
         if (IS_WINDOWS) {
-            const resolved = this.resolveCursorBinary();
-            if (resolved) {
-                // Direct invocation: node.exe index.js <args>
-                // No shell wrappers, no nesting, no hangs.
-                const directArgs = [resolved.indexPath, ...args];
-                this.log(`Windows direct mode: ${resolved.nodePath}`);
-                this.log(`Args: ${directArgs.map((a) => (a.length > 80 ? `${a.slice(0, 77)}...` : a)).join(' ')}`);
-                const proc = this.spawn(resolved.nodePath, directArgs, {
-                    cwd,
-                    stdio: ['pipe', 'pipe', 'pipe'],
-                    windowsHide: true,
-                    env: this.buildEnv({ CURSOR_INVOKED_AS: 'agent' }),
-                });
-                this.log(`Direct PID: ${proc.pid ?? 'undefined (spawn may have failed)'}`);
-                if (proc.stdin)
-                    proc.stdin.end();
-                return { proc, tmpFile: undefined };
-            }
-            // Fallback: if direct resolution fails, use PowerShell + temp file
-            this.log('Windows fallback: cursor binary not resolved, using PowerShell');
+            // Write prompt to temp file to bypass cmd.exe argument mangling
             const tmpFile = join(tmpdir(), `shep-cursor-${Date.now()}-${Math.random().toString(36).slice(2, 6)}.txt`);
             writeFileSync(tmpFile, prompt, 'utf8');
+            // Build the agent args WITHOUT -p and the prompt (they go via temp file)
             const agentFlags = args.filter((a) => a !== '-p' && a !== prompt).join(' ');
             const safePath = tmpFile.replace(/'/g, "''");
             const psCmd = `$p = Get-Content -Raw '${safePath}'; & agent ${agentFlags} -p $p`;
+            this.log(`Windows PowerShell mode: wrote ${prompt.length} chars to ${tmpFile}`);
+            this.log(`PS command: ${psCmd.replace(prompt, `<${prompt.length} chars>`)}`);
             const proc = this.spawn('powershell.exe', ['-NoProfile', '-NonInteractive', '-Command', psCmd], {
                 cwd,
                 stdio: ['pipe', 'pipe', 'pipe'],
                 windowsHide: true,
-                env: this.buildEnv(),
+                env: cleanEnv,
             });
+            this.log(`PowerShell PID: ${proc.pid ?? 'undefined (spawn may have failed)'}`);
             if (proc.stdin)
                 proc.stdin.end();
             return { proc, tmpFile };
@@ -444,7 +347,7 @@ export class CursorExecutorService {
         // Linux/macOS: spawn agent directly, no shell needed
         const spawnOpts = {
             stdio: ['pipe', 'pipe', 'pipe'],
-            env: this.buildEnv(),
+            env: cleanEnv,
         };
         if (cwd)
             spawnOpts.cwd = cwd;

package/dist/src/presentation/web/components/layouts/app-shell/app-shell.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"app-shell.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/layouts/app-shell/app-shell.tsx"],"names":[],"mappings":"AAEA,OAAO,EAAkC,KAAK,SAAS,EAAE,MAAM,OAAO,CAAC;AAsBvE,UAAU,aAAa;IACrB,QAAQ,EAAE,SAAS,CAAC;CACrB;AAiFD,wBAAgB,QAAQ,CAAC,EAAE,QAAQ,EAAE,EAAE,aAAa,2CAUnD"}
+{"version":3,"file":"app-shell.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/layouts/app-shell/app-shell.tsx"],"names":[],"mappings":"AAEA,OAAO,EAAkC,KAAK,SAAS,EAAE,MAAM,OAAO,CAAC;AAsBvE,UAAU,aAAa;IACrB,QAAQ,EAAE,SAAS,CAAC;CACrB;AA0FD,wBAAgB,QAAQ,CAAC,EAAE,QAAQ,EAAE,EAAE,aAAa,2CAUnD"}

package/dist/src/presentation/web/components/layouts/app-shell/app-shell.js

@@ -1,7 +1,7 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { useCallback, useMemo, useState } from 'react';
-import { useRouter } from 'next/navigation';
+import { useRouter, usePathname } from 'next/navigation';
 import { Plus, FolderPlus } from 'lucide-react';
 import { SidebarProvider, SidebarInset } from '../../ui/sidebar.js';
 import { AppSidebar } from '../../layouts/app-sidebar/index.js';
@@ -14,9 +14,14 @@ import { DrawerCloseGuardProvider, useDrawerCloseGuard } from '../../../hooks/dr
 import { SidebarFeaturesProvider, useSidebarFeaturesContext, } from '../../../hooks/sidebar-features-context.js';
 import { useNotifications } from '../../../hooks/use-notifications.js';
 import { useFeatureFlags } from '../../../hooks/feature-flags-context.js';
-/** Inner shell that consumes the agent-events context for notifications. */
+/** Control center route prefixes where the FAB should be visible. */
+const CONTROL_CENTER_PREFIXES = ['/', '/create', '/feature', '/repository'];
+function isControlCenterRoute(pathname) {
+    return CONTROL_CENTER_PREFIXES.some((prefix) => pathname === prefix || (prefix !== '/' && pathname.startsWith(`${prefix}/`)));
+}
 function AppShellInner({ children }) {
     const router = useRouter();
+    const pathname = usePathname();
     const { guardedNavigate } = useDrawerCloseGuard();
     const featureFlags = useFeatureFlags();
     // Subscribe to agent lifecycle events and dispatch toast/browser notifications
@@ -58,7 +63,7 @@ function AppShellInner({ children }) {
             loading: addingRepo,
         },
     ], [handleNewFeature, handleAddRepository, addingRepo]);
-    return (_jsxs(SidebarProvider, { defaultOpen: false, children: [_jsx(AppSidebar, { features: features, featureFlags: featureFlags, onFeatureClick: handleFeatureClick }), _jsx(SidebarInset, { children: _jsxs("div", { className: "relative h-full", children: [_jsxs("div", { className: "absolute top-3 right-3 z-50 flex gap-1", "data-test-id": "canvas-actions-toolbar", children: [_jsx(SoundToggle, {}), _jsx(ThemeToggle, {})] }), _jsx("main", { className: "h-full", children: children }), _jsx(FloatingActionButton, { actions: fabActions })] }) })] }));
+    return (_jsxs(SidebarProvider, { defaultOpen: false, children: [_jsx(AppSidebar, { features: features, featureFlags: featureFlags, onFeatureClick: handleFeatureClick }), _jsx(SidebarInset, { children: _jsxs("div", { className: "relative h-full", children: [_jsxs("div", { className: "absolute top-3 right-3 z-50 flex gap-1", "data-test-id": "canvas-actions-toolbar", children: [_jsx(SoundToggle, {}), _jsx(ThemeToggle, {})] }), _jsx("main", { className: "h-full", children: children }), isControlCenterRoute(pathname) && _jsx(FloatingActionButton, { actions: fabActions })] }) })] }));
 }
 export function AppShell({ children }) {
     return (_jsx(AgentEventsProvider, { children: _jsx(DrawerCloseGuardProvider, { children: _jsx(SidebarFeaturesProvider, { children: _jsx(AppShellInner, { children: children }) }) }) }));