@lifeaitools/rdc-skills 0.21.1 → 0.23.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.claude-plugin/plugin.json +1 -1
- package/.github/workflows/self-test.yml +34 -34
- package/commands/build.md +181 -181
- package/commands/collab.md +180 -180
- package/commands/deploy.md +148 -148
- package/commands/fixit.md +105 -105
- package/commands/handoff.md +173 -173
- package/commands/overnight.md +218 -218
- package/commands/plan.md +158 -158
- package/commands/preplan.md +131 -131
- package/commands/prototype.md +145 -145
- package/commands/report.md +99 -99
- package/commands/review.md +120 -120
- package/commands/status.md +86 -86
- package/commands/workitems.md +127 -127
- package/git-sha.json +1 -1
- package/guides/agent-bootstrap.md +195 -195
- package/guides/agents/backend.md +102 -102
- package/guides/agents/content.md +94 -94
- package/guides/agents/cs2.md +56 -56
- package/guides/agents/data.md +86 -86
- package/guides/agents/design.md +77 -77
- package/guides/agents/frontend.md +91 -91
- package/guides/agents/infrastructure.md +81 -81
- package/guides/agents/setup.md +272 -272
- package/guides/agents/verify.md +119 -119
- package/guides/agents/viz.md +106 -106
- package/package.json +1 -1
- package/scripts/self-test.mjs +1458 -1458
- package/skills/build/SKILL.md +517 -478
- package/skills/collab/SKILL.md +239 -239
- package/skills/deploy/SKILL.md +541 -522
- package/skills/design/SKILL.md +205 -205
- package/skills/fixit/SKILL.md +165 -165
- package/skills/fs-mcp/SKILL.md +17 -0
- package/skills/handoff/SKILL.md +200 -200
- package/skills/housekeeping/SKILL.md +14 -1
- package/skills/overnight/SKILL.md +251 -230
- package/skills/plan/SKILL.md +314 -274
- package/skills/preplan/SKILL.md +90 -90
- package/skills/prototype/SKILL.md +150 -150
- package/skills/release/SKILL.md +140 -140
- package/skills/report/SKILL.md +100 -100
- package/skills/review/SKILL.md +152 -152
- package/skills/rpms-filemap/SKILL.cloud.md +107 -0
- package/skills/rpms-filemap/SKILL.md +107 -0
- package/skills/self-test/SKILL.md +123 -123
- package/skills/status/SKILL.md +99 -99
- package/skills/watch/SKILL.md +90 -90
- package/skills/workitems/SKILL.md +151 -151
package/skills/review/SKILL.md
CHANGED
|
@@ -1,152 +1,152 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: rdc:review
|
|
3
|
-
description: "Usage `rdc:review [--unattended]` — Post-build quality gate: tsc, tests, stale docs, export conflicts, and a mandatory pr-review-toolkit:code-reviewer pass across modified packages. Fixes what it can automatically, escalates the rest. Call after rdc:build and before merging to main."
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
> **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`
|
|
7
|
-
> Checklist-only output. No tool-call narration. No raw MCP/JSON/log dumps.
|
|
8
|
-
> One checklist upfront, updated in place, shown again at end with a 1-line verdict.
|
|
9
|
-
|
|
10
|
-
> 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`), then `{PROJECT_ROOT}/.rdc/guides/engineering-behavior.md` (fallback: `{PROJECT_ROOT}/.rdc/guides/engineering-behavior.md`).
|
|
11
|
-
|
|
12
|
-
> **Sandbox contract:** This skill honors `RDC_TEST=1` per `guides/agent-bootstrap.md` § RDC_TEST Sandbox Contract. Destructive external calls short-circuit under the flag.
|
|
13
|
-
|
|
14
|
-
|
|
15
|
-
# rdc:review — Quality Gate
|
|
16
|
-
|
|
17
|
-
## When to Use
|
|
18
|
-
- After a build session (especially overnight builds)
|
|
19
|
-
- Before merging development → main/production
|
|
20
|
-
- Project lead asks "review the work", "is everything clean"
|
|
21
|
-
- Before any production deployment
|
|
22
|
-
- Called by `rdc:overnight` after each epic build completes
|
|
23
|
-
|
|
24
|
-
## Arguments
|
|
25
|
-
- `rdc:review` — interactive review, pauses on issues needing judgment
|
|
26
|
-
- `rdc:review --unattended` — silent mode, auto-fixes everything fixable
|
|
27
|
-
|
|
28
|
-
## Procedure
|
|
29
|
-
|
|
30
|
-
1. **Identify modified packages:**
|
|
31
|
-
```bash
|
|
32
|
-
git diff --name-only origin/main...HEAD | grep "^packages/" | cut -d/ -f2 | sort -u
|
|
33
|
-
```
|
|
34
|
-
|
|
35
|
-
2. **Run tests for each modified package:**
|
|
36
|
-
```bash
|
|
37
|
-
cd packages/<name> && npx vitest run 2>&1 | tail -10
|
|
38
|
-
```
|
|
39
|
-
Report: package → test count → pass/fail → new tests added
|
|
40
|
-
|
|
41
|
-
**IMPORTANT:** `pnpm build` must NEVER be run (crashes system). Use `npx tsc --noEmit --project <path>/tsconfig.json` for typecheck instead. For packages without tests, typecheck is the verification method. Do NOT run vitest across the entire monorepo — check only modified packages individually.
|
|
42
|
-
|
|
43
|
-
3. **Check test coverage delta:**
|
|
44
|
-
```bash
|
|
45
|
-
git diff origin/main...HEAD -- packages/*/src/ | grep -c "^+" | head -5
|
|
46
|
-
git diff origin/main...HEAD -- packages/*/test* packages/*/src/**/*.test.* packages/*/src/**/*.spec.* 2>/dev/null | grep -c "^+" || echo 0
|
|
47
|
-
```
|
|
48
|
-
Flag any package where implementation lines added > 50 but test lines added = 0.
|
|
49
|
-
|
|
50
|
-
4. **Check for export conflicts:**
|
|
51
|
-
- Read `packages/*/src/index.ts` for any package with new exports
|
|
52
|
-
- Look for duplicate export names across the barrel
|
|
53
|
-
- Verify aliased exports don't shadow each other
|
|
54
|
-
|
|
55
|
-
5. **Check for TODO/FIXME/HACK:**
|
|
56
|
-
```bash
|
|
57
|
-
grep -rn "TODO\|FIXME\|HACK\|XXX" packages/*/src/ --include="*.ts" --include="*.tsx"
|
|
58
|
-
```
|
|
59
|
-
|
|
60
|
-
6. **Check package versions:**
|
|
61
|
-
- Any package with significant new code should have a version bump
|
|
62
|
-
- Compare package.json versions to what's in `docs/SYSTEM-STATE.md`
|
|
63
|
-
|
|
64
|
-
7. **Check for stale CLAUDE.md:**
|
|
65
|
-
- If new modules were added to a package, does its CLAUDE.md mention them?
|
|
66
|
-
- Flag any package where exports grew by >10 lines but CLAUDE.md wasn't updated
|
|
67
|
-
|
|
68
|
-
8. **Orphan work item audit:**
|
|
69
|
-
```sql
|
|
70
|
-
SELECT id, title, item_type, status, source, created_at::date
|
|
71
|
-
FROM work_items
|
|
72
|
-
WHERE parent_id IS NULL
|
|
73
|
-
AND item_type NOT IN ('epic', 'bug')
|
|
74
|
-
AND status NOT IN ('done', 'archived')
|
|
75
|
-
ORDER BY created_at DESC;
|
|
76
|
-
```
|
|
77
|
-
For each orphaned task found:
|
|
78
|
-
- If it clearly belongs to an open epic → **do not run raw SQL**. No RPC exists for re-parenting. Create a work item: `SELECT insert_work_item(p_title := 'Re-parent orphaned task <task-id> to epic <epic-id>', p_priority := 'normal', p_source := 'agent')` and flag for supervisor to re-parent manually.
|
|
79
|
-
- If unclear → report (interactive) or flag in REVIEW_STATUS (unattended)
|
|
80
|
-
- Never silently leave orphaned tasks
|
|
81
|
-
- **⛔ Raw `UPDATE work_items SET parent_id = ...` is forbidden** — bypasses RLS and all constraint checks
|
|
82
|
-
|
|
83
|
-
8b. **Mandatory code-review gate — dispatch pr-review-toolkit:code-reviewer:**
|
|
84
|
-
|
|
85
|
-
⛔ **No CLEAN verdict without a code-review pass.** This is the second half of the quality gate; tsc/vitest catch type and behavior errors, code-reviewer catches logic, security, and convention drift.
|
|
86
|
-
|
|
87
|
-
Dispatch ONE `pr-review-toolkit:code-reviewer` agent on the full diff under review:
|
|
88
|
-
|
|
89
|
-
```
|
|
90
|
-
Agent({
|
|
91
|
-
subagent_type: "pr-review-toolkit:code-reviewer",
|
|
92
|
-
description: "rdc:review code-review pass",
|
|
93
|
-
prompt: "Review `git diff origin/main...HEAD` on the development branch.
|
|
94
|
-
Focus on: bugs, logic errors, security vulnerabilities, project-convention
|
|
95
|
-
adherence (.claude/rules/*, CLAUDE.md, engineering-behavior.md).
|
|
96
|
-
Confidence-based filtering — high-confidence findings only.
|
|
97
|
-
Return CODE_REVIEW_COMPLETE with: { critical_count, high_count, medium_count,
|
|
98
|
-
low_count, findings: [{severity, file:line, issue, suggested_fix}] }."
|
|
99
|
-
})
|
|
100
|
-
```
|
|
101
|
-
|
|
102
|
-
**Severity gate:**
|
|
103
|
-
- `critical` or `high` → verdict cannot be CLEAN. Interactive: report and pause. Unattended: auto-fix if mechanical (rename, missing await, obvious null check), otherwise escalate via advisor and mark HAS_ISSUES.
|
|
104
|
-
- `medium` or `low` → record in the review report's "Issues" section; verdict can still be CLEAN.
|
|
105
|
-
- Zero findings → log `CODE_REVIEW: CLEAN` and continue.
|
|
106
|
-
|
|
107
|
-
Under `RDC_TEST=1`: echo `[RDC_TEST] skipping code-review dispatch` and continue.
|
|
108
|
-
|
|
109
|
-
9. **Verification gate — dispatch the verify agent:**
|
|
110
|
-
After any fixes land, run the verify gate on every touched package. See `guides/agents/verify.md`.
|
|
111
|
-
Apply `guides/engineering-behavior.md` while reviewing: flag unnecessary abstraction, drive-by refactors, missing assumptions, hidden uncertainty, out-of-scope edits, and prose-only verification.
|
|
112
|
-
**Iron Law: no CLEAN verdict without fresh evidence.** Quote the checklist decomposition verdict, vitest output, and tsc output in the report.
|
|
113
|
-
The verify agent must reject any work item that lacks passed `decomp-*` checklist rows or whose rows are too coarse to prove one observable behavior at a time.
|
|
114
|
-
If verify fails → do NOT emit CLEAN. Loop back, fix, re-run verify.
|
|
115
|
-
|
|
116
|
-
10. **Fix issues found:**
|
|
117
|
-
- Failing tests → fix and commit
|
|
118
|
-
- Export conflicts → resolve and commit
|
|
119
|
-
- Missing version bumps → bump and commit
|
|
120
|
-
- All fixes as separate commits with descriptive messages
|
|
121
|
-
|
|
122
|
-
**Judgment calls:**
|
|
123
|
-
- Interactive: report — don't guess
|
|
124
|
-
- Unattended: escalate via advisor tool with: error message, surrounding context,
|
|
125
|
-
two most likely fix paths. Resume with advisor's recommendation.
|
|
126
|
-
If advisor unavailable: take the most conservative path, flag in status block.
|
|
127
|
-
|
|
128
|
-
11. **Report:**
|
|
129
|
-
- Interactive:
|
|
130
|
-
```
|
|
131
|
-
## Review Results
|
|
132
|
-
| Package | Tests | Pass/Fail | New Tests | Issues |
|
|
133
|
-
## Fixed
|
|
134
|
-
## Remaining Issues
|
|
135
|
-
## Verdict: CLEAN / HAS ISSUES
|
|
136
|
-
```
|
|
137
|
-
- Unattended: emit status block only:
|
|
138
|
-
```
|
|
139
|
-
REVIEW_STATUS: { verdict: "CLEAN|HAS_ISSUES", packages_checked, tests_passed, tests_failed, new_tests_added, fixes_applied, escalations }
|
|
140
|
-
```
|
|
141
|
-
|
|
142
|
-
## Rules
|
|
143
|
-
- Do NOT run `pnpm build` (crashes system) — vitest only
|
|
144
|
-
- Interactive: fix what you can, flag what needs decision
|
|
145
|
-
- Unattended: fix everything fixable; escalate judgment calls to advisor
|
|
146
|
-
- Each fix is a separate commit (not batched)
|
|
147
|
-
- Always push fixes to origin after committing *(skip if `$RDC_TEST=1` — echo `[RDC_TEST] skipping git push` instead)*
|
|
148
|
-
- Unattended: NEVER pause for input
|
|
149
|
-
|
|
150
|
-
## Capture lessons (exit step)
|
|
151
|
-
|
|
152
|
-
Before the final verdict line, follow `.rdc/guides/lessons-learned-spec.md` § Capture procedure. If this run taught something non-obvious — a first root-cause theory that turned out wrong, the documented/standard path not working, a missing gate or check that cost a round, or a surprising tool/infra behavior — write one `.rdc/lessons/<YYYY-MM-DD>-review-<short-slug>.md` per lesson using the schema in that spec. Set `scope` (`simple` | `architectural`) and `status` (`open`, or `applied` if you shipped the fix in this same run, with the commit linked). Commit the lesson file(s) on `develop` alongside the run's other commits, and note "N lessons captured" in your verdict/summary. A run that taught nothing writes nothing — absence is the default.
|
|
1
|
+
---
|
|
2
|
+
name: rdc:review
|
|
3
|
+
description: "Usage `rdc:review [--unattended]` — Post-build quality gate: tsc, tests, stale docs, export conflicts, and a mandatory pr-review-toolkit:code-reviewer pass across modified packages. Fixes what it can automatically, escalates the rest. Call after rdc:build and before merging to main."
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
> **⚠️ OUTPUT CONTRACT (READ FIRST):** `guides/output-contract.md`
|
|
7
|
+
> Checklist-only output. No tool-call narration. No raw MCP/JSON/log dumps.
|
|
8
|
+
> One checklist upfront, updated in place, shown again at end with a 1-line verdict.
|
|
9
|
+
|
|
10
|
+
> 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`), then `{PROJECT_ROOT}/.rdc/guides/engineering-behavior.md` (fallback: `{PROJECT_ROOT}/.rdc/guides/engineering-behavior.md`).
|
|
11
|
+
|
|
12
|
+
> **Sandbox contract:** This skill honors `RDC_TEST=1` per `guides/agent-bootstrap.md` § RDC_TEST Sandbox Contract. Destructive external calls short-circuit under the flag.
|
|
13
|
+
|
|
14
|
+
|
|
15
|
+
# rdc:review — Quality Gate
|
|
16
|
+
|
|
17
|
+
## When to Use
|
|
18
|
+
- After a build session (especially overnight builds)
|
|
19
|
+
- Before merging development → main/production
|
|
20
|
+
- Project lead asks "review the work", "is everything clean"
|
|
21
|
+
- Before any production deployment
|
|
22
|
+
- Called by `rdc:overnight` after each epic build completes
|
|
23
|
+
|
|
24
|
+
## Arguments
|
|
25
|
+
- `rdc:review` — interactive review, pauses on issues needing judgment
|
|
26
|
+
- `rdc:review --unattended` — silent mode, auto-fixes everything fixable
|
|
27
|
+
|
|
28
|
+
## Procedure
|
|
29
|
+
|
|
30
|
+
1. **Identify modified packages:**
|
|
31
|
+
```bash
|
|
32
|
+
git diff --name-only origin/main...HEAD | grep "^packages/" | cut -d/ -f2 | sort -u
|
|
33
|
+
```
|
|
34
|
+
|
|
35
|
+
2. **Run tests for each modified package:**
|
|
36
|
+
```bash
|
|
37
|
+
cd packages/<name> && npx vitest run 2>&1 | tail -10
|
|
38
|
+
```
|
|
39
|
+
Report: package → test count → pass/fail → new tests added
|
|
40
|
+
|
|
41
|
+
**IMPORTANT:** `pnpm build` must NEVER be run (crashes system). Use `npx tsc --noEmit --project <path>/tsconfig.json` for typecheck instead. For packages without tests, typecheck is the verification method. Do NOT run vitest across the entire monorepo — check only modified packages individually.
|
|
42
|
+
|
|
43
|
+
3. **Check test coverage delta:**
|
|
44
|
+
```bash
|
|
45
|
+
git diff origin/main...HEAD -- packages/*/src/ | grep -c "^+" | head -5
|
|
46
|
+
git diff origin/main...HEAD -- packages/*/test* packages/*/src/**/*.test.* packages/*/src/**/*.spec.* 2>/dev/null | grep -c "^+" || echo 0
|
|
47
|
+
```
|
|
48
|
+
Flag any package where implementation lines added > 50 but test lines added = 0.
|
|
49
|
+
|
|
50
|
+
4. **Check for export conflicts:**
|
|
51
|
+
- Read `packages/*/src/index.ts` for any package with new exports
|
|
52
|
+
- Look for duplicate export names across the barrel
|
|
53
|
+
- Verify aliased exports don't shadow each other
|
|
54
|
+
|
|
55
|
+
5. **Check for TODO/FIXME/HACK:**
|
|
56
|
+
```bash
|
|
57
|
+
grep -rn "TODO\|FIXME\|HACK\|XXX" packages/*/src/ --include="*.ts" --include="*.tsx"
|
|
58
|
+
```
|
|
59
|
+
|
|
60
|
+
6. **Check package versions:**
|
|
61
|
+
- Any package with significant new code should have a version bump
|
|
62
|
+
- Compare package.json versions to what's in `docs/SYSTEM-STATE.md`
|
|
63
|
+
|
|
64
|
+
7. **Check for stale CLAUDE.md:**
|
|
65
|
+
- If new modules were added to a package, does its CLAUDE.md mention them?
|
|
66
|
+
- Flag any package where exports grew by >10 lines but CLAUDE.md wasn't updated
|
|
67
|
+
|
|
68
|
+
8. **Orphan work item audit:**
|
|
69
|
+
```sql
|
|
70
|
+
SELECT id, title, item_type, status, source, created_at::date
|
|
71
|
+
FROM work_items
|
|
72
|
+
WHERE parent_id IS NULL
|
|
73
|
+
AND item_type NOT IN ('epic', 'bug')
|
|
74
|
+
AND status NOT IN ('done', 'archived')
|
|
75
|
+
ORDER BY created_at DESC;
|
|
76
|
+
```
|
|
77
|
+
For each orphaned task found:
|
|
78
|
+
- If it clearly belongs to an open epic → **do not run raw SQL**. No RPC exists for re-parenting. Create a work item: `SELECT insert_work_item(p_title := 'Re-parent orphaned task <task-id> to epic <epic-id>', p_priority := 'normal', p_source := 'agent')` and flag for supervisor to re-parent manually.
|
|
79
|
+
- If unclear → report (interactive) or flag in REVIEW_STATUS (unattended)
|
|
80
|
+
- Never silently leave orphaned tasks
|
|
81
|
+
- **⛔ Raw `UPDATE work_items SET parent_id = ...` is forbidden** — bypasses RLS and all constraint checks
|
|
82
|
+
|
|
83
|
+
8b. **Mandatory code-review gate — dispatch pr-review-toolkit:code-reviewer:**
|
|
84
|
+
|
|
85
|
+
⛔ **No CLEAN verdict without a code-review pass.** This is the second half of the quality gate; tsc/vitest catch type and behavior errors, code-reviewer catches logic, security, and convention drift.
|
|
86
|
+
|
|
87
|
+
Dispatch ONE `pr-review-toolkit:code-reviewer` agent on the full diff under review:
|
|
88
|
+
|
|
89
|
+
```
|
|
90
|
+
Agent({
|
|
91
|
+
subagent_type: "pr-review-toolkit:code-reviewer",
|
|
92
|
+
description: "rdc:review code-review pass",
|
|
93
|
+
prompt: "Review `git diff origin/main...HEAD` on the development branch.
|
|
94
|
+
Focus on: bugs, logic errors, security vulnerabilities, project-convention
|
|
95
|
+
adherence (.claude/rules/*, CLAUDE.md, engineering-behavior.md).
|
|
96
|
+
Confidence-based filtering — high-confidence findings only.
|
|
97
|
+
Return CODE_REVIEW_COMPLETE with: { critical_count, high_count, medium_count,
|
|
98
|
+
low_count, findings: [{severity, file:line, issue, suggested_fix}] }."
|
|
99
|
+
})
|
|
100
|
+
```
|
|
101
|
+
|
|
102
|
+
**Severity gate:**
|
|
103
|
+
- `critical` or `high` → verdict cannot be CLEAN. Interactive: report and pause. Unattended: auto-fix if mechanical (rename, missing await, obvious null check), otherwise escalate via advisor and mark HAS_ISSUES.
|
|
104
|
+
- `medium` or `low` → record in the review report's "Issues" section; verdict can still be CLEAN.
|
|
105
|
+
- Zero findings → log `CODE_REVIEW: CLEAN` and continue.
|
|
106
|
+
|
|
107
|
+
Under `RDC_TEST=1`: echo `[RDC_TEST] skipping code-review dispatch` and continue.
|
|
108
|
+
|
|
109
|
+
9. **Verification gate — dispatch the verify agent:**
|
|
110
|
+
After any fixes land, run the verify gate on every touched package. See `guides/agents/verify.md`.
|
|
111
|
+
Apply `guides/engineering-behavior.md` while reviewing: flag unnecessary abstraction, drive-by refactors, missing assumptions, hidden uncertainty, out-of-scope edits, and prose-only verification.
|
|
112
|
+
**Iron Law: no CLEAN verdict without fresh evidence.** Quote the checklist decomposition verdict, vitest output, and tsc output in the report.
|
|
113
|
+
The verify agent must reject any work item that lacks passed `decomp-*` checklist rows or whose rows are too coarse to prove one observable behavior at a time.
|
|
114
|
+
If verify fails → do NOT emit CLEAN. Loop back, fix, re-run verify.
|
|
115
|
+
|
|
116
|
+
10. **Fix issues found:**
|
|
117
|
+
- Failing tests → fix and commit
|
|
118
|
+
- Export conflicts → resolve and commit
|
|
119
|
+
- Missing version bumps → bump and commit
|
|
120
|
+
- All fixes as separate commits with descriptive messages
|
|
121
|
+
|
|
122
|
+
**Judgment calls:**
|
|
123
|
+
- Interactive: report — don't guess
|
|
124
|
+
- Unattended: escalate via advisor tool with: error message, surrounding context,
|
|
125
|
+
two most likely fix paths. Resume with advisor's recommendation.
|
|
126
|
+
If advisor unavailable: take the most conservative path, flag in status block.
|
|
127
|
+
|
|
128
|
+
11. **Report:**
|
|
129
|
+
- Interactive:
|
|
130
|
+
```
|
|
131
|
+
## Review Results
|
|
132
|
+
| Package | Tests | Pass/Fail | New Tests | Issues |
|
|
133
|
+
## Fixed
|
|
134
|
+
## Remaining Issues
|
|
135
|
+
## Verdict: CLEAN / HAS ISSUES
|
|
136
|
+
```
|
|
137
|
+
- Unattended: emit status block only:
|
|
138
|
+
```
|
|
139
|
+
REVIEW_STATUS: { verdict: "CLEAN|HAS_ISSUES", packages_checked, tests_passed, tests_failed, new_tests_added, fixes_applied, escalations }
|
|
140
|
+
```
|
|
141
|
+
|
|
142
|
+
## Rules
|
|
143
|
+
- Do NOT run `pnpm build` (crashes system) — vitest only
|
|
144
|
+
- Interactive: fix what you can, flag what needs decision
|
|
145
|
+
- Unattended: fix everything fixable; escalate judgment calls to advisor
|
|
146
|
+
- Each fix is a separate commit (not batched)
|
|
147
|
+
- Always push fixes to origin after committing *(skip if `$RDC_TEST=1` — echo `[RDC_TEST] skipping git push` instead)*
|
|
148
|
+
- Unattended: NEVER pause for input
|
|
149
|
+
|
|
150
|
+
## Capture lessons (exit step)
|
|
151
|
+
|
|
152
|
+
Before the final verdict line, follow `.rdc/guides/lessons-learned-spec.md` § Capture procedure. If this run taught something non-obvious — a first root-cause theory that turned out wrong, the documented/standard path not working, a missing gate or check that cost a round, or a surprising tool/infra behavior — write one `.rdc/lessons/<YYYY-MM-DD>-review-<short-slug>.md` per lesson using the schema in that spec. Set `scope` (`simple` | `architectural`) and `status` (`open`, or `applied` if you shipped the fix in this same run, with the commit linked). Commit the lesson file(s) on `develop` alongside the run's other commits, and note "N lessons captured" in your verdict/summary. A run that taught nothing writes nothing — absence is the default.
|
|
@@ -0,0 +1,107 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: "Generated RPMS file map — RULE #1, canonical homes, and Context Export pointers served from regen-root manifest."
|
|
3
|
+
slash: "rdc:rpms-filemap"
|
|
4
|
+
category: "tooling"
|
|
5
|
+
usage: "rdc:rpms-filemap"
|
|
6
|
+
requires: []
|
|
7
|
+
triggers:
|
|
8
|
+
- "rpms file map"
|
|
9
|
+
- "rule #1"
|
|
10
|
+
- "context export"
|
|
11
|
+
- "where does this file belong"
|
|
12
|
+
- "where should pm artifacts go"
|
|
13
|
+
---
|
|
14
|
+
# RPMS File Map
|
|
15
|
+
> GENERATED FILE - DO NOT HAND-EDIT.
|
|
16
|
+
> Source of truth: `docs/architecture/rpms.locations.json`
|
|
17
|
+
> Regenerate: `pnpm rpms:gen-filemap`
|
|
18
|
+
|
|
19
|
+
## RULE #1
|
|
20
|
+
|
|
21
|
+
**Name the entity -> read its `_context.md` -> you are oriented.**
|
|
22
|
+
|
|
23
|
+
The operational door is pinned in `docs/architecture/CONTEXT-EXPORT.md` and in RPMS section 8.1.
|
|
24
|
+
If you are touching an entity, read its generated `_context.md` before any local reasoning, and never hand-edit that projection.
|
|
25
|
+
|
|
26
|
+
## Pointers
|
|
27
|
+
|
|
28
|
+
- RPMS section 6a: Asset lens and branch-store split
|
|
29
|
+
- RPMS section 6b: Corpus disk layout
|
|
30
|
+
- RPMS section 8.1: Session-start ritual and locked-decision read order
|
|
31
|
+
|
|
32
|
+
## Canonical Map
|
|
33
|
+
|
|
34
|
+
### Canonical Sources
|
|
35
|
+
|
|
36
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
37
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
38
|
+
| `canon.rpms.architecture` | `docs/architecture/regenerative-project-management-system.md` | `read_only` | One Home Per Fact, One Door | `block_wrong_home`<br />`path_conformance` | docs/architecture/regenerative-project-management-system.md | `active` |
|
|
39
|
+
| `canon.context-export.single-door` | `docs/architecture/CONTEXT-EXPORT.md` | `read_only` | One Direction, One Door | `block_wrong_home`<br />`path_conformance` | docs/architecture/CONTEXT-EXPORT.md | `active` |
|
|
40
|
+
|
|
41
|
+
### Session Start
|
|
42
|
+
|
|
43
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
44
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
45
|
+
| `session-start.claude-ai.root` | `CLAUDE.ai.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | repo-root | `active` |
|
|
46
|
+
| `session-start.claude.root` | `CLAUDE.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | repo-root | `active` |
|
|
47
|
+
| `session-start.decision.spine-lock` | `.rdc/plans/project-relationship-graph-approval.md` | `read_only` | One Spine, One Door | `block_wrong_home`<br />`path_conformance` | .rdc/plans/project-relationship-graph-approval.md | `active` |
|
|
48
|
+
| `session-start.decision.cs2-lock` | `.rdc/plans/mdk-knowledge-decisions-and-workitems-2026-06-02.md` | `read_only` | One Direction, One Door | `block_wrong_home`<br />`path_conformance` | .rdc/plans/mdk-knowledge-decisions-and-workitems-2026-06-02.md | `active` |
|
|
49
|
+
| `session-start.glossary.layer-model` | `docs/systems/cs2/LAYER-MODEL-AND-VOCABULARY.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | docs/systems/cs2/LAYER-MODEL-AND-VOCABULARY.md | `active` |
|
|
50
|
+
|
|
51
|
+
### Database Gates
|
|
52
|
+
|
|
53
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
54
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
55
|
+
| `session-start.work-items.queue` | `table:work_items via rpc:get_open_epics` | `rpc_only` | One Home Per Fact, One Door | `rpc_only`<br />`rpc_contract`<br />CRUD stays behind insert_work_item/update_work_item_status/update_checklist_item/submit_implementation_report. | public.work_items + RPC gate | `active` |
|
|
56
|
+
|
|
57
|
+
### Place Corpus
|
|
58
|
+
|
|
59
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
60
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
61
|
+
| `session-start.entity-context.place` | `places/*/_context.md`<br />created_by: `apps/studio/src/lib/context-export-writer.ts` | `generated` | One Direction, One Door | `block_manual_edit`<br />`projection_drift` | context-export writer | `active` |
|
|
62
|
+
| `place.identity.place-md` | `places/*/PLACE.md` | `authored` | One Home Per Fact, One Door | `allow_declared_home`<br />`path_conformance` | place stewards | `active` |
|
|
63
|
+
| `place.history.history-md` | `places/*/HISTORY.md` | `append_only` | One Direction, One Home Per Fact | `allow_declared_home`<br />`path_conformance` | place stewards | `active` |
|
|
64
|
+
| `place.corpus.index` | `places/*/corpus/INDEX.md` | `authored` | One Home Per Fact, One Door | `allow_declared_home`<br />`path_conformance` | place corpus | `active` |
|
|
65
|
+
| `place.corpus.documents` | `places/*/corpus/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban`<br />Text and markdown live here; binaries route to branch stores. | place corpus | `active` |
|
|
66
|
+
| `place.artifacts.outputs` | `places/*/artifacts/**` | `generated` | One Direction | `allow_declared_home`<br />`path_conformance` | render pipelines | `active` |
|
|
67
|
+
| `place.legacy.asset-bundles` | `places/*/*.assets/**` | `legacy_frozen` | One Direction, One Home Per Fact | `block_manual_edit`<br />`path_conformance`<br />Bridge-state allowlist for current repo reality; no new writes here. | legacy bridge until asset-lens migration | `active` |
|
|
68
|
+
| `place.tracker.pm-docs` | `places/*/tracker/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban` | place operators | `active` |
|
|
69
|
+
|
|
70
|
+
### Shared Corpus
|
|
71
|
+
|
|
72
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
73
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
74
|
+
| `session-start.entity-context.project-node` | `corpus/_shared/*/_context.md`<br />created_by: `apps/studio/src/lib/context-export-writer.ts` | `generated` | One Direction, One Door | `block_manual_edit`<br />`projection_drift` | context-export writer | `active` |
|
|
75
|
+
| `shared.index` | `corpus/_shared/*/INDEX.md` | `authored` | One Home Per Fact, One Door | `allow_declared_home`<br />`path_conformance` | shared corpus | `active` |
|
|
76
|
+
| `shared.framework.documents` | `corpus/_shared/*/framework/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban` | shared corpus | `active` |
|
|
77
|
+
| `shared.research.documents` | `corpus/_shared/*/research/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban` | shared corpus | `active` |
|
|
78
|
+
| `shared.legacy.design-assets` | `corpus/_shared/*/design/**/assets/**` | `legacy_frozen` | One Direction, One Home Per Fact | `block_manual_edit`<br />`path_conformance`<br />Bridge-state allowlist for current repo reality; no new writes here. | legacy bridge until asset-lens migration | `active` |
|
|
79
|
+
| `shared.corpus.documents` | `corpus/_shared/` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban`<br />Generated _context.md remains separately governed by projection drift. | shared corpus | `active` |
|
|
80
|
+
|
|
81
|
+
### Asset Lens
|
|
82
|
+
|
|
83
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
84
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
85
|
+
| `asset-lens.entity-assets.registry` | `table:entity_assets` | `deferred` | One Spine, One Home Per Fact | `deferred`<br />`deferred`<br />Manifest records the canonical home now; implementation remains deferred per plan section 6a E7. | RPMS E7 | `deferred` |
|
|
86
|
+
|
|
87
|
+
### Operational Door
|
|
88
|
+
|
|
89
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
90
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
91
|
+
| `operational-door.rdc.plans` | `.rdc/plans/**` | `authored` | One Home Per Fact, One Direction | `allow_declared_home`<br />`path_conformance` | RDC planning workflow | `active` |
|
|
92
|
+
| `operational-door.rdc.reports` | `.rdc/reports/**` | `generated` | One Direction | `allow_declared_home`<br />`path_conformance` | rdc:report | `active` |
|
|
93
|
+
| `operational-door.rdc.lessons` | `.rdc/lessons/**` | `append_only` | One Direction, One Home Per Fact | `allow_declared_home`<br />`path_conformance` | RDC lesson capture workflow | `active` |
|
|
94
|
+
| `operational-door.rpms-filemap.rule` | `.claude/rules/rpms-filemap.md`<br />created_by: `scripts/rpms/gen-filemap.ts` | `generated` | One Home Per Fact, One Direction, One Door | `block_manual_edit`<br />`projection_drift` | scripts/rpms/gen-filemap.ts | `active` |
|
|
95
|
+
|
|
96
|
+
### App Guides
|
|
97
|
+
|
|
98
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
99
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
100
|
+
| `session-start.app-guide` | `apps/*/CLAUDE.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | app-local architecture guides | `active` |
|
|
101
|
+
|
|
102
|
+
## Projection Contract
|
|
103
|
+
|
|
104
|
+
- Generated projections (`_context.md`, this rule) are never hand-edited.
|
|
105
|
+
- Authored markdown stays in its declared home; do not create second homes elsewhere.
|
|
106
|
+
- `work_items` stays RPC-only even when the location is documented here.
|
|
107
|
+
- Corpus homes are text-first; binaries route to branch stores, not git under `places/` or `corpus/`.
|
|
@@ -0,0 +1,107 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: "Generated RPMS file map — RULE #1, canonical homes, and Context Export pointers served from regen-root manifest."
|
|
3
|
+
slash: "rdc:rpms-filemap"
|
|
4
|
+
category: "tooling"
|
|
5
|
+
usage: "rdc:rpms-filemap"
|
|
6
|
+
requires: []
|
|
7
|
+
triggers:
|
|
8
|
+
- "rpms file map"
|
|
9
|
+
- "rule #1"
|
|
10
|
+
- "context export"
|
|
11
|
+
- "where does this file belong"
|
|
12
|
+
- "where should pm artifacts go"
|
|
13
|
+
---
|
|
14
|
+
# RPMS File Map
|
|
15
|
+
> GENERATED FILE - DO NOT HAND-EDIT.
|
|
16
|
+
> Source of truth: `docs/architecture/rpms.locations.json`
|
|
17
|
+
> Regenerate: `pnpm rpms:gen-filemap`
|
|
18
|
+
|
|
19
|
+
## RULE #1
|
|
20
|
+
|
|
21
|
+
**Name the entity -> read its `_context.md` -> you are oriented.**
|
|
22
|
+
|
|
23
|
+
The operational door is pinned in `docs/architecture/CONTEXT-EXPORT.md` and in RPMS section 8.1.
|
|
24
|
+
If you are touching an entity, read its generated `_context.md` before any local reasoning, and never hand-edit that projection.
|
|
25
|
+
|
|
26
|
+
## Pointers
|
|
27
|
+
|
|
28
|
+
- RPMS section 6a: Asset lens and branch-store split
|
|
29
|
+
- RPMS section 6b: Corpus disk layout
|
|
30
|
+
- RPMS section 8.1: Session-start ritual and locked-decision read order
|
|
31
|
+
|
|
32
|
+
## Canonical Map
|
|
33
|
+
|
|
34
|
+
### Canonical Sources
|
|
35
|
+
|
|
36
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
37
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
38
|
+
| `canon.rpms.architecture` | `docs/architecture/regenerative-project-management-system.md` | `read_only` | One Home Per Fact, One Door | `block_wrong_home`<br />`path_conformance` | docs/architecture/regenerative-project-management-system.md | `active` |
|
|
39
|
+
| `canon.context-export.single-door` | `docs/architecture/CONTEXT-EXPORT.md` | `read_only` | One Direction, One Door | `block_wrong_home`<br />`path_conformance` | docs/architecture/CONTEXT-EXPORT.md | `active` |
|
|
40
|
+
|
|
41
|
+
### Session Start
|
|
42
|
+
|
|
43
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
44
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
45
|
+
| `session-start.claude-ai.root` | `CLAUDE.ai.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | repo-root | `active` |
|
|
46
|
+
| `session-start.claude.root` | `CLAUDE.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | repo-root | `active` |
|
|
47
|
+
| `session-start.decision.spine-lock` | `.rdc/plans/project-relationship-graph-approval.md` | `read_only` | One Spine, One Door | `block_wrong_home`<br />`path_conformance` | .rdc/plans/project-relationship-graph-approval.md | `active` |
|
|
48
|
+
| `session-start.decision.cs2-lock` | `.rdc/plans/mdk-knowledge-decisions-and-workitems-2026-06-02.md` | `read_only` | One Direction, One Door | `block_wrong_home`<br />`path_conformance` | .rdc/plans/mdk-knowledge-decisions-and-workitems-2026-06-02.md | `active` |
|
|
49
|
+
| `session-start.glossary.layer-model` | `docs/systems/cs2/LAYER-MODEL-AND-VOCABULARY.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | docs/systems/cs2/LAYER-MODEL-AND-VOCABULARY.md | `active` |
|
|
50
|
+
|
|
51
|
+
### Database Gates
|
|
52
|
+
|
|
53
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
54
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
55
|
+
| `session-start.work-items.queue` | `table:work_items via rpc:get_open_epics` | `rpc_only` | One Home Per Fact, One Door | `rpc_only`<br />`rpc_contract`<br />CRUD stays behind insert_work_item/update_work_item_status/update_checklist_item/submit_implementation_report. | public.work_items + RPC gate | `active` |
|
|
56
|
+
|
|
57
|
+
### Place Corpus
|
|
58
|
+
|
|
59
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
60
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
61
|
+
| `session-start.entity-context.place` | `places/*/_context.md`<br />created_by: `apps/studio/src/lib/context-export-writer.ts` | `generated` | One Direction, One Door | `block_manual_edit`<br />`projection_drift` | context-export writer | `active` |
|
|
62
|
+
| `place.identity.place-md` | `places/*/PLACE.md` | `authored` | One Home Per Fact, One Door | `allow_declared_home`<br />`path_conformance` | place stewards | `active` |
|
|
63
|
+
| `place.history.history-md` | `places/*/HISTORY.md` | `append_only` | One Direction, One Home Per Fact | `allow_declared_home`<br />`path_conformance` | place stewards | `active` |
|
|
64
|
+
| `place.corpus.index` | `places/*/corpus/INDEX.md` | `authored` | One Home Per Fact, One Door | `allow_declared_home`<br />`path_conformance` | place corpus | `active` |
|
|
65
|
+
| `place.corpus.documents` | `places/*/corpus/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban`<br />Text and markdown live here; binaries route to branch stores. | place corpus | `active` |
|
|
66
|
+
| `place.artifacts.outputs` | `places/*/artifacts/**` | `generated` | One Direction | `allow_declared_home`<br />`path_conformance` | render pipelines | `active` |
|
|
67
|
+
| `place.legacy.asset-bundles` | `places/*/*.assets/**` | `legacy_frozen` | One Direction, One Home Per Fact | `block_manual_edit`<br />`path_conformance`<br />Bridge-state allowlist for current repo reality; no new writes here. | legacy bridge until asset-lens migration | `active` |
|
|
68
|
+
| `place.tracker.pm-docs` | `places/*/tracker/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban` | place operators | `active` |
|
|
69
|
+
|
|
70
|
+
### Shared Corpus
|
|
71
|
+
|
|
72
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
73
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
74
|
+
| `session-start.entity-context.project-node` | `corpus/_shared/*/_context.md`<br />created_by: `apps/studio/src/lib/context-export-writer.ts` | `generated` | One Direction, One Door | `block_manual_edit`<br />`projection_drift` | context-export writer | `active` |
|
|
75
|
+
| `shared.index` | `corpus/_shared/*/INDEX.md` | `authored` | One Home Per Fact, One Door | `allow_declared_home`<br />`path_conformance` | shared corpus | `active` |
|
|
76
|
+
| `shared.framework.documents` | `corpus/_shared/*/framework/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban` | shared corpus | `active` |
|
|
77
|
+
| `shared.research.documents` | `corpus/_shared/*/research/**` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban` | shared corpus | `active` |
|
|
78
|
+
| `shared.legacy.design-assets` | `corpus/_shared/*/design/**/assets/**` | `legacy_frozen` | One Direction, One Home Per Fact | `block_manual_edit`<br />`path_conformance`<br />Bridge-state allowlist for current repo reality; no new writes here. | legacy bridge until asset-lens migration | `active` |
|
|
79
|
+
| `shared.corpus.documents` | `corpus/_shared/` | `authored` | One Home Per Fact | `block_binary_commit`<br />`binary_ban`<br />Generated _context.md remains separately governed by projection drift. | shared corpus | `active` |
|
|
80
|
+
|
|
81
|
+
### Asset Lens
|
|
82
|
+
|
|
83
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
84
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
85
|
+
| `asset-lens.entity-assets.registry` | `table:entity_assets` | `deferred` | One Spine, One Home Per Fact | `deferred`<br />`deferred`<br />Manifest records the canonical home now; implementation remains deferred per plan section 6a E7. | RPMS E7 | `deferred` |
|
|
86
|
+
|
|
87
|
+
### Operational Door
|
|
88
|
+
|
|
89
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
90
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
91
|
+
| `operational-door.rdc.plans` | `.rdc/plans/**` | `authored` | One Home Per Fact, One Direction | `allow_declared_home`<br />`path_conformance` | RDC planning workflow | `active` |
|
|
92
|
+
| `operational-door.rdc.reports` | `.rdc/reports/**` | `generated` | One Direction | `allow_declared_home`<br />`path_conformance` | rdc:report | `active` |
|
|
93
|
+
| `operational-door.rdc.lessons` | `.rdc/lessons/**` | `append_only` | One Direction, One Home Per Fact | `allow_declared_home`<br />`path_conformance` | RDC lesson capture workflow | `active` |
|
|
94
|
+
| `operational-door.rpms-filemap.rule` | `.claude/rules/rpms-filemap.md`<br />created_by: `scripts/rpms/gen-filemap.ts` | `generated` | One Home Per Fact, One Direction, One Door | `block_manual_edit`<br />`projection_drift` | scripts/rpms/gen-filemap.ts | `active` |
|
|
95
|
+
|
|
96
|
+
### App Guides
|
|
97
|
+
|
|
98
|
+
| Artifact | Home | RW | Law | Enforcement | Owner | Status |
|
|
99
|
+
|----------|------|----|-----|-------------|-------|--------|
|
|
100
|
+
| `session-start.app-guide` | `apps/*/CLAUDE.md` | `read_only` | One Door | `block_wrong_home`<br />`path_conformance` | app-local architecture guides | `active` |
|
|
101
|
+
|
|
102
|
+
## Projection Contract
|
|
103
|
+
|
|
104
|
+
- Generated projections (`_context.md`, this rule) are never hand-edited.
|
|
105
|
+
- Authored markdown stays in its declared home; do not create second homes elsewhere.
|
|
106
|
+
- `work_items` stays RPC-only even when the location is documented here.
|
|
107
|
+
- Corpus homes are text-first; binaries route to branch stores, not git under `places/` or `corpus/`.
|