fix(subagents): hide thinking text

Author: Sg312

apps/sim/.env.example

@@ -17,6 +17,7 @@ BETTER_AUTH_URL=http://localhost:3000
 
 # NextJS (Required)
 NEXT_PUBLIC_APP_URL=http://localhost:3000
+# MOTHERSHIP_SUBAGENT_NARRATION=true  # Optional: show free-form Workflow/Research/etc. agent narration; defaults to false.
 # INTERNAL_API_BASE_URL=http://sim-app.default.svc.cluster.local:3000  # Optional: internal URL for server-side /api self-calls; defaults to NEXT_PUBLIC_APP_URL
 # TRUSTED_ORIGINS=https://www.example.com,https://app.example.com  # Optional: comma-separated additional public origins to trust for auth (apex+www, alias domains). Merged into Better Auth trustedOrigins.
 

apps/sim/lib/copilot/entitlements.test.ts

@@ -0,0 +1,40 @@
+/**
+ * @vitest-environment node
+ */
+import { describe, expect, it, vi } from 'vitest'
+
+const { mockIsFeatureEnabled, mockIsCustomBlocksEligible } = vi.hoisted(() => ({
+  mockIsFeatureEnabled: vi.fn(),
+  mockIsCustomBlocksEligible: vi.fn(),
+}))
+
+vi.mock('@/lib/core/config/feature-flags', () => ({
+  isFeatureEnabled: mockIsFeatureEnabled,
+}))
+
+vi.mock('@/lib/workflows/custom-blocks/operations', () => ({
+  isCustomBlocksEligible: mockIsCustomBlocksEligible,
+}))
+
+import {
+  computeWorkspaceEntitlements,
+  SUBAGENT_NARRATION_ENTITLEMENT,
+} from '@/lib/copilot/entitlements'
+
+describe('computeWorkspaceEntitlements', () => {
+  it('includes subagent narration only when its feature flag is enabled', async () => {
+    mockIsCustomBlocksEligible.mockResolvedValue(false)
+    mockIsFeatureEnabled.mockImplementation(
+      async (_flag: string, context: { userId?: string }) => context.userId === 'enabled-user'
+    )
+
+    await expect(computeWorkspaceEntitlements('workspace-1', 'disabled-user')).resolves.toEqual([])
+    await expect(computeWorkspaceEntitlements('workspace-1', 'enabled-user')).resolves.toEqual([
+      SUBAGENT_NARRATION_ENTITLEMENT,
+    ])
+
+    expect(mockIsFeatureEnabled).toHaveBeenCalledWith('mothership-subagent-narration', {
+      userId: 'enabled-user',
+    })
+  })
+})

apps/sim/lib/copilot/entitlements.ts

@@ -1,6 +1,7 @@
 import { createLogger } from '@sim/logger'
 import { getErrorMessage } from '@sim/utils/errors'
 import { LRUCache } from 'lru-cache'
+import { isFeatureEnabled } from '@/lib/core/config/feature-flags'
 import { isCustomBlocksEligible } from '@/lib/workflows/custom-blocks/operations'
 
 const logger = createLogger('CopilotEntitlements')
@@ -10,6 +11,11 @@ const logger = createLogger('CopilotEntitlements')
  * its `core.Entitlement*` constants to gate agent surfaces.
  */
 export const CUSTOM_BLOCKS_ENTITLEMENT = 'custom-blocks'
+export const SUBAGENT_NARRATION_ENTITLEMENT = 'mothership-subagent-narration'
+
+function isSubagentNarrationEnabled(_workspaceId: string, userId?: string): Promise<boolean> {
+  return isFeatureEnabled('mothership-subagent-narration', { userId })
+}
 
 /**
  * Workspace entitlements — plan/flag-gated org capabilities sent to the
@@ -33,6 +39,7 @@ const ENTITLEMENT_EVALUATORS: Record<
   (workspaceId: string, userId?: string) => Promise<boolean>
 > = {
   [CUSTOM_BLOCKS_ENTITLEMENT]: isCustomBlocksEligible,
+  [SUBAGENT_NARRATION_ENTITLEMENT]: isSubagentNarrationEnabled,
 }
 
 const entitlementsCache = new LRUCache<string, Promise<string[]>>({

apps/sim/lib/core/config/env.ts

@@ -451,6 +451,7 @@ export const env = createEnv({
     DISABLE_INVITATIONS:                   z.boolean().optional(),                 // Disable workspace invitations globally (for self-hosted deployments)
     DISABLE_PUBLIC_API:                    z.boolean().optional(),                 // Disable public API access globally (for self-hosted deployments)
     MOTHERSHIP_BETA_FEATURES:              z.boolean().optional(),                 // Enable beta Mothership planning/changelog artifact surfaces
+    MOTHERSHIP_SUBAGENT_NARRATION:         z.boolean().optional(),                 // Show and stream subagent narration in Mothership chat (defaults to false)
 
     // Development Tools
     REACT_GRAB_ENABLED:                    z.boolean().optional(),                 // Enable React Grab for UI element debugging in Cursor/AI agents (dev only)

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

@@ -12,6 +12,7 @@ const { mockFetch, mockIsPlatformAdmin, envRef, flagRef } = vi.hoisted(() => ({
     APPCONFIG_ENVIRONMENT: 'staging' as string | undefined,
     FORKING_ENABLED: undefined as boolean | undefined,
     DEPLOY_AS_BLOCK: undefined as boolean | undefined,
+    MOTHERSHIP_SUBAGENT_NARRATION: undefined as boolean | undefined,
   },
   flagRef: { isAppConfigEnabled: false },
 }))
@@ -63,6 +64,7 @@ describe('getFeatureFlags', () => {
     const flags = await getFeatureFlags()
     // All registered flags should be present, disabled (env vars unset in test env)
     expect(flags['mothership-beta']).toEqual({ enabled: false })
+    expect(flags['mothership-subagent-narration']).toEqual({ enabled: false })
     expect(flags['pii-redaction']).toEqual({ enabled: false })
     expect(flags['pii-granular-redaction']).toEqual({ enabled: false })
     expect(flags['trigger-eu-region']).toEqual({ enabled: false })
@@ -110,6 +112,7 @@ describe('isFeatureEnabled', () => {
     flagRef.isAppConfigEnabled = false
     envRef.FORKING_ENABLED = undefined
     envRef.DEPLOY_AS_BLOCK = undefined
+    envRef.MOTHERSHIP_SUBAGENT_NARRATION = undefined
   })
 
   describe('workspace-forking flag', () => {
@@ -148,6 +151,21 @@ describe('isFeatureEnabled', () => {
     })
   })
 
+  describe('mothership-subagent-narration flag', () => {
+    it('defaults off and uses its fallback only when explicitly enabled', async () => {
+      expect(await isFeatureEnabled('mothership-subagent-narration', { userId: 'u1' })).toBe(false)
+
+      envRef.MOTHERSHIP_SUBAGENT_NARRATION = true
+      expect(await isFeatureEnabled('mothership-subagent-narration', { userId: 'u1' })).toBe(true)
+    })
+
+    it('supports targeted rollout through AppConfig', async () => {
+      withAppConfig({ 'mothership-subagent-narration': { userIds: ['u1'] } })
+      expect(await isFeatureEnabled('mothership-subagent-narration', { userId: 'u1' })).toBe(true)
+      expect(await isFeatureEnabled('mothership-subagent-narration', { userId: 'u2' })).toBe(false)
+    })
+  })
+
   it('returns false for an unknown flag', async () => {
     withAppConfig({})
     expect(await enabled('missing', { userId: 'u1' })).toBe(false)

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

@@ -64,6 +64,12 @@ const FEATURE_FLAGS = {
       'user context — use enabled:true for global rollout rather than per-user targeting.',
     fallback: 'MOTHERSHIP_BETA_FEATURES',
   },
+  'mothership-subagent-narration': {
+    description:
+      'Stream and render free-form assistant narration from Mothership subagents such as ' +
+      'Workflow Agent. Disabled by default so only subagent lifecycle and tool activity are shown.',
+    fallback: 'MOTHERSHIP_SUBAGENT_NARRATION',
+  },
   'table-snapshot-cache': {
     description:
       'Mount Sim tables into code sandboxes by reference via a version-keyed CSV snapshot in ' +