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

推荐订阅源

WordPress大学
WordPress大学
L
LangChain Blog
酷 壳 – CoolShell
酷 壳 – CoolShell
罗磊的独立博客
J
Java Code Geeks
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
博客园 - 叶小钗
小众软件
小众软件
博客园 - Franky
D
Docker
Google DeepMind News
Google DeepMind News
Microsoft Azure Blog
Microsoft Azure Blog
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
U
Unit 42
宝玉的分享
宝玉的分享
C
Check Point Blog
B
Blog
V
V2EX
博客园 - 三生石上(FineUI控件)
MyScale Blog
MyScale Blog
The Cloudflare Blog
博客园 - 聂微东
博客园_首页
Engineering at Meta
Engineering at Meta

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
chore: add codex review skill · openclaw/openclaw@beea866
steipete · 2026-05-14 · via Recent Commits to openclaw:main

@@ -0,0 +1,188 @@

1+

#!/usr/bin/env bash

2+

set -euo pipefail

3+4+

usage() {

5+

cat <<'EOF'

6+

Usage: codex-review [options]

7+8+

Options:

9+

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

10+

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

11+

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

12+

--output FILE Also save output to file.

13+

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

14+

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

15+

-h, --help Show help.

16+17+

Modes:

18+

local codex review --uncommitted

19+

branch codex review --base <base>

20+

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

21+

EOF

22+

}

23+24+

mode=auto

25+

base_ref=

26+

codex_bin=${CODEX_BIN:-codex}

27+

output=${CODEX_REVIEW_OUTPUT:-}

28+

parallel_tests=

29+

dry_run=false

30+31+

while [[ $# -gt 0 ]]; do

32+

case "$1" in

33+

--mode)

34+

mode=${2:-}

35+

shift 2

36+

;;

37+

--base)

38+

base_ref=${2:-}

39+

shift 2

40+

;;

41+

--codex-bin)

42+

codex_bin=${2:-}

43+

shift 2

44+

;;

45+

--output)

46+

output=${2:-}

47+

shift 2

48+

;;

49+

--parallel-tests)

50+

parallel_tests=${2:-}

51+

shift 2

52+

;;

53+

--dry-run)

54+

dry_run=true

55+

shift

56+

;;

57+

-h|--help)

58+

usage

59+

exit 0

60+

;;

61+

*)

62+

usage >&2

63+

exit 2

64+

;;

65+

esac

66+

done

67+68+

case "$mode" in

69+

auto|local|branch) ;;

70+

*)

71+

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

72+

exit 2

73+

;;

74+

esac

75+76+

git rev-parse --show-toplevel >/dev/null

77+78+

current_branch=$(git branch --show-current 2>/dev/null || true)

79+

dirty=false

80+

if [[ -n "$(git status --porcelain)" ]]; then

81+

dirty=true

82+

fi

83+84+

pr_url=

85+

if [[ -z "$base_ref" && "$mode" != local ]] && command -v gh >/dev/null 2>&1; then

86+

if pr_lines=$(gh pr view --json baseRefName,url --jq '[.baseRefName, .url] | @tsv' 2>/dev/null); then

87+

base_name=${pr_lines%%$'\t'*}

88+

pr_url=${pr_lines#*$'\t'}

89+

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

90+

base_ref="origin/$base_name"

91+

fi

92+

fi

93+

fi

94+95+

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

96+

base_ref=origin/main

97+

fi

98+99+

review_kind=

100+

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

101+

review_kind=local

102+

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

103+

review_kind=branch

104+

else

105+

echo "no review target: clean main checkout and no forced mode" >&2

106+

exit 1

107+

fi

108+109+

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

110+

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

111+

else

112+

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

113+

fi

114+115+

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

116+

printf 'branch: %s\n' "${current_branch:-detached}"

117+

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

118+

printf 'pr: %s\n' "$pr_url"

119+

fi

120+

printf 'review:'

121+

printf ' %q' "${review_cmd[@]}"

122+

printf '\n'

123+

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

124+

printf 'tests: %s\n' "$parallel_tests"

125+

fi

126+

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

127+

printf 'fetch: git fetch origin --quiet\n'

128+

fi

129+

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

130+

printf 'output: %s\n' "$output"

131+

fi

132+133+

if [[ "$dry_run" == true ]]; then

134+

exit 0

135+

fi

136+137+

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

138+

git fetch origin --quiet || {

139+

echo "warning: git fetch origin failed; reviewing with existing refs" >&2

140+

}

141+

fi

142+143+

run_review() {

144+

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

145+

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

146+

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

147+

else

148+

"${review_cmd[@]}"

149+

fi

150+

}

151+152+

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

153+

run_review

154+

exit $?

155+

fi

156+157+

review_status_file=$(mktemp)

158+

tests_status_file=$(mktemp)

159+160+

(

161+

set +e

162+

run_review

163+

status=$?

164+

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

165+

) &

166+

review_pid=$!

167+168+

(

169+

set +e

170+

bash -lc "$parallel_tests"

171+

status=$?

172+

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

173+

) &

174+

tests_pid=$!

175+176+

wait "$review_pid" || true

177+

wait "$tests_pid" || true

178+179+

review_status=$(cat "$review_status_file")

180+

tests_status=$(cat "$tests_status_file")

181+

rm -f "$review_status_file" "$tests_status_file"

182+183+

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

184+

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

185+186+

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

187+

exit 1

188+

fi