Repository navigation
Start new workspaces on the last model chosen in a footer - #8
Closed
HugoHeneault wants to merge 1 commit into
Closed
HugoHeneault wants to merge 1 commit into
HugoHeneault wants to merge 1 commit into
Conversation
The last model, effort and backend picked in the composer or create window feed the next workspace. A default preset and a repository models.default still outrank it, and Settings sits below. A Codex model missing from the account's fetched model list is never used as a default, so the next layer answers instead.
Author
|
Opened against spatie/bloom instead. |
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
1 participant
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.
Fixes spatie#12. A new workspace opened on the Settings default whatever had been picked in the footer a minute before.
LastModelChoice) and feed the next workspace.models.default, then a default preset, then the last choice, then Settings, then the machine-wide file, then the built-in. Permission mode and output style are not carried over.model/listis never used as a default. A last choice in that state is skipped, and a pinned one falls back to the built-in. This only applies once the list has arrived; an empty list disqualifies nothing, and Claude Code ids are not checked because ids such asopus-5-1mare on no list.I could not establish where
gpt-5.5came from on the reporting machine without reading the real database, so this removes the unusable default rather than explaining it.Tests:
LastModelChoiceTestscovers the round trip, the precedence and the availability rule. Not run locally; the test target compiles,swift build -Xswiftc -warnings-as-errors,make lintandmake swiftlintare green, and I checked the behaviour by hand in Bloom Dev.