bdtui 0.7.0 → 0.7.2

package/CHANGELOG.md

@@ -4,6 +4,30 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+## [0.7.2] - 2026-06-29
+
+### Added
+
+- `w` now works on epics. Previously the workflow-command picker was disabled for epics ("workflow skills apply to tasks only"); pressing `w` on an epic now opens a picker offering `/executor-epic-sequential` and `/executor-epic-sequential-worktree` (which run every ready child of the epic on a single branch / isolated worktree). Tasks, chores, and bugs keep their existing per-bead executors.
+
+### Fixed
+
+- `bd` errors now render as readable text in the status bar. When a `bd` command failed, bdtui threw the entire structured JSON error body (`{"error": "…", "schema_version": 1}`) verbatim — a multi-line blob whose leading `{` corrupted the one-line status bar markup (the "error with `{`" seen when a tab failed to load). bdtui now parses that body and surfaces just the human-readable `error` field, collapsing newlines and escaping tag markup so the status bar stays a clean single line.
+- Tag-escaping no longer mangles text containing a literal `}`. The two-step escape (`replace('{')` then `replace('}')`) re-processed the `}` it had just inserted into `{open}`, turning any `}` in a bead description (e.g. JSON/code snippets) or in a `/` search query into a stray `{open}` token. Escaping is now a single pass in both the detail pane and the status bar.
+
+### Changed
+
+- npm package metadata: expanded `description` and added `keywords` for discoverability; normalized the repository URL to the `git+https://` form.
+- Removed internal agent-workflow tooling (`.beads/`, `.claude/` agents and skills, `CLAUDE.md`, `BEADS_WORKFLOW.md`, `docs/TROUBLESHOOTING.md`) from the repository.
+
+## [0.7.1] - 2026-06-21
+
+### Fixed
+
+- The **Keybindings** (`?`) help can now be dismissed after a mouse click moves focus to the Beads list or Detail pane. Its close keys were bound to the help box itself, so once a click stole focus the modal became unclosable (every global key is suppressed while a modal is open). `q`/`Esc`/`h` are now bound at the screen level and dismiss it regardless of which pane holds focus.
+- Reopening the Keybindings help with `?` works again. `?` is now a single open/close toggle, fixing a re-entrancy where the freshly-registered `?` close handler fired on the very keypress that opened the modal and slammed it shut.
+- The boot status bar no longer shows the `? help` hint twice. `setStatus` already appends it, so the redundant hint baked into the "Ready" message was removed.
+
 ## [0.7.0] - 2026-06-21
 
 ### Added

package/README.md

@@ -82,7 +82,7 @@ be-12  P2 ready   T  └ Another child
 | `p` | Change priority |
 | `h` | Change parent (epic / standalone) |
 | `y` | Yank bead ID to clipboard |
-| `w` | Pick a workflow skill (`/executor-task`, `/executor-task-worktree`, `/executor-epic-task`, `/executor-epic-task-worktree`) and copy it with the selected task's ID. Disabled for epics. See [workflow-template](https://github.com/jsiovn/workflow-template). |
+| `w` | Pick a workflow command and copy it with the selected bead's ID. Tasks list the per-bead executors (`/executor-task`, `/executor-task-worktree`, `/executor-epic-task`, `/executor-epic-task-worktree`, `/executor-rework-in-place`); epics list `/executor-epic-sequential` and `/executor-epic-sequential-worktree`. See [workflow-template](https://github.com/jsiovn/workflow-template). |
 | `?` | Help overlay |
 | `q` / `Ctrl-C` | Quit |
 

package/package.json

@@ -1,7 +1,24 @@
 {
   "name": "bdtui",
-  "version": "0.7.0",
-  "description": "Terminal UI for the bd beads issue tracker",
+  "version": "0.7.2",
+  "description": "Terminal UI (TUI) for the bd beads issue tracker - browse, filter, and update beads from the CLI with vim keybindings",
+  "keywords": [
+    "bd",
+    "beads",
+    "bdtui",
+    "issue-tracker",
+    "tui",
+    "terminal-ui",
+    "terminal",
+    "cli",
+    "bug-tracker",
+    "project-management",
+    "blessed",
+    "vim",
+    "dependency-aware",
+    "developer-tools",
+    "nodejs"
+  ],
   "type": "module",
   "bin": {
     "bdtui": "bin/bdtui.js"
@@ -14,7 +31,7 @@
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/jsiovn/bdtui.git"
+    "url": "git+https://github.com/jsiovn/bdtui.git"
   },
   "homepage": "https://github.com/jsiovn/bdtui#readme",
   "bugs": {

package/src/app.js

@@ -13,7 +13,10 @@ const TYPE_FILTERS = ['all', 'epic', 'task', 'chore', 'bug'];
 // Neutralize blessed tag markup in free text before it lands in a tags:true
 // box (mirrors esc() in views/detail.js). The title search is user-typed, so a
 // query like "{bold}" would otherwise corrupt the status line.
-const escTags = (s) => String(s ?? '').replace(/\{/g, '{open}').replace(/\}/g, '{close}');
+// Single pass: a two-step .replace(/\{/).replace(/\}/) would re-process the "}"
+// it just inserted into "{open}", corrupting any text that contains a literal
+// "}" (e.g. a bd JSON error blob) into a stray "{open}" token.
+const escTags = (s) => String(s ?? '').replace(/[{}]/g, (c) => (c === '{' ? '{open}' : '{close}'));
 
 export async function run(cwd) {
   state.cwd = cwd;
@@ -48,7 +51,9 @@ export async function run(cwd) {
   // active so the caller can fall back to a plain "Ready".
   function filterSummary() {
     const parts = [];
-    if (state.textFilter) parts.push(`"${escTags(state.textFilter)}"`);
+    // Plain text only — setStatus() escapes the whole summary, so pre-escaping
+    // textFilter here would double-escape any braces in the search query.
+    if (state.textFilter) parts.push(`"${state.textFilter}"`);
     if (state.epicFilter) parts.push(`epic: ${state.epicFilter}`);
     if (state.typeFilter !== 'all') parts.push(`type: ${state.typeFilter}`);
     if (parts.length === 0) return null;
@@ -59,8 +64,13 @@ export async function run(cwd) {
   }
   function setStatus(msg, isError = false, transient = false) {
     if (statusTimer) { clearTimeout(statusTimer); statusTimer = null; }
+    // The status bar is a single tags:true row. Neutralize blessed tag markup
+    // and collapse newlines so dynamic text — bd error messages especially,
+    // which can be multi-line and contain "{" — renders as plain text instead
+    // of corrupting the markup or overflowing the one-line height.
+    const safe = escTags(String(msg ?? '').replace(/\s*\n\s*/g, ' ').trim());
     const icon = isError ? '{red-fg}✗{/}' : '{green-fg}●{/}';
-    const text = isError ? `{red-fg}${msg}{/}` : msg;
+    const text = isError ? `{red-fg}${safe}{/}` : safe;
     statusBar.setContent(` ${icon} ${text}  {gray-fg}? help · q quit{/}`);
     screen.render();
     if (transient) {
@@ -282,9 +292,19 @@ export async function run(cwd) {
     setStatus(defaultStatus());
   });
 
-  key(['?'], () => {
+  // '?' toggles the help modal. Registered directly (NOT through key(), which
+  // swallows every global while a modal is open) so the same key that opens the
+  // help can also close it. Driving open AND close from this one handler — kept
+  // out of showHelp's own close keys — avoids re-entrancy: a separate '?' close
+  // binding would fire on the very keypress that registered it and slam the
+  // modal shut again. closeHelp is non-null exactly while the help is open.
+  let closeHelp = null;
+  screen.key(['?'], () => {
+    if (closeHelp) { closeHelp(); return; }
+    if (modalOpen) return; // another modal (a picker) owns the screen
     modalOpen = true;
-    showHelp(screen, () => {
+    closeHelp = showHelp(screen, () => {
+      closeHelp = null;
       modalOpen = false;
       list.focus();
       screen.render();
@@ -534,12 +554,8 @@ export async function run(cwd) {
     if (screen.focused !== list || !state.selectedId) return;
     const id = state.selectedId;
     const bead = state.beadsById.get(id);
-    if (bead?.issue_type === 'epic') {
-      setStatus(`${id} is an epic — workflow skills apply to tasks only`, true, true);
-      return;
-    }
     try {
-      const line = await skillPicker(screen, id);
+      const line = await skillPicker(screen, id, bead?.issue_type);
       list.focus();
       await copyToClipboard(line, {
         busyMsg: `Copying ${line}…`,
@@ -564,5 +580,7 @@ export async function run(cwd) {
     setStatus(err.message, true);
   }
   render();
-  setStatus('Ready | ? help');
+  // setStatus already appends "? help · q quit"; defaultStatus() yields "Ready"
+  // (or the active-filter summary) without duplicating the help hint.
+  setStatus(defaultStatus());
 }

package/src/bd.js

@@ -7,12 +7,32 @@ const execFileP = promisify(execFile);
 // closed beads) can exceed that and throw "stdout maxBuffer length exceeded".
 const MAX_BUFFER = 64 * 1024 * 1024; // 64MB
 
+// bd reports failures with a non-zero exit and a structured JSON body
+// ({"error": "…", "schema_version": 1}) — usually on stderr, occasionally on
+// stdout. Surface the human-readable `error` field; fall back to the raw text,
+// then execFile's own message. Without this the whole multi-line JSON blob —
+// leading "{" and all — gets thrown verbatim and dumped into the one-line
+// status bar, which is the "error with {" users see when a tab fails to load.
+function bdErrorMessage(err) {
+  const raw = (err.stderr?.trim() || err.stdout?.trim() || '');
+  if (raw) {
+    try {
+      const parsed = JSON.parse(raw);
+      if (parsed && typeof parsed.error === 'string' && parsed.error.trim()) {
+        return parsed.error.trim();
+      }
+    } catch {}
+    return raw;
+  }
+  return err.message;
+}
+
 async function runBd(args, cwd) {
   try {
     const { stdout } = await execFileP('bd', args, { cwd, maxBuffer: MAX_BUFFER });
     return JSON.parse(stdout);
   } catch (err) {
-    throw new Error(err.stderr?.trim() || err.message);
+    throw new Error(bdErrorMessage(err));
   }
 }
 
@@ -24,7 +44,7 @@ async function runBdVoid(args, cwd) {
   try {
     await execFileP('bd', args, { cwd, maxBuffer: MAX_BUFFER });
   } catch (err) {
-    throw new Error(err.stderr?.trim() || err.message);
+    throw new Error(bdErrorMessage(err));
   }
 }
 

package/src/keys.js

@@ -34,7 +34,7 @@ export const HELP_TEXT = [
   '  Other',
   '  ──────────────────────────────────────',
   '  y           Yank bead ID to clipboard',
-  '  w           Copy workflow skill command for selected task',
+  '  w           Copy workflow command (tasks: executors; epics: epic-sequential)',
   '  ?           Toggle this help',
   '  q / Ctrl-C  Quit',
   '',
@@ -65,11 +65,34 @@ export function showHelp(screen, onClose) {
   box.focus();
   screen.render();
 
+  // '?' is deliberately NOT a close key here — the caller (app.js) drives it as
+  // an open/close toggle. Binding '?' as a close key would re-enter on the very
+  // keypress that opens the modal: the toggle adds this listener mid-dispatch
+  // and blessed's emitter would then fire it in the same 'key ?' pass, slamming
+  // the help shut the instant it opened. q/Esc/h are safe — they never open the
+  // help, so they can only ever close it.
+  const CLOSE_KEYS = ['q', 'escape', 'h'];
+  let closed = false;
   const close = () => {
+    if (closed) return;
+    closed = true;
+    screen.unkey(CLOSE_KEYS, close);
     box.destroy();
     screen.render();
     onClose?.();
   };
 
-  box.key(['?', 'q', 'escape', 'h'], close);
+  // Bind the close keys on the screen, not the box. A box-level binding only
+  // fires while the box holds keyboard focus, but the Beads list and Detail
+  // pane are both mouse:true — a click on either steals focus from the help
+  // box. app.js suppresses every global key while a modal is open, so a
+  // focus-stealing click would otherwise leave the help permanently stuck
+  // open (its own close keys dead, every global key inert). Screen-level keys
+  // fire regardless of which pane holds focus, so q/Esc/h always dismiss it.
+  // unkey() on close keeps the binding from leaking across repeated opens.
+  screen.key(CLOSE_KEYS, close);
+
+  // Returned so the caller's '?' toggle can dismiss the help regardless of
+  // which pane currently holds focus.
+  return close;
 }

package/src/views/detail.js

@@ -24,7 +24,10 @@ const STATUS_ICON = {
 const PRIORITY_COLOR  = ['red', 'yellow', 'green', 'cyan', 'gray'];
 const PRIORITY_LABEL  = ['Critical', 'High', 'Normal', 'Low', 'Trivial'];
 
-function esc(s)          { return String(s ?? '').replace(/\{/g, '{open}').replace(/\}/g, '{close}'); }
+// Single pass: a two-step .replace(/\{/).replace(/\}/) would re-process the "}"
+// it just inserted into "{open}", corrupting any text containing a literal "}"
+// (e.g. a JSON/code snippet in a description) into a stray "{open}" token.
+function esc(s)          { return String(s ?? '').replace(/[{}]/g, (c) => (c === '{' ? '{open}' : '{close}')); }
 function t(color, text)  { return `{${color}-fg}${text}{/}`; }
 function bold(text)      { return `{bold}${text}{/bold}`; }
 function fmtDate(iso)    { return iso ? iso.slice(0, 10) : '—'; }

package/src/views/modals.js

@@ -175,14 +175,21 @@ export function epicPicker(screen, epics, currentEpicId = null) {
   });
 }
 
-export function skillPicker(screen, beadId) {
-  const items = [
-    `/executor-task ${beadId}`,
-    `/executor-task-worktree ${beadId}`,
-    `/executor-epic-task ${beadId}`,
-    `/executor-epic-task-worktree ${beadId}`,
-    `/executor-rework-in-place ${beadId}`,
-  ];
+export function skillPicker(screen, beadId, issueType) {
+  // Epics get the epic-level executor (runs every ready child sequentially on a
+  // single branch); tasks/chores/bugs get the per-bead executors.
+  const items = issueType === 'epic'
+    ? [
+        `/executor-epic-sequential ${beadId}`,
+        `/executor-epic-sequential-worktree ${beadId}`,
+      ]
+    : [
+        `/executor-task ${beadId}`,
+        `/executor-task-worktree ${beadId}`,
+        `/executor-epic-task ${beadId}`,
+        `/executor-epic-task-worktree ${beadId}`,
+        `/executor-rework-in-place ${beadId}`,
+      ];
   return listPicker(screen, 'Copy workflow command', items, 'magenta', 56, 1);
 }