Skip to content

fix(status): correct pinned model clear hint#93231

Merged
clawsweeper[bot] merged 3 commits into
mainfrom
fix/status-reset-clear-hint
Jun 15, 2026
Merged

fix(status): correct pinned model clear hint#93231
clawsweeper[bot] merged 3 commits into
mainfrom
fix/status-reset-clear-hint

Conversation

@hxy91819

Copy link
Copy Markdown
Member

Summary

  • update pinned-session /status guidance to use /model default
  • remove misleading /reset suggestion because resets preserve user-selected model overrides
  • update status message and CLI status section tests

Test 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.ts
    • passed 2 Vitest shards: 90 tests
  • git diff --check HEAD~1..HEAD

@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@openclaw-barnacle openclaw-barnacle Bot added commands Command implementations size: XS maintainer Maintainer-authored PR labels Jun 15, 2026
@clawsweeper

clawsweeper Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Codex review: passed. Reviewed June 15, 2026, 6:14 AM ET / 10:14 UTC.

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.

Review metrics: none identified.

Merge readiness
Overall: 🐚 platinum hermit
Proof: 🌊 off-meta tidepool
Patch quality: 🐚 platinum hermit
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

  • none.

Next step before merge

  • [P2] No repair job is needed because the automerge-opted PR has no actionable findings; let exact-head checks and protected-label merge gates finish.

Security
Cleared: The diff changes status copy, docs, and tests only; it does not touch dependencies, workflows, secrets, auth, package metadata, or code execution paths.

Review details

Best 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 /reset hint, while /model default clears persisted model overrides and /reset intentionally preserves user-selected overrides.

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 /reset would be a broader reset-contract change.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against 0314819f918a.

Label changes

Label justifications:

  • P3: This is a small user-facing status/docs correction with limited blast radius and no config, migration, provider, protocol, or dependency change.
  • rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🌊 off-meta tidepool and patch quality is 🐚 platinum hermit.
  • status: 🚀 automerge armed: This PR is in ClawSweeper's automerge lane. Not applicable: The contributor proof gate does not apply to this MEMBER-authored, maintainer-labeled PR; the PR body and live checks provide focused test evidence for the changed status paths.
Evidence reviewed

PR surface:

Source 0, Tests 0, Docs +4. Total +4 across 7 files.

View PR surface stats
Area Files Added Removed Net
Source 2 2 2 0
Tests 2 2 2 0
Docs 3 7 3 +4
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 7 11 7 +4

What I checked:

Likely related people:

  • steipete: git shortlog --all shows Peter Steinberger with the heaviest history across the central status/model docs and status helper files, including status helper consolidation and model-selection docs work. (role: area history owner; confidence: high; commits: 72dcf9422138, 756cb22f15a7, c72f539ced72; files: src/commands/status.command-sections.ts, docs/concepts/models.md, docs/cli/status.md)
  • kevinslin: git blame attributes the current old status hint and the existing /model default resolver lines to commit e82d19f, merged through feat(codex): add auto plugin approvals #92625. (role: current-line introducer; confidence: high; commits: e82d19fb06b5; files: src/status/status-message.ts, src/commands/status.command-sections.ts, src/auto-reply/reply/directive-handling.model-selection.ts)
  • frankekn: The reset-preservation behavior central to this fix was introduced through the merged reset model override work in fix(gateway): clear auto-fallback model override on session reset #63155. (role: reset behavior contributor; confidence: high; commits: 5d46e4dc4f45; files: src/config/sessions/reset-preserved-selection.ts, src/gateway/session-reset-service.ts)
  • vincentkoc: Recent history shows Vincent Koc touching adjacent rich status output in the same status-message surface and authoring the latest release commit checked for shipped behavior. (role: recent adjacent contributor; confidence: medium; commits: 5b460c4669d4, 8c802aa68351; files: src/status/status-message.ts)
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

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 keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. labels Jun 15, 2026
@openclaw-barnacle openclaw-barnacle Bot added the docs Improvements or additions to documentation label Jun 15, 2026
@hxy91819

Copy link
Copy Markdown
Member Author

@clawsweeper re-review

@clawsweeper

clawsweeper Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@hxy91819

Copy link
Copy Markdown
Member Author

@clawsweeper automerge

@clawsweeper

clawsweeper Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=1181624daa707485beebd1b48c912a6a31c7109c)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-06-15T10:15:55Z
Merge commit: 3ce3ed668d92

What merged:

  • 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

The automerge loop is complete.

Automerge progress:

  • 2026-06-15 09:46:11 UTC review queued eb9b5b294569 (queued)
  • 2026-06-15 09:52:46 UTC review passed eb9b5b294569 (structured ClawSweeper verdict: pass (sha=eb9b5b2945697d01869e796d30be03ef60642...)
  • 2026-06-15 10:09:26 UTC review queued 1181624daa70 (after repair)
  • 2026-06-15 10:15:38 UTC review passed 1181624daa70 (structured ClawSweeper verdict: pass (sha=1181624daa707485beebd1b48c912a6a31c71...)
  • 2026-06-15 10:15:57 UTC merged 1181624daa70 (merged by ClawSweeper automerge)

@clawsweeper clawsweeper Bot added clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane. and removed status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jun 15, 2026
@clawsweeper
clawsweeper Bot force-pushed the fix/status-reset-clear-hint branch from eb9b5b2 to 1181624 Compare June 15, 2026 10:09
@clawsweeper
clawsweeper Bot merged commit 3ce3ed6 into main Jun 15, 2026
163 checks passed
@clawsweeper
clawsweeper Bot deleted the fix/status-reset-clear-hint branch June 15, 2026 10:15
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jun 16, 2026
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]>
crh-code pushed a commit to crh-code/openclaw that referenced this pull request Jun 18, 2026
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]>
badgerbees pushed a commit to badgerbees/openclaw that referenced this pull request Jul 8, 2026
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge commands Command implementations docs Improvements or additions to documentation maintainer Maintainer-authored PR P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: XS status: 🚀 automerge armed This PR is in ClawSweeper's automerge lane.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants