Repository navigation
feat: register Claude models from OMP catalog - #7
LaurensBosscher wants to merge 2 commits into
Conversation
Deep @oh-my-pi/* imports fail under OMP 18.2.5 extension loading because host onLoad only rewrites legacy scopes onto in-process shims. Switch to @earendil-works root imports (proven on this host) and register claude-opus-5 so existing modelRoles/fallbackChains keep resolving.
…, Fable 5.1, Mythos) + max effort Derive the picker from the host's anthropic catalog instead of a hardcoded id list, keeping the 1M/200K variant entries. Measured [1m] entitlements stay in tables; unmeasured 1M-capable models default to 200K with a -1m alternate. Map thinking levels onto each model's catalog effort ladder (adds max). Fix unsuffixed entries throwing when their preferred window has no runtime.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bdd567a2df
ℹ️ 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".
| for (let i = rank; i >= 0; i--) { | ||
| if (efforts.includes(CLAUDE_CODE_EFFORTS[i])) return CLAUDE_CODE_EFFORTS[i]; |
There was a problem hiding this comment.
When AskClaude uses the newly allowed thinking: "max" with claude-sonnet-4-6, the injected map only contains xhigh: "max", so a direct max request falls through to this ladder scan. With the Sonnet 4.6 catalog shape added in this commit (low/medium/high), this returns high, even though requesting xhigh on the same model returns Claude Code's supported max; selecting the nominally stronger setting therefore silently reduces effort. Add a direct max mapping for these known max-backed models rather than clamping it through the advertised ladder.
Useful? React with 👍 / 👎.
Changes
Stacked after #6: this branch includes its commit until #6 merges.
Verification