fix(ci): restore test isolation and memory file scope checks

Author: Sg312

apps/sim/app/_shell/desktop-title-bar-surfaces.test.ts

@@ -166,7 +166,10 @@ describe('desktop title-bar surface audit', () => {
     // (KnowledgeBase et al) whose header bars reserve the lane. It inherits the
     // variable, so without this reset those bars gain the inset while sitting nowhere
     // near the traffic lights.
-    expect(mothershipView).toContain('[--workspace-content-title-bar-inset:0px]')
+    expect(mothershipView).toContain('<ChatPanelContent')
+    expect(read('../workspace/[workspaceId]/home/components/chat-panel-layout.tsx')).toContain(
+      '[--workspace-content-title-bar-inset:0px]'
+    )
   })
 
   it('reserves that lane in every top-of-pane header bar', () => {

apps/sim/app/api/copilot/api-keys/validate/route.ts

@@ -23,6 +23,8 @@ import {
 import { getHighestPrioritySubscription } from '@/lib/billing/core/plan'
 import { isEnterprisePlan } from '@/lib/billing/core/subscription'
 import { deriveBillingContext } from '@/lib/billing/core/usage-log'
+import { isBillingEnabled, isHosted } from '@/lib/core/config/env-flags'
+import { asOrchestrationError } from '@/lib/core/orchestration/types'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import {
   authorizeCopilotChatCallback,
@@ -48,8 +50,6 @@ import { TraceAttr } from '@/lib/mothership/generated/trace-attributes-v1'
 import { TraceSpan } from '@/lib/mothership/generated/trace-spans-v1'
 import { checkInternalApiKey } from '@/lib/mothership/request/http'
 import { withIncomingGoSpan } from '@/lib/mothership/request/otel'
-import { isBillingEnabled, isHosted } from '@/lib/core/config/env-flags'
-import { asOrchestrationError } from '@/lib/core/orchestration/types'
 
 const logger = createLogger('CopilotApiKeysValidate')
 

apps/sim/app/api/copilot/chat/stop/route.ts

@@ -4,8 +4,8 @@ import { type NextRequest, NextResponse } from 'next/server'
 import { copilotChatStopContract } from '@/lib/api/contracts/copilot'
 import { parseRequest } from '@/lib/api/server'
 import { getSession } from '@/lib/auth'
-import { getAccessibleCopilotChatAuth } from '@/lib/mothership/chat/lifecycle'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
+import { getAccessibleCopilotChatAuth } from '@/lib/mothership/chat/lifecycle'
 import {
   normalizeMessage,
   type PersistedMessage,

apps/sim/app/api/copilot/chat/stream/route.test.ts

@@ -3,25 +3,21 @@
  */
 
 import { authMockFns } from '@sim/testing'
-import { OrchestrationError } from '@/lib/core/orchestration/types'
 import { NextRequest } from 'next/server'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { OrchestrationError } from '@/lib/core/orchestration/types'
 import {
   MothershipStreamV1CompletionStatus,
   MothershipStreamV1EventType,
 } from '@/lib/mothership/generated/mothership-stream-v1'
 
-const {
-  getLatestRunForStream,
-  readEvents,
-  readFilePreviewSessions,
-  checkForReplayGap,
-} = vi.hoisted(() => ({
-  getLatestRunForStream: vi.fn(),
-  readEvents: vi.fn(),
-  readFilePreviewSessions: vi.fn(),
-  checkForReplayGap: vi.fn(),
-}))
+const { getLatestRunForStream, readEvents, readFilePreviewSessions, checkForReplayGap } =
+  vi.hoisted(() => ({
+    getLatestRunForStream: vi.fn(),
+    readEvents: vi.fn(),
+    readFilePreviewSessions: vi.fn(),
+    checkForReplayGap: vi.fn(),
+  }))
 
 vi.mock('@/lib/mothership/request/application/recover-stream', () => ({
   readChatStream: { execute: getLatestRunForStream },

apps/sim/app/api/copilot/chat/stream/route.ts

@@ -11,6 +11,7 @@ import {
   internalOrchestrationErrorPolicy,
   internalSessionAuth,
 } from '@/lib/api/server/routes'
+import { encodeSSEComment } from '@/lib/core/utils/sse'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import {
   MothershipStreamV1CompletionStatus,
@@ -35,7 +36,6 @@ import {
   SSE_RESPONSE_HEADERS,
 } from '@/lib/mothership/request/session'
 import { toReplayEnvelope, toStreamBatchEvent } from '@/lib/mothership/request/session/types'
-import { encodeSSEComment } from '@/lib/core/utils/sse'
 
 export const maxDuration = 3600
 

apps/sim/app/api/knowledge/search/utils.test.ts

@@ -740,7 +740,7 @@ describe('Knowledge Search Utils', () => {
       })
 
       await expect(generateSearchEmbedding('test query', DEFAULT_EMBEDDING_TARGET)).rejects.toThrow(
-        'OPENAI_API_KEY is not configured'
+        'Semantic retrieval is unavailable because its embedding provider is not configured.'
       )
     })
 

apps/sim/app/api/knowledge/utils.test.ts

@@ -341,7 +341,7 @@ describe('Knowledge Utils', () => {
 
       try {
         await expect(generateEmbeddings(['test text'], DEFAULT_EMBEDDING_TARGET)).rejects.toThrow(
-          'OPENAI_API_KEY is not configured'
+          'Semantic retrieval is unavailable because its embedding provider is not configured.'
         )
         expect(rotationSpy).toHaveBeenCalledWith('openai')
         expect(fetch).not.toHaveBeenCalled()

apps/sim/app/api/mothership/chat/route.ts

@@ -5,8 +5,8 @@ import {
 } from '@/lib/api/contracts/mothership-chats'
 import { validationErrorResponse } from '@/lib/api/server'
 import { getSession } from '@/lib/auth'
-import { handleUnifiedChatPost } from '@/lib/mothership/chat/post'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
+import { handleUnifiedChatPost } from '@/lib/mothership/chat/post'
 import { validateShimEnvelope } from '@/lib/mothership/request/http'
 import { GET as copilotChatGet } from '@/app/api/copilot/chat/queries'
 

apps/sim/app/api/mothership/events/route.ts

@@ -15,12 +15,12 @@ import {
   InternalUnauthenticatedError,
   internalSessionAuth,
 } from '@/lib/api/server/routes/internal-json-route'
-import { authorizeOrganizationChatEvents } from '@/lib/mothership/chat/organization-chats'
-import { chatPubSub } from '@/lib/mothership/chat-status'
 import { isChatEnabled } from '@/lib/core/config/env-flags'
 import { asOrchestrationError } from '@/lib/core/orchestration/types'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 import { createSSEStream, createWorkspaceSSE } from '@/lib/events/sse-endpoint'
+import { authorizeOrganizationChatEvents } from '@/lib/mothership/chat/organization-chats'
+import { chatPubSub } from '@/lib/mothership/chat-status'
 
 export const dynamic = 'force-dynamic'
 

apps/sim/app/api/v2/workflows/[workflowId]/export/route.test.ts

@@ -48,7 +48,11 @@ describe('/api/v2/workflows/[workflowId]/export route definition', () => {
     })
     expect(
       mapInput({ params: { workflowId: 'workflow-1' }, query: { includeWorkspaceBindings: true } })
-    ).toEqual({ workflowId: 'workflow-1', includeReferences: false, includeWorkspaceBindings: true })
+    ).toEqual({
+      workflowId: 'workflow-1',
+      includeReferences: false,
+      includeWorkspaceBindings: true,
+    })
     expect(
       mapInput({
         params: { workflowId: 'workflow-1' },