Skip to content

gut(docs): remove /think directive references — Area 6 of #2336 (#2466)#2472

Merged
alexey-pelykh merged 1 commit intomainfrom
gut/docs-remove-think-2466
Apr 22, 2026
Merged

gut(docs): remove /think directive references — Area 6 of #2336 (#2466)#2472
alexey-pelykh merged 1 commit intomainfrom
gut/docs-remove-think-2466

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Summary

Changes

  • docs/tools/slash-commands.md — remove /think from the directives list (L17), delete the /think <off|minimal|low|medium|high|xhigh> command entry (L114), and drop /think: high from the colon-separator example (L133).
  • docs/concepts/context.md:139 — remove /think from the directives list.
  • docs/web/tui.md:94 — delete the /think <off|...> TUI entry.
  • docs/channels/group-messages.md — drop or /think high at L18 and /think high at L70 from the session-level-directives examples; /verbose on remains 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 on origin/main.
  • Fresh-context validate + polish subprocesses both returned CLEAN.

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:2354set /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:40 status-line string model + 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 flag model/thinking compound phrases and /thinking off alias references — none of which are the /think slash command. The parent umbrella #2336 AC uses /think\b (word boundary) to scope precisely to the directive. This PR matches /think\b behavior and documents the remaining substring matches as out-of-scope.

Closes #2466.

Test plan

  • git grep "/think\b" docs/ returns zero hits
  • cd docs && npm run build succeeds (296 pages)
  • pnpm format:docs:check clean
  • pnpm lint:docs shows no new errors on edited files
  • Page-level links to /tools/slash-commands, /web/tui, /concepts/context, /channels/group-messages survive (no #think anchors existed)

🤖 Generated with Claude Code

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>
@alexey-pelykh alexey-pelykh enabled auto-merge (squash) April 22, 2026 14:59
@alexey-pelykh alexey-pelykh merged commit 41ba28f into main Apr 22, 2026
15 checks passed
@alexey-pelykh alexey-pelykh deleted the gut/docs-remove-think-2466 branch April 22, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: remove /think references (#2336 Area 6)

1 participant