Skip to content

fix: converge review findings before implementation - #73

Merged
shinpr merged 1 commit into
mainfrom
fix/reviewee-judgment-convergence
Aug 31, 2026
Merged

fix: converge review findings before implementation#73
shinpr merged 1 commit into
mainfrom
fix/reviewee-judgment-convergence

Conversation

@shinpr

@shinpr shinpr commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • require review findings to pass the ordered reviewee-judgment resolution method before disposition
  • make change necessity and decision-changing alternatives explicit in the resolution record
  • route orchestration review resolution through the shared reviewee-judgment skill and align disposition labels
  • bump the package version to 1.3.2

Behavioral verification

A fresh codex exec review loop independently classified an introduced decoder abstraction as unnecessary, selected deletion and reuse of the existing formatter owner, delegated only the accepted corrections, and passed the same reviewer on rerun.

Checks

  • npm test
  • npm run check:skills-index
  • git diff --check

@shinpr shinpr self-assigned this Aug 31, 2026
@shinpr
shinpr merged commit 545afa6 into main Aug 31, 2026
2 checks passed
@shinpr
shinpr deleted the fix/reviewee-judgment-convergence branch August 31, 2026 14:23
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