tldr-experts 0.3.1 → 0.5.0

package/dist/hooks/{chunk-ytvmc5ns.js → chunk-889tybxc.js}

@@ -1,8 +1,9 @@
 import {
   emitFactsYaml,
   withWorkspaceLock,
-  workspaceRootOfFactsPath
-} from "./chunk-9kkm6q0t.js";
+  workspaceRootOfFactsPath,
+  writeAtomic
+} from "./chunk-qw73rdbr.js";
 import {
   FACT_CONFIDENCES,
   FACT_KINDS,
@@ -10,9 +11,10 @@ import {
   MAX_FACT_CHARS,
   factNumber,
   formatFactId,
+  isLive,
   isRetired,
   noteDeprecations
-} from "./chunk-3t91gvpp.js";
+} from "./chunk-nadqsr3w.js";
 import {
   asDocument,
   isRecord,
@@ -24,16 +26,17 @@ import {
   result
 } from "./chunk-39zh2e44.js";
 import {
-  parseYaml
-} from "./chunk-ztczwtj0.js";
+  SRC_PATTERNS,
+  parseYaml,
+  readableSource
+} from "./chunk-a6rpj2cp.js";
 
 // src/core/facts/FactsStore.ts
-import { existsSync, mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
-import { dirname } from "node:path";
+import { existsSync, readFileSync } from "node:fs";
 
 // src/core/facts/validateFactsFile.ts
-var ID_RE = /^F\d{3,6}$/;
-var Q_RE = /^Q\d{1,6}$/;
+var ID_RE = SRC_PATTERNS.fact;
+var Q_RE = SRC_PATTERNS.answer;
 function validateFactsFile(input) {
   const issues = [];
   const deprecations = [];
@@ -59,7 +62,7 @@ function validateFactsFile(input) {
     requireKeys(row, ["id", "fact", "area", "repos", "kind", "confidence", "source", "supersedes", "superseded_by", "retired"], path, issues);
     const id = typeof row.id === "string" ? row.id : "";
     if (!ID_RE.test(id))
-      issues.push({ path: `${path}.id`, message: "id must match ^F\\d{3,6}$" });
+      issues.push({ path: `${path}.id`, message: `id must match ${readableSource(ID_RE)}` });
     else if (byId.has(id))
       issues.push({ path: `${path}.id`, message: `duplicate fact id ${id}` });
     else {
@@ -81,7 +84,7 @@ function validateFactsFile(input) {
       requireKeys(row.source, ["who", "when", "run", "q"], `${path}.source`, issues);
       const q = row.source.q;
       if (typeof q === "string" && !Q_RE.test(q)) {
-        issues.push({ path: `${path}.source.q`, message: "expected ^Q\\d+$ or null" });
+        issues.push({ path: `${path}.source.q`, message: `expected ${readableSource(Q_RE)} or null` });
       }
     } else if (row.source !== undefined) {
       issues.push({ path: `${path}.source`, message: "expected a mapping" });
@@ -153,7 +156,7 @@ function findDuplicate(question, area, facts, threshold = DEFAULT_JACCARD_THRESH
   for (const fact of facts) {
     if (fact.area !== area)
       continue;
-    if (isRetired(fact))
+    if (!isLive(fact))
       continue;
     const score = jaccard(asked, tokenize(fact.fact));
     if (score < threshold)
@@ -199,7 +202,7 @@ class FactsStore {
     return this.rows;
   }
   get active() {
-    return this.rows.filter((f) => !isRetired(f));
+    return this.rows.filter(isLive);
   }
   get(id) {
     return this.rows.find((f) => f.id === id);
@@ -226,6 +229,15 @@ class FactsStore {
     this.rows.push(fact);
     return fact;
   }
+  headOf(id) {
+    let fact = this.get(id);
+    for (let step = 0;fact !== undefined && fact.superseded_by !== null; step++) {
+      if (step >= this.rows.length)
+        return;
+      fact = this.get(fact.superseded_by);
+    }
+    return fact;
+  }
   supersede(oldId, input) {
     const index = this.rows.findIndex((f) => f.id === oldId);
     if (index === -1)
@@ -269,7 +281,7 @@ class FactsStore {
       throw new Error(`refusing to write an invalid facts.yml: ${first?.path ?? ""} ${first?.message ?? ""}`);
     }
     withWorkspaceLock(workspaceRootOfFactsPath(path), () => {
-      writeFactsAtomic(path, text);
+      writeAtomic(path, text);
     });
   }
   static update(path, fn) {
@@ -281,19 +293,5 @@ class FactsStore {
     });
   }
 }
-function writeFactsAtomic(path, text) {
-  const temp = `${path}.tmp-${String(process.pid)}`;
-  mkdirSync(dirname(path), { recursive: true });
-  try {
-    writeFileSync(temp, text, "utf8");
-    renameSync(temp, path);
-  } catch (error) {
-    try {
-      if (existsSync(temp))
-        rmSync(temp, { force: true });
-    } catch {}
-    throw error;
-  }
-}
 
 export { FactsStore };

package/dist/hooks/{chunk-9gb21660.js → chunk-a5dq2dcp.js}

@@ -1,27 +1,67 @@
+import {
+  BULLET_RULE,
+  EMPTY_SECTION_RULE,
+  noneBullet
+} from "./chunk-q8d3sff9.js";
+import {
+  srcRule
+} from "./chunk-a6rpj2cp.js";
+
 // src/hooks/lib/messages.ts
 var GRAMMAR = "[src: <repo:path:line> | https://… | Q<n> | F<n> | $ <cmd> → exit <n> | graph:<node> | absent:<path>]";
 function usd(amount) {
   return `$${amount.toFixed(2)}`;
 }
-function claimSourcesDeny(relPath, lines) {
-  const list = lines.map((n) => `L${n}`).join(", ");
-  return `[tldrx] claim-sources: ${lines.length} unsourced bullet(s) in ${relPath} — ${list}.
-` + `Every bullet under Findings/Decisions/Unknowns/Evidence ledger must end with ${GRAMMAR}. ` + "Add the source or delete the claim.";
+var MAX_QUOTED_CHARS = 200;
+function quoteLine(text, line) {
+  if (text === "" || line < 1)
+    return null;
+  const found = text.split(`
+`)[line - 1];
+  if (found === undefined || found.trim() === "")
+    return null;
+  const trimmed = found.trim();
+  return trimmed.length <= MAX_QUOTED_CHARS ? trimmed : `${trimmed.slice(0, MAX_QUOTED_CHARS)}…`;
+}
+function issueBlock(issue, text) {
+  const out = [`L${String(issue.line)}: ${issue.message}`];
+  const quoted = quoteLine(text, issue.line);
+  if (quoted !== null)
+    out.push(`      you wrote: ${quoted}`);
+  if (issue.rule !== undefined) {
+    const rule = srcRule(issue.rule);
+    out.push(issue.message.includes(rule.rule) ? `      rule:      \`${rule.id}\`` : `      rule \`${rule.id}\`: ${rule.rule}`);
+    out.push(`      corrected: ${rule.good}`);
+  }
+  return out.join(`
+`);
+}
+function claimSourcesDeny(relPath, lines, text = "") {
+  const list = lines.map((n) => `L${String(n)}`).join(", ");
+  const quotes = lines.map((n) => ({ n, quoted: quoteLine(text, n) })).filter((row) => row.quoted !== null).map((row) => `L${String(row.n)}: ${row.quoted}`);
+  return `[tldrx] claim-sources: ${String(lines.length)} unsourced bullet(s) in ${relPath} — ${list}.
+` + (quotes.length === 0 ? "" : `${quotes.join(`
+`)}
+`) + `Rule: ${BULLET_RULE}. The token goes at the END of the line: ${GRAMMAR}.
+` + "Add the source or delete the claim.";
 }
 function claimSourcesEmptySectionDeny(relPath, sections) {
-  const list = sections.map((s) => `"${s.name}" (L${s.line})`).join(", ");
-  return `[tldrx] claim-sources: ${sections.length} checked section(s) in ${relPath} contain no list ` + `items — ${list}.
-` + "Findings/Decisions/Unknowns/Evidence ledger must each hold at least one sourced item; prose alone " + "is not a claim anything can check. If there is genuinely nothing, say so as an item: " + "`- none [src: absent:<what you looked at>]`.";
+  const list = sections.map((s) => `"${s.name}" (L${String(s.line)})`).join(", ");
+  return `[tldrx] claim-sources: ${String(sections.length)} checked section(s) in ${relPath} contain no list ` + `items — ${list}.
+` + `Rule: ${EMPTY_SECTION_RULE}.
+` + "If there is genuinely nothing, say so as an item: " + `\`${noneBullet("<what you looked at>")}\`.`;
 }
-function claimSourcesMalformedDeny(relPath, issues) {
-  const list = issues.map((i) => `L${i.line}`).join(", ");
-  return `[tldrx] claim-sources: ${issues.length} malformed citation(s) in ${relPath} — ${list}.
-` + "The bullet cites a source but the token could not be read. The `[src: …]` token must be the LAST " + "thing on the line: remove the backticks around it and any words after it. A closing quote, bracket " + "or a final `.` after the `]` is fine.";
+function claimSourcesMalformedDeny(relPath, issues, text = "") {
+  const list = issues.map((i) => `L${String(i.line)}`).join(", ");
+  return `[tldrx] claim-sources: ${String(issues.length)} malformed citation(s) in ${relPath} — ${list}.
+` + `${issues.map((issue) => issueBlock(issue, text)).join(`
+`)}
+` + "The bullet cites a source but the token could not be read. Fix the citation or delete the claim.";
 }
-function claimSourcesUnresolvedDeny(relPath, issues) {
-  const detail = issues.map((i) => `L${i.line}: ${i.message}`).join(`
+function claimSourcesUnresolvedDeny(relPath, issues, text = "") {
+  const detail = issues.map((issue) => issueBlock(issue, text)).join(`
 `);
-  return `[tldrx] claim-sources: ${issues.length} unresolvable source(s) in ${relPath}.
+  return `[tldrx] claim-sources: ${String(issues.length)} unresolvable source(s) in ${relPath}.
 ` + `${detail}
 ` + "A cited file must exist with the line in range, a command must be one of workspace.yml's, and " + "`$ … → exit n` belongs only in the Evidence ledger. Fix the citation or delete the claim.";
 }