-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
GitHub Copilot: claude-opus-4.8 hard-coded to 128K context (native 1M) + missing thinking support #91869
Copy link
Copy link
Closed as not planned
Closed as not planned
Copy link
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:auth-providerAuth, provider routing, model choice, or SecretRef resolution may break.Auth, provider routing, model choice, or SecretRef resolution may break.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-live-reproClawSweeper needs live local, crabbox, or manual validation to confirm this issue.ClawSweeper needs live local, crabbox, or manual validation to confirm this issue.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:auth-providerAuth, provider routing, model choice, or SecretRef resolution may break.Auth, provider routing, model choice, or SecretRef resolution may break.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.issue-rating: 🐚 platinum hermitGood issue quality with a plausible reproduction path needing some confirmation.Good issue quality with a plausible reproduction path needing some confirmation.
Type
Fields
Priority
None yet
Summary
The GitHub Copilot provider hard-codes
claude-opus-4.8withcontextWindow: 128000(and noreasoningflag). Copilot's live/modelsendpoint returns 1,000,000 for this model with full reasoning-effort support. The static catalog value shadows the real capability, so users on entitled accounts silently get a 128K context budget and no thinking, even though the model and the provider both support 1M + adaptive thinking.Root cause: PR #88547 added
claude-opus-4.8by mirroring the 4.7 metadata shape, which carried overcontextWindow: 128000. Unlikeclaude-opus-4.7-high/-xhigh(which really are 200K split variants), Opus 4.8 is natively 1M on the Anthropic API / Bedrock / Vertex — no-1msuffix, no beta header, no long-context premium (per Anthropic's 4.8 release notes).Reproduction
claude-opus-4.8via GitHub Copilot.GET https://api.githubcopilot.com/modelsreturns (abridged) forclaude-opus-4.8:openclaw models listshowsgithub-copilot/claude-opus-4.8with 128k context./status→Context: x/128k), triggering premature compaction on long contexts.For comparison, the same
/modelsresponse givesclaude-opus-4.7(no suffix) = 1,000,000 too, whileclaude-opus-4.7-high/claude-opus-4.7-xhigh= 200,000. So the 4.8 row is simply wrong, not a deliberate cap.Two distinct places need the fix
The catalog value feeds two different consumers, and they resolve from different sources:
extensions/github-copilot/openclaw.plugin.json→modelCatalog.providers.github-copilot.models[id=claude-opus-4.8]— this is what seeds the session context budget (/statusContext line). CurrentlycontextWindow: 128000.extensions/github-copilot/models-defaults.ts/ static metadata — feeds the picker andmodels list.A user-config override of
models.providers.github-copilot.models[]fixesmodels listand routing, but the session budget still reads the bundled manifest, so editing config alone is not enough — the manifest itself has to be correct (or the budget resolver should prefer the live/override value).Proposed fix
claude-opus-4.8contextWindow: 1000000,maxTokens: 64000,reasoning: true,input: ["text","image"]in the bundled manifest + defaults.supportedReasoningEfforts: ["low","medium","high","xhigh","max"](matching upstream/models)./modelsmetadata (or a user override) over the static manifest, so newly-entitled / re-spec'd models don't require a release bump. (This overlaps with GitHub Copilot: static default model list shadows live entitlement discovery #88548.)Environment
api.githubcopilot.com/modelsreturnsclaude-opus-4.8ctx=1,000,000 + reasoning_effort [low,medium,high,xhigh,max] for the affected account.Interim workaround (confirmed working)
models.providers.github-copilot.models[]withcontextWindow:1000000+reasoning:true+compat.supportedReasoningEffortsfixesmodels list+ routing + thinking.openclaw.plugin.jsonmanifest's 4.8contextWindowto1000000and restart. After that,/statusshowsContext: …/1.0m. (This is overwritten on upgrade.)