@orchestrator-claude/definitions 3.31.1 → 3.32.0

package/skills/backlog-synthesis/SKILL.md

@@ -18,7 +18,7 @@ cat .claude/agent-memory/orchestrator/MEMORY.md 2>/dev/null || echo "(empty)"
 ```
 
 ### Step 2: Query Active Workflows
-Use MCP tool getWorkflows or getStatus to check active workflow state.
+Use MCP tool listWorkflowDefinitions or getStatus to check active workflow state.
 
 ### Step 3: Cross-Reference and Synthesize
 Look for:

package/skills/workflow-status/SKILL.md

@@ -14,7 +14,7 @@ All workflow state is in **PostgreSQL** via MCP tools. NEVER read `.orchestrator
 | Tool | What it returns |
 |------|----------------|
 | `getStatus` | Current workflow: phase, status, pending action |
-| `getWorkflows` | All workflows with phase configs |
+| `listWorkflowDefinitions` | All workflow type definitions (phases, modes) |
 | `getNextAction` | Next pending action (agent + status) |
 | `getInvocations` | Agent invocation history |
 | `getMetrics` | Workflow metrics (duration, counts) |

package/hooks/pre-agent-dispatch.ts

@@ -1,57 +0,0 @@
-/**
- * pre-agent-dispatch.ts — PreToolUse hook for Agent tool (RFC-027 Sprint 4)
- *
- * Warns when dispatching known agents without subagent_type.
- * Non-blocking: always exits 0.
- */
-
-import { log, readStdin, getField, outputContext } from "./lib/api-client.js";
-
-const HOOK = "PRE-AGENT-DISPATCH";
-
-const KNOWN_AGENTS = [
-  "implementer",
-  "planner",
-  "specifier",
-  "reviewer",
-  "task-generator",
-  "researcher",
-];
-
-const KNOWN_AGENTS_REGEX = new RegExp(
-  `\\b(${KNOWN_AGENTS.join("|")})\\b`,
-  "i"
-);
-
-async function main(): Promise<void> {
-  const stdin = await readStdin();
-  const toolInput = getField(stdin, "tool_input") || "";
-  const subagentType =
-    getField(stdin, "tool_input.subagent_type") ||
-    getField(stdin, "tool_input.subagentType") ||
-    "";
-
-  // If subagent_type is present, no warning needed
-  if (subagentType) {
-    log(HOOK, `Agent dispatched with subagent_type="${subagentType}", OK`);
-    return;
-  }
-
-  // Check if prompt mentions a known agent
-  const prompt =
-    getField(stdin, "tool_input.prompt") ||
-    getField(stdin, "tool_input.description") ||
-    (typeof toolInput === "string" ? toolInput : JSON.stringify(toolInput));
-
-  const match = prompt.match(KNOWN_AGENTS_REGEX);
-  if (match) {
-    const detected = match[1].toLowerCase();
-    outputContext(
-      HOOK,
-      `[WARN] Agent dispatched without subagent_type. Known agent detected: "${detected}". Add subagent_type for proper frontmatter loading.`
-    );
-    log(HOOK, `WARN: known agent "${detected}" without subagent_type`);
-  }
-}
-
-main().catch(() => process.exit(0));

package/hooks/verification-before-completion.ts

@@ -1,97 +0,0 @@
-/**
- * verification-before-completion.ts — PreToolUse[Bash] hook
- *
- * Injects a verification reminder when the model is about to run
- * `git commit` or `git push`. Does NOT block — only adds context.
- *
- * Principle: Evidence before claims, always.
- * Source: verification-before-completion skill (superpowers 5.0.7)
- *
- * Flow:
- *   1. readStdin() → extract bash command
- *   2. commandMatchesGitOp() → check for commit/push
- *   3. If match → output allow + context reminder
- *   4. If no match → silent pass-through (no output)
- */
-
-import { log, readStdin, getField, outputPreToolUse } from "./lib/api-client.js";
-
-const HOOK = "VERIFICATION-REMINDER";
-
-// ─────────────────────────────────────────────────────────────
-// Dependency injection interface (enables testing without IO)
-// ─────────────────────────────────────────────────────────────
-
-export interface HookDeps {
-  readStdin: () => Promise<Record<string, unknown>>;
-  output: (opts: { decision: "allow" | "deny"; reason?: string; context?: string }) => void;
-  log: (prefix: string, msg: string) => void;
-}
-
-const defaultDeps: HookDeps = {
-  readStdin,
-  output: outputPreToolUse,
-  log,
-};
-
-// ─────────────────────────────────────────────────────────────
-// Pure helpers (exported for testing)
-// ─────────────────────────────────────────────────────────────
-
-/**
- * Returns true when the bash command string contains a git commit or git push
- * operation. Handles common variations: `git commit`, `git push`, multi-command
- * pipelines (`&&`, `;`), and quoted forms.
- */
-export function commandMatchesGitOp(command: string): boolean {
-  return /git\s+(commit|push)/.test(command);
-}
-
-// ─────────────────────────────────────────────────────────────
-// Core logic (exported for testing)
-// ─────────────────────────────────────────────────────────────
-
-export async function mainWithDeps(deps: HookDeps): Promise<void> {
-  const { readStdin, output, log } = deps;
-
-  const stdin = await readStdin();
-
-  // Extract the bash command from the tool input envelope
-  const command =
-    (getField(stdin, "tool_input.command") as string | undefined) ||
-    (getField(stdin, "command") as string | undefined) ||
-    "";
-
-  if (!commandMatchesGitOp(command)) {
-    // Silent pass-through — not a git commit/push, nothing to do
-    log(HOOK, `PASS (no git op): ${command.slice(0, 60)}`);
-    return;
-  }
-
-  log(HOOK, `REMIND git op detected: ${command.slice(0, 80)}`);
-
-  // Allow the command but inject a verification reminder as context.
-  // This hook NEVER blocks — it only nudges the model to verify first.
-  output({
-    decision: "allow",
-    context: [
-      "[VERIFICATION-REMINDER] You are about to run a git commit/push.",
-      "Before proceeding, confirm:",
-      "  1. Tests pass — run the test command and paste the output.",
-      "  2. Lint clean — run the lint command and confirm 0 errors.",
-      "  3. Build succeeds — run the build command and confirm exit 0.",
-      "  4. All acceptance criteria have matching test evidence.",
-      "Evidence before claims. No shortcuts for verification.",
-    ].join("\n"),
-  });
-}
-
-// ─────────────────────────────────────────────────────────────
-// Entry point (production use)
-// ─────────────────────────────────────────────────────────────
-
-async function main(): Promise<void> {
-  await mainWithDeps(defaultDeps);
-}
-
-main().catch(() => process.exit(0));