pi 0.85.1 added a built-in /thinking command — extension's registerThinkingCommand now conflicts (startup warning)
What happens
On pi 0.85.1, every session start prints:
[Extension issues]
npm:@companion-ai/feynman (user) extensions/research-tools.ts
Extension command '/thinking' conflicts with built-in interactive command. Skipping in autocomplete.Cause
pi 0.85.x added a built-in interactive /thinking command (view/set thinking level). Feynman 0.3.47 still registers its own /thinking via extensions/research-tools/thinking.ts → registerThinkingCommand(pi) (originally added per #196). pi resolves the collision by skipping the extension command in autocomplete, and prints the warning banner on each session start.
Impact
Cosmetic — both commands do the same thing, pi's built-in wins, no functionality is lost. But the startup warning shows up for every user on pi ≥ 0.85 running feynman 0.3.47.
Suggested fix
Drop the extension's /thinking registration in favor of the built-in one once the minimum supported pi version includes the built-in command (pi ≥ 0.85.1). That means:
- remove the
registerThinkingCommandimport and call inextensions/research-tools.ts - delete
extensions/research-tools/thinking.ts - remove
tests/thinking-command.test.ts
PR to follow.
Environment
@companion-ai/feynman0.3.47 (installed as a pi package)- pi 0.85.1, Node v26.8.1, Linux (WSL2)
Source: companion-inc/feynman