Skip to content

feat: add agent to cli Workspace#20

Merged
feloy merged 1 commit intoopenkaiden:mainfrom
feloy:agent
Mar 26, 2026
Merged

feat: add agent to cli Workspace#20
feloy merged 1 commit intoopenkaiden:mainfrom
feloy:agent

Conversation

@feloy
Copy link
Copy Markdown
Contributor

@feloy feloy commented Mar 26, 2026

Fixes #19

Signed-off-by: Philippe Martin <phmartin@redhat.com>

Co-Authored-By: Claude Code (Claude Sonnet 4.5) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b8d9bfae-65ff-48d1-ad7b-48de69993178

📥 Commits

Reviewing files that changed from the base of the PR and between 1d01419 and 1bc9cd1.

📒 Files selected for processing (2)
  • cli/README.md
  • cli/openapi.yaml

📝 Walkthrough

Walkthrough

The PR updates the CLI README and OpenAPI specification to include a new agent field in workspace information. The README examples now show the --agent claude flag in init command invocations and updated output, while the OpenAPI Workspace schema adds the agent property as a required string field with corresponding example payloads.

Changes

Cohort / File(s) Summary
Documentation Updates
cli/README.md
Updated init command examples to include --agent claude flag and added "agent": "claude" field to example JSON outputs for verbose initialization and workspace listing scenarios.
OpenAPI Schema
cli/openapi.yaml
Extended the Workspace schema by adding a required agent string property. Updated example payloads for /init/verbose and /list endpoints to include agent: claude values.

Possibly related PRs

  • kortex-hub/kortex-cli-api#10 — Modifies the same Workspace schema in cli/openapi.yaml, introducing the agent property structure that this PR builds upon.
  • kortex-hub/kortex-cli-api#17 — Updates the Workspace schema with another required top-level property, demonstrating concurrent schema expansion work.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding an agent field to the CLI Workspace.
Description check ✅ Passed The description references issue #19 which is related to adding agent information to Workspace, matching the changeset scope.
Linked Issues check ✅ Passed The PR successfully adds the Agent field to Workspace and updates both init and list command outputs to include agent information as required by issue #19.
Out of Scope Changes check ✅ Passed All changes are directly related to adding the agent field to Workspace; no out-of-scope modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@feloy feloy merged commit 6222a0f into openkaiden:main Mar 26, 2026
2 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.

Add Agent to the Workspace information

2 participants