Add global synthesis and terminal settings - #18
Merged
Merged
Conversation
milanshen
force-pushed
the
milan/settings
branch
from
August 4, 2026 22:26
f2e7448 to
4414744
Compare
luhe19001
requested changes
Aug 4, 2026
luhe19001
left a comment
Collaborator
There was a problem hiding this comment.
Requested change
First-run can save synthesis On with an unavailable backend still selected.
The consent draft defaults to enabled: true and claude-cli. Unavailable backends are not clickable, but the current selection is not cleared and Save is not blocked — so a Codex-only machine can accept the dialog as-is and persist a config that fails every synthesis run.
Please block that path, e.g. auto-select the first available backend, disable Save when synthesis is on and the selected backend is unavailable, or default draft enabled to false when no backend is available.
milanshen
force-pushed
the
milan/settings
branch
from
August 5, 2026 16:40
65eb091 to
285f789
Compare
luhe19001
approved these changes
Aug 5, 2026
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
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.
Context
coSlash currently hard-codes its synthesis backend, model, and macOS terminal. This adds secure machine-wide settings with explicit synthesis consent, configurable runtime choices, and fail-closed handling for invalid configuration.
Changes
~/.coslash/settings.jsonschema and restricted atomic persistence.Test
fresh start or delete
~/.coslash/settings.jsonScreenshots