beads-ui 0.5.0 → 0.7.0

package/app/protocol.js

@@ -9,7 +9,7 @@
  * - Server can also send unsolicited events (e.g., subscription `snapshot`).
  */
 
-/** @typedef {'list-issues'|'update-status'|'edit-text'|'update-priority'|'create-issue'|'list-ready'|'dep-add'|'dep-remove'|'epic-status'|'update-assignee'|'label-add'|'label-remove'|'subscribe-list'|'unsubscribe-list'|'snapshot'|'upsert'|'delete'} MessageType */
+/** @typedef {'list-issues'|'update-status'|'edit-text'|'update-priority'|'create-issue'|'list-ready'|'dep-add'|'dep-remove'|'epic-status'|'update-assignee'|'label-add'|'label-remove'|'subscribe-list'|'unsubscribe-list'|'snapshot'|'upsert'|'delete'|'get-comments'|'add-comment'} MessageType */
 
 /**
  * @typedef {Object} RequestEnvelope
@@ -53,7 +53,10 @@ export const MESSAGE_TYPES = /** @type {const} */ ([
   // vNext per-subscription full-issue push events
   'snapshot',
   'upsert',
-  'delete'
+  'delete',
+  // Comments
+  'get-comments',
+  'add-comment'
 ]);
 
 /**

package/app/styles.css

@@ -1656,3 +1656,67 @@ html[data-theme='dark'] {
     grid-template-columns: 1fr;
   }
 }
+
+/* Comments section */
+.comments {
+  margin-top: 24px;
+  padding-top: 16px;
+  border-top: 1px solid var(--border);
+}
+
+.comment-item {
+  padding: 12px;
+  margin-bottom: 8px;
+  background: color-mix(in srgb, var(--panel-bg) 95%, transparent);
+  border: 1px solid var(--border);
+  border-radius: 4px;
+}
+
+.comment-header {
+  display: flex;
+  justify-content: space-between;
+  align-items: center;
+  margin-bottom: 6px;
+  font-size: 12px;
+}
+
+.comment-author {
+  font-weight: 600;
+  color: var(--fg);
+}
+
+.comment-date {
+  color: var(--muted);
+}
+
+.comment-text {
+  font-size: 14px;
+  line-height: 1.5;
+  white-space: pre-wrap;
+}
+
+.comment-input {
+  margin-top: 12px;
+}
+
+.comment-input textarea {
+  width: 100%;
+  min-height: 60px;
+  padding: 8px;
+  border: 1px solid var(--control-border);
+  border-radius: 4px;
+  background: var(--control-bg);
+  color: var(--fg);
+  font-family: inherit;
+  font-size: 14px;
+  resize: vertical;
+}
+
+.comment-input textarea:focus {
+  outline: none;
+  border-color: var(--link);
+}
+
+.comment-input button {
+  margin-top: 8px;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "beads-ui",
-  "version": "0.5.0",
+  "version": "0.7.0",
   "description": "Local UI for Beads — Collaborate on issues with your coding agent.",
   "keywords": [
     "agent",

package/server/bd.js

@@ -4,6 +4,38 @@ import { debug } from './logging.js';
 
 const log = debug('bd');
 
+/**
+ * Get the git user name from git config.
+ *
+ * @param {{ cwd?: string }} [options]
+ * @returns {Promise<string>}
+ */
+export async function getGitUserName(options = {}) {
+  return new Promise((resolve) => {
+    const child = spawn('git', ['config', 'user.name'], {
+      cwd: options.cwd || process.cwd(),
+      shell: false
+    });
+
+    /** @type {string[]} */
+    const chunks = [];
+
+    if (child.stdout) {
+      child.stdout.setEncoding('utf8');
+      child.stdout.on('data', (chunk) => chunks.push(String(chunk)));
+    }
+
+    child.on('error', () => resolve(''));
+    child.on('close', (code) => {
+      if (code !== 0) {
+        resolve('');
+        return;
+      }
+      resolve(chunks.join('').trim());
+    });
+  });
+}
+
 /**
  * Resolve the bd executable path.
  *

package/server/cli/commands.js

@@ -14,11 +14,9 @@ import { openUrl, waitForServer } from './open.js';
  * - Spawns a detached server process, writes PID file, returns 0.
  * - If already running (PID file present and process alive), prints URL and returns 0.
  *
+ * @param {{ open?: boolean, is_debug?: boolean, host?: string, port?: number }} [options]
  * @returns {Promise<number>} Exit code (0 on success)
  */
-/**
- * @param {{ open?: boolean, is_debug?: boolean }} [options]
- */
 export async function handleStart(options) {
   // Default: do not open a browser unless explicitly requested via `open: true`.
   const should_open = options?.open === true;
@@ -32,7 +30,19 @@ export async function handleStart(options) {
     removePidFile();
   }
 
-  const started = startDaemon({ is_debug: options?.is_debug });
+  // Set env vars in current process so getConfig() reflects the overrides
+  if (options?.host) {
+    process.env.HOST = options.host;
+  }
+  if (options?.port) {
+    process.env.PORT = String(options.port);
+  }
+
+  const started = startDaemon({
+    is_debug: options?.is_debug,
+    host: options?.host,
+    port: options?.port
+  });
   if (started && started.pid > 0) {
     printServerUrl();
     // Auto-open the browser once for a fresh daemon start

package/server/cli/daemon.js

@@ -140,7 +140,7 @@ export function getServerEntryPath() {
  * Spawn the server as a detached daemon, redirecting stdio to the log file.
  * Writes the PID file upon success.
  *
- * @param {{ is_debug?: boolean }} [options]
+ * @param {{ is_debug?: boolean, host?: string, port?: number }} [options]
  * @returns {{ pid: number } | null} Returns child PID on success; null on failure.
  */
 export function startDaemon(options = {}) {
@@ -160,10 +160,19 @@ export function startDaemon(options = {}) {
     log_fd = -1;
   }
 
+  /** @type {Record<string, string | undefined>} */
+  const spawn_env = { ...process.env };
+  if (options.host) {
+    spawn_env.HOST = options.host;
+  }
+  if (options.port) {
+    spawn_env.PORT = String(options.port);
+  }
+
   /** @type {SpawnOptions} */
   const opts = {
     detached: true,
-    env: { ...process.env },
+    env: spawn_env,
     stdio: log_fd >= 0 ? ['ignore', log_fd, log_fd] : 'ignore',
     windowsHide: true
   };

package/server/cli/index.js

@@ -3,18 +3,21 @@ import { handleRestart, handleStart, handleStop } from './commands.js';
 import { printUsage } from './usage.js';
 
 /**
- * Parse argv into a command token and flags.
+ * Parse argv into a command token, flags, and options.
  *
  * @param {string[]} args
- * @returns {{ command: string | null, flags: string[] }}
+ * @returns {{ command: string | null, flags: string[], options: { host?: string, port?: number } }}
  */
 export function parseArgs(args) {
   /** @type {string[]} */
   const flags = [];
   /** @type {string | null} */
   let command = null;
+  /** @type {{ host?: string, port?: number }} */
+  const options = {};
 
-  for (const token of args) {
+  for (let i = 0; i < args.length; i++) {
+    const token = args[i];
     if (token === '--help' || token === '-h') {
       flags.push('help');
       continue;
@@ -27,6 +30,17 @@ export function parseArgs(args) {
       flags.push('open');
       continue;
     }
+    if (token === '--host' && i + 1 < args.length) {
+      options.host = args[++i];
+      continue;
+    }
+    if (token === '--port' && i + 1 < args.length) {
+      const port_value = Number.parseInt(args[++i], 10);
+      if (Number.isFinite(port_value) && port_value > 0) {
+        options.port = port_value;
+      }
+      continue;
+    }
     if (
       !command &&
       (token === 'start' || token === 'stop' || token === 'restart')
@@ -37,7 +51,7 @@ export function parseArgs(args) {
     // Ignore unrecognized tokens for now; future flags may be parsed here.
   }
 
-  return { command, flags };
+  return { command, flags, options };
 }
 
 /**
@@ -48,7 +62,7 @@ export function parseArgs(args) {
  * @returns {Promise<number>}
  */
 export async function main(args) {
-  const { command, flags } = parseArgs(args);
+  const { command, flags, options } = parseArgs(args);
 
   const is_debug = flags.includes('debug');
   if (is_debug) {
@@ -68,21 +82,25 @@ export async function main(args) {
     /**
      * Default behavior: do NOT open a browser. `--open` explicitly opens.
      */
-    const options = {
+    const start_options = {
       open: flags.includes('open'),
-      is_debug: is_debug || Boolean(process.env.DEBUG)
+      is_debug: is_debug || Boolean(process.env.DEBUG),
+      host: options.host,
+      port: options.port
     };
-    return await handleStart(options);
+    return await handleStart(start_options);
   }
   if (command === 'stop') {
     return await handleStop();
   }
   if (command === 'restart') {
-    const options = {
+    const restart_options = {
       open: flags.includes('open'),
-      is_debug: is_debug || Boolean(process.env.DEBUG)
+      is_debug: is_debug || Boolean(process.env.DEBUG),
+      host: options.host,
+      port: options.port
     };
-    return await handleRestart(options);
+    return await handleRestart(restart_options);
   }
 
   // Unknown command path (should not happen due to parseArgs guard)

package/server/cli/usage.js

@@ -13,9 +13,11 @@ export function printUsage(out_stream) {
     '  restart     Restart the UI server',
     '',
     'Options:',
-    '  -h, --help    Show this help message',
-    '  -d, --debug   Enable debug logging',
-    '      --open    Open the browser after start/restart',
+    '  -h, --help        Show this help message',
+    '  -d, --debug       Enable debug logging',
+    '      --open        Open the browser after start/restart',
+    '      --host <addr> Bind to a specific host (default: 127.0.0.1)',
+    '      --port <num>  Bind to a specific port (default: 3000)',
     ''
   ];
   for (const line of lines) {

package/server/config.js

@@ -23,7 +23,8 @@ export function getConfig() {
     port_value = 3000;
   }
 
-  const host_value = '127.0.0.1';
+  const host_env = process.env.HOST;
+  const host_value = host_env && host_env.length > 0 ? host_env : '127.0.0.1';
 
   return {
     host: host_value,

package/server/index.js

@@ -10,6 +10,16 @@ if (process.argv.includes('--debug') || process.argv.includes('-d')) {
   enableAllDebug();
 }
 
+// Parse --host and --port from argv and set env vars before getConfig()
+for (let i = 0; i < process.argv.length; i++) {
+  if (process.argv[i] === '--host' && process.argv[i + 1]) {
+    process.env.HOST = process.argv[++i];
+  }
+  if (process.argv[i] === '--port' && process.argv[i + 1]) {
+    process.env.PORT = process.argv[++i];
+  }
+}
+
 const config = getConfig();
 const app = createApp(config);
 const server = createServer(app);

package/server/ws.js

@@ -5,7 +5,7 @@
  */
 import { WebSocketServer } from 'ws';
 import { isRequest, makeError, makeOk } from '../app/protocol.js';
-import { runBd, runBdJson } from './bd.js';
+import { getGitUserName, runBd, runBdJson } from './bd.js';
 import { fetchListForSubscription } from './list-adapters.js';
 import { debug } from './logging.js';
 import { keyOf, registry } from './subscriptions.js';
@@ -1081,6 +1081,78 @@ export async function handleMessage(ws, data) {
     return;
   }
 
+  // get-comments: payload { id: string }
+  if (req.type === 'get-comments') {
+    const { id } = /** @type {any} */ (req.payload || {});
+    if (typeof id !== 'string' || id.length === 0) {
+      ws.send(
+        JSON.stringify(
+          makeError(req, 'bad_request', 'payload requires { id: string }')
+        )
+      );
+      return;
+    }
+    const res = await runBdJson(['comments', id, '--json']);
+    if (res.code !== 0) {
+      ws.send(
+        JSON.stringify(makeError(req, 'bd_error', res.stderr || 'bd failed'))
+      );
+      return;
+    }
+    ws.send(JSON.stringify(makeOk(req, res.stdoutJson || [])));
+    return;
+  }
+
+  // add-comment: payload { id: string, text: string }
+  if (req.type === 'add-comment') {
+    const { id, text } = /** @type {any} */ (req.payload || {});
+    if (
+      typeof id !== 'string' ||
+      id.length === 0 ||
+      typeof text !== 'string' ||
+      text.trim().length === 0
+    ) {
+      ws.send(
+        JSON.stringify(
+          makeError(
+            req,
+            'bad_request',
+            'payload requires { id: string, text: non-empty string }'
+          )
+        )
+      );
+      return;
+    }
+
+    // Get git user name for author attribution
+    const author = await getGitUserName();
+    const args = ['comment', id, text.trim()];
+    if (author) {
+      args.push('--author', author);
+    }
+
+    const res = await runBd(args);
+    if (res.code !== 0) {
+      ws.send(
+        JSON.stringify(makeError(req, 'bd_error', res.stderr || 'bd failed'))
+      );
+      return;
+    }
+
+    // Return updated comments list
+    const comments = await runBdJson(['comments', id, '--json']);
+    if (comments.code !== 0) {
+      ws.send(
+        JSON.stringify(
+          makeError(req, 'bd_error', comments.stderr || 'bd failed')
+        )
+      );
+      return;
+    }
+    ws.send(JSON.stringify(makeOk(req, comments.stdoutJson || [])));
+    return;
+  }
+
   // Unknown type
   const err = makeError(
     req,