Skip to content

Add review-solution-simplicity skill - #14

Merged
shaug merged 1 commit into
mainfrom
scott/issue-7-solution-simplicity
Jul 20, 2026
Merged

Add review-solution-simplicity skill#14
shaug merged 1 commit into
mainfrom
scott/issue-7-solution-simplicity

Conversation

@shaug

@shaug shaug commented Jul 20, 2026

Copy link
Copy Markdown
Owner

TL;DR

Adds a whole-solution review skill that removes requirement-free machinery while protecting complexity needed for correctness and operations.

Summary

  • Introduces the standalone review-solution-simplicity skill and a mechanism-to-requirement comparison rubric.
  • Requires every proposed simplification to be concrete, requirement-complete, and explicit about preserved failure semantics.
  • Protects justified concurrency, idempotency, security, migration, compatibility, rollout, and recovery behavior.
  • Adds packet and standalone evaluations for speculative frameworks, pre-release backfills, necessary concurrency controls, and missing design decisions.
  • Aligns explicit ticket-scope violations with the shared blocking-severity rule and documents the new skill.

Validation

  • Repository formatting, linting, skill validation, and all test gates pass.
  • Fresh forward evaluations found unsupported machinery, preserved necessary complexity, and returned blocked when a smaller design required missing product decisions.
  • An independent final review found no material ticket-scoped issues at the published head.

Tickets

Fixes #7

## Summary

- Add a standalone read-only solution-simplicity skill with a mechanism-to-requirement rubric
- Add over-engineering, required-complexity, missing-requirement, and raw standalone evaluations
- Align explicit ticket-scope violations with the shared blocking severity and document the skill

## Why

- Identify materially smaller requirement-complete designs without trading away correctness or operational invariants
- Keep speculative frameworks, compatibility layers, and migrations out of narrowly scoped changes
@shaug
shaug merged commit e552a98 into main Jul 20, 2026
1 check passed
@shaug
shaug deleted the scott/issue-7-solution-simplicity branch July 20, 2026 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the review-solution-simplicity skill

1 participant