beads-ui 0.4.2 → 0.4.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "beads-ui",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "Local UI for Beads — Collaborate on issues with your coding agent.",
   "keywords": [
     "agent",
@@ -9,6 +9,7 @@
     "ai-tools"
   ],
   "homepage": "https://github.com/mantoni/beads-ui",
+  "author": "Maximilian Antoni <mail@maxantoni.de>",
   "type": "module",
   "bin": {
     "bdui": "bin/bdui.js"
@@ -57,7 +58,7 @@
     "jsdom": "^27.0.1",
     "prettier": "^3.3.3",
     "typescript": "^5.6.3",
-    "vitest": "^2.1.3"
+    "vitest": "^4.0.6"
   },
   "files": [
     "app/index.html",

package/server/bd.js

@@ -1,5 +1,8 @@
 import { spawn } from 'node:child_process';
 import { resolveDbPath } from './db.js';
+import { debug } from './logging.js';
+
+const log = debug('bd');
 
 /**
  * Resolve the bd executable path.
@@ -24,15 +27,25 @@ export function getBdBin() {
  */
 export function runBd(args, options = {}) {
   const bin = getBdBin();
+
+  // Ensure a consistent DB by setting BEADS_DB environment variable
+  const db_path = resolveDbPath({
+    cwd: options.cwd || process.cwd(),
+    env: options.env || process.env
+  });
+  const env_with_db = {
+    ...(options.env || process.env),
+    BEADS_DB: db_path.path
+  };
+
   const spawn_opts = {
     cwd: options.cwd || process.cwd(),
-    env: options.env ? options.env : process.env,
+    env: env_with_db,
     shell: false
   };
 
-  // Ensure a consistent DB by injecting --db if missing, following beads precedence.
   /** @type {string[]} */
-  const final_args = withDbArg(args, spawn_opts.cwd, spawn_opts.env);
+  const final_args = args.slice();
 
   return new Promise((resolve) => {
     const child = spawn(bin, final_args, spawn_opts);
@@ -78,8 +91,9 @@ export function runBd(args, options = {}) {
       });
     };
 
-    child.on('error', () => {
-      // Treat spawn error as an immediate non-zero exit with captured stderr message.
+    child.on('error', (err) => {
+      // Treat spawn error as an immediate non-zero exit; log for diagnostics.
+      log('spawn error running %s %o', bin, err);
       finish(127);
     });
     child.on('close', (code) => {
@@ -98,30 +112,21 @@ export function runBd(args, options = {}) {
 export async function runBdJson(args, options = {}) {
   const result = await runBd(args, options);
   if (result.code !== 0) {
+    log(
+      'bd exited with code %d (args=%o) stderr=%s',
+      result.code,
+      args,
+      result.stderr
+    );
     return { code: result.code, stderr: result.stderr };
   }
   /** @type {unknown} */
   let parsed;
   try {
     parsed = JSON.parse(result.stdout || 'null');
-  } catch {
+  } catch (err) {
+    log('bd returned invalid JSON (args=%o): %o', args, err);
     return { code: 0, stderr: 'Invalid JSON from bd' };
   }
   return { code: 0, stdoutJson: parsed };
 }
-
-/**
- * Add a resolved "--db <path>" pair to args when none present.
- *
- * @param {string[]} args
- * @param {string} cwd
- * @param {Record<string, string | undefined>} env
- * @returns {string[]}
- */
-function withDbArg(args, cwd, env) {
-  if (args.includes('--db')) {
-    return args.slice();
-  }
-  const resolved = resolveDbPath({ cwd, env });
-  return ['--db', resolved.path, ...args];
-}

package/server/list-adapters.js

@@ -1,4 +1,7 @@
 import { runBdJson } from './bd.js';
+import { debug } from './logging.js';
+
+const log = debug('list-adapters');
 
 /**
  * Build concrete `bd` CLI args for a subscription type + params.
@@ -107,6 +110,8 @@ export async function fetchListForSubscription(spec) {
   try {
     args = mapSubscriptionToBdArgs(spec);
   } catch (err) {
+    // Surface bad requests (e.g., missing params)
+    log('mapSubscriptionToBdArgs failed for %o: %o', spec, err);
     const e = toErrorObject(err);
     return { ok: false, error: e };
   }
@@ -114,6 +119,13 @@ export async function fetchListForSubscription(spec) {
   try {
     const res = await runBdJson(args);
     if (!res || res.code !== 0 || !('stdoutJson' in res)) {
+      log(
+        'bd failed for %o (args=%o) code=%s stderr=%s',
+        spec,
+        args,
+        res?.code,
+        res?.stderr || ''
+      );
       return {
         ok: false,
         error: {
@@ -161,6 +173,7 @@ export async function fetchListForSubscription(spec) {
     const items = normalizeIssueList(raw);
     return { ok: true, items };
   } catch (err) {
+    log('bd invocation failed for %o (args=%o): %o', spec, args, err);
     return {
       ok: false,
       error: {

package/server/ws.js

@@ -250,8 +250,8 @@ function emitSubscriptionSnapshot(ws, client_id, key, issues) {
   });
   try {
     ws.send(msg);
-  } catch {
-    // ignore per-socket errors
+  } catch (err) {
+    log('emit snapshot send failed key=%s id=%s: %o', key, client_id, err);
   }
 }
 
@@ -277,8 +277,8 @@ function emitSubscriptionUpsert(ws, client_id, key, issue) {
   });
   try {
     ws.send(msg);
-  } catch {
-    // ignore
+  } catch (err) {
+    log('emit upsert send failed key=%s id=%s: %o', key, client_id, err);
   }
 }
 
@@ -304,8 +304,8 @@ function emitSubscriptionDelete(ws, client_id, key, issue_id) {
   });
   try {
     ws.send(msg);
-  } catch {
-    // ignore
+  } catch (err) {
+    log('emit delete send failed key=%s id=%s: %o', key, client_id, err);
   }
 }
 
@@ -322,6 +322,7 @@ async function refreshAndPublish(spec) {
   await registry.withKeyLock(key, async () => {
     const res = await fetchListForSubscription(spec);
     if (!res.ok) {
+      log('refresh failed for %s: %s %o', key, res.error.message, res.error);
       return;
     }
     const items = applyClosedIssuesFilter(spec, res.items);
@@ -554,14 +555,20 @@ export async function handleMessage(ws, data) {
       await registry.withKeyLock(key, async () => {
         const res = await fetchListForSubscription(spec);
         if (!res.ok) {
+          log(
+            'initial snapshot failed for %s: %s %o',
+            key,
+            res.error.message,
+            res.error
+          );
           return;
         }
         const items = applyClosedIssuesFilter(spec, res.items);
         void registry.applyItems(key, items);
         emitSubscriptionSnapshot(ws, client_id, key, items);
       });
-    } catch {
-      // ignore snapshot errors
+    } catch (err) {
+      log('subscribe-list snapshot error for %s: %o', key, err);
     }
     ws.send(JSON.stringify(makeOk(req, { id: client_id, key })));
     return;