pnpm-dash 0.1.2 → 0.1.4

package/dist/cli.js

@@ -4,7 +4,7 @@ export function parseCLI() {
     program
         .name('pnpm-dash')
         .description('A TUI dashboard for pnpm workspaces - run scripts across packages with a split-pane interface')
-        .version('0.1.2')
+        .version('0.1.3')
         .argument('<script>', 'Script name to run across workspace packages (e.g., dev, start)')
         .option('-F, --filter <pattern...>', 'Filter packages by name pattern, supports * for wildcard and ! for exclusions')
         .parse();

package/dist/runner.js

@@ -1,4 +1,4 @@
-import { execa } from 'execa';
+import { spawn } from 'node:child_process';
 import { EventEmitter } from 'node:events';
 import { RingBuffer } from './ringbuf.js';
 import { MAX_LOG_LINES } from './constants.js';
@@ -21,7 +21,6 @@ export class Runner extends EventEmitter {
         let state = this.states.get(pkg.name);
         if (state) {
             if (state.subprocess && state.subprocess.exitCode == null) {
-                console.error(`${pkg.name} subprocess is still running`, state.subprocess.pid);
                 return;
             }
             state.status = 'running';
@@ -35,21 +34,19 @@ export class Runner extends EventEmitter {
             };
             this.states.set(pkg.name, state);
         }
-        const subprocess = execa('pnpm', ['run', this.scriptName], {
+        const cmd = process.platform === 'win32' ? 'pnpm.cmd' : 'pnpm';
+        const subprocess = spawn(cmd, ['run', this.scriptName], {
             cwd: pkg.path,
             env: {
                 ...process.env,
                 FORCE_COLOR: '1',
             },
-            all: true,
-            buffer: false,
-            reject: false,
-            cleanup: false,
+            stdio: ['ignore', 'pipe', 'pipe'],
             detached: true,
         });
         state.subprocess = subprocess;
         this.emit('start', pkg.name);
-        subprocess.all.on('data', (data) => {
+        const handleData = (data) => {
             const lines = data.toString().split('\n');
             for (const line of lines) {
                 if (line) {
@@ -57,15 +54,20 @@ export class Runner extends EventEmitter {
                     this.emit('log', pkg.name, line);
                 }
             }
-        });
-        subprocess.then((result) => {
-            state.status = result.exitCode === 0 ? 'success' : 'error';
+        };
+        subprocess.stdout?.on('data', handleData);
+        subprocess.stderr?.on('data', handleData);
+        subprocess.on('close', (code) => {
+            state.status = code === 0 ? 'success' : 'error';
             state.subprocess = null;
-            this.emit('exit', pkg.name, result.exitCode ?? null);
-        }).catch((error) => {
+            this.emit('exit', pkg.name, code);
+        });
+        subprocess.on('error', (error) => {
             state.status = 'error';
-            state.logs.push(`Error: ${error.message}`);
             state.subprocess = null;
+            const logLine = `Error: ${error.message}`;
+            state.logs.push(logLine);
+            this.emit('log', pkg.name, logLine);
             this.emit('error', pkg.name, error);
         });
     }
@@ -103,16 +105,16 @@ export class Runner extends EventEmitter {
         if (!subprocess?.pid)
             return;
         const pid = subprocess.pid;
-        this.killProcessGroup(pid);
-        const forceKillTimeout = setTimeout(() => {
-            this.killProcessGroup(pid, true);
-        }, 2000);
-        try {
-            await subprocess;
-        }
-        finally {
-            clearTimeout(forceKillTimeout);
-        }
+        return new Promise((resolve) => {
+            const forceKillTimeout = setTimeout(() => {
+                this.killProcessGroup(pid, true);
+            }, 2000);
+            subprocess.once('close', () => {
+                clearTimeout(forceKillTimeout);
+                resolve();
+            });
+            this.killProcessGroup(pid);
+        });
     }
     async stopAll() {
         await Promise.all(Array.from(this.states.keys()).map((name) => this.stopPackage(name)));

package/dist/ui/dashboard.js

@@ -2,6 +2,7 @@ import blessed from 'reblessed';
 import { createSidebar, updateSidebarItems } from './sidebar.js';
 import { createLogView, updateLogView, appendLog, toggleLogAutoScroll, expandLogView, shrinkLogView } from './logview.js';
 import { createStatusBar, updateStatusBar } from './statusbar.js';
+const RENDER_INTERVAL = 33;
 export class Dashboard {
     screen;
     sidebar;
@@ -10,6 +11,9 @@ export class Dashboard {
     runner;
     state;
     packageNames = [];
+    renderTimer = null;
+    needsRender = false;
+    pendingLogs = [];
     constructor(runner, packages) {
         this.runner = runner;
         this.packageNames = packages.map((p) => p.name);
@@ -62,20 +66,33 @@ export class Dashboard {
     setupRunnerEvents() {
         this.runner.on('start', (packageName) => {
             this.refreshSidebar();
-            this.screen.render();
+            this.needsRender = true;
         });
         this.runner.on('log', (packageName, line) => {
-            appendLog(this.logView, this.getSelectedPackageName(), packageName, line);
+            if (packageName === this.getSelectedPackageName()) {
+                this.pendingLogs.push(line);
+                this.needsRender = true;
+            }
         });
         this.runner.on('exit', (packageName, code) => {
             this.refreshSidebar();
-            this.screen.render();
+            this.needsRender = true;
         });
         this.runner.on('error', (packageName, error) => {
             this.refreshSidebar();
-            this.screen.render();
+            this.needsRender = true;
         });
     }
+    flushRender() {
+        if (this.pendingLogs.length > 0) {
+            appendLog(this.logView, this.pendingLogs);
+            this.pendingLogs = [];
+        }
+        if (this.needsRender) {
+            this.screen.render();
+            this.needsRender = false;
+        }
+    }
     getSelectedPackageName() {
         return this.packageNames[this.state.selectedIndex];
     }
@@ -92,6 +109,8 @@ export class Dashboard {
         }
         this.refreshSidebar();
         this.refreshLogView();
+        this.pendingLogs = [];
+        this.needsRender = true;
     }
     selectPrev() {
         if (this.state.selectedIndex > 0) {
@@ -102,12 +121,15 @@ export class Dashboard {
         }
         this.refreshSidebar();
         this.refreshLogView();
+        this.pendingLogs = [];
+        this.needsRender = true;
     }
     clearSelected() {
         const state = this.getSelectedState();
         if (state) {
             state.logs.clear();
             this.refreshLogView();
+            this.needsRender = true;
         }
     }
     stopSelected() {
@@ -128,8 +150,8 @@ export class Dashboard {
     toggleAutoScroll() {
         this.state.autoScroll = !this.state.autoScroll;
         toggleLogAutoScroll(this.logView, this.state.autoScroll);
-        updateStatusBar(this.statusBar, this.state.autoScroll);
-        this.screen.render();
+        this.refreshStatusBar();
+        this.needsRender = true;
     }
     toggleSidebar() {
         this.state.sidebarHidden = !this.state.sidebarHidden;
@@ -141,8 +163,7 @@ export class Dashboard {
             this.sidebar.show();
             shrinkLogView(this.logView);
         }
-        updateStatusBar(this.statusBar, this.state.autoScroll);
-        this.screen.render();
+        this.needsRender = true;
     }
     refreshSidebar() {
         updateSidebarItems(this.sidebar, this.state.packages, this.state.selectedIndex);
@@ -150,7 +171,14 @@ export class Dashboard {
     refreshLogView() {
         updateLogView(this.logView, this.getSelectedState());
     }
+    refreshStatusBar() {
+        updateStatusBar(this.statusBar, this.state.autoScroll);
+    }
     async quit() {
+        if (this.renderTimer) {
+            clearInterval(this.renderTimer);
+            this.renderTimer = null;
+        }
         await this.runner.stopAll();
         this.screen.destroy();
         process.exit(0);
@@ -158,7 +186,9 @@ export class Dashboard {
     start() {
         this.refreshSidebar();
         this.refreshLogView();
+        this.refreshStatusBar();
         this.logView.focus();
         this.screen.render();
+        this.renderTimer = setInterval(() => this.flushRender(), RENDER_INTERVAL);
     }
 }

package/dist/ui/logview.js

@@ -39,11 +39,8 @@ export function updateLogView(logView, state) {
     logView.setContent(state.logs.toArray().join('\n'));
     logView.setScroll(0);
 }
-export function appendLog(logView, currentPackage, packageName, line) {
-    if (currentPackage !== packageName) {
-        return;
-    }
-    logView.add(line);
+export function appendLog(logView, lines) {
+    logView.add(lines.join("\n"));
 }
 export function toggleLogAutoScroll(logView, autoScroll) {
     logView.scrollOnInput = autoScroll;

package/dist/ui/statusbar.js

@@ -12,7 +12,6 @@ export function createStatusBar(screen) {
         },
         tags: true,
     });
-    updateStatusBar(statusBar, true);
     return statusBar;
 }
 export function updateStatusBar(statusBar, autoScroll) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pnpm-dash",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "A TUI dashboard for pnpm workspaces - run scripts across packages with a split-pane interface",
   "type": "module",
   "main": "./dist/index.js",
@@ -25,11 +25,14 @@
   ],
   "author": "artygus",
   "license": "Unlicense",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/artygus/pnpm-dash.git"
+  },
   "dependencies": {
     "@pnpm/find-workspace-dir": "^7.0.2",
     "@pnpm/workspace.find-packages": "^4.0.5",
     "commander": "^13.1.0",
-    "execa": "^9.6.1",
     "reblessed": "^0.2.1"
   },
   "devDependencies": {