claude-code+codex+opencode+gemini-cli: unify context and skills options#9068
claude-code+codex+opencode+gemini-cli: unify context and skills options#9068khaneliman merged 5 commits intonix-community:masterfrom
Conversation
|
I had an alternative method in a local branch that I haven't gotten back to. Might take a look again soon |
Should I close this PR then? |
97ac646 to
18ce61c
Compare
18ce61c to
8cdc3c3
Compare
8cdc3c3 to
b48ddf8
Compare
|
Added support for |
Sorry, been super busy and forgot to respond. I actually think this is a fine first step for what I was planning around these modules that all share similar concepts. I don't think it conflicts with the general concept so we can get this in first to improve things from a UX perspective. |
Description
these assistant modules exposed the same concepts through different option names and types. global instructions lived under
memory,custom-instructions, orrules, and claude code split skill configuration betweenskillsandskillsDir. that made shared configurations harder to move between assistants and left claude code less flexible than the others.this patch adds a canonical
contextoption for the shared AGENTS-style file and keep the old option names as compatibility shims with migration warnings.also let
programs.claude-code.skillsaccept either an attribute set or a directory path so it matches the bulk-directory workflow already supported elsewhere.it also updates the module tests and add a news entry so the migration path is explicit.
Checklist
Change is backwards compatible.
Code formatted with
nix fmtornix-shell -p treefmt nixfmt deadnix keep-sorted nixf-diagnose --run treefmt.Code tested through
nix run .#tests -- test-allornix-shell --pure tests -A run.all.Test cases updated/added. See example.
Commit messages are formatted like
See CONTRIBUTING for more information and recent commit messages for examples.
If this PR adds a new module
If this PR adds an exciting new feature or contains a breaking change.