Skip to content

feat(cli): add optional model field to Workspace schema#26

Open
feloy wants to merge 1 commit intokortex-hub:mainfrom
feloy:model
Open

feat(cli): add optional model field to Workspace schema#26
feloy wants to merge 1 commit intokortex-hub:mainfrom
feloy:model

Conversation

@feloy
Copy link
Copy Markdown
Contributor

@feloy feloy commented Apr 4, 2026

Add an optional model field to the Workspace schema to allow specifying which model is used for the workspace agent.

Closes #25

Add an optional model field to the Workspace schema to allow specifying which model is used for the workspace agent.

Closes kortex-hub#25

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 Apr 4, 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: 287ee933-663f-448c-aa84-f5fbd141b603

📥 Commits

Reviewing files that changed from the base of the PR and between ce0c181 and 23aeb4d.

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

📝 Walkthrough

Walkthrough

The pull request adds a new optional model field to the Kortex CLI workspace schema and documentation. The field is introduced to the Workspace schema in the OpenAPI specification and reflected in example JSON outputs for workspace initialization and listing operations.

Changes

Cohort / File(s) Summary
Documentation
cli/README.md
Updated JSON output examples for workspace operations (init, init --project, workspace list) to include the new model field with sample value "model1".
Schema Definition
cli/openapi.yaml
Added model property (string type) to components/schemas/Workspace and included model: model1 in example payloads for /init/verbose and /list endpoints.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically summarizes the main change: adding an optional model field to the Workspace schema.
Description check ✅ Passed The description is directly related to the changeset, explaining the purpose of adding the model field and referencing the linked issue.
Linked Issues check ✅ Passed The pull request successfully implements the requirement from issue #25 by adding an optional model field to the Workspace schema in both the README and OpenAPI specification.
Out of Scope Changes check ✅ Passed All changes are in scope: README and OpenAPI schema updates directly address the linked issue requirement without introducing unrelated modifications.
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.

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.

(cli) Add a model field to the workspace info

1 participant