Skip to content

Add /thinking REPL command - #200

Closed
Konstantin Baltsat (Baltsat) wants to merge 1 commit into
companion-inc:mainfrom
Baltsat:codex/feynman-196-thinking-command
Closed

Add /thinking REPL command#200
Konstantin Baltsat (Baltsat) wants to merge 1 commit into
companion-inc:mainfrom
Baltsat:codex/feynman-196-thinking-command

Conversation

@Baltsat

Copy link
Copy Markdown

Summary

  • add /thinking as a discoverable REPL command for viewing and changing the current thinking level
  • open a picker with the current level marked when no argument is provided
  • accept off, minimal, low, medium, high, or xhigh directly
  • preserve the existing Shift+Tab cycle behavior

Fixes #196

Validation

  • node --import tsx --test tests/thinking-command.test.ts tests/help-command.test.ts — 6 passed
  • npm test — 588 passed
  • npm run typecheck
  • npm run build
  • cd website && npm run build — 34 pages built

AI-assisted contribution

This change was prepared with AI assistance. I reviewed the final diff, verified it against the installed Pi extension API, and ran the checks listed above.

Expose the current thinking level through a discoverable picker and direct level arguments while preserving the existing Shift+Tab cycle.

Assisted-by: OpenAI Codex

User-Request: build reviewed OSS contributions | codex:019f8d3f-8848-72b1-9e8c-c2c71648e981
Signed-off-by: Konstantin Baltsat <baltsat2002@mail.ru>
Copilot AI review requested due to automatic review settings July 26, 2026 06:24
@vercel

vercel Bot commented Jul 26, 2026

Copy link
Copy Markdown

Konstantin Baltsat (@Baltsat) is attempting to deploy a commit to the Companion Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@advaitpaliwal

Copy link
Copy Markdown
Contributor

Thanks for the focused contribution. I am closing this as superseded by the independently prepared implementation in #192. That version uses Pi 0.82.1's model-supported levels, includes max, supports direct arguments and completions, reports status in non-UI contexts, and surfaces clamp behavior; no source code from this PR was ported. Issue #196 remains open until the replacement is merged and released.

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.

FEATURE REQUEST: Add a /thinking (/effort) slash command to change reasoning effort in the TUI

3 participants