rhachet-roles-bhuild 0.15.0 → 0.15.2

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.
@@ -6,7 +6,59 @@ protect:
6
6
 
7
7
  reviews:
8
8
  self:
9
- # 1. delete before optimize
9
+ # 1. research traceability
10
+ - slug: has-research-traceability
11
+ say: |
12
+ review that research recommendations were leveraged or explicitly omitted.
13
+
14
+ the research stone(s) produced recommendations. we must ensure the
15
+ blueprint either:
16
+ - leverages each recommendation, or
17
+ - provides clear rationale for why it was omitted
18
+
19
+ go through each research artifact in the route:
20
+ 1. list every recommendation from the research
21
+ 2. for each recommendation, check:
22
+ - is it reflected in the blueprint?
23
+ - if not, is there a clear rationale for the omission?
24
+ - did we silently ignore useful research?
25
+
26
+ for omissions, ensure the rationale is documented:
27
+ - "not applicable because..." (explain why)
28
+ - "deferred to future work because..." (explain scope)
29
+ - "contradicts requirement X because..." (cite conflict)
30
+
31
+ research done but not used is wasted effort. if we researched it,
32
+ we should either use it or explain why not.
33
+
34
+ # 2. zero deferrals
35
+ - slug: has-zero-deferrals
36
+ say: |
37
+ review that no item from the vision is deferred. zero leniance.
38
+
39
+ if the vision included it, it is not deferrable. the vision is the
40
+ contract — we deliver what was promised.
41
+
42
+ go through the blueprint and check:
43
+ 1. are any items marked as "deferred", "future work", or "out of scope"?
44
+ 2. for each deferral, check:
45
+ - was this item in the vision or criteria?
46
+ - if yes, it cannot be deferred — implement it or escalate to wisher
47
+ - if no, the deferral is acceptable (we can defer extras)
48
+
49
+ acceptable deferrals:
50
+ - nice-to-haves we identified ourselves
51
+ - optimizations beyond the stated requirements
52
+
53
+ unacceptable deferrals:
54
+ - any requirement from the vision
55
+ - any criterion from the criteria
56
+ - any explicit ask from the wisher
57
+
58
+ if vision items are deferred, either implement them or flag as blocker
59
+ for the wisher to re-scope.
60
+
61
+ # 3. delete before optimize
10
62
  - slug: has-questioned-deletables
11
63
  say: |
12
64
  try hard to delete before you optimize:
@@ -32,7 +84,7 @@ reviews:
32
84
 
33
85
  delete and simplify before we proceed.
34
86
 
35
- # 2. question assumptions
87
+ # 4. question assumptions
36
88
  - slug: has-questioned-assumptions
37
89
  say: |
38
90
  a junior recently modified files in this repo. we need to carefully
@@ -49,7 +101,7 @@ reviews:
49
101
 
50
102
  surface all technical assumptions and question each one.
51
103
 
52
- # 3. drill with 5 whys
104
+ # 5. drill with 5 whys
53
105
  - slug: has-questioned-5whys
54
106
  say: |
55
107
  for each technical decision, drill deep with 5 whys:
@@ -62,7 +114,7 @@ reviews:
62
114
 
63
115
  ensure we understand the root reasons for these choices.
64
116
 
65
- # 4. pre-mortem
117
+ # 6. pre-mortem
66
118
  - slug: has-questioned-premortem
67
119
  say: |
68
120
  imagine this blueprint was implemented and failed miserably.
@@ -75,7 +127,7 @@ reviews:
75
127
 
76
128
  address all failure modes identified before we proceed.
77
129
 
78
- # 5. inverse
130
+ # 7. inverse
79
131
  - slug: has-questioned-inverse
80
132
  say: |
81
133
  did we consider the inverse?
@@ -88,7 +140,7 @@ reviews:
88
140
 
89
141
  ensure we know what to avoid, not just what to build.
90
142
 
91
- # 7. devil's advocate
143
+ # 8. devils advocate
92
144
  - slug: has-questioned-devils-advocate
93
145
  say: |
94
146
  argue against this blueprint — play devil's advocate:
@@ -101,7 +153,7 @@ reviews:
101
153
 
102
154
  expose weak points before they undermine us.
103
155
 
104
- # 8. minimalism - yagni
156
+ # 9. yagni
105
157
  - slug: has-pruned-yagni
106
158
  say: |
107
159
  review for extras that were not prescribed.
@@ -118,7 +170,7 @@ reviews:
118
170
  if a component was not requested, delete it or flag it as an open question
119
171
  for the wisher to decide.
120
172
 
121
- # 9. minimalism - backwards compat
173
+ # 10. backwards compat
122
174
  - slug: has-pruned-backcompat
123
175
  say: |
124
176
  review for backwards compatibility that was not explicitly requested.
@@ -133,7 +185,7 @@ reviews:
133
185
  2. eliminate it if not confirmed as required
134
186
  3. make the open question very clearly reported
135
187
 
136
- # 10. consistency - mechanisms
188
+ # 11. consistent mechanisms
137
189
  - slug: has-consistent-mechanisms
138
190
  say: |
139
191
  review for new mechanisms that duplicate extant functionality.
@@ -152,7 +204,7 @@ reviews:
152
204
  1. replace with the extant mechanism
153
205
  2. or flag as an open question if unsure
154
206
 
155
- # 11. consistency - conventions
207
+ # 12. consistent conventions
156
208
  - slug: has-consistent-conventions
157
209
  say: |
158
210
  review for divergence from extant names and patterns.
@@ -172,7 +224,7 @@ reviews:
172
224
  1. align with the extant convention
173
225
  2. or flag as an open question if the extant convention seems wrong
174
226
 
175
- # 12. behavior declaration - coverage
227
+ # 13. behavior coverage
176
228
  - slug: has-behavior-declaration-coverage
177
229
  say: |
178
230
  review for coverage of the behavior declaration.
@@ -189,7 +241,7 @@ reviews:
189
241
 
190
242
  fix all gaps before you continue.
191
243
 
192
- # 13. behavior declaration - adherance
244
+ # 14. behavior adherance
193
245
  - slug: has-behavior-declaration-adherance
194
246
  say: |
195
247
  review for adherance to the behavior declaration.
@@ -206,7 +258,7 @@ reviews:
206
258
 
207
259
  fix all gaps before you continue.
208
260
 
209
- # 14. role standards - adherance
261
+ # 15. standards adherance
210
262
  - slug: has-role-standards-adherance
211
263
  say: |
212
264
  review for adherance to mechanic role standards.
@@ -226,7 +278,7 @@ reviews:
226
278
 
227
279
  fix all gaps before you continue.
228
280
 
229
- # 15. role standards - coverage
281
+ # 16. standards coverage
230
282
  - slug: has-role-standards-coverage
231
283
  say: |
232
284
  review for coverage of mechanic role standards.
@@ -247,7 +299,7 @@ reviews:
247
299
  fix all gaps before you continue.
248
300
 
249
301
  peer:
250
- - npx rhachet run --repo bhrain --skill review --rules '.agent/repo=ehmpathy/role=mechanic/briefs/practices/code.prod/pitofsuccess.errors/rule.*.md' --diffs since-main --paths-with '$route/3.3.blueprint.*.md' --join intersect --output '$route/.reviews/$stone.peer-review.failhides.md' --mode hard
302
+ - npx rhachet run --repo bhrain --skill review --rules '.agent/repo=ehmpathy/role=mechanic/briefs/practices/code.{prod,test}/pitofsuccess.errors/rule.*.md' --diffs since-main --paths-with '$route/3.3.blueprint.*.md' --join intersect --output '$route/.reviews/$stone.peer-review.failhides.md' --mode hard
251
303
 
252
304
  judges:
253
305
  - npx rhachet run --repo bhrain --skill route.stone.judge --mechanism reviewed? --stone $stone --route $route --allow-blockers 0 --allow-nitpicks 3
@@ -6,7 +6,59 @@ protect:
6
6
 
7
7
  reviews:
8
8
  self:
9
- # 1. delete before optimize
9
+ # 1. research traceability
10
+ - slug: has-research-traceability
11
+ say: |
12
+ review that research recommendations were leveraged or explicitly omitted.
13
+
14
+ the research stone(s) produced recommendations. we must ensure the
15
+ blueprint either:
16
+ - leverages each recommendation, or
17
+ - provides clear rationale for why it was omitted
18
+
19
+ go through each research artifact in the route:
20
+ 1. list every recommendation from the research
21
+ 2. for each recommendation, check:
22
+ - is it reflected in the blueprint?
23
+ - if not, is there a clear rationale for the omission?
24
+ - did we silently ignore useful research?
25
+
26
+ for omissions, ensure the rationale is documented:
27
+ - "not applicable because..." (explain why)
28
+ - "deferred to future work because..." (explain scope)
29
+ - "contradicts requirement X because..." (cite conflict)
30
+
31
+ research done but not used is wasted effort. if we researched it,
32
+ we should either use it or explain why not.
33
+
34
+ # 2. zero deferrals
35
+ - slug: has-zero-deferrals
36
+ say: |
37
+ review that no item from the vision is deferred. zero leniance.
38
+
39
+ if the vision included it, it is not deferrable. the vision is the
40
+ contract — we deliver what was promised.
41
+
42
+ go through the blueprint and check:
43
+ 1. are any items marked as "deferred", "future work", or "out of scope"?
44
+ 2. for each deferral, check:
45
+ - was this item in the vision or criteria?
46
+ - if yes, it cannot be deferred — implement it or escalate to wisher
47
+ - if no, the deferral is acceptable (we can defer extras)
48
+
49
+ acceptable deferrals:
50
+ - nice-to-haves we identified ourselves
51
+ - optimizations beyond the stated requirements
52
+
53
+ unacceptable deferrals:
54
+ - any requirement from the vision
55
+ - any criterion from the criteria
56
+ - any explicit ask from the wisher
57
+
58
+ if vision items are deferred, either implement them or flag as blocker
59
+ for the wisher to re-scope.
60
+
61
+ # 3. delete before optimize
10
62
  - slug: has-questioned-deletables
11
63
  say: |
12
64
  try hard to delete before you optimize:
@@ -32,7 +84,7 @@ reviews:
32
84
 
33
85
  delete and simplify before we proceed.
34
86
 
35
- # 2. question assumptions
87
+ # 4. question assumptions
36
88
  - slug: has-questioned-assumptions
37
89
  say: |
38
90
  a junior recently modified files in this repo. we need to carefully
@@ -49,7 +101,7 @@ reviews:
49
101
 
50
102
  surface all technical assumptions and question each one.
51
103
 
52
- # 3. minimalism - yagni
104
+ # 5. yagni
53
105
  - slug: has-pruned-yagni
54
106
  say: |
55
107
  review for extras that were not prescribed.
@@ -66,7 +118,7 @@ reviews:
66
118
  if a component was not requested, delete it or flag it as an open question
67
119
  for the wisher to decide.
68
120
 
69
- # 4. minimalism - backwards compat
121
+ # 6. backwards compat
70
122
  - slug: has-pruned-backcompat
71
123
  say: |
72
124
  review for backwards compatibility that was not explicitly requested.
@@ -81,7 +133,7 @@ reviews:
81
133
  2. eliminate it if not confirmed as required
82
134
  3. make the open question very clearly reported
83
135
 
84
- # 5. consistency - mechanisms
136
+ # 7. consistent mechanisms
85
137
  - slug: has-consistent-mechanisms
86
138
  say: |
87
139
  review for new mechanisms that duplicate extant functionality.
@@ -100,7 +152,7 @@ reviews:
100
152
  1. replace with the extant mechanism
101
153
  2. or flag as an open question if unsure
102
154
 
103
- # 6. consistency - conventions
155
+ # 8. consistent conventions
104
156
  - slug: has-consistent-conventions
105
157
  say: |
106
158
  review for divergence from extant names and patterns.
@@ -120,7 +172,7 @@ reviews:
120
172
  1. align with the extant convention
121
173
  2. or flag as an open question if the extant convention seems wrong
122
174
 
123
- # 7. behavior declaration - coverage
175
+ # 9. behavior coverage
124
176
  - slug: has-behavior-declaration-coverage
125
177
  say: |
126
178
  review for coverage of the behavior declaration.
@@ -137,7 +189,7 @@ reviews:
137
189
 
138
190
  fix all gaps before you continue.
139
191
 
140
- # 8. behavior declaration - adherance
192
+ # 10. behavior adherance
141
193
  - slug: has-behavior-declaration-adherance
142
194
  say: |
143
195
  review for adherance to the behavior declaration.
@@ -154,7 +206,7 @@ reviews:
154
206
 
155
207
  fix all gaps before you continue.
156
208
 
157
- # 9. role standards - adherance
209
+ # 11. standards adherance
158
210
  - slug: has-role-standards-adherance
159
211
  say: |
160
212
  review for adherance to mechanic role standards.
@@ -174,7 +226,7 @@ reviews:
174
226
 
175
227
  fix all gaps before you continue.
176
228
 
177
- # 10. role standards - coverage
229
+ # 12. standards coverage
178
230
  - slug: has-role-standards-coverage
179
231
  say: |
180
232
  review for coverage of mechanic role standards.
@@ -195,7 +247,7 @@ reviews:
195
247
  fix all gaps before you continue.
196
248
 
197
249
  peer:
198
- - npx rhachet run --repo bhrain --skill review --rules '.agent/repo=ehmpathy/role=mechanic/briefs/practices/code.prod/pitofsuccess.errors/rule.*.md' --diffs since-main --paths-with '$route/3.3.blueprint.*.md' --join intersect --output '$route/.reviews/$stone.peer-review.failhides.md' --mode hard
250
+ - npx rhachet run --repo bhrain --skill review --rules '.agent/repo=ehmpathy/role=mechanic/briefs/practices/code.{prod,test}/pitofsuccess.errors/rule.*.md' --diffs since-main --paths-with '$route/3.3.blueprint.*.md' --join intersect --output '$route/.reviews/$stone.peer-review.failhides.md' --mode hard
199
251
 
200
252
  judges:
201
253
  - npx rhachet run --repo bhrain --skill route.stone.judge --mechanism reviewed? --stone $stone --route $route --allow-blockers 0 --allow-nitpicks 3
@@ -157,7 +157,7 @@ reviews:
157
157
  fix all gaps before you continue.
158
158
 
159
159
  peer:
160
- - npx rhachet run --repo bhrain --skill review --rules '.agent/repo=ehmpathy/role=mechanic/briefs/practices/code.prod/pitofsuccess.errors/rule.*.md' --diffs since-main --paths-with 'src/**/*.ts' --join intersect --output '$route/.reviews/$stone.peer-review.failhides.md' --mode hard
160
+ - npx rhachet run --repo bhrain --skill review --rules '.agent/repo=ehmpathy/role=mechanic/briefs/practices/code.{prod,test}/pitofsuccess.errors/rule.*.md' --diffs since-main --paths-with 'src/**/*.ts' --join intersect --output '$route/.reviews/$stone.peer-review.failhides.md' --mode hard
161
161
 
162
162
  judges:
163
163
  - npx rhachet run --repo bhrain --skill route.stone.judge --mechanism reviewed? --stone $stone --route $route --allow-blockers 0 --allow-nitpicks 3
package/package.json CHANGED
@@ -2,7 +2,7 @@
2
2
  "name": "rhachet-roles-bhuild",
3
3
  "author": "ehmpathy",
4
4
  "description": "roles for building resilient systems, via rhachet",
5
- "version": "0.15.0",
5
+ "version": "0.15.2",
6
6
  "repository": "ehmpathy/rhachet-roles-bhuild",
7
7
  "homepage": "https://github.com/ehmpathy/rhachet-roles-bhuild",
8
8
  "keywords": [
@@ -59,7 +59,7 @@
59
59
  "prepare:husky": "husky install && chmod ug+x .husky/*",
60
60
  "prepare": "if [ -e .git ] && [ -z \"${CI:-}\" ]; then npm run prepare:husky && npm run prepare:rhachet; fi",
61
61
  "test:format:biome": "biome format",
62
- "prepare:rhachet": "npm run build && rhachet init --hooks --roles behaver driver mechanic reviewer librarian"
62
+ "prepare:rhachet": "npm run build && rhachet init --hooks --roles mechanic behaver driver reviewer librarian ergonomist architect reflector dreamer dispatcher"
63
63
  },
64
64
  "dependencies": {
65
65
  "domain-objects": "0.31.9",
@@ -89,11 +89,11 @@
89
89
  "esbuild-register": "3.6.0",
90
90
  "husky": "8.0.3",
91
91
  "jest": "30.2.0",
92
- "rhachet": "1.39.0",
92
+ "rhachet": "1.39.7",
93
93
  "rhachet-brains-anthropic": "0.4.0",
94
94
  "rhachet-roles-bhrain": "0.23.10",
95
95
  "rhachet-roles-bhuild": "link:.",
96
- "rhachet-roles-ehmpathy": "1.34.11",
96
+ "rhachet-roles-ehmpathy": "1.34.19",
97
97
  "tsc-alias": "1.8.10",
98
98
  "tsx": "4.20.6",
99
99
  "typescript": "5.4.5",