fix(audit): remove redundant metadata fields duplicating top-level au…

…dit fields Remove metadata entries that duplicate resourceName, workspaceId, or other top-level recordAudit fields. Also remove noisy fileNames arrays from bulk document upload audits (kept fileCount). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
simstudioai · waleedlatif1 · Apr 11, 2026 · Apr 11, 2026 · Apr 11, 2026 · Apr 11, 2026
commit e7d1d0c76e0a0b356c9b5eae9f84f3f39bc1459f
diff --git a/apps/sim/app/api/credential-sets/[id]/members/route.ts b/apps/sim/app/api/credential-sets/[id]/members/route.ts
@@ -202,7 +202,6 @@ export async function DELETE(req: NextRequest, { params }: { params: Promise<{ i
         memberUserId: memberToRemove.userId,
         targetEmail: memberToRemove.email ?? undefined,
         providerId: result.set.providerId,
-        credentialSetName: result.set.name,
       },
       request: req,
     })

diff --git a/apps/sim/app/api/knowledge/[id]/documents/route.ts b/apps/sim/app/api/knowledge/[id]/documents/route.ts
@@ -280,8 +280,6 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
           metadata: {
             knowledgeBaseName: accessCheck.knowledgeBase?.name,
             fileCount: createdDocuments.length,
-            fileNames: createdDocuments.map((doc) => doc.filename),
-            uploadType: 'bulk',
           },
           request: req,
         })
@@ -364,7 +362,6 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
             fileName: validatedData.filename,
             fileType: validatedData.mimeType,
             fileSize: validatedData.fileSize,
-            uploadType: 'single',
           },
           request: req,
         })

diff --git a/apps/sim/app/api/workflows/[id]/deployments/[version]/revert/route.ts b/apps/sim/app/api/workflows/[id]/deployments/[version]/revert/route.ts
@@ -129,8 +129,6 @@ export async function POST(
       description: `Reverted workflow to deployment version ${version}`,
       metadata: {
         targetVersion: version,
-        workflowName: workflowRecord?.name ?? undefined,
-        workspaceId: workflowRecord?.workspaceId ?? undefined,
       },
       request,
     })

diff --git a/apps/sim/app/api/workflows/[id]/duplicate/route.ts b/apps/sim/app/api/workflows/[id]/duplicate/route.ts
@@ -90,8 +90,6 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
       metadata: {
         sourceWorkflowId,
         newWorkflowId: result.id,
-        newWorkflowName: result.name,
-        workspaceId: workspaceId || undefined,
         folderId: folderId || undefined,
       },
       request: req,

diff --git a/apps/sim/lib/workflows/orchestration/chat-deploy.ts b/apps/sim/lib/workflows/orchestration/chat-deploy.ts
@@ -213,7 +213,6 @@ export async function performChatUndeploy(
     resourceName: chatRecord.title || chatId,
     description: `Deleted chat deployment "${chatRecord.title || chatId}"`,
     metadata: {
-      chatTitle: chatRecord.title || undefined,
       workflowId: chatRecord.workflowId || undefined,
       identifier: chatRecord.identifier || undefined,
       authType: chatRecord.authType || undefined,

diff --git a/apps/sim/lib/workflows/orchestration/deploy.ts b/apps/sim/lib/workflows/orchestration/deploy.ts
@@ -212,7 +212,6 @@ export async function performFullDeploy(
     metadata: {
       deploymentVersionId,
       version: deployResult.version,
-      workflowName: (workflowData.name as string) || undefined,
       previousVersionId: previousVersionId || undefined,
       triggerWarnings: triggerSaveResult.warnings?.length ? triggerSaveResult.warnings : undefined,
     },
@@ -291,9 +290,6 @@ export async function performFullUndeploy(
     resourceId: workflowId,
     resourceName: (workflowData.name as string) || undefined,
     description: `Undeployed workflow "${(workflowData.name as string) || workflowId}"`,
-    metadata: {
-      workflowName: (workflowData.name as string) || undefined,
-    },
   })
 
   return { success: true }
@@ -487,7 +483,6 @@ export async function performActivateVersion(
       version,
       deploymentVersionId: versionRow.id,
       previousVersionId: previousVersionId || undefined,
-      workflowName: (workflow.name as string) || undefined,
     },
   })
 

diff --git a/apps/sim/lib/workflows/orchestration/workflow-lifecycle.ts b/apps/sim/lib/workflows/orchestration/workflow-lifecycle.ts
@@ -109,10 +109,8 @@ export async function performDeleteWorkflow(
     resourceName: workflowRecord.name,
     description: `Archived workflow "${workflowRecord.name}"`,
     metadata: {
-      workflowName: workflowRecord.name,
       archived: archiveResult.archived,
       templateAction,
-      workspaceId: workflowRecord.workspaceId || undefined,
     },
   })