Skip to content

feat: Fix-First Review — auto-fix obvious issues, ask about hard ones (v0.4.5)#116

Merged
garrytan merged 4 commits intomainfrom
garrytan/actually-fix-in-ship-review
Mar 17, 2026
Merged

feat: Fix-First Review — auto-fix obvious issues, ask about hard ones (v0.4.5)#116
garrytan merged 4 commits intomainfrom
garrytan/actually-fix-in-ship-review

Conversation

@garrytan
Copy link
Owner

@garrytan garrytan commented Mar 17, 2026

Documentation

  • CHANGELOG.md: polished v0.4.5 entry voice — "New Fix-First Heuristic in checklist.md" rewritten to lead with user benefit ("You control the line between 'just fix it' and 'ask me first'")
  • README.md: current — no changes needed (skill table, descriptions already accurate)
  • CLAUDE.md: current — project structure already includes document-release/
  • CONTRIBUTING.md: current — no changes needed
  • ARCHITECTURE.md: current — preamble description still accurate
  • TODOS.md: current — no completed items from this change
  • VERSION: already bumped to v0.4.5

garrytan and others added 4 commits March 16, 2026 19:21
Replace the CRITICAL-only AskUserQuestion flow with Fix-First:
- Every finding gets action (not just critical ones)
- AUTO-FIX items (dead code, N+1, stale comments) applied directly
- ASK items (security, race conditions, design decisions) batched
  into at most one AskUserQuestion
- Fix-First Heuristic in checklist.md (single source of truth)
- Gate Classification → Severity Classification rename

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@garrytan garrytan merged commit a30f707 into main Mar 17, 2026
lucaslim pushed a commit to lucaslim/gstack-governed that referenced this pull request Mar 17, 2026
… (v0.4.5) (garrytan#116)

* feat: Fix-First Review — auto-fix obvious issues, ask about hard ones

Replace the CRITICAL-only AskUserQuestion flow with Fix-First:
- Every finding gets action (not just critical ones)
- AUTO-FIX items (dead code, N+1, stale comments) applied directly
- ASK items (security, race conditions, design decisions) batched
  into at most one AskUserQuestion
- Fix-First Heuristic in checklist.md (single source of truth)
- Gate Classification → Severity Classification rename

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: bump version and changelog (v0.4.5)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* docs: polish CHANGELOG v0.4.5 voice — lead with user benefit

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant