-
Notifications
You must be signed in to change notification settings - Fork 2
correct: Handle mixed-case damage explicitly #37
Copy link
Copy link
Closed
Labels
area: cliCommand-line interface behavior.Command-line interface behavior.area: correctionCorrection engine and correction UX.Correction engine and correction UX.bugSomething isn't workingSomething isn't workinggate: adversarial reviewResolve, merge, or explicitly defer before the next full adversarial review.Resolve, merge, or explicitly defer before the next full adversarial review.
Description
Activity
Metadata
Metadata
Assignees
Labels
area: cliCommand-line interface behavior.Command-line interface behavior.area: correctionCorrection engine and correction UX.Correction engine and correction UX.bugSomething isn't workingSomething isn't workinggate: adversarial reviewResolve, merge, or explicitly defer before the next full adversarial review.Resolve, merge, or explicitly defer before the next full adversarial review.
correct()and the standalonecorrectcommand previously reported no candidate / “No valid correction found” for otherwise repairable mixed-case input, while interactive recovery already applied majority-case interpretation.Implemented and reviewed in PR #42: apply the same majority-case interpretation to the public correction API and standalone CLI, retry minority-case characters as erasures, preserve the shared deadline/capture ledger, keep explicit
--bytesenforcement, preserve the operator's original characters inCorrectionEdit.observed, and never Unicode-case-fold non-ASCII text. The parser itself remains strict: mixed-case codex32 strings are invalid until corrected.PR #45's distinct
correctexit statuses are already merged into #42's branch, so the current #42 head carries both fixes and has green exact-head CI. Keep this issue open until #42 is human-integrated into currentreviewability-v1; the branch has ordinary base divergence after #7/#51 rather than a known semantic blocker. Rerun the existing mixed-case/status regressions and frozen differential verifier on the integrated tip.