fix(status): correct pinned model clear hint#93231
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Codex review: passed. Reviewed June 15, 2026, 6:14 AM ET / 10:14 UTC. Summary PR surface: Source 0, Tests 0, Docs +4. Total +4 across 7 files. Reproducibility: yes. from source inspection. Current main and v2026.6.6 emit the old Review metrics: none identified. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Next step before merge
Security Review detailsBest possible solution: Land the narrow status-copy, docs, and test correction after exact-head checks and automerge gates pass. Do we have a high-confidence way to reproduce the issue? Yes, from source inspection. Current main and v2026.6.6 emit the old Is this the best way to solve the issue? Yes. Updating the status copy, docs, and focused expectations is the narrowest maintainable fix because the clearing behavior already exists; changing AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 0314819f918a. Label changesLabel justifications:
Evidence reviewedPR surface: Source 0, Tests 0, Docs +4. Total +4 across 7 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
@clawsweeper automerge |
|
🦞✅ Source: What merged:
Automerge notes:
The automerge loop is complete. Automerge progress:
|
eb9b5b2 to
1181624
Compare
Summary: - This PR changes pinned-session `/status` guidance, model-selection docs, and status tests to recommend `/model default` instead of `/model <configured>` or `/reset` for clearing a session model pin. - PR surface: Source 0, Tests 0, Docs +4. Total +4 across 7 files. - Reproducibility: yes. from source inspection. Current main and v2026.6.6 emit the old `/reset` hint, while `/model default` clears persisted model overrides and `/reset` intentionally preserves user-selected overrides. Automerge notes: - PR branch already contained follow-up commit before automerge: docs: align model clear hint docs - PR branch already contained follow-up commit before automerge: fix(status): correct pinned model clear hint Validation: - ClawSweeper review passed for head 1181624. - Required merge gates passed before the squash merge. Prepared head SHA: 1181624 Review: openclaw#93231 (comment) Co-authored-by: masonxhuang <[email protected]> Co-authored-by: Mason Huang <[email protected]> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: hxy91819 Co-authored-by: hxy91819 <[email protected]>
Summary: - This PR changes pinned-session `/status` guidance, model-selection docs, and status tests to recommend `/model default` instead of `/model <configured>` or `/reset` for clearing a session model pin. - PR surface: Source 0, Tests 0, Docs +4. Total +4 across 7 files. - Reproducibility: yes. from source inspection. Current main and v2026.6.6 emit the old `/reset` hint, while `/model default` clears persisted model overrides and `/reset` intentionally preserves user-selected overrides. Automerge notes: - PR branch already contained follow-up commit before automerge: docs: align model clear hint docs - PR branch already contained follow-up commit before automerge: fix(status): correct pinned model clear hint Validation: - ClawSweeper review passed for head 1181624. - Required merge gates passed before the squash merge. Prepared head SHA: 1181624 Review: openclaw#93231 (comment) Co-authored-by: masonxhuang <[email protected]> Co-authored-by: Mason Huang <[email protected]> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: hxy91819 Co-authored-by: hxy91819 <[email protected]>
Summary: - This PR changes pinned-session `/status` guidance, model-selection docs, and status tests to recommend `/model default` instead of `/model <configured>` or `/reset` for clearing a session model pin. - PR surface: Source 0, Tests 0, Docs +4. Total +4 across 7 files. - Reproducibility: yes. from source inspection. Current main and v2026.6.6 emit the old `/reset` hint, while `/model default` clears persisted model overrides and `/reset` intentionally preserves user-selected overrides. Automerge notes: - PR branch already contained follow-up commit before automerge: docs: align model clear hint docs - PR branch already contained follow-up commit before automerge: fix(status): correct pinned model clear hint Validation: - ClawSweeper review passed for head 1181624. - Required merge gates passed before the squash merge. Prepared head SHA: 1181624 Review: openclaw#93231 (comment) Co-authored-by: masonxhuang <[email protected]> Co-authored-by: Mason Huang <[email protected]> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: hxy91819 Co-authored-by: hxy91819 <[email protected]>
Summary
/model default/resetsuggestion because resets preserve user-selected model overridesTest evidence
env -u OPENCLAW_STATE_DIR -u OPENCLAW_VITEST_INCLUDE_FILE OPENCLAW_HOME=/tmp/openclaw-test-home-status-reset XDG_CONFIG_HOME=/tmp/openclaw-test-home-status-reset HOME=/tmp/openclaw-test-home-status-reset node scripts/test-projects.mjs src/auto-reply/status.test.ts src/commands/status.command-sections.test.tsgit diff --check HEAD~1..HEAD