fix: await dataset preview access links

Author: xqvvu

packages/service/common/s3/uploadPolicy/service.ts

@@ -376,7 +376,7 @@ export const resolveUploadFile = ({
 
     const resolvedExtension = explicitExtension
       ? explicitExtension
-      : matchedAllowedExtension || evidence.officeExtension || evidence.detectedExtension;
+      : matchedAllowedExtension || evidence.officeExtension || evidence.detectedExtension || '';
     return {
       filename: resolveAcceptedFilename({ filename, extension: resolvedExtension }),
       contentType: evidence.detectedMime,

projects/app/src/pages/api/core/dataset/file/getRawTextPreviewChunks.ts

@@ -54,13 +54,15 @@ async function handler(
     maxChunks: maxPreviewChunkCount
   });
 
-  const chunksWithJWT = chunks.slice(0, 10).map((chunk) => ({
-    q: replaceS3KeyToPreviewUrl(chunk.q, addDays(new Date(), 1)),
-    a: replaceS3KeyToPreviewUrl(chunk.a, addDays(new Date(), 1))
-  }));
+  const chunksWithPreviewUrls = await Promise.all(
+    chunks.slice(0, 10).map(async (chunk) => ({
+      q: await replaceS3KeyToPreviewUrl(chunk.q, addDays(new Date(), 1)),
+      a: await replaceS3KeyToPreviewUrl(chunk.a, addDays(new Date(), 1))
+    }))
+  );
 
   return GetPreviewChunksResponseSchema.parse({
-    chunks: chunksWithJWT,
+    chunks: chunksWithPreviewUrls,
     total: chunks.length
   });
 }

projects/app/test/pages/api/core/dataset/file/getRawTextPreviewChunks.test.ts

@@ -4,7 +4,7 @@ import { beforeEach, describe, expect, it, vi } from 'vitest';
 const mocks = vi.hoisted(() => ({
   authDataset: vi.fn(),
   rawText2Chunks: vi.fn(),
-  replaceS3KeyToPreviewUrl: vi.fn((value: string) => value)
+  replaceS3KeyToPreviewUrl: vi.fn(async (value: string) => value)
 }));
 
 vi.mock('@/service/middleware/entry', () => ({