@bonesofspring/ai-rules 0.2.26 → 0.2.27

package/presets/claude/next/hooks/chain-team-phases.sh

@@ -42,7 +42,10 @@ try {
 }
 
 const slug = active.slug;
-if (!slug) {
+// Reject non-kebab / traversal slugs: the runtime writes status.json and
+// metrics.json under tasks/<slug>, so an unchecked slug could escape the tree.
+const SLUG_RE = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+if (typeof slug !== 'string' || slug.length > 48 || !SLUG_RE.test(slug)) {
   process.stdout.write('{}');
   process.exit(0);
 }
@@ -69,6 +72,9 @@ function waitForMetricsLock() {
   // and never blocked on the lock file, so two concurrent subagentStop events
   // could both acquire the lock and corrupt metrics.json. Retry EEXIST with a
   // 10 ms sleep up to 200 attempts (2 s ceiling). On any other error, throw.
+  // A lock left behind by a killed process is reclaimed once its mtime is older
+  // than STALE_LOCK_MS, so one crash cannot brick the task forever.
+  const STALE_LOCK_MS = 30000;
   for (let attempt = 0; attempt < 200; attempt += 1) {
     try {
       const fd = fs.openSync(METRICS_LOCK, 'wx');
@@ -76,6 +82,13 @@ function waitForMetricsLock() {
       return;
     } catch (error) {
       if (!error || error.code !== 'EEXIST') throw error;
+      try {
+        const lockStat = fs.statSync(METRICS_LOCK);
+        if (Date.now() - lockStat.mtimeMs > STALE_LOCK_MS) {
+          fs.unlinkSync(METRICS_LOCK);
+          continue;
+        }
+      } catch {}
       const until = Date.now() + 10;
       while (Date.now() < until) { /* 10 ms busy-wait */ }
     }
@@ -493,7 +506,10 @@ function legacyFlow() {
 
 
 if (hookInput.action === 'end_human_gate') {
-    status.awaitingHumanGate = false;
+  // Only advance when a gate was actually open. A duplicate or retried
+  // end_human_gate must be a no-op, not a silent pipeline skip.
+  const gateWasOpen = Boolean(status.activeHumanGateEventId) || status.awaitingHumanGate === true;
+  status.awaitingHumanGate = false;
   // A2 (retro §8): pin a single runId per pipeline so subsequent events don't
   // fragment the metrics ledger across multiple runId sequences. Prefer
   // metrics.json.task.runId when metrics already exist, then status.metricsRunId,
@@ -520,7 +536,7 @@ if (hookInput.action === 'end_human_gate') {
     });
     delete status.activeHumanGateEventId;
   }
-  if (hookInput.activateNext && fs.existsSync(pipelinePath)) {
+  if (gateWasOpen && hookInput.activateNext && fs.existsSync(pipelinePath)) {
     let resumePipeline = null;
     try { resumePipeline = JSON.parse(fs.readFileSync(pipelinePath, 'utf8')); } catch {}
     if (resumePipeline) {
@@ -542,6 +558,19 @@ if (hookInput.action === 'end_human_gate') {
         else out(handoffForAgent(resumeAgents[0], resumeNext.step.scope, resumeNext.step));
         process.exit(0);
       }
+      // Gate on the final step: no executable step remains. Complete the
+      // pipeline instead of leaving it in_progress forever.
+      status.state = 'completed';
+      completeTaskMetrics();
+      writeStatus({
+        pipelineIndex: resumeSteps.length,
+        phase: 'done',
+        state: 'completed',
+        currentAgent: null,
+        awaitingHumanGate: false,
+      });
+      out(`Pipeline complete for "${slug}". Run /technical-retro ${slug}.`);
+      process.exit(0);
     }
   }
   // A2 (retro §8): if gate closes the final human gate of the pipeline

package/presets/claude/next/hooks/examples/README.md

@@ -8,11 +8,13 @@ Optional scripts under `hooks/examples/`. **Not** referenced from default `hooks
 | `format-edited.example.sh` | Stack formatter / linter on edited files |
 | `test-on-save.example.sh` | Scoped tests for the stack |
 
-## Wiring (Cursor)
+## Wiring (Claude Code)
+
+Claude Code has no preset `hooks.json`; hooks are registered in `.claude/settings.json` and fire on events such as `PreToolUse`, `PostToolUse`, and `Stop`.
 
 1. Copy an example into `hooks/` (drop `.example` suffix), e.g. `cp hooks/examples/secret-guard.example.sh hooks/secret-guard.sh`.
 2. `chmod +x hooks/secret-guard.sh`.
-3. Add a hook entry in `.cursor/hooks.json` (e.g. `beforeShellExecution` or `afterFileEdit` — see Cursor hooks docs).
-4. Keep cookbook scripts **out** of the default preset `hooks.json` when contributing back upstream.
+3. Register the command in `.claude/settings.json` (see Claude Code hooks docs for event/matcher syntax).
+4. Keep cookbook scripts **out** of the default preset when contributing back upstream.
 
 Stack: **next** — recipes must stay stack-safe (no cross-stack lint/test commands).

package/presets/claude/next/rules/README.md

@@ -14,7 +14,7 @@ In Claude Code all `.md` files under `rules/` are **discovered recursively**. Bo
 
 ## Loading strategy
 
-- **Session start (no `paths:`):** exactly **3** — `next-app-core`, `package-manager`, `code-quality`
+- **Session start (no `paths:`):** 3–5 — `next-app-core`, `package-manager`, `code-quality`
 - **On-demand (`paths:` or via commands):** `post-change-lint` (**must after code edits**), `agent-team-intake`, `agent-team-orchestrator`, `code-review-mr`, `security-next`, architecture, imports, UI, tests, feature-delivery-workflow, `design-guidance`, `mcp-usage`, `anti-sycophancy-discipline`, `react-next-anti-hallucination`, reference-features, next-app-router, react-a11y-coding, `technical-retro`, `mockup-review`, `ux-design`, **preset-*** meta (paths on `packages/ai-rules/presets/**`)
 - **Skills:** long workflows (`feature-delivery`, `spec-authoring`, `spec-discovery`, `code-review`, …)
 

package/presets/claude/next/rules/api-and-data/api-services.md

@@ -20,8 +20,8 @@ impact: HIGH
 - For each domain area (for example, orders, billing, account settings):
   - a separate directory in `app/src/api/services/<ServiceName or feature>/**` following repository convention.
 - Inside a module:
-  - files with API calls (`index.ts` or `*.service.ts`);
-  - mapper files (`*responseMappers.ts`);
+  - files with API calls — follow the repo's shape: `index.ts` / `*.service.ts`, **or** a per-domain module exporting an object (`XxxApiMethods` in `<Domain>.apiMethods.ts` with `<Domain>.endpoints.ts` and `<Domain>.responseTypes.ts`) as in neighboring services;
+  - mapper files (`*responseMappers.ts`) — pure `format…` / `map…` functions named as in the repo;
   - request/response-specific types (if not moved to `app/src/types/**` with export through the `@/types` barrel).
   - when needed — local `index.ts` / barrel inside the module for structure **within** `app/src/api/**`; **outside** this layer UI, store, and other code import only from the root barrel `app/src/api/index.ts` (`import … from '@/api'`, **`architecture/public-imports.md`**).
 
@@ -70,10 +70,12 @@ export const getCard = async (id: string) => {
 
 ```ts
 import type { TCard } from '@/types'
-import { mapCardResponse } from './cardResponseMappers'
+import { formatCardResponse } from './cardResponseMappers'
 
 export const getCard = async (id: string): Promise<TCard> => {
   const { data } = await cardsClient.get(`/cards/${id}`)
-  return mapCardResponse(data)
+  return formatCardResponse(data)
 }
 ```
+
+Mapper function verb (`format…` vs `map…`) follows the repo — match neighboring mappers.

package/presets/claude/next/rules/api-and-data/store-rtk.md

@@ -17,13 +17,14 @@ impact: HIGH
 # Slice structure
 
 - Each domain module — its own slice in `app/src/store/slices/**`.
-- A slice exports:
-  - default `reducer`;
-  - `actions` as named exports;
-  - selectors `selectSomething(state: RootState): Type`.
+- A slice exports — follow the repo's shape:
+  - a named slice (`xxxSlice`) re-exported through a per-slice barrel (`index.ts`), **or** a default `reducer`;
+  - actions as named exports;
+  - selectors named as in the repo (`selectSomething(state: RootState): Type`, or a curried `xxxSelector = (args) => (state) => …`); memoize derived values with `createSelector`.
 - State:
   - explicit state type;
-  - careful `initialState` initialization.
+  - careful `initialState` initialization;
+  - typed root state — use the repo's `RootState` if present, otherwise the local `IRootState` interface neighboring slices use.
 
 # Async and side effects
 

package/presets/claude/next/rules/architecture/feature-delivery-workflow.md

@@ -8,6 +8,10 @@ paths:
 
 When the prompt or `brief.md` contains a `figma.com` URL, **mockup review** (`mockup-reviewer` / skill `mockup-review`) must complete — or the user must continue past FAIL/`BLOCKED_MCP` via `/task-continue` — **before UI implementation**. Non-UI tasks no-op this gate. When required-signal and **no** `figma.com`, **`ux-design.md`** (on the analysis manifest entry) must exist and not be `INCOMPLETE` before UI work — else **blocked**, hand back to `task-analyst`. Parallel to the mockup gate.
 
+## Style discovery (mandatory)
+
+Before writing a new file in a layer, open **1–2 analog files** in the same layer and mirror: export shape, name prefixes (`T` / `I` where the repo uses them), file suffixes (`.apiMethods.ts`, `.endpoints.ts`, `.data.ts`, `.hooks.ts`, `styles.ts`), mapper / selector / thunk naming, and barrel shape. Record the analogs in the handoff. Where this preset's examples differ from the repo, **the repo wins**.
+
 <!-- shared-core: architecture/feature-delivery-workflow.md -->
 
 

package/presets/claude/next/rules/architecture/reference-features.md

@@ -13,7 +13,7 @@ paths:
 
 Before implementing, find a feature of the same type and **mirror its structure**; do not invent a new file organization.
 
-Taxonomy / shared core: [`presets/_shared/README.md`](../../../../_shared/README.md) · [Preset contribution guide](../../../../docs/PRESET-CONTRIBUTION.md).
+Taxonomy / shared core: [`presets/_shared/README.md`](../../../../_shared/README.md) · [Preset contribution guide](../../../../../docs/PRESET-CONTRIBUTION.md).
 
 ## Reference table
 

package/presets/claude/next/rules/stack/next-app-core.md

@@ -5,8 +5,9 @@ Concrete versions come **from `package.json` and configs of the target repositor
 # Project structure
 
 - `app/` — Next.js root; `app/src/**` — code; `app/__tests__/e2e/**` — e2e.
-- `app/tsconfig.json`: `baseUrl: "."`, `paths: { "@/*": ["./src/*"] }`.
+- `app/tsconfig.json`: `paths: { "@/*": ["./src/*"] }` (`baseUrl` may be absent with TS ≥5 / `moduleResolution: bundler` — check the file, do not add it blindly).
 - **Requirement:** `@/*` instead of relative imports that climb the tree.
+- **Router mode:** detect from the repo before adding routes — App Router (`app/src/app/**`, `page.tsx`) vs Pages Router (`app/src/pages/**`, `_app.tsx`). Do **not** assume App Router; follow the mode already in use (`stack/next-app-router.md`, `stack/navigation-router.md`).
 
 # Architectural layers
 

package/presets/claude/next/rules/testing/tests-unit.md

@@ -3,6 +3,7 @@ description: Unit and integration tests (Testing Library and project runner)
 paths:
   - app/__tests__/unit/**/*.spec.{ts,tsx}
   - app/__tests__/unit/**/*.test.{ts,tsx}
+impact: HIGH
 ---
 
 # Testing principles

package/presets/claude/next/rules/tooling-and-review/code-quality.md

@@ -18,6 +18,12 @@ description: Maintain and improve code quality and architecture
 - **CUPID:** Composable; Unix philosophy (one responsibility); Predictable; Idiomatic (repo); Domain-based.
 - **Composition over inheritance;** Law of Demeter — no `a.b.c.d` chains; via public API / facade.
 
+# Over-abstraction guardrails
+
+- **No unnecessary constants:** do not hoist a one-off literal, string, or boolean into a named constant. Inline it, or use an existing token/enum when the value is meaningful and reused. Named constants are for shared, semantic, or repeated values.
+- **No invented utilities:** do not add a `utils`/helper for a single call site or a thin pass-through wrapper. Inline it or extend the existing helper; a new shared utility is justified only after the Rule of Three.
+- **Decompose over conditionals:** a unit (function, component, hook) that accumulates several branches (`if` / ternary / `switch`) should be split into smaller units with one reason to change instead of piling up conditions.
+
 # Refactoring during changes
 
 - Light refactor is OK if it reduces duplication, improves readability, and does not break public contracts.

package/presets/claude/next/rules/tooling-and-review/preset-layering.md

@@ -1,5 +1,5 @@
 ---
-description: Preset authoring: Core → Stack → Platform layering. Use when editing packages/ai-rules/presets/**.
+description: "Preset authoring: Core → Stack → Platform layering. Use when editing packages/ai-rules/presets/**."
 paths:
   - packages/ai-rules/presets/**/*
   - .claude/**/*
@@ -18,7 +18,7 @@ Rules for **preset authors** (`packages/ai-rules/presets/**`). Not always-on in
 | Layer | Where | What to put |
 |-------|-------|-------------|
 | **Core** | `presets/_shared/core/**` | Stack-agnostic prose + checklists; no frontmatter; no paths `app/**`, `Features/**`, Playwright, xcodebuild |
-| **Stack** | `presets/{cursor\|claude\|mavis\|opencode}/<stack>/` | Domain/tooling, stack always-on trio, stack examples in **Stack notes** |
+| **Stack** | `presets/{cursor\|claude\|mavis\|opencode}/<stack>/` | Domain/tooling, stack always-on set (3–5), stack examples in **Stack notes** |
 | **Platform** | Cursor `.mdc` / Claude `.md` / Mavis `.md` / OpenCode `.md` | Frontmatter (`alwaysApply`/`globs` vs `paths:` vs OpenCode `mode`/`permission`), platform-only (`BUGBOT.md`, `CLAUDE.md`) |
 
 ## Where to put a new rule

package/presets/claude/next/rules/tooling-and-review/preset-no-cross-stack-leakage.md

@@ -1,5 +1,5 @@
 ---
-description: Preset authoring: forbid cross-stack agent/path leakage in hooks and agents.
+description: "Preset authoring: forbid cross-stack agent/path leakage in hooks and agents."
 paths:
   - packages/ai-rules/presets/**/*
   - .claude/**/*

package/presets/claude/next/rules/tooling-and-review/preset-pr-checklist.md

@@ -1,5 +1,5 @@
 ---
-description: Preset authoring: PR checklist for twin sync, budget, leakage, README. Use on preset PRs.
+description: "Preset authoring: PR checklist for twin sync, budget, leakage, README. Use on preset PRs."
 paths:
   - packages/ai-rules/presets/**/*
   - .claude/**/*
@@ -21,7 +21,7 @@ Checklist for PRs that touch `packages/ai-rules/presets/**` (and dogfood `.curso
 - [ ] **Mavis derive:** mapped `cursor/<stack>/` change → `mavis/<stack>/` + `adapter-map.json` **same PR** when the stack is in `releasedStacks` (run `generate-mavis.mjs --stack <stack>` + `check-mavis-adapter-map.mjs`; see `preset-twin-sync` § Mavis derivation)
 - [ ] **OpenCode derive:** mapped `cursor/<stack>/` change → `opencode/<stack>/` + `adapter-map.json` **same PR** when the stack is in `releasedStacks` (run `generate-opencode.mjs --stack <stack>` + `check-opencode-adapter-map.mjs`; see `preset-twin-sync` § OpenCode derivation)
 - [ ] **Mapping:** `claude/<stack>/rules/README.md` mapping table is current
-- [ ] **Token budget:** Cursor `alwaysApply` ≤ 4 (4th slot reserved for `ai-local-overlay`); Claude session-start exactly 3; meta rules not always-on
+- [ ] **Token budget:** Cursor `alwaysApply` and Claude session-start both **3–5** (extra slots reserved for `ai-local-overlay`); body sum ≤200 (stretch ≤170); meta rules not always-on
 - [ ] **Local overlay coverage:** every installable stack ships `ai-local-overlay.mdc` (Cursor) and `ai-local-overlay.md` (Claude) with `MANDATORY` + acknowledgement format + `docs/LOCAL-OVERLAY.md` pointer; Claude twin listed in `claude/<stack>/rules/README.md` mapping table. Run `packages/ai-rules/scripts/check-local-overlay.sh`.
 - [ ] **Leakage:** no foreign stack agent stems / handoff strings (`preset-no-cross-stack-leakage`)
 - [ ] **Hook-only lifecycle:** `status.json` / `metrics.json` writes happen only inside `hooks/`; agents read but never rewrite lifecycle fields. Run `packages/ai-rules/scripts/check-hook-only-lifecycle.sh`.

package/presets/claude/next/rules/tooling-and-review/preset-token-budget.md

@@ -1,5 +1,5 @@
 ---
-description: Preset authoring: ≤3 always-on / session-start; meta rules never alwaysApply.
+description: "Preset authoring: ≤3 always-on / session-start; meta rules never alwaysApply."
 paths:
   - packages/ai-rules/presets/**/*
   - .claude/**/*
@@ -17,19 +17,19 @@ Invariants for rule loading in an **installed** preset (consumer app).
 
 | Platform | Always-on | Cap |
 |----------|-----------|-----|
-| Cursor | `alwaysApply: true` | **≤ 4** |
-| Claude | rules **without** `paths:` | **exactly 3** |
-| Mavis | `adapter-map` rules with `loadMode: root-entry` | **≤ 4** |
-| OpenCode | `adapter-map` rules with `loadMode: root-entry` | **≤ 4** |
+| Cursor | `alwaysApply: true` | **3–5** |
+| Claude | rules **without** `paths:` | **3–5** |
+| Mavis | `adapter-map` rules with `loadMode: root-entry` | **3–5** |
+| OpenCode | `adapter-map` rules with `loadMode: root-entry` | **3–5** |
 
-**Cursor 4th slot is reserved for `ai-local-overlay`** (consumer-owned `.ai.local/` rules). If you do not need local overlay, demote that rule to `globs: ["**/.ai.local/**"]` and revert to ≤3.
+Caps are **aligned** across platforms: minimum **3** (base invariants), maximum **5**. Slots beyond the base invariants are reserved for `ai-local-overlay` (consumer-owned `.ai.local/` rules). If you do not need local overlay, demote that rule to `globs: ["**/.ai.local/**"]` / Claude `paths:`, or a non-root load mode on Mavis / OpenCode.
 
-**Claude Option A (README exclusion):** when counting Claude rules without `paths:` / session-start, **exclude `**/README.md`** (topic indexes are not session-start rules). The intended session-start trio remains **exactly 3**; body budget sums **only those 3** rules.
+**Claude Option A (README exclusion):** when counting Claude rules without `paths:` / session-start, **exclude `**/README.md`** (topic indexes are not session-start rules). Body budget sums only the intended session-start rules (≤5).
 
 Session-start body budget:
-- Claude trio: ≤140 lines total (stretch ≤120).
-- Cursor 4-set: ≤170 lines total (4 rules including `ai-local-overlay`).
-- Mavis / OpenCode: ≤4 root-entry rule files; combined root `AGENTS.md` + platform entry ≤170 nonempty lines. Do not dump the full rule catalog into `AGENTS.md`.
+- Claude session-start: ≤200 lines total (stretch ≤170).
+- Cursor always-on: ≤200 lines total (stretch ≤170).
+- Mavis / OpenCode: ≤5 root-entry rule files; combined root `AGENTS.md` + platform entry ≤200 nonempty lines. Do not dump the full rule catalog into `AGENTS.md`.
 
 ## Meta / author rules
 
@@ -45,4 +45,4 @@ Orchestrator, feature-delivery, code-review, post-change gates, domain globs —
 
 ## Verifier
 
-build-verifier **FAIL** if Cursor `alwaysApply` count exceeds 4, Claude session-start count exceeds 3 (Claude count **excludes `**/README.md`**), or if a `preset-*` meta rule is marked always-on.
+build-verifier **FAIL** if Cursor `alwaysApply` count or Claude session-start count is outside **3–5** (Claude count **excludes `**/README.md`**), or if a `preset-*` meta rule is marked always-on.

package/presets/claude/next/rules/tooling-and-review/preset-twin-sync.md

@@ -1,5 +1,5 @@
 ---
-description: Preset authoring: Cursor SoT → Claude twin same PR. Use when editing preset rules.
+description: "Preset authoring: Cursor SoT → Claude twin same PR. Use when editing preset rules."
 paths:
   - packages/ai-rules/presets/**/*
   - .claude/**/*

package/presets/claude/next/rules/ui-and-accessibility/react-ui.md

@@ -8,6 +8,7 @@ impact: HIGH
 # General component rules
 
 - Use **functional components** with hooks, no classes.
+- Export as `export const ComponentName = ({ ... }: IProps) => …` (arrow + props type). No `FC` / `React.FC`.
 - A component should:
   - stay as "thin" as possible on business logic;
   - delegate state to store/hooks when needed in multiple places;
@@ -16,6 +17,41 @@ impact: HIGH
 - For repeated logic, create `useSomething` hooks nearby or in `src/ui/hooks/**`.
 - When accessing store, API, shared helpers, and utilities, components and hooks must use only their **public API** (for example, `@/store`, `@/store/slices/Foo`, `@/lib/utils`), not internal implementation files of other modules.
 
+# Conditional rendering
+
+- Prefer **decomposition over conditionals**: a component with more than one or two branches (nested ternaries, long `&&` chains, several early returns) should extract each branch into a child under `./components/<Name>/`, keeping the parent a thin switch.
+- Keep the condition in the parent; the child receives resolved props.
+- No nested ternaries in JSX. For not-ready / empty / error, use an early return or a dedicated `EmptyState` / `ErrorState` component.
+- A `variant` / `mode` flag that renders two unrelated layouts is **two** components, not one.
+
+## Incorrect
+
+```tsx
+export const OrderPanel = ({ order }: Props) => (
+  <div>
+    {order == null ? (
+      <Spinner />
+    ) : order.items.length === 0 ? (
+      <p>No items</p>
+    ) : order.status === 'paid' ? (
+      <PaidList items={order.items} />
+    ) : (
+      <UnpaidList items={order.items} />
+    )}
+  </div>
+)
+```
+
+## Correct
+
+```tsx
+export const OrderPanel = ({ order }: Props) => {
+  if (order == null) return <Spinner />
+  if (order.items.length === 0) return <EmptyState />
+  return <OrderItems order={order} />
+}
+```
+
 # File structure
 
 For a page/large block:
@@ -72,7 +108,7 @@ For a reusable component:
 # Props and typing
 
 - **Do not pass props via spread** (`<Foo {...x} />`). Only explicit attributes; details — `ui-and-accessibility/no-props-spread.md` (in `app/src/ui` ESLint catches this too).
-- Describe props with `type Props = { ... }` or `interface Props { ... }`.
+- Describe props with a local `interface` / `type` named as the repo does (`interface IProps` / `type TProps`); match the neighboring component. No `FC` / `React.FC`.
 - Do not use `any`; when needed — generics (`<T>`), `unknown`, type guards.
 - For domain entities use types from `@/types` (and enums from `@/types/enums`), not redefined locally (`architecture/public-imports.md`).
 

package/presets/claude/next/team/fixtures/metrics-scenarios/tasks/human-gate/metrics.json

@@ -0,0 +1,69 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "human-gate-run",
+    "slug": "human-gate",
+    "preset": "ai-rules",
+    "stack": "next",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "human-gate-run:task",
+      "type": "task_started",
+      "at": "2026-07-31T17:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:gate:1",
+      "type": "human_gate_started",
+      "at": "2026-07-31T17:01:00.000Z",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:gate:end",
+      "type": "human_gate_ended",
+      "at": "2026-07-31T17:11:00.000Z",
+      "gateStartedEventId": "human-gate-run:gate:1",
+      "source": "command",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:1:feature-developer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T17:11:00.000Z",
+      "attemptId": "human-gate-run:step-1:feature-developer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "feature-developer",
+      "role": "implementation",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:1:feature-developer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T17:12:00.000Z",
+      "attemptId": "human-gate-run:step-1:feature-developer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:done",
+      "type": "task_completed",
+      "at": "2026-07-31T17:12:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}

package/presets/claude/next/team/fixtures/metrics-scenarios/tasks/missing-ledger/status.json

@@ -0,0 +1,5 @@
+{
+  "slug": "missing-ledger",
+  "state": "completed",
+  "note": "Intentional fixture: metrics.json is absent."
+}

package/presets/claude/next/team/fixtures/metrics-scenarios/tasks/parallel-receipts/metrics.json

@@ -0,0 +1,79 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "parallel-receipts-run",
+    "slug": "parallel-receipts",
+    "preset": "ai-rules",
+    "stack": "next",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "parallel-receipts-run:task",
+      "type": "task_started",
+      "at": "2026-07-31T16:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:code-reviewer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T16:00:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:code-reviewer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "code-reviewer",
+      "role": "review",
+      "attempt": 1,
+      "parallelGroupId": "parallel-receipts-run:step-1",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:security-reviewer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T16:00:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:security-reviewer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "security-reviewer",
+      "role": "review",
+      "attempt": 1,
+      "parallelGroupId": "parallel-receipts-run:step-1",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:security-reviewer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T16:01:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:security-reviewer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "agent",
+      "attributionConfidence": "fallback"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:code-reviewer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T16:02:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:code-reviewer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 120000,
+      "retryAfterFix": null,
+      "source": "agent",
+      "attributionConfidence": "fallback"
+    },
+    {
+      "eventId": "parallel-receipts-run:done",
+      "type": "task_completed",
+      "at": "2026-07-31T16:02:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}

package/presets/claude/next/team/fixtures/metrics-scenarios/tasks/serial-retry/metrics.json

@@ -0,0 +1,124 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "serial-retry-run",
+    "slug": "serial-retry",
+    "preset": "ai-rules",
+    "stack": "next",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "serial-retry-run:task",
+      "type": "task_started",
+      "at": "2026-07-31T14:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T14:00:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-1",
+      "stepIndex": 2,
+      "stepLabel": null,
+      "agent": "code-reviewer",
+      "role": "review",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T14:01:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-1",
+      "outcome": "changes_requested",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:retry:1",
+      "type": "retry_scheduled",
+      "at": "2026-07-31T14:01:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-1",
+      "nextAgent": "feature-developer",
+      "retryAfterFix": "code-reviewer",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:1:feature-developer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T14:01:00.000Z",
+      "attemptId": "serial-retry-run:step-1:feature-developer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "feature-developer",
+      "role": "implementation",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:1:feature-developer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T14:03:00.000Z",
+      "attemptId": "serial-retry-run:step-1:feature-developer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 120000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:retry:2",
+      "type": "retry_scheduled",
+      "at": "2026-07-31T14:03:00.000Z",
+      "attemptId": "serial-retry-run:step-1:feature-developer:attempt-1",
+      "nextAgent": "code-reviewer",
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:2:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T14:03:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-2",
+      "stepIndex": 2,
+      "stepLabel": null,
+      "agent": "code-reviewer",
+      "role": "review",
+      "attempt": 2,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:2:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T14:04:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-2",
+      "outcome": "completed",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:done",
+      "type": "task_completed",
+      "at": "2026-07-31T14:04:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}