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

Check if this issue is valid — if so, understand the root cause and fix it. At apps/sim/app/workspace/[workspaceId]/home/hooks/use-resource-transition-guard.ts, line 57:

<comment>When first-message routing replaces the sentinel while a draft is still dirty, browser Back has already reached the previous home entry before this handler re-seeds a sentinel. Confirming discard then calls `history.back()` again and skips that destination; preserve whether the original pop already performed the requested navigation and avoid replaying Back in that case.</comment>

<file context>
@@ -0,0 +1,220 @@
+
+  const retireHistorySentinel = useCallback((afterRetirement?: () => void) => {
+    const sentinel = historySentinelRef.current
+    historySentinelRef.current = null
+    const currentState = window.history.state
+    const ownsCurrentEntry =
</file context>

Check if this issue is valid — if so, understand the root cause and fix it. At apps/sim/app/workspace/[workspaceId]/home/home.tsx, line 271:

<comment>When the visible tab is the fallback resource after the URL selection is cleared, this guard sees no active resource and allows agent focus to switch away from a dirty editor. Pass the effective active resource ID to the automatic-focus guard, or keep the URL-selection ref synchronized with the rendered fallback.</comment>

<file context>
@@ -220,20 +261,30 @@ export function Home({ chatId, userName, userId, tableViewsEnabled }: HomeProps)
+      markAttention()
+      return
     }
+    routeAutomaticResourceFocus(
+      activeResourceId,
+      resourceId,
</file context>

Check if this issue is valid — if so, understand the root cause and fix it. At apps/sim/app/workspace/[workspaceId]/settings/components/custom-tools/components/custom-tool-detail/custom-tool-detail.tsx, line 114:

<comment>When an update fails after the optimistic cache update, this treats the optimistic tool as the new baseline. The rollback then replaces the user's edits with the old server value and clears the embedded dirty guard; compare the draft with `previousToolSource` before reseeding.</comment>

<file context>
@@ -75,10 +84,46 @@ export function CustomToolDetail({
+        previousToolSource.code !== nextSource.code)
+
+    if (switchedTool || sourceChanged) {
+      const hadLocalDraft = jsonSchema !== seededSchema || functionCode !== seededCode
+      setPreviousToolSource(nextSource)
+      setSeededSchema(nextSource.schema)
</file context>

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>

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/resource-tabs/resource-tabs.tsx, line 180:

<comment>When an MCP server has an empty name, this lookup replaces the resource's existing title with an empty string, so its tab renders blank. Use the same `Unnamed server` fallback as the resource picker.</comment>

<file context>
@@ -141,36 +138,48 @@ const PREVIEW_MODE_LABELS: Record<PreviewMode, string> = {
     for (const folder of folders ?? []) map.set(`folder:${folder.id}`, folder.name)
+    for (const skill of skills ?? []) map.set(`skill:${skill.id}`, skill.name)
+    for (const tool of customTools ?? []) map.set(`custom_tool:${tool.id}`, tool.title)
+    for (const server of mcpServers ?? []) map.set(`mcp_server:${server.id}`, server.name)
     return map
-  }, [enabled, workflows, tables, files, knowledgeBases, folders])
</file context>

Check if this issue is valid — if so, understand the root cause and fix it. At apps/sim/app/workspace/[workspaceId]/skills/[skillId]/skill-detail.tsx, line 126:

<comment>When an update fails after the optimistic cache update, the rollback is treated as a clean server change and `seedDrafts(skill)` erases the edits the user needs to retry. Ignore optimistic update/rollback transitions when synchronizing drafts so a failed save preserves the local values.</comment>

<file context>
@@ -82,11 +98,33 @@ export function SkillDetail({ workspaceId, skillId }: SkillDetailProps) {
+          descriptionDraft === previousSkillSource.description &&
+          contentDraft === previousSkillSource.content)
+      setPreviousSkillSource(nextSource)
+      if (shouldReseed) seedDrafts(skill)
+    }
   }
</file context>

Check if this issue is valid — if so, understand the root cause and fix it. At apps/sim/app/workspace/[workspaceId]/settings/components/mcp/mcp.tsx, line 457:

<comment>When a background server-list refetch fails after cached data loaded, this branch hides a still-resolvable MCP server detail. Show the load error only when `serverId` cannot be resolved from `servers`.</comment>

<file context>
@@ -425,6 +443,30 @@ export function MCP() {
+    )
+  }
+
+  if (serverId && serversError) {
+    return (
+      <SettingsPanel
</file context>

Check if this issue is valid — if so, understand the root cause and fix it. At apps/sim/lib/copilot/chat/post.ts, line 123:

<comment>When an integration attachment omits `title`, this fallback stores `Integration`, but later specific titles cannot replace it because the shared placeholder set does not recognize that value. Add `Integration` to `GENERIC_RESOURCE_TITLES`.</comment>

<file context>
@@ -134,6 +120,10 @@ const GENERIC_RESOURCE_TITLE: Record<z.infer<typeof ResourceAttachmentSchema>['t
   filefolder: 'File Folder',
   task: 'Task',
   log: 'Log',
+  integration: 'Integration',
+  skill: 'Skill',
+  custom_tool: 'Custom Tool',
</file context>