Skip to content

Add integration for various AI CLI commands#20

Merged
afewyards merged 8 commits into
afewyards:mainfrom
anthraxx:feat/ai-cli
Mar 28, 2026
Merged

Add integration for various AI CLI commands#20
afewyards merged 8 commits into
afewyards:mainfrom
anthraxx:feat/ai-cli

Conversation

@anthraxx

Copy link
Copy Markdown
Contributor

Generalize the cli command invocation into a reusable utils module and integrate various CLI commands:

  • Codex
  • Copilot
  • Gemini
  • OpenCode
  • Qwen

@anthraxx anthraxx force-pushed the feat/ai-cli branch 2 times, most recently from d12c44a to dd46145 Compare March 24, 2026 22:09

@afewyards afewyards left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work on extracting it to a utils. couple comments around it that need fixing. also maybe a test file to cover it as it's trivial to this plugin.

Comment thread doc/codereview.txt Outdated
Comment thread lua/codereview/ai/providers/codex_cli.lua Outdated
Comment thread lua/codereview/ai/providers/gemini_cli.lua Outdated
Comment thread lua/codereview/ai/providers/opencode_cli.lua Outdated
Comment thread lua/codereview/ai/providers/opencode_cli.lua Outdated
Comment thread lua/codereview/ai/providers/utils.lua Outdated
Comment thread lua/codereview/ai/providers/utils.lua Outdated
Comment thread lua/codereview/ai/providers/utils.lua
Comment thread lua/codereview/ai/providers/utils.lua Outdated
Comment thread lua/codereview/config.lua Outdated
This preparation unifies the claud_cli and custom_cmd invocation
into a general purpose utils runner that can be reused. This will
allow future cli integrations to be done simpler without code
duplication.
@anthraxx

Copy link
Copy Markdown
Contributor Author

Pushed and rbeased adjustments, feel free to give it another spin

@afewyards afewyards merged commit 722542e into afewyards:main Mar 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants