@xera-ai/cli 0.12.1 → 0.12.3

package/dist/index.js

@@ -15,7 +15,95 @@ import pc from "picocolors";
 import { existsSync, readFileSync } from "fs";
 import { join } from "path";
 import { loadConfig, readAuthState } from "@xera-ai/core";
-async function runChecks(cwd) {
+import { parse as parseYaml } from "yaml";
+function pushTicketChecks(checks, cwd, ticket, acFieldConfigured) {
+  const ticketDir = join(cwd, ".xera", ticket);
+  if (!existsSync(ticketDir)) {
+    checks.push({
+      name: `${ticket}: .xera/${ticket}/ exists`,
+      ok: false,
+      message: `no artifact dir \u2014 run \`/xera-fetch ${ticket}\` first`
+    });
+    return;
+  }
+  const giPath = join(ticketDir, "graph-input.json");
+  if (!existsSync(giPath)) {
+    checks.push({
+      name: `${ticket}: graph-input.json present`,
+      ok: false,
+      message: `missing \u2014 modifiesAreas will be []; run step 5 of /xera-fetch (extract-areas prompt)`
+    });
+  } else {
+    try {
+      const data = JSON.parse(readFileSync(giPath, "utf8"));
+      if (!Array.isArray(data.modifiesAreas)) {
+        checks.push({
+          name: `${ticket}: graph-input.json present`,
+          ok: false,
+          message: `parsed but modifiesAreas is not an array \u2014 re-run step 5 of /xera-fetch`
+        });
+      } else {
+        checks.push({
+          name: `${ticket}: graph-input.json present`,
+          ok: true,
+          message: `${data.modifiesAreas.length} area(s)`
+        });
+      }
+    } catch (e) {
+      checks.push({
+        name: `${ticket}: graph-input.json present`,
+        ok: false,
+        message: `invalid JSON (${e.message}) \u2014 re-run step 5 of /xera-fetch`
+      });
+    }
+  }
+  const storyPath = join(ticketDir, "story.md");
+  if (!existsSync(storyPath)) {
+    checks.push({
+      name: `${ticket}: story.md acceptanceCriteria`,
+      ok: false,
+      message: `story.md missing \u2014 re-run /xera-fetch ${ticket}`
+    });
+    return;
+  }
+  const raw = readFileSync(storyPath, "utf8");
+  const m = raw.match(/^---\n([\s\S]*?)\n---/);
+  if (!m) {
+    checks.push({
+      name: `${ticket}: story.md acceptanceCriteria`,
+      ok: false,
+      message: `frontmatter missing \u2014 re-run /xera-fetch ${ticket}`
+    });
+    return;
+  }
+  let fm;
+  try {
+    fm = parseYaml(m[1]);
+  } catch (e) {
+    checks.push({
+      name: `${ticket}: story.md acceptanceCriteria`,
+      ok: false,
+      message: `frontmatter unparseable (${e.message})`
+    });
+    return;
+  }
+  const ac = Array.isArray(fm.acceptanceCriteria) ? fm.acceptanceCriteria : [];
+  if (ac.length === 0) {
+    const hint = acFieldConfigured ? `jira.fields.acceptanceCriteria is configured but Jira returned no AC for this ticket \u2014 check the ticket in Jira` : `no AC in frontmatter; AC-level coverage will be empty. Set jira.fields.acceptanceCriteria in xera.config.ts if your project stores AC in a dedicated Jira field`;
+    checks.push({
+      name: `${ticket}: story.md acceptanceCriteria`,
+      ok: false,
+      message: hint
+    });
+  } else {
+    checks.push({
+      name: `${ticket}: story.md acceptanceCriteria`,
+      ok: true,
+      message: `${ac.length} AC item(s)`
+    });
+  }
+}
+async function runChecks(cwd, opts = {}) {
   const checks = [];
   checks.push({
     name: `bun ${process.versions.bun ?? "unknown"}`,
@@ -181,6 +269,10 @@ async function runChecks(cwd) {
         }
       } catch {}
     }
+    if (opts.ticket) {
+      const acFieldConfigured = Boolean(cfg.jira?.fields?.acceptanceCriteria);
+      pushTicketChecks(checks, cwd, opts.ticket, acFieldConfigured);
+    }
   } catch (e) {
     checks.push({
       name: "xera.config.ts found and valid",
@@ -210,21 +302,34 @@ async function runChecks(cwd) {
     checks.push({ name: "xera skills present", ok: false, message: "run `xera init`" });
   } else {
     const required = [
-      "xera-run.md",
-      "xera-fetch.md",
-      "xera-feature.md",
-      "xera-script.md",
-      "xera-exec.md",
-      "xera-report.md",
-      "xera-promote.md"
+      "xera-run",
+      "xera-fetch",
+      "xera-feature",
+      "xera-script",
+      "xera-exec",
+      "xera-report",
+      "xera-promote"
     ];
-    const missing = required.filter((n) => !existsSync(join(skillsDir, n)));
+    const missing = [];
+    const legacyFlat = [];
+    for (const base of required) {
+      if (existsSync(join(skillsDir, base, "SKILL.md")))
+        continue;
+      if (existsSync(join(skillsDir, `${base}.md`))) {
+        legacyFlat.push(base);
+      } else {
+        missing.push(base);
+      }
+    }
     const skillsCheck = {
       name: "xera skills present",
-      ok: missing.length === 0
+      ok: missing.length === 0 && legacyFlat.length === 0
     };
-    if (missing.length)
-      skillsCheck.message = `missing: ${missing.join(", ")}`;
+    if (missing.length) {
+      skillsCheck.message = `missing: ${missing.map((b) => `${b}/SKILL.md`).join(", ")}`;
+    } else if (legacyFlat.length) {
+      skillsCheck.message = `legacy flat layout in .claude/skills/ \u2014 run \`xera init --update\` to migrate to <name>/SKILL.md (Claude Code Skill tool requires the directory layout)`;
+    }
     checks.push(skillsCheck);
   }
   return checks;
@@ -248,7 +353,7 @@ async function doctorCommand(opts) {
     console.log("Token usage estimation requires log lines with tokens_in/tokens_out fields (added by skills).");
     return 0;
   }
-  const checks = await runChecks(cwd);
+  const checks = await runChecks(cwd, opts.strict ? { ticket: opts.strict } : {});
   for (const c of checks) {
     const icon = c.ok ? pc.green("\u2713") : pc.red("\u2717");
     console.log(`${icon} ${c.name}${c.message ? pc.dim(` \u2014 ${c.message}`) : ""}`);
@@ -261,9 +366,16 @@ async function doctorCommand(opts) {
 }
 
 // src/commands/init.ts
-import { appendFileSync, existsSync as existsSync3, readFileSync as readFileSync3, unlinkSync, writeFileSync as writeFileSync2 } from "fs";
+import {
+  appendFileSync,
+  existsSync as existsSync3,
+  mkdirSync as mkdirSync2,
+  readdirSync as readdirSync2,
+  readFileSync as readFileSync3,
+  writeFileSync as writeFileSync2
+} from "fs";
 import { createRequire } from "module";
-import { join as join3 } from "path";
+import { dirname as dirname2, join as join3 } from "path";
 import * as p from "@clack/prompts";
 import { generateKey } from "@xera-ai/core";
 import pc2 from "picocolors";
@@ -293,17 +405,6 @@ function scaffoldFile(targetPath, templateName, vars) {
   mkdirSync(dirname(targetPath), { recursive: true });
   writeFileSync(targetPath, render(tmpl, vars));
 }
-function copyDir(src, dest) {
-  mkdirSync(dest, { recursive: true });
-  for (const entry of readdirSync(src, { withFileTypes: true })) {
-    const s = join2(src, entry.name);
-    const d = join2(dest, entry.name);
-    if (entry.isDirectory())
-      copyDir(s, d);
-    else
-      writeFileSync(d, readFileSync2(s));
-  }
-}
 
 // src/commands/init.ts
 var require2 = createRequire(import.meta.url);
@@ -446,15 +547,22 @@ async function initCommand(opts) {
     writeFileSync2(gitignorePath, `${gitignoreAdditions.trim()}
 `);
   }
-  const skillsSrc = require2.resolve("@xera-ai/skills/package.json");
-  const skillsDir = join3(skillsSrc, "..");
-  for (const target of [".claude/skills", ".claude/commands"]) {
-    copyDir(skillsDir, join3(cwd, target));
-    for (const name of ["package.json", "version.json", "CHANGELOG.md"]) {
-      const f = join3(cwd, target, name);
-      if (existsSync3(f))
-        unlinkSync(f);
-    }
+  const skillsPkgPath = require2.resolve("@xera-ai/skills/package.json");
+  const skillsSrcDir = join3(skillsPkgPath, "..");
+  const SKILL_IGNORE = new Set(["package.json", "version.json", "CHANGELOG.md"]);
+  for (const name of readdirSync2(skillsSrcDir)) {
+    if (SKILL_IGNORE.has(name))
+      continue;
+    if (!name.endsWith(".md"))
+      continue;
+    const content = readFileSync3(join3(skillsSrcDir, name));
+    const base = name.replace(/\.md$/, "");
+    const skillFile = join3(cwd, ".claude/skills", base, "SKILL.md");
+    mkdirSync2(dirname2(skillFile), { recursive: true });
+    writeFileSync2(skillFile, content);
+    const cmdFile = join3(cwd, ".claude/commands", name);
+    mkdirSync2(dirname2(cmdFile), { recursive: true });
+    writeFileSync2(cmdFile, content);
   }
   const pkgPath = join3(cwd, "package.json");
   const pkg = existsSync3(pkgPath) ? JSON.parse(readFileSync3(pkgPath, "utf8")) : { name: "xera-project", private: true, type: "module" };
@@ -528,13 +636,14 @@ Next:
 import {
   copyFileSync,
   existsSync as existsSync4,
-  mkdirSync as mkdirSync2,
-  readdirSync as readdirSync2,
+  mkdirSync as mkdirSync3,
+  readdirSync as readdirSync3,
   readFileSync as readFileSync4,
+  unlinkSync,
   writeFileSync as writeFileSync3
 } from "fs";
 import { createRequire as createRequire2 } from "module";
-import { dirname as dirname2, join as join4 } from "path";
+import { dirname as dirname3, join as join4 } from "path";
 import * as p2 from "@clack/prompts";
 import pc3 from "picocolors";
 var require3 = createRequire2(import.meta.url);
@@ -647,7 +756,7 @@ async function initUpdateCommand(opts) {
   pkg.scripts["xera:disputes"] = "xera-internal disputes";
   writeFileSync3(pkgPath, JSON.stringify(pkg, null, 2));
   const wfDir = join4(cwd, ".github/workflows");
-  mkdirSync2(wfDir, { recursive: true });
+  mkdirSync3(wfDir, { recursive: true });
   try {
     const cliPkgPath = require3.resolve("@xera-ai/cli/package.json");
     const cliTplPath = join4(cliPkgPath, "..", "templates/xera-graph.yml.template");
@@ -658,28 +767,47 @@ async function initUpdateCommand(opts) {
   }
   const skillsSrc = require3.resolve("@xera-ai/skills/package.json");
   const newSkillsDir = join4(skillsSrc, "..");
-  const targetDirs = [join4(cwd, ".claude/skills"), join4(cwd, ".claude/commands")];
-  for (const name of readdirSync2(newSkillsDir)) {
+  const SKILL_IGNORE = new Set(["package.json", "version.json", "CHANGELOG.md"]);
+  for (const name of readdirSync3(newSkillsDir)) {
+    if (SKILL_IGNORE.has(name))
+      continue;
     if (!name.endsWith(".md"))
       continue;
     const newContent = readFileSync4(join4(newSkillsDir, name), "utf8");
-    const localStates = targetDirs.map((dir) => {
-      const path = join4(dir, name);
-      if (!existsSync4(path))
-        return { path, state: "missing" };
-      const content = readFileSync4(path, "utf8");
-      return { path, state: content === newContent ? "same" : "diff" };
-    });
-    if (localStates.every((s) => s.state === "missing")) {
-      for (const { path } of localStates) {
-        mkdirSync2(dirname2(path), { recursive: true });
+    const base = name.replace(/\.md$/, "");
+    const skillPath = join4(cwd, ".claude/skills", base, "SKILL.md");
+    const legacyFlatSkillPath = join4(cwd, ".claude/skills", name);
+    const cmdPath = join4(cwd, ".claude/commands", name);
+    let migratedLegacy = false;
+    if (existsSync4(legacyFlatSkillPath) && !existsSync4(skillPath)) {
+      const legacyContent = readFileSync4(legacyFlatSkillPath, "utf8");
+      mkdirSync3(dirname3(skillPath), { recursive: true });
+      writeFileSync3(skillPath, legacyContent);
+      unlinkSync(legacyFlatSkillPath);
+      migratedLegacy = true;
+    }
+    const targets = [];
+    for (const path of [skillPath, cmdPath]) {
+      if (!existsSync4(path)) {
+        targets.push({ path, state: "missing" });
+      } else {
+        const content = readFileSync4(path, "utf8");
+        targets.push({ path, state: content === newContent ? "same" : "diff" });
+      }
+    }
+    if (targets.every((s) => s.state === "missing")) {
+      for (const { path } of targets) {
+        mkdirSync3(dirname3(path), { recursive: true });
         writeFileSync3(path, newContent);
       }
       p2.log.info(`+ ${name}`);
       continue;
     }
-    if (localStates.every((s) => s.state === "same")) {
-      p2.log.info(`= ${name}`);
+    if (targets.every((s) => s.state === "same")) {
+      if (migratedLegacy)
+        p2.log.success(`migrated ${name} to .claude/skills/${base}/SKILL.md`);
+      else
+        p2.log.info(`= ${name}`);
       continue;
     }
     const choice = await p2.select({
@@ -690,12 +818,14 @@ async function initUpdateCommand(opts) {
       ]
     });
     if (choice === "overwrite") {
-      for (const { path } of localStates) {
-        mkdirSync2(dirname2(path), { recursive: true });
+      for (const { path } of targets) {
+        mkdirSync3(dirname3(path), { recursive: true });
         writeFileSync3(path, newContent);
       }
       p2.log.success(`overwrote ${name}`);
     } else {
+      if (migratedLegacy)
+        p2.log.success(`migrated ${name} to .claude/skills/${base}/SKILL.md`);
       p2.log.warn(`kept local ${name}`);
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xera-ai/cli",
-  "version": "0.12.1",
+  "version": "0.12.3",
   "type": "module",
   "bin": {
     "xera": "./bin/xera"
@@ -15,10 +15,11 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@xera-ai/core": "^0.12.1",
-    "@xera-ai/skills": "^0.12.1",
+    "@xera-ai/core": "^0.12.3",
+    "@xera-ai/skills": "^0.12.3",
     "@clack/prompts": "1.4.0",
     "cac": "7.0.0",
-    "picocolors": "1.1.1"
+    "picocolors": "1.1.1",
+    "yaml": "2.9.0"
   }
 }