@pixelbyte-software/pixcode 1.48.2 → 1.48.4

package/scripts/smoke/vscode-workbench-polish.mjs

@@ -12,6 +12,10 @@ const chatComposer = read('src/components/chat/view/subcomponents/ChatComposer.t
 const workerSlots = read('src/components/chat/view/subcomponents/WorkerSlotsControl.tsx');
 const wizard = read('src/components/project-creation-wizard/ProjectCreationWizard.tsx');
 const sidebar = read('src/components/sidebar/view/Sidebar.tsx');
+const shellTerminal = read('src/components/shell/hooks/useShellTerminal.ts');
+const gitPanel = read('src/components/git-panel/view/GitPanel.tsx');
+const gitPanelHeader = read('src/components/git-panel/view/GitPanelHeader.tsx');
+const gitViewTabs = read('src/components/git-panel/view/GitViewTabs.tsx');
 
 assert.match(
   workbench,
@@ -59,6 +63,51 @@ assert.match(
   'Workbench should render Chrome-style workspace tabs directly under the menu bar.',
 );
 
+assert.doesNotMatch(
+  workbench,
+  /\.\.\.currentTabs\.filter\(\(tab\) => tab\.id !== tabId\)/,
+  'Selecting an existing workspace tab must preserve tab order instead of moving it to the end.',
+);
+
+assert.match(
+  workbench,
+  /function WorkspaceTabContextMenu/,
+  'Workspace tabs should expose their actions through a right-click context menu.',
+);
+
+assert.match(
+  workbench,
+  /onContextMenu=\{\(event\) => openWorkspaceContextMenu\(event, tab\)\}/,
+  'Workspace tab actions should open from right-click on the tab.',
+);
+
+const workspaceTabsSource = workbench.slice(
+  workbench.indexOf('function WorkbenchWorkspaceTabs'),
+  workbench.indexOf('function EditorTabContextMenu'),
+);
+
+assert.doesNotMatch(
+  workspaceTabsSource,
+  /MoreHorizontal/,
+  'Workspace tabs should not render a three-dot action button.',
+);
+
+for (const token of ['closeOtherWorkspaces', 'closeAllWorkspaces']) {
+  assert.match(workbench, new RegExp(token), `Workspace tab context menu should support ${token}.`);
+}
+
+assert.match(
+  workspaceTabsSource,
+  /items-center justify-center/,
+  'Workspace add button should center its plus icon instead of rendering an off-center bare icon.',
+);
+
+assert.match(
+  workspaceTabsSource,
+  /border-r border-border/,
+  'Workspace add button should read as part of the tab strip instead of a floating bare button.',
+);
+
 assert.match(
   workbench,
   /WORKBENCH_WORKSPACE_TABS_STORAGE_KEY/,
@@ -113,8 +162,8 @@ assert.match(
 
 assert.match(
   workbench,
-  /onClose=\{\(\) => setIsTerminalOpen\(false\)\}/,
-  'Right CLI terminal close button should return to the picker instead of leaving a dead reconnect overlay.',
+  /onClose=\{closeTerminal\}/,
+  'Right CLI terminal close button should return to the picker through the persisted close flow.',
 );
 
 assert.match(
@@ -123,6 +172,48 @@ assert.match(
   'CLI history should be integrated as a polished project-scoped panel.',
 );
 
+assert.match(
+  workbench,
+  /terminalSession/,
+  'Right CLI panel should track whether the terminal is running a new session or a selected history session.',
+);
+
+assert.match(
+  workbench,
+  /WORKBENCH_CLI_STATE_STORAGE_KEY/,
+  'Right CLI panel should persist per-project terminal state when switching workspaces.',
+);
+
+assert.match(
+  workbench,
+  /openNewCliSessionPicker/,
+  'Right CLI panel toolbar plus should stop the current terminal view and return to CLI selection.',
+);
+
+assert.match(
+  workbench,
+  /terminateCurrentCliSession\(selectedProvider\)/,
+  'Right CLI panel toolbar plus should explicitly terminate the current provider PTY before showing the picker.',
+);
+
+assert.match(
+  workbench,
+  /forceNewSession=\{terminalLaunch\.forceNewSession\}/,
+  'Right CLI panel should mark explicitly started new sessions so the backend does not reconnect the old PTY.',
+);
+
+assert.match(
+  workbench,
+  /function WorkbenchCliPanelToolbar/,
+  'Right CLI terminal should keep compact History and New Session actions visible while the terminal is open.',
+);
+
+assert.match(
+  workbench,
+  /onNewSession=\{openNewCliSessionPicker\}/,
+  'Right CLI terminal toolbar should wire the plus button to the new-session picker flow.',
+);
+
 assert.doesNotMatch(
   workbench,
   /return <Sidebar \{\.\.\.sidebarProps\} isMobile=\{false\} \/>/,
@@ -170,6 +261,28 @@ assert.match(
   'Projects activity should stay selected while the center chat tab is active.',
 );
 
+assert.match(
+  workbench,
+  /setActivityPanel\('explorer'\)/,
+  'Selecting a project from Projects should switch the left pane back to Explorer.',
+);
+
+assert.match(
+  workbench,
+  /<GitPanel selectedProject=\{selectedProject\} isMobile=\{false\} compact onFileOpen=\{handleFileOpen\}/,
+  'Source Control should render in compact icon-first mode inside the VS Code workbench side panel.',
+);
+
+assert.match(gitPanel, /compact = false/, 'GitPanel should accept a compact prop.');
+assert.match(gitPanelHeader, /compact/, 'GitPanelHeader should receive compact mode for narrow panes.');
+assert.match(gitViewTabs, /compact/, 'GitViewTabs should render compact icon-only tabs.');
+
+assert.doesNotMatch(
+  shellTerminal,
+  /new WebglAddon\(\)/,
+  'Shell terminal should avoid the WebGL renderer that can leave stale glyph trails with OpenCode output.',
+);
+
 assert.match(chatInterface, /compactComposer\?: boolean/, 'ChatInterface should expose compactComposer for narrow workbench panes.');
 assert.match(chatComposer, /compact\?: boolean/, 'ChatComposer should expose a compact prop.');
 assert.match(chatComposer, /flex-wrap/, 'ChatComposer footer should wrap controls in narrow panes.');

package/server/index.js

@@ -285,8 +285,44 @@ const server = http.createServer(app);
 const ptySessionsMap = new Map();
 const PTY_SESSION_TIMEOUT = 30 * 60 * 1000;
 const SHELL_URL_PARSE_BUFFER_LIMIT = 32768;
+const SHELL_CLI_PROVIDERS = new Set(['claude', 'codex', 'cursor', 'gemini', 'qwen', 'opencode']);
 import { stripAnsiSequences, normalizeDetectedUrl, extractUrlsFromText, shouldAutoOpenUrlFromOutput } from './utils/url-detection.js';
 
+function terminatePtySession(sessionKey, session, reason) {
+    if (!session) return false;
+
+    console.log(`🧹 Terminating PTY session (${reason}):`, sessionKey);
+    if (session.timeoutId) {
+        clearTimeout(session.timeoutId);
+    }
+
+    try {
+        if (session.pty && session.pty.kill) {
+            session.pty.kill();
+        }
+    } catch (error) {
+        console.warn('Failed to kill PTY session:', error.message);
+    }
+
+    ptySessionsMap.delete(sessionKey);
+    return true;
+}
+
+function killProviderPtySessions(projectPath, provider) {
+    let killed = 0;
+    for (const [sessionKey, session] of ptySessionsMap.entries()) {
+        if (
+            session?.projectPath === projectPath &&
+            session?.provider === provider &&
+            !session?.isPlainShell
+        ) {
+            killed += terminatePtySession(sessionKey, session, 'fresh provider session') ? 1 : 0;
+        }
+    }
+
+    return killed;
+}
+
 // Single WebSocket server that handles both paths
 const wss = new WebSocketServer({
     server,
@@ -358,6 +394,18 @@ app.get('/health', (req, res) => {
 // Optional API key validation (if configured)
 app.use('/api', validateApiKey);
 
+app.post('/api/shell/sessions/terminate', authenticateToken, (req, res) => {
+    const provider = req.body?.provider || 'claude';
+    const projectPath = req.body?.projectPath || os.homedir();
+
+    if (!SHELL_CLI_PROVIDERS.has(provider)) {
+        return res.status(400).json({ error: 'Unsupported provider' });
+    }
+
+    const killedSessions = killProviderPtySessions(projectPath, provider);
+    res.json({ success: true, killedSessions });
+});
+
 // Authentication routes (public)
 app.use('/api/auth', authRoutes);
 
@@ -2044,6 +2092,7 @@ function handleShellConnection(ws) {
                 const provider = data.provider || 'claude';
                 const initialCommand = data.initialCommand;
                 const isPlainShell = data.isPlainShell || (!!initialCommand && !hasSession) || provider === 'plain-shell';
+                const forceNewSession = Boolean(data.forceNewSession && !isPlainShell);
                 urlDetectionBuffer = '';
                 announcedAuthUrls.clear();
 
@@ -2077,14 +2126,16 @@ function handleShellConnection(ws) {
                 if (isLoginCommand) {
                     const oldSession = ptySessionsMap.get(ptySessionKey);
                     if (oldSession) {
-                        console.log('🧹 Cleaning up existing login session:', ptySessionKey);
-                        if (oldSession.timeoutId) clearTimeout(oldSession.timeoutId);
-                        if (oldSession.pty && oldSession.pty.kill) oldSession.pty.kill();
-                        ptySessionsMap.delete(ptySessionKey);
+                        terminatePtySession(ptySessionKey, oldSession, 'fresh login');
+                    }
+                } else if (forceNewSession) {
+                    const killedSessions = killProviderPtySessions(projectPath, provider);
+                    if (killedSessions > 0) {
+                        console.log(`🧹 Fresh ${provider} session requested; terminated ${killedSessions} cached PTY session(s).`);
                     }
                 }
 
-                const existingSession = isLoginCommand ? null : ptySessionsMap.get(ptySessionKey);
+                const existingSession = (isLoginCommand || forceNewSession) ? null : ptySessionsMap.get(ptySessionKey);
                 if (existingSession) {
                     console.log('♻️  Reconnecting to existing PTY session:', ptySessionKey);
                     shellProcess = existingSession.pty;
@@ -2292,7 +2343,9 @@ function handleShellConnection(ws) {
                         buffer: [],
                         timeoutId: null,
                         projectPath,
-                        sessionId
+                        sessionId,
+                        provider,
+                        isPlainShell
                     });
 
                     // Handle data output