fix(files): reflect the content-version guard outcome in the dimensions response

The route returned success:true even when updateWorkspaceFileDimensions matched
0 rows (the CAS rejected a write whose measured key no longer matches the row).
Return success:<whether a row was written> and widen the contract response to
{ success: boolean }. Not an error path — the client's next measurement persists
once its file list has the new key; this just stops the API claiming a persist
that did not happen.

Author: waleedlatif1

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

@@ -56,6 +56,13 @@ describe('PATCH /api/workspaces/[id]/files/[fileId]/dimensions', () => {
     expect(mockUpdateWorkspaceFileDimensions).toHaveBeenCalledOnce()
   })
 
+  it('reports success:false when the content-version guard rejects the write (key changed)', async () => {
+    mockUpdateWorkspaceFileDimensions.mockResolvedValue(false)
+    const res = await PATCH(buildRequest({ key: KEY, width: 10, height: 20 }), routeContext)
+    expect(res.status).toBe(200)
+    expect(await res.json()).toEqual({ success: false })
+  })
+
   it('rejects an unauthenticated caller before touching the DB', async () => {
     authMockFns.mockGetSession.mockResolvedValue(null)
     const res = await PATCH(buildRequest({ key: KEY, width: 10, height: 10 }), routeContext)

apps/sim/app/api/workspaces/[id]/files/[fileId]/dimensions/route.ts

@@ -36,8 +36,15 @@ export const PATCH = withRouteHandler(
     }
 
     try {
-      await updateWorkspaceFileDimensions(workspaceId, fileId, { key, width, height })
-      return NextResponse.json({ success: true as const })
+      // `written` is false when the content-version guard rejected the write (the row's storage key no
+      // longer matches the key the client measured — the content was replaced since). That is not an
+      // error; the client's next measurement, once its file list has the new key, persists correctly.
+      const written = await updateWorkspaceFileDimensions(workspaceId, fileId, {
+        key,
+        width,
+        height,
+      })
+      return NextResponse.json({ success: written })
     } catch (error) {
       logger.error('Failed to backfill workspace file dimensions', {
         workspaceId,

apps/sim/lib/api/contracts/workspace-files.ts

@@ -137,7 +137,9 @@ export const updateWorkspaceFileDimensionsContract = defineRouteContract({
   body: updateWorkspaceFileDimensionsBodySchema,
   response: {
     mode: 'json',
-    schema: z.object({ success: z.literal(true) }),
+    // `success` reflects whether the row was actually written: false when the content-version guard
+    // rejected the write (the storage key changed since the client measured), not just on error.
+    schema: z.object({ success: z.boolean() }),
   },
 })