From 7ecb365e63415f134d7a7279560d5e2cdc60132c Mon Sep 17 00:00:00 2001 From: Danijel Martinek Date: Wed, 13 May 2026 08:53:00 +0200 Subject: [PATCH] feat(sandcastle): implementer + reviewer prompts include fallow audit --- .sandcastle/implementer.prompt.md | 3 ++- .sandcastle/reviewer.prompt.md | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/.sandcastle/implementer.prompt.md b/.sandcastle/implementer.prompt.md index c088a50..1fbefa9 100644 --- a/.sandcastle/implementer.prompt.md +++ b/.sandcastle/implementer.prompt.md @@ -41,9 +41,10 @@ pnpm typecheck # TS brand-slot enforcement, 0s pnpm lint # ESLint rules incl. conformance/* — <1s pnpm test --filter @repo/ # tests for the feature you touched pnpm conformance # cross-feature event closure +pnpm fallow:audit # whole-codebase analysis: dead exports, dupes, circular deps, complexity ``` -All four pass before you commit. If any fail, fix or report BLOCKED — do not paper over. +All five pass before you commit. If any fail, fix or report BLOCKED — do not paper over. ## Commit message format diff --git a/.sandcastle/reviewer.prompt.md b/.sandcastle/reviewer.prompt.md index 0c08395..5fe2330 100644 --- a/.sandcastle/reviewer.prompt.md +++ b/.sandcastle/reviewer.prompt.md @@ -31,6 +31,7 @@ If you suspect the implementer hand-rolled what should have been generator outpu 3. **Manifest-first ordering**: if a new use case landed, the manifest was updated; tests exist; the factory was wrapped at bind time. 4. **Conformance gates**: the diff's tests + lint + typecheck pass. (You don't run them yourself; sandcastle's CI step does. Trust the CI status, reject if it's red.) 5. **Generator-first**: see the section above. Hand-rolled code that should have been generated is a rejection. +6. **Fallow audit**: verify the implementer ran `pnpm fallow:audit` and it passed. If their diff increases dead exports / dupes / circular deps / complexity beyond the baseline, that's a rejection cause unless the implementer's notes explicitly justify it. ## Output format