Merged
Conversation
Use ansible_facts.env.HOME instead of ansible_env.HOME Reorder and normalize dock_remove defaults
- changed edit prediction provider to follow new settngs standard - added subpixel text rendering mode as a default - added YAML settings (mainly for disabling typing formatting) - added ansible plugin
Migrate Zed AI settings from copilot_chat to OpenRouter-backed Anthropic models and add Codestral edit predictions. Update default, inline assistant, commit-message and thread-summary models to Anthropic 4.6 variants (enable thinking for Opus/Sonnet where appropriate) and add a favorite_models list for quick selection. Introduce an edit_predictions block using Codestral (codestral-latest, eager mode). Also adjust editor UI defaults: enable tab file icons, close-position on left, show git status in tabs, and activate left neighbour on close. Enable minimap current-line highlight for all and set tabs/close behavior. Reorder and re-enable extend_comment_on_newline and tweak gutter/minimap settings to improve readability and workflow. These changes centralize model configuration to OpenRouter, enable code completion predictions, and make small UI ergonomics improvements.
- zsh/starship.toml: prevent noisy kubectl stderr by redirecting stderr to /dev/null for current-context and namespace commands. This avoids showing error messages in the prompt when kubectl is not configured or when contexts/namespaces are unavailable. - zed/settings.json: change AI edit_predictions provider from 'codestral' to 'zed', set mode to 'eager', add provider-specific api_url for the previous codestral configuration, and keep the model field. This reorganizes provider settings and enables eager predictions with the intended 'zed' provider.
This ensures privacy_mode is set, while preserving codestral settings for future use.
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the macOS development environment setup (Ansible + dotfiles) to refine Dock defaults, reduce noisy shell prompt output, and modernize Zed editor UI/AI configuration.
Changes:
- Switch macOS defaults “done” marker path to use
ansible_facts.env.HOME. - Silence
kubectlstderr noise in Starship custom prompt modules. - Refresh Zed settings: AI provider/model configuration, edit predictions, UI/ergonomic defaults, and enable YAML/Ansible-related settings.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
tasks/defaults.yml |
Uses ansible_facts.env.HOME for .osx.sh.done marker file path. |
dotfiles/zsh/starship.toml |
Adjusts Starship kubectl prompt commands to reduce stderr noise. |
dotfiles/zed/settings.json |
Updates Zed AI provider/model settings and UI/editor defaults; adds YAML/Ansible configuration. |
defaults.yml |
Reorders/normalizes Dock removal defaults. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [custom.kubens] | ||
| style = 'bold red' | ||
| command = 'kubectl config view --minify -o "jsonpath={..namespace}"' | ||
| command = 'kubectl config view --minify -o "jsonpath={..namespace} 2>/dev/null"' |
There was a problem hiding this comment.
The stderr redirection is currently inside the jsonpath string (after jsonpath={..namespace}), so it will be passed to kubectl as part of the -o argument and won’t actually silence stderr. Move 2>/dev/null outside the quoted -o "jsonpath=..." portion so the shell performs the redirection.
Suggested change
| command = 'kubectl config view --minify -o "jsonpath={..namespace} 2>/dev/null"' | |
| command = 'kubectl config view --minify -o "jsonpath={..namespace}" 2>/dev/null' |
| [custom.kubectx] | ||
| style = 'bold blue' | ||
| command = "echo \ue81d ${$(kubectl config current-context)##*/}" | ||
| command = "echo \ue81d ${$(kubectl config current-context 2>/dev/null)##*/} " |
There was a problem hiding this comment.
This command adds a trailing space to the module output (...##*/} ). That space will be included in $output and can create inconsistent spacing around the (:${custom.kubens}) segment in the prompt. Consider removing the trailing space (or using printf if you need precise formatting).
Suggested change
| command = "echo \ue81d ${$(kubectl config current-context 2>/dev/null)##*/} " | |
| command = "echo \ue81d ${$(kubectl config current-context 2>/dev/null)##*/}" |
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.
Summary
Changes