@joshski/dust 0.1.55 → 0.1.57

package/dist/cli/types.d.ts

@@ -4,6 +4,7 @@
 export interface CommandContext {
     cwd: string;
     stdout: (message: string) => void;
+    stdoutInline?: (message: string) => void;
     stderr: (message: string) => void;
 }
 export interface CommandResult {

package/dist/dust.js

@@ -1177,7 +1177,7 @@ function getLogLines(buffer) {
 }
 
 // lib/bucket/repository.ts
-import { dirname as dirname2, join as join8 } from "node:path";
+import { dirname as dirname3, join as join8 } from "node:path";
 
 // lib/claude/spawn-claude-code.ts
 import { spawn as nodeSpawn } from "node:child_process";
@@ -1595,6 +1595,9 @@ async function run(prompt, options = {}, dependencies = defaultRunnerDependencie
   await dependencies.streamEvents(events, sink, onRawEvent);
 }
 
+// lib/logging/index.ts
+import { join as join5 } from "node:path";
+
 // lib/logging/match.ts
 function parsePatterns(debug) {
   if (!debug)
@@ -1617,41 +1620,47 @@ function formatLine(name, messages) {
 
 // lib/logging/sink.ts
 import { appendFileSync, mkdirSync } from "node:fs";
-import { join as join5 } from "node:path";
-var logPath;
-var ready = false;
-var scope = process.env.DEBUG_LOG_SCOPE || "debug";
-function ensureLogFile() {
-  if (ready)
-    return logPath;
-  ready = true;
-  const dir = join5(process.cwd(), "log", "dust");
-  logPath = join5(dir, `${scope}.log`);
-  try {
-    mkdirSync(dir, { recursive: true });
-  } catch {
-    logPath = undefined;
-  }
-  return logPath;
-}
-function setLogScope(name) {
-  scope = name;
-  process.env.DEBUG_LOG_SCOPE = name;
-  logPath = undefined;
+import { dirname } from "node:path";
+
+class FileSink {
+  logPath;
+  _appendFileSync;
+  _mkdirSync;
+  resolvedPath;
   ready = false;
+  constructor(logPath, _appendFileSync = appendFileSync, _mkdirSync = mkdirSync) {
+    this.logPath = logPath;
+    this._appendFileSync = _appendFileSync;
+    this._mkdirSync = _mkdirSync;
+  }
+  ensureLogFile() {
+    if (this.ready)
+      return this.resolvedPath;
+    this.ready = true;
+    this.resolvedPath = this.logPath;
+    try {
+      this._mkdirSync(dirname(this.logPath), { recursive: true });
+    } catch {
+      this.resolvedPath = undefined;
+    }
+    return this.resolvedPath;
+  }
+  write(line) {
+    const path = this.ensureLogFile();
+    if (!path)
+      return;
+    try {
+      this._appendFileSync(path, line);
+    } catch {}
+  }
 }
-var writeToFile = (line) => {
-  const path = ensureLogFile();
-  if (!path)
-    return;
-  try {
-    appendFileSync(path, line);
-  } catch {}
-};
 
 // lib/logging/index.ts
+var DUST_LOG_FILE = "DUST_LOG_FILE";
 var patterns = null;
 var initialized = false;
+var activeFileSink = null;
+var ownedDustLogFile = false;
 function init() {
   if (initialized)
     return;
@@ -1659,12 +1668,26 @@ function init() {
   const parsed = parsePatterns(process.env.DEBUG);
   patterns = parsed.length > 0 ? parsed : null;
 }
-function createLogger(name, write = writeToFile) {
+function enableFileLogs(scope, _sinkForTesting) {
+  const existing = process.env[DUST_LOG_FILE];
+  const logDir = process.env.DUST_LOG_DIR ?? join5(process.cwd(), "log");
+  const path = existing ?? join5(logDir, `${scope}.log`);
+  if (!existing) {
+    process.env[DUST_LOG_FILE] = path;
+    ownedDustLogFile = true;
+  }
+  activeFileSink = _sinkForTesting ?? new FileSink(path);
+}
+function createLogger(name) {
   return (...messages) => {
     init();
-    if (!patterns || !matchesAny(name, patterns))
-      return;
-    write(formatLine(name, messages));
+    const line = formatLine(name, messages);
+    if (activeFileSink) {
+      activeFileSink.write(line);
+    }
+    if (patterns && matchesAny(name, patterns)) {
+      process.stdout.write(line);
+    }
   };
 }
 
@@ -1742,7 +1765,7 @@ function formatAgentEvent(event) {
 import { spawn as nodeSpawn2 } from "node:child_process";
 import { readFileSync } from "node:fs";
 import os from "node:os";
-import { dirname, join as join7 } from "node:path";
+import { dirname as dirname2, join as join7 } from "node:path";
 import { fileURLToPath } from "node:url";
 
 // lib/workflow-tasks.ts
@@ -1887,7 +1910,7 @@ async function next(dependencies) {
 }
 
 // lib/cli/commands/loop.ts
-var __dirname2 = dirname(fileURLToPath(import.meta.url));
+var __dirname2 = dirname2(fileURLToPath(import.meta.url));
 function getDustVersion() {
   const candidates = [
     join7(__dirname2, "../../../package.json"),
@@ -1925,8 +1948,7 @@ function formatLoopEvent(event) {
     case "loop.checking_tasks":
       return null;
     case "loop.no_tasks":
-      return `\uD83D\uDE34 No tasks available. Sleeping...
-`;
+      return "\uD83D\uDE34 No tasks available. Sleeping...";
     case "loop.tasks_found":
       return `✨ Found a task. Going to work!
 `;
@@ -1973,7 +1995,18 @@ function createWireEventSender(eventsUrl, sessionId, postEvent, onError, getAgen
 }
 var log = createLogger("dust.cli.commands.loop");
 var SLEEP_INTERVAL_MS = 30000;
+var SLEEP_STEP_MS = 1000;
 var DEFAULT_MAX_ITERATIONS = 10;
+async function sleepWithProgress(sleep, totalMs, writeInline, writeLine) {
+  let remainingMs = totalMs;
+  while (remainingMs > 0) {
+    const stepMs = Math.min(SLEEP_STEP_MS, remainingMs);
+    await sleep(stepMs);
+    writeInline(".");
+    remainingMs -= stepMs;
+  }
+  writeLine("");
+}
 async function gitPull(cwd, spawn) {
   return new Promise((resolve) => {
     const proc = spawn("git", ["pull"], {
@@ -2127,6 +2160,7 @@ function parseMaxIterations(commandArguments) {
   return parsed;
 }
 async function loopClaude(dependencies, loopDependencies = createDefaultDependencies()) {
+  enableFileLogs("loop");
   const { context, settings } = dependencies;
   const { postEvent } = loopDependencies;
   const maxIterations = parseMaxIterations(dependencies.arguments);
@@ -2172,7 +2206,8 @@ async function loopClaude(dependencies, loopDependencies = createDefaultDependen
     const result = await runOneIteration(dependencies, loopDependencies, onLoopEvent, onAgentEvent, iterationOptions);
     if (result === "no_tasks") {
       log("sleeping, no tasks");
-      await loopDependencies.sleep(SLEEP_INTERVAL_MS);
+      const writeInline = context.stdoutInline ?? context.stdout;
+      await sleepWithProgress(loopDependencies.sleep, SLEEP_INTERVAL_MS, writeInline, context.stdout);
     } else {
       completedIterations++;
       log(`iteration ${completedIterations}/${maxIterations} complete, result=${result}`);
@@ -2380,7 +2415,7 @@ async function addRepository(repository, manager, repoDeps, context) {
   }
   log3(`adding repository ${repository.name}`);
   const repoPath = getRepoPath(repository.name, repoDeps.getReposDir());
-  await repoDeps.fileSystem.mkdir(dirname2(repoPath), { recursive: true });
+  await repoDeps.fileSystem.mkdir(dirname3(repoPath), { recursive: true });
   if (repoDeps.fileSystem.exists(repoPath)) {
     await removeRepository(repoPath, repoDeps.spawn, context);
   }
@@ -3301,6 +3336,7 @@ async function resolveToken(authDeps, context) {
   }
 }
 async function bucket(dependencies, bucketDeps = createDefaultBucketDependencies()) {
+  enableFileLogs("bucket");
   const { context, fileSystem } = dependencies;
   const token = await resolveToken(bucketDeps.auth, context);
   if (!token) {
@@ -3591,7 +3627,7 @@ function validateTitleFilenameMatch(filePath, content) {
 }
 
 // lib/lint/validators/goal-hierarchy.ts
-import { dirname as dirname3, resolve } from "node:path";
+import { dirname as dirname4, resolve } from "node:path";
 var REQUIRED_GOAL_HEADINGS = ["## Parent Goal", "## Sub-Goals"];
 function validateGoalHierarchySections(filePath, content) {
   const violations = [];
@@ -3608,7 +3644,7 @@ function validateGoalHierarchySections(filePath, content) {
 function extractGoalRelationships(filePath, content) {
   const lines = content.split(`
 `);
-  const fileDir = dirname3(filePath);
+  const fileDir = dirname4(filePath);
   const parentGoals = [];
   const subGoals = [];
   let currentSection = null;
@@ -3706,17 +3742,15 @@ function validateIdeaOpenQuestions(filePath, content) {
   const violations = [];
   const lines = content.split(`
 `);
+  const topLevelStructureMessage = "Open Questions must use `### Question?` headings and `#### Option` headings at the top level. Put supporting markdown (including lists and code blocks) under an option heading. Run `dust new idea` to see the expected format.";
   let inOpenQuestions = false;
   let currentQuestionLine = null;
+  let inOption = false;
   let inCodeBlock = false;
   for (let i = 0;i < lines.length; i++) {
     const line = lines[i];
-    if (line.startsWith("```")) {
-      inCodeBlock = !inCodeBlock;
-      continue;
-    }
-    if (inCodeBlock)
-      continue;
+    const trimmedLine = line.trimEnd();
+    const nonWhitespaceLine = line.trim();
     if (line.startsWith("## ")) {
       if (inOpenQuestions && currentQuestionLine !== null) {
         violations.push({
@@ -3735,19 +3769,27 @@ function validateIdeaOpenQuestions(filePath, content) {
       }
       inOpenQuestions = line === "## Open Questions";
       currentQuestionLine = null;
+      inOption = false;
+      inCodeBlock = false;
       continue;
     }
     if (!inOpenQuestions)
       continue;
-    if (/^[-*] /.test(line.trimStart())) {
-      violations.push({
-        file: filePath,
-        message: "Open Questions must use ### headings for questions and #### headings for options, not bullet points. Run `dust new idea` to see the expected format.",
-        line: i + 1
-      });
+    if (line.startsWith("```")) {
+      if (!inOption && !inCodeBlock) {
+        violations.push({
+          file: filePath,
+          message: topLevelStructureMessage,
+          line: i + 1
+        });
+      }
+      inCodeBlock = !inCodeBlock;
       continue;
     }
+    if (inCodeBlock)
+      continue;
     if (line.startsWith("### ")) {
+      inOption = false;
       if (currentQuestionLine !== null) {
         violations.push({
           file: filePath,
@@ -3755,7 +3797,7 @@ function validateIdeaOpenQuestions(filePath, content) {
           line: currentQuestionLine
         });
       }
-      if (!line.trimEnd().endsWith("?")) {
+      if (!trimmedLine.endsWith("?")) {
         violations.push({
           file: filePath,
           message: 'Questions must end with "?" (e.g., "### Should we take our own payments?")',
@@ -3769,6 +3811,15 @@ function validateIdeaOpenQuestions(filePath, content) {
     }
     if (line.startsWith("#### ")) {
       currentQuestionLine = null;
+      inOption = true;
+      continue;
+    }
+    if (nonWhitespaceLine && !inOption) {
+      violations.push({
+        file: filePath,
+        message: topLevelStructureMessage,
+        line: i + 1
+      });
     }
   }
   if (inOpenQuestions && currentQuestionLine !== null) {
@@ -3802,7 +3853,7 @@ function validateIdeaTransitionTitle(filePath, content, ideasPath, fileSystem) {
 }
 
 // lib/lint/validators/link-validator.ts
-import { dirname as dirname4, resolve as resolve2 } from "node:path";
+import { dirname as dirname5, resolve as resolve2 } from "node:path";
 var SEMANTIC_RULES = [
   {
     section: "## Goals",
@@ -3819,7 +3870,7 @@ function validateLinks(filePath, content, fileSystem) {
   const violations = [];
   const lines = content.split(`
 `);
-  const fileDir = dirname4(filePath);
+  const fileDir = dirname5(filePath);
   for (let i = 0;i < lines.length; i++) {
     const line = lines[i];
     const linkPattern = new RegExp(MARKDOWN_LINK_PATTERN.source, "g");
@@ -3846,7 +3897,7 @@ function validateSemanticLinks(filePath, content) {
   const violations = [];
   const lines = content.split(`
 `);
-  const fileDir = dirname4(filePath);
+  const fileDir = dirname5(filePath);
   let currentSection = null;
   for (let i = 0;i < lines.length; i++) {
     const line = lines[i];
@@ -3897,7 +3948,7 @@ function validateGoalHierarchyLinks(filePath, content) {
   const violations = [];
   const lines = content.split(`
 `);
-  const fileDir = dirname4(filePath);
+  const fileDir = dirname5(filePath);
   let currentSection = null;
   for (let i = 0;i < lines.length; i++) {
     const line = lines[i];
@@ -4136,6 +4187,24 @@ async function lintMarkdown(dependencies) {
 // lib/cli/commands/check.ts
 var log5 = createLogger("dust.cli.commands.check");
 var DEFAULT_CHECK_TIMEOUT_MS = 13000;
+var MAX_OUTPUT_LINES = 500;
+var KEEP_LINES = 250;
+function truncateOutput(output) {
+  const lines = output.split(`
+`);
+  if (lines.length <= MAX_OUTPUT_LINES) {
+    return output;
+  }
+  const snippedCount = lines.length - KEEP_LINES * 2;
+  const firstLines = lines.slice(0, KEEP_LINES);
+  const lastLines = lines.slice(-KEEP_LINES);
+  return [
+    ...firstLines,
+    `[...snip ${snippedCount} lines...]`,
+    ...lastLines
+  ].join(`
+`);
+}
 async function runSingleCheck(check, cwd, runner) {
   const timeoutMs = check.timeoutMilliseconds ?? DEFAULT_CHECK_TIMEOUT_MS;
   log5(`running check ${check.name}: ${check.command}`);
@@ -4215,7 +4284,7 @@ function displayResults(results, context) {
       context.stdout(`Note: This check was killed after ${result.timeoutSeconds}s. To configure a different timeout, set "timeoutMilliseconds" in the check configuration in .dust/config/settings.json`);
     }
     if (result.output.trim()) {
-      context.stdout(result.output.trimEnd());
+      context.stdout(truncateOutput(result.output).trimEnd());
     }
     if (result.hints && result.hints.length > 0) {
       context.stdout("");
@@ -4238,7 +4307,7 @@ async function check(dependencies, shellRunner = defaultShellRunner) {
     settings
   } = dependencies;
   const serial = commandArguments.includes("--serial");
-  setLogScope("check");
+  enableFileLogs("check");
   if (!settings.checks || settings.checks.length === 0) {
     context.stderr("Error: No checks configured in .dust/config/settings.json");
     context.stderr("");
@@ -5125,6 +5194,7 @@ async function wireEntry(fsPrimitives, processPrimitives, consolePrimitives) {
     context: {
       cwd: processPrimitives.cwd(),
       stdout: consolePrimitives.log,
+      stdoutInline: consolePrimitives.write,
       stderr: consolePrimitives.error
     },
     fileSystem,
@@ -5140,4 +5210,10 @@ await wireEntry({ existsSync, statSync: statSync2, readFile: readFile2, writeFil
   exit: (code) => {
     process.exitCode = code;
   }
-}, { log: console.log, error: console.error });
+}, {
+  log: console.log,
+  write: (message) => {
+    process.stdout.write(message);
+  },
+  error: console.error
+});

package/dist/logging/index.d.ts

@@ -1,12 +1,19 @@
 /**
  * Minimal debug logging framework.
  *
- * When the DEBUG environment variable is set, matching loggers write
- * timestamped lines to `<cwd>/log/dust/<scope>.log`.
+ * Two independent output channels:
  *
- * The scope defaults to "debug" but can be changed via setLogScope()
- * so that different commands (e.g. `loop`, `check`, `bucket`) write
- * to separate log files.
+ * - **File logging** — activated by `enableFileLogs(scope)` at command startup.
+ *   Writes all logs to `./log/<scope>.log` by default. Two env vars control routing:
+ *
+ *   Routing rules for enableFileLogs(scope):
+ *   1. If DUST_LOG_FILE is already set (inherited from a parent process such as
+ *      `dust check`), use that path — all scopes land in the same file.
+ *   2. Otherwise compute the path from DUST_LOG_DIR (if set) or `<cwd>/log`, set
+ *      DUST_LOG_FILE so child processes inherit the same destination, then write there.
+ *
+ * - **Stdout logging** — activated by `DEBUG=pattern`. Writes matching logs to
+ *   stdout. Works in any command, regardless of whether file logging is enabled.
  *
  * DEBUG is a comma-separated list of match expressions. Each expression
  * can contain `*` as a wildcard (matches any sequence of characters).
@@ -23,22 +30,32 @@
  *
  * No external dependencies.
  */
-import { type WriteFn } from './sink';
-export { setLogScope } from './sink';
+import { type LogSink } from './sink';
 export type LogFn = (...messages: unknown[]) => void;
 /**
- * Create a named logger function. The returned function writes to
- * `log/dust/<scope>.log` when the logger name matches the DEBUG patterns.
+ * Activate file logging for this command. Determines the log path as follows:
+ * - If DUST_LOG_FILE is already set (inherited from a parent process such as
+ *   `dust check`), use that path — all scopes land in the same file.
+ * - Otherwise compute the path using DUST_LOG_DIR (if set) or `<cwd>/log`, set
+ *   DUST_LOG_FILE so that any child processes inherit the same destination, then write there.
+ *
+ * Pass a LogSink as the second argument to override for testing.
+ */
+export declare function enableFileLogs(scope: string, _sinkForTesting?: LogSink): void;
+/**
+ * Create a named logger function. The returned function writes to:
+ * - The active file sink (if `enableFileLogs` was called), always, no filtering.
+ * - `process.stdout` if DEBUG is set and `name` matches the pattern.
  *
  * @param name - Logger name, e.g. `dust.bucket.loop`
- * @param write - Override the default file writer (for testing)
  */
-export declare function createLogger(name: string, write?: WriteFn): LogFn;
+export declare function createLogger(name: string): LogFn;
 /**
- * Check whether a logger name would be enabled under the current DEBUG value.
+ * Check whether a logger name would produce stdout output under the current DEBUG value.
  */
 export declare function isEnabled(name: string): boolean;
 /**
  * Reset internal state (for testing only).
+ * Clears DUST_LOG_FILE only if this module set it (not if it was inherited).
  */
 export declare function _reset(): void;

package/dist/logging/sink.d.ts

@@ -1,24 +1,29 @@
 /**
  * File-based log sink — the imperative shell for debug logging.
  *
- * Lazily creates `<cwd>/log/dust/<scope>.log` and appends lines to it.
- * The scope defaults to "debug" but can be changed via setLogScope()
- * so that different commands write to separate log files.
+ * Writes log lines to an arbitrary file path, creating the directory lazily
+ * on first write. The path is determined by the caller (enableFileLogs in
+ * index.ts) rather than this class.
  */
-export type WriteFn = (line: string) => void;
-/**
- * Set the log scope, which determines the output filename.
- * Must be called before any logger writes (i.e. at command startup).
- *
- * For example, `setLogScope('loop')` writes to `log/dust/loop.log`.
- */
-export declare function setLogScope(name: string): void;
-/**
- * Write a line to the debug log file.
- * Silently no-ops if the file cannot be opened.
- */
-export declare const writeToFile: WriteFn;
-/**
- * Reset sink state (for testing only).
- */
-export declare function _resetSink(): void;
+export interface LogSink {
+    write(line: string): void;
+}
+type AppendFileSyncFn = (path: string, data: string) => void;
+type MkdirSyncFn = (path: string, options: {
+    recursive: boolean;
+}) => void;
+export declare class FileSink implements LogSink {
+    private readonly logPath;
+    private readonly _appendFileSync;
+    private readonly _mkdirSync;
+    private resolvedPath;
+    private ready;
+    constructor(logPath: string, _appendFileSync?: AppendFileSyncFn, _mkdirSync?: MkdirSyncFn);
+    private ensureLogFile;
+    /**
+     * Write a line to the log file.
+     * Silently no-ops if the file cannot be opened.
+     */
+    write(line: string): void;
+}
+export {};

package/dist/logging.js

@@ -1,3 +1,6 @@
+// lib/logging/index.ts
+import { join } from "node:path";
+
 // lib/logging/match.ts
 function parsePatterns(debug) {
   if (!debug)
@@ -20,41 +23,47 @@ function formatLine(name, messages) {
 
 // lib/logging/sink.ts
 import { appendFileSync, mkdirSync } from "node:fs";
-import { join } from "node:path";
-var logPath;
-var ready = false;
-var scope = process.env.DEBUG_LOG_SCOPE || "debug";
-function ensureLogFile() {
-  if (ready)
-    return logPath;
-  ready = true;
-  const dir = join(process.cwd(), "log", "dust");
-  logPath = join(dir, `${scope}.log`);
-  try {
-    mkdirSync(dir, { recursive: true });
-  } catch {
-    logPath = undefined;
-  }
-  return logPath;
-}
-function setLogScope(name) {
-  scope = name;
-  process.env.DEBUG_LOG_SCOPE = name;
-  logPath = undefined;
+import { dirname } from "node:path";
+
+class FileSink {
+  logPath;
+  _appendFileSync;
+  _mkdirSync;
+  resolvedPath;
   ready = false;
+  constructor(logPath, _appendFileSync = appendFileSync, _mkdirSync = mkdirSync) {
+    this.logPath = logPath;
+    this._appendFileSync = _appendFileSync;
+    this._mkdirSync = _mkdirSync;
+  }
+  ensureLogFile() {
+    if (this.ready)
+      return this.resolvedPath;
+    this.ready = true;
+    this.resolvedPath = this.logPath;
+    try {
+      this._mkdirSync(dirname(this.logPath), { recursive: true });
+    } catch {
+      this.resolvedPath = undefined;
+    }
+    return this.resolvedPath;
+  }
+  write(line) {
+    const path = this.ensureLogFile();
+    if (!path)
+      return;
+    try {
+      this._appendFileSync(path, line);
+    } catch {}
+  }
 }
-var writeToFile = (line) => {
-  const path = ensureLogFile();
-  if (!path)
-    return;
-  try {
-    appendFileSync(path, line);
-  } catch {}
-};
 
 // lib/logging/index.ts
+var DUST_LOG_FILE = "DUST_LOG_FILE";
 var patterns = null;
 var initialized = false;
+var activeFileSink = null;
+var ownedDustLogFile = false;
 function init() {
   if (initialized)
     return;
@@ -62,12 +71,26 @@ function init() {
   const parsed = parsePatterns(process.env.DEBUG);
   patterns = parsed.length > 0 ? parsed : null;
 }
-function createLogger(name, write = writeToFile) {
+function enableFileLogs(scope, _sinkForTesting) {
+  const existing = process.env[DUST_LOG_FILE];
+  const logDir = process.env.DUST_LOG_DIR ?? join(process.cwd(), "log");
+  const path = existing ?? join(logDir, `${scope}.log`);
+  if (!existing) {
+    process.env[DUST_LOG_FILE] = path;
+    ownedDustLogFile = true;
+  }
+  activeFileSink = _sinkForTesting ?? new FileSink(path);
+}
+function createLogger(name) {
   return (...messages) => {
     init();
-    if (!patterns || !matchesAny(name, patterns))
-      return;
-    write(formatLine(name, messages));
+    const line = formatLine(name, messages);
+    if (activeFileSink) {
+      activeFileSink.write(line);
+    }
+    if (patterns && matchesAny(name, patterns)) {
+      process.stdout.write(line);
+    }
   };
 }
 function isEnabled(name) {
@@ -77,10 +100,15 @@ function isEnabled(name) {
 function _reset() {
   initialized = false;
   patterns = null;
+  activeFileSink = null;
+  if (ownedDustLogFile) {
+    delete process.env[DUST_LOG_FILE];
+    ownedDustLogFile = false;
+  }
 }
 export {
-  setLogScope,
   isEnabled,
+  enableFileLogs,
   createLogger,
   _reset
 };

package/dist/workflow-tasks.js

@@ -111,7 +111,7 @@ async function createIdeaTask(fileSystem, dustPath, prefix, ideaSlug, openingSen
   return { filePath };
 }
 async function createRefineIdeaTask(fileSystem, dustPath, ideaSlug, description) {
-  return createIdeaTask(fileSystem, dustPath, "Refine Idea: ", ideaSlug, (ideaTitle) => `Thoroughly research this idea and refine it into a well-defined proposal. Read the idea file, explore the codebase for relevant context, and identify any ambiguity. Where aspects are unclear or could go multiple ways, add open questions to the idea file. Review \`.dust/goals/\` for alignment and \`.dust/facts/\` for relevant design decisions. See [${ideaTitle}](../ideas/${ideaSlug}.md). If you add open questions, use \`## Open Questions\` with \`### Question?\` headings and \`#### Option\` headings, and only add questions that are meaningful decisions worth asking.`, [
+  return createIdeaTask(fileSystem, dustPath, "Refine Idea: ", ideaSlug, (ideaTitle) => `Thoroughly research this idea and refine it into a well-defined proposal. Read the idea file, explore the codebase for relevant context, and identify any ambiguity. Where aspects are unclear or could go multiple ways, add open questions to the idea file. Review \`.dust/goals/\` for alignment and \`.dust/facts/\` for relevant design decisions. See [${ideaTitle}](../ideas/${ideaSlug}.md). If you add open questions, use \`## Open Questions\` with \`### Question?\` headings and one or more \`#### Option\` headings beneath each question, and only add questions that are meaningful decisions worth asking.`, [
     "Idea is thoroughly researched with relevant codebase context",
     "Open questions are added for any ambiguous or underspecified aspects",
     "Open questions follow the required heading format and focus on high-value decisions",
@@ -175,7 +175,7 @@ ${description}
   const ideaPath = `.dust/ideas/${ideaFilename}`;
   const content = `# ${taskTitle}
 
-Research this idea thoroughly, then create an idea file at \`${ideaPath}\`. Read the codebase for relevant context, flesh out the description, and identify any ambiguity. Where aspects are unclear or could go multiple ways, add open questions to the idea file. If you add open questions, use \`## Open Questions\` with \`### Question?\` headings and \`#### Option\` headings, and only add questions that are meaningful decisions worth asking. Review \`.dust/goals/\` and \`.dust/facts/\` for relevant context.
+Research this idea thoroughly, then create an idea file at \`${ideaPath}\`. Read the codebase for relevant context, flesh out the description, and identify any ambiguity. Where aspects are unclear or could go multiple ways, add open questions to the idea file. If you add open questions, use \`## Open Questions\` with \`### Question?\` headings and one or more \`#### Option\` headings beneath each question, and only add questions that are meaningful decisions worth asking. Review \`.dust/goals/\` and \`.dust/facts/\` for relevant context.
 
 ## Idea Description
 

package/lib/istanbul/minimal-reporter.cjs

@@ -21,25 +21,36 @@ function formatMetrics(metrics) {
   return parts.join(', ')
 }
 
-function getUncoveredLines(fileCoverage) {
+function getGapLines(fileCoverage) {
+  const gapLines = new Set()
+
   const lineCoverage = fileCoverage.getLineCoverage()
+  for (const [lineStr, hits] of Object.entries(lineCoverage)) {
+    if (hits === 0) gapLines.add(Number.parseInt(lineStr, 10))
+  }
+
+  const { branchMap, b } = fileCoverage
+  for (const [id, branch] of Object.entries(branchMap)) {
+    if (b[id].some(hits => hits === 0)) {
+      gapLines.add(branch.loc.start.line)
+    }
+  }
+
+  const sorted = [...gapLines].sort((a, b) => a - b)
   const ranges = []
   let rangeStart = null
   let rangeEnd = null
 
-  for (const [lineStr, hits] of Object.entries(lineCoverage)) {
-    const line = Number.parseInt(lineStr, 10)
-    if (hits === 0) {
-      if (rangeStart === null) {
-        rangeStart = line
-        rangeEnd = line
-      } else if (line === rangeEnd + 1) {
-        rangeEnd = line
-      } else {
-        ranges.push([rangeStart, rangeEnd])
-        rangeStart = line
-        rangeEnd = line
-      }
+  for (const line of sorted) {
+    if (rangeStart === null) {
+      rangeStart = line
+      rangeEnd = line
+    } else if (line === rangeEnd + 1) {
+      rangeEnd = line
+    } else {
+      ranges.push([rangeStart, rangeEnd])
+      rangeStart = line
+      rangeEnd = line
     }
   }
 
@@ -68,7 +79,12 @@ class IncompleteCoverageReporter extends ReportBase {
       },
     })
 
-    if (incompleteFiles.length === 0) return
+    if (incompleteFiles.length === 0) {
+      const cw = context.writer.writeFile(null)
+      cw.println('✔ 100% coverage!')
+      cw.close()
+      return
+    }
 
     const cw = context.writer.writeFile(null)
     const count = incompleteFiles.length
@@ -78,7 +94,7 @@ class IncompleteCoverageReporter extends ReportBase {
     for (const file of incompleteFiles) {
       cw.println('')
       cw.println(`${file.name} (${formatMetrics(file.metrics)})`)
-      for (const line of getUncoveredLines(file.fileCoverage)) {
+      for (const line of getGapLines(file.fileCoverage)) {
         cw.println(`- ${line}`)
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@joshski/dust",
-  "version": "0.1.55",
+  "version": "0.1.57",
   "description": "Flow state for AI coding agents",
   "type": "module",
   "bin": {