@staff0rd/assist 0.658.0 → 0.659.1

package/dist/index.js

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.658.0",
+  version: "0.659.1",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -561,6 +561,23 @@ function stripLegacyConfigKeys(config) {
 // src/shared/types.ts
 import { z as z3 } from "zod";
 
+// src/shared/adviceFragmentNames.ts
+var adviceFragmentNames = [
+  "assist-global",
+  "backlog-ids",
+  "backlog-prs",
+  "code-comments",
+  "drafting-messages",
+  "editing-files",
+  "filename-convention",
+  "jira-context",
+  "jira-smart-links",
+  "markdown",
+  "refactor",
+  "settings-json",
+  "verify"
+];
+
 // src/shared/runConfigSchema.ts
 import { z } from "zod";
 var runParamSchema = z.strictObject({
@@ -619,8 +636,8 @@ var DEFAULT_BACKUP_DIR = "~/.assist/backups";
 var DEFAULT_CLONE_DIR = "~/git";
 var assistConfigShape = {
   advice: z3.strictObject({
-    include: z3.array(z3.string()).default([]),
-    exclude: z3.array(z3.string()).default([]),
+    include: z3.array(z3.enum(adviceFragmentNames)).default([]),
+    exclude: z3.array(z3.enum(adviceFragmentNames)).default([]),
     extra: z3.string().optional(),
     verify: z3.string().optional()
   }).optional(),
@@ -2013,7 +2030,96 @@ function lint(options2 = {}) {
 
 // src/commands/new/registerNew/newCli/index.ts
 import { execSync as execSync12 } from "child_process";
-import { basename as basename3, resolve as resolve6 } from "path";
+import { basename as basename4, resolve as resolve6 } from "path";
+
+// src/commands/advise/loadAdviceFragments.ts
+import { readdirSync, readFileSync as readFileSync10 } from "fs";
+import { basename as basename2, join as join10 } from "path";
+
+// src/commands/advise/adviceDir.ts
+import { existsSync as existsSync13 } from "fs";
+import { dirname as dirname10, join as join9 } from "path";
+import { fileURLToPath } from "url";
+function adviceDir() {
+  let current = dirname10(fileURLToPath(import.meta.url));
+  while (current !== dirname10(current)) {
+    const candidate = join9(current, "claude", "advice");
+    if (existsSync13(candidate)) return candidate;
+    current = dirname10(current);
+  }
+  throw new Error("Could not locate the shipped claude/advice directory");
+}
+
+// src/commands/advise/parseAdviceFragment.ts
+import { parse as parseYaml2 } from "yaml";
+var frontmatter = /^---\r?\n([\s\S]*?)\r?\n---\r?\n?/;
+function parseAdviceFragment(name, content) {
+  const match = frontmatter.exec(content);
+  if (!match) throw new Error(`Advice fragment ${name} has no frontmatter`);
+  const meta = parseYaml2(match[1]) ?? {};
+  const title = meta.title;
+  const when = meta.when;
+  if (typeof title !== "string" || typeof when !== "string")
+    throw new Error(
+      `Advice fragment ${name} needs a title and a when condition`
+    );
+  return {
+    name,
+    title,
+    when,
+    body: content.slice(match[0].length).trim()
+  };
+}
+
+// src/commands/advise/loadAdviceFragments.ts
+function loadAdviceFragments(dir = adviceDir()) {
+  return readdirSync(dir).filter((file) => file.endsWith(".md")).sort().map(
+    (file) => parseAdviceFragment(
+      basename2(file, ".md"),
+      readFileSync10(join10(dir, file), "utf8")
+    )
+  );
+}
+
+// src/commands/verify/reportVerifyProblems.ts
+function verifySection(label2, entries) {
+  if (entries.length === 0) return void 0;
+  const list5 = entries.sort().map((entry) => `  ${entry}`).join("\n");
+  return `${label2}:
+${list5}`;
+}
+function reportVerifyProblems(problems, success) {
+  const found = problems.filter(
+    (problem) => problem !== void 0
+  );
+  if (found.length > 0) {
+    console.log(found.join("\n\n"));
+    process.exit(1);
+  }
+  console.log(success);
+  process.exit(0);
+}
+
+// src/commands/verify/adviceFragments.ts
+function adviceFragments() {
+  const shipped = new Set(
+    loadAdviceFragments().map((fragment) => fragment.name)
+  );
+  const declared = new Set(adviceFragmentNames);
+  reportVerifyProblems(
+    [
+      verifySection(
+        "Shipped in claude/advice but missing from adviceFragmentNames (add them, or advice.include/exclude cannot name them)",
+        [...shipped].filter((name) => !declared.has(name))
+      ),
+      verifySection(
+        "Listed in adviceFragmentNames but no longer shipped in claude/advice (remove them)",
+        [...declared].filter((name) => !shipped.has(name))
+      )
+    ],
+    `All ${shipped.size} advice fragments are named in adviceFragmentNames.`
+  );
+}
 
 // src/commands/verify/blockCodeComments/findComments.ts
 import { execSync as execSync4 } from "child_process";
@@ -2960,12 +3066,12 @@ var adviceConfigHelp = [
   {
     key: "advice.include",
     setter: 'assist config set advice.include "refactor"',
-    note: "fragment names included whatever their condition says"
+    note: "fragment names included whatever their condition says; 'assist advise --explain' lists every name"
   },
   {
     key: "advice.exclude",
     setter: 'assist config set advice.exclude "jira-context"',
-    note: "fragment names dropped even when their condition matches"
+    note: "fragment names dropped even when their condition matches; 'assist advise --explain' lists every name"
   },
   {
     key: "advice.extra",
@@ -3573,57 +3679,46 @@ var pendingConfigDocumentation = /* @__PURE__ */ new Set([
 ]);
 
 // src/commands/verify/configKeys.ts
-function section(label2, keys) {
-  if (keys.length === 0) return void 0;
-  const list5 = keys.sort().map((key) => `  ${key}`).join("\n");
-  return `${label2}:
-${list5}`;
-}
 function configKeys() {
   const schemaKeys = new Set(enumerateConfigLeafKeys(assistConfigSchema));
   const documented = getDocumentedConfigKeys();
   const pending = pendingConfigDocumentation;
   const aggregated = new Set(configHelpEntries.map((entry) => entry.key));
-  const problems = [
-    section(
-      "Config keys documented by no command (surface them with configHelp)",
-      [...schemaKeys].filter(
-        (key) => !documented.has(key) && !pending.has(key)
+  reportVerifyProblems(
+    [
+      verifySection(
+        "Config keys documented by no command (surface them with configHelp)",
+        [...schemaKeys].filter(
+          (key) => !documented.has(key) && !pending.has(key)
+        )
+      ),
+      verifySection(
+        "Documented config keys not present in assistConfigSchema",
+        [...documented].filter((key) => !schemaKeys.has(key))
+      ),
+      verifySection(
+        "Keys now documented but still in pendingConfigDocumentation (remove them)",
+        [...pending].filter((key) => documented.has(key))
+      ),
+      verifySection(
+        "pendingConfigDocumentation lists keys not in assistConfigSchema (remove them)",
+        [...pending].filter((key) => !schemaKeys.has(key))
+      ),
+      verifySection(
+        "Keys documented by a command but missing from configHelpEntries (add their module to src/commands/configHelpEntries.ts)",
+        [...documented].filter((key) => !aggregated.has(key))
+      ),
+      verifySection(
+        "configHelpEntries lists keys no command documents (remove them)",
+        [...aggregated].filter((key) => !documented.has(key))
       )
-    ),
-    section(
-      "Documented config keys not present in assistConfigSchema",
-      [...documented].filter((key) => !schemaKeys.has(key))
-    ),
-    section(
-      "Keys now documented but still in pendingConfigDocumentation (remove them)",
-      [...pending].filter((key) => documented.has(key))
-    ),
-    section(
-      "pendingConfigDocumentation lists keys not in assistConfigSchema (remove them)",
-      [...pending].filter((key) => !schemaKeys.has(key))
-    ),
-    section(
-      "Keys documented by a command but missing from configHelpEntries (add their module to src/commands/configHelpEntries.ts)",
-      [...documented].filter((key) => !aggregated.has(key))
-    ),
-    section(
-      "configHelpEntries lists keys no command documents (remove them)",
-      [...aggregated].filter((key) => !documented.has(key))
-    )
-  ].filter((problem) => problem !== void 0);
-  if (problems.length > 0) {
-    console.log(problems.join("\n\n"));
-    process.exit(1);
-  }
-  console.log(
+    ],
     `All ${schemaKeys.size} config keys are surfaced in --help or pending documentation.`
   );
-  process.exit(0);
 }
 
 // src/commands/verify/forbiddenStrings/index.ts
-import { existsSync as existsSync13, readFileSync as readFileSync10 } from "fs";
+import { existsSync as existsSync14, readFileSync as readFileSync11 } from "fs";
 
 // src/commands/verify/forbiddenStrings/findForbiddenStrings.ts
 import { minimatch as minimatch2 } from "minimatch";
@@ -3664,13 +3759,13 @@ function forbiddenStrings() {
   const cache5 = /* @__PURE__ */ new Map();
   const readJson = (file) => {
     if (cache5.has(file)) return cache5.get(file);
-    if (!existsSync13(file)) {
+    if (!existsSync14(file)) {
       console.log(`Forbidden-strings file not found: ${file}`);
       process.exit(1);
     }
     let parsed;
     try {
-      parsed = JSON.parse(readFileSync10(file, "utf8"));
+      parsed = JSON.parse(readFileSync11(file, "utf8"));
     } catch (error) {
       console.log(`Could not parse ${file}: ${error.message}`);
       process.exit(1);
@@ -3743,7 +3838,7 @@ Total: ${lines2.length} hardcoded color(s)`);
 import * as path20 from "path";
 
 // src/shared/resolveRunConfigs.ts
-import { dirname as dirname10, relative, resolve as resolve4 } from "path";
+import { dirname as dirname11, relative, resolve as resolve4 } from "path";
 
 // src/shared/assertNoDuplicateRunNames.ts
 function findDuplicateNames(configs) {
@@ -3765,14 +3860,14 @@ function assertNoDuplicateRunNames(configs) {
 }
 
 // src/shared/findLinkedConfigPath.ts
-import { existsSync as existsSync14 } from "fs";
-import { join as join9, resolve as resolve2 } from "path";
+import { existsSync as existsSync15 } from "fs";
+import { join as join11, resolve as resolve2 } from "path";
 function findLinkedConfigPath(linkPath, fromDir) {
   const resolved = resolve2(fromDir, linkPath);
-  const claudePath = join9(resolved, ".claude", "assist.yml");
-  if (existsSync14(claudePath)) return claudePath;
-  const rootPath = join9(resolved, "assist.yml");
-  if (existsSync14(rootPath)) return rootPath;
+  const claudePath = join11(resolved, ".claude", "assist.yml");
+  if (existsSync15(claudePath)) return claudePath;
+  const rootPath = join11(resolved, "assist.yml");
+  if (existsSync15(rootPath)) return rootPath;
   throw new Error(`No assist.yml found in linked project: ${resolved}`);
 }
 
@@ -3814,7 +3909,7 @@ function loadAndResolveLink(linkPath, configDir, ctx) {
   const entries = loadLinkedEntries(configPath, ctx.visited);
   const defaultCwd = relativeToRoot(ctx, resolve4(configDir, linkPath));
   return setDefaultCwd(
-    resolveRecursive(entries, dirname10(configPath), ctx),
+    resolveRecursive(entries, dirname11(configPath), ctx),
     defaultCwd
   );
 }
@@ -3841,12 +3936,12 @@ function resolveLocalCwd(config, configDir, ctx) {
 }
 
 // src/shared/findRepoRoot.ts
-import { existsSync as existsSync15 } from "fs";
+import { existsSync as existsSync16 } from "fs";
 import path19 from "path";
 function findRepoRoot(dir) {
   let current = dir;
   while (current !== path19.dirname(current)) {
-    if (existsSync15(path19.join(current, ".git"))) {
+    if (existsSync16(path19.join(current, ".git"))) {
       return current;
     }
     current = path19.dirname(current);
@@ -3917,9 +4012,9 @@ function list() {
 }
 
 // src/commands/verify/migrations/index.ts
-import { readFileSync as readFileSync11 } from "fs";
+import { readFileSync as readFileSync12 } from "fs";
 import path21 from "path";
-import { fileURLToPath } from "url";
+import { fileURLToPath as fileURLToPath2 } from "url";
 
 // src/shared/db/migrations/migration0001Baseline.ts
 var sql = `
@@ -4267,15 +4362,15 @@ function checkSequential(migrations3, files) {
 }
 
 // src/commands/verify/migrations/listMigrationFiles.ts
-import { readdirSync } from "fs";
+import { readdirSync as readdirSync2 } from "fs";
 var MIGRATION_FILE2 = /^migration\d+[A-Za-z0-9]*\.ts$/;
 function listMigrationFiles(dir) {
-  return readdirSync(dir).filter((file) => MIGRATION_FILE2.test(file));
+  return readdirSync2(dir).filter((file) => MIGRATION_FILE2.test(file));
 }
 
 // src/commands/verify/migrations/readBaselineMigrations.ts
 import { execSync as execSync6 } from "child_process";
-import { basename as basename2 } from "path";
+import { basename as basename3 } from "path";
 var MIGRATION_FILE3 = /^migration\d+[A-Za-z0-9]*\.ts$/;
 function readBaselineMigrations(repoRelativeDir, ref) {
   const baseline = /* @__PURE__ */ new Map();
@@ -4288,7 +4383,7 @@ function readBaselineMigrations(repoRelativeDir, ref) {
   } catch {
     return baseline;
   }
-  const paths = listing.split("\n").map((line) => line.trim()).filter(Boolean).filter((path91) => MIGRATION_FILE3.test(basename2(path91)));
+  const paths = listing.split("\n").map((line) => line.trim()).filter(Boolean).filter((path91) => MIGRATION_FILE3.test(basename3(path91)));
   for (const path91 of paths) {
     try {
       const content = execSync6(`git show "${ref}:${path91}"`, {
@@ -4296,7 +4391,7 @@ function readBaselineMigrations(repoRelativeDir, ref) {
         maxBuffer: 16 * 1024 * 1024,
         stdio: ["pipe", "pipe", "pipe"]
       });
-      baseline.set(basename2(path91), content);
+      baseline.set(basename3(path91), content);
     } catch {
     }
   }
@@ -4338,7 +4433,7 @@ function resolveBaselineRef() {
 var REPO_RELATIVE_DIR = "src/shared/db/migrations";
 function migrationsDir() {
   return path21.resolve(
-    path21.dirname(fileURLToPath(import.meta.url)),
+    path21.dirname(fileURLToPath2(import.meta.url)),
     "../../../shared/db/migrations"
   );
 }
@@ -4356,7 +4451,7 @@ function migrations2() {
   if (ref) {
     const baseline = readBaselineMigrations(REPO_RELATIVE_DIR, ref);
     const current = new Map(
-      files.map((file) => [file, readFileSync11(path21.join(dir, file), "utf8")])
+      files.map((file) => [file, readFileSync12(path21.join(dir, file), "utf8")])
     );
     for (const finding of checkAppendOnly(baseline, current)) {
       problems.push(
@@ -4415,15 +4510,15 @@ import * as net from "net";
 
 // src/commands/sessions/daemon/daemonPaths.ts
 import { homedir as homedir2 } from "os";
-import { join as join10 } from "path";
-var DAEMON_DIR = join10(homedir2(), ".assist", "daemon");
+import { join as join12 } from "path";
+var DAEMON_DIR = join12(homedir2(), ".assist", "daemon");
 var daemonPaths = {
   dir: DAEMON_DIR,
-  socket: process.platform === "win32" ? String.raw`\\.\pipe\assist-sessions-daemon` : join10(DAEMON_DIR, "daemon.sock"),
-  log: join10(DAEMON_DIR, "daemon.log"),
-  pid: join10(DAEMON_DIR, "daemon.pid"),
-  spawnLock: join10(DAEMON_DIR, "spawn.lock"),
-  hooksSettings: join10(DAEMON_DIR, "hooks-settings.json")
+  socket: process.platform === "win32" ? String.raw`\\.\pipe\assist-sessions-daemon` : join12(DAEMON_DIR, "daemon.sock"),
+  log: join12(DAEMON_DIR, "daemon.log"),
+  pid: join12(DAEMON_DIR, "daemon.pid"),
+  spawnLock: join12(DAEMON_DIR, "spawn.lock"),
+  hooksSettings: join12(DAEMON_DIR, "hooks-settings.json")
 };
 
 // src/commands/sessions/daemon/connectToDaemon.ts
@@ -4731,7 +4826,7 @@ program.parse();
 
 // src/commands/new/registerNew/newCli/index.ts
 async function newCli() {
-  const name = basename3(resolve6("."));
+  const name = basename4(resolve6("."));
   initGit();
   initPackageJson(name);
   console.log("Installing dependencies...");
@@ -4746,7 +4841,7 @@ async function newCli() {
 
 // src/commands/new/registerNew/newProject.ts
 import { execSync as execSync14 } from "child_process";
-import { existsSync as existsSync17, readFileSync as readFileSync13, writeFileSync as writeFileSync13 } from "fs";
+import { existsSync as existsSync18, readFileSync as readFileSync14, writeFileSync as writeFileSync13 } from "fs";
 
 // src/commands/deploy/init/index.ts
 import { execSync as execSync13 } from "child_process";
@@ -4754,33 +4849,33 @@ import chalk27 from "chalk";
 import enquirer3 from "enquirer";
 
 // src/commands/deploy/init/updateWorkflow.ts
-import { existsSync as existsSync16, mkdirSync as mkdirSync3, readFileSync as readFileSync12, writeFileSync as writeFileSync12 } from "fs";
-import { dirname as dirname12, join as join11 } from "path";
-import { fileURLToPath as fileURLToPath2 } from "url";
+import { existsSync as existsSync17, mkdirSync as mkdirSync3, readFileSync as readFileSync13, writeFileSync as writeFileSync12 } from "fs";
+import { dirname as dirname13, join as join13 } from "path";
+import { fileURLToPath as fileURLToPath3 } from "url";
 import chalk26 from "chalk";
 var WORKFLOW_PATH = ".github/workflows/build.yml";
-var __dirname2 = dirname12(fileURLToPath2(import.meta.url));
+var __dirname2 = dirname13(fileURLToPath3(import.meta.url));
 function getExistingSiteId() {
-  if (!existsSync16(WORKFLOW_PATH)) {
+  if (!existsSync17(WORKFLOW_PATH)) {
     return null;
   }
-  const content = readFileSync12(WORKFLOW_PATH, "utf8");
+  const content = readFileSync13(WORKFLOW_PATH, "utf8");
   const match = content.match(/-s\s+([a-f0-9-]{36})/);
   return match ? match[1] : null;
 }
 function getTemplateContent(siteId) {
-  const templatePath = join11(__dirname2, "commands/deploy/build.yml");
-  const template = readFileSync12(templatePath, "utf8");
+  const templatePath = join13(__dirname2, "commands/deploy/build.yml");
+  const template = readFileSync13(templatePath, "utf8");
   return template.replace("{{NETLIFY_SITE_ID}}", siteId);
 }
 async function updateWorkflow(siteId) {
   const newContent = getTemplateContent(siteId);
   const workflowDir = ".github/workflows";
-  if (!existsSync16(workflowDir)) {
+  if (!existsSync17(workflowDir)) {
     mkdirSync3(workflowDir, { recursive: true });
   }
-  if (existsSync16(WORKFLOW_PATH)) {
-    const oldContent = readFileSync12(WORKFLOW_PATH, "utf8");
+  if (existsSync17(WORKFLOW_PATH)) {
+    const oldContent = readFileSync13(WORKFLOW_PATH, "utf8");
     if (oldContent === newContent) {
       console.log(chalk26.green("build.yml is already up to date"));
       return;
@@ -4874,11 +4969,11 @@ async function newProject() {
 }
 function addViteBaseConfig() {
   const viteConfigPath = "vite.config.ts";
-  if (!existsSync17(viteConfigPath)) {
+  if (!existsSync18(viteConfigPath)) {
     console.log("No vite.config.ts found, skipping base config");
     return;
   }
-  const content = readFileSync13(viteConfigPath, "utf8");
+  const content = readFileSync14(viteConfigPath, "utf8");
   if (content.includes("base:")) {
     console.log("vite.config.ts already has base config");
     return;
@@ -5038,7 +5133,9 @@ function renderLineChart({
   seriesTitle,
   labels,
   values,
-  wholeNumbersOnly = false
+  wholeNumbersOnly = false,
+  minY,
+  maxY
 }) {
   const input = keyboardInput();
   const screen = blessed.screen({
@@ -5053,7 +5150,9 @@ function renderLineChart({
     legend: { width: Math.max(12, seriesTitle.length + 2) },
     xLabelPadding: 3,
     xPadding: 5,
-    wholeNumbersOnly
+    wholeNumbersOnly,
+    minY,
+    maxY
   });
   line.setData([
     {
@@ -5151,55 +5250,6 @@ function adviceContextFor(cwd) {
   };
 }
 
-// src/commands/advise/loadAdviceFragments.ts
-import { readdirSync as readdirSync2, readFileSync as readFileSync14 } from "fs";
-import { basename as basename4, join as join13 } from "path";
-
-// src/commands/advise/adviceDir.ts
-import { existsSync as existsSync18 } from "fs";
-import { dirname as dirname13, join as join12 } from "path";
-import { fileURLToPath as fileURLToPath3 } from "url";
-function adviceDir() {
-  let current = dirname13(fileURLToPath3(import.meta.url));
-  while (current !== dirname13(current)) {
-    const candidate = join12(current, "claude", "advice");
-    if (existsSync18(candidate)) return candidate;
-    current = dirname13(current);
-  }
-  throw new Error("Could not locate the shipped claude/advice directory");
-}
-
-// src/commands/advise/parseAdviceFragment.ts
-import { parse as parseYaml2 } from "yaml";
-var frontmatter = /^---\r?\n([\s\S]*?)\r?\n---\r?\n?/;
-function parseAdviceFragment(name, content) {
-  const match = frontmatter.exec(content);
-  if (!match) throw new Error(`Advice fragment ${name} has no frontmatter`);
-  const meta = parseYaml2(match[1]) ?? {};
-  const title = meta.title;
-  const when = meta.when;
-  if (typeof title !== "string" || typeof when !== "string")
-    throw new Error(
-      `Advice fragment ${name} needs a title and a when condition`
-    );
-  return {
-    name,
-    title,
-    when,
-    body: content.slice(match[0].length).trim()
-  };
-}
-
-// src/commands/advise/loadAdviceFragments.ts
-function loadAdviceFragments(dir = adviceDir()) {
-  return readdirSync2(dir).filter((file) => file.endsWith(".md")).sort().map(
-    (file) => parseAdviceFragment(
-      basename4(file, ".md"),
-      readFileSync14(join13(dir, file), "utf8")
-    )
-  );
-}
-
 // src/commands/advise/verifyRunCommandNames.ts
 function verifyRunCommandNames({
   config,
@@ -5268,9 +5318,11 @@ var adviceConditions = {
 // src/commands/advise/selectAdvice.ts
 function decide(fragment, context) {
   const advice = context.config.advice;
-  if (advice?.exclude.includes(fragment.name))
+  const exclude = advice?.exclude ?? [];
+  const include = advice?.include ?? [];
+  if (exclude.includes(fragment.name))
     return { fragment, included: false, reason: "excluded by advice.exclude" };
-  if (advice?.include.includes(fragment.name))
+  if (include.includes(fragment.name))
     return { fragment, included: true, reason: "included by advice.include" };
   const condition = adviceConditions[fragment.when];
   if (!condition)
@@ -6362,6 +6414,19 @@ function registerBackup(program2) {
   configHelp(backupCommand, backupConfigHelp);
 }
 
+// src/commands/chart/chartYRange.ts
+var toCentsWithoutFloatError = (value) => Number((value * 100).toFixed(6));
+function chartYRange(values) {
+  const min = Math.min(...values);
+  const max = Math.max(...values);
+  const span = max - min;
+  const pad2 = span === 0 ? Math.max(Math.abs(max) * 0.1, 1) : span * 0.2;
+  return {
+    minY: Math.floor(toCentsWithoutFloatError(min - pad2)) / 100,
+    maxY: Math.ceil(toCentsWithoutFloatError(max + pad2)) / 100
+  };
+}
+
 // src/commands/chart/parseChartSeries.ts
 function parseChartSeries(lines2) {
   const points = [];
@@ -6412,12 +6477,14 @@ async function chart(options2) {
     return;
   }
   const title = options2.title ?? "Chart";
+  const values = points.map((p) => p.value);
   renderLineChart({
     title,
     label: title,
     seriesTitle: title,
     labels: points.map((p) => p.label),
-    values: points.map((p) => p.value)
+    values,
+    ...chartYRange(values)
   });
 }
 
@@ -16800,13 +16867,13 @@ function phaseNotes(entry, position) {
 }
 function renderDiffPhase(entry, idx) {
   const notes = phaseNotes(entry, idx + 1);
-  const section3 = renderPhaseSection(
+  const section2 = renderPhaseSection(
     { ...entry.phase, name: `${entry.phase.name} (${notes.join(", ")})` },
     idx
   );
-  if (!entry.previousTasks) return section3;
+  if (!entry.previousTasks) return section2;
   return [
-    section3,
+    section2,
     "**Previously:**",
     entry.previousTasks.map((task) => `- ${task}`).join("\n")
   ].join("\n\n");
@@ -26319,9 +26386,9 @@ function parsePrBody(body) {
   return sections;
 }
 function serializePrBody(sections) {
-  return sections.map((section3) => `## ${section3.heading}
+  return sections.map((section2) => `## ${section2.heading}
 
-${section3.content}`).join("\n\n");
+${section2.content}`).join("\n\n");
 }
 
 // src/commands/prs/editPrBody.ts
@@ -26370,11 +26437,11 @@ function findWallOfText(body) {
 }
 function splitParagraphs(body) {
   const paragraphs = [];
-  let section3 = "(intro)";
+  let section2 = "(intro)";
   let lines2 = [];
   const flush = () => {
     if (lines2.length > 0) {
-      paragraphs.push({ section: section3, lines: lines2 });
+      paragraphs.push({ section: section2, lines: lines2 });
       lines2 = [];
     }
   };
@@ -26382,7 +26449,7 @@ function splitParagraphs(body) {
     const heading2 = line.match(/^#{1,6}\s+(.*)$/);
     if (heading2) {
       flush();
-      section3 = heading2[1].trim();
+      section2 = heading2[1].trim();
     } else if (line.trim() === "") {
       flush();
     } else {
@@ -29103,13 +29170,13 @@ function buildPlan2(functionName, sourceFile, sourcePath, destPath, project) {
 // src/commands/refactor/extract/displayPlan.ts
 import path50 from "path";
 import chalk199 from "chalk";
-function section2(title) {
+function section(title) {
   return `
   ${chalk199.cyan(title)}`;
 }
 function displayImporters(plan2, cwd) {
   if (plan2.importersToUpdate.length === 0) return;
-  console.log(section2("Update importers:"));
+  console.log(section("Update importers:"));
   for (const imp of plan2.importersToUpdate) {
     const rel = path50.relative(cwd, imp.file.getFilePath());
     console.log(`    ${chalk199.dim(rel)}: \u2192 import from "${imp.relPath}"`);
@@ -29123,18 +29190,18 @@ function displayPlan(functionName, relDest, plan2, cwd) {
     console.log(`    ${name}`);
   }
   if (plan2.imports.length > 0) {
-    console.log(section2("Imports to copy:"));
+    console.log(section("Imports to copy:"));
     for (const imp of plan2.imports) {
       console.log(`    ${formatImportLine(imp)}`);
     }
   }
   if (plan2.exportedDeps.length > 0) {
-    console.log(section2("New imports from source:"));
+    console.log(section("New imports from source:"));
     console.log(
       `    import { ${plan2.exportedDeps.join(", ")} } from "${plan2.sourceRelPath}";`
     );
   }
-  console.log(section2("Source file changes:"));
+  console.log(section("Source file changes:"));
   console.log(`    Remove: ${plan2.extractedNames.join(", ")}`);
   if (plan2.sourceNeedsReimport) {
     console.log(
@@ -29143,7 +29210,7 @@ function displayPlan(functionName, relDest, plan2, cwd) {
   }
   displayImporters(plan2, cwd);
   if (plan2.barrel) {
-    console.log(section2("Barrel export:"));
+    console.log(section("Barrel export:"));
     console.log(
       `    Add: export { ${functionName} } from "${plan2.barrelRelPath}";`
     );
@@ -34080,6 +34147,11 @@ var checks = [
     description: "Check every assistConfigSchema key is surfaced in some command's --help via configHelp",
     action: configKeys
   },
+  {
+    name: "advice-fragments",
+    description: "Check ADVICE_FRAGMENT_NAMES matches the fragments shipped in claude/advice, so advice.include/exclude can name every one",
+    action: adviceFragments
+  },
   {
     name: "migrations",
     description: "Check DB migrations are sequentially numbered, append-only, and gate destructive DDL behind an acknowledgement marker",