Uh oh!
There was an error while loading. Please reload this page.
feat(desktop): show the model's reported context window beside the declared setting - #4575
Merged
Merged
Conversation
6 tasks
Joob1nforce-pushed
the
feat/context-window-setting-hint
branch
12 times, most recently
from
September 3, 2026 08:58
7f865d3 to
2b2aac6Compare6 tasks
…clared setting The context-window setting is the Maka window: the compaction target the user declares. When nothing is declared and the provider's /models report carries a window for the model, show that number beside the empty field with a one-click "Use it" that fills the declaration. The reported value never takes effect on its own (apache#4559); the help text now says what the setting does and what leaving it empty means. Refs apache#4559 Generated-by: Claude Code Claude-Session: https://claude.ai/code/session_014ajaRxC4jydavY9nYUFj5J
Joob1nforce-pushed
the
feat/context-window-setting-hint
branch
from
September 3, 2026 13:29
2b2aac6 to
d06758cCompareJoob1n
marked this pull request as ready for review
September 3, 2026 13:29
M4n5ter
approved these changes
Sep 3, 2026
Uh oh!
There was an error while loading. Please reload this page.
6 tasks
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 freeto 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.
Summary
Follows #4653, which is merged; this branch is a single commit on
main.After #4653 the context-window setting is the Maka window: the compaction target the user declares, and the only proactive threshold. A provider's
/modelsreport no longer takes effect on its own. This PR makes that visible where the user sets it: when nothing is declared and the connection's model catalog reports a window, the setting shows "This model declares a N-token window" with a one-click Use it that fills the declaration. The reported value never applies by itself. The help text now says what the setting does ("Maka compacts once the previous request's real usage exceeds it") and what leaving it empty means ("never compact proactively; the provider decides").Refs #4559
Verification
npm --workspace @maka/desktop run typecheck,npm run check:renderer-architecture -- --base origin/main(no debt increase inprovider-connection-detail.tsx),npm run astryx:surface-inventory,npm run lint,npm run format:check— clean. Not run locally: Desktop e2e — relying on CI.Self-review
connection.models[].contextWindow(the Host's catalog entry) rather than models.dev directly, so it stays inside the dependency debt the ratchet allows for this legacy file.setDraftContextWindowpath as typing the value, so the declaration is persisted and validated exactly as a manual entry would be.AI use
Select exactly one:
Tool(s) and scope: Claude Code — implementation; reviewed and verified by the author.
Checklist
Does this PR entail a change in behavior?
https://claude.ai/code/session_014ajaRxC4jydavY9nYUFj5J