Add refactor skill cross-references to implement, test, and review skills

Co-authored-by: afranken <[email protected]>

Author: Copilot

.claude/skills/implement/SKILL.md

@@ -9,11 +9,14 @@ description: Implement features, fix bugs, or refactor code. Use when asked to a
 
 ## Skill Workflow
 
-A complete feature implementation requires all three skills — run them in sequence:
+A complete feature implementation requires these skills — run them in sequence:
 
-1. **implement** (this skill) — write the production code
-2. **test** skill — add/update unit and integration tests (`*Test.kt`, `*IT.kt`)
-3. **document** skill — update `CHANGELOG.md`, `README.md`, and `AGENTS.md` where applicable
+1. **refactor** skill — if nearby existing code, tests, or configuration would benefit from cleanup *before* adding new code, do it first rather than working around it
+2. **implement** (this skill) — write the production code
+3. **test** skill — add/update unit and integration tests (`*Test.kt`, `*IT.kt`)
+4. **document** skill — update `CHANGELOG.md`, `README.md`, and `AGENTS.md` where applicable
+
+> **Prefer refactoring over workarounds**: if you find yourself adding complexity to work around existing code, stop and invoke the `refactor` skill to clean it up first.
 
 ## Implementation Flow for New S3 Operations
 

.claude/skills/review/SKILL.md

@@ -48,7 +48,7 @@ See **[docs/KOTLIN.md](../../../docs/KOTLIN.md)** for the full list of idioms an
 Structure feedback as:
 
 - **Must fix** — blocks merge (correctness issues, convention violations, missing tests)
-- **Should fix** — strongly recommended (idiom improvements, missing docs)
+- **Should fix** — strongly recommended (idiom improvements, missing docs); suggest invoking the **`refactor` skill** for purely readability/style improvements rather than asking the author to add more code
 - **Nit** — optional style suggestions
 
 For each finding, reference the specific AGENTS.md rule or AWS API doc where applicable.

.claude/skills/test/SKILL.md

@@ -19,6 +19,7 @@ Read **[docs/TESTING.md](../../../docs/TESTING.md)**, **[docs/KOTLIN.md](../../.
 ## Checklist
 
 - [ ] Read `docs/TESTING.md` and root + module `AGENTS.md`
+- [ ] If existing tests have structural problems (poor naming, shared state, weak assertions), invoke the **`refactor` skill** to fix them rather than working around them
 - [ ] Tests pass locally
 - [ ] Both success and failure cases covered
 - [ ] Tests are independent (no shared state, UUID bucket names)