@witchpot/gmc 0.8.0 → 0.10.0

package/scripts/check-release-readiness.mjs

@@ -1,177 +0,0 @@
-#!/usr/bin/env node
-
-import { spawnSync } from "node:child_process";
-import { existsSync, readFileSync } from "node:fs";
-import path from "node:path";
-
-const packageDir = path.resolve(new URL("..", import.meta.url).pathname);
-const packageJsonPath = path.join(packageDir, "package.json");
-const packageJson = JSON.parse(readFileSync(packageJsonPath, "utf8"));
-
-const requiredFiles = [
-  "bin/gmc.mjs",
-  "src/index.js",
-  "src/main.js",
-  "src/requests.js",
-  "src/workflows.js",
-  "skill/gmc-analysis/SKILL.md",
-  "skill/gmc-analysis/references/recipes.md",
-  "skill/gmc-analysis/references/pitfalls.md",
-  "skill/gmc-analysis/references/validity.md",
-  "skill/gmc-analysis/references/quota.md",
-  "skill/gmc-analysis/references/charts.md",
-  "skill/gmc-analysis/CHANGELOG.md",
-  "README.md",
-  "CHANGELOG.md"
-];
-
-const requiredFileGlobs = ["bin", "src", "scripts", "skill", "README.md", "CHANGELOG.md"];
-const forbiddenPatterns = [
-  { pattern: /@supabase\//, message: "CLI package must not import Supabase SDKs." },
-  { pattern: /@clerk\//, message: "CLI package must not import Clerk SDKs." },
-  { pattern: /src\/server|server\/gmc|server-only/, message: "CLI package must not import server-only app code." },
-  { pattern: /SUPABASE_[A-Z0-9_]*|CLERK_[A-Z0-9_]*|STRIPE_[A-Z0-9_]*/, message: "CLI package must not reference server secret environment variables." }
-];
-
-const sourceFiles = [
-  "bin/gmc.mjs",
-  "src/index.js",
-  "src/args.js",
-  "src/browser-login.js",
-  "src/config.js",
-  "src/constants.js",
-  "src/help.js",
-  "src/main.js",
-  "src/output.js",
-  "src/projection.js",
-  "src/requests.js",
-  "src/workflows.js"
-];
-
-const failures = [];
-
-if (packageJson.private === true) {
-  failures.push("packages/gmc-cli/package.json must not have private:true for beta publishing.");
-}
-
-if (packageJson.name !== "@witchpot/gmc") {
-  failures.push(`Unexpected package name: ${packageJson.name}`);
-}
-
-if (packageJson.publishConfig?.access !== "public") {
-  failures.push("publishConfig.access must be public for npm beta-tag publishing.");
-}
-
-for (const binName of ["gmc", "gmcp", "game-market-copilot"]) {
-  if (packageJson.bin?.[binName] !== "bin/gmc.mjs") {
-    failures.push(`bin.${binName} must point to bin/gmc.mjs.`);
-  }
-}
-
-if (packageJson.main !== "./src/index.js") {
-  failures.push("main must point to ./src/index.js.");
-}
-
-if (packageJson.exports?.["."] !== "./src/index.js") {
-  failures.push("exports['.'] must point to ./src/index.js.");
-}
-
-for (const required of requiredFileGlobs) {
-  if (!packageJson.files?.includes(required)) {
-    failures.push(`package.json files must include ${required}.`);
-  }
-}
-
-for (const file of requiredFiles) {
-  if (!existsSync(path.join(packageDir, file))) {
-    failures.push(`Missing required package file: ${file}`);
-  }
-}
-
-const constantsPath = path.join(packageDir, "src/constants.js");
-const constantsSource = readFileSync(constantsPath, "utf8");
-const versionMatch = constantsSource.match(/CLI_VERSION\s*=\s*"([^"]+)"/);
-if (!versionMatch) {
-  failures.push("src/constants.js must export a string CLI_VERSION.");
-} else if (versionMatch[1] !== packageJson.version) {
-  failures.push(`CLI_VERSION (${versionMatch[1]}) must match package version (${packageJson.version}).`);
-}
-
-const changelog = readFileSync(path.join(packageDir, "CHANGELOG.md"), "utf8");
-if (!changelog.includes(`## ${packageJson.version} -`)) {
-  failures.push(`CHANGELOG.md must include a section for ${packageJson.version}.`);
-}
-
-const skillSource = readFileSync(path.join(packageDir, "skill/gmc-analysis/SKILL.md"), "utf8");
-const skillVersionMatch = skillSource.match(/^\s*version:\s*"?([0-9][0-9A-Za-z.\-]*)"?\s*$/m);
-if (!skillVersionMatch) {
-  failures.push("skill/gmc-analysis/SKILL.md must declare metadata.version in its frontmatter.");
-} else {
-  const skillChangelog = readFileSync(path.join(packageDir, "skill/gmc-analysis/CHANGELOG.md"), "utf8");
-  if (!skillChangelog.includes(`## ${skillVersionMatch[1]} -`)) {
-    failures.push(`skill CHANGELOG.md must include a section for skill version ${skillVersionMatch[1]}.`);
-  }
-}
-
-for (const file of sourceFiles) {
-  const source = readFileSync(path.join(packageDir, file), "utf8");
-  for (const forbidden of forbiddenPatterns) {
-    if (forbidden.pattern.test(source)) {
-      failures.push(`${file}: ${forbidden.message}`);
-    }
-  }
-}
-
-// The skill content-hash manifest (NAK-190) is also enforced by
-// src/skill-manifest.test.js, but release:check/prepublishOnly run WITHOUT
-// the test suite — recheck here so a stale manifest can never reach npm.
-const { buildSkillManifest, MANIFEST_PATH } = await import("./gen-skill-manifest.mjs");
-if (!existsSync(MANIFEST_PATH)) {
-  failures.push("skill/gmc-analysis.manifest.json is missing — run npm run skill:manifest.");
-} else {
-  const committedManifest = JSON.parse(readFileSync(MANIFEST_PATH, "utf8"));
-  const currentManifest = buildSkillManifest();
-  if (
-    committedManifest.contentHash !== currentManifest.contentHash ||
-    committedManifest.version !== currentManifest.version
-  ) {
-    failures.push(
-      "skill/gmc-analysis.manifest.json is stale: bump metadata.version in SKILL.md, run npm run skill:manifest, and re-sync the gmc-agent-skills distribution repo."
-    );
-  }
-}
-
-// The public distribution repo (github.com/witchpot-studio/gmc-agent-skills)
-// mirrors skill/gmc-analysis. When it is checked out as a workspace sibling,
-// a drift between the two is a release blocker — MCP-only users install the
-// skill from that repo and must get the same content the npm package ships.
-const explicitAgentSkillsDir = process.env.GMC_AGENT_SKILLS_DIR;
-const agentSkillsRepo =
-  explicitAgentSkillsDir ?? path.join(packageDir, "..", "..", "..", "gmc-agent-skills");
-const agentSkillsSyncScript = path.join(agentSkillsRepo, "scripts", "sync-from-cli.mjs");
-if (explicitAgentSkillsDir && !existsSync(agentSkillsSyncScript)) {
-  failures.push(
-    `GMC_AGENT_SKILLS_DIR is set but ${agentSkillsSyncScript} does not exist — fix the path or unset the variable.`
-  );
-} else if (existsSync(agentSkillsSyncScript)) {
-  const drift = spawnSync(process.execPath, ["scripts/sync-from-cli.mjs", "--check"], {
-    cwd: agentSkillsRepo,
-    encoding: "utf8",
-    // Pin the canonical source to THIS checkout — the sync script's own
-    // default (../GMC relative to the skills repo) could silently compare
-    // against a different GMC checkout.
-    env: { ...process.env, GMC_REPO_DIR: path.join(packageDir, "..", "..") }
-  });
-  if (drift.status !== 0) {
-    failures.push(
-      `gmc-agent-skills repo has drifted from skill/gmc-analysis — run node scripts/sync-from-cli.mjs there.\n${(drift.stderr || "").trim()}`
-    );
-  }
-}
-
-if (failures.length > 0) {
-  process.stderr.write(`GMC CLI release readiness failed:\n${failures.map((failure) => `- ${failure}`).join("\n")}\n`);
-  process.exit(1);
-}
-
-process.stdout.write(`GMC CLI release readiness ok: ${packageJson.name}@${packageJson.version}\n`);

package/scripts/gen-skill-manifest.mjs

@@ -1,151 +0,0 @@
-#!/usr/bin/env node
-
-// Regenerates skill/gmc-analysis.manifest.json — a content-hash fingerprint
-// of the packaged gmc-analysis skill (SKILL.md + references/ + CHANGELOG.md).
-//
-// Maintainer rule: any time skill/gmc-analysis/** content changes, bump
-// metadata.version in SKILL.md, then run `npm run skill:manifest` (in
-// packages/gmc-cli) to regenerate this manifest, and re-sync the
-// gmc-agent-skills distribution repo (node scripts/sync-from-cli.mjs there).
-// src/skill-manifest.test.js recomputes the same hash on every
-// `npm run test:cli` / `npm run verify` and fails loudly if it no longer
-// matches the committed manifest — this is the drift guard that would have
-// caught the unversioned charts.md changes in PR #18 / PR #43 (NAK-190).
-//
-// The manifest is written OUTSIDE skill/gmc-analysis/ on purpose: that
-// directory is mirrored byte-for-byte into the public gmc-agent-skills
-// distribution repo, and the manifest must never be part of (or affect the
-// hash of) the content it describes.
-
-import { spawnSync } from "node:child_process";
-import { createHash } from "node:crypto";
-import { existsSync, readdirSync, readFileSync, writeFileSync } from "node:fs";
-import path from "node:path";
-import { fileURLToPath } from "node:url";
-
-const packageDir = path.dirname(path.dirname(fileURLToPath(import.meta.url)));
-export const SKILL_DIR = path.join(packageDir, "skill", "gmc-analysis");
-export const MANIFEST_PATH = path.join(packageDir, "skill", "gmc-analysis.manifest.json");
-
-/** List every file under `dir`, as sorted relative POSIX paths. */
-export function listSkillFiles(dir = SKILL_DIR) {
-  const results = [];
-  walk(dir, dir, results);
-  return results.sort();
-}
-
-function walk(root, current, results) {
-  const entries = readdirSync(current, { withFileTypes: true }).sort((a, b) => a.name.localeCompare(b.name));
-  for (const entry of entries) {
-    const abs = path.join(current, entry.name);
-    if (entry.isDirectory()) {
-      walk(root, abs, results);
-    } else if (entry.isFile()) {
-      results.push(path.relative(root, abs).split(path.sep).join("/"));
-    }
-  }
-}
-
-function sha256Hex(buffer) {
-  return createHash("sha256").update(buffer).digest("hex");
-}
-
-/**
- * Hash every file under `dir` and fold the per-file hashes into one
- * top-level sha256 over `relpath\nfilehash\n` lines (sorted, deterministic).
- */
-export function computeSkillContentHash(dir = SKILL_DIR) {
-  const files = listSkillFiles(dir);
-  const digest = createHash("sha256");
-  for (const relPath of files) {
-    const fileHash = sha256Hex(readFileSync(path.join(dir, relPath)));
-    digest.update(`${relPath}\n${fileHash}\n`);
-  }
-  return { contentHash: digest.digest("hex"), files: files.length };
-}
-
-/** Parse metadata.version out of SKILL.md source text; undefined when absent. */
-export function parseSkillVersion(source) {
-  const frontmatter = source.match(/^---\n([\s\S]*?)\n---/);
-  return frontmatter?.[1].match(/^\s*version:\s*"?([0-9][0-9A-Za-z.\-]*)"?\s*$/m)?.[1];
-}
-
-/** Read metadata.version from SKILL.md's YAML frontmatter. */
-export function readSkillManifestVersion(dir = SKILL_DIR) {
-  const version = parseSkillVersion(readFileSync(path.join(dir, "SKILL.md"), "utf8"));
-  if (!version) {
-    throw new Error(`${path.join(dir, "SKILL.md")} must declare metadata.version in its frontmatter.`);
-  }
-  return version;
-}
-
-/** Build the manifest object (version + contentHash + files) for `dir`. */
-export function buildSkillManifest(dir = SKILL_DIR) {
-  const { contentHash, files } = computeSkillContentHash(dir);
-  return { version: readSkillManifestVersion(dir), contentHash, files };
-}
-
-/**
- * Baseline manifest for the version-bump rule. The authoritative baseline is
- * the manifest COMMITTED ON origin/main (read via `git show`, so deleting or
- * hand-editing the local file cannot dodge the rule); the local file is only
- * a fallback for environments where the git baseline is unreadable.
- */
-function baselineSkillManifest() {
-  const result = spawnSync("git", ["show", "origin/main:packages/gmc-cli/skill/gmc-analysis.manifest.json"], {
-    cwd: packageDir,
-    encoding: "utf8"
-  });
-  if (result.status === 0 && typeof result.stdout === "string") {
-    try {
-      return JSON.parse(result.stdout);
-    } catch {
-      // fall through to the local file
-    }
-  }
-  if (existsSync(MANIFEST_PATH)) {
-    try {
-      return JSON.parse(readFileSync(MANIFEST_PATH, "utf8"));
-    } catch {
-      return undefined;
-    }
-  }
-  return undefined;
-}
-
-/**
- * Pure regen-permission rule (unit-tested in src/skill-manifest.test.js).
- * Content that differs from the baseline while keeping the baseline's
- * version is refused: once a version's content is on origin/main, changing
- * content under that version always requires a bump. A differing version
- * (the in-flight bump case, iterated freely within one branch) and a
- * missing baseline (true first generation) are allowed. There is
- * deliberately NO force escape hatch: an override flag would let
- * unversioned content changes reach npm and the public skills repo.
- */
-export function assertRegenAllowed({ baseline, next }) {
-  if (!baseline) return;
-  if (baseline.contentHash === next.contentHash) return;
-  if (baseline.version !== next.version) return;
-  throw new Error(
-    `skill content changed but metadata.version is still ${next.version}, whose content is already fixed by the baseline manifest (origin/main) — bump the version in SKILL.md, then rerun npm run skill:manifest.`
-  );
-}
-
-function main() {
-  const manifest = buildSkillManifest();
-  try {
-    assertRegenAllowed({ baseline: baselineSkillManifest(), next: manifest });
-  } catch (error) {
-    console.error(error.message);
-    process.exit(1);
-  }
-  writeFileSync(MANIFEST_PATH, `${JSON.stringify(manifest, null, 2)}\n`);
-  process.stdout.write(
-    `Wrote ${path.relative(packageDir, MANIFEST_PATH)}: version ${manifest.version}, ${manifest.files} files, contentHash ${manifest.contentHash}\n`
-  );
-}
-
-if (process.argv[1] && path.resolve(process.argv[1]) === fileURLToPath(import.meta.url)) {
-  main();
-}

package/scripts/smoke-beta.mjs

@@ -1,246 +0,0 @@
-#!/usr/bin/env node
-
-import { spawnSync } from "node:child_process";
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import path from "node:path";
-
-const packageDir = path.resolve(new URL("..", import.meta.url).pathname);
-const cliBin = path.join(packageDir, "bin", "gmc.mjs");
-const workDir = mkdtempSync(path.join(tmpdir(), "gmc-cli-beta-smoke-"));
-const configPath = path.join(workDir, "config.json");
-const allowApiErrors = process.argv.includes("--allow-api-errors");
-
-const smokeCommands = [
-  {
-    name: "version",
-    args: ["--version", "--json"],
-    validate: (payload) => requireFields(payload, ["data.name", "data.version"])
-  },
-  {
-    name: "update",
-    args: ["update", "--json"],
-    validate: (payload) => requireFields(payload, ["data.currentVersion", "data.commands.npmGlobal"])
-  },
-  {
-    name: "auth status",
-    args: ["auth", "status", "--json"],
-    validate: (payload) => requireSuccessEnvelope(payload, ["data.caller", "data.cli"])
-  },
-  {
-    name: "schema",
-    args: ["schema", "--json"],
-    validate: (payload) => requireSuccessEnvelope(payload, ["data.version", "data.cliCommands"])
-  },
-  {
-    name: "search recent reviewed games",
-    args: [
-      "games",
-      "search",
-      "--source",
-      "steam",
-      "--release-year",
-      "2025",
-      "--reviews-min",
-      "1000",
-      "--limit",
-      "5",
-      "--json"
-    ],
-    validate: (payload) => requireSuccessEnvelope(payload, ["data"])
-  },
-  {
-    name: "aggregate recent reviewed games",
-    args: [
-      "games",
-      "aggregate",
-      "--source",
-      "steam",
-      "--release-year",
-      "2025",
-      "--metric",
-      "count,median_reviews",
-      "--group-by",
-      "release_year",
-      "--reviews-min",
-      "1000",
-      "--json"
-    ],
-    validate: (payload) => requireSuccessEnvelope(payload, ["data"])
-  },
-  {
-    name: "benchmark city builder automation",
-    args: [
-      "games",
-      "benchmark",
-      "--source",
-      "steam",
-      "--tags",
-      "city_builder,automation",
-      "--reviews-min",
-      "1000",
-      "--preset",
-      "summary",
-      "--json"
-    ],
-    validate: (payload) => requireComposedEnvelope(payload, ["data.summary"])
-  },
-  {
-    name: "showcase patterns by tag",
-    args: ["showcases", "patterns", "--source", "steam", "--tags", "city_builder,automation", "--json"],
-    validate: (payload) => requireComposedEnvelope(payload, ["data.summary"])
-  }
-];
-
-try {
-  const results = smokeCommands.map(runSmokeCommand);
-  const failed = results.filter((result) => result.status === "failed");
-  const summary = {
-    ok: failed.length === 0,
-    apiBaseUrl: process.env.GMC_API_BASE_URL || "https://gamemarketcopilot.com",
-    auth: {
-      apiKeyConfigured: Boolean(process.env.GMC_API_KEY)
-    },
-    allowApiErrors,
-    commands: results
-  };
-
-  process.stdout.write(`${JSON.stringify(summary, null, 2)}\n`);
-  if (failed.length > 0) process.exit(1);
-} finally {
-  rmSync(workDir, { recursive: true, force: true });
-}
-
-function runSmokeCommand(command) {
-  const result = spawnSync(process.execPath, [cliBin, ...command.args], {
-    encoding: "utf8",
-    env: {
-      ...process.env,
-      GMC_CONFIG_PATH: configPath
-    }
-  });
-
-  const payload = parseJson(result.stdout);
-  if (!payload.ok) {
-    return {
-      name: command.name,
-      status: "failed",
-      exitCode: result.status,
-      error: `stdout was not valid JSON: ${payload.error}`
-    };
-  }
-
-  if (payload.value?.error) {
-    const stable = isStableError(payload.value.error);
-    const status = allowApiErrors && stable ? "api_error_allowed" : "failed";
-    return {
-      name: command.name,
-      status,
-      exitCode: result.status,
-      error: summarizeError(payload.value.error)
-    };
-  }
-
-  if (result.status !== 0) {
-    return {
-      name: command.name,
-      status: "failed",
-      exitCode: result.status,
-      error: "command exited non-zero without a structured error envelope"
-    };
-  }
-
-  const validation = command.validate(payload.value);
-  if (!validation.ok) {
-    return {
-      name: command.name,
-      status: "failed",
-      exitCode: result.status,
-      error: validation.error
-    };
-  }
-
-  return {
-    name: command.name,
-    status: "passed",
-    exitCode: result.status,
-    requestId: payload.value.meta?.requestId,
-    upstreamRequestIds: Array.isArray(payload.value.meta?.upstreamRequests)
-      ? payload.value.meta.upstreamRequests.map((request) => request?.requestId).filter(Boolean)
-      : undefined,
-    warnings: Array.isArray(payload.value.warnings) ? payload.value.warnings.length : 0,
-    hasPagination: Boolean(payload.value.pagination),
-    hasEntitlements: Boolean(payload.value.meta?.entitlements),
-    hasUpstreamEntitlements: Array.isArray(payload.value.meta?.upstreamRequests)
-      ? payload.value.meta.upstreamRequests.some((request) => request?.entitlements)
-      : undefined
-  };
-}
-
-function parseJson(stdout) {
-  try {
-    return {
-      ok: true,
-      value: JSON.parse(stdout)
-    };
-  } catch (error) {
-    return {
-      ok: false,
-      error: error instanceof Error ? error.message : "unknown parse error"
-    };
-  }
-}
-
-function requireSuccessEnvelope(payload, fields) {
-  const base = requireFields(payload, ["data", "meta.requestId", ...fields]);
-  if (!base.ok) return base;
-  return {
-    ok: true
-  };
-}
-
-function requireComposedEnvelope(payload, fields) {
-  const base = requireFields(payload, ["data", "meta.requestId", "meta.upstreamRequests", ...fields]);
-  if (!base.ok) return base;
-  if (!Array.isArray(payload.meta.upstreamRequests) || payload.meta.upstreamRequests.length === 0) {
-    return {
-      ok: false,
-      error: "meta.upstreamRequests must include at least one request"
-    };
-  }
-  return {
-    ok: true
-  };
-}
-
-function requireFields(payload, fields) {
-  const missing = fields.filter((field) => getPath(payload, field) == null);
-  if (missing.length > 0) {
-    return {
-      ok: false,
-      error: `missing fields: ${missing.join(", ")}`
-    };
-  }
-  return {
-    ok: true
-  };
-}
-
-function getPath(value, pathName) {
-  return pathName.split(".").reduce((current, part) => current?.[part], value);
-}
-
-function isStableError(error) {
-  return Boolean(error && typeof error.code === "string" && typeof error.message === "string");
-}
-
-function summarizeError(error) {
-  return {
-    code: error.code,
-    message: error.message,
-    requestId: error.requestId,
-    status: error.status,
-    contentType: error.contentType,
-    path: error.path
-  };
-}

package/scripts/smoke-pack.mjs

@@ -1,123 +0,0 @@
-#!/usr/bin/env node
-
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import path from "node:path";
-import { spawnSync } from "node:child_process";
-
-const packageDir = path.resolve(new URL("..", import.meta.url).pathname);
-const dryRunOnly = process.argv.includes("--dry-run");
-const workDir = mkdtempSync(path.join(tmpdir(), "gmc-cli-pack-"));
-const npmCache = path.join(workDir, "npm-cache");
-
-try {
-  run("npm", ["pack", "--dry-run"], {
-    cwd: packageDir,
-    env: {
-      npm_config_cache: npmCache
-    }
-  });
-
-  if (dryRunOnly) process.exit(0);
-
-  const packResult = run("npm", ["pack", "--json", "--pack-destination", workDir], {
-    cwd: packageDir,
-    env: {
-      npm_config_cache: npmCache
-    },
-    capture: true
-  });
-  const packed = JSON.parse(packResult.stdout);
-  const filename = packed?.[0]?.filename;
-  if (!filename) throw new Error("npm pack did not return a tarball filename");
-
-  const projectDir = path.join(workDir, "project");
-  run("npm", ["init", "-y"], {
-    cwd: workDir,
-    env: {
-      npm_config_cache: npmCache
-    }
-  });
-  run("npm", ["install", path.join(workDir, filename)], {
-    cwd: workDir,
-    env: {
-      npm_config_cache: npmCache
-    }
-  });
-
-  const binName = process.platform === "win32" ? "gmc.cmd" : "gmc";
-  const gmcBin = path.join(workDir, "node_modules", ".bin", binName);
-  const configPath = path.join(projectDir, "config.json");
-  const helpResult = run(gmcBin, ["--help"], {
-    env: { GMC_CONFIG_PATH: configPath },
-    capture: true
-  });
-  if (!helpResult.stdout.includes("Game Market Copilot CLI") || !helpResult.stdout.includes("gmc --version")) {
-    throw new Error("Packed gmc --help did not print the expected help text");
-  }
-
-  const configResult = run(gmcBin, ["config", "get", "--json"], {
-    env: { GMC_CONFIG_PATH: configPath },
-    capture: true
-  });
-  const configPayload = JSON.parse(configResult.stdout);
-  if (configPayload?.data?.configPath !== configPath || configPayload?.data?.authMode !== "anonymous") {
-    throw new Error("Packed gmc config get --json did not print the expected config payload");
-  }
-
-  const skillStatusResult = run(gmcBin, ["skill", "status", "--json"], {
-    env: { GMC_CONFIG_PATH: configPath },
-    capture: true
-  });
-  const skillStatus = JSON.parse(skillStatusResult.stdout);
-  if (!skillStatus?.data?.packaged?.version) {
-    throw new Error("Packed gmc skill status did not report a packaged skill version");
-  }
-
-  const skillPrintResult = run(gmcBin, ["skill", "print", "--format", "agents-md"], {
-    env: { GMC_CONFIG_PATH: configPath },
-    capture: true
-  });
-  if (!skillPrintResult.stdout.includes("GMC Analysis") || !skillPrintResult.stdout.includes("Pitfalls")) {
-    throw new Error("Packed gmc skill print did not emit the inlined agents-md section");
-  }
-  if (!skillPrintResult.stdout.includes("Chart cards") || !skillPrintResult.stdout.includes("gmc-lockup.svg")) {
-    throw new Error("Packed gmc skill print did not inline the chart cards reference");
-  }
-
-  // gmc is shadowed by Oh My Zsh's `git merge --continue` alias, so the
-  // package ships alias binaries; make sure npm actually installed them.
-  for (const aliasName of ["gmcp", "game-market-copilot"]) {
-    const aliasBinName = process.platform === "win32" ? `${aliasName}.cmd` : aliasName;
-    const aliasBin = path.join(workDir, "node_modules", ".bin", aliasBinName);
-    const aliasResult = run(aliasBin, ["--version"], {
-      env: { GMC_CONFIG_PATH: configPath },
-      capture: true
-    });
-    if (!/^gmc \d+\.\d+\.\d+/.test(aliasResult.stdout)) {
-      throw new Error(`Packed ${aliasName} --version did not print the CLI version`);
-    }
-  }
-} finally {
-  rmSync(workDir, { recursive: true, force: true });
-}
-
-function run(command, args, options = {}) {
-  const result = spawnSync(command, args, {
-    cwd: options.cwd,
-    encoding: "utf8",
-    env: {
-      ...process.env,
-      npm_config_dry_run: "false",
-      ...(options.env ?? {})
-    },
-    stdio: options.capture ? ["ignore", "pipe", "pipe"] : "inherit"
-  });
-
-  if (result.status !== 0) {
-    const stderr = result.stderr ? `\n${result.stderr}` : "";
-    throw new Error(`${command} ${args.join(" ")} failed with exit code ${result.status}${stderr}`);
-  }
-
-  return result;
-}