@bonesofspring/ai-rules 0.2.19 → 0.2.21

package/scripts/check-chain-hook-coverage.sh

@@ -0,0 +1,109 @@
+#!/usr/bin/env bash
+# FAIL when chain-team-phases.sh has an orphaned entry:
+#  - DEFAULT_ARTIFACT_OUTPUT_IDS entry without an AGENT_HANDOFF entry, or
+#  - AGENT_HANDOFF entry without a DEFAULT_ARTIFACT_OUTPUT_IDS entry.
+#
+# Also FAIL when a stack ships an agent file that the hook ignores entirely
+# (no entry in either map) AND the stack has a pipeline intent that can route
+# to it — for example, intent `openspec-change` requires `openspec-author` to
+# be wired through both maps or the pipeline silently halts in
+# `awaiting_artifact_receipt`. We check the documented intent-to-agent table in
+# each stack's task-router.md against the hook's actual coverage.
+set -euo pipefail
+
+ROOT="$(cd "$(dirname "$0")/../../.." && pwd)"
+export AI_RULES_PRESETS_ROOT="$ROOT/packages/ai-rules/presets"
+
+python3 <<'PY'
+from __future__ import annotations
+
+import os
+import re
+import sys
+from pathlib import Path
+
+presets = Path(os.environ["AI_RULES_PRESETS_ROOT"])
+
+# Stacks that ship a /task pipeline. mcp-ts ships a Phase-1 chain but not the
+# standard openspec-change intent; do not require openspec-author there.
+STACKS_WITH_OPENSPEC = {
+    ("cursor", "android-kotlin"),
+    ("cursor", "go"),
+    ("cursor", "ios-swift"),
+    ("cursor", "java"),
+    ("cursor", "next"),
+    ("cursor", "nuxt"),
+    ("cursor", "php-hexagonal"),
+    ("cursor", "php-laravel"),
+    ("cursor", "svelte"),
+    ("claude", "android-kotlin"),
+    ("claude", "go"),
+    ("claude", "ios-swift"),
+    ("claude", "java"),
+    ("claude", "next"),
+    ("claude", "nuxt"),
+    ("claude", "php-hexagonal"),
+    ("claude", "php-laravel"),
+    ("claude", "svelte"),
+}
+
+# Agents used only outside the pipeline (no chain hook entry expected).
+PIPELINE_EXEMPT = {"task-router", "codebase-analyzer"}
+
+failures: list[str] = []
+checks = 0
+
+
+def parse_block(text: str, header_re: str, line_re: str) -> set[str]:
+    m = re.search(header_re, text, re.DOTALL)
+    if not m:
+        return set()
+    return {x.group(1) for x in line_re.finditer(m.group(1))}
+
+
+for hook in sorted(presets.glob("*/**/hooks/chain-team-phases.sh")):
+    rel = hook.relative_to(presets)
+    platform, preset = rel.parts[0], rel.parts[1]
+    text = hook.read_text(encoding="utf-8")
+
+    artifact_ids = parse_block(
+        text,
+        r"const DEFAULT_ARTIFACT_OUTPUT_IDS = \{(.*?)\};\n",
+        re.compile(r"^\s*['\"]([a-z][a-z0-9.\-]*)['\"]\s*:\s*['\"][a-z][a-z0-9.\-]*['\"]\s*,?", re.MULTILINE),
+    )
+    handoff_keys = parse_block(
+        text,
+        r"const AGENT_HANDOFF = \{(.*?)\n\};\n",
+        re.compile(r"^\s*['\"]([a-z][a-z0-9.\-]*)['\"]\s*:\s*\(s", re.MULTILINE),
+    )
+
+    # 1) Symmetric invariant: any entry in either map must appear in the other.
+    only_artifact = artifact_ids - handoff_keys
+    only_handoff = handoff_keys - artifact_ids
+    for agent in sorted(only_artifact - PIPELINE_EXEMPT):
+        checks += 1
+        failures.append(f"{rel}: DEFAULT_ARTIFACT_OUTPUT_IDS '{agent}' has no AGENT_HANDOFF entry")
+    for agent in sorted(only_handoff - PIPELINE_EXEMPT):
+        checks += 1
+        failures.append(f"{rel}: AGENT_HANDOFF '{agent}' has no DEFAULT_ARTIFACT_OUTPUT_IDS entry")
+
+    # 2) openspec-author must be in BOTH maps for any stack that ships the
+    # agent. The hook without this entry silently breaks the openspec-change
+    # intent (the original bug).
+    agents_dir = presets / platform / preset / "agents"
+    if (platform, preset) in STACKS_WITH_OPENSPEC and (agents_dir / "openspec-author.md").exists():
+        checks += 1
+        if "openspec-author" not in artifact_ids:
+            failures.append(f"{rel}: ships openspec-author.md but hook is missing DEFAULT_ARTIFACT_OUTPUT_IDS['openspec-author'] (openspec-change intent will silently stall in awaiting_artifact_receipt)")
+        if "openspec-author" not in handoff_keys:
+            failures.append(f"{rel}: ships openspec-author.md but hook is missing AGENT_HANDOFF['openspec-author']")
+
+if failures:
+    print(f"FAIL: chain-team-phases.sh coverage gap ({len(failures)} issue(s) / {checks} checks):")
+    for f in failures:
+        print(f"  - {f}")
+    sys.exit(1)
+
+print(f"OK: chain-team-phases.sh hook coverage is consistent ({checks} checks across all hooks)")
+sys.exit(0)
+PY

package/scripts/check-local-overlay.sh

@@ -0,0 +1,136 @@
+#!/usr/bin/env bash
+# Verify that every installable stack ships an `ai-local-overlay` rule on both
+# sides (Cursor `.mdc` and Claude `.md`) and that the body contains the
+# mandatory acknowledgement format.
+#
+# Exit 0 on OK; exit 1 on any FAIL.
+set -euo pipefail
+
+ROOT="$(cd "$(dirname "$0")/../../.." && pwd)"
+export AI_RULES_PRESETS_ROOT="$ROOT/packages/ai-rules/presets"
+
+python3 <<'PY'
+from __future__ import annotations
+
+import os
+import re
+import sys
+from pathlib import Path
+
+presets = Path(os.environ["AI_RULES_PRESETS_ROOT"])
+STACKS = (
+    "next",
+    "nuxt",
+    "svelte",
+    "ios-swift",
+    "android-kotlin",
+    "go",
+    "java",
+    "mcp-ts",
+    "php-hexagonal",
+    "php-laravel",
+)
+
+CURSOR_NAME = "ai-local-overlay.mdc"
+CLAUDE_NAME = "ai-local-overlay.md"
+# The body line that the agent MUST emit when it applies the overlay.
+ACK_RE = re.compile(r"Using\s+\.ai\.local:\s+<rule-name>")
+# Sentinel strings that signal the rule is mandatory / explains authoring /
+# points at the consumer guide.
+FRONTMATTER_DESCRIPTION = "MANDATORY"
+POINTER_DOCS = "docs/LOCAL-OVERLAY.md"
+
+
+def strip_frontmatter(text: str) -> tuple[str, str]:
+    if not text.startswith("---\n"):
+        return "", text
+    end = text.find("\n---\n", 4)
+    if end < 0:
+        return "", text
+    return text[4:end], text[end + 5 :]
+
+
+failed = 0
+stack_failed: dict[str, int] = {}
+
+print("Local overlay coverage (every stack must ship ai-local-overlay)")
+print(f"Presets: {presets}")
+print()
+
+for stack in STACKS:
+    print(f"=== {stack} ===")
+    cursor_path = presets / "cursor" / stack / "rules" / CURSOR_NAME
+    claude_path = presets / "claude" / stack / "rules" / CLAUDE_NAME
+
+    # Per-stack failure counter — keeps the OK message for healthy stacks
+    # even when an earlier stack has failed.
+    sf = 0
+
+    # Cursor must exist + alwaysApply: true + ack format + pointer.
+    if not cursor_path.is_file():
+        print(f"FAIL: missing {cursor_path}")
+        sf += 1
+    else:
+        text = cursor_path.read_text(encoding="utf-8")
+        fm, body = strip_frontmatter(text)
+        if "alwaysApply: true" not in fm:
+            print(f"FAIL: {cursor_path.name} must have alwaysApply: true")
+            sf += 1
+        if FRONTMATTER_DESCRIPTION not in fm:
+            print(f"FAIL: {cursor_path.name} frontmatter description must include 'MANDATORY'")
+            sf += 1
+        if not ACK_RE.search(body):
+            print(f"FAIL: {cursor_path.name} body missing acknowledgement format `Using .ai.local: <rule-name>`")
+            sf += 1
+        if POINTER_DOCS not in body:
+            print(f"FAIL: {cursor_path.name} body must reference `docs/LOCAL-OVERLAY.md`")
+            sf += 1
+        if sf == 0:
+            print(f"  OK: {cursor_path.name} (alwaysApply + MANDATORY + ack + pointer)")
+
+    # Claude must exist (flat or topic folder) + paths: includes .ai.local + ack format + pointer.
+    if not claude_path.is_file():
+        # Try to find a topic-folder placement (e.g. tooling-and-review/ai-local-overlay.md).
+        candidates = list((presets / "claude" / stack / "rules").rglob(CLAUDE_NAME))
+        if candidates:
+            claude_path = candidates[0]
+        else:
+            print(f"FAIL: missing {claude_path}")
+            sf += 1
+    if claude_path.is_file():
+        text = claude_path.read_text(encoding="utf-8")
+        fm, body = strip_frontmatter(text)
+        if "**/.ai.local/**" not in fm:
+            print(f"FAIL: {claude_path.name} frontmatter must include `**/.ai.local/**` in paths")
+            sf += 1
+        if FRONTMATTER_DESCRIPTION not in fm:
+            print(f"FAIL: {claude_path.name} frontmatter description must include 'MANDATORY'")
+            sf += 1
+        if not ACK_RE.search(body):
+            print(f"FAIL: {claude_path.name} body missing acknowledgement format `Using .ai.local: <rule-name>`")
+            sf += 1
+        if POINTER_DOCS not in body:
+            print(f"FAIL: {claude_path.name} body must reference `docs/LOCAL-OVERLAY.md`")
+            sf += 1
+        if sf == 0:
+            print(f"  OK: {claude_path.name} (paths + MANDATORY + ack + pointer)")
+
+    # Twin-map coverage: the Claude twin must be listed in claude/<stack>/rules/README.md.
+    claude_readme = presets / "claude" / stack / "rules" / "README.md"
+    if claude_readme.is_file():
+        readme_text = claude_readme.read_text(encoding="utf-8")
+        if CURSOR_NAME not in readme_text:
+            print(f"FAIL: {claude_readme} mapping table must include `{CURSOR_NAME}`")
+            sf += 1
+
+    stack_failed[stack] = sf
+    failed += sf
+    print()
+
+if failed:
+    print(f"Result: FAIL ({failed} issue(s) across {sum(1 for v in stack_failed.values() if v > 0)} stack(s))")
+    sys.exit(1)
+
+print("Result: OK")
+sys.exit(0)
+PY

package/scripts/check-preset-i18n.sh

@@ -54,8 +54,8 @@ if len(violations) > 50:
     print(f"  ... and {len(violations) - 50} more")
 
 if violations and (fail or not soft):
-    if fail:
-        sys.exit(1)
+    print("Result: FAIL (unmarked Cyrillic)" if fail else "Result: WARN (soft)")
+    sys.exit(1)
 print("Result:", "WARN (soft)" if violations else "OK")
 sys.exit(0)
 PY

package/scripts/check-preset-leakage.sh

@@ -278,7 +278,10 @@ def line_is_negation(line: str) -> bool:
 
 
 def is_forbidden_agent(name: str, bad_stems: tuple[str, ...]) -> bool:
-    return name in bad_stems or any(name.startswith(s) for s in bad_stems)
+    # Exact match, or kebab-case prefix (so `playwright-test-planner` matches
+    # `playwright-test-planner-foo` too — kebab-case is the only naming style in
+    # use across all presets, so the naive `startswith` is intentional).
+    return name in bad_stems or any(name.startswith(s + "-") or name == s for s in bad_stems)
 
 
 def collect_step_agents(steps) -> list[str]:

package/scripts/check-preset-structure.sh

@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-# Orchestrator: twin map + token budget + leakage + hook-only lifecycle + shared-core drift + i18n (hard).
+# Orchestrator: twin map + token budget + leakage + hook-only lifecycle + local-overlay + shared-core drift + i18n (hard).
 # All gates exit non-zero on FAIL.
 set -euo pipefail
 
@@ -33,7 +33,9 @@ run_gate "twin-map" "$SCRIPT_DIR/check-preset-twin-map.sh"
 run_gate "token-budget" "$SCRIPT_DIR/check-preset-token-budget.sh"
 run_gate "leakage" "$SCRIPT_DIR/check-preset-leakage.sh"
 run_gate "hook-only-lifecycle" "$SCRIPT_DIR/check-hook-only-lifecycle.sh"
+run_gate "local-overlay" "$SCRIPT_DIR/check-local-overlay.sh"
 run_gate "shared-core-drift" "$SCRIPT_DIR/check-shared-core-drift.sh" AI_RULES_DRIFT_FAIL=1
+run_gate "chain-hook-coverage" "$SCRIPT_DIR/check-chain-hook-coverage.sh"
 run_gate "i18n" "$SCRIPT_DIR/check-preset-i18n.sh" AI_RULES_I18N_FAIL=1 AI_RULES_I18N_SOFT=0
 
 echo ">>> cli-mcp-merge"
@@ -45,6 +47,24 @@ else
 fi
 echo
 
+echo ">>> chain-team-phases coverage"
+if node "$SCRIPT_DIR/test-chain-team-phases-coverage.mjs"; then
+  echo "<<< chain-team-phases coverage: OK"
+else
+  echo "<<< chain-team-phases coverage: FAIL"
+  status=1
+fi
+echo
+
+echo ">>> agent-task-metrics hooks"
+if node "$SCRIPT_DIR/test-agent-task-metrics-hooks.mjs"; then
+  echo "<<< agent-task-metrics hooks: OK"
+else
+  echo "<<< agent-task-metrics hooks: FAIL"
+  status=1
+fi
+echo
+
 # Soft WARN: dogfood product-specs (never fails the orchestrator).
 # Default check-product-specs exits 0 even when issues are found unless --strict.
 # Use --strict here only to surface problems as WARN in logs; do not flip status.

package/scripts/check-preset-token-budget.sh

@@ -16,7 +16,8 @@ from pathlib import Path
 
 presets = Path(os.environ["AI_RULES_PRESETS_ROOT"])
 STACKS = ("next", "nuxt", "svelte", "ios-swift", "android-kotlin", "go", "java", "mcp-ts", "php-hexagonal", "php-laravel")
-BODY_CAP = 140
+BODY_CAP = 140  # applied to 3-rule trios (Claude session-start, Cursor trio when len==3)
+BODY_CAP_CURSOR_4 = 170  # Cursor now allows 4 always-on (next-app-core, package-manager, code-quality, ai-local-overlay)
 MARKER_LINE_RE = re.compile(r"^<!--\s*shared-core:\s*[^>]+?\s*-->\s*$")
 ALWAYS_APPLY_RE = re.compile(r"(?m)^alwaysApply:\s*true\s*$")
 PATHS_RE = re.compile(r"(?m)^paths:\s*")
@@ -70,7 +71,7 @@ failed = 0
 
 print("Preset token / session-start budget (Option A)")
 print(f"Presets: {presets}")
-print(f"Body line cap (trio sum): {BODY_CAP}")
+print(f"Body line cap (Claude trio): {BODY_CAP}; (Cursor 4-set): {BODY_CAP_CURSOR_4}")
 print()
 
 for stack in STACKS:
@@ -87,9 +88,15 @@ for stack in STACKS:
             print(f"FAIL: meta preset-* must not be alwaysApply: {p.name}")
             failed += 1
 
-    if len(always) != 3:
-        print(f"FAIL: Cursor alwaysApply count is {len(always)}, expected exactly 3")
+    if len(always) not in (3, 4):
+        print(f"FAIL: Cursor alwaysApply count is {len(always)}, expected 3 or 4 (4th slot reserved for ai-local-overlay)")
         failed += 1
+    elif len(always) == 4:
+        # 4-set: ensure the 4th rule is ai-local-overlay (or declares itself as such).
+        extra = [p for p in always if p.stem not in {"code-quality-and-refactoring", "package-manager", "next-app-core", "nuxt-app-core", "svelte-app-core", "ios-app-core", "android-app-core", "go-app-core", "java-app-core", "mcp-ts-app-core", "php-app-core", "laravel-app-core", "architecture-boundaries", "mcp-server-boundaries"}]
+        if not any(p.stem == "ai-local-overlay" for p in always) and extra:
+            print(f"FAIL: Cursor alwaysApply=4 but no ai-local-overlay.mdc; the 4th slot is reserved for the local overlay rule")
+            failed += 1
 
     # --- Claude no-paths (excl README) ---
     session = claude_session_start(claude_rules)
@@ -108,10 +115,12 @@ for stack in STACKS:
         )
         failed += 1
 
-    # --- Body sum of intended trio (Cursor alwaysApply + Claude session) ---
-    # Prefer Cursor trio for budget; also check Claude trio independently.
+    # --- Body sum of intended session-start set (Cursor alwaysApply + Claude session) ---
+    # Cursor allows 3 or 4 rules (4th slot reserved for ai-local-overlay); Claude is exactly 3.
     for label, files in (("Cursor", always), ("Claude", session)):
-        if len(files) != 3:
+        if label == "Cursor" and len(files) not in (3, 4):
+            continue
+        if label == "Claude" and len(files) != 3:
             continue
         total = 0
         for path in files:
@@ -121,10 +130,11 @@ for stack in STACKS:
             total += n
             name = path.name if label == "Cursor" else path.relative_to(claude_rules).as_posix()
             print(f"  {label} body lines {name}: {n}")
-        print(f"  {label} trio body sum: {total}")
-        if total > BODY_CAP:
+        cap = BODY_CAP_CURSOR_4 if label == "Cursor" and len(always) == 4 else BODY_CAP
+        print(f"  {label} session-start body sum: {total} (cap {cap}, files={len(files)})")
+        if total > cap:
             print(
-                f"FAIL: {label} session-start body sum {total} > {BODY_CAP} ({stack})"
+                f"FAIL: {label} session-start body sum {total} > {cap} ({stack})"
             )
             failed += 1
 

package/scripts/check-rule-eval.mjs

@@ -99,6 +99,22 @@ function checkFile(relPath, minPairs, requireImpact) {
       ) && ok;
   }
 
+  // Order check: each Incorrect heading must precede its matching Correct heading.
+  // Without this, an agent reading the rule may learn a "correct" pattern before
+  // seeing the "incorrect" example it contrasts with.
+  if (incorrectCount > 0 && correctCount > 0) {
+    const incRe = /^#{2,3}\s+Incorrect\s*$/gim;
+    const corRe = /^#{2,3}\s+Correct\s*$/gim;
+    const firstInc = text.search(incRe);
+    const firstCor = text.search(corRe);
+    if (firstInc >= 0 && firstCor >= 0 && firstInc >= firstCor) {
+      ok =
+        fail(
+          `${relPath}: first "Incorrect" heading (offset ${firstInc}) must precede first "Correct" heading (offset ${firstCor})`,
+        ) && ok;
+    }
+  }
+
   const incorrectBodies = sectionBodies(text, 'Incorrect');
   const correctBodies = sectionBodies(text, 'Correct');
   for (let i = 0; i < Math.min(incorrectBodies.length, correctBodies.length); i++) {

package/scripts/check-shared-core-drift.sh

@@ -7,42 +7,26 @@ set -euo pipefail
 ROOT="$(cd "$(dirname "$0")/../../.." && pwd)"
 export AI_RULES_PRESETS_ROOT="$ROOT/packages/ai-rules/presets"
 
-python3 <<'PY'
+SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
+EMBED_LIST_FILE="$SCRIPT_DIR/_lib/embed-stems.json"
+
+python3 - "$EMBED_LIST_FILE" <<'PY'
 from __future__ import annotations
 
+import json
 import os
 import re
 import sys
 from pathlib import Path
 
+embed_list_file = Path(sys.argv[1])
+_stems = json.loads(embed_list_file.read_text(encoding="utf-8"))
+EMBED = _stems["embed"]
+LINEAGE = _stems["lineage"]
+
 presets = Path(os.environ["AI_RULES_PRESETS_ROOT"])
 core_root = presets / "_shared" / "core"
 
-EMBED = [
-    "quality/code-quality-and-refactoring.md",
-    "quality/anti-sycophancy-discipline.md",
-    "quality/design-guidance.md",
-    "quality/mcp-usage.md",
-    "meta/preset-layering.md",
-    "meta/preset-twin-sync.md",
-    "meta/preset-token-budget.md",
-    "meta/preset-pr-checklist.md",
-    "meta/preset-no-cross-stack-leakage.md",
-    "review/code-review-mr.md",
-    "review/technical-retro.md",
-    "review/mockup-review.md",
-    "review/ux-design.md",
-    "agent-team/agent-team-intake.md",
-    "architecture/product-specs.md",
-    "architecture/product-specs-authoring.md",
-]
-
-LINEAGE = [
-    "architecture/feature-delivery-workflow.md",
-    "architecture/reference-features.template.md",
-    "agent-team/agent-team-orchestrator.md",
-]
-
 # Full-line sync marker only (do not strip prose that *mentions* the marker).
 MARKER_LINE_RE = re.compile(r"^<!--\s*shared-core:\s*[^>]+?\s*-->\s*$")
 

package/scripts/close-agent-task-human-gate.mjs

@@ -13,18 +13,35 @@ import path from 'node:path';
 
 function parseArgs(argv) {
   const options = { cwd: process.cwd(), platform: 'cursor', slug: null };
+  // Read the next token from argv for value-taking flags. Returns the value or null
+  // when the flag is missing its value; the caller exits with a friendly error.
+  function nextValue(flag, i) {
+    const next = argv[i + 1];
+    if (next === undefined || next.startsWith('-')) {
+      console.error(`Missing value for ${flag} (use ${flag} <value>)`);
+      process.exit(2);
+    }
+    return next;
+  }
   for (let i = 2; i < argv.length; i += 1) {
     const arg = argv[i];
-    if (arg === '--cwd') options.cwd = path.resolve(argv[++i]);
-    else if (arg === '--platform') options.platform = argv[++i];
-    else if (arg === '--slug') options.slug = argv[++i];
-    else if (arg === '--activate-next') {
+    if (arg === '--cwd') {
+      options.cwd = path.resolve(nextValue(arg, i));
+      i += 1;
+    } else if (arg === '--platform') {
+      options.platform = nextValue(arg, i);
+      i += 1;
+    } else if (arg === '--slug') {
+      options.slug = nextValue(arg, i);
+      i += 1;
+    } else if (arg === '--activate-next') {
       /* always activate via hook payload */
     } else if (arg === '--team-root') {
       // Compatibility with earlier docs: derive platform + cwd from team root.
-      const teamRoot = path.resolve(argv[++i]);
+      const teamRoot = path.resolve(nextValue(arg, i));
       options.cwd = path.resolve(teamRoot, '..', '..');
       options.platform = path.basename(path.dirname(teamRoot)) === '.claude' ? 'claude' : 'cursor';
+      i += 1;
     } else {
       console.error(`Unknown argument: ${arg}`);
       process.exit(2);

package/scripts/sync-shared-core-embeds.sh

@@ -8,35 +8,24 @@ set -euo pipefail
 ROOT="$(cd "$(dirname "$0")/../../.." && pwd)"
 export AI_RULES_PRESETS_ROOT="$ROOT/packages/ai-rules/presets"
 
-python3 <<'PY'
+SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
+EMBED_LIST_FILE="$SCRIPT_DIR/_lib/embed-stems.json"
+
+python3 - "$EMBED_LIST_FILE" <<'PY'
 from __future__ import annotations
 
+import json
 import os
 import re
+import sys
 from pathlib import Path
 
+embed_list_file = Path(sys.argv[1])
+EMBED = json.loads(embed_list_file.read_text(encoding="utf-8"))["embed"]
+
 presets = Path(os.environ["AI_RULES_PRESETS_ROOT"])
 core_root = presets / "_shared" / "core"
 
-EMBED = [
-    "quality/code-quality-and-refactoring.md",
-    "quality/anti-sycophancy-discipline.md",
-    "quality/design-guidance.md",
-    "quality/mcp-usage.md",
-    "meta/preset-layering.md",
-    "meta/preset-twin-sync.md",
-    "meta/preset-token-budget.md",
-    "meta/preset-pr-checklist.md",
-    "meta/preset-no-cross-stack-leakage.md",
-    "review/code-review-mr.md",
-    "review/technical-retro.md",
-    "review/mockup-review.md",
-    "review/ux-design.md",
-    "agent-team/agent-team-intake.md",
-    "architecture/product-specs.md",
-    "architecture/product-specs-authoring.md",
-]
-
 MARKER_LINE_RE = re.compile(r"^<!--\s*shared-core:\s*([^>]+?)\s*-->\s*$")
 
 

package/scripts/test-agent-task-metrics-hooks.mjs

@@ -14,7 +14,7 @@ const hooks = [
   ...fs.readdirSync(path.join(presetsDirectory, 'claude')).map((stack) => path.join(presetsDirectory, 'claude', stack, 'hooks', 'chain-team-phases.sh')),
 ];
 
-assert.equal(hooks.length, 14, 'expected every packaged Cursor and Claude hook');
+assert.equal(hooks.length, 20, 'expected every packaged Cursor and Claude hook (10 stacks × 2 platforms)');
 
 function readJson(filePath) {
   return JSON.parse(fs.readFileSync(filePath, 'utf8'));