@bramblex/codex-workbench 0.1.17 → 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.17",
+  "version": "0.1.18",
   "description": "Terminal workbench for browsing and managing local and SSH Codex sessions.",
   "license": "MIT",
   "repository": {

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';