@lifeaitools/rdc-skills 0.24.41 → 0.25.0

package/guides/agents/verify.md

@@ -1,151 +1,151 @@
-> **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`
-> Checklist-only output. No tool-call narration. No raw MCP/JSON/log dumps.
-> One checklist upfront, updated in place, shown again at end with a 1-line verdict.
-
-
-# rdc:verify — Verification Before Completion
-
-> Adapted from obra/superpowers `verification-before-completion`.
-> Stack-specific: uses `npx vitest run` + `npx tsc --noEmit`. NEVER `pnpm build` (crashes machine — 800MB/process).
-
-## The Iron Law
-
-**NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE.**
-
-You may not say "done", "complete", "working", "fixed", "passing", or imply success until you have:
-
-1. **Identified** the exact commands that prove the claim
-2. **Run** those commands fresh in the current state (not memory, not "earlier", not "should")
-3. **Read** the full output
-4. **Verified** the output matches the claim
-5. Only THEN made the claim, quoting the evidence
-
-If you cannot run the commands, say so explicitly. Do not substitute reasoning for evidence.
-
-## Red Flags — Stop and Verify
-
-If you're about to write any of these, STOP:
-
-- "should work" / "should pass" / "should be fine"
-- "probably" / "seems to" / "looks good"
-- "I think the tests pass"
-- "the types check out" (without running tsc)
-- "everything compiles" (without fresh output)
-- Expressing satisfaction or wrapping up before the gate ran
-
-## The Gate — Commands for This Stack
-
-Run these in order. All must pass. Capture output.
-
-### 0. Checklist decomposition quality
-
-Before code verification, inspect each work item checklist and the matching plan section.
-
-Required plan evidence:
-- `## Checklist Decomposition Matrix`
-- `## Checklist Quality Gate`
-- `verdict: PASS`
-
-Required work item evidence:
-- At least one `decomp-*` checklist item for every implementation work item
-- At least one `test-*` checklist item for every implementation work item
-- Each `decomp-*` item includes a route or file path, one action, one expected result, and one evidence artifact
-- Each `decomp-*` item is independently pass/fail; vague rows like "UI complete", "theme management works", or "integration verified" are failures
-
-If this check fails, stop validation for that item and reopen it. Do not mark a work item `done` when the checklist itself is not decomposed enough to prove completion.
-
-### 1. Per-package vitest (every package touched)
-
-```bash
-cd <repo-root>
-for pkg in <changed-packages>; do
-  npx vitest run --dir packages/$pkg 2>&1 | tee /tmp/verify-$pkg.log
-done
-```
-
-Or for a single package:
-```bash
-npx vitest run --dir packages/<pkg>
-npx vitest run --dir apps/<app>
-```
-
-**NEVER** run `pnpm build` or `pnpm test` at the repo root — spawns 5-7 node processes at 800MB each and crashes the machine.
-
-### 2. Typecheck per package
-
-```bash
-npx tsc --noEmit --project packages/<pkg>/tsconfig.json
-npx tsc --noEmit --project apps/<app>/tsconfig.json
-```
-
-One package at a time. Read the full output — zero errors required.
-
-### 3. Lint (if configured for the package)
-
-```bash
-npx eslint <paths> --max-warnings=0
-```
-
-### 4. Smoke check exports (if package adds public API)
-
-```bash
-node -e "const m = require('<pkg>'); console.log(Object.keys(m));"
-```
-
-Or for ESM:
-```bash
-node --input-type=module -e "import * as m from '<pkg>'; console.log(Object.keys(m));"
-```
-
-## Rationalization Prevention
-
-| You're tempted to think... | Reality |
-|---|---|
-| "Tests passed earlier, should still pass" | Run them now. Code changed. |
-| "TypeScript is strict, if it built it works" | You didn't build. Run tsc. |
-| "Small change, no need to retest" | Small changes are where regressions hide. |
-| "I'll note it as pending verification" | No. Verify now or mark incomplete. |
-| "The test file looks right" | Reading ≠ running. Run it. |
-
-## Output Format — Required
-
-When reporting completion, use this structure:
-
-```
-## Verification Evidence
-
-### vitest
-$ npx vitest run --dir packages/hail
- Test Files  12 passed (12)
-      Tests  147 passed (147)
-
-### tsc
-$ npx tsc --noEmit --project packages/hail/tsconfig.json
-(no output — clean)
-
-### Status
-Checklist quality: PASS
-PASS — safe to mark tasks done.
-```
-
-If any step fails: do NOT claim completion. Report the failure, fix it, re-run the entire gate.
-
-## When to Invoke
-
-- **End of `/rdc:build`** — mandatory final phase before marking epic done
-- **During `/rdc:review`** — verification gate after fixes applied
-- **Before any "done" declaration** — if an agent or supervisor is about to say work is complete
-- **After merging waves** — when parallel agents finish and their work is combined
-
-## Agent Delegation
-
-When dispatching a verification agent, give them:
-- Exact list of packages/apps to verify
-- Explicit ban on `pnpm build` / `pnpm test` / `pnpm -r`
-- Work item checklists and the matching plan doc so `decomp-*` quality can be verified first
-- Required output format (above)
-- Instruction: "If any command fails, STOP and report. Do not fix. Do not continue."
-
-## The Bottom Line
-
-Evidence before claims. Fresh evidence, not remembered evidence. Full output, not selective reading. If you skip the gate, you are lying about the work being done.
+> **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`
+> Checklist-only output. No tool-call narration. No raw MCP/JSON/log dumps.
+> One checklist upfront, updated in place, shown again at end with a 1-line verdict.
+
+
+# rdc:verify — Verification Before Completion
+
+> Adapted from obra/superpowers `verification-before-completion`.
+> Stack-specific: uses `npx vitest run` + `npx tsc --noEmit`. NEVER `pnpm build` (crashes machine — 800MB/process).
+
+## The Iron Law
+
+**NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE.**
+
+You may not say "done", "complete", "working", "fixed", "passing", or imply success until you have:
+
+1. **Identified** the exact commands that prove the claim
+2. **Run** those commands fresh in the current state (not memory, not "earlier", not "should")
+3. **Read** the full output
+4. **Verified** the output matches the claim
+5. Only THEN made the claim, quoting the evidence
+
+If you cannot run the commands, say so explicitly. Do not substitute reasoning for evidence.
+
+## Red Flags — Stop and Verify
+
+If you're about to write any of these, STOP:
+
+- "should work" / "should pass" / "should be fine"
+- "probably" / "seems to" / "looks good"
+- "I think the tests pass"
+- "the types check out" (without running tsc)
+- "everything compiles" (without fresh output)
+- Expressing satisfaction or wrapping up before the gate ran
+
+## The Gate — Commands for This Stack
+
+Run these in order. All must pass. Capture output.
+
+### 0. Checklist decomposition quality
+
+Before code verification, inspect each work item checklist and the matching plan section.
+
+Required plan evidence:
+- `## Checklist Decomposition Matrix`
+- `## Checklist Quality Gate`
+- `verdict: PASS`
+
+Required work item evidence:
+- At least one `decomp-*` checklist item for every implementation work item
+- At least one `test-*` checklist item for every implementation work item
+- Each `decomp-*` item includes a route or file path, one action, one expected result, and one evidence artifact
+- Each `decomp-*` item is independently pass/fail; vague rows like "UI complete", "theme management works", or "integration verified" are failures
+
+If this check fails, stop validation for that item and reopen it. Do not mark a work item `done` when the checklist itself is not decomposed enough to prove completion.
+
+### 1. Per-package vitest (every package touched)
+
+```bash
+cd <repo-root>
+for pkg in <changed-packages>; do
+  npx vitest run --dir packages/$pkg 2>&1 | tee /tmp/verify-$pkg.log
+done
+```
+
+Or for a single package:
+```bash
+npx vitest run --dir packages/<pkg>
+npx vitest run --dir apps/<app>
+```
+
+**NEVER** run `pnpm build` or `pnpm test` at the repo root — spawns 5-7 node processes at 800MB each and crashes the machine.
+
+### 2. Typecheck per package
+
+```bash
+npx tsc --noEmit --project packages/<pkg>/tsconfig.json
+npx tsc --noEmit --project apps/<app>/tsconfig.json
+```
+
+One package at a time. Read the full output — zero errors required.
+
+### 3. Lint (if configured for the package)
+
+```bash
+npx eslint <paths> --max-warnings=0
+```
+
+### 4. Smoke check exports (if package adds public API)
+
+```bash
+node -e "const m = require('<pkg>'); console.log(Object.keys(m));"
+```
+
+Or for ESM:
+```bash
+node --input-type=module -e "import * as m from '<pkg>'; console.log(Object.keys(m));"
+```
+
+## Rationalization Prevention
+
+| You're tempted to think... | Reality |
+|---|---|
+| "Tests passed earlier, should still pass" | Run them now. Code changed. |
+| "TypeScript is strict, if it built it works" | You didn't build. Run tsc. |
+| "Small change, no need to retest" | Small changes are where regressions hide. |
+| "I'll note it as pending verification" | No. Verify now or mark incomplete. |
+| "The test file looks right" | Reading ≠ running. Run it. |
+
+## Output Format — Required
+
+When reporting completion, use this structure:
+
+```
+## Verification Evidence
+
+### vitest
+$ npx vitest run --dir packages/hail
+ Test Files  12 passed (12)
+      Tests  147 passed (147)
+
+### tsc
+$ npx tsc --noEmit --project packages/hail/tsconfig.json
+(no output — clean)
+
+### Status
+Checklist quality: PASS
+PASS — safe to mark tasks done.
+```
+
+If any step fails: do NOT claim completion. Report the failure, fix it, re-run the entire gate.
+
+## When to Invoke
+
+- **End of `/rdc:build`** — mandatory final phase before marking epic done
+- **During `/rdc:review`** — verification gate after fixes applied
+- **Before any "done" declaration** — if an agent or supervisor is about to say work is complete
+- **After merging waves** — when parallel agents finish and their work is combined
+
+## Agent Delegation
+
+When dispatching a verification agent, give them:
+- Exact list of packages/apps to verify
+- Explicit ban on `pnpm build` / `pnpm test` / `pnpm -r`
+- Work item checklists and the matching plan doc so `decomp-*` quality can be verified first
+- Required output format (above)
+- Instruction: "If any command fails, STOP and report. Do not fix. Do not continue."
+
+## The Bottom Line
+
+Evidence before claims. Fresh evidence, not remembered evidence. Full output, not selective reading. If you skip the gate, you are lying about the work being done.

package/guides/agents/viz.md

@@ -1,106 +1,106 @@
-> **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`
-> Checklist-only output. No tool-call narration. No raw MCP/JSON/log dumps.
-> One checklist upfront, updated in place, shown again at end with a 1-line verdict.
-
-> If dispatching subagents or running as a subagent: read `{PROJECT_ROOT}/.rdc/guides/agent-bootstrap.md` first (fallback: `{PROJECT_ROOT}/.rdc/guides/agent-bootstrap.md`).
-
-
-# rdc:viz — Visualization Agent
-
-## Mandatory First Steps
-
-Read BOTH guides before ANY code:
-```
-{PROJECT_ROOT}/.rdc/guides/frontend.md (fallback: {PROJECT_ROOT}/.rdc/guides/frontend.md)
-{PROJECT_ROOT}/.rdc/guides/design.md (fallback: {PROJECT_ROOT}/.rdc/guides/design.md)
-```
-
-## Before Writing Any Code
-
-1. **Check for existing prototypes:**
-   ```sql
-   SELECT name, component, source_path, status, notes
-   FROM prototype_registry
-   WHERE status IN ('prototype', 'converting')
-   ORDER BY created_at DESC;
-   ```
-   Source path is usually `docs/source/`. **Read it first.**
-
-2. **Check design decisions:**
-   ```sql
-   SELECT topic, summary FROM design_context
-   WHERE topic ILIKE '%<viz-name>%';
-   ```
-
-## SVG Layout Patterns
-
-When building new SVG-based viz:
-- Define all layout constants at top (`W`, `H`, column x-centers, box dimensions)
-- Build path builders as pure functions
-- Separate data from rendering
-- Use `viewBox` for responsive scaling
-- Animate with appropriate motion libraries
-
-```tsx
-const W = 980, H = 570;
-const COL_A = 80, COL_B = 240, COL_C = 435;
-
-function pathTo(x1: number, y1: number, x2: number, y2: number): string {
-  const mx = (x1 + x2) / 2;
-  return `M ${x1} ${y1} C ${mx} ${y1} ${mx} ${y2} ${x2} ${y2}`;
-}
-```
-
-## Viz Component Contract
-
-```tsx
-"use client";
-import * as React from "react";
-import { cva, type VariantProps } from "class-variance-authority";
-import { cn } from "../lib/utils";
-
-// CVA variants required on every new viz component
-const myVizVariants = cva("relative w-full", {
-  variants: {
-    layout: { single: "", double: "flex flex-col", grid: "" },
-    density: { compact: "", comfortable: "", spacious: "" },
-  },
-  defaultVariants: { layout: "single", density: "comfortable" },
-});
-
-interface MyVizProps
-  extends React.HTMLAttributes<HTMLDivElement>,
-    VariantProps<typeof myVizVariants> {}
-
-const MyViz = React.forwardRef<HTMLDivElement, MyVizProps>(
-  ({ className, layout, density, ...props }, ref) => (
-    <div ref={ref} className={cn(myVizVariants({ layout, density }), className)} {...props} />
-  )
-);
-MyViz.displayName = "MyViz";
-export { MyViz, myVizVariants };
-```
-
-Export from package's index after building.
-
-## Registering New Viz Components
-
-After completing:
-```sql
-INSERT INTO prototype_registry (name, component, source_path, notes, created_by)
-VALUES (
-  'MyViz v1.0',
-  'MyViz',
-  'packages/ui/src/components/my-viz.tsx',
-  'Production component. Props: <list key props>',
-  'agent'
-);
-```
-
-## Safety Rules
-
-- Branch: development branch — auto-commit
-- NEVER run `pnpm build`
-- Reduce motion: always wrap animations with appropriate hooks
-- NEVER overlap with other agents on the same component file
-- Write tests FIRST — red → implement → green
+> **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`
+> Checklist-only output. No tool-call narration. No raw MCP/JSON/log dumps.
+> One checklist upfront, updated in place, shown again at end with a 1-line verdict.
+
+> If dispatching subagents or running as a subagent: read `{PROJECT_ROOT}/.rdc/guides/agent-bootstrap.md` first (fallback: `{PROJECT_ROOT}/.rdc/guides/agent-bootstrap.md`).
+
+
+# rdc:viz — Visualization Agent
+
+## Mandatory First Steps
+
+Read BOTH guides before ANY code:
+```
+{PROJECT_ROOT}/.rdc/guides/frontend.md (fallback: {PROJECT_ROOT}/.rdc/guides/frontend.md)
+{PROJECT_ROOT}/.rdc/guides/design.md (fallback: {PROJECT_ROOT}/.rdc/guides/design.md)
+```
+
+## Before Writing Any Code
+
+1. **Check for existing prototypes:**
+   ```sql
+   SELECT name, component, source_path, status, notes
+   FROM prototype_registry
+   WHERE status IN ('prototype', 'converting')
+   ORDER BY created_at DESC;
+   ```
+   Source path is usually `docs/source/`. **Read it first.**
+
+2. **Check design decisions:**
+   ```sql
+   SELECT topic, summary FROM design_context
+   WHERE topic ILIKE '%<viz-name>%';
+   ```
+
+## SVG Layout Patterns
+
+When building new SVG-based viz:
+- Define all layout constants at top (`W`, `H`, column x-centers, box dimensions)
+- Build path builders as pure functions
+- Separate data from rendering
+- Use `viewBox` for responsive scaling
+- Animate with appropriate motion libraries
+
+```tsx
+const W = 980, H = 570;
+const COL_A = 80, COL_B = 240, COL_C = 435;
+
+function pathTo(x1: number, y1: number, x2: number, y2: number): string {
+  const mx = (x1 + x2) / 2;
+  return `M ${x1} ${y1} C ${mx} ${y1} ${mx} ${y2} ${x2} ${y2}`;
+}
+```
+
+## Viz Component Contract
+
+```tsx
+"use client";
+import * as React from "react";
+import { cva, type VariantProps } from "class-variance-authority";
+import { cn } from "../lib/utils";
+
+// CVA variants required on every new viz component
+const myVizVariants = cva("relative w-full", {
+  variants: {
+    layout: { single: "", double: "flex flex-col", grid: "" },
+    density: { compact: "", comfortable: "", spacious: "" },
+  },
+  defaultVariants: { layout: "single", density: "comfortable" },
+});
+
+interface MyVizProps
+  extends React.HTMLAttributes<HTMLDivElement>,
+    VariantProps<typeof myVizVariants> {}
+
+const MyViz = React.forwardRef<HTMLDivElement, MyVizProps>(
+  ({ className, layout, density, ...props }, ref) => (
+    <div ref={ref} className={cn(myVizVariants({ layout, density }), className)} {...props} />
+  )
+);
+MyViz.displayName = "MyViz";
+export { MyViz, myVizVariants };
+```
+
+Export from package's index after building.
+
+## Registering New Viz Components
+
+After completing:
+```sql
+INSERT INTO prototype_registry (name, component, source_path, notes, created_by)
+VALUES (
+  'MyViz v1.0',
+  'MyViz',
+  'packages/ui/src/components/my-viz.tsx',
+  'Production component. Props: <list key props>',
+  'agent'
+);
+```
+
+## Safety Rules
+
+- Branch: development branch — auto-commit
+- NEVER run `pnpm build`
+- Reduce motion: always wrap animations with appropriate hooks
+- NEVER overlap with other agents on the same component file
+- Write tests FIRST — red → implement → green