chore(uploads): trim verbose comments

Drop inline comments that restate code ("Use the full storageKey as fileName"),
collapse a multi-line block comment into a tighter TSDoc on the existence
check, and prune verbose vitest file headers — describe blocks already
document what's tested.

Author: waleedlatif1

apps/sim/app/api/files/multipart/route.test.ts

@@ -1,7 +1,4 @@
 /**
- * Tests for the generic multipart upload route — focuses on parts shape
- * normalization across S3 and Azure Blob providers.
- *
  * @vitest-environment node
  */
 import { authMockFns, permissionsMock, permissionsMockFns } from '@sim/testing'

apps/sim/app/api/workspaces/[id]/files/presigned/route.test.ts

@@ -1,6 +1,4 @@
 /**
- * Tests for workspace presigned upload API route.
- *
  * @vitest-environment node
  */
 import {

apps/sim/app/api/workspaces/[id]/files/register/route.test.ts

@@ -1,6 +1,4 @@
 /**
- * Tests for workspace file register API route.
- *
  * @vitest-environment node
  */
 import {

apps/sim/lib/uploads/client/direct-upload.test.ts

@@ -1,9 +1,4 @@
 /**
- * Tests for the client-side direct-upload transport. Focuses on:
- * - Strategy selection at the LARGE_FILE_THRESHOLD boundary.
- * - Parts shape `{ partNumber, etag? }` sent to /complete.
- * - FALLBACK_REQUIRED propagation when the server reports no cloud storage.
- *
  * @vitest-environment jsdom
  */
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'

apps/sim/lib/uploads/contexts/workspace/workspace-file-manager.ts

@@ -175,12 +175,12 @@ export async function uploadWorkspaceFile(
 
       const uploadResult = await uploadFile({
         file: fileBuffer,
-        fileName: storageKey, // Use the full storageKey as fileName
+        fileName: storageKey,
         contentType,
         context: 'workspace',
-        preserveKey: true, // Don't add timestamp prefix
-        customKey: storageKey, // Explicitly set the key
-        metadata, // Pass metadata for cloud storage consistency
+        preserveKey: true,
+        customKey: storageKey,
+        metadata,
       })
 
       logger.info(`Upload returned key: ${uploadResult.key}`)
@@ -240,7 +240,7 @@ export async function uploadWorkspaceFile(
         name: uniqueName,
         size: fileBuffer.length,
         type: contentType,
-        url: serveUrl, // Use authenticated serve URL (enforces context)
+        url: serveUrl,
         key: uploadResult.key,
         context: 'workspace',
       }
@@ -323,10 +323,8 @@ export async function registerUploadedWorkspaceFile(params: {
   }
 
   /**
-   * Existence check runs before the quota guard so a retry after a
-   * successful-but-network-dropped register does not (a) fail quota because
-   * the bytes are now counted, or (b) invoke cleanupOrphan on an already
-   * registered object.
+   * Existence check precedes the quota guard so a network-dropped retry doesn't
+   * double-charge quota or orphan-cleanup an already-registered object.
    */
   const existing = await getFileMetadataByKey(key, 'workspace')