feat(cmd): add info command to list supported agents and runtimes#121
feat(cmd): add info command to list supported agents and runtimes#121bmahabirbu wants to merge 1 commit intokortex-hub:mainfrom
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 3 minutes and 27 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Add `kortex-cli info` command that shows CLI version, available agents, and available runtimes. Agents are dynamically discovered from config files on disk (*.json in the podman config directory, excluding image.json). Supports `--output json` for machine-readable output. Also adds `ListAgents()` method to the podman config interface to enable agent discovery by scanning the config directory. Closes kortex-hub#35 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Brian <bmahabir@bu.edu>
bmahabirbu
force-pushed
the
feat/info-command
branch
from
b7ca6a6 to
4e3404a
Compare
| configDir := filepath.Join(absStorageDir, "runtimes", "podman", "config") | ||
| cfg, err := config.NewConfig(configDir) | ||
| if err != nil { | ||
| return outputErrorIfJSON(cmd, i.output, fmt.Errorf("failed to create config: %w", err)) | ||
| } |
There was a problem hiding this comment.
This must be generic, the command cannot know about specific runtimes, only the instance.Manager knows about them, in a generic way (though interface/implementation). You'll have to add a ListAgents method to the Runtime interfaec, and the manager can collect all the agents from all runtimes. Finally, the Manager will expose a ListAgents() method so the command can get them.
I suppose that we will differentiate agents per runtime, as one agent can be configured for a specific runtime, but not others
3 participants
Summary
kortex-cli infocommand that displays CLI version, available agents, and available runtimes*.jsonin the podman config directory, excludingimage.json)--output jsonfor machine-readable outputListAgents()method to the podmanConfiginterfaceText output:
JSON output:
Closes #35
Test plan
make test)kortex-cli infoshows text outputkortex-cli info -o jsonshows JSON outputkortex-cli info --helpshows help with examples🤖 Generated with Claude Code