@geraldmaron/construct 1.1.0 → 1.1.1

package/README.md

@@ -149,6 +149,8 @@ The embed daemon writes its supervisor stdout log to `~/.cx/runtime/embed-daemon
 | `construct ask` | One-shot ask against the active knowledge index |
 | `construct bootstrap` | Import seed observation corpus into local memory store for cold-start acceleration |
 | `construct customer` | Manage customer profiles for product intelligence |
+| `construct demo` | Record reproducible terminal demos via VHS/asciinema (optional system binaries; ADR-0001) |
+| `construct diagram` | Render code-driven diagrams via D2/Graphviz (optional system binaries; ADR-0001) |
 | `construct distill` | Distill documents with query-focused chunking |
 | `construct drop` | Ingest file from Downloads/Desktop |
 | `construct export` | Export markdown to PDF/DOCX/HTML via Pandoc + Typst (optional system binaries; ADR-0024) |

package/bin/construct

@@ -1229,7 +1229,7 @@ async function resolveServiceSelection(args, selectable) {
   return null;
 }
 
-async function cmdUp(args = []) {
+async function cmdDev(args = []) {
   const { SELECTABLE_SERVICES } = await import('../lib/service-manager.mjs');
 
   // --only=a,b,c picks services non-interactively; --select opens a checklist.
@@ -1365,7 +1365,7 @@ async function cmdUp(args = []) {
   opLog.close(failed.length ? 'degraded' : 'ok', { failed: failed.map((r) => r.name) });
 }
 
-async function cmdDown() {
+async function cmdStop() {
   const { stopServices } = await import('../lib/service-manager.mjs');
   const result = await stopServices({ homeDir: HOME, rootDir: ROOT_DIR });
   for (const svc of result.results) {
@@ -1384,7 +1384,7 @@ async function cmdDown() {
   } catch { /* best effort */ }
 }
 
-async function cmdServe() {
+async function cmdDashboard() {
   if (restArgsCache.includes('--token')) {
     const { generateToken, setDashboardToken, getDashboardToken } = await import('../lib/server/auth.mjs');
     const existing = getDashboardToken();
@@ -1951,7 +1951,7 @@ async function cmdHosts() {
     printHostCapabilities();
   }
 }
-async function cmdSetup(args) {
+async function cmdInstall(args) {
   const { startOpLog } = await import('../lib/op-log.mjs');
   const opLog = startOpLog('install', { homeDir: HOME });
   opLog.event('args', { args });
@@ -2402,7 +2402,7 @@ async function cmdIntakeProcess(args, cwd) {
       errorln('Choose one:');
       errorln('  construct intake process --wait        Block until the holder finishes (default 30s).');
       errorln('  construct intake process --wait=<sec>  Block for a custom window.');
-      errorln('  construct down                          Stop background services, then retry.');
+      errorln('  construct stop                          Stop background services, then retry.');
       process.exit(2);
     }
     throw err;
@@ -5209,11 +5209,11 @@ async function readStdin() {
 
 const handlers = new Map([
   // Core
-  ['dev', cmdUp],
-  ['dashboard', cmdServe],
-  ['stop', cmdDown],
+  ['dev', cmdDev],
+  ['dashboard', cmdDashboard],
+  ['stop', cmdStop],
   ['status', cmdStatus],
-  ['install', cmdSetup],
+  ['install', cmdInstall],
   ['config', cmdConfig],
   ['intake', cmdIntake],
   ['recommendations', cmdRecommendations],
@@ -5838,6 +5838,14 @@ const handlers = new Map([
     const { runWireframeCli } = await import('../lib/wireframe.mjs');
     return runWireframeCli(args);
   }],
+  ['diagram', async (args) => {
+    const { runDiagramCli } = await import('../lib/diagram.mjs');
+    return runDiagramCli(args);
+  }],
+  ['demo', async (args) => {
+    const { runDemoCli } = await import('../lib/demo.mjs');
+    return runDemoCli(args);
+  }],
   ['skills', async (args) => {
     const sub = args[0];
     if (sub === 'scope' || !sub) {
@@ -6355,6 +6363,67 @@ if (command === '--version' || command === '-V') {
   process.exit(0);
 }
 
+// Unknown-flag guidance for commands that fully declare their flags (strictFlags
+// in the spec). A stray or typo'd flag on these is a user error, not a no-op:
+// reject it with the nearest declared flag and the command's help, matching how
+// install (lib/setup.mjs KNOWN_FLAGS) already behaves, so no flag is silently
+// swallowed. Opt-in keeps commands that accept undeclared flags unaffected.
+
+function validateKnownFlags(name, argv) {
+  const spec = CLI_COMMANDS.find((c) => c.name === name);
+  if (!spec || !spec.strictFlags) return;
+
+  const exact = new Set(['--help', '-h']);
+  const prefixes = [];
+  for (const opt of spec.options || []) {
+    for (const part of String(opt.flag).split(',')) {
+      const token = part.trim().split('=')[0].split(/\s/)[0];
+      if (!token.startsWith('-')) continue;
+      if (/-<[^>]+>$/.test(token)) prefixes.push(token.replace(/<[^>]+>$/, ''));
+      else exact.add(token);
+    }
+  }
+
+  const declared = [...exact].filter((f) => f.startsWith('--'));
+  const dist = (a, b) => {
+    const row = Array(b.length + 1).fill(0).map((_, i) => i);
+    for (let i = 1; i <= a.length; i += 1) {
+      let prev = i;
+      for (let j = 1; j <= b.length; j += 1) {
+        const cur = a[i - 1] === b[j - 1] ? row[j - 1] : 1 + Math.min(row[j - 1], row[j], prev);
+        row[j - 1] = prev;
+        prev = cur;
+      }
+      row[b.length] = prev;
+    }
+    return row[b.length];
+  };
+
+  const unknown = [];
+  for (const tok of argv) {
+    if (tok === '--') break;
+    if (!/^--?[A-Za-z]/.test(tok)) continue;
+    const bare = tok.split('=')[0];
+    if (exact.has(bare) || prefixes.some((p) => bare.startsWith(p))) continue;
+    unknown.push(bare);
+  }
+  if (unknown.length === 0) return;
+
+  for (const flag of unknown) {
+    errorln(`Unknown flag: ${flag}`);
+    let best = 4;
+    let suggestion = null;
+    for (const cand of declared) {
+      const d = dist(flag, cand);
+      if (d < best) { best = d; suggestion = cand; }
+    }
+    if (suggestion) errorln(`Did you mean: ${suggestion}?`);
+  }
+  println('');
+  println(`${COLORS.dim}Run 'construct ${name} --help' for valid options${COLORS.reset}`);
+  process.exit(1);
+}
+
 const handler = handlers.get(command);
 if (!handler) {
   errorln(`Unknown command: ${command}`);
@@ -6433,7 +6502,9 @@ if (command !== 'hook') {
 
 const probe = await maybeFirstInvocationProbe({ command, homeDir: HOME, env: process.env });
 if (probe?.runSetup) {
-  await cmdSetup([]);
+  await cmdInstall([]);
 }
 
+validateKnownFlags(command, rest);
+
 await handler(rest);

package/lib/audit-trail.mjs

@@ -1,22 +1,68 @@
 /**
- * lib/audit-trail.mjs — reads the append-only audit log produced by
- * lib/hooks/audit-trail.mjs and renders it for `construct audit trail`.
+ * lib/audit-trail.mjs — owns the append-only audit log written by
+ * lib/hooks/audit-trail.mjs and rendered for `construct audit trail`.
  *
  * The log is JSONL at ~/.cx/audit-trail.jsonl. Each line carries a
  * prev_line_hash pointing at the SHA-256 of the previous line — tampering
  * (reorder, delete, edit) breaks the chain and is surfaced by --verify.
+ *
+ * A serial hash chain cannot survive concurrent appends: a read-prev-hash +
+ * append done outside a critical section lets two processes (hook, daemon,
+ * parallel tool calls in one session) both chain off the same predecessor, so
+ * the second link points at a stale tail. appendAuditRecord
+ * closes that window by computing prev_line_hash and appending inside a single
+ * cross-process file lock, making every writer effectively serialized.
+ *
+ * Corrupted links cannot be re-derived, so a `chain_reset` boundary line seals
+ * the damaged prefix as immutable legacy and re-bases the live chain;
+ * verifyChain validates only from the last boundary forward. See
+ * docs/concepts/observability.mdx.
  */
-import { readFileSync, existsSync, statSync } from 'node:fs';
+import { readFileSync, existsSync, statSync, mkdirSync } from 'node:fs';
 import { createHash } from 'node:crypto';
-import { join } from 'node:path';
+import { join, dirname } from 'node:path';
 import { homedir } from 'node:os';
 
+import { appendBounded, readLastLineAcrossSegments } from './logging/rotate.mjs';
+import { withFileLockSync } from './storage/file-lock.mjs';
+
 const AUDIT_FILE = join(homedir(), '.cx', 'audit-trail.jsonl');
 
 function sha256(input) {
   return createHash('sha256').update(input).digest('hex');
 }
 
+// prev_line_hash and the append must be one atomic step or concurrent writers
+// race; the file lock serializes them across processes.
+
+export function appendAuditRecord(partial, { file = AUDIT_FILE } = {}) {
+  return withFileLockSync(file, () => {
+    let prev = null;
+    try {
+      const last = readLastLineAcrossSegments(file);
+      if (last !== null) prev = sha256(last);
+    } catch { /* first record, or unreadable tail */ }
+    const record = { ...partial, prev_line_hash: prev };
+    mkdirSync(dirname(file), { recursive: true });
+    appendBounded('audit-trail', file, JSON.stringify(record) + '\n');
+    return record;
+  });
+}
+
+// A reset boundary seals the records it follows as legacy excluded from
+// verification and re-bases the live chain from this line.
+
+export function writeChainReset({ file = AUDIT_FILE, reason = 'manual reset' } = {}) {
+  return appendAuditRecord({
+    ts: new Date().toISOString(),
+    chain_reset: true,
+    reason,
+    agent: 'construct',
+    tool: null,
+    target: null,
+  }, { file });
+}
+
 export function readAuditTrail({ limit = 50, since = null, agent = null, tool = null } = {}) {
   if (!existsSync(AUDIT_FILE)) return [];
   const lines = readFileSync(AUDIT_FILE, 'utf8').split('\n').filter(Boolean);
@@ -34,19 +80,29 @@ export function readAuditTrail({ limit = 50, since = null, agent = null, tool =
   return filtered.slice(-limit);
 }
 
-export function verifyChain() {
-  if (!existsSync(AUDIT_FILE)) return { ok: true, verified: 0, broken: [] };
-  const lines = readFileSync(AUDIT_FILE, 'utf8').split('\n').filter(Boolean);
+export function verifyChain(file = AUDIT_FILE) {
+  if (!existsSync(file)) return { ok: true, verified: 0, broken: [], legacy: 0 };
+  const lines = readFileSync(file, 'utf8').split('\n').filter(Boolean);
+
+  // Validation starts at the last reset boundary; the prefix it seals is legacy
+  // and exempt, so concurrency damage predating the migration can't fail the chain.
+
+  let start = 0;
+  for (let i = 0; i < lines.length; i += 1) {
+    try { if (JSON.parse(lines[i]).chain_reset) start = i; } catch { /* legacy malformed */ }
+  }
+
   const broken = [];
   let prevHash = null;
-  for (let i = 0; i < lines.length; i += 1) {
+  for (let i = start; i < lines.length; i += 1) {
     const line = lines[i];
     let row;
     try { row = JSON.parse(line); } catch {
       broken.push({ line: i + 1, reason: 'malformed JSON' });
+      prevHash = sha256(line);
       continue;
     }
-    if (i > 0 && row.prev_line_hash !== prevHash) {
+    if (i > start && !row.chain_reset && row.prev_line_hash !== prevHash) {
       broken.push({
         line: i + 1,
         expected: prevHash,
@@ -56,7 +112,7 @@ export function verifyChain() {
     }
     prevHash = sha256(line);
   }
-  return { ok: broken.length === 0, verified: lines.length, broken };
+  return { ok: broken.length === 0, verified: lines.length - start, broken, legacy: start };
 }
 
 function formatRow(row) {
@@ -74,6 +130,9 @@ export async function runAuditTrailCli(args = []) {
   for (let i = 0; i < args.length; i += 1) {
     const arg = args[i];
     if (arg === '--verify') flags.add('verify');
+    else if (arg === '--reset') flags.add('reset');
+    else if (arg.startsWith('--reason=')) options.reason = arg.slice(9);
+    else if (arg === '--reason') options.reason = args[++i];
     else if (arg === '--json') flags.add('json');
     else if (arg === '--all') flags.add('all');
     else if (arg.startsWith('--limit=')) options.limit = Number(arg.slice(8));
@@ -97,6 +156,8 @@ Options:
   --agent <name>    Filter to a specific agent (e.g. cx-engineer)
   --tool <name>     Filter to a specific tool (Edit, Write, MultiEdit, NotebookEdit, Bash)
   --verify          Verify the tamper-evidence chain and exit
+  --reset           Seal the current chain as legacy and re-base a fresh chain
+  --reason <text>   Reason recorded on the --reset boundary
   --json            Emit raw JSONL lines instead of formatted output
   -h, --help        Show this message
 
@@ -105,6 +166,12 @@ Log location: ${AUDIT_FILE}
     return;
   }
 
+  if (flags.has('reset')) {
+    writeChainReset({ reason: options.reason || 'manual reset' });
+    console.log(`Audit chain re-based at ${AUDIT_FILE}. Prior records sealed as legacy; verification resumes from the new boundary.`);
+    return;
+  }
+
   if (flags.has('verify')) {
     const result = verifyChain();
     if (result.ok) {

package/lib/cli-commands.mjs

@@ -23,6 +23,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Start services for development',
     usage: 'construct dev [--select] [--only=postgres,dashboard,...]',
+    examples: [
+      { cmd: 'construct dev', desc: 'Start the default service set' },
+      { cmd: 'construct dev --only=dashboard', desc: 'Start just the dashboard' },
+    ],
     options: [
       { flag: '--select', desc: 'Pick which services to start from an interactive checklist' },
       { flag: '--only=<a,b,c>', desc: 'Start only the named services (postgres, dashboard, telemetry, memory, opencode)' },
@@ -35,6 +39,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Start the local dashboard/orchestration daemon (or --token to mint a dashboard token)',
     usage: 'construct dashboard [--token]',
+    examples: [
+      { cmd: 'construct dashboard', desc: 'Launch the local dashboard daemon' },
+      { cmd: 'construct dashboard --token', desc: 'Mint a dashboard access token' },
+    ],
   },
   {
     name: 'stop',
@@ -43,6 +51,9 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Stop all running services',
     usage: 'construct stop',
+    examples: [
+      { cmd: 'construct stop', desc: 'Stop every running service' },
+    ],
   },
   {
     name: 'status',
@@ -51,6 +62,11 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Show system health and credentials',
     usage: 'construct status',
+    strictFlags: true,
+    examples: [
+      { cmd: 'construct status', desc: 'Human-readable health summary' },
+      { cmd: 'construct status --json', desc: 'Full payload for scripting' },
+    ],
     options: [
       { flag: '--json', desc: 'Output full status payload as JSON' },
     ],
@@ -61,10 +77,15 @@ export const CLI_COMMANDS = [
     category: 'Core',
     core: true,
     description: 'Machine setup (scoped per ADR-0029): --scope=project|user|both, default project',
-    usage: 'construct install [--scope=project|user|both] [--yes] [--no-docker] [--no-launch-agent] [--reconfigure] [--with-docling]',
+    usage: 'construct install [--scope=project|user|both] [--yes] [--dry-run] [--no-docker] [--no-launch-agent] [--reconfigure] [--with-docling]',
+    examples: [
+      { cmd: 'construct install --dry-run', desc: 'Preview the plan before writing anything' },
+      { cmd: 'construct install --scope=user --yes', desc: 'Non-interactive user-scope install' },
+    ],
     options: [
       { flag: '--scope=<s>',      desc: 'project (default, no-op + guidance) | user (writes ~/.construct/, MCP, ~/.claude/* via consent) | both' },
       { flag: '--yes',            desc: 'Apply defaults without prompts (only meaningful with --scope=user|both)' },
+      { flag: '--dry-run',        desc: 'Preview the install plan (scopes, files, services) without writing anything' },
       { flag: '--no-docker',      desc: 'Skip Docker-based service setup (local Postgres)' },
       { flag: '--no-launch-agent', desc: 'Skip background macOS LaunchAgent registration' },
       { flag: '--reconfigure',    desc: 'Re-prompt for service consent, ignoring cached answers' },
@@ -78,6 +99,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Project setup (once per repo): scaffold .cx/, AGENTS.md, plan.md, adapters',
     usage: 'construct init [path] [options]',
+    examples: [
+      { cmd: 'construct init', desc: 'Scaffold the current repo interactively' },
+      { cmd: 'construct init --yes --with-adrs', desc: 'Non-interactive init with the ADR doc lane' },
+    ],
     options: [
       { flag: '--yes', desc: 'Accept all defaults (non-interactive)' },
       { flag: '--no-start', desc: 'Do not start services after init' },
@@ -103,6 +128,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Sync agent adapters to AI tools',
     usage: 'construct sync [--project] [--dry-run] [--no-docs] [--compress-personas]',
+    examples: [
+      { cmd: 'construct sync', desc: 'Sync adapters to detected AI tools' },
+      { cmd: 'construct sync --dry-run', desc: 'Preview adapter changes without writing' },
+    ],
     options: [
       { flag: '--project', desc: 'Write project-local Claude adapters into the current repo only' },
       { flag: '--dry-run', desc: 'Preview adapter changes without writing files' },
@@ -117,6 +146,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'View and process the active profile\'s intake queue (queue label varies by profile)',
     usage: 'construct intake list|show|done|skip|reopen|integrate|classify',
+    examples: [
+      { cmd: 'construct intake list', desc: 'See pending packets' },
+      { cmd: 'construct intake done <id> --output=path', desc: 'Mark processed and stamp the artifact' },
+    ],
     subcommands: [
       { name: 'list', desc: 'List pending packets' },
       { name: 'show <id>', desc: 'Show one packet (triage, related docs, excerpt, tag suggestions)' },
@@ -134,6 +167,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'View and manage artifact recommendations',
     usage: 'construct recommendations list|show|dismiss|stats',
+    examples: [
+      { cmd: 'construct recommendations list', desc: 'See open recommendations' },
+      { cmd: 'construct recommendations dismiss <id>', desc: 'Dismiss one by id' },
+    ],
   },
   {
     name: 'integrations',
@@ -180,6 +217,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Isolated tmpdir-based environment for QA / specialist dry-runs',
     usage: 'construct sandbox create|list|delete|prune [--profile=<id>]',
+    examples: [
+      { cmd: 'construct sandbox create --profile=<id>', desc: 'Spin up an isolated sandbox' },
+      { cmd: 'construct sandbox prune --days=7', desc: 'Remove sandboxes older than a week' },
+    ],
     subcommands: [
       { name: 'create [--profile=<id>]', desc: 'Create a new sandbox under ~/.cx/sandboxes/' },
       { name: 'list', desc: 'List existing sandboxes, newest first' },
@@ -194,6 +235,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Manage the active org profile and its lifecycle (draft, promote, archive, health)',
     usage: 'construct profile show|list|set|create|drafts|archive|health',
+    examples: [
+      { cmd: 'construct profile list', desc: 'List curated profiles' },
+      { cmd: 'construct profile set <id>', desc: 'Switch the active profile' },
+    ],
     subcommands: [
       { name: 'show', desc: 'Show the active profile' },
       { name: 'list', desc: 'List curated profiles' },
@@ -225,6 +270,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Documentation commands',
     usage: 'construct docs check|verify|update',
+    examples: [
+      { cmd: 'construct docs verify', desc: 'Validate the docs tree' },
+      { cmd: 'construct docs update', desc: 'Regenerate AUTO-managed regions' },
+    ],
     subcommands: [
       { name: 'check', desc: 'Check for missing how-to guides' },
       { name: 'verify', desc: 'Validate documentation quality' },
@@ -238,6 +287,10 @@ export const CLI_COMMANDS = [
     core: true,
     description: 'Check installation health',
     usage: 'construct doctor [<status|logs|tick|report|consistency|watch|stop|credentials>] [--fix-legacy-agents]',
+    examples: [
+      { cmd: 'construct doctor', desc: 'Run all health checks once' },
+      { cmd: 'construct doctor report', desc: 'Print the latest health report' },
+    ],
     subcommands: [
       { name: 'status', desc: 'Doctor daemon status' },
       { name: 'logs', desc: 'Tail doctor daemon logs' },
@@ -488,6 +541,7 @@ export const CLI_COMMANDS = [
     core: false,
     description: 'Release dev-agent memory pressure by cleaning stale helper and bridge processes',
     usage: 'construct cleanup [--dry-run] [--quiet] [--pressure-release] [--pressure-only] [--disk-only]',
+    strictFlags: true,
     options: [
       { flag: '--dry-run',          desc: 'Show what would be cleaned without changing anything' },
       { flag: '--quiet',            desc: 'Minimal output' },
@@ -560,6 +614,36 @@ export const CLI_COMMANDS = [
     description: 'Generate wireframes from description',
     usage: 'construct wireframe <description>',
   },
+  {
+    name: 'diagram',
+    emoji: '📊',
+    category: 'Work',
+    core: false,
+    description: 'Render code-driven diagrams via D2/Graphviz (optional system binaries; ADR-0001)',
+    usage: 'construct diagram <description> [--type=architecture|flow|sequence|state|er|class] [--format=svg|png] [--theme=<name>] [--out=<path>] [--source-only]',
+    strictFlags: true,
+    options: [
+      { flag: '--type=<t>',     desc: 'architecture (default) | flow | sequence | state | er | class' },
+      { flag: '--format=<f>',   desc: 'svg (default) | png' },
+      { flag: '--theme=<name>', desc: 'D2 theme name (e.g. neutral, sketch, cool-classics)' },
+      { flag: '--out=<path>',   desc: 'Output path (default: .cx/diagrams/<slug>-<ts>.<ext>)' },
+      { flag: '--source-only',  desc: 'Always write the source file; skip rendering' },
+    ],
+  },
+  {
+    name: 'demo',
+    emoji: '🎬',
+    category: 'Work',
+    core: false,
+    description: 'Record reproducible terminal demos via VHS/asciinema (optional system binaries; ADR-0001)',
+    usage: 'construct demo <name> [--format=gif|mp4|webm] [--out=<path>] [--source-only]',
+    strictFlags: true,
+    options: [
+      { flag: '--format=<f>',  desc: 'gif (default) | mp4 | webm (VHS only)' },
+      { flag: '--out=<path>',  desc: 'Output path (default: .cx/demos/<name>-<ts>.<ext>)' },
+      { flag: '--source-only', desc: 'Always write the .tape source; skip recording' },
+    ],
+  },
   {
     name: 'ollama',
     emoji: '🦙',
@@ -1035,5 +1119,28 @@ export function formatCommandHelp(name, { colors = false } = {}) {
     }
     lines.push('');
   }
+
+  if (spec.examples && spec.examples.length > 0) {
+    lines.push(`${c.bold}Examples${c.reset}`);
+    for (const ex of spec.examples) {
+      lines.push(`  ${ex.cmd}`);
+      if (ex.desc) lines.push(`    ${c.dim}${ex.desc}${c.reset}`);
+    }
+    lines.push('');
+  }
+
+  // Every help block closes with the obvious next action: an explicit `next`,
+  // else "run a subcommand" for grouped commands, plus a See also pointer to the
+  // full reference — so no surface is a dead end.
+
+  const firstSub = spec.subcommands?.[0]?.name.split(/\s+/)[0];
+  const next = spec.next
+    || (firstSub ? `Run \`construct ${spec.name} ${firstSub}\` to start; add --help to any subcommand for details.` : null);
+  if (next) {
+    lines.push(`${c.dim}Next:${c.reset} ${next}`);
+    lines.push('');
+  }
+  lines.push(`${c.dim}See also:${c.reset} run \`construct --help\` for all commands; full reference in docs/reference/cli/.`);
+  lines.push('');
   return lines.join('\n');
 }