bdtui 0.7.2 → 0.8.1

package/src/bd.js

@@ -1,130 +1,130 @@
-import { execFile } from 'child_process';
-import { promisify } from 'util';
-
-const execFileP = promisify(execFile);
-
-// Default execFile maxBuffer is 1MB; bd JSON output (esp. `list --all` with many
-// 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(bdErrorMessage(err));
-  }
-}
-
-// Run a bd command whose stdout we don't consume (e.g. `dep add`/`dep remove`,
-// which print a human-readable "✓ Added dependency…" line, not JSON). We can't
-// JSON.parse that — doing so was the source of the "Unexpected token '✓'" error.
-// Failures still surface: a non-zero exit rejects execFile and we throw stderr.
-async function runBdVoid(args, cwd) {
-  try {
-    await execFileP('bd', args, { cwd, maxBuffer: MAX_BUFFER });
-  } catch (err) {
-    throw new Error(bdErrorMessage(err));
-  }
-}
-
-// `bd blocked` only returns *derived*-blocked beads (open beads whose blockers
-// aren't all closed); it omits beads whose status was explicitly set to
-// "blocked". The Blocked tab should show both, so merge the two sources, deduped.
-async function bdBlocked(cwd) {
-  const [derived, explicit] = await Promise.all([
-    runBd(['blocked', '--json'], cwd).catch(() => []),
-    runBd(['list', '--status', 'blocked', '--json', '--limit', '0'], cwd).catch(() => []),
-  ]);
-  const byId = new Map();
-  for (const b of [...(derived || []), ...(explicit || [])]) {
-    if (b?.id && !byId.has(b.id)) byId.set(b.id, b);
-  }
-  return [...byId.values()];
-}
-
-export const bdList = (filter, cwd, { unlimited = false } = {}) => {
-  if (filter === 'ready') return runBd(['ready', '--json'], cwd);
-  if (filter === 'blocked') return bdBlocked(cwd);
-  const args = ['list', '--json'];
-  if (filter === 'all') args.push('--all');
-  else if (filter) args.push('--status', filter);
-  // `--limit 0` lifts bd's default 50-row cap for the paginated flat views.
-  if (unlimited) args.push('--limit', '0');
-  return runBd(args, cwd);
-};
-
-// bdList('all') with the 50-row cap lifted — used for enumerating every epic.
-export const bdListAll = (cwd) => bdList('all', cwd, { unlimited: true });
-
-export const bdShow = async (id, cwd) => {
-  const res = await runBd(['show', id, '--json'], cwd);
-  return Array.isArray(res) ? res[0] : res;
-};
-
-export const bdDeps = async (id, cwd) => {
-  const [down, up] = await Promise.all([
-    runBd(['dep', 'list', id, '--json'], cwd).catch(() => []),
-    runBd(['dep', 'list', id, '--direction=up', '--json'], cwd).catch(() => []),
-  ]);
-  return { down: Array.isArray(down) ? down : [], up: Array.isArray(up) ? up : [] };
-};
-
-export const bdDepListDown = async (id, cwd) => {
-  const res = await runBd(['dep', 'list', id, '--json'], cwd).catch(() => []);
-  return Array.isArray(res) ? res : [];
-};
-
-export const bdUpdate = (id, opts, cwd) => {
-  const args = ['update', id];
-  if (opts.status) args.push('--status', opts.status);
-  if (opts.priority !== undefined) args.push('--priority', String(opts.priority));
-  return runBd([...args, '--json'], cwd);
-};
-
-export const bdClose = (id, reason, cwd) => {
-  const args = ['close', id];
-  if (reason) args.push('--reason', reason);
-  return runBd([...args, '--json'], cwd);
-};
-
-export const bdClaim = (id, cwd) => runBd(['update', id, '--claim', '--json'], cwd);
-
-export const bdReopen = (id, cwd) => runBd(['reopen', id, '--json'], cwd);
-
-export const bdDepAdd = (child, parent, type, cwd) => {
-  const args = ['dep', 'add', child, parent];
-  if (type && type !== 'blocks') args.push('--type', type);
-  return runBdVoid(args, cwd);
-};
-
-export const bdDepRemove = (child, parent, cwd) =>
-  runBdVoid(['dep', 'remove', child, parent], cwd);
-
-// Returns all children of an epic (all statuses — bd children includes closed by default)
-export const bdChildren = (id, cwd) => runBd(['children', id, '--json'], cwd);
-
-export const bdEpics = async (cwd) => {
-  const all = await bdListAll(cwd);
-  return Array.isArray(all) ? all.filter((b) => b.issue_type === 'epic') : [];
-};
+import { execFile } from 'child_process';
+import { promisify } from 'util';
+
+const execFileP = promisify(execFile);
+
+// Default execFile maxBuffer is 1MB; bd JSON output (esp. `list --all` with many
+// 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(bdErrorMessage(err));
+  }
+}
+
+// Run a bd command whose stdout we don't consume (e.g. `dep add`/`dep remove`,
+// which print a human-readable "✓ Added dependency…" line, not JSON). We can't
+// JSON.parse that — doing so was the source of the "Unexpected token '✓'" error.
+// Failures still surface: a non-zero exit rejects execFile and we throw stderr.
+async function runBdVoid(args, cwd) {
+  try {
+    await execFileP('bd', args, { cwd, maxBuffer: MAX_BUFFER });
+  } catch (err) {
+    throw new Error(bdErrorMessage(err));
+  }
+}
+
+// `bd blocked` only returns *derived*-blocked beads (open beads whose blockers
+// aren't all closed); it omits beads whose status was explicitly set to
+// "blocked". The Blocked tab should show both, so merge the two sources, deduped.
+async function bdBlocked(cwd) {
+  const [derived, explicit] = await Promise.all([
+    runBd(['blocked', '--json'], cwd).catch(() => []),
+    runBd(['list', '--status', 'blocked', '--json', '--limit', '0'], cwd).catch(() => []),
+  ]);
+  const byId = new Map();
+  for (const b of [...(derived || []), ...(explicit || [])]) {
+    if (b?.id && !byId.has(b.id)) byId.set(b.id, b);
+  }
+  return [...byId.values()];
+}
+
+export const bdList = (filter, cwd, { unlimited = false } = {}) => {
+  if (filter === 'ready') return runBd(['ready', '--json'], cwd);
+  if (filter === 'blocked') return bdBlocked(cwd);
+  const args = ['list', '--json'];
+  if (filter === 'all') args.push('--all');
+  else if (filter) args.push('--status', filter);
+  // `--limit 0` lifts bd's default 50-row cap for the paginated flat views.
+  if (unlimited) args.push('--limit', '0');
+  return runBd(args, cwd);
+};
+
+// bdList('all') with the 50-row cap lifted — used for enumerating every epic.
+export const bdListAll = (cwd) => bdList('all', cwd, { unlimited: true });
+
+export const bdShow = async (id, cwd) => {
+  const res = await runBd(['show', id, '--json'], cwd);
+  return Array.isArray(res) ? res[0] : res;
+};
+
+export const bdDeps = async (id, cwd) => {
+  const [down, up] = await Promise.all([
+    runBd(['dep', 'list', id, '--json'], cwd).catch(() => []),
+    runBd(['dep', 'list', id, '--direction=up', '--json'], cwd).catch(() => []),
+  ]);
+  return { down: Array.isArray(down) ? down : [], up: Array.isArray(up) ? up : [] };
+};
+
+export const bdDepListDown = async (id, cwd) => {
+  const res = await runBd(['dep', 'list', id, '--json'], cwd).catch(() => []);
+  return Array.isArray(res) ? res : [];
+};
+
+export const bdUpdate = (id, opts, cwd) => {
+  const args = ['update', id];
+  if (opts.status) args.push('--status', opts.status);
+  if (opts.priority !== undefined) args.push('--priority', String(opts.priority));
+  return runBd([...args, '--json'], cwd);
+};
+
+export const bdClose = (id, reason, cwd) => {
+  const args = ['close', id];
+  if (reason) args.push('--reason', reason);
+  return runBd([...args, '--json'], cwd);
+};
+
+export const bdClaim = (id, cwd) => runBd(['update', id, '--claim', '--json'], cwd);
+
+export const bdReopen = (id, cwd) => runBd(['reopen', id, '--json'], cwd);
+
+export const bdDepAdd = (child, parent, type, cwd) => {
+  const args = ['dep', 'add', child, parent];
+  if (type && type !== 'blocks') args.push('--type', type);
+  return runBdVoid(args, cwd);
+};
+
+export const bdDepRemove = (child, parent, cwd) =>
+  runBdVoid(['dep', 'remove', child, parent], cwd);
+
+// Returns all children of an epic (all statuses — bd children includes closed by default)
+export const bdChildren = (id, cwd) => runBd(['children', id, '--json'], cwd);
+
+export const bdEpics = async (cwd) => {
+  const all = await bdListAll(cwd);
+  return Array.isArray(all) ? all.filter((b) => b.issue_type === 'epic') : [];
+};

package/src/keys.js

@@ -1,98 +1,100 @@
-import blessed from 'blessed';
-
-export const HELP_TEXT = [
-  '',
-  '  Navigation',
-  '  ──────────────────────────────────────',
-  '  j / ↓       Move down in list',
-  '  k / ↑       Move up in list',
-  '  g           Jump to top',
-  '  Shift+g     Jump to bottom',
-  '  Enter / l   Focus detail pane',
-  '  h / Esc     Back to list (from detail)',
-  '',
-  '  Filters & refresh',
-  '  ──────────────────────────────────────',
-  '  Tab         Next status filter (blocked/ready/in_progress/closed/all)',
-  '  Shift+Tab   Previous status filter',
-  '  t           Cycle type filter (all/epic/task/chore/bug)',
-  '  e           Filter by epic (modal picker; x clears; switches status to all)',
-  '  /           In-memory title filter (empty submit clears it)',
-  '  r           Reload (keeps active filters)',
-  '  Shift+r     Reset all filters (title / epic / type) and reload',
-  '  (all & closed tabs load more rows as you scroll to the bottom)',
-  '',
-  '  Mutations',
-  '  ──────────────────────────────────────',
-  '  s           Change status',
-  '  c           Close with reason',
-  '  Shift+c     Claim bead (in_progress + assign self)',
-  '  o           Reopen',
-  '  p           Change priority',
-  '  h           Change parent (epic / standalone)',
-  '',
-  '  Other',
-  '  ──────────────────────────────────────',
-  '  y           Yank bead ID to clipboard',
-  '  w           Copy workflow command (tasks: executors; epics: epic-sequential)',
-  '  ?           Toggle this help',
-  '  q / Ctrl-C  Quit',
-  '',
-  '  Press ? q or Esc to close',
-].join('\n');
-
-export function showHelp(screen, onClose) {
-  const box = blessed.box({
-    parent: screen,
-    label: ' Keybindings ',
-    border: { type: 'line' },
-    top: 'center',
-    left: 'center',
-    width: '55%',
-    height: '80%',
-    content: HELP_TEXT,
-    tags: false,
-    keys: true,
-    scrollable: true,
-    alwaysScroll: true,
-    scrollbar: { ch: '│' },
-    style: {
-      border: { fg: 'cyan' },
-      label: { fg: 'cyan', bold: true },
-    },
-  });
-
-  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?.();
-  };
-
-  // 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;
-}
+import blessed from 'blessed';
+
+export const HELP_TEXT = [
+  '',
+  '  Navigation',
+  '  ──────────────────────────────────────',
+  '  j / ↓       Move down in list',
+  '  k / ↑       Move up in list',
+  '  g           Jump to top',
+  '  Shift+g     Jump to bottom',
+  '  Enter / l   Focus detail pane',
+  '  h / Esc     Back to list (from detail)',
+  '  j / k       Scroll detail (when focused; g / Shift+g top/bottom, Ctrl-d/u half page)',
+  '',
+  '  Filters & refresh',
+  '  ──────────────────────────────────────',
+  '  Tab         Next status filter (deferred/blocked/ready/in_progress/closed/all)',
+  '  Shift+Tab   Previous status filter',
+  '  t           Cycle type filter (all/epic/task/chore/bug)',
+  '  e           Filter by epic (modal picker; x clears; switches status to all)',
+  '  /           In-memory title filter (empty submit clears it)',
+  '  r           Reload (keeps active filters)',
+  '  Shift+r     Reset all filters (title / epic / type) and reload',
+  '  (all & closed tabs load more rows as you scroll to the bottom)',
+  '',
+  '  Mutations',
+  '  ──────────────────────────────────────',
+  '  s           Change status',
+  '  c           Close with reason',
+  '  Shift+c     Claim bead (in_progress + assign self)',
+  '  o           Reopen',
+  '  p           Change priority',
+  '  h           Change parent (epic / standalone)',
+  '',
+  '  Other',
+  '  ──────────────────────────────────────',
+  '  y           Yank bead ID to clipboard',
+  '  w           Copy workflow command (Claude / OpenCode sections; tasks: executors; epics: epic-sequential)',
+  '  m           Release mouse to select/copy detail text (toggle)',
+  '  ?           Toggle this help',
+  '  q / Ctrl-C  Quit',
+  '',
+  '  Press ? q or Esc to close',
+].join('\n');
+
+export function showHelp(screen, onClose) {
+  const box = blessed.box({
+    parent: screen,
+    label: ' Keybindings ',
+    border: { type: 'line' },
+    top: 'center',
+    left: 'center',
+    width: '55%',
+    height: '80%',
+    content: HELP_TEXT,
+    tags: false,
+    keys: true,
+    scrollable: true,
+    alwaysScroll: true,
+    scrollbar: { ch: '│' },
+    style: {
+      border: { fg: 'cyan' },
+      label: { fg: 'cyan', bold: true },
+    },
+  });
+
+  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?.();
+  };
+
+  // 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;
+}