@punks/cli 1.0.7 → 2.0.0-beta.1

package/dist/data/catalog/skills.ts

@@ -12,44 +12,358 @@ export interface SkillCatalogEntry {
 }
 
 export const skillCatalog = [
-  { id: "agent-browser", sourceDirectory: "skills/agnostic/frontend/agent-browser", tier: "framework", language: null, framework: "browser-automation", requiresTools: ["agent-browser"] },
-  { id: "backend-domain-structure", sourceDirectory: "skills/agnostic/backend/backend-domain-structure", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "backend-recoverable-actions", sourceDirectory: "skills/agnostic/backend/backend-recoverable-actions", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "logging-best-practices", sourceDirectory: "skills/agnostic/backend/logging-best-practices", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "better-auth-best-practices", sourceDirectory: "skills/frameworks/better-auth/better-auth-best-practices", tier: "framework", language: "typescript", framework: "better-auth", requiresTools: [] },
-  { id: "create-plan", sourceDirectory: "skills/agnostic/planning/create-plan", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "create-spec", sourceDirectory: "skills/agnostic/planning/create-spec", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "debug-agent", sourceDirectory: "skills/agnostic/debug/debug-agent", tier: "agnostic", language: null, framework: null, requiresTools: ["debug-agent"] },
-  { id: "docs-maintenance", sourceDirectory: "skills/agnostic/docs/docs-maintenance", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "effect-authoring", sourceDirectory: "skills/frameworks/effect/effect-authoring", tier: "framework", language: "typescript", framework: "effect", requiresTools: ["opensrc"] },
-  { id: "effect-backend-structure", sourceDirectory: "skills/frameworks/effect/effect-backend-structure", tier: "framework", language: "typescript", framework: "effect", requiresTools: ["opensrc"] },
-  { id: "effect-best-practices", sourceDirectory: "skills/frameworks/effect/effect-best-practices", tier: "framework", language: "typescript", framework: "effect", requiresTools: [] },
-  { id: "effect-recoverable-actions", sourceDirectory: "skills/frameworks/effect/effect-recoverable-actions", tier: "framework", language: "typescript", framework: "effect", requiresTools: ["opensrc"] },
-  { id: "elysiajs", sourceDirectory: "skills/frameworks/elysia/elysiajs", tier: "framework", language: "typescript", framework: "elysiajs", requiresTools: [] },
-  { id: "design-taste-frontend", sourceDirectory: "skills/agnostic/frontend/design-taste-frontend", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "frontend-domain-structure", sourceDirectory: "skills/agnostic/frontend/frontend-domain-structure", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "grill-me", sourceDirectory: "skills/agnostic/planning/grill-me", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "improve-codebase-architecture", sourceDirectory: "skills/agnostic/research/improve-codebase-architecture", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "implement-spec", sourceDirectory: "skills/agnostic/planning/implement-spec", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "next-best-practices", sourceDirectory: "skills/frameworks/nextjs/next-best-practices", tier: "framework", language: "typescript", framework: "nextjs", requiresTools: [] },
-  { id: "next-cache-components", sourceDirectory: "skills/frameworks/nextjs/next-cache-components", tier: "framework", language: "typescript", framework: "nextjs", requiresTools: [] },
-  { id: "parallel-research", sourceDirectory: "skills/agnostic/research/parallel-research", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "async-python-patterns", sourceDirectory: "skills/languages/python/async-python-patterns", tier: "language", language: "python", framework: null, requiresTools: [] },
-  { id: "python-code-style", sourceDirectory: "skills/languages/python/python-code-style", tier: "language", language: "python", framework: null, requiresTools: [] },
-  { id: "python-design-patterns", sourceDirectory: "skills/languages/python/python-design-patterns", tier: "language", language: "python", framework: null, requiresTools: [] },
-  { id: "python-project-structure", sourceDirectory: "skills/languages/python/python-project-structure", tier: "language", language: "python", framework: null, requiresTools: [] },
-  { id: "python-testing-patterns", sourceDirectory: "skills/languages/python/python-testing-patterns", tier: "language", language: "python", framework: null, requiresTools: [] },
-  { id: "quality-types", sourceDirectory: "skills/languages/typescript/quality-types", tier: "language", language: "typescript", framework: null, requiresTools: [] },
-  { id: "async-react-patterns", sourceDirectory: "skills/frameworks/react/async-react-patterns", tier: "framework", language: "typescript", framework: "react", requiresTools: [] },
-  { id: "requirements-grill", sourceDirectory: "skills/agnostic/requirements/requirements-grill", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "simplify", sourceDirectory: "skills/agnostic/quality/simplify", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "swarm-planner", sourceDirectory: "skills/agnostic/subagents/swarm-planner", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "tanstack-query", sourceDirectory: "skills/frameworks/tanstack-query/tanstack-query", tier: "framework", language: "typescript", framework: "tanstack-query", requiresTools: ["opensrc"] },
-  { id: "tdd", sourceDirectory: "skills/agnostic/quality/tdd", tier: "agnostic", language: null, framework: null, requiresTools: [] },
-  { id: "turborepo", sourceDirectory: "skills/frameworks/turborepo/turborepo", tier: "framework", language: null, framework: "turborepo", requiresTools: [] },
-  { id: "vercel-composition-patterns", sourceDirectory: "skills/frameworks/react/vercel-composition-patterns", tier: "framework", language: "typescript", framework: "react", requiresTools: [] },
-  { id: "vercel-react-best-practices", sourceDirectory: "skills/frameworks/react/vercel-react-best-practices", tier: "framework", language: "typescript", framework: "react", requiresTools: [] },
-  { id: "write-backlog", sourceDirectory: "skills/agnostic/requirements/write-backlog", tier: "agnostic", language: null, framework: null, requiresTools: [] },
+  {
+    id: "agent-browser",
+    sourceDirectory: "skills/agnostic/frontend/agent-browser",
+    tier: "framework",
+    language: null,
+    framework: "browser-automation",
+    requiresTools: ["agent-browser"],
+  },
+  {
+    id: "backend-domain-structure",
+    sourceDirectory: "skills/agnostic/backend/backend-domain-structure",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "backend-recoverable-actions",
+    sourceDirectory: "skills/agnostic/backend/backend-recoverable-actions",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "logging-best-practices",
+    sourceDirectory: "skills/agnostic/backend/logging-best-practices",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "better-auth-best-practices",
+    sourceDirectory: "skills/frameworks/better-auth/better-auth-best-practices",
+    tier: "framework",
+    language: "typescript",
+    framework: "better-auth",
+    requiresTools: [],
+  },
+  {
+    id: "create-plan",
+    sourceDirectory: "skills/agnostic/planning/create-plan",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "create-spec",
+    sourceDirectory: "skills/agnostic/planning/create-spec",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "delivery-phase",
+    sourceDirectory: "skills/agnostic/planning/delivery-phase",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "debug-agent",
+    sourceDirectory: "skills/agnostic/debug/debug-agent",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: ["debug-agent"],
+  },
+  {
+    id: "debugging-phase",
+    sourceDirectory: "skills/agnostic/debug/debugging-phase",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: ["debug-agent"],
+  },
+  {
+    id: "docs-ingest-phase",
+    sourceDirectory: "skills/agnostic/docs/docs-ingest-phase",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "effect-authoring",
+    sourceDirectory: "skills/frameworks/effect/effect-authoring",
+    tier: "framework",
+    language: "typescript",
+    framework: "effect",
+    requiresTools: ["opensrc"],
+  },
+  {
+    id: "effect-backend-structure",
+    sourceDirectory: "skills/frameworks/effect/effect-backend-structure",
+    tier: "framework",
+    language: "typescript",
+    framework: "effect",
+    requiresTools: ["opensrc"],
+  },
+  {
+    id: "effect-best-practices",
+    sourceDirectory: "skills/frameworks/effect/effect-best-practices",
+    tier: "framework",
+    language: "typescript",
+    framework: "effect",
+    requiresTools: [],
+  },
+  {
+    id: "effect-recoverable-actions",
+    sourceDirectory: "skills/frameworks/effect/effect-recoverable-actions",
+    tier: "framework",
+    language: "typescript",
+    framework: "effect",
+    requiresTools: ["opensrc"],
+  },
+  {
+    id: "elysiajs",
+    sourceDirectory: "skills/frameworks/elysia/elysiajs",
+    tier: "framework",
+    language: "typescript",
+    framework: "elysiajs",
+    requiresTools: [],
+  },
+  {
+    id: "design-taste-frontend",
+    sourceDirectory: "skills/agnostic/frontend/design-taste-frontend",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "frontend-domain-structure",
+    sourceDirectory: "skills/agnostic/frontend/frontend-domain-structure",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "goalify",
+    sourceDirectory: "skills/agnostic/planning/goalify",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "grill-me",
+    sourceDirectory: "skills/agnostic/planning/grill-me",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "improve-codebase-architecture",
+    sourceDirectory: "skills/agnostic/research/improve-codebase-architecture",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "implement-spec",
+    sourceDirectory: "skills/agnostic/planning/implement-spec",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "next-best-practices",
+    sourceDirectory: "skills/frameworks/nextjs/next-best-practices",
+    tier: "framework",
+    language: "typescript",
+    framework: "nextjs",
+    requiresTools: [],
+  },
+  {
+    id: "next-cache-components",
+    sourceDirectory: "skills/frameworks/nextjs/next-cache-components",
+    tier: "framework",
+    language: "typescript",
+    framework: "nextjs",
+    requiresTools: [],
+  },
+  {
+    id: "parallel-research",
+    sourceDirectory: "skills/agnostic/research/parallel-research",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "async-python-patterns",
+    sourceDirectory: "skills/languages/python/async-python-patterns",
+    tier: "language",
+    language: "python",
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "python-code-style",
+    sourceDirectory: "skills/languages/python/python-code-style",
+    tier: "language",
+    language: "python",
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "python-design-patterns",
+    sourceDirectory: "skills/languages/python/python-design-patterns",
+    tier: "language",
+    language: "python",
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "python-project-structure",
+    sourceDirectory: "skills/languages/python/python-project-structure",
+    tier: "language",
+    language: "python",
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "python-testing-patterns",
+    sourceDirectory: "skills/languages/python/python-testing-patterns",
+    tier: "language",
+    language: "python",
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "quality-types",
+    sourceDirectory: "skills/languages/typescript/quality-types",
+    tier: "language",
+    language: "typescript",
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "async-react-patterns",
+    sourceDirectory: "skills/frameworks/react/async-react-patterns",
+    tier: "framework",
+    language: "typescript",
+    framework: "react",
+    requiresTools: [],
+  },
+  {
+    id: "requirements-grill",
+    sourceDirectory: "skills/agnostic/requirements/requirements-grill",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "requirements-phase",
+    sourceDirectory: "skills/agnostic/requirements/requirements-phase",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "resolve-debt-phase",
+    sourceDirectory: "skills/agnostic/planning/resolve-debt-phase",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "review-phase",
+    sourceDirectory: "skills/agnostic/research/review-phase",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "simplify",
+    sourceDirectory: "skills/agnostic/quality/simplify",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "swarm-planner",
+    sourceDirectory: "skills/agnostic/subagents/swarm-planner",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "tanstack-query",
+    sourceDirectory: "skills/frameworks/tanstack-query/tanstack-query",
+    tier: "framework",
+    language: "typescript",
+    framework: "tanstack-query",
+    requiresTools: ["opensrc"],
+  },
+  {
+    id: "tdd",
+    sourceDirectory: "skills/agnostic/quality/tdd",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
+  {
+    id: "turborepo",
+    sourceDirectory: "skills/frameworks/turborepo/turborepo",
+    tier: "framework",
+    language: null,
+    framework: "turborepo",
+    requiresTools: [],
+  },
+  {
+    id: "vercel-composition-patterns",
+    sourceDirectory: "skills/frameworks/react/vercel-composition-patterns",
+    tier: "framework",
+    language: "typescript",
+    framework: "react",
+    requiresTools: [],
+  },
+  {
+    id: "vercel-react-best-practices",
+    sourceDirectory: "skills/frameworks/react/vercel-react-best-practices",
+    tier: "framework",
+    language: "typescript",
+    framework: "react",
+    requiresTools: [],
+  },
+  {
+    id: "write-backlog",
+    sourceDirectory: "skills/agnostic/requirements/write-backlog",
+    tier: "agnostic",
+    language: null,
+    framework: null,
+    requiresTools: [],
+  },
 ] as const satisfies ReadonlyArray<SkillCatalogEntry>;
 
 export type SkillId = string;

package/dist/data/hooks/require-tests-for-pr.mjs

@@ -0,0 +1,206 @@
+#!/usr/bin/env node
+
+import { spawnSync } from "node:child_process";
+import { existsSync, readFileSync } from "node:fs";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+const denyMessage = "Tests are failing. Fix all test failures before creating a PR.";
+const prCommandPattern = /\bgh\s+pr\s+create\b/i;
+const prToolPattern = /(^|[._-])create[_-]?pull[_-]?request([._-]|$)/i;
+const packageManagers = new Set(["bun", "pnpm", "npm", "yarn"]);
+
+function readStdinJson() {
+  try {
+    return JSON.parse(readFileSync(0, "utf8"));
+  } catch {
+    return {};
+  }
+}
+
+function runGit(args, cwd) {
+  const result = spawnSync("git", ["-C", cwd, ...args], {
+    encoding: "utf8",
+    stdio: ["ignore", "pipe", "ignore"],
+  });
+
+  return result.status === 0 ? result.stdout : null;
+}
+
+function repoRoot(cwd) {
+  return runGit(["rev-parse", "--show-toplevel"], cwd)?.trim() ?? "";
+}
+
+function collectStrings(value, values = []) {
+  if (typeof value === "string") {
+    const trimmed = value.trim();
+    if (trimmed) {
+      values.push(trimmed);
+    }
+    return values;
+  }
+
+  if (Array.isArray(value)) {
+    for (const entry of value) {
+      collectStrings(entry, values);
+    }
+    return values;
+  }
+
+  if (!value || typeof value !== "object") {
+    return values;
+  }
+
+  for (const entry of Object.values(value)) {
+    collectStrings(entry, values);
+  }
+
+  return values;
+}
+
+function matchesPrAction(...values) {
+  const strings = [...new Set(values.flatMap((value) => collectStrings(value)))];
+
+  return strings.some((value) => prCommandPattern.test(value) || prToolPattern.test(value));
+}
+
+function readPackageJson(root) {
+  try {
+    return JSON.parse(readFileSync(path.join(root, "package.json"), "utf8"));
+  } catch {
+    return {};
+  }
+}
+
+function packageManagerFromPackageJson(root) {
+  const packageManager = readPackageJson(root)?.packageManager;
+
+  if (typeof packageManager !== "string") {
+    return null;
+  }
+
+  const name = packageManager.split("@")[0];
+  return packageManagers.has(name) ? name : null;
+}
+
+function packageManagerFromLockfile(root) {
+  if (existsSync(path.join(root, "bun.lock")) || existsSync(path.join(root, "bun.lockb"))) {
+    return "bun";
+  }
+
+  if (
+    existsSync(path.join(root, "pnpm-lock.yaml")) ||
+    existsSync(path.join(root, "pnpm-workspace.yaml"))
+  ) {
+    return "pnpm";
+  }
+
+  if (existsSync(path.join(root, "yarn.lock"))) {
+    return "yarn";
+  }
+
+  if (existsSync(path.join(root, "package-lock.json"))) {
+    return "npm";
+  }
+
+  return null;
+}
+
+function testCommand(root) {
+  switch (packageManagerFromPackageJson(root) ?? packageManagerFromLockfile(root) ?? "npm") {
+    case "bun":
+      return ["bun", "run", "test"];
+    case "pnpm":
+      return ["pnpm", "--silent", "test"];
+    case "yarn":
+      return ["yarn", "--silent", "test"];
+    case "npm":
+    default:
+      return ["npm", "--silent", "test"];
+  }
+}
+
+function runTests(root) {
+  const command = testCommand(root);
+  const result = spawnSync(command[0], command.slice(1), {
+    cwd: root,
+    stdio: "ignore",
+  });
+
+  return result.status === 0;
+}
+
+function emitCodexOrClaudeBlock(message) {
+  process.stdout.write(
+    JSON.stringify({
+      hookSpecificOutput: {
+        hookEventName: "PreToolUse",
+        permissionDecision: "deny",
+        permissionDecisionReason: message,
+      },
+      systemMessage: message,
+    }),
+  );
+}
+
+function emitCursorBlock(message) {
+  process.stdout.write(
+    JSON.stringify({
+      continue: false,
+      permission: "deny",
+      agent_message: message,
+      user_message: message,
+    }),
+  );
+}
+
+function runCommandHook(style) {
+  const payload = readStdinJson();
+  const root = repoRoot(String(payload.cwd ?? process.cwd()));
+
+  if (!root || !matchesPrAction(payload) || runTests(root)) {
+    return;
+  }
+
+  if (style === "cursor") {
+    emitCursorBlock(denyMessage);
+    return;
+  }
+
+  emitCodexOrClaudeBlock(denyMessage);
+}
+
+export const RequireTestsForPrPlugin = async ({ worktree }) => {
+  return {
+    "tool.execute.before": async (input, output) => {
+      if (!matchesPrAction(input?.tool, output?.args?.command)) {
+        return;
+      }
+
+      if (runTests(worktree)) {
+        return;
+      }
+
+      throw new Error(denyMessage);
+    },
+  };
+};
+
+export const testHooks = {
+  testCommand,
+};
+
+function main() {
+  const mode = process.argv[2] ?? "";
+
+  if (mode === "claude" || mode === "codex" || mode === "cursor") {
+    runCommandHook(mode);
+  }
+}
+
+const invokedPath = process.argv[1] ? path.resolve(process.argv[1]) : "";
+const modulePath = fileURLToPath(import.meta.url);
+
+if (invokedPath && path.basename(invokedPath) === path.basename(modulePath)) {
+  main();
+}

package/dist/data/hooks.test.ts

@@ -31,6 +31,35 @@ const tempRepo = (files: Record<string, string>) => {
 };
 
 describe("scaffolded hooks", () => {
+  it("runs the PR test gate with the repo package manager", async () => {
+    const { testHooks } = await loadHookModule("require-tests-for-pr.mjs");
+
+    expect(
+      testHooks.testCommand?.(
+        tempRepo({
+          "package.json": JSON.stringify({ packageManager: "bun@1.3.5" }),
+        }),
+      ),
+    ).toEqual(["bun", "run", "test"]);
+
+    expect(
+      testHooks.testCommand?.(
+        tempRepo({
+          "package.json": JSON.stringify({ packageManager: "pnpm@10.0.0" }),
+        }),
+      ),
+    ).toEqual(["pnpm", "--silent", "test"]);
+
+    expect(
+      testHooks.testCommand?.(
+        tempRepo({
+          "package.json": "{}",
+          "package-lock.json": "",
+        }),
+      ),
+    ).toEqual(["npm", "--silent", "test"]);
+  });
+
   it("executes formatter tools with the repo package manager", async () => {
     const { testHooks } = await loadHookModule("format-edited-file.mjs");