@bramblex/codex-workbench 0.1.16 → 0.1.18

package/README.md

@@ -140,6 +140,8 @@ codex-workbench auto-detects installed backends by checking their session direct
 
 Session metadata such as custom names, notes, and archive state is stored in workbench's own metadata file, not inside backend session files.
 
+Every provider owns the full workbench command surface it advertises: new, resume, fork, archive, unarchive, and delete. A provider can implement an operation through its native CLI, workbench metadata, or file operations, but callers should not need provider-specific fallback logic.
+
 ---
 
 ## CLI commands

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bramblex/codex-workbench",
-  "version": "0.1.16",
+  "version": "0.1.18",
   "description": "Terminal workbench for browsing and managing local and SSH Codex sessions.",
   "license": "MIT",
   "repository": {
@@ -33,7 +33,7 @@
     "LICENSE"
   ],
   "scripts": {
-    "test": "node -e \"const fs=require('fs'),{spawnSync}=require('child_process');function files(d){return fs.readdirSync(d,{withFileTypes:true}).flatMap(e=>e.isDirectory()?files(d+'/'+e.name):e.name.endsWith('.js')?[d+'/'+e.name]:[])}for(const f of files('src')){const r=spawnSync(process.execPath,['--check',f],{stdio:'inherit'});if(r.status)process.exit(r.status)}\" && node --check bin/codex-workbench && node --check scripts/pty-codex.js && node --check scripts/tui-pty-codex.js && node --check scripts/blessed-xterm-codex.js && node test/codex-bin.test.js && node test/blessed-compat.test.js && node test/config-paths.test.js && node test/session-sources.test.js && node test/workbench-config.test.js && node test/smoke.js",
+    "test": "node -e \"const fs=require('fs'),{spawnSync}=require('child_process');function files(d){return fs.readdirSync(d,{withFileTypes:true}).flatMap(e=>e.isDirectory()?files(d+'/'+e.name):e.name.endsWith('.js')?[d+'/'+e.name]:[])}for(const f of files('src')){const r=spawnSync(process.execPath,['--check',f],{stdio:'inherit'});if(r.status)process.exit(r.status)}\" && node --check bin/codex-workbench && node --check scripts/pty-codex.js && node --check scripts/tui-pty-codex.js && node --check scripts/blessed-xterm-codex.js && node test/codex-bin.test.js && node test/blessed-compat.test.js && node test/config-paths.test.js && node test/session-sources.test.js && node test/update-checker.test.js && node test/workbench-config.test.js && node test/smoke.js",
     "pty:codex": "node scripts/pty-codex.js",
     "tui:codex": "node scripts/tui-pty-codex.js",
     "xterm:codex": "node scripts/blessed-xterm-codex.js"

package/src/providers/codex.js

@@ -249,6 +249,14 @@ function runSessionCommand(command, session, args, inherit) {
 module.exports = {
   id: 'codex',
   label: 'Codex',
+  capabilities: {
+    new: true,
+    resume: true,
+    fork: true,
+    archive: true,
+    unarchive: true,
+    delete: true,
+  },
   isAvailable,
   getSessionFiles,
   parseSession,

package/src/providers/pi.js

@@ -8,7 +8,7 @@ const fs = require('fs');
 const path = require('path');
 const { spawn, spawnSync } = require('child_process');
 const { HOME, PI_CODING_AGENT_DIR } = require('../config');
-const { updateMetadata } = require('../model/metadata');
+const { removeMetadata, updateMetadata } = require('../model/metadata');
 
 // ---------------------------------------------------------------------------
 // Paths
@@ -272,9 +272,9 @@ function runSessionCommand(command, session, args, inherit) {
       return runArgv(argv, cwd, inherit);
     }
     case 'delete': {
-      // pi has no delete CLI – just remove the file
-      // A force flag is handled by session-sources calling deleteSessionFile
-      return -1; // signal that file-based deletion should be used
+      fs.unlinkSync(session.file);
+      removeMetadata(session);
+      return 0;
     }
     case 'archive':
     case 'unarchive': {
@@ -312,6 +312,14 @@ function resolveBin() {
 module.exports = {
   id: 'pi',
   label: 'pi',
+  capabilities: {
+    new: true,
+    resume: true,
+    fork: true,
+    archive: true,
+    unarchive: true,
+    delete: true,
+  },
   isAvailable,
   getSessionFiles,
   parseSession,

package/src/services/session-sources.js

@@ -105,6 +105,7 @@ function providerSummary(provider) {
   return {
     id: provider.id,
     label: provider.label || provider.id,
+    capabilities: provider.capabilities || {},
   };
 }
 
@@ -121,19 +122,15 @@ function listSourceBackends(source) {
     .map((backend) => ({
       id: String(backend.id),
       label: backend.label ? String(backend.label) : String(backend.id),
+      capabilities: backend.capabilities && typeof backend.capabilities === 'object'
+        ? backend.capabilities
+        : {},
     }));
 }
 
 function runSourceSessionCommand(session, command, args) {
   if (!session.sourceRemote) {
-    const status = runCodexCommand(command, session, args);
-    // If the provider returns -1 (e.g. pi delete = file-based), fall through to file deletion
-    if (status === -1) {
-      const { deleteSessionFile } = require('../model/session-store');
-      deleteSessionFile(session);
-      return 0;
-    }
-    return status;
+    return runCodexCommand(command, session, args);
   }
   const source = configuredSourceOrThrow(session.sourceId);
   const tty = command === 'resume' || command === 'fork';

package/src/services/update-checker.js

@@ -0,0 +1,72 @@
+'use strict';
+
+const https = require('https');
+const pkg = require('../../package.json');
+
+const REGISTRY_URL = `https://registry.npmjs.org/${encodeURIComponent(pkg.name).replace(/^%40/, '@')}/latest`;
+
+function parseVersion(version) {
+  return String(version || '')
+    .replace(/^v/, '')
+    .split('.')
+    .map((part) => Number.parseInt(part, 10))
+    .map((part) => (Number.isFinite(part) ? part : 0));
+}
+
+function compareVersions(a, b) {
+  const left = parseVersion(a);
+  const right = parseVersion(b);
+  const length = Math.max(left.length, right.length);
+  for (let i = 0; i < length; i += 1) {
+    const delta = (left[i] || 0) - (right[i] || 0);
+    if (delta !== 0) return delta;
+  }
+  return 0;
+}
+
+function fetchLatestVersion(timeoutMs = 1500) {
+  return new Promise((resolve) => {
+    const req = https.get(REGISTRY_URL, {
+      headers: { accept: 'application/json', 'user-agent': `${pkg.name}/${pkg.version}` },
+      timeout: timeoutMs,
+    }, (res) => {
+      if (res.statusCode !== 200) {
+        res.resume();
+        resolve(null);
+        return;
+      }
+      let body = '';
+      res.setEncoding('utf8');
+      res.on('data', (chunk) => { body += chunk; });
+      res.on('end', () => {
+        try {
+          const payload = JSON.parse(body);
+          resolve(payload && payload.version ? String(payload.version) : null);
+        } catch {
+          resolve(null);
+        }
+      });
+    });
+
+    req.on('timeout', () => {
+      req.destroy();
+      resolve(null);
+    });
+    req.on('error', () => resolve(null));
+  });
+}
+
+async function checkForUpdate(currentVersion = pkg.version) {
+  const latestVersion = await fetchLatestVersion();
+  if (!latestVersion || compareVersions(latestVersion, currentVersion) <= 0) return null;
+  return {
+    currentVersion,
+    latestVersion,
+  };
+}
+
+module.exports = {
+  checkForUpdate,
+  compareVersions,
+  fetchLatestVersion,
+};

package/src/ui/workbench.js

@@ -21,6 +21,7 @@ const {
   updateSourceMetadata,
 } = require('../services/session-sources');
 const { usableCwd } = require('../services/codex-runner');
+const { checkForUpdate } = require('../services/update-checker');
 const { createDirectoryPicker } = require('./directory-picker');
 
 async function runWorkbench() {
@@ -42,6 +43,7 @@ async function runWorkbench() {
   let activePanel = 'projects';
   let remoteLoadId = 0;
   let remoteLoading = false;
+  let updateInfo = null;
   let closed = false;
 
   const screen = blessed.screen({
@@ -466,7 +468,8 @@ async function runWorkbench() {
   const render = () => {
     applyLayout();
     const visible = currentSessions();
-    header.setContent(` ${appTitle}\n ${visible.length}/${sessions.length} visible  ${groupDisplayName(currentGroup())}`);
+    const updateText = updateInfo ? `  Update available: v${updateInfo.latestVersion}` : '';
+    header.setContent(` ${appTitle}${updateText}\n ${visible.length}/${sessions.length} visible  ${groupDisplayName(currentGroup())}`);
     detailBox.setContent(detailContent(selectedSession()));
     updateFocusStyles();
     screen.render();
@@ -891,6 +894,11 @@ async function runWorkbench() {
   projectsList.focus();
   render();
   startRemoteReload(true);
+  checkForUpdate(pkg.version).then((nextUpdateInfo) => {
+    if (closed || !nextUpdateInfo) return;
+    updateInfo = nextUpdateInfo;
+    render();
+  });
 
   return new Promise(() => {});
 }