fix(chat): preserve configured origin in deployment URLs

Author: Sg312

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

@@ -4,14 +4,16 @@
 import {
   MockV2ApiKeyUnauthenticatedError,
   resetDbChainMock,
+  resetEnvMock,
+  setEnv,
   V2_OPERATION_RATE_LIMIT_ALLOWED,
   V2_PREAUTH_RATE_LIMIT_ALLOWED,
   v2ApiKeyAuthModuleMock,
   v2RateLimiterModuleMock,
   v2RouteMocks,
 } from '@sim/testing'
 import { NextRequest } from 'next/server'
-import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 import { PermissionGroupCapabilityError } from '@/lib/permission-groups/capability-error'
 
 const mocks = vi.hoisted(() => ({
@@ -149,6 +151,7 @@ function uniqueViolation(constraint: string) {
 const validBody = { identifier: 'support', title: 'Support chat' }
 
 describe('/api/v2/workflows/[workflowId]/deployments/chat', () => {
+  afterEach(resetEnvMock)
   beforeEach(() => {
     vi.clearAllMocks()
     resetDbChainMock()
@@ -250,6 +253,16 @@ describe('/api/v2/workflows/[workflowId]/deployments/chat', () => {
   })
 
   describe('PUT', () => {
+    it('returns the configured www chat URL to CLI callers on deploy and read', async () => {
+      setEnv({ NEXT_PUBLIC_APP_URL: 'https://www.dev.sim.ai' })
+      mocks.getLiveChatDeployment.mockResolvedValueOnce(null).mockResolvedValue(chatRow())
+
+      for (const response of [await put(validBody), await get()]) {
+        expect(response.status).toBe(200)
+        expect((await response.json()).data.url).toBe('https://www.dev.sim.ai/chat/support')
+      }
+    })
+
     it('creates the chat when the workflow publishes none', async () => {
       mocks.getLiveChatDeployment.mockResolvedValueOnce(null).mockResolvedValue(chatRow())
 

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/components/deploy-modal/components/chat/chat.tsx

@@ -20,8 +20,8 @@ import { Check, TriangleAlert } from '@sim/emcn/icons'
 import { createLogger } from '@sim/logger'
 import { getErrorMessage } from '@sim/utils/errors'
 import { GeneratedPasswordInput } from '@/components/ui'
+import { buildChatDeploymentUrl } from '@/lib/chat-deployments/urls'
 import { useDeploymentShape } from '@/lib/core/config/deployment-shape'
-import { getBaseUrl, getEmailDomain } from '@/lib/core/utils/urls'
 import { validateAllowlistEntry } from '@/lib/messaging/email/validation'
 import { formatInternalOutputSelector } from '@/lib/workflows/streaming/output-selector'
 import { OutputSelect } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/chat/components/output-select/output-select'
@@ -497,7 +497,7 @@ export function ChatDeploy({
           { text: existingChat?.title || 'this chat', bold: true },
           '? ',
           {
-            text: `This will remove the chat at "${getEmailDomain()}/chat/${existingChat?.identifier ?? ''}" and make it unavailable to all users.`,
+            text: `This will remove the chat at "${buildChatDeploymentUrl(existingChat?.identifier ?? '').replace(/^https?:\/\//, '')}" and make it unavailable to all users.`,
             error: true,
           },
           ' This action cannot be undone.',
@@ -554,7 +554,7 @@ interface IdentifierInputProps {
 }
 
 const getDomainPrefix = (() => {
-  const prefix = `${getEmailDomain()}/chat/`
+  const prefix = buildChatDeploymentUrl('').replace(/^https?:\/\//, '')
   return () => prefix
 })()
 
@@ -581,7 +581,7 @@ function IdentifierInput({
     onChange(lowercaseValue)
   }
 
-  const fullUrl = `${getBaseUrl()}/chat/${value}`
+  const fullUrl = buildChatDeploymentUrl(value)
   const displayUrl = fullUrl.replace(/^https?:\/\//, '')
 
   return (

apps/sim/lib/chat-deployments/urls.test.ts

@@ -17,10 +17,16 @@ describe('buildChatDeploymentUrl', () => {
     expect(buildChatDeploymentUrl('support')).toBe('https://sim.ai/chat/support')
   })
 
-  it('strips the www prefix, because the deployed chat answers on the bare host', () => {
-    setEnv({ NEXT_PUBLIC_APP_URL: 'https://www.sim.ai' })
+  it.each([
+    'https://www.dev.sim.ai',
+    'https://www.staging.sim.ai',
+    'https://www.sim.ai',
+    'https://www.custom.example:8443',
+    'http://localhost:3000',
+  ])('preserves the configured origin %s', (origin) => {
+    setEnv({ NEXT_PUBLIC_APP_URL: `${origin}/` })
 
-    expect(buildChatDeploymentUrl('support')).toBe('https://sim.ai/chat/support')
+    expect(buildChatDeploymentUrl('support')).toBe(`${origin}/chat/support`)
   })
 
   /**

apps/sim/lib/chat-deployments/urls.ts

@@ -5,8 +5,9 @@ import { getBaseUrl, getEmailDomain } from '@/lib/core/utils/urls'
  * The public URL a deployed chat answers on.
  *
  * There is no chat subdomain: `proxy.ts` routes chat purely by the `/chat/`
- * path, so the URL is the app host plus the identifier. The `www.` prefix is
- * stripped because the deployed chat is served on the bare host.
+ * path, so the URL must preserve the configured app origin, including `www.`
+ * and any port. Host aliases may redirect, drop the chat path, or have a
+ * different login cookie scope.
  *
  * Single source of truth for the previously independent constructions — the
  * deploy orchestration, the manage read, and the manage write — which had
@@ -25,11 +26,5 @@ export function buildChatDeploymentUrl(identifier: string): string {
   } catch {
     return `${isDev ? 'http' : 'https'}://${getEmailDomain()}/chat/${identifier}`
   }
-  try {
-    const url = new URL(baseUrl)
-    const host = url.host.startsWith('www.') ? url.host.slice('www.'.length) : url.host
-    return `${url.protocol}//${host}/chat/${identifier}`
-  } catch {
-    return `${baseUrl}/chat/${identifier}`
-  }
+  return `${baseUrl}/chat/${identifier}`
 }