Skip to content

claude-code+codex+opencode+gemini-cli: unify context and skills options#9068

Merged
khaneliman merged 5 commits intonix-community:masterfrom
iniw:ai-agents-unification
Apr 12, 2026
Merged

claude-code+codex+opencode+gemini-cli: unify context and skills options#9068
khaneliman merged 5 commits intonix-community:masterfrom
iniw:ai-agents-unification

Conversation

@iniw
Copy link
Copy Markdown
Contributor

@iniw iniw commented Apr 8, 2026

Description

these assistant modules exposed the same concepts through different option names and types. global instructions lived under memory, custom-instructions, or rules, and claude code split skill configuration between skills and skillsDir. that made shared configurations harder to move between assistants and left claude code less flexible than the others.

this patch adds a canonical context option for the shared AGENTS-style file and keep the old option names as compatibility shims with migration warnings.
also let programs.claude-code.skills accept 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 fmt or
    nix-shell -p treefmt nixfmt deadnix keep-sorted nixf-diagnose --run treefmt.

  • Code tested through nix run .#tests -- test-all or
    nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.
    • Generate a news entry. See News
    • Basic tests added. See Tests
  • If this PR adds an exciting new feature or contains a breaking change.

    • Generate a news entry. See News

@github-actions github-actions bot added the ai label Apr 8, 2026
@home-manager-ci home-manager-ci bot requested review from delafthi and khaneliman April 8, 2026 14:52
@khaneliman
Copy link
Copy Markdown
Collaborator

I had an alternative method in a local branch that I haven't gotten back to. Might take a look again soon

@iniw
Copy link
Copy Markdown
Contributor Author

iniw commented Apr 8, 2026

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?

@iniw iniw force-pushed the ai-agents-unification branch from 97ac646 to 18ce61c Compare April 8, 2026 20:36
@home-manager-ci home-manager-ci bot requested a review from rrvsh April 8, 2026 20:37
@iniw iniw force-pushed the ai-agents-unification branch from 18ce61c to 8cdc3c3 Compare April 8, 2026 20:38
delafthi

This comment was marked as low quality.

@iniw iniw force-pushed the ai-agents-unification branch from 8cdc3c3 to b48ddf8 Compare April 10, 2026 00:33
@iniw
Copy link
Copy Markdown
Contributor Author

iniw commented Apr 10, 2026

Added support for gemini-cli.skills to be set to a directory, an addition to #9009 now that it's merged

@iniw iniw changed the title claude-code+codex+opencode: unify context and skills options claude-code+codex+opencode+gemini-cli: unify context and skills options Apr 10, 2026
@khaneliman
Copy link
Copy Markdown
Collaborator

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?

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.

@khaneliman khaneliman merged commit 30ca31c into nix-community:master Apr 12, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants