docs: clarify /new vs /reset semantics in slash-commands#81073
docs: clarify /new vs /reset semantics in slash-commands#81073clawsweeper[bot] merged 5 commits into
Conversation
|
Codex review: passed. Workflow note: Future ClawSweeper reviews update this same comment in place. How this review workflow works
Summary Reproducibility: yes. Reading current main reproduces the misleading docs line at PR rating 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. PR egg Rarity: 🥚 common. What is this egg doing here?
Real behavior proof Next step before merge Security Review detailsBest possible solution: Merge the one-line docs correction through the exact-head automerge path; treat command renaming or stronger reset-warning policy as separate product work. Do we have a high-confidence way to reproduce the issue? Yes. Reading current main reproduces the misleading docs line at Is this the best way to solve the issue? Yes. For this PR's scope, replacing the misleading sentence is the narrowest maintainable fix; the optional Label justifications:
What I checked:
Likely related people:
Codex review notes: model gpt-5.5, reasoning high; reviewed against edcf862da5d0. |
|
Thanks @clawsweeper — addressed the P3 finding in Final line:
|
|
FYI on the two red checks (
Happy to rebase once main has been patched, or you can apply a |
|
@clawsweeper automerge |
|
🦞✅ Source: What merged:
Automerge notes:
The automerge loop is complete. Automerge progress:
|
Summary: - The PR changes one bullet in `docs/tools/slash-commands.md` to distinguish `/new` from `/reset` and remove the misleading alias wording. - Reproducibility: yes. Reading current main reproduces the misleading docs line at `docs/tools/slash-commands.md:127`, and adjacent source/tests show `/new` and `/reset` take different paths in the Control UI. Automerge notes: - PR branch already contained follow-up commit before automerge: docs/slash-commands: drop inaccurate Control UI/ACP cross-reference (… - PR branch already contained follow-up commit before automerge: Merge branch 'main' into docs/fix-reset-alias-misleading Validation: - ClawSweeper review passed for head bb92b6050aab85335061028404d59c762cee3125. - Required merge gates passed before the squash merge. Prepared head SHA: bb92b6050aab85335061028404d59c762cee3125 Review: openclaw/openclaw#81073 (comment) Co-authored-by: Md. Al-Mosabbir Rakib <mrakib50.cse@gmail.com> Co-authored-by: Md. Al-Mosabbir Rakib <34891461+mosabbirrakib@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
Summary: - The PR changes one bullet in `docs/tools/slash-commands.md` to distinguish `/new` from `/reset` and remove the misleading alias wording. - Reproducibility: yes. Reading current main reproduces the misleading docs line at `docs/tools/slash-commands.md:127`, and adjacent source/tests show `/new` and `/reset` take different paths in the Control UI. Automerge notes: - PR branch already contained follow-up commit before automerge: docs/slash-commands: drop inaccurate Control UI/ACP cross-reference (… - PR branch already contained follow-up commit before automerge: Merge branch 'main' into docs/fix-reset-alias-misleading Validation: - ClawSweeper review passed for head bb92b6050aab85335061028404d59c762cee3125. - Required merge gates passed before the squash merge. Prepared head SHA: bb92b6050aab85335061028404d59c762cee3125 Review: openclaw/openclaw#81073 (comment) Co-authored-by: Md. Al-Mosabbir Rakib <mrakib50.cse@gmail.com> Co-authored-by: Md. Al-Mosabbir Rakib <34891461+mosabbirrakib@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>
Summary
docs/tools/slash-commands.md:127summarizes/newand/resetas aliases. They are not./newarchives the current session and starts a fresh one;/resetwipes the current session in place. The line directly below already admits this (Control UI intercepts/newto create a fresh dashboard session, while typed/resetruns the Gateway's in-place reset), so the summary contradicts itself./resetthinking they can recover the old context. They cannot — the previous transcript is gone. Reported as docs: "/reset is the reset alias" is misleading — /new and /reset are not aliases #76592 with a real user-impact framing ("until they lose their session data").docs/tools/slash-commands.mdto give an accurate one-line distinction and explicitly state they are not aliases. Pointer to the existing follow-up bullets for the Control UIsession.dmScope: "main"case and the ACP-bound channel case where they do collapse to an in-place reset./resetto/clearor/wipe) because that is a behavior/UX change, not a docs fix.Change Type (select all)
Scope (select all touched areas)
Linked Issue/PR
Real behavior proof (required for external PRs)
Behavior or issue addressed:
docs/tools/slash-commands.md:127documented/resetas "the reset alias" of/new. That phrasing told users the commands were interchangeable, while the very next bullet in the same file already noted the Control UI//resetpaths behave differently. This PR replaces the misleading bullet with an accurate one-line distinction (/newarchives + starts fresh;/resetwipes in place; not aliases) and points to the existing follow-up bullets for the contexts where both collapse to the same in-place reset.Real environment tested: Local macOS workstation with Node 22.17.1 against the new branch
docs/fix-reset-alias-misleadingbranched fromopenclaw/openclaw:mainat7a9613e66b. Verified the misleading wording occurrence count, the rewritten bullet, MDX validity, doc formatting, and internal link health using the repo's own docs scripts vianode.Exact steps or command run after this patch:
Evidence after fix (screenshot, recording, terminal capture, console output, redacted runtime log, linked artifact, or copied live output): Terminal output captured from the steps above against the branch tip:
Observed result after fix: The repo-wide grep for "reset alias" or "is the reset alias" returns zero matches, confirming the misleading phrasing has been removed from both
docs/andsrc/. The rewritten bullet at line 127 reads as a clear one-sentence distinction (/newarchives + starts fresh;/resetwipes in place) followed by an explicit "They are not aliases" statement and a pointer to the existing follow-up bullet that already documents the Control UIsession.dmScope: "main"carve-out and the ACP-bound channel context (the only places where both commands collapse to the same in-place reset). MDX parses cleanly, oxfmt reports the touched file is already formatted, and the docs link audit reports zero broken internal links across 4345 checked references. The diff is exactly 1 line changed on 1 file.What was not tested: Production Mintlify rendering at
docs.openclaw.ai/tools/slash-commands#sessions-and-runs, which only resolves after merge. The actual runtime behavior of/newvs/resetwas not exercised because this PR makes no runtime change; the new wording is verified against the existing, separately-documented behavior in the next bullet of the same accordion (Control UI dmScope main carve-out) and the cross-references indocs/tools/acp-agents.mdand the per-channel docs.Before evidence (optional but encouraged): The misleading line as it stood before this patch:
Public-docs URL of the existing page on production Mintlify: https://docs.openclaw.ai/tools/slash-commands. The same accordion already contained the line
Typed /reset still runs the Gateway's in-place reset.directly below the removed bullet, which is the self-contradiction the original issue flagged.Root Cause (if applicable)
N/A — this is a docs-only correction, not a bug fix.
Regression Test Plan (if applicable)
N/A — no behavior change. A docs/grammar regression of this exact kind would be caught by a reviewer reading the file or by a maintainer noticing the contradiction across adjacent bullets. There is no narrowly-scoped automated test that would flag a misleading English summary inside an accordion bullet.
User-visible / Behavior Changes
https://docs.openclaw.ai/tools/slash-commandswill no longer be told that/resetis an alias of/new. They will instead see a one-sentence distinction (/newarchives + starts fresh;/resetwipes in place; not aliases) plus a pointer to the existing follow-up bullets that already document the Control UI and ACP-bound channel contexts where both collapse to the same in-place reset.Diagram (if applicable)
N/A
Security Impact (required)
Repro + Verification
Environment
Steps
docs/tools/slash-commands.mdand locate theSessions and runsaccordion underCore built-in commands./resetbehavior.Expected
/newand/resetaccurately distinguishes the two commands and does not call/resetan alias./newto create a fresh dashboard session while typed/resetruns the Gateway's in-place reset.Actual
/new [model]archives the current session and starts a fresh one;/resetwipes the current session in place. They are not aliases — see the bullet below for Control UI and ACP-bound channel contexts where both collapse to the same in-place reset."reset alias"phrasing returns zero matches indocs/andsrc/.Evidence
grep,check-docs-mdx,format-docs --check,docs-link-audit, andgit diff --statabove)Human Verification (required)
/reset is the reset alias) is the only occurrence in the repo and has been removed./resetruns in-place reset).docs/tools/acp-agents.md:459,docs/channels/discord.md:846,docs/channels/imessage.md:331, anddocs/channels/matrix.md:544, which describe the ACP-bound channel context where both commands do collapse to an in-place reset of the bound session.docs/(grep across the tree)./reset softcontinues to be documented separately on the next-next bullet and is unaffected by this change.docs.openclaw.ai/tools/slash-commandsafter merge./resetto/clearor/wipe) — out of scope for a docs PR.Review Conversations
Compatibility / Migration
Risks and Mitigations
Risk: the new wording overstates the difference and confuses users in ACP-bound channels who do experience both commands as equivalent.
docs/tools/acp-agents.md,docs/channels/discord.md,docs/channels/imessage.md,docs/channels/matrix.md) continue to document the "both reset the bound ACP session in place" behavior in their own context. The cross-reference keeps users from being surprised.Risk: future code changes that genuinely make
/resetan alias of/newwould silently leave this docs line wrong.session.dmScope: "main") and the ACP-bound channel binding mechanism both rely on the commands behaving differently in the default case. If the policy is ever reversed, the file will need to be updated again — but that is true of any behavior-specific documentation.