Skip to content

Remove legacy review config compatibility - #311

Merged
yvette-carlisle merged 1 commit into
mainfrom
xy/remove-legacy-review-config-compat
Jun 11, 2026
Merged

Remove legacy review config compatibility#311
yvette-carlisle merged 1 commit into
mainfrom
xy/remove-legacy-review-config-compat

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • remove the legacy internal_review_mode / external_review_enabled parser reintroduced by [XY-902] Fix stale closeout lease target dispatch #301
  • restore the v0.2 no-compat config contract so only [codex].review controls review behavior
  • replace legacy acceptance coverage with a rejection regression test

Validation

  • cargo test -p decodex config --all-features -- --test-threads=1
  • cargo make fmt
  • cargo make check-rust
  • cargo make lint-fix
  • cargo make test (1038 passed, 1 skipped)
  • git diff --check

@yvette-carlisle
yvette-carlisle merged commit 25e797e into main Jun 11, 2026
14 checks passed
@yvette-carlisle
yvette-carlisle deleted the xy/remove-legacy-review-config-compat branch June 11, 2026 02:25
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