@dfosco/storyboard 0.6.12 → 0.6.13

package/src/core/cli/canvasUndo.js

@@ -0,0 +1,40 @@
+/**
+ * storyboard canvas undo — append the inverse of a previously-applied event.
+ *
+ * Usage:
+ *   storyboard canvas undo --canvas my-canvas --event evt_xxx
+ *
+ * Flags:
+ *   -c, --canvas             Canvas name (required)
+ *   -e, --event              Event ID to undo (required)
+ *   --json                   Output as JSON
+ */
+
+import { post, parseSimpleArgs, jsonOut, die } from './cliHelpers.js'
+
+const args = process.argv.slice(4) // skip: node, sb, canvas, undo
+
+if (args.includes('--help') || args.includes('-h')) {
+  console.log(`
+  canvas undo flags:
+
+    -c, --canvas             Canvas name (required)
+    -e, --event              Event ID to undo (required)
+    --json                   Output as JSON
+`)
+  process.exit(0)
+}
+
+const { flags } = parseSimpleArgs(args)
+const canvas = flags.canvas || flags.c
+const eventId = flags.event || flags.e
+
+if (!canvas) die('--canvas is required')
+if (!eventId) die('--event is required')
+
+try {
+  const result = await post('/_storyboard/canvas/undo', { name: canvas, eventId })
+  if (flags.json) { jsonOut(result) } else {
+    console.log(`Undid event ${eventId} → appended ${result.eventId} (${result.inverseEvent?.event})`)
+  }
+} catch (err) { die(err.message) }

package/src/core/cli/dev.js

@@ -256,14 +256,11 @@ async function main() {
   }
 
   const renameWatcher = startRenameWatcher(targetCwd)
-  const compactInterval = setInterval(() => {
-    try {
-      const r = compactAll(targetCwd)
-      if (verbose) {
-        for (const x of r) p.log.info(`[compact] ${x.name}: ${(x.before / 1024).toFixed(0)}KB → ${(x.after / 1024).toFixed(0)}KB`)
-      }
-    } catch { /* non-critical */ }
-  }, 15 * 60 * 1000)
+  // Note: no background compaction timer — the previous 15-min interval
+  // would silently wipe per-tab undo history mid-session. Files are now
+  // compacted on dev startup (above) and via a 2 MB hard-ceiling check
+  // inside the canvas server's appendEvent (kicks in only for runaway
+  // sessions that genuinely need it).
 
   const npmBin = process.platform === 'win32' ? 'npx.cmd' : 'npx'
   const viteArgs = ['vite', '--port', String(port)]
@@ -375,7 +372,6 @@ async function main() {
       process.exit(130)
     }
     shuttingDown = true
-    clearInterval(compactInterval)
     renameWatcher.close()
     // Suppress Vite's shutdown-time esbuild noise ("Pre-transform error:
     // The service was stopped" for every in-flight transform) AND the
@@ -397,7 +393,6 @@ async function main() {
   process.on('SIGTERM', shutdown)
 
   child.on('exit', (code) => {
-    clearInterval(compactInterval)
     renameWatcher.close()
     releasePort(worktreeName)
     process.exit(code ?? 0)

package/src/core/cli/index.js

@@ -86,6 +86,8 @@ function helpScreen(version) {
     cmd('canvas delete-canvas', 'Delete a canvas and its directory'),
     cmd('canvas roles', 'List available hub roles'),
     cmd('canvas batch', 'Execute multiple canvas operations in one call'),
+    cmd('canvas undo --event <id>', 'Undo a previously-applied canvas event'),
+    cmd('canvas redo --event <id>', 'Redo a previously-undone canvas event'),
     cmd('compact [name]', 'Compact canvas JSONL files (removes bloat)'),
     cmd('compact --all', 'Force compact all canvases'),
     '',
@@ -196,6 +198,10 @@ switch (command) {
       import('./canvasRoles.js')
     } else if (process.argv[3] === 'batch') {
       import('./canvasBatch.js')
+    } else if (process.argv[3] === 'undo') {
+      import('./canvasUndo.js')
+    } else if (process.argv[3] === 'redo') {
+      import('./canvasRedo.js')
     } else {
       const version = getVersion()
       console.log(helpScreen(version))