chore: stop managing Codex config#19
Merged
Merged
Conversation
## What Removed the Codex config directory from dotfiles, stopped the setup script from linking ~/.codex, and removed the Codex Doctor pre-commit hook. ## Why Codex local state and config should stay outside this dotfiles repository while shared agent instructions can remain linked separately. ## Notes - Existing Codex state now needs to be managed outside this repository after pulling this change. - pre-commit now only runs ShellCheck; Codex health checks are no longer tied to dotfiles commits. Signed-off-by: jmeridth <jmeridth@gmail.com>
Owner
Author
|
I DO NOT like that the trusted projects pieces in ~/.codex/config.toml have to be absolute path. Not ready to have 2 diff configs between work and personal laptops. And not comfortable source controlling my local directory repo names. Should be able to mark a parent directory trusted like: [projects."~/code"]
trust_level = "trusted" |
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.
What
Removed Codex config from dotfiles by deleting the tracked Codex config entries, stopping setup from linking ~/.codex, and removing the Codex Doctor pre-commit hook.
Why
Codex local state and config should live outside this dotfiles repository while shared agent instructions can remain linked separately.
Notes
Testing
Validated that the edited setup script parses with Bash. Manually verified ~/.codex is a real directory and ~/.codex/AGENTS.md points to configs/AGENTS.md.