remove ci flag

Author: PetrusZ

DESIGN-zh-CN.md

@@ -145,7 +145,7 @@ export ZADIG_REVIEW_MODEL_API_KEY=...
 
 `config show` 会隐藏 API Key；`config get model.api_key` 返回真实值。正式 review 检测到默认模型占位值时返回配置错误，preview 不要求模型配置。
 
-`--ci` 默认将控制台模式切换为 `summary`，显式 `--console` 可覆盖。审查控制、模型协议/名称/Endpoint/Timeout 和输出设置可通过 review flags 覆盖；API Key 只通过配置文件或环境变量传入。
+审查控制、模型协议/名称/Endpoint/Timeout 和输出设置可通过 review flags 覆盖；API Key 只通过配置文件或环境变量传入。
 
 ## 5. Rules
 

DESIGN.md

@@ -112,7 +112,7 @@ built-in defaults < config file < ZADIG_REVIEW_MODEL_* < review flags
 
 Environment variables are `ZADIG_REVIEW_MODEL_PROTOCOL`, `ZADIG_REVIEW_MODEL_NAME`, `ZADIG_REVIEW_MODEL_ENDPOINT`, `ZADIG_REVIEW_MODEL_TIMEOUT`, and `ZADIG_REVIEW_MODEL_API_KEY`.
 
-`config show` redacts API keys, while `config get model.api_key` returns the actual value. A real review rejects the placeholder default model; preview does not require model configuration. `--ci` selects summary console output unless an explicit `--console` overrides it. API keys can only come from configuration or the environment, not review flags.
+`config show` redacts API keys, while `config get model.api_key` returns the actual value. A real review rejects the placeholder default model; preview does not require model configuration. Review controls, model settings, and output settings can be overridden by review flags. API keys can only come from configuration or the environment, not review flags.
 
 ## 5. Rules
 

Makefile

@@ -24,10 +24,10 @@ help:
 	@echo "  clean       Remove build output"
 
 fmt:
-	gofmt -w cmd internal
+	gofmt -w main.go internal
 
 fmt-check:
-	@test -z "$$(gofmt -l cmd internal)" || (gofmt -l cmd internal && exit 1)
+	@test -z "$$(gofmt -l main.go internal)" || (gofmt -l main.go internal && exit 1)
 
 verify:
 	go mod verify

README-en.md

@@ -139,13 +139,13 @@ zadig-review-agent rules check internal/reviewer/reviewer.go
 
 ## CI usage
 
-The `--ci` flag selects concise console output. Explicit report paths are convenient for CI artifacts:
+In CI, use `--console summary` for concise final output and explicit report paths for convenient artifact upload:
 
 ```bash
 zadig-review-agent review \
   --from origin/main \
   --to HEAD \
-  --ci \
+  --console summary \
   --output-json "$PWD/review-report.json" \
   --output-md "$PWD/review-report.md"
 ```

README.md

@@ -139,13 +139,13 @@ zadig-review-agent rules check internal/reviewer/reviewer.go
 
 ## CI 用法
 
-`--ci` 会启用精简控制台输出。在 CI 中显式指定报告路径便于上传制品：
+在 CI 中可使用 `--console summary` 精简最终结果，并显式指定报告路径以便上传制品：
 
 ```bash
 zadig-review-agent review \
   --from origin/main \
   --to HEAD \
-  --ci \
+  --console summary \
   --output-json "$PWD/review-report.json" \
   --output-md "$PWD/review-report.md"
 ```

internal/cli/cli.go

@@ -59,7 +59,6 @@ func runReview(ctx context.Context, args []string, stdout, stderr io.Writer) (in
 	commit := fs.String("commit", "", "commit sha to review")
 	rulePath := fs.String("rule", "", "path to Zadig review rules JSON")
 	preview := fs.Bool("preview", false, "print file filtering and rule resolution without invoking the model")
-	ciMode := fs.Bool("ci", false, "use CI-friendly console output")
 	concurrency := fs.Int("concurrency", 0, "max concurrent file reviews")
 	contextLines := fs.Int("context-lines", 0, "context lines around changes")
 	maxToolRounds := fs.Int("max-tool-rounds", 0, "max main tool-loop request rounds")
@@ -88,7 +87,6 @@ func runReview(ctx context.Context, args []string, stdout, stderr io.Writer) (in
 	}
 	visited := visitedFlags(fs)
 	if err := applyCLIOverrides(&cfg, visited, cliOverrides{
-		ciMode:              *ciMode,
 		concurrency:         *concurrency,
 		contextLines:        *contextLines,
 		maxToolRounds:       *maxToolRounds,
@@ -575,7 +573,6 @@ func usage(w io.Writer) {
 }
 
 type cliOverrides struct {
-	ciMode              bool
 	concurrency         int
 	contextLines        int
 	maxToolRounds       int
@@ -603,9 +600,6 @@ func visitedFlags(fs *flag.FlagSet) map[string]bool {
 }
 
 func applyCLIOverrides(cfg *config.Config, visited map[string]bool, o cliOverrides) error {
-	if visited["ci"] && o.ciMode {
-		cfg.Output.Console = "summary"
-	}
 	if visited["concurrency"] {
 		if o.concurrency < 1 {
 			return fmt.Errorf("--concurrency must be a positive integer")

internal/cli/cli_test.go

@@ -189,6 +189,17 @@ func TestRepositoryPathGroupingUsesFlattenedFullPath(t *testing.T) {
 	}
 }
 
+func TestReviewCIFlagRejected(t *testing.T) {
+	var stdout, stderr bytes.Buffer
+	code, err := Run(context.Background(), []string{"review", "--ci", "--preview"}, &stdout, &stderr)
+	if err == nil || code == 0 {
+		t.Fatalf("expected --ci to be rejected, code=%d stdout=%s stderr=%s", code, stdout.String(), stderr.String())
+	}
+	if !strings.Contains(stderr.String(), "flag provided but not defined") {
+		t.Fatalf("unexpected stderr: %s", stderr.String())
+	}
+}
+
 func TestReviewBaseFlagRejected(t *testing.T) {
 	var stdout, stderr bytes.Buffer
 	code, err := Run(context.Background(), []string{"review", "--base", "origin/main", "--preview"}, &stdout, &stderr)
@@ -278,7 +289,6 @@ func TestConfigSetHelpListsKeys(t *testing.T) {
 func TestApplyCLIOverrides(t *testing.T) {
 	cfg := config.Default()
 	visited := map[string]bool{
-		"ci":                     true,
 		"concurrency":            true,
 		"context-lines":          true,
 		"max-tool-rounds":        true,
@@ -297,7 +307,6 @@ func TestApplyCLIOverrides(t *testing.T) {
 		"progress":               true,
 	}
 	err := applyCLIOverrides(&cfg, visited, cliOverrides{
-		ciMode:              true,
 		concurrency:         8,
 		contextLines:        12,
 		maxToolRounds:       3,