claude-nomad 0.26.2 → 0.28.0

package/src/summary.ts

@@ -1,51 +1,99 @@
+import { green, okGlyph, warnGlyph, yellow } from './color.ts';
 import { ok, warn } from './utils.ts';
 
 /**
- * Emit the single end-of-run summary line shared by cmdPull, cmdPush, and
- * cmdDiff. Canonical phrasing:
+ * Pure phrasing core for the end-of-run summary line shared by cmdPull,
+ * cmdPush, and cmdDiff. Returns the message `text` (without any status glyph)
+ * plus a `clean` flag so callers can pick the right glyph/stream. Canonical
+ * phrasing:
  *   - `summary: clean` when nothing was unmapped (and, for push, no
- *     collisions or extras skipped). Always printed so users see a consistent
- *     terminator and can spot when behavior changes.
+ *     collisions or extras skipped).
  *   - `summary: <N> unmapped on pull (run nomad doctor to list)`
  *   - `summary: <N> unmapped on pull, <X> extras skipped (run nomad doctor to list)`
  *   - `summary: <N> unmapped on diff (run nomad doctor to list)`
  *   - `summary: <N> unmapped on push, <M> collisions (run nomad doctor to list)`
  *   - `summary: <N> unmapped on push, <M> collisions, <X> extras skipped (run nomad doctor to list)`
  *
- * Clean outcomes go through `ok()` (green `✓` glyph, stdout) and unmapped /
- * collision / extras-skipped outcomes go through `warn()` (yellow `⚠︎` glyph,
- * stderr). The status glyph carries the success/warn semantics; users see e.g.
- * `✓ summary: clean` or `⚠︎ summary: 3 unmapped on pull (...)`. Note: clean
- * still goes to stdout so it survives backgrounded shell-rc invocations
- * like `nomad pull 2>/dev/null &`. `collisions` is meaningful only for
- * `'push'`; for `'pull'` / `'diff'` it is ignored and defaults to 0.
- * `extrasSkipped` counts dirnames that the per-project whitelist
- * (`SUPPORTED_EXTRAS`) declined to sync; surfaces from `remapExtrasPush`
- * and `remapExtrasPull`. The fourth positional parameter defaults to 0 so
- * legacy three-arg call sites continue to work unchanged (D-03 additive
- * contract). This module is the SINGLE source of truth for the phrasing,
- * eliminating drift risk across the three callers by construction.
+ * `collisions` is meaningful only for `'push'`; for `'pull'` / `'diff'` it is
+ * ignored and defaults to 0. `extrasSkipped` counts dirnames that the
+ * per-project whitelist (`SUPPORTED_EXTRAS`) declined to sync. This function is
+ * the SINGLE source of truth for the phrasing, so `emitSummary` (standalone
+ * line) and `summaryRow` (tree row) cannot drift apart.
+ *
+ * @param verb - the originating command.
+ * @param unmapped - count of path-map entries skipped for this host.
+ * @param collisions - push-only collision count (ignored for pull/diff).
+ * @param extrasSkipped - count of extras dirnames the whitelist declined.
+ * @returns `{ text, clean }` where `clean` is true on the no-warning outcome.
  */
-export function emitSummary(
+export function summaryText(
   verb: 'pull' | 'push' | 'diff',
   unmapped: number,
   collisions = 0,
   extrasSkipped = 0,
-): void {
+): { text: string; clean: boolean } {
+  const extras = extrasSkipped > 0 ? `, ${extrasSkipped} extras skipped` : '';
   if (verb === 'push') {
     if (unmapped === 0 && collisions === 0 && extrasSkipped === 0) {
-      ok('summary: clean');
-      return;
+      return { text: 'summary: clean', clean: true };
     }
     const base = `summary: ${unmapped} unmapped on push, ${collisions} collisions`;
-    const extras = extrasSkipped > 0 ? `, ${extrasSkipped} extras skipped` : '';
-    warn(`${base}${extras} (run nomad doctor to list)`);
-    return;
+    return { text: `${base}${extras} (run nomad doctor to list)`, clean: false };
   }
   if (unmapped === 0 && extrasSkipped === 0) {
-    ok('summary: clean');
+    return { text: 'summary: clean', clean: true };
+  }
+  return {
+    text: `summary: ${unmapped} unmapped on ${verb}${extras} (run nomad doctor to list)`,
+    clean: false,
+  };
+}
+
+/**
+ * Build the fully-rendered Summary-section row (status glyph embedded) for the
+ * grouped push/pull tree. Delegates phrasing to `summaryText` so the row text
+ * matches `emitSummary` byte-for-byte. A clean outcome renders
+ * `${green(okGlyph)} <text>`; any warning outcome renders
+ * `${yellow(warnGlyph)} <text>`.
+ *
+ * @param verb - the originating command.
+ * @param unmapped - count of path-map entries skipped for this host.
+ * @param collisions - push-only collision count (ignored for pull/diff).
+ * @param extrasSkipped - count of extras dirnames the whitelist declined.
+ * @returns the rendered row string for the Summary section.
+ */
+export function summaryRow(
+  verb: 'pull' | 'push' | 'diff',
+  unmapped: number,
+  collisions = 0,
+  extrasSkipped = 0,
+): string {
+  const { text, clean } = summaryText(verb, unmapped, collisions, extrasSkipped);
+  return clean ? `${green(okGlyph)} ${text}` : `${yellow(warnGlyph)} ${text}`;
+}
+
+/**
+ * Emit the single end-of-run summary line shared by cmdPull, cmdPush, and
+ * cmdDiff. Delegates phrasing to `summaryText` so the wording cannot drift from
+ * `summaryRow`. Clean outcomes go through `ok()` (green `✓` glyph, stdout) and
+ * unmapped / collision / extras-skipped outcomes go through `warn()` (yellow
+ * `⚠︎` glyph, stderr). The status glyph carries the success/warn semantics;
+ * users see e.g. `✓ summary: clean` or `⚠︎ summary: 3 unmapped on pull (...)`.
+ * Clean still goes to stdout so it survives backgrounded shell-rc invocations
+ * like `nomad pull 2>/dev/null &`. The fourth positional parameter defaults to
+ * 0 so legacy three-arg call sites continue to work unchanged (D-03 additive
+ * contract). `cmdDiff` still calls this for its standalone summary line.
+ */
+export function emitSummary(
+  verb: 'pull' | 'push' | 'diff',
+  unmapped: number,
+  collisions = 0,
+  extrasSkipped = 0,
+): void {
+  const { text, clean } = summaryText(verb, unmapped, collisions, extrasSkipped);
+  if (clean) {
+    ok(text);
     return;
   }
-  const extras = extrasSkipped > 0 ? `, ${extrasSkipped} extras skipped` : '';
-  warn(`summary: ${unmapped} unmapped on ${verb}${extras} (run nomad doctor to list)`);
+  warn(text);
 }