edits to seam review skill to use naming docs

2026-05-25 20:04:02 -06:00
parent 858bf67043
commit 57a691236e
1 changed files with 2 additions and 2 deletions
@@ -22,7 +22,7 @@ Activate when the user:

 ## Core Function: The Seam Review Protocol

-**Goal:** explain whether the code presents a reviewable bounded seam, why it is or is not working, and rank the improvements
+**Goal:** explain whether the code presents a reviewable bounded seam, why it is or is not working, and what the smallest high-value improvements would be.

 **Required input:**

@@ -142,7 +142,7 @@ If naming is part of the problem:
 1. state why the current name is weak
 2. propose three alternatives when the name is important
 3. choose the option that best preserves caller intent across likely model evolution
-4. prefer intent-first names over vague ready/process labels when those better describe the protected invariant. follow the naming guidelines in "docs/explanation/naming-for-domain-modeling.md"
+4. prefer intent-first names such as accepted, confirmed, authorized, selected, or emitted outcomes over vague ready/process labels when those better describe the protected invariant

 ## Constraints