Skip to content

fix(workspace): trim model-facing capability output - #148

Closed
Waishnav wants to merge 5 commits into
codex/dw-onboardingfrom
codex/dw-output-polish
Closed

fix(workspace): trim model-facing capability output#148
Waishnav wants to merge 5 commits into
codex/dw-onboardingfrom
codex/dw-output-polish

Conversation

@Waishnav

@Waishnav Waishnav commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Workspace responses still exposed more agent/provider detail than models need, and CLI orchestration launched through MCP could lose the opened workspace scope.

This layer keeps capability summaries compact, removes unavailable providers and profiles from model-facing target discovery, and propagates the opened workspace identity into CLI orchestration. It also aligns the bundled documentation and skills with the CLI-first workflow model.

Depends on #144; it is the final layer of the local stack.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e2bf29bc-3e77-46da-8691-0f02e3fe4863

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@greptile-apps

greptile-apps Bot commented Aug 8, 2026

Copy link
Copy Markdown

Greptile Summary

The PR narrows model-facing workspace capability data while retaining compact provider and profile execution hints. It also propagates the opened workspace identity into CLI orchestration launched through the MCP shell.

  • Replaces provider-name lists with filtered structured capability summaries.
  • Adds provider and optional model/effort defaults to visible agent profiles.
  • Carries workspace root and ID through shell-spawned DevSpace CLI commands.
  • Updates dashboard rendering, schemas, tests, documentation, and bundled skills for the revised contract.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete blocking or independently actionable non-blocking issue identified.

The changed capability objects align with the catalog’s filtered data shapes, and shell working directories and injected identity values remain consistently scoped to the validated opened workspace.

Important Files Changed

Filename Overview
src/server.ts Updates the open-workspace response schema and formatting, permits CLI orchestration through the shell tool, and passes the validated workspace identity into shell execution.
src/pi-tools.ts Injects the opened workspace ID and root into spawned shell commands so nested CLI orchestration retains workspace scope.
src/local-agent-resolution.ts Restricts profile suggestions in resolution errors to profiles backed by currently available providers.
src/local-agent-targets.ts Filters formatted profile targets against the supplied provider availability list.
src/ui/workspace-dashboard.ts Renders structured provider capabilities and profile provider/model/effort metadata.
src/ui/card-types.ts Aligns dashboard card types with the revised structured capability response.
src/open-workspace-capabilities.test.ts Updates schema assertions for structured provider capability and agent-profile output.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[open_workspace] --> B[Filtered provider and profile catalog]
  B --> C[Structured model-facing capabilities]
  B --> D[Workspace dashboard]
  A --> E[workspaceId and workspace root]
  E --> F[MCP shell tool]
  F --> G[Spawn environment]
  G --> H[devspace agents and workflow CLI]
  H --> I[Orchestration scoped to opened workspace]
Loading

Reviews (1): Last reviewed commit: "fix(agents): hide unavailable profiles f..." | Re-trigger Greptile

@Waishnav Waishnav closed this Aug 8, 2026
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.

1 participant