@joshski/dust 0.1.56 → 0.1.57

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"),
@@ -2137,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);
@@ -2391,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);
   }
@@ -3312,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) {
@@ -3602,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 = [];
@@ -3619,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;
@@ -3828,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",
@@ -3845,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");
@@ -3872,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];
@@ -3923,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];
@@ -4162,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}`);
@@ -4241,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("");
@@ -4264,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("");

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/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.56",
+  "version": "0.1.57",
   "description": "Flow state for AI coding agents",
   "type": "module",
   "bin": {