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

推荐订阅源

Y
Y Combinator Blog
The GitHub Blog
The GitHub Blog
Vercel News
Vercel News
D
DataBreaches.Net
MongoDB | Blog
MongoDB | Blog
H
Help Net Security
小众软件
小众软件
美团技术团队
T
The Blog of Author Tim Ferriss
爱范儿
爱范儿
D
Docker
Martin Fowler
Martin Fowler
大猫的无限游戏
大猫的无限游戏
博客园 - 聂微东
Blog — PlanetScale
Blog — PlanetScale
H
Hackread – Cybersecurity News, Data Breaches, AI and More
罗磊的独立博客
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
V
V2EX
S
SegmentFault 最新的问题
云风的 BLOG
云风的 BLOG
B
Blog
雷峰网
雷峰网
The Cloudflare Blog

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(autoreview): route OpenClaw validation to Testbox ·...
vincentkoc · 2026-05-18 · via Recent Commits to openclaw:main

@@ -51,6 +51,7 @@ yolo=${AUTOREVIEW_YOLO:-${CODEX_REVIEW_YOLO:-1}}

5151

output=${AUTOREVIEW_OUTPUT:-${CODEX_REVIEW_OUTPUT:-}}

5252

parallel_tests=

5353

dry_run=false

54+

codex_review_prompt=

54555556

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

5657

case "$1" in

@@ -201,6 +202,19 @@ else

201202

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

202203

fi

203204205+

repo_url=$(git -C "$repo_root" config --get remote.origin.url 2>/dev/null || true)

206+

if [[ "$repo_url" == *"openclaw/openclaw"* ]]; then

207+

codex_review_prompt=$(cat <<'EOF'

208+

OpenClaw autoreview validation policy:

209+

- Review the diff by reading code, tests, and dependency contracts.

210+

- Do not run local memory-heavy Node validation from review mode. This includes local pnpm checks/tests, Vitest, tsgo, npm test, and node scripts/run-vitest.mjs.

211+

- If runtime proof is needed, use existing proof or route validation through OPENCLAW_TESTBOX=1 / Crabbox / Blacksmith Testbox and report the exact provider and id.

212+

- If remote validation is not necessary for the finding, state the targeted proof that should be run instead of starting local tests.

213+

EOF

214+

)

215+

review_cmd+=(-)

216+

fi

217+204218

printf 'autoreview target: %s\n' "$review_kind"

205219

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

206220

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

@@ -221,6 +235,9 @@ if [[ "$reviewer" == auto || "$reviewer" == codex ]]; then

221235

printf 'review:'

222236

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

223237

printf '\n'

238+

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

239+

printf 'review policy: OpenClaw Crabbox/Testbox-aware validation prompt injected\n'

240+

fi

224241

else

225242

printf 'review: %s prompt review\n' "$reviewer"

226243

fi

@@ -265,7 +282,11 @@ trap cleanup EXIT

265282266283

run_review() {

267284

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

268-

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

285+

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

286+

printf '%s\n' "$codex_review_prompt" | "${review_cmd[@]}" 2>&1 | tee "$review_output"

287+

else

288+

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

289+

fi

269290

}

270291271292

diff_for_review() {

@@ -306,6 +327,7 @@ Rules:

306327

- Review the proposed code change as a closeout reviewer.

307328

- Focus on the diff below. If your CLI exposes read-only repository tools, inspect surrounding code and tests to verify findings; never modify files.

308329

- Do not modify files.

330+

- In OpenClaw, do not run local memory-heavy Node validation from review mode. Avoid local pnpm checks/tests, Vitest, tsgo, npm test, and node scripts/run-vitest.mjs. If runtime proof is needed, use existing proof or route validation through OPENCLAW_TESTBOX=1 / Crabbox / Blacksmith Testbox and report the exact provider and id.

309331

- Report only discrete, actionable issues introduced by this change.

310332

- Prioritize correctness, regressions, security, data loss, performance cliffs, and missing tests that would catch a real bug.

311333

- Do not report pre-existing issues, speculative risks, broad rewrites, style nits, changelog gaps, or findings that depend on unstated assumptions.