@brainervirus/workit-core 0.5.6 → 0.6.1

package/src/core/youtrack.ts

@@ -1,18 +1,17 @@
 import fs from "node:fs";
-import os from "node:os";
 import path from "node:path";
 import { spawnSync } from "node:child_process";
 import { readTemplate } from "./templates";
 import { resolveWorkspaceRoot } from "./scripts";
+import { configDir } from "./config";
 
 const ISSUE_RE = /^[A-Z]+-\d+$/;
 const TOKEN_PLACEHOLDER = "YOUR_TOKEN_HERE";
 
 // Port of scripts/youtrack/config.sh chain: WORKFLOW_YOUTRACK_CONFIG ->
-// XDG_CONFIG_HOME / HOME .config + workflow-toolkit/youtrack.json.
+// configDir() (XDG_CONFIG_HOME / HOME .config + workit).
 export const youTrackConfigPath = (): string =>
-  process.env.WORKFLOW_YOUTRACK_CONFIG ??
-  path.join(process.env.XDG_CONFIG_HOME || path.join(os.homedir(), ".config"), "workflow-toolkit", "youtrack.json");
+  process.env.WORKFLOW_YOUTRACK_CONFIG ?? path.join(configDir(), "youtrack.json");
 
 const youTrackTokenModeOk = (p: string): boolean => {
   if (process.platform === "win32") return true;
@@ -20,7 +19,9 @@ const youTrackTokenModeOk = (p: string): boolean => {
   return mode === 0o600;
 };
 
-function readYouTrackConfig(required: boolean): { config: Record<string, any>; path: string } | { error: string } {
+function readYouTrackConfig(
+  required: boolean,
+): { config: Record<string, any>; path: string } | { error: string } {
   const cfgPath = youTrackConfigPath();
   if (!fs.existsSync(cfgPath)) {
     return required ? { error: "ERROR: missing youtrack.json" } : { config: {}, path: cfgPath };
@@ -40,7 +41,9 @@ export function youTrackConfigLoad(): { data: Record<string, any> } | { error: s
   const cfgPath = loaded.path;
   const tokenFile = String(loaded.config.tokenFile ?? "");
   const tokenPath = tokenFile
-    ? (path.isAbsolute(tokenFile) ? path.resolve(tokenFile) : path.resolve(process.cwd(), tokenFile))
+    ? path.isAbsolute(tokenFile)
+      ? path.resolve(tokenFile)
+      : path.resolve(process.cwd(), tokenFile)
     : "";
   if (!tokenPath || !fs.existsSync(tokenPath)) {
     return { error: "missing youtrack.token" };
@@ -60,17 +63,29 @@ export function youTrackConfigLoad(): { data: Record<string, any> } | { error: s
   return { data: redacted };
 }
 
-function tzParts(date: Date, tz: string): { y: string; m: string; d: string; hour: string; minute: string } {
+function tzParts(
+  date: Date,
+  tz: string,
+): { y: string; m: string; d: string; hour: string; minute: string } {
   const parts = new Intl.DateTimeFormat("en-GB", {
-    timeZone: tz, year: "numeric", month: "2-digit", day: "2-digit",
-    hour: "2-digit", minute: "2-digit", hourCycle: "h23",
+    timeZone: tz,
+    year: "numeric",
+    month: "2-digit",
+    day: "2-digit",
+    hour: "2-digit",
+    minute: "2-digit",
+    hourCycle: "h23",
   }).formatToParts(date);
   const map = Object.fromEntries(parts.map((p) => [p.type, p.value]));
   return { y: map.year, m: map.month, d: map.day, hour: map.hour, minute: map.minute };
 }
 
 /** Port of scripts/youtrack/greeting.sh. */
-export function youTrackGreeting(configOverride?: string): { stdout: string; exitCode: number; stderr: string } {
+export function youTrackGreeting(configOverride?: string): {
+  stdout: string;
+  exitCode: number;
+  stderr: string;
+} {
   const cfgPath = configOverride ?? youTrackConfigPath();
   try {
     const config = JSON.parse(fs.readFileSync(cfgPath, "utf8")) as Record<string, any>;
@@ -81,20 +96,29 @@ export function youTrackGreeting(configOverride?: string): { stdout: string; exi
     const cutoffHour = Number(cutoff[0]);
     const cutoffMinute = Number(cutoff[1] ?? 0);
     const greetings = (config.greetings ?? {}) as Record<string, string>;
-    const isMorning = Number(hour) < cutoffHour || (Number(hour) === cutoffHour && Number(minute) < cutoffMinute);
+    const isMorning =
+      Number(hour) < cutoffHour || (Number(hour) === cutoffHour && Number(minute) < cutoffMinute);
     const greeting = isMorning
-      ? greetings.morning ?? "buenos días"
-      : greetings.afternoon ?? "buenas tardes";
+      ? (greetings.morning ?? "buenos días")
+      : (greetings.afternoon ?? "buenas tardes");
     const mention = String(config.defaultMention ?? "Alejandra.Flores");
-    void y; void m; void d;
+    void y;
+    void m;
+    void d;
     return { stdout: `@${mention} Hola, ${greeting}.\n`, exitCode: 0, stderr: "" };
   } catch (err) {
-    return { stdout: "", exitCode: 1, stderr: err instanceof Error ? err.message : "greeting failed" };
+    return {
+      stdout: "",
+      exitCode: 1,
+      stderr: err instanceof Error ? err.message : "greeting failed",
+    };
   }
 }
 
 /** Port of scripts/youtrack/parse-duration.sh. */
-export function youTrackParseDuration(text: string): { data: { minutes: number; text: string } } | { error: string } {
+export function youTrackParseDuration(
+  text: string,
+): { data: { minutes: number; text: string } } | { error: string } {
   const lower = String(text).toLowerCase().trim();
   let total = 0;
   for (const match of lower.matchAll(/(\d+)\s*h/g)) total += Number(match[1]) * 60;
@@ -105,13 +129,17 @@ export function youTrackParseDuration(text: string): { data: { minutes: number;
 }
 
 /** Port of scripts/youtrack/work-date-ms.sh — resolve work-item date as epoch ms. */
-export function youTrackWorkDateMs(dateRaw: string): { data: { dateMs: number; timezone: string; localDate: string } } | { error: string } {
+export function youTrackWorkDateMs(
+  dateRaw: string,
+): { data: { dateMs: number; timezone: string; localDate: string } } | { error: string } {
   const cfgPath = youTrackConfigPath();
   let tz = "America/Santiago";
   try {
     const config = JSON.parse(fs.readFileSync(cfgPath, "utf8")) as Record<string, any>;
     tz = String(config.timezone ?? "America/Santiago");
-  } catch { /* defaults */ }
+  } catch {
+    /* defaults */
+  }
   const raw = dateRaw || "auto";
   try {
     if (raw === "auto" || !raw) {
@@ -128,7 +156,13 @@ export function youTrackWorkDateMs(dateRaw: string): { data: { dateMs: number; t
     const [y, m, d] = raw.split("-").map(Number);
     const iso = `${y}-${String(m).padStart(2, "0")}-${String(d).padStart(2, "0")}T00:00:00`;
     const dateMs = Math.floor(Date.parse(iso) / 86400000) * 86400000;
-    return { data: { dateMs, timezone: tz, localDate: `${y}-${String(m).padStart(2, "0")}-${String(d).padStart(2, "0")}` } };
+    return {
+      data: {
+        dateMs,
+        timezone: tz,
+        localDate: `${y}-${String(m).padStart(2, "0")}-${String(d).padStart(2, "0")}`,
+      },
+    };
   } catch (err) {
     return { error: err instanceof Error ? err.message : "could not resolve date" };
   }
@@ -139,14 +173,19 @@ const youTrackToken = (): { token: string; base: string } | { error: string } =>
   if ("error" in loaded) return loaded;
   const tokenFile = String(loaded.config.tokenFile ?? "");
   const tokenPath = tokenFile
-    ? (path.isAbsolute(tokenFile) ? path.resolve(tokenFile) : path.resolve(process.cwd(), tokenFile))
+    ? path.isAbsolute(tokenFile)
+      ? path.resolve(tokenFile)
+      : path.resolve(process.cwd(), tokenFile)
     : "";
   if (!tokenPath || !fs.existsSync(tokenPath)) return { error: "missing youtrack.token" };
   if (!youTrackTokenModeOk(tokenPath)) return { error: "youtrack.token mode must be 0600" };
   const token = fs.readFileSync(tokenPath, "utf8").trim();
   if (!token) return { error: "empty token file" };
   if (token === TOKEN_PLACEHOLDER || token.startsWith(TOKEN_PLACEHOLDER)) {
-    return { error: "token file still has placeholder YOUR_TOKEN_HERE — edit the file locally, then run /wk-status" };
+    return {
+      error:
+        "token file still has placeholder YOUR_TOKEN_HERE — edit the file locally, then run /wk-status",
+    };
   }
   const base = String(loaded.config.baseUrl ?? "").replace(/\/+$/, "");
   if (!base) return { error: "baseUrl missing in config" };
@@ -159,11 +198,17 @@ function youTrackCurl(args: string[]): { status: number; stdout: string; stderr:
 }
 
 /** Port of scripts/youtrack/api.sh — log-time / post-comment with the WORKFLOW_YT_WRITE guard. */
-export function youTrackApi(args: string[], writeFlag = process.env.WORKFLOW_YT_WRITE ?? ""): { data: Record<string, any> } | { error: string } {
+export function youTrackApi(
+  args: string[],
+  writeFlag = process.env.WORKFLOW_YT_WRITE ?? "",
+): { data: Record<string, any> } | { error: string } {
   const cmd = args[0];
   if (cmd === "log-time" || cmd === "post-comment") {
     if (writeFlag !== "1") {
-      return { error: "YouTrack write operations require WORKFLOW_YT_WRITE=1 (refusing to mutate production)" };
+      return {
+        error:
+          "YouTrack write operations require WORKFLOW_YT_WRITE=1 (refusing to mutate production)",
+      };
     }
   }
   const creds = youTrackToken();
@@ -178,13 +223,25 @@ export function youTrackApi(args: string[], writeFlag = process.env.WORKFLOW_YT_
     if ("error" in dateMs) return dateMs;
     const body = JSON.stringify({ duration: { minutes }, text, date: dateMs.data.dateMs });
     const out = youTrackCurl([
-      ...auth, "-H", "Content-Type: application/json", "-d", body,
+      ...auth,
+      "-H",
+      "Content-Type: application/json",
+      "-d",
+      body,
       `${base}/api/issues/${issue}/timeTracking/workItems?fields=id,idReadable`,
     ]);
     if (out.status !== 0) return { error: "YouTrack HTTP request failed" };
     try {
       const created = JSON.parse(out.stdout) as Record<string, any>;
-      return { data: { ok: true, issueId: issue, workItemId: created.id, dateMs: dateMs.data.dateMs, minutes } };
+      return {
+        data: {
+          ok: true,
+          issueId: issue,
+          workItemId: created.id,
+          dateMs: dateMs.data.dateMs,
+          minutes,
+        },
+      };
     } catch {
       return { error: "invalid JSON from YouTrack API" };
     }
@@ -193,7 +250,11 @@ export function youTrackApi(args: string[], writeFlag = process.env.WORKFLOW_YT_
     const [issue, text] = args.slice(1);
     const body = JSON.stringify({ text });
     const out = youTrackCurl([
-      ...auth, "-H", "Content-Type: application/json", "-d", body,
+      ...auth,
+      "-H",
+      "Content-Type: application/json",
+      "-d",
+      body,
       `${base}/api/issues/${issue}/comments`,
     ]);
     if (out.status !== 0) return { error: "YouTrack HTTP request failed" };
@@ -203,18 +264,26 @@ export function youTrackApi(args: string[], writeFlag = process.env.WORKFLOW_YT_
 }
 
 /** Port of scripts/youtrack/verify-token.sh — read-only GET /api/users/me. */
-export function youTrackVerifyToken(): { data: Record<string, any> } | { error: string; http_status?: number; path?: string } {
+export function youTrackVerifyToken():
+  | { data: Record<string, any> }
+  | { error: string; http_status?: number; path?: string } {
   const cfgPath = youTrackConfigPath();
   if (!fs.existsSync(cfgPath)) return { error: "missing youtrack.json" };
   const creds = youTrackToken();
   if ("error" in creds) return { error: creds.error };
   const { token, base } = creds;
 
-  const me = youTrackCurl(["-H", `Authorization: Bearer ${token}`, "-H", "Accept: application/json",
-    `${base}/api/users/me?fields=id,login,name,email`]);
+  const me = youTrackCurl([
+    "-H",
+    `Authorization: Bearer ${token}`,
+    "-H",
+    "Accept: application/json",
+    `${base}/api/users/me?fields=id,login,name,email`,
+  ]);
   if (me.status !== 0) {
     const body = me.stderr.trim() || me.stdout.trim();
-    const err = me.status === 22 ? "authentication failed (401/403)" : `HTTP error: ${body.slice(0, 200)}`;
+    const err =
+      me.status === 22 ? "authentication failed (401/403)" : `HTTP error: ${body.slice(0, 200)}`;
     return { error: err, http_status: me.status };
   }
   let user: Record<string, any>;
@@ -224,21 +293,33 @@ export function youTrackVerifyToken(): { data: Record<string, any> } | { error:
     return { error: "invalid JSON from YouTrack /api/users/me" };
   }
   const result: Record<string, any> = {
-    ok: true, method: "GET /api/users/me", baseUrl: base,
-    login: user.login, name: user.name, email: user.email, id: user.id,
+    ok: true,
+    method: "GET /api/users/me",
+    baseUrl: base,
+    login: user.login,
+    name: user.name,
+    email: user.email,
+    id: user.id,
   };
   const meeting = readYouTrackConfig(false);
   const meetingIssue = "config" in meeting ? meeting.config.meetingIssue : undefined;
   if (meetingIssue) {
-    const issue = youTrackCurl(["-H", `Authorization: Bearer ${token}`, "-H", "Accept: application/json",
-      `${base}/api/issues/${meetingIssue}?fields=id,idReadable,summary`]);
+    const issue = youTrackCurl([
+      "-H",
+      `Authorization: Bearer ${token}`,
+      "-H",
+      "Accept: application/json",
+      `${base}/api/issues/${meetingIssue}?fields=id,idReadable,summary`,
+    ]);
     if (issue.status === 0) {
       try {
         const parsed = JSON.parse(issue.stdout) as Record<string, any>;
         result.meetingIssue = meetingIssue;
         result.meetingIssueReadable = true;
         result.meetingIssueSummary = parsed.summary;
-      } catch { /* unreadable */ }
+      } catch {
+        /* unreadable */
+      }
     } else {
       result.meetingIssue = meetingIssue;
       result.meetingIssueReadable = false;
@@ -256,7 +337,9 @@ export function youTrackTokenCreateUrl(): { data: Record<string, any> } {
   const cfgPath = loaded && "config" in loaded ? loaded.path : youTrackConfigPath();
   const defaults = (config.tokenDefaults ?? {}) as Record<string, any>;
   const name = String(defaults.name ?? tokenName);
-  const desc = String(defaults.description ?? "OpenCode workit — /wk-issue-update and /wk-meetings");
+  const desc = String(
+    defaults.description ?? "OpenCode workit — /wk-issue-update and /wk-meetings",
+  );
   const scopes = Array.isArray(defaults.scopes) ? defaults.scopes : ["YouTrack"];
   const base = String(config.baseUrl ?? "https://enghouseamg.youtrack.cloud").replace(/\/+$/, "");
   const tokenFile = String(config.tokenFile ?? path.join(path.dirname(cfgPath), "youtrack.token"));
@@ -284,7 +367,9 @@ export function youTrackTokenCreateUrl(): { data: Record<string, any> } {
 }
 
 /** Parse bare id (NSR-40) or YouTrack URL into issue id. */
-export function parseIssueRef(input: unknown): { issueId: string; source: string } | { error: string } {
+export function parseIssueRef(
+  input: unknown,
+): { issueId: string; source: string } | { error: string } {
   const trimmed = String(input ?? "").trim();
   if (!trimmed) return { error: "empty issue reference" };
 
@@ -319,11 +404,17 @@ const defaultScripts: YouTrackScripts = {
   api: (args) => youTrackApi(args, process.env.WORKFLOW_YT_WRITE ?? ""),
 };
 
-export function verifyYouTrackToken(scripts: YouTrackScripts = defaultScripts): Record<string, any> {
+export function verifyYouTrackToken(
+  scripts: YouTrackScripts = defaultScripts,
+): Record<string, any> {
   return scripts.config();
 }
 
-function resolveYouTrackFromPaths(spec_path: string | undefined, plan_path: string | undefined, workspace_root: string): string | null {
+function resolveYouTrackFromPaths(
+  spec_path: string | undefined,
+  plan_path: string | undefined,
+  workspace_root: string,
+): string | null {
   const root = resolveWorkspaceRoot(workspace_root);
   for (const rel of [spec_path, plan_path].filter(Boolean) as string[]) {
     const full = path.isAbsolute(rel) ? rel : path.join(root, rel);
@@ -358,7 +449,26 @@ function meetingOptionsFromConfig(cfg: any): Record<string, any>[] {
   ];
 }
 
-export function context({ spec_path, plan_path, issue_id, issue_url, issue_ref, mode, workspace_root }: { spec_path?: string; plan_path?: string; issue_id?: string; issue_url?: string; issue_ref?: string; mode?: string; workspace_root: string }, scripts: YouTrackScripts = defaultScripts): Record<string, any> {
+export function context(
+  {
+    spec_path,
+    plan_path,
+    issue_id,
+    issue_url,
+    issue_ref,
+    mode,
+    workspace_root,
+  }: {
+    spec_path?: string;
+    plan_path?: string;
+    issue_id?: string;
+    issue_url?: string;
+    issue_ref?: string;
+    mode?: string;
+    workspace_root: string;
+  },
+  scripts: YouTrackScripts = defaultScripts,
+): Record<string, any> {
   const cfg = scripts.config();
   if (cfg.error) return { error: cfg.error };
 
@@ -390,7 +500,8 @@ export function context({ spec_path, plan_path, issue_id, issue_url, issue_ref,
   if (!issue) issue = resolveYouTrackFromPaths(spec_path, plan_path, workspace_root) ?? undefined;
   if (!issue || !ISSUE_RE.test(issue)) {
     return {
-      error: "invalid or missing issue id — pass issue_url, issue_id, or spec/plan with **YouTrack:**",
+      error:
+        "invalid or missing issue id — pass issue_url, issue_id, or spec/plan with **YouTrack:**",
       requiresIssueInput: true,
     };
   }
@@ -411,39 +522,86 @@ export function context({ spec_path, plan_path, issue_id, issue_url, issue_ref,
   };
 }
 
-export function parseDuration(text: string, _workspace_root: string, scripts: YouTrackScripts = defaultScripts): Record<string, any> {
+export function parseDuration(
+  text: string,
+  _workspace_root: string,
+  scripts: YouTrackScripts = defaultScripts,
+): Record<string, any> {
   const out = scripts.parseDuration(text);
   if (out.error) return { error: out.error };
   return out.data;
 }
 
-export function logTime({ issueId, minutes, text, date, dateMs, workspace_root }: { issueId: string; minutes: number; text?: string; date?: string; dateMs?: number; workspace_root: string }, scripts: YouTrackScripts = defaultScripts): Record<string, any> {
+export function logTime(
+  {
+    issueId,
+    minutes,
+    text,
+    date,
+    dateMs,
+    workspace_root: _workspace_root,
+  }: {
+    issueId: string;
+    minutes: number;
+    text?: string;
+    date?: string;
+    dateMs?: number;
+    workspace_root: string;
+  },
+  scripts: YouTrackScripts = defaultScripts,
+): Record<string, any> {
   if (!issueId || !ISSUE_RE.test(issueId)) return { error: "invalid issueId" };
   if (!minutes || minutes <= 0) return { error: "minutes must be positive" };
   const workText = text ?? "workit";
   const dateArg =
-    dateMs != null
-      ? String(dateMs)
-      : date && /^\d+$/.test(String(date))
-        ? String(date)
-        : "auto";
+    dateMs != null ? String(dateMs) : date && /^\d+$/.test(String(date)) ? String(date) : "auto";
   const out = scripts.api(["log-time", issueId, String(minutes), workText, dateArg]);
   if (out.error) return { error: out.error };
   return { issueId, minutes, text: workText, ...out.data, ok: true };
 }
 
-
-export function buildDraft({ issueId, projectName, userNotes, greeting, facts, includeProjectOpener, includeFacts }: { issueId: string; projectName?: string; userNotes?: string; greeting?: string; facts?: any; includeProjectOpener?: boolean; includeFacts?: boolean }): Record<string, any> {
+export function buildDraft({
+  issueId,
+  projectName,
+  userNotes,
+  greeting,
+  facts,
+  includeProjectOpener,
+  includeFacts,
+}: {
+  issueId: string;
+  projectName?: string;
+  userNotes?: string;
+  greeting?: string;
+  facts?: any;
+  includeProjectOpener?: boolean;
+  includeFacts?: boolean;
+}): Record<string, any> {
   const tpl = readTemplate("issue-update").content;
   const para = (value: string): string => (value ? `\n\n${value}` : "");
   const filled = tpl
     .replaceAll("{{greetingSection}}", para(greeting ? `${greeting}` : ""))
-    .replaceAll("{{projectSection}}", para(includeProjectOpener && projectName ? `Hoy estuve full con ${projectName}.` : ""))
+    .replaceAll(
+      "{{projectSection}}",
+      para(includeProjectOpener && projectName ? `Hoy estuve full con ${projectName}.` : ""),
+    )
     .replaceAll("{{userNotesSection}}", para((userNotes ?? "").trim()))
-    .replaceAll("{{progressSection}}", para(includeFacts && facts?.progress_excerpt?.length
-      ? facts.progress_excerpt.map((l: string) => `- ${l}`).join("\n") : ""))
-    .replaceAll("{{gitCommitsSection}}", para(includeFacts && facts?.git_commits?.length
-      ? facts.git_commits.map((c: string) => `- ${c}`).join("\n") : ""));
+    .replaceAll(
+      "{{progressSection}}",
+      para(
+        includeFacts && facts?.progress_excerpt?.length
+          ? facts.progress_excerpt.map((l: string) => `- ${l}`).join("\n")
+          : "",
+      ),
+    )
+    .replaceAll(
+      "{{gitCommitsSection}}",
+      para(
+        includeFacts && facts?.git_commits?.length
+          ? facts.git_commits.map((c: string) => `- ${c}`).join("\n")
+          : "",
+      ),
+    );
   const collapsed = filled.replace(/\n{3,}/g, "\n\n").trimEnd();
   // Bare draft keeps the header's trailing blank line (matches legacy output);
   // drafts with sections end right after the last one.
@@ -451,14 +609,31 @@ export function buildDraft({ issueId, projectName, userNotes, greeting, facts, i
   return { issueId, markdown };
 }
 
-export function postUpdate({ confirmed, issueId, markdown, minutes, workspace_root }: { confirmed: boolean; issueId: string; markdown: string; minutes?: number; workspace_root?: string }, operations?: Record<string, any>): Record<string, any> {
+export function postUpdate(
+  {
+    confirmed,
+    issueId,
+    markdown,
+    minutes,
+    workspace_root,
+  }: {
+    confirmed: boolean;
+    issueId: string;
+    markdown: string;
+    minutes?: number;
+    workspace_root?: string;
+  },
+  operations?: Record<string, any>,
+): Record<string, any> {
   operations ??= {};
   if (!confirmed) return { error: "confirmed: true required" };
   if (!issueId || !ISSUE_RE.test(issueId)) return { error: "invalid issueId" };
   if (!markdown?.trim()) return { error: "markdown required" };
 
-  const postComment = operations.postComment ?? ((id: string, text: string, root: string) =>
-    youTrackApi(["post-comment", id, text], process.env.WORKFLOW_YT_WRITE ?? ""));
+  const postComment =
+    operations.postComment ??
+    ((id: string, text: string, _root: string) =>
+      youTrackApi(["post-comment", id, text], process.env.WORKFLOW_YT_WRITE ?? ""));
   const logTimeOperation = operations.logTime ?? logTime;
   const comment = postComment(issueId, markdown, workspace_root);
   if (comment.error) return { error: comment.error };

package/src/core.ts

@@ -7,7 +7,11 @@ export type Result<T> =
   | { ok: false; data: T | null; error: string };
 
 export const ok = <T>(data: T): Result<T> => ({ ok: true, data, error: null });
-export const fail = <T = never>(error: string, data: T | null = null): Result<T> => ({ ok: false, data, error });
+export const fail = <T = never>(error: string, data: T | null = null): Result<T> => ({
+  ok: false,
+  data,
+  error,
+});
 
 const revision = /^[A-Za-z0-9@][A-Za-z0-9@._/~^{}-]*$/;
 
@@ -25,7 +29,13 @@ export function gitRevisionParts(value: string): string[] {
 
 export function resolveGitRevision(root: string, value: string): void {
   for (const part of gitRevisionParts(value)) {
-    const result = run(root, "git", ["rev-parse", "--verify", "--quiet", "--end-of-options", `${part}^{commit}`]);
+    const result = run(root, "git", [
+      "rev-parse",
+      "--verify",
+      "--quiet",
+      "--end-of-options",
+      `${part}^{commit}`,
+    ]);
     if (result.exitCode !== 0) throw new Error(`invalid Git revision or range: ${value}`);
   }
 }
@@ -46,7 +56,12 @@ export function resolveInside(root: string, candidate: string): string {
   return target;
 }
 
-export function run(root: string, executable: string, args: string[], env: Record<string, string> = {}) {
+export function run(
+  root: string,
+  executable: string,
+  args: string[],
+  env: Record<string, string> = {},
+) {
   const cwd = realpathSync(root);
   const result = spawnSync(executable, args, {
     cwd,

package/src/tools/docs-repo.ts

@@ -7,7 +7,8 @@ const output = (value: unknown) => JSON.stringify(value, null, 2);
 export function createDocsRepoTools() {
   return {
     workflow_docs_repo_link: tool({
-      description: "Link the component docs repo in the toolkit config (validates git repo + features/)",
+      description:
+        "Link the component docs repo in the toolkit config (validates git repo + features/)",
       args: {
         path: tool.schema.string(),
         confirmed: tool.schema.boolean(),
@@ -26,7 +27,8 @@ export function createDocsRepoTools() {
       },
     }),
     workflow_docs_promote: tool({
-      description: "Promote a spec (+plan) to the linked docs repo features/YYYY-MM-<slug>/ with quality gate",
+      description:
+        "Promote a spec (+plan) to the linked docs repo features/YYYY-MM-<slug>/ with quality gate",
       args: {
         slug: tool.schema.string(),
         confirmed: tool.schema.boolean(),
@@ -34,7 +36,14 @@ export function createDocsRepoTools() {
       },
       execute: async ({ slug, confirmed, force }, context) => {
         const result = promoteSpec(context.directory, slug, { confirmed, force });
-        if (result.ok) return output(ok({ target_dir: result.target_dir, files: result.files, index_updated: result.index_updated }));
+        if (result.ok)
+          return output(
+            ok({
+              target_dir: result.target_dir,
+              files: result.files,
+              index_updated: result.index_updated,
+            }),
+          );
         return output(fail(result.error, { findings: result.findings ?? [] } as never));
       },
     }),

package/src/tools/flow.ts

@@ -11,8 +11,7 @@ import path from "node:path";
 
 const output = (value: unknown) => JSON.stringify(value, null, 2);
 
-const flowPathFor = (slug: string) =>
-  path.posix.join("docs", slug, "sdd", "flow.json");
+const flowPathFor = (slug: string) => path.posix.join("docs", slug, "sdd", "flow.json");
 
 export function createFlowTools() {
   return {
@@ -27,20 +26,23 @@ export function createFlowTools() {
           const slug = slugFromPath(plan_path ?? spec_path ?? "");
           if (!slug) return output(fail("plan_path or spec_path required"));
           const state = readFlowState(context.directory, slug);
-          return output(ok({
-            slug,
-            spec: state.spec,
-            plan: state.plan,
-            menu: state.menu,
-            flow_path: flowPathFor(slug),
-          }));
+          return output(
+            ok({
+              slug,
+              spec: state.spec,
+              plan: state.plan,
+              menu: state.menu,
+              flow_path: flowPathFor(slug),
+            }),
+          );
         } catch (error) {
           return output(fail(error instanceof Error ? error.message : "flow status failed"));
         }
       },
     }),
     workflow_spec_approve: tool({
-      description: "Advance spec status: first call self_reviewed, second call approved (after user approval)",
+      description:
+        "Advance spec status: first call self_reviewed, second call approved (after user approval)",
       args: {
         confirmed: tool.schema.boolean(),
         spec_path: tool.schema.string(),
@@ -56,7 +58,8 @@ export function createFlowTools() {
       },
     }),
     workflow_plan_approve: tool({
-      description: "Advance plan status: first call self_reviewed, second call approved. Requires approved spec.",
+      description:
+        "Advance plan status: first call self_reviewed, second call approved. Requires approved spec.",
       args: {
         confirmed: tool.schema.boolean(),
         plan_path: tool.schema.string(),
@@ -76,12 +79,20 @@ export function createFlowTools() {
       args: {
         confirmed: tool.schema.boolean(),
         plan_path: tool.schema.string(),
-        choice: tool.schema.enum(["subagent-driven", "inline", "handoff", "review-spec", "review-plan"]),
+        choice: tool.schema.enum([
+          "subagent-driven",
+          "inline",
+          "handoff",
+          "review-spec",
+          "review-plan",
+        ]),
       },
       execute: async ({ confirmed, plan_path, choice }, context) => {
         const slug = slugFromPath(plan_path);
         const result = recordMenuChoice(context.directory, slug, plan_path, choice, confirmed);
-        return output(result.ok ? ok({ menu: { presented: true, chosen: choice } }) : fail(result.error));
+        return output(
+          result.ok ? ok({ menu: { presented: true, chosen: choice } }) : fail(result.error),
+        );
       },
     }),
   };