惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

Hugging Face - Blog
Hugging Face - Blog
Recent Announcements
Recent Announcements
V
Visual Studio Blog
博客园 - 叶小钗
H
Help Net Security
aimingoo的专栏
aimingoo的专栏
宝玉的分享
宝玉的分享
U
Unit 42
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
F
Fortinet All Blogs
V
V2EX
Stack Overflow Blog
Stack Overflow Blog
WordPress大学
WordPress大学
D
DataBreaches.Net
J
Java Code Geeks
H
Hackread – Cybersecurity News, Data Breaches, AI and More
A
About on SuperTechFans
酷 壳 – CoolShell
酷 壳 – CoolShell
量子位
C
Check Point Blog
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
小众软件
小众软件
Microsoft Azure Blog
Microsoft Azure Blog
M
MIT News - Artificial intelligence

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(skills): default codex review to yolo mode · openclaw...
steipete · 2026-05-17 · via Recent Commits to openclaw:main

@@ -6,9 +6,13 @@ usage() {

66

Usage: codex-review [options]

7788

Options:

9-

--mode auto|local|branch Target selection. Default: auto.

9+

--mode auto|local|branch|commit

10+

Target selection. Default: auto.

1011

--base REF Base ref for branch review. Default: PR base or origin/main.

12+

--commit REF Commit ref for commit review. Default: HEAD.

1113

--codex-bin PATH Codex binary. Default: codex.

14+

--full-access Keep yolo/full-access mode enabled. Default.

15+

--no-yolo Run nested Codex review with normal sandbox/approval prompts.

1216

--output FILE Also save output to file.

1317

--parallel-tests CMD Run review and test command concurrently.

1418

--dry-run Print selected commands, do not run.

@@ -17,13 +21,17 @@ Options:

1721

Modes:

1822

local codex review --uncommitted

1923

branch codex review --base <base>

24+

commit codex review --commit <commit>

2025

auto dirty tree -> local, else PR/current branch -> branch

2126

EOF

2227

}

23282429

mode=auto

2530

base_ref=

31+

commit_ref=HEAD

2632

codex_bin=${CODEX_BIN:-codex}

33+

codex_args=()

34+

yolo=${CODEX_REVIEW_YOLO:-1}

2735

output=${CODEX_REVIEW_OUTPUT:-}

2836

parallel_tests=

2937

dry_run=false

@@ -38,10 +46,22 @@ while [[ $# -gt 0 ]]; do

3846

base_ref=${2:-}

3947

shift 2

4048

;;

49+

--commit)

50+

commit_ref=${2:-}

51+

shift 2

52+

;;

4153

--codex-bin)

4254

codex_bin=${2:-}

4355

shift 2

4456

;;

57+

--full-access)

58+

yolo=1

59+

shift

60+

;;

61+

--no-yolo)

62+

yolo=0

63+

shift

64+

;;

4565

--output)

4666

output=${2:-}

4767

shift 2

@@ -65,8 +85,13 @@ while [[ $# -gt 0 ]]; do

6585

esac

6686

done

678788+

case "$yolo" in

89+

0|false|False|FALSE|no|No|NO|off|Off|OFF) ;;

90+

*) codex_args+=(--dangerously-bypass-approvals-and-sandbox) ;;

91+

esac

92+6893

case "$mode" in

69-

auto|local|branch) ;;

94+

auto|local|branch|commit) ;;

7095

*)

7196

echo "invalid --mode: $mode" >&2

7297

exit 2

@@ -99,6 +124,8 @@ fi

99124

review_kind=

100125

if [[ "$mode" == local || ( "$mode" == auto && "$dirty" == true ) ]]; then

101126

review_kind=local

127+

elif [[ "$mode" == commit ]]; then

128+

review_kind=commit

102129

elif [[ "$mode" == branch || ( "$mode" == auto && -n "$current_branch" && "$current_branch" != "main" ) ]]; then

103130

review_kind=branch

104131

else

@@ -107,9 +134,11 @@ else

107134

fi

108135109136

if [[ "$review_kind" == local ]]; then

110-

review_cmd=("$codex_bin" review --uncommitted)

137+

review_cmd=("$codex_bin" "${codex_args[@]}" review --uncommitted)

138+

elif [[ "$review_kind" == commit ]]; then

139+

review_cmd=("$codex_bin" "${codex_args[@]}" review --commit "$commit_ref")

111140

else

112-

review_cmd=("$codex_bin" review --base "$base_ref")

141+

review_cmd=("$codex_bin" "${codex_args[@]}" review --base "$base_ref")

113142

fi

114143115144

printf 'codex-review target: %s\n' "$review_kind"

@@ -140,28 +169,92 @@ if [[ "$review_kind" == branch ]]; then

140169

}

141170

fi

142171172+

review_output=$output

173+

review_output_is_temp=false

174+

if [[ -z "$review_output" ]]; then

175+

review_output=$(mktemp)

176+

review_output_is_temp=true

177+

fi

178+179+

cleanup() {

180+

if [[ "${review_output_is_temp:-false}" == true && -n "${review_output:-}" ]]; then

181+

rm -f "$review_output"

182+

fi

183+

}

184+

trap cleanup EXIT

185+143186

run_review() {

144-

if [[ -n "$output" ]]; then

145-

mkdir -p "$(dirname "$output")"

146-

"${review_cmd[@]}" 2>&1 | tee "$output"

187+

mkdir -p "$(dirname "$review_output")"

188+

"${review_cmd[@]}" 2>&1 | tee "$review_output"

189+

}

190+191+

elapsed_since() {

192+

local started_at=$1

193+

local finished_at

194+

finished_at=$(date +%s)

195+

printf '%s\n' "$((finished_at - started_at))"

196+

}

197+198+

format_elapsed() {

199+

local seconds=$1

200+

if (( seconds < 60 )); then

201+

printf '%ss\n' "$seconds"

147202

else

148-

"${review_cmd[@]}"

203+

printf '%sm%ss\n' "$((seconds / 60))" "$((seconds % 60))"

204+

fi

205+

}

206+207+

review_output_empty() {

208+

[[ ! -s "$review_output" ]] || ! grep -q '[^[:space:]]' "$review_output"

209+

}

210+211+

review_output_has_findings() {

212+

grep -Eq '\[P[0-3]\]' "$review_output"

213+

}

214+215+

report_clean_review_or_fail() {

216+

local elapsed_text

217+

elapsed_text=$(format_elapsed "${review_elapsed_seconds:-0}")

218+219+

if review_output_has_findings; then

220+

printf 'codex-review complete after %s\n' "$elapsed_text"

221+

printf 'codex-review findings: accepted/actionable findings reported\n'

222+

return 1

223+

fi

224+

if review_output_empty; then

225+

printf 'codex-review complete after %s; no output\n' "$elapsed_text"

226+

return 1

149227

fi

228+

printf 'codex-review complete after %s\n' "$elapsed_text"

229+

printf 'codex-review clean: no accepted/actionable findings reported\n'

150230

}

151231152232

if [[ -z "$parallel_tests" ]]; then

233+

review_started_at=$(date +%s)

234+

set +e

153235

run_review

154-

exit $?

236+

review_status=$?

237+

review_elapsed_seconds=$(elapsed_since "$review_started_at")

238+

set -e

239+

if [[ "$review_status" == 0 ]]; then

240+

report_clean_review_or_fail

241+

exit $?

242+

fi

243+

exit "$review_status"

155244

fi

156245157246

review_status_file=$(mktemp)

247+

review_elapsed_file=$(mktemp)

158248

tests_status_file=$(mktemp)

159249160250

(

161251

set +e

252+

review_started_at=$(date +%s)

162253

run_review

163254

status=$?

255+

elapsed=$(elapsed_since "$review_started_at")

164256

printf '%s\n' "$status" > "$review_status_file"

257+

printf '%s\n' "$elapsed" > "$review_elapsed_file"

165258

) &

166259

review_pid=$!

167260

@@ -177,12 +270,15 @@ wait "$review_pid" || true

177270

wait "$tests_pid" || true

178271179272

review_status=$(cat "$review_status_file")

273+

review_elapsed_seconds=$(cat "$review_elapsed_file")

180274

tests_status=$(cat "$tests_status_file")

181-

rm -f "$review_status_file" "$tests_status_file"

275+

rm -f "$review_status_file" "$review_elapsed_file" "$tests_status_file"

182276183277

printf 'codex-review exit: %s\n' "$review_status"

184278

printf 'tests exit: %s\n' "$tests_status"

185279186280

if [[ "$review_status" != 0 || "$tests_status" != 0 ]]; then

187281

exit 1

188282

fi

283+284+

report_clean_review_or_fail