trackops 2.0.6 → 2.2.0

package/scripts/skills-marketplace-smoke.js

@@ -1,124 +1,156 @@
-#!/usr/bin/env node
-
-const assert = require("assert");
-const fs = require("fs");
-const os = require("os");
-const path = require("path");
-const { spawnSync } = require("child_process");
-
-const ROOT = path.resolve(__dirname, "..");
-const SKILL_DIR = path.join(ROOT, "skills", "trackops");
-const SKILL_CONFIG = JSON.parse(fs.readFileSync(path.join(SKILL_DIR, "skill.json"), "utf8"));
-
-function run(command, args, cwd, envOverrides = {}) {
-  const shell = process.platform === "win32" && /\.(cmd|bat)$/i.test(command);
-  return spawnSync(command, args, {
-    cwd,
-    encoding: "utf8",
-    env: { ...process.env, ...envOverrides },
-    shell,
-  });
-}
-
-function getNpxCommand() {
-  return process.platform === "win32" ? "npx.cmd" : "npx";
-}
-
-function ensureOk(result, context) {
-  const output = `${result.stdout || ""}\n${result.stderr || ""}`.trim();
-  assert.strictEqual(result.status, 0, output || context);
-  return output;
-}
-
-function initGitRepo(repo) {
-  ensureOk(run("git", ["init"], repo), "git init failed");
-  ensureOk(run("git", ["config", "user.email", "skills-smoke@example.com"], repo), "git config email failed");
-  ensureOk(run("git", ["config", "user.name", "Skills Smoke"], repo), "git config name failed");
-  ensureOk(run("git", ["add", "."], repo), "git add failed");
-  ensureOk(run("git", ["commit", "-m", "skills smoke fixture"], repo), "git commit failed");
-}
-
-function buildIsolatedEnv(homeRoot) {
-  const env = {
-    HOME: homeRoot,
-    USERPROFILE: homeRoot,
-    APPDATA: path.join(homeRoot, "AppData", "Roaming"),
-    LOCALAPPDATA: path.join(homeRoot, "AppData", "Local"),
-    XDG_CONFIG_HOME: path.join(homeRoot, ".config"),
-  };
-  for (const value of Object.values(env)) {
-    fs.mkdirSync(value, { recursive: true });
-  }
-  return env;
-}
-
-function findInstalledSkill(rootDir, skillName) {
-  const matches = [];
-
-  function walk(dir) {
-    const entries = fs.readdirSync(dir, { withFileTypes: true });
-    for (const entry of entries) {
-      const fullPath = path.join(dir, entry.name);
-      if (entry.isDirectory()) {
-        walk(fullPath);
-        continue;
-      }
-      if (entry.isFile() && entry.name === "SKILL.md" && path.basename(path.dirname(fullPath)) === skillName) {
-        matches.push(path.dirname(fullPath));
-      }
-    }
-  }
-
-  walk(rootDir);
-  return matches;
-}
-
-function main() {
-  const tempRoot = fs.mkdtempSync(path.join(os.tmpdir(), "trackops-skills-smoke-"));
-  const sourceRepo = path.join(tempRoot, "source");
-  const sourceSkillDir = path.join(sourceRepo, "skills", "trackops");
-  const homeRoot = path.join(tempRoot, "home");
-
-  fs.mkdirSync(path.dirname(sourceSkillDir), { recursive: true });
-  fs.cpSync(SKILL_DIR, sourceSkillDir, { recursive: true });
-  initGitRepo(sourceRepo);
-
-  const env = buildIsolatedEnv(homeRoot);
-  const distribution = SKILL_CONFIG.distribution || {};
-  const skillName = distribution.skill || SKILL_CONFIG.name;
-
-  const listResult = run(
-    getNpxCommand(),
-    ["--yes", "skills", "add", sourceRepo, "--list", "--skill", skillName, "--full-depth", "-y"],
-    ROOT,
-    env,
-  );
-  const listOutput = ensureOk(listResult, "skills list failed");
-  assert.match(listOutput, /\btrackops\b/i, listOutput);
-
-  const installResult = run(
-    getNpxCommand(),
-    ["--yes", "skills", "add", sourceRepo, "--skill", skillName, "--full-depth", "--global", "--agent", "codex", "--copy", "-y"],
-    ROOT,
-    env,
-  );
-  ensureOk(installResult, "skills install failed");
-
-  const installed = findInstalledSkill(homeRoot, skillName);
-  assert.ok(installed.length >= 1, `trackops skill was not installed under ${homeRoot}`);
-
-  const installedSkillDir = installed[0];
-  assert.ok(fs.existsSync(path.join(installedSkillDir, "references", "activation.md")));
-  assert.ok(fs.existsSync(path.join(installedSkillDir, "skill.json")));
-  assert.ok(!fs.existsSync(path.join(installedSkillDir, "scripts", "bootstrap-trackops.js")));
-
-  fs.rmSync(tempRoot, { recursive: true, force: true });
-  console.log("skills marketplace smoke OK");
-}
-
-try {
-  main();
-} catch (error) {
-  console.error(error.message);
-  process.exit(1);
-}
+#!/usr/bin/env node
+
+const assert = require("assert");
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
+const { spawnSync } = require("child_process");
+
+const ROOT = path.resolve(__dirname, "..");
+const SKILLS_ROOT = path.join(ROOT, "skills");
+const REQUIRED_PUBLIC_SKILLS = ["trackops", "trackops-quality-guard"];
+
+const REQUIRED_FILES_BY_SKILL = {
+  trackops: [
+    "references/activation.md",
+    "references/workflow.md",
+    "references/troubleshooting.md",
+    "skill.json",
+  ],
+  "trackops-quality-guard": [
+    "references/commands.md",
+    "references/decision-policy.md",
+    "references/output-format.md",
+    "skill.json",
+  ],
+};
+
+function run(command, args, cwd, envOverrides = {}) {
+  const shell = process.platform === "win32" && /\.(cmd|bat)$/i.test(command);
+  return spawnSync(command, args, {
+    cwd,
+    encoding: "utf8",
+    env: { ...process.env, ...envOverrides },
+    shell,
+  });
+}
+
+function getNpxCommand() {
+  return process.platform === "win32" ? "npx.cmd" : "npx";
+}
+
+function ensureOk(result, context) {
+  const output = `${result.stdout || ""}\n${result.stderr || ""}`.trim();
+  assert.strictEqual(result.status, 0, output || context);
+  return output;
+}
+
+function initGitRepo(repo) {
+  ensureOk(run("git", ["init"], repo), "git init failed");
+  ensureOk(run("git", ["config", "user.email", "skills-smoke@example.com"], repo), "git config email failed");
+  ensureOk(run("git", ["config", "user.name", "Skills Smoke"], repo), "git config name failed");
+  ensureOk(run("git", ["add", "."], repo), "git add failed");
+  ensureOk(run("git", ["commit", "-m", "skills smoke fixture"], repo), "git commit failed");
+}
+
+function buildIsolatedEnv(homeRoot) {
+  const env = {
+    HOME: homeRoot,
+    USERPROFILE: homeRoot,
+    APPDATA: path.join(homeRoot, "AppData", "Roaming"),
+    LOCALAPPDATA: path.join(homeRoot, "AppData", "Local"),
+    XDG_CONFIG_HOME: path.join(homeRoot, ".config"),
+  };
+  for (const value of Object.values(env)) {
+    fs.mkdirSync(value, { recursive: true });
+  }
+  return env;
+}
+
+function findInstalledSkill(rootDir, skillName) {
+  const matches = [];
+
+  function walk(dir) {
+    const entries = fs.readdirSync(dir, { withFileTypes: true });
+    for (const entry of entries) {
+      const fullPath = path.join(dir, entry.name);
+      if (entry.isDirectory()) {
+        walk(fullPath);
+        continue;
+      }
+      if (entry.isFile() && entry.name === "SKILL.md" && path.basename(path.dirname(fullPath)) === skillName) {
+        matches.push(path.dirname(fullPath));
+      }
+    }
+  }
+
+  walk(rootDir);
+  return matches;
+}
+
+function discoverPublicSkills() {
+  return fs.readdirSync(SKILLS_ROOT, { withFileTypes: true })
+    .filter((entry) => entry.isDirectory() && fs.existsSync(path.join(SKILLS_ROOT, entry.name, "skill.json")))
+    .map((entry) => {
+      const dir = path.join(SKILLS_ROOT, entry.name);
+      const config = JSON.parse(fs.readFileSync(path.join(dir, "skill.json"), "utf8"));
+      return { name: entry.name, dir, config };
+    })
+    .sort((a, b) => a.name.localeCompare(b.name));
+}
+
+function main() {
+  const tempRoot = fs.mkdtempSync(path.join(os.tmpdir(), "trackops-skills-smoke-"));
+  const sourceRepo = path.join(tempRoot, "source");
+  const homeRoot = path.join(tempRoot, "home");
+
+  fs.mkdirSync(sourceRepo, { recursive: true });
+  fs.cpSync(SKILLS_ROOT, path.join(sourceRepo, "skills"), { recursive: true });
+  initGitRepo(sourceRepo);
+
+  const env = buildIsolatedEnv(homeRoot);
+  const skills = discoverPublicSkills();
+  for (const skillName of REQUIRED_PUBLIC_SKILLS) {
+    assert.ok(skills.some((entry) => entry.name === skillName), `missing public skill ${skillName}`);
+  }
+
+  for (const entry of skills) {
+    const distribution = entry.config.distribution || {};
+    const skillName = distribution.skill || entry.config.name;
+
+    const listResult = run(
+      getNpxCommand(),
+      ["--yes", "skills", "add", sourceRepo, "--list", "--skill", skillName, "--full-depth", "-y"],
+      ROOT,
+      env,
+    );
+    const listOutput = ensureOk(listResult, `skills list failed for ${skillName}`);
+    assert.ok(listOutput.includes(skillName), listOutput);
+
+    const installResult = run(
+      getNpxCommand(),
+      ["--yes", "skills", "add", sourceRepo, "--skill", skillName, "--full-depth", "--global", "--agent", "codex", "--copy", "-y"],
+      ROOT,
+      env,
+    );
+    ensureOk(installResult, `skills install failed for ${skillName}`);
+
+    const installed = findInstalledSkill(homeRoot, skillName);
+    assert.ok(installed.length >= 1, `${skillName} was not installed under ${homeRoot}`);
+
+    const installedSkillDir = installed[0];
+    for (const relativeFile of REQUIRED_FILES_BY_SKILL[skillName] || []) {
+      assert.ok(fs.existsSync(path.join(installedSkillDir, relativeFile)), `missing ${relativeFile} for ${skillName}`);
+    }
+  }
+
+  fs.rmSync(tempRoot, { recursive: true, force: true });
+  console.log("skills marketplace smoke OK");
+}
+
+try {
+  main();
+} catch (error) {
+  console.error(error.message);
+  process.exit(1);
+}