Uh oh!
There was an error while loading. Please reload this page.
feat(copilot): add the platform subagent - #6395
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Adds Also extracts shared Reviewed by Cursor Bugbot for commit 3adc397. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryRegisters the client-side half of the new context-isolated Platform subagent, intended to be deployed with its paired Mothership implementation.
Confidence Score: 5/5The client-side changes appear safe to merge as part of the explicitly paired Platform-agent deployment. The generated catalog and schema agree on the new subagent contract, routing is derived from the catalog, and the relevant workspace display paths all recognize the platform identifier.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/copilot/generated/tool-catalog-v1.ts | Adds the Platform subagent’s catalog entry, unions, and registry mapping consistently with existing subagent definitions. |
| apps/sim/lib/copilot/generated/tool-schemas-v1.ts | Adds the matching runtime schema requiring a self-contained string task. |
| apps/sim/lib/copilot/tools/tool-display.ts | Adds the human-readable Platform Agent title to the central tool-title map. |
| apps/sim/app/workspace/[workspaceId]/home/types.ts | Adds the Platform Agent label used by workspace subagent rendering and dispatch recognition. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/utils.ts | Assigns the Library icon to Platform agent groups in message content. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
Copilot[Copilot orchestrator] -->|platform task| Catalog[Generated tool catalog]
Catalog -->|route: subagent| Mothership[Mothership]
Mothership --> Platform[Platform agent]
Platform -->|subagent events| Workspace[Workspace message UI]
Workspace --> Label[Platform Agent label]
Workspace --> Icon[Library icon]
Reviews (1): Last reviewed commit: "feat(copilot): describe platform agent d..." | Re-trigger Greptile
Adds the no-argument get_account_billing tool: the handler combines the org-aware billing lookups (usage data, credit balance, usage-limit info) into one snapshot — plan, current-period usage vs limit with remaining, and purchased credit balance — always scoped to the requesting user. Catalog and schema mirrors regenerated from the mothership definitions. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
dac3284 to
e4e2abcCompareThere was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 3adc397. Configure here.
| governingPermissionGroup: accessControl.permissionGroup, | ||
| effectiveConfig: accessControl.config, | ||
| activeRestrictions: getActivePermissionGroupRestrictions(accessControl.config), | ||
| }, |
There was a problem hiding this comment.
Inactive AC exposes env restrictions
Medium Severity
get_enterprise_context always derives effectiveConfig and activeRestrictions from accessControl.config, even when entitled is false. Inactive contexts still run mergeEnvAllowlist(null), so a self-hosted ALLOWED_INTEGRATIONS value surfaces as Access Control restrictions with no governing group. The platform agent can then describe deployment env limits as permission-group policy.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 3adc397. Configure here.


What
Registers the new platform subagent client-side — the sim half of a paired change (mothership: simstudioai/mothership#417, to be merged together). The platform agent is a context-isolated Sim-product specialist that takes over the docs beat (
search_docs+ thedocs/corpus) from the search agent.platformtrigger tool and the slimmed search-agent surface.tool-display.tsand the home message-content types, so a platform delegation renders with a proper label instead of a raw tool ID.Stacked PR
Stacked on #6389 (base =
feat/docs-vfs-search-docs) — this diff shows only the platform-agent delta. The base auto-retargets to staging when #6389 merges; merge in order.Deploy note
Same pairing rule as #6389: merge this and simstudioai/mothership#417 together and deploy normally.
Testing
Type-check clean; 669 copilot tool tests pass. Validated end-to-end at this stack level: Sim product questions route to the platform agent, which answers in ~1
search_docsround with named markdown-link citations (verified via traces).🤖 Generated with Claude Code