Skip to content
This repository was archived by the owner on Jun 18, 2026. It is now read-only.

Commit 5ff5e86

Browse files
author
catlog22
committed
feat: 更新 Codex 代码审查描述,改为基于提示的分析模式
1 parent 491828a commit 5ff5e86

8 files changed

Lines changed: 18 additions & 17 deletions

File tree

.ccw/workflows/cli-tools-usage.md

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ ccw cli -p "<PROMPT>" [options]
2020
|--------|-------------|---------|
2121
| `-p, --prompt` | **Required**. Prompt text ||
2222
| `--tool <name>` | Tool: gemini, qwen, codex, claude, opencode | First enabled in config |
23-
| `--mode <mode>` | `analysis` (read-only), `write` (create/modify/delete), `review` (codex-only) | `analysis` |
23+
| `--mode <mode>` | `analysis` (read-only) or `write` (create/modify/delete) | `analysis` |
2424
| `--model <model>` | Model override | Tool's `primaryModel` |
2525
| `--cd <dir>` | Working directory | Current directory |
2626
| `--includeDirs <dirs>` | Additional directories (comma-separated) ||
@@ -34,12 +34,8 @@ ccw cli -p "<PROMPT>" [options]
3434
|------|-----------|------------------|---------|
3535
| `analysis` | Read-only | Yes | Review, exploration, diagnosis, architecture analysis |
3636
| `write` | Create/Modify/Delete | No — requires explicit intent | Implementation, bug fixes, refactoring |
37-
| `review` | Read-only (git-aware) | Yes | **Codex only**. Uncommitted changes, branch diffs, specific commits |
38-
3937
> `--mode` is the **authoritative** permission control for ccw cli. The `MODE:` field inside prompt text is a hint for the agent — both should be consistent, but `--mode` governs actual behavior.
4038
41-
**Codex review mode**: Target flags (`--uncommitted`, `--base`, `--commit`) are codex-only and mutually exclusive with `-p`.
42-
4339
---
4440

4541
## 2. Configuration

.claude/skills/ccw-help/command.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@
4848
{
4949
"name": "codex-review",
5050
"command": "/cli:codex-review",
51-
"description": "Interactive code review using Codex CLI via ccw endpoint with configurable review target, model, and custom instructions",
52-
"arguments": "[--uncommitted|--base <branch>|--commit <sha>] [--model <model>] [--title <title>] [prompt]",
51+
"description": "Prompt-based code review using Codex CLI via ccw endpoint with --mode analysis",
52+
"arguments": "[--model <model>] [--title <title>] [prompt]",
5353
"category": "cli",
5454
"subcategory": null,
5555
"usage_scenario": "analysis",

.claude/skills/ccw-help/index/all-commands.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@
3535
{
3636
"name": "codex-review",
3737
"command": "/cli:codex-review",
38-
"description": "Interactive code review using Codex CLI via ccw endpoint with configurable review target, model, and custom instructions",
39-
"arguments": "[--uncommitted|--base <branch>|--commit <sha>] [--model <model>] [--title <title>] [prompt]",
38+
"description": "Prompt-based code review using Codex CLI via ccw endpoint with --mode analysis",
39+
"arguments": "[--model <model>] [--title <title>] [prompt]",
4040
"category": "cli",
4141
"subcategory": null,
4242
"usage_scenario": "analysis",

.claude/skills/ccw-help/index/by-category.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@
5252
{
5353
"name": "codex-review",
5454
"command": "/cli:codex-review",
55-
"description": "Interactive code review using Codex CLI via ccw endpoint with configurable review target, model, and custom instructions",
56-
"arguments": "[--uncommitted|--base <branch>|--commit <sha>] [--model <model>] [--title <title>] [prompt]",
55+
"description": "Prompt-based code review using Codex CLI via ccw endpoint with --mode analysis",
56+
"arguments": "[--model <model>] [--title <title>] [prompt]",
5757
"category": "cli",
5858
"subcategory": null,
5959
"usage_scenario": "analysis",

.claude/skills/ccw-help/index/by-use-case.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -302,8 +302,8 @@
302302
{
303303
"name": "codex-review",
304304
"command": "/cli:codex-review",
305-
"description": "Interactive code review using Codex CLI via ccw endpoint with configurable review target, model, and custom instructions",
306-
"arguments": "[--uncommitted|--base <branch>|--commit <sha>] [--model <model>] [--title <title>] [prompt]",
305+
"description": "Prompt-based code review using Codex CLI via ccw endpoint with --mode analysis",
306+
"arguments": "[--model <model>] [--title <title>] [prompt]",
307307
"category": "cli",
308308
"subcategory": null,
309309
"usage_scenario": "analysis",

.claude/skills/workflow-lite-execute/SKILL.md

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ function selectExecutionOptions() {
9595
multiSelect: false,
9696
options: [
9797
{ label: "Gemini Review", description: "Gemini CLI: git diff quality review" },
98-
{ label: "Codex Review", description: "Codex CLI: git-aware code review (--mode review)" },
98+
{ label: "Codex Review", description: "Codex CLI: prompt-based code quality review (--mode analysis)" },
9999
{ label: "Agent Review", description: "@code-reviewer agent" },
100100
{ label: "Skip", description: "No code review" }
101101
]
@@ -435,7 +435,12 @@ Run \`git diff --name-only HEAD~${getTasks(planObject).length}..HEAD\` to identi
435435
436436
```javascript
437437
const reviewId = `${sessionId}-code-review`
438-
Bash(`ccw cli -p "Review code changes for quality, correctness, security, and pattern compliance. Focus: ${planObject.summary}" --tool codex --mode review --id ${reviewId}`, { run_in_background: true })
438+
Bash(`ccw cli -p "PURPOSE: Post-execution code quality review for: ${planObject.summary}
439+
TASK: • Run git diff to identify all changes • Review each changed file for quality, correctness, security • Check pattern compliance with existing codebase • Identify potential bugs, edge cases, performance issues
440+
MODE: analysis
441+
CONTEXT: @**/* | Memory: lite-execute completed, reviewing code quality
442+
EXPECTED: Per-file review with severity levels (Critical/High/Medium/Low), file:line references, fix suggestions, overall verdict
443+
CONSTRAINTS: Read-only | Focus on code quality not convergence" --tool codex --mode analysis --rule analysis-review-code-quality --id ${reviewId}`, { run_in_background: true })
439444
// STOP - wait for hook callback
440445
```
441446

.claude/skills/workflow-lite-plan/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,7 @@ if (workflowPreferences.autoYes) {
531531
multiSelect: false,
532532
options: [
533533
{ label: "Gemini Review", description: "Gemini CLI: git diff quality review" },
534-
{ label: "Codex Review", description: "Codex CLI: git-aware code review (--mode review)" },
534+
{ label: "Codex Review", description: "Codex CLI: prompt-based code quality review (--mode analysis)" },
535535
{ label: "Agent Review", description: "@code-reviewer agent" },
536536
{ label: "Skip", description: "No code review" }
537537
]

.claude/skills/workflow-multi-cli-plan/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ AskUserQuestion({
185185
options: [
186186
{ label: "Skip", description: "No review" },
187187
{ label: "Gemini Review", description: "Gemini CLI tool" },
188-
{ label: "Codex Review", description: "codex review --uncommitted" },
188+
{ label: "Codex Review", description: "Codex CLI: prompt-based code quality review (--mode analysis)" },
189189
{ label: "Agent Review", description: "Current agent review" }
190190
]
191191
}

0 commit comments

Comments
 (0)