feat(review): procedural correctness finders, effort levels, and posting/verify guardrails

docs/users/features/commands.md

@@ -130,12 +130,12 @@ Commands for managing AI tools and models.
 
 These commands invoke bundled skills that provide specialized workflows.
 
-| Command      | Description                                                 | Usage Examples                                    |
-| ------------ | ----------------------------------------------------------- | ------------------------------------------------- |
-| `/review`    | Review code changes with 9 parallel review agents           | `/review`, `/review 123`, `/review 123 --comment` |
-| `/loop`      | Run a prompt on a recurring schedule                        | `/loop 5m check the build`                        |
-| `/simplify`  | Review recent changes and apply safe cleanup edits directly | `/simplify`, `/simplify focus on duplication`     |
-| `/qc-helper` | Answer questions about Qwen Code usage and configuration    | `/qc-helper how do I configure MCP?`              |
+| Command      | Description                                                 | Usage Examples                                                            |
+| ------------ | ----------------------------------------------------------- | ------------------------------------------------------------------------- |
+| `/review`    | Multi-agent code review (12 parallel agents at high effort) | `/review`, `/review 123`, `/review 123 --comment`, `/review --effort low` |
+| `/loop`      | Run a prompt on a recurring schedule                        | `/loop 5m check the build`                                                |
+| `/simplify`  | Review recent changes and apply safe cleanup edits directly | `/simplify`, `/simplify focus on duplication`                             |
+| `/qc-helper` | Answer questions about Qwen Code usage and configuration    | `/qc-helper how do I configure MCP?`                                      |
 
 See [Code Review](./code-review.md) for full `/review` documentation.
 

packages/cli/src/commands/review.test.ts

@@ -15,31 +15,50 @@ import { reviewCommand } from './review.js';
 // can't silently re-add `deterministic`, drop one of the others, or let the
 // `describe` / demand text drift.
 describe('reviewCommand', () => {
-  function registeredSubcommands(): string[] {
+  function inspectBuilder(): { names: string[]; demandMessage: string } {
     const names: string[] = [];
+    let demandMessage = '';
     const stub = {
       command: (m: CommandModule) => {
         names.push(String(m.command).split(' ')[0]);
         return stub;
       },
-      demandCommand: () => stub,
+      demandCommand: (_min: number, msg: string) => {
+        demandMessage = msg;
+        return stub;
+      },
       version: () => stub,
     } as unknown as Argv;
     (reviewCommand.builder as (y: Argv) => Argv)(stub);
-    return names;
+    return { names, demandMessage };
+  }
+
+  function registeredSubcommands(): string[] {
+    return inspectBuilder().names;
   }
 
   it('registers exactly the expected internal helper subcommands', () => {
     expect(registeredSubcommands()).toEqual([
+      'parse-args',
       'fetch-pr',
       'plan-diff',
       'pr-context',
       'load-rules',
       'presubmit',
+      'compose-review',
       'cleanup',
     ]);
   });
 
+  it('the demandCommand message names every registered subcommand', () => {
+    // The error message is the one place that enumerates the interface for
+    // a user who typed `qwen review` bare; it once omitted plan-diff.
+    const { names, demandMessage } = inspectBuilder();
+    for (const name of names) {
+      expect(demandMessage).toContain(name);
+    }
+  });
+
   it('does not register the removed `post-suggestions` subcommand', () => {
     expect(registeredSubcommands()).not.toContain('post-suggestions');
   });

packages/cli/src/commands/review.ts

@@ -9,6 +9,8 @@
 // can stay short and the logic stays testable.
 
 import type { Argv, CommandModule } from 'yargs';
+import { parseArgsCommand } from './review/parse-args.js';
+import { composeReviewCommand } from './review/compose-review.js';
 import { fetchPrCommand } from './review/fetch-pr.js';
 import { planDiffCommand } from './review/plan-diff.js';
 import { prContextCommand } from './review/pr-context.js';
@@ -22,15 +24,17 @@ export const reviewCommand: CommandModule = {
     'Internal helpers used by the /review skill (PR worktree setup, context fetch, rules loading, presubmit checks, cleanup)',
   builder: (yargs: Argv) =>
     yargs
+      .command(parseArgsCommand)
       .command(fetchPrCommand)
       .command(planDiffCommand)
       .command(prContextCommand)
       .command(loadRulesCommand)
       .command(presubmitCommand)
+      .command(composeReviewCommand)
       .command(cleanupCommand)
       .demandCommand(
         1,
-        'Specify a subcommand: fetch-pr, pr-context, load-rules, presubmit, or cleanup.',
+        'Specify a subcommand: parse-args, fetch-pr, plan-diff, pr-context, load-rules, presubmit, compose-review, or cleanup.',
       )
       .version(false),
   handler: () => {