Add Pi runtime context and fast model settings - #21
Conversation
- Expose context-window controls when supported by the active Pi profile - Synchronize reasoning, context, and service tier before prompts - Enable Fast service for supported GPT-5.6 models
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 53734dd4ab
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- Fall back to high when catalog defaults are unavailable - Apply the same default to discovered Pi Codex models
- Apply the high reasoning default to every supported Pi provider - Expand coverage for Codex, Claude SDK, and OpenCode models - Remove the obsolete root README
|
@macroscope-app review |
|
Manual reviews triggered for commit All prior checks · these links stay valid even if you push more commits. |
|
Review triggered! The Approvability Check and Correctness Check are now running. Results will be posted when they complete. |
ApprovabilityVerdict: Needs human review This PR adds new runtime context window and fast model settings that change Pi provider behavior. An unresolved review comment identifies a profile-mismatch issue where advertised options may cause errors with certain profile selections. You can customize Macroscope's approvability policy. Learn more. |
Summary
/contextcommand support.Testing
Note
Add Pi runtime context window selection and fast model settings
/contextcommand support to the Pi adapter, allowing context-window size to be synchronized before each prompt viasyncContextWindowin PiAdapter.ts.contextcommand.gpt-5.6-*variants and addsparsePiContextWindowto validate/normalize context values before sending them to Pi.highwhen supported, both in Pi model capabilities and in Codex model capabilities whendefaultReasoningEffortis unavailable./contextand/fastselections now emit runtime warnings and are silently ignored instead of throwing errors.Macroscope summarized 86fac02.