Skip to content

Commit e7d1d0c

Browse files
waleedlatif1claude
andcommitted
fix(audit): remove redundant metadata fields duplicating top-level audit 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>
1 parent 4021cb2 commit e7d1d0c

7 files changed

Lines changed: 0 additions & 16 deletions

File tree

apps/sim/app/api/credential-sets/[id]/members/route.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,6 @@ export async function DELETE(req: NextRequest, { params }: { params: Promise<{ i
202202
memberUserId: memberToRemove.userId,
203203
targetEmail: memberToRemove.email ?? undefined,
204204
providerId: result.set.providerId,
205-
credentialSetName: result.set.name,
206205
},
207206
request: req,
208207
})

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

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -280,8 +280,6 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
280280
metadata: {
281281
knowledgeBaseName: accessCheck.knowledgeBase?.name,
282282
fileCount: createdDocuments.length,
283-
fileNames: createdDocuments.map((doc) => doc.filename),
284-
uploadType: 'bulk',
285283
},
286284
request: req,
287285
})
@@ -364,7 +362,6 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
364362
fileName: validatedData.filename,
365363
fileType: validatedData.mimeType,
366364
fileSize: validatedData.fileSize,
367-
uploadType: 'single',
368365
},
369366
request: req,
370367
})

apps/sim/app/api/workflows/[id]/deployments/[version]/revert/route.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,6 @@ export async function POST(
129129
description: `Reverted workflow to deployment version ${version}`,
130130
metadata: {
131131
targetVersion: version,
132-
workflowName: workflowRecord?.name ?? undefined,
133-
workspaceId: workflowRecord?.workspaceId ?? undefined,
134132
},
135133
request,
136134
})

apps/sim/app/api/workflows/[id]/duplicate/route.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,6 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
9090
metadata: {
9191
sourceWorkflowId,
9292
newWorkflowId: result.id,
93-
newWorkflowName: result.name,
94-
workspaceId: workspaceId || undefined,
9593
folderId: folderId || undefined,
9694
},
9795
request: req,

apps/sim/lib/workflows/orchestration/chat-deploy.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,6 @@ export async function performChatUndeploy(
213213
resourceName: chatRecord.title || chatId,
214214
description: `Deleted chat deployment "${chatRecord.title || chatId}"`,
215215
metadata: {
216-
chatTitle: chatRecord.title || undefined,
217216
workflowId: chatRecord.workflowId || undefined,
218217
identifier: chatRecord.identifier || undefined,
219218
authType: chatRecord.authType || undefined,

apps/sim/lib/workflows/orchestration/deploy.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,6 @@ export async function performFullDeploy(
212212
metadata: {
213213
deploymentVersionId,
214214
version: deployResult.version,
215-
workflowName: (workflowData.name as string) || undefined,
216215
previousVersionId: previousVersionId || undefined,
217216
triggerWarnings: triggerSaveResult.warnings?.length ? triggerSaveResult.warnings : undefined,
218217
},
@@ -291,9 +290,6 @@ export async function performFullUndeploy(
291290
resourceId: workflowId,
292291
resourceName: (workflowData.name as string) || undefined,
293292
description: `Undeployed workflow "${(workflowData.name as string) || workflowId}"`,
294-
metadata: {
295-
workflowName: (workflowData.name as string) || undefined,
296-
},
297293
})
298294

299295
return { success: true }
@@ -487,7 +483,6 @@ export async function performActivateVersion(
487483
version,
488484
deploymentVersionId: versionRow.id,
489485
previousVersionId: previousVersionId || undefined,
490-
workflowName: (workflow.name as string) || undefined,
491486
},
492487
})
493488

apps/sim/lib/workflows/orchestration/workflow-lifecycle.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,10 +109,8 @@ export async function performDeleteWorkflow(
109109
resourceName: workflowRecord.name,
110110
description: `Archived workflow "${workflowRecord.name}"`,
111111
metadata: {
112-
workflowName: workflowRecord.name,
113112
archived: archiveResult.archived,
114113
templateAction,
115-
workspaceId: workflowRecord.workspaceId || undefined,
116114
},
117115
})
118116

0 commit comments

Comments
 (0)