Skip to content

feat: compose repository-owned code review - #16

Merged
shaug merged 1 commit into
mainfrom
scott/issue-9-review-code-change
Jul 20, 2026
Merged

feat: compose repository-owned code review#16
shaug merged 1 commit into
mainfrom
scott/issue-9-review-code-change

Conversation

@shaug

@shaug shaug commented Jul 20, 2026

Copy link
Copy Markdown
Owner

TL;DR

Adds one repository-owned entrypoint that runs solution, correctness, and code-simplicity review as a bounded, fail-closed sequence.

Summary

  • Add the standalone review-code-change skill with one evidence packet, ordered local-lens delegation, early redesign exit, targeted rechecks, deduplication, and aggregate verdicts.
  • Extend the shared result contract with explicit correctness dispositions for simplification proposals, so unsafe proposals are rejected without misreporting correct candidate code as defective.
  • Fail closed on missing dependencies, incomplete evidence, malformed results, stale candidate identity, wrong-lens output, and exhausted review cycles.
  • Add ten orchestration evaluations, a raw standalone evaluation, and deterministic contract tests while keeping candidate mutation and GitHub workflow actions outside the review skill.

Tickets

Fixes #9

## Summary

- Add the standalone review-code-change entrypoint with one-packet evidence preparation and ordered lens delegation
- Define fail-closed sequencing, explicit correctness dispositions for simplification proposals, root-cause deduplication, targeted rechecks, and bounded cycles
- Add ten orchestration traces plus a raw standalone three-lens evaluation and deterministic contract tests

## Why

- Give callers one repository-owned review interface without duplicating lens rubrics or depending on third-party skills
- Stop wasteful downstream review after redesign or correctness failures while preserving a trustworthy aggregate verdict
@shaug
shaug merged commit a1a3591 into main Jul 20, 2026
1 check passed
@shaug
shaug deleted the scott/issue-9-review-code-change branch July 20, 2026 04:35
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.

Compose the review-code-change orchestration skill

1 participant