Skip to content

Make review CLI commands honor reject and next actions #77

Description

@getOffIt

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions