chore(db): drop retired storage whose readers have shipped

Drops tables and an index whose last readers and writers were removed in
already-released versions, makes chat.include_tool_calls NOT NULL, and
un-declares six dead columns ahead of a next-release DROP COLUMN.
embedding_keyword_tin is dropped by script migration 0031 so it runs after
0030 retires the triggers that still write it on skipped-release upgrades.

Author: waleedlatif1

apps/sim/app/api/chat/[identifier]/otp/route.ts

@@ -231,8 +231,8 @@ export const PUT = withRouteHandler(
         customizations: deployment.customizations,
         authType: deployment.authType,
         outputConfigs: deployment.outputConfigs,
-        includeThinking: deployment.includeThinking ?? false,
-        includeToolCalls: deployment.includeToolCalls ?? false,
+        includeThinking: deployment.includeThinking,
+        includeToolCalls: deployment.includeToolCalls,
       })
       await setChatAuthCookie(response, deployment, email)
 

apps/sim/app/api/chat/[identifier]/route.test.ts

@@ -140,7 +140,7 @@ describe('Chat Identifier API Route', () => {
       },
       outputConfigs: [{ blockId: 'block-1', path: 'output' }],
       includeThinking: false,
-      includeToolCalls: null,
+      includeToolCalls: false,
     },
   ]
 
@@ -362,13 +362,10 @@ describe('Chat Identifier API Route', () => {
       })
     }, 10000)
 
-    /**
-     * A row predating the column has no tool policy, so it has not opted in.
-     * Thinking must not drag tool frames along with it.
-     */
-    it('reads a null tool policy as off rather than inheriting thinking', async () => {
+    /** Thinking and tool calls are independent policies: thinking must not drag tool frames along. */
+    it('keeps tool calls off when only thinking is enabled', async () => {
       const thinkingChatResult = [
-        { ...mockChatResult[0], includeThinking: true, includeToolCalls: null },
+        { ...mockChatResult[0], includeThinking: true, includeToolCalls: false },
       ]
       dbChainMockFns.select.mockImplementation((fields: Record<string, unknown>) => {
         if (fields && fields.isDeployed !== undefined) {

apps/sim/app/api/chat/[identifier]/route.ts

@@ -33,8 +33,8 @@ interface ChatConfigSource {
   customizations: unknown
   authType: string | null
   outputConfigs: unknown
-  includeThinking?: boolean | null
-  includeToolCalls?: boolean | null
+  includeThinking: boolean
+  includeToolCalls: boolean
 }
 
 function toChatConfigResponse(deployment: ChatConfigSource) {
@@ -45,8 +45,8 @@ function toChatConfigResponse(deployment: ChatConfigSource) {
     customizations: deployment.customizations,
     authType: deployment.authType,
     outputConfigs: deployment.outputConfigs,
-    includeThinking: deployment.includeThinking ?? false,
-    includeToolCalls: deployment.includeToolCalls ?? false,
+    includeThinking: deployment.includeThinking,
+    includeToolCalls: deployment.includeToolCalls,
   }
 }
 
@@ -332,8 +332,7 @@ export const POST = withRouteHandler(
           variables: (workflowRecord?.variables as Record<string, unknown>) ?? undefined,
         }
 
-        const includeThinking = deployment.includeThinking ?? false
-        const includeToolCalls = deployment.includeToolCalls ?? false
+        const { includeThinking, includeToolCalls } = deployment
         const agentEvents = shouldEmitAgentStreamEvents({
           includeThinking,
           includeToolCalls,

apps/sim/app/api/chat/presenters.ts

@@ -19,7 +19,7 @@ export function toChatDetailResponse(deployment: ChatDeploymentView, chatUrl: st
     allowedEmails: (deployment.allowedEmails as string[] | null) ?? [],
     outputConfigs: (deployment.outputConfigs as ChatDetail['outputConfigs'] | null) ?? [],
     includeThinking: deployment.includeThinking,
-    includeToolCalls: deployment.includeToolCalls ?? false,
+    includeToolCalls: deployment.includeToolCalls,
     customizations: (deployment.customizations as ChatDetail['customizations']) ?? undefined,
     isActive: deployment.isActive,
     chatUrl,

apps/sim/app/api/v2/chat-deployments/route.test.ts

@@ -87,7 +87,7 @@ function chatRow(overrides: Record<string, unknown> = {}) {
     allowedEmails: [],
     outputConfigs: [],
     includeThinking: false,
-    includeToolCalls: null,
+    includeToolCalls: false,
     archivedAt: null,
     createdAt: new Date('2026-06-12T10:30:00.000Z'),
     updatedAt: new Date('2026-06-12T10:30:00.000Z'),

apps/sim/app/api/v2/chat-deployments/utils.ts

@@ -99,7 +99,7 @@ export function toV2ChatDeployment(
     customizations: normalizeStoredCustomizations(deployment.customizations),
     outputConfigs: normalizeStoredOutputConfigs(deployment.outputConfigs),
     includeThinking: deployment.includeThinking,
-    includeToolCalls: deployment.includeToolCalls ?? false,
+    includeToolCalls: deployment.includeToolCalls,
     createdAt: deployment.createdAt.toISOString(),
     updatedAt: deployment.updatedAt.toISOString(),
   })
@@ -130,7 +130,7 @@ export function toV2ChatDeploymentListItem(
     authType: deployment.authType,
     outputConfigs: normalizeStoredOutputConfigs(deployment.outputConfigs),
     includeThinking: deployment.includeThinking,
-    includeToolCalls: deployment.includeToolCalls ?? false,
+    includeToolCalls: deployment.includeToolCalls,
     createdAt: deployment.createdAt.toISOString(),
     updatedAt: deployment.updatedAt.toISOString(),
   })

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

@@ -107,7 +107,7 @@ function chatRow(overrides: Record<string, unknown> = {}) {
     allowedEmails: [],
     outputConfigs: [],
     includeThinking: false,
-    includeToolCalls: null,
+    includeToolCalls: false,
     archivedAt: null,
     createdAt: new Date('2026-06-12T10:30:00.000Z'),
     updatedAt: new Date('2026-06-12T10:30:00.000Z'),

apps/sim/app/api/workflows/[id]/chat/status/route.test.ts

@@ -67,7 +67,7 @@ function chatRow(overrides: Record<string, unknown> = {}) {
     allowedEmails: ['ceo@victim-corp.com', '@victim-corp.com'],
     outputConfigs: [{ blockId: 'block-1', path: 'output' }],
     includeThinking: true,
-    includeToolCalls: null,
+    includeToolCalls: false,
     archivedAt: null,
     createdAt: new Date('2026-06-12T10:30:00.000Z'),
     updatedAt: new Date('2026-06-12T10:30:00.000Z'),

apps/sim/lib/chat-deployments/application/read-chat-deployments.ts

@@ -24,11 +24,7 @@ export interface ChatDeploymentView extends Omit<ChatDeploymentRow, 'password'>
 
 export function toChatDeploymentView(row: ChatDeploymentRow): ChatDeploymentView {
   const { password, ...rest } = row
-  return {
-    ...rest,
-    includeToolCalls: rest.includeToolCalls ?? false,
-    hasPassword: Boolean(password),
-  }
+  return { ...rest, hasPassword: Boolean(password) }
 }
 
 /** A chat can answer only while both it and its workflow deployment are active. */

apps/sim/lib/chat-deployments/application/update-chat-deployment.ts

@@ -217,8 +217,7 @@ export const updateChatDeployment = defineAuthorizedWorkspaceUseCase({
     if (input.allowedEmails) values.allowedEmails = input.allowedEmails
     if (input.outputConfigs) values.outputConfigs = input.outputConfigs
     if (input.includeThinking !== undefined) values.includeThinking = input.includeThinking
-    /** Partial updates keep the stored value; a row predating the column reads false. */
-    values.includeToolCalls = input.includeToolCalls ?? existing.includeToolCalls ?? false
+    if (input.includeToolCalls !== undefined) values.includeToolCalls = input.includeToolCalls
 
     const updated = await updateChatDeploymentRow(existing.id, values).catch(
       chatIdentifierUniquenessConflict(values.identifier ?? existing.identifier)