fix(db): change document.connectorId FK from cascade to set null

The cascade behavior meant deleting a connector would always delete
its documents, contradicting the "keep documents" option. With set null,
the database automatically nullifies connectorId when a connector is
removed, and we only need explicit deletion when the user opts in.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/app/api/knowledge/[id]/connectors/[connectorId]/route.ts

@@ -314,17 +314,6 @@ export async function DELETE(request: NextRequest, { params }: RouteParams) {
           await tx.delete(embedding).where(inArray(embedding.documentId, documentIds))
           await tx.delete(document).where(inArray(document.id, documentIds))
         }
-      } else if (docs.length > 0) {
-        await tx
-          .update(document)
-          .set({ connectorId: null })
-          .where(
-            and(
-              eq(document.connectorId, connectorId),
-              isNull(document.archivedAt),
-              isNull(document.deletedAt)
-            )
-          )
       }
 
       const deletedConnectors = await tx

packages/db/migrations/0182_luxuriant_quasar.sql

@@ -0,0 +1,3 @@
+ALTER TABLE "document" DROP CONSTRAINT "document_connector_id_knowledge_connector_id_fk";
+--> statement-breakpoint
+ALTER TABLE "document" ADD CONSTRAINT "document_connector_id_knowledge_connector_id_fk" FOREIGN KEY ("connector_id") REFERENCES "public"."knowledge_connector"("id") ON DELETE set null ON UPDATE no action;

packages/db/schema.ts

@@ -1275,7 +1275,7 @@ export const document = pgTable(
 
     // Connector-sourced document fields
     connectorId: text('connector_id').references(() => knowledgeConnector.id, {
-      onDelete: 'cascade',
+      onDelete: 'set null',
     }),
     externalId: text('external_id'),
     contentHash: text('content_hash'),