gut(docs): remove /think directive references — Area 6 of #2336 (#2466)#2472
Merged
alexey-pelykh merged 1 commit intomainfrom Apr 22, 2026
Merged
gut(docs): remove /think directive references — Area 6 of #2336 (#2466)#2472alexey-pelykh merged 1 commit intomainfrom
alexey-pelykh merged 1 commit intomainfrom
Conversation
Part of the umbrella thinking-level / model-selection sweep (#2336). Sibling PR #2463 landed Areas 1+2+4 (runtime `/think` parsing, helpers, and passthrough plumbing); this PR covers Area 6 (docs). RemoteClaw does not control thinking levels — CLI runtimes (Claude, Gemini, Codex, OpenCode) own that decision. User-facing documentation advertising `/think` as a directive was inherited from upstream OpenClaw and is now stale. ## Removals - `docs/tools/slash-commands.md`: - L17: drop `/think` from the directives list. - L114: delete the `/think <off|minimal|low|medium|high|xhigh>` command entry. - L133: drop `/think: high` from the colon-separator example (`/send: on`, `/help:` still illustrate both syntactic variants). - `docs/concepts/context.md:139`: drop `/think` from the directives list. - `docs/web/tui.md:94`: delete the `/think <off|...>` TUI entry. - `docs/channels/group-messages.md`: - L18: drop `or /think high` (kept `/verbose on` as primary example). - L70: drop `/think high` from the session-level directives list. ## Verification - `git grep "/think\b" docs/` → 0 hits (matches parent #2336 AC). - `cd docs && npm run build` → 296 pages built, no errors. - `pnpm format:docs:check` → clean. - `pnpm lint:docs` → no errors on edited files (pre-existing errors on unrelated files documented; identical on `origin/main`). ## Out of scope (tracked as informational for future #2336 waves) - `/thinking off` advice at `docs/help/faq.md:2354` — alias of the removed directive. - `/tools/thinking` URL references in `faq.md`, `messages.md`, `hubs.md` — pre-existing broken links; target page was gutted in an earlier wave. - `docs/web/tui.md:40` status-line string `model + think/fast/...` — UI indicator shorthand, not a slash command. - `src/agents/agent-helpers/errors.ts` still references `/think` in an error message — src cleanup belongs to Areas 1/2/4. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/thinkslash-command directive from user-facing docs./thinkas a RemoteClaw directive is stale upstream surface./thinkparsing, helpers, and passthrough plumbing). This completes the docs half.Changes
docs/tools/slash-commands.md— remove/thinkfrom the directives list (L17), delete the/think <off|minimal|low|medium|high|xhigh>command entry (L114), and drop/think: highfrom the colon-separator example (L133).docs/concepts/context.md:139— remove/thinkfrom the directives list.docs/web/tui.md:94— delete the/think <off|...>TUI entry.docs/channels/group-messages.md— dropor /think highat L18 and/think highat L70 from the session-level-directives examples;/verbose onremains as the primary example in both spots.Verification
git grep "/think\b" docs/→ 0 hits (matches the parent gut(auto-reply): finish thinking-level and model-selection sweep — follow-up to #2335 #2336 AC).cd docs && npm run build→ 296 pages built, no errors.pnpm format:docs:check→ clean.pnpm lint:docs→ no errors on the 4 edited files. Pre-existing lint errors on unrelated files (docs/start/*.mdx,docs/node_modules/**) are identical onorigin/main.Out of scope for Area 6 (tracked as informational)
Found during rescan, explicitly outside the 4-file scope defined by Area 6. Flag for future #2336 waves if not already covered elsewhere:
docs/help/faq.md:2354—set /thinking off for that agent(alias of/think).docs/help/faq.md:2901,docs/concepts/messages.md:145,docs/start/hubs.md:115— links to/tools/thinking; the target page was gutted in an earlier wave (pre-existing broken links, not introduced here).docs/web/tui.md:40status-line stringmodel + think/fast/...— UI indicator shorthand, not a slash command.src/agents/agent-helpers/errors.ts— error message still references/think; belongs to Areas 1/2/4 (src cleanup).AC interpretation note
Issue AC says
git grep "/think" docs/returns zero hits. Taken literally (substring match), that would also flagmodel/thinkingcompound phrases and/thinking offalias references — none of which are the/thinkslash command. The parent umbrella #2336 AC uses/think\b(word boundary) to scope precisely to the directive. This PR matches/think\bbehavior and documents the remaining substring matches as out-of-scope.Closes #2466.
Test plan
git grep "/think\b" docs/returns zero hitscd docs && npm run buildsucceeds (296 pages)pnpm format:docs:checkcleanpnpm lint:docsshows no new errors on edited files/tools/slash-commands,/web/tui,/concepts/context,/channels/group-messagessurvive (no#thinkanchors existed)🤖 Generated with Claude Code