tokenmaxx install pi - #23
Merged
Merged
Conversation
|
try this branch: |
DexterStorey
force-pushed
the
pi-install
branch
from
August 18, 2026 23:09
f5f7191 to
428377c
Compare
DexterStorey
force-pushed
the
pi-install
branch
from
August 18, 2026 23:12
428377c to
6bac863
Compare
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.
Towards #11, replacing #14 at a smaller scope: pi only, since that is the harness actually in use.
What
tokenmaxx install pimerges atokenmaxx-anthropicand atokenmaxx-openaiprovider into~/.pi/agent/models.json, pointed at the local proxy. pi re-reads models.json every time/modelopens, so no restart is needed.tokenmaxx uninstall piremoves exactly the two providers we added and nothing else, anddoctorshows a pi line whenever pi is on the machine.An unparseable models.json is never touched: the CLI prints the exact block to add manually instead.
The proxy speaks the ChatGPT backend's dialect
Carried over from #14 unchanged: harnesses send standard Responses API requests — system prompts as messages in the
inputarray,max_output_tokensset — and the ChatGPT codex backend rejects both. On the openai route for OAuth accounts, the proxy now lifts system and developer messages intoinstructionsand dropsmax_output_tokens. Codex's own requests already have that shape, so the adapter is a no-op for them; API-key accounts hit api.openai.com and are left untouched.Cut from #14
openclaw and hermes writers, the settings-tab harness panel, and the settings scroll machinery. The CLI plus doctor covers the pi story on its own.
Tested live
bun run check: 84 tests, typecheck, biome all green.uninstall pi→ providers gone from models.json, neighbours untouched;install pi→ providers back;doctorshowsok pi.pi --provider tokenmaxx-openai --model gpt-5.6-sol -p) into the running daemon: answered correctly and metered under the active codex OAuth account intoken_events.tokenmaxx login claude --api-key) and none is registered on this machine. The provider entry and route mechanics are identical to the claude routing exercised daily.🤖 Generated with Claude Code