refactor(dashboards): move authoring guidance to Mothership like Sim Pages

Drop the create-dashboard built-in skill and its rollout gating in the skill
lists. Mothership now learns the dashboard format from a sim-dashboards
reference in its own research-and-deliverables skill, the pattern Sim Pages
use; Sim workspace built-ins stay Agent-block documents. The dashboards flag
still gates the viewer and table analytics.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01S6aTRnkiu7PxYZPNPXYEMV

Author: TheodoreSpeaks

apps/sim/app/o/[organizationId]/home/components/composer/composer.tsx

@@ -25,7 +25,6 @@ import {
 } from '@/app/workspace/[workspaceId]/home/components/user-input/components/prompt-editor'
 import { organizationSkillOptions } from '@/app/workspace/[workspaceId]/home/components/user-input/components/skills-menu-dropdown/organization-skill-options'
 import type { ChatRequestMode } from '@/app/workspace/[workspaceId]/home/types'
-import { useFeatureFlag } from '@/app/workspace/[workspaceId]/providers/feature-flags-provider'
 import type { useFileAttachments } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/user-input/hooks/use-file-attachments'
 import { SKILL_CHIP_TRIGGER } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/user-input/utils'
 import { getSkillsQueryOptions } from '@/hooks/queries/skills'
@@ -76,7 +75,6 @@ export function Composer({
   attachedFilesRef.current = files.attachedFiles
   const imagesOnly = requestMode === 'assistant'
   const { organization } = useOrganizationContext()
-  const dashboardsEnabled = useFeatureFlag('dashboards')
   const { data: allWorkspaces = [] } = useWorkspacesQuery(!imagesOnly)
   const workspaces = (imagesOnly ? [] : allWorkspaces).filter(
     (workspace) => workspace.organizationId === organization.id
@@ -90,8 +88,7 @@ export function Composer({
         workspaces.map((workspace, index) => ({
           ...workspace,
           skills: skillQueries[index].isPlaceholderData ? [] : (skillQueries[index].data ?? []),
-        })),
-        dashboardsEnabled ? [] : ['builtin-create-dashboard']
+        }))
       )
   const editor = usePromptEditor({
     workspaceId: '',

apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/skills-menu-dropdown/organization-skill-options.ts

@@ -1,20 +1,16 @@
 import { BUILTIN_SKILLS, isBuiltinSkillId } from '@/lib/workflows/skills/builtin-skills'
 import type { SkillDefinition } from '@/hooks/queries/skills'
 
-/**
- * Built-ins are global templates; only user-defined skills carry a workspace. Rollout-gated
- * built-ins (the dashboard skill) are excluded the same way the server skill lists exclude them.
- */
+/** Built-ins are global templates; only user-defined skills carry a workspace. */
 export function organizationSkillOptions(
   workspaces: ReadonlyArray<{
     id: string
     name: string
     skills: readonly SkillDefinition[]
-  }>,
-  excludedBuiltinIds: readonly string[] = []
+  }>
 ): (SkillDefinition & { workspaceName?: string })[] {
   return [
-    ...BUILTIN_SKILLS.filter((skill) => !excludedBuiltinIds.includes(skill.id)).map((skill) => ({
+    ...BUILTIN_SKILLS.map((skill) => ({
       ...skill,
       workspaceId: null,
       userId: null,

apps/sim/lib/core/config/feature-flags.ts

@@ -48,7 +48,7 @@ interface FeatureFlagDefinition {
 const FEATURE_FLAGS = {
   dashboards: {
     description:
-      'Enable .dashboard file rendering, table analytics, and the dashboard authoring skill. Supports global and organization rollout; disabled by default.',
+      'Enable .dashboard file rendering and table analytics. Supports global and organization rollout; disabled by default.',
     fallback: 'DASHBOARDS',
   },
   'mothership-model-selector': {

apps/sim/lib/dashboards/README.md

@@ -1,6 +1,6 @@
 # Table-backed dashboards
 
-A dashboard is a workspace file with MIME type `text/x-sim-dashboard`, handled like Sim pages. Creating or uploading `<Name>.dashboard` is the ingestion signal: the stored name drops the suffix and the content type alone marks the kind, which content updates never reset. It lives in Files and its folders, is created and edited with the ordinary file tools, and shows up in Chat as a file. The Files viewer renders it live. The built-in **create-dashboard** skill documents the syntax. A public share link shows a workspace-only notice and issues no table queries; live public dashboards are deferred.
+A dashboard is a workspace file with MIME type `text/x-sim-dashboard`, handled like Sim pages. Creating or uploading `<Name>.dashboard` is the ingestion signal: the stored name drops the suffix and the content type alone marks the kind, which content updates never reset. It lives in Files and its folders, is created and edited with the ordinary file tools, and shows up in Chat as a file. The Files viewer renders it live. Mothership learns the syntax from the `sim-dashboards` reference in its `research-and-deliverables` skill, the same way it learns Sim Pages. A public share link shows a workspace-only notice and issues no table queries; live public dashboards are deferred.
 
 The implementation has three boundaries:
 
@@ -20,7 +20,7 @@ The toolbar has side-by-side time-range and timezone dropdowns, followed by an i
 
 EMCN is an ECharts theme, so authored `option.color`, series styles, text, and axis styles override its defaults. Standard string axis formatters also override adaptive labels; omit them to get timezone-aware dates and intraday times automatically. Floating tooltips, hover readouts, cursor synchronization, and range selection belong to the framework. Tabs, controls, typography outside the plot, and responsive layout remain EMCN-owned.
 
-Authoring guidance defaults to the muted theme: single-measure panels share a neutral color, comparisons use the shared palette or line patterns, and explicit colors are reserved for user-requested meaning. It contains no example dashboards. Text blocks are optional brief annotations, not viewer instructions or implementation caveats. Chart grids use ECharts 6 outer bounds to fit axis names and end ticks inside the canvas, and horizontal category labels leave space above the configured bar thickness.
+The authoring reference defaults to the muted theme: panels rely on the renderer's colors, and explicit colors are reserved for user-requested meaning. Text blocks are optional brief annotations, not viewer instructions or implementation caveats. Chart grids use ECharts 6 outer bounds to fit axis names and end ticks inside the canvas, and horizontal category labels leave space above the configured bar thickness.
 
 ## Time and results
 
@@ -67,5 +67,5 @@ global switch. Local development uses `DASHBOARDS=true` in the app's ignored
 environment file.
 
 The server resolves the flag for the canonical workspace organization. It gates
-the `.dashboard` viewer, table analytics, and the built-in authoring skill. With
+the `.dashboard` viewer and table analytics. The Mothership authoring reference is not flag-gated, like Sim Pages; in an organization without the flag a `.dashboard` file still saves but does not render. With
 the flag off, the viewer shows a notice and the YAML stays editable.

apps/sim/lib/dashboards/authoring.ts

@@ -14,13 +14,11 @@ import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 const { hoisted } = vi.hoisted(() => ({
   hoisted: {
-    flag: vi.fn(),
     getById: vi.fn(),
     update: vi.fn(),
   },
 }))
 
-vi.mock('@/lib/dashboards/feature-flag', () => ({ requireDashboardsEnabled: hoisted.flag }))
 vi.mock('@/lib/uploads/contexts/workspace', () => workspaceUploadsMock)
 vi.mock('@sim/platform-authz/workspace', () => workspaceAuthzMock)
 vi.mock('@sim/audit', () => auditMock)
@@ -63,24 +61,12 @@ const skill = {
 
 describe('skill application use cases', () => {
   beforeEach(() => {
-    mocks.flag.mockResolvedValue(undefined)
     mocks.loadContext.mockResolvedValue(workspace)
     mocks.resolvePermission.mockResolvedValue('read')
     mocks.getById.mockResolvedValue(skill)
     mocks.update.mockResolvedValue({ ...skill, content: '# Updated' })
   })
 
-  it('refuses the dashboard authoring skill when dashboards are disabled', async () => {
-    mocks.getById.mockResolvedValue({ ...skill, id: 'builtin-create-dashboard' })
-    mocks.flag.mockRejectedValue(new Error('Dashboards are not enabled'))
-    await expect(
-      getSkillUseCase.execute({
-        principal: { kind: 'session', userId: 'reader' },
-        input: { workspaceId: workspace.workspaceId, skillId: 'builtin-create-dashboard' },
-      })
-    ).rejects.toThrow('Dashboards are not enabled')
-  })
-
   it.each(['skill-1', 'builtin-research'])(
     'reads %s with read permission in the asserted workspace',
     async (skillId) => {

apps/sim/lib/skills/application/use-cases.test.ts

@@ -11,7 +11,6 @@ import {
   ForbiddenOperationError,
 } from '@/lib/core/application'
 import { OrchestrationError } from '@/lib/core/orchestration/types'
-import { isDashboardsEnabled, requireDashboardsEnabled } from '@/lib/dashboards/feature-flag'
 import { getSkillActorContext, listSkillEditors, type SkillEditor } from '@/lib/skills/access'
 import { skillDelegationPolicy } from '@/lib/skills/application/authorization'
 import { skillOperations } from '@/lib/skills/application/operations'
@@ -212,9 +211,6 @@ export const listSkillsUseCase = defineAuthorizedWorkspaceUseCase({
   async execute({ input, context }) {
     const page = await listSkillSummariesPage({
       workspaceId: context.workspaceId,
-      excludedBuiltinIds: (await isDashboardsEnabled(context.workspaceOrganizationId))
-        ? []
-        : ['builtin-create-dashboard'],
       search: input.search,
       sortBy: input.sortBy,
       sortOrder: input.sortOrder,
@@ -243,9 +239,6 @@ export const listAvailableSkillsUseCase = defineAuthorizedWorkspaceUseCase({
     const skills = await listSkillsForUser({
       workspaceId: context.workspaceId,
       userId: requirePrincipalSubjectUserId(principal),
-      excludedBuiltinIds: (await isDashboardsEnabled(context.workspaceOrganizationId))
-        ? []
-        : ['builtin-create-dashboard'],
     })
     return { skills }
   },
@@ -261,11 +254,6 @@ export const getSkillUseCase = defineAuthorizedWorkspaceUseCase({
   resolveContext: ({ input }: { input: GetSkillInput }) =>
     resolveSkillContext(input.workspaceId, input.skillId),
   authorizationOptions,
-  async authorizeResource({ context }) {
-    if (context.skill.id === 'builtin-create-dashboard') {
-      await requireDashboardsEnabled(context.workspaceOrganizationId)
-    }
-  },
   async execute({ context }) {
     return { skill: context.skill }
   },

apps/sim/lib/skills/application/use-cases.ts

@@ -1,5 +1,3 @@
-import { DASHBOARD_AUTHORING_CONTENT } from '@/lib/dashboards/authoring'
-
 /**
  * Built-in (template) skills that ship with every workspace.
  *
@@ -192,13 +190,6 @@ You can attach an existing workflow to a table so it runs once for every row, wi
 `
 
 export const BUILTIN_SKILLS: readonly BuiltinSkill[] = [
-  {
-    id: 'builtin-create-dashboard',
-    name: 'create-dashboard',
-    description:
-      'Build a live table-backed dashboard with YAML, ECharts, time filters, rows, and tabs.',
-    content: DASHBOARD_AUTHORING_CONTENT,
-  },
   {
     id: 'builtin-connect-integration',
     name: 'connect-integration',

apps/sim/lib/workflows/skills/builtin-skills.ts

@@ -64,17 +64,10 @@ function builtinSkillRow(workspaceId: string, builtin: BuiltinSkill): SkillRow {
  * built-in by sharing its name, and a name that matches the search on the
  * built-in matches it on the DB row too.
  */
-function visibleBuiltins(
-  dbNames: Set<string>,
-  search?: string,
-  excludedIds: readonly string[] = []
-): BuiltinSkill[] {
+function visibleBuiltins(dbNames: Set<string>, search?: string): BuiltinSkill[] {
   const term = search?.toLowerCase()
   return BUILTIN_SKILLS.filter(
-    (b) =>
-      !excludedIds.includes(b.id) &&
-      !dbNames.has(b.name.toLowerCase()) &&
-      (!term || b.name.toLowerCase().includes(term))
+    (b) => !dbNames.has(b.name.toLowerCase()) && (!term || b.name.toLowerCase().includes(term))
   )
 }
 export type SkillSortBy = 'name' | 'createdAt' | 'updatedAt'
@@ -186,7 +179,6 @@ export interface SkillSummaryPage {
  */
 export async function listSkillSummariesPage(params: {
   workspaceId: string
-  excludedBuiltinIds?: readonly string[]
   search?: string
   sortBy: SkillSortBy
   sortOrder: ListSortOrder
@@ -202,7 +194,7 @@ export async function listSkillSummariesPage(params: {
     .orderBy(...listOrderBy(SKILL_SORTS[sortBy], sortOrder))
 
   const dbNames = new Set(dbRows.map((r) => r.name.toLowerCase()))
-  const builtins = visibleBuiltins(dbNames, params.search, params.excludedBuiltinIds).map((b) =>
+  const builtins = visibleBuiltins(dbNames, params.search).map((b) =>
     builtinSkillSummaryRow(params.workspaceId, b)
   )
 
@@ -231,7 +223,6 @@ export type SkillWithAccess = typeof skill.$inferSelect & { canEdit: boolean }
  */
 export async function listSkillsForUser(params: {
   workspaceId: string
-  excludedBuiltinIds?: readonly string[]
   userId: string
   includeBuiltins?: boolean
   workspaceAccess?: WorkspaceAccess
@@ -252,10 +243,8 @@ export async function listSkillsForUser(params: {
 
   // A workspace skill that shares a built-in's name overrides it for everyone.
   const dbNames = new Set(tagged.map((r) => r.name.toLowerCase()))
-  const builtins: SkillWithAccess[] = visibleBuiltins(
-    dbNames,
-    undefined,
-    params.excludedBuiltinIds
+  const builtins: SkillWithAccess[] = BUILTIN_SKILLS.filter(
+    (b) => !dbNames.has(b.name.toLowerCase())
   ).map((b) => ({ ...builtinSkillRow(params.workspaceId, b), canEdit: false }))
   return [...builtins, ...tagged]
 }