Summary
The TypeScript review command path exposes review-reject and review-next, but only accept has real behavior.
Problem
executeReview() in voice-tool/src/services/CommandModeHandler.ts plays audio and handles accept, but it does not implement the reject/next decision path. Those commands currently succeed without doing what their names imply.
Impact
Automation and operator workflows can report success while leaving review state unchanged.
Relevant code
voice-tool/src/services/CommandModeHandler.ts:113-133
voice-tool/src/cli/CLIInterface.ts:252-260
Expected behavior
review-reject and review-next should either perform the documented action or fail explicitly.
Suggested direction
- Implement the reject/next decision flow in the TypeScript command handler.
- If that functionality is intentionally unsupported in this path, remove the commands or return a clear error instead of success.
- Add command-level tests for accept, reject, and next behavior.
Summary
The TypeScript review command path exposes
review-rejectandreview-next, but onlyaccepthas real behavior.Problem
executeReview()invoice-tool/src/services/CommandModeHandler.tsplays audio and handlesaccept, but it does not implement the reject/next decision path. Those commands currently succeed without doing what their names imply.Impact
Automation and operator workflows can report success while leaving review state unchanged.
Relevant code
voice-tool/src/services/CommandModeHandler.ts:113-133voice-tool/src/cli/CLIInterface.ts:252-260Expected behavior
review-rejectandreview-nextshould either perform the documented action or fail explicitly.Suggested direction