ryderfreeman4logos/cli-sub-agent/review-loop

Review Loop

Bounded diagnostic review-fix front end. It may run at most 2 identical serial review → fix → review rounds. If the branch is still not clean, the serial loop MUST stop and change topology: freeze HEAD, fan out read-only discovery by semantic scope × lens, verify/deduplicate, cluster root causes, batch repairs, and finish with a fresh whole-range clean-room gate. Reaching the budget never authorizes PASS, push, or merge.

Usage

csa run --sa-mode true --skill review-loop "Review and fix current changes"

SA Mode Propagation (MANDATORY)

When operating under SA mode (e.g., dispatched by /sa or any autonomous workflow), ALL `csa` invocations MUST include `--sa-mode true`. This includes csa run, csa review, csa debate, and any other execution commands. Omitting --sa-mode at root depth causes a hard error; passing false when the caller is in SA mode breaks prompt-guard propagation.

Behavior

1. Reviews current diff with csa review --diff
2. If issues found: fixes them and re-reviews (up to MAX_ROUNDS)

• Multi-finding optimization: When 2+ findings affect different files,

uses parallel-fix pattern (parallel RECON / serial EDIT) for faster fix rounds.

1. If clean: exits successfully
2. If max rounds are reached: keeps the branch blocked, persists remaining

findings, and escalates to convergence topology; never returns PASS merely because the serial budget is exhausted

Variables

• MAX_ROUNDS: Maximum review-fix iterations (default: 2)