docs: record ci snapshot closeout notes · openclaw/openclaw@5089459
steipete
·
2026-05-18
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
@@ -136,3 +136,10 @@ Include:
|
136 | 136 | - the clean review result from the final helper/review run, or why a remaining finding was consciously rejected |
137 | 137 | |
138 | 138 | Do not run another Codex review solely to improve the final report wording. If the final helper run exited 0 and produced no accepted/actionable findings, report that exact run as clean. |
| 139 | + |
| 140 | +## PR / CI Closeout |
| 141 | + |
| 142 | +- Prefer direct run/job APIs after CI starts: `gh run view <run-id> --json jobs`; use PR rollup only for final mergeability. |
| 143 | +- After rebase, compare `origin/main..HEAD`; drop CI-fix commits already upstream before pushing. |
| 144 | +- For prompt snapshot CI failures, prove/generate with Linux Node 24 before rerunning the failed job. |
| 145 | +- Update PR body once near the final head unless proof labels are missing or stale enough to block CI. |
| Original file line number | Diff line number | Diff line change |
|---|
@@ -27,7 +27,7 @@ Prove the touched surface first. Do not reflexively run the whole suite.
|
27 | 27 | use the Crabbox wrapper with the provider that matches the proof surface. |
28 | 28 | For maintainer heavy `pnpm` gates, that is usually delegated Blacksmith |
29 | 29 | Testbox through Crabbox, e.g. `node scripts/crabbox-wrapper.mjs run |
30 | | - --provider blacksmith-testbox ... -- pnpm check:changed`. For direct AWS |
| 30 | +--provider blacksmith-testbox ... -- pnpm check:changed`. For direct AWS |
31 | 31 | Crabbox proof, omit `--provider` and let `.crabbox.yaml` choose AWS. |
32 | 32 | - workflow-only: `git diff --check`, workflow syntax/lint (`actionlint` when available) |
33 | 33 | - docs-only: `pnpm docs:list`, docs formatter/lint only if docs tooling changed or requested |
@@ -131,6 +131,8 @@ gh run view <run-id> --job <job-id> --log
|
131 | 131 | - Check exact SHA. Ignore newer unrelated `main` unless asked. |
132 | 132 | - For cancelled same-branch runs, confirm whether a newer run superseded it. |
133 | 133 | - Fetch full logs only for failed or relevant jobs. |
| 134 | +- Prefer `gh run view <run-id> --json jobs` over PR rollup while debugging; rollup can be stale/noisy. |
| 135 | +- For `prompt:snapshots:check` failures, treat Linux Node 24 as CI truth. If macOS passes but CI drifts, reproduce in a Linux Node 24 container or Testbox, commit that generated output, then rerun. |
134 | 136 | |
135 | 137 | ## GitHub Release Workflows |
136 | 138 | |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -79,6 +79,7 @@ Skills own workflows; root owns hard policy and routing.
|
79 | 79 | - If proof is blocked, say exactly what is missing and why. |
80 | 80 | - Do not land related failing format/lint/type/build/tests. If unrelated on latest `origin/main`, say so with scoped proof. |
81 | 81 | - Docs/changelog-only and CI/workflow metadata-only: `git diff --check` plus relevant docs/workflow sanity; escalate only if scripts/config/generated/package/runtime behavior changed. |
| 82 | +- Prompt snapshots: CI truth is Linux Node 24. If macOS local passes but CI drifts, reproduce/generate in Linux before rerun. |
82 | 83 | |
83 | 84 | ## GitHub / PRs |
84 | 85 | |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。
