feat(mothership): add editable skills and tools to the resource panel

[BillLeoutsakosvl346]

Summary

Adds Skills, Custom Tools, and connected MCP servers to Sim Chat's persisted resource panel. Canonical resource persistence now drives attachment validation, while the existing chat lifecycle remains the single owner of resource add, remove, and reorder operations. The panel reuses the existing editors, routes, permissions, and query hooks; embedded drafts are protected across panel transitions, browser navigation, supported workspace navigation, and active-chat deletion.

Fixes #N/A

Type of Change

• Bug fix
• New feature
• Breaking change
• Documentation
• Other: ___________

Testing

• Ran the complete root test suite with bun run test after rebasing onto the latest staging (19 workspace tasks; 2,308 app test files and 32,573 app tests passed).
• Ran bun run lint:check, all-workspace type-check, the relevant React Query, client-boundary, and import audits, the production app build, and git diff --check.
• Verified the resource picker, persisted tabs and deep links, existing full-editor routes, Skill and Custom Tool saves, built-in Skill read-only behavior, MCP discovered tools and editing, Skill/MCP chat-context focus, panel-only composer exclusion, guarded browser Back and command-palette navigation, dirty MCP behavior, first-message URL replacement, and guarded active-chat deletion in the local app. Deleted all disposable browser fixtures afterward.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

Evidence is attached in the first PR comment

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview	Aug 26, 2026 2:49am

[BillLeoutsakosvl346]

Resource panel evidence

Panel picker — Skills, Custom Tools, and MCP Servers

Editable Skill

Built-in Skill remains read-only

Custom Tool editor

MCP server with discovered child tools

Mothership edit with the affected MCP tab focused

[BillLeoutsakosvl346]

@cursor bugbot
@greptile

[cursor]

You need to increase your spend limit or enable usage-based billing to run background agents. Go to Cursor

[greptile-apps]

Greptile Summary

Adds persisted, editable Skill, Custom Tool, and MCP server resources to Sim Chat’s resource panel while centralizing resource mutations and unsaved-draft navigation protection.

• Extends resource discovery, rendering, naming, persistence, extraction, and management-tool handling for the three new resource families.
• Reuses existing editors and permission checks in embedded panel views, with dirty-state reporting and deletion handling.
• Adds a shared transition guard covering tab operations, browser navigation, workspace navigation, and active-chat lifecycle changes.
• Keeps panel-only resources out of prompt attachments while allowing them to persist with chats.

Confidence Score: 5/5

The PR appears safe to merge; no concrete blocking or independently actionable non-blocking defects were identified.

The new resource families are handled consistently across discovery, persistence, rendering, permissions, prompt exclusion, and management extraction, while dirty embedded drafts are protected through centralized transition and lifecycle handling.

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/home/hooks/use-resource-transition-guard.ts	Introduces centralized dirty-resource tracking, history-sentinel handling, navigation interception, and deferred transition replay.
apps/sim/app/workspace/[workspaceId]/home/home.tsx	Moves resource persistence ownership into the chat lifecycle and integrates the shared transition guard with resource and chat state.
apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx	Embeds Skill, Custom Tool, and MCP editors with existing permission checks, dirty reporting, deletion handling, and full-editor actions.
apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-tabs/resource-tabs.tsx	Adds live names for the new resource types and routes tab additions, selections, and closures through centralized ownership and transition protection.
apps/sim/lib/copilot/resources/types.ts	Extends the canonical resource policy so Skills, Custom Tools, and MCP servers are persisted and sanitized consistently.
apps/sim/lib/copilot/resources/extraction.ts	Extends management-tool result extraction to add and remove the new persisted panel resources.
apps/sim/app/workspace/[workspaceId]/skills/[skillId]/skill-detail.tsx	Makes the Skill editor reusable in the panel while preserving drafts across background data updates and reporting dirty state.
apps/sim/app/workspace/[workspaceId]/settings/components/custom-tools/components/custom-tool-detail/custom-tool-detail.tsx	Adds embedded operation, cache-aware draft baselines, host-owned navigation protection, and deletion callbacks to the Custom Tool editor.
apps/sim/app/workspace/[workspaceId]/settings/components/mcp/mcp.tsx	Supports directly opening an MCP server in the embedded settings surface and forwarding modal dirty state to the panel guard.
apps/sim/lib/copilot/chat/post.ts	Uses canonical resource persistence during first-message chat creation so the persisted panel state drives subsequent validation.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  Picker[Resource picker or management tool] --> Canonical[Canonical resource policy and sanitization]
  Canonical --> ChatState[Chat-owned resource state]
  ChatState --> Persistence[Persisted chat resources]
  ChatState --> Panel[Resource panel tabs]
  Panel --> Skill[Embedded Skill editor]
  Panel --> Tool[Embedded Custom Tool editor]
  Panel --> MCP[Embedded MCP server editor]
  Skill --> Dirty[Shared dirty-resource guard]
  Tool --> Dirty
  MCP --> Dirty
  Dirty -->|clean| Transition[Add, remove, reorder, navigate]
  Dirty -->|dirty| Confirm[Discard confirmation]
  Confirm -->|discard| Transition
  Confirm -->|cancel| Panel
  Panel --> ContextMap[Prompt context mapping]
  ContextMap -->|panel-only types| Excluded[No prompt attachment]

Loading

_{Reviews (2): Last reviewed commit: "fix(mothership): use canonical sentinel ..." | Re-trigger Greptile}

[cubic-dev-ai]

1 issue found and verified against the latest diff

Confidence score: 3/5

• In apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/add-resource-dropdown/add-resource-dropdown.tsx, a picker that remains mounted during a workspace switch can serve the previous workspace’s placeholder rows while isHydrating is false, risking stale resource options; track isPlaceholderData for all three queries or suppress placeholder data during the switch.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/add-resource-dropdown/add-resource-dropdown.tsx">

<violation number="1" location="apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/add-resource-dropdown/add-resource-dropdown.tsx:189">
P2: When the picker remains mounted across a workspace switch, these hooks serve the previous workspace's placeholder rows while `isHydrating` stays false. Track `isPlaceholderData` for all three queries or suppress placeholder rows before allowing selection, otherwise the panel can attach a resource from the prior workspace.</violation>
</file>

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

P2: When the picker remains mounted across a workspace switch, these hooks serve the previous workspace's placeholder rows while isHydrating stays false. Track isPlaceholderData for all three queries or suppress placeholder rows before allowing selection, otherwise the panel can attach a resource from the prior workspace.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/add-resource-dropdown/add-resource-dropdown.tsx, line 189:

<comment>When the picker remains mounted across a workspace switch, these hooks serve the previous workspace's placeholder rows while `isHydrating` stays false. Track `isPlaceholderData` for all three queries or suppress placeholder rows before allowing selection, otherwise the panel can attach a resource from the prior workspace.</comment>

<file context>
@@ -180,6 +183,18 @@ export function useAvailableResources(
+  const skillsEnabled = enabled && !excludeTypes?.includes('skill')
+  const customToolsEnabled = enabled && !excludeTypes?.includes('custom_tool')
+  const mcpServersEnabled = enabled && !excludeTypes?.includes('mcp_server')
+  const { data: skills, isPending: skillsPending } = useSkills(workspaceId, {
+    enabled: skillsEnabled,
+  })
</file context>