fix: patch banner filename error

KUN1007 · KUN1007 · commit 9419ea286a5d · 2025-11-11T00:38:00.000+08:00
diff --git a/migration/sync-ts/_syncPatchesToS3Dry.ts b/migration/sync-ts/_syncPatchesToS3Dry.ts
@@ -3,6 +3,7 @@ import { readdir, mkdir, copyFile, writeFile } from 'fs/promises'
 import { vndbGetVnById } from './api/vndb'
 import { parsePatchFileName } from './vn-sync/parse'
 import sharp from 'sharp'
+import { sanitizeFileName } from '../../utils/sanitizeFileName'
 
 async function ensureDir(dir: string) {
   await mkdir(dir, { recursive: true })
@@ -35,33 +36,29 @@ export async function syncPatchesToS3Dry(
         results.push({ file: name, ok: false, error: 'Unrecognized filename' })
         continue
       }
-      // Copy patch file for inspection
-      const dstPatchPath = path.posix.join(dstPatchDir, name)
+      // Copy patch file for inspection (sanitized file name)
+      const sanitized = sanitizeFileName(name)
+      const dstPatchPath = path.posix.join(dstPatchDir, sanitized)
       await copyFile(filePath, dstPatchPath)
 
-      // Fetch VN and write banner webp files
+      // Fetch VN and write banner avif files
       const vn = await vndbGetVnById(parsed.vndbId)
       const shotUrl = pickSfwScreenshotUrlLocal(vn)
       if (shotUrl) {
         const res = await fetch(shotUrl)
         const ab = await res.arrayBuffer()
         const banner = await sharp(ab)
           .resize(1920, 1080, { fit: 'inside', withoutEnlargement: true })
-          .webp({ quality: 70 })
+          .avif({ quality: 60 })
           .toBuffer()
         const mini = await sharp(ab)
           .resize(460, 259, { fit: 'inside', withoutEnlargement: true })
-          .webp({ quality: 70 })
+          .avif({ quality: 60 })
           .toBuffer()
-        const base = `${parsed.vndbId}`
-        await writeFile(
-          path.posix.join(dstBannerDir, `${base}-banner.webp`),
-          banner
-        )
-        await writeFile(
-          path.posix.join(dstBannerDir, `${base}-banner-mini.webp`),
-          mini
-        )
+        const bannerDir = path.posix.join(dstBannerDir, parsed.vndbId)
+        await ensureDir(bannerDir)
+        await writeFile(path.posix.join(bannerDir, `banner.avif`), banner)
+        await writeFile(path.posix.join(bannerDir, `banner-mini.avif`), mini)
       }
 
       results.push({ file: name, ok: true })
diff --git a/migration/sync-ts/syncPatchesToS3.ts b/migration/sync-ts/syncPatchesToS3.ts
@@ -1,19 +1,46 @@
 import path from 'path'
 import { readdir } from 'fs/promises'
-import { processOnePatchFile } from './vn-sync/index'
+import { prisma } from './db/prisma'
+import { parsePatchFileName } from './vn-sync/parse'
+import {
+  createPatchIfMissing,
+  createPatchResourceForFile
+} from './vn-sync/index'
 
 export async function syncPatchesToS3(dir = 'migration/sync-ts/patch') {
   const files = await readdir(dir)
   const results: Array<{ file: string; ok: boolean; error?: string }> = []
   for (const name of files) {
     const filePath = path.posix.join(dir, name)
     try {
-      const res = await processOnePatchFile(filePath)
-      if (typeof res === 'string') {
-        results.push({ file: name, ok: false, error: res })
-      } else {
-        results.push({ file: name, ok: true })
+      const parsed = parsePatchFileName(filePath)
+      if (!parsed) {
+        results.push({ file: name, ok: false, error: 'Unrecognized filename' })
+        continue
       }
+
+      const patchId = await createPatchIfMissing(parsed)
+      const resource = await createPatchResourceForFile(patchId, parsed)
+
+      const patch = await prisma.patch.findUnique({
+        where: { id: patchId },
+        select: { name_ja_jp: true, banner: true }
+      })
+
+      const jaName = patch?.name_ja_jp || ''
+      const banner = patch?.banner || ''
+      const bannerMini = banner
+        ? banner.replace(/\/banner\.(avif|webp)$/i, '/banner-mini.$1')
+        : ''
+      const fileUrl = (resource as any)?.content || ''
+
+      // Print requested details
+      console.log(`Patch: ${jaName}`)
+      if (banner) console.log(`banner: ${banner}`)
+      if (bannerMini) console.log(`banner-mini: ${bannerMini}`)
+      if (fileUrl) console.log(`file: ${fileUrl}`)
+
+      results.push({ file: name, ok: true })
     } catch (e: any) {
       results.push({ file: name, ok: false, error: e?.message || String(e) })
     }
diff --git a/migration/sync-ts/vn-sync/index.ts b/migration/sync-ts/vn-sync/index.ts
@@ -1,10 +1,10 @@
 import { prisma } from '../db/prisma'
 import { vndbGetVnById } from '../api/vndb'
 import { parsePatchFileName, type ParsedPatchFileName } from './parse'
-import { uploadBannerForPatchWebp, uploadPatchFileToS3 } from './upload'
+import { uploadBannerForPatch, uploadPatchFileToS3 } from './upload'
 import { generateFileHash } from '../../../app/api/upload/calculateFileStreamHash'
 import { loadNoteTemplate, renderNoteFromTemplate } from './note'
-import sharp from 'sharp'
+import { sanitizeFileName } from '../../../utils/sanitizeFileName'
 
 export async function pickSfwScreenshotUrl(vn: any): Promise<string | null> {
   if (!vn?.screenshots?.length) return null
@@ -41,7 +41,7 @@ export async function createPatchIfMissing(parsed: ParsedPatchFileName) {
   const nameEn = (vn as any)?.title || ''
   const nameJa = parsed.gameName || deriveJaTitle(vn)
   const releasedRaw = String((vn as any)?.released || '').trim()
-  const released = releasedRaw ? formatDateYMD(releasedRaw) : 'unknown'
+  const released = releasedRaw ? formatDateYYMMDD(releasedRaw) : 'unknown'
 
   const patch = await prisma.patch.create({
     data: {
@@ -66,7 +66,7 @@ export async function createPatchIfMissing(parsed: ParsedPatchFileName) {
     if (screenshotUrl) {
       const res = await fetch(screenshotUrl)
       const arrayBuffer = await res.arrayBuffer()
-      const uploadRes = await uploadBannerForPatchWebp(patch.id, arrayBuffer)
+      const uploadRes = await uploadBannerForPatch(patch.id, arrayBuffer)
       if (typeof uploadRes !== 'string') {
         await prisma.patch.update({
           where: { id: patch.id },
@@ -102,11 +102,11 @@ export async function createPatchResourceForFile(
     gameName: parsed.gameName,
     groupName: parsed.groupName,
     language: parsed.language,
-    publishDate: formatDateYMD(parsed.publishDate),
-    startDate: formatDateYMD(parsed.startDate),
+    publishDate: formatDateYYMMDD(parsed.publishDate),
+    startDate: formatDateYYMMDD(parsed.startDate),
     vndbId: parsed.vndbId,
     platform: parsed.platform,
-    fileName: parsed.fileName
+    fileName: sanitizeFileName(parsed.fileName)
   })
 
   const resource = await prisma.patch_resource.create({
@@ -159,14 +159,14 @@ export async function processOnePatchFile(filePath: string) {
   return await createPatchResourceForFile(patchId, parsed)
 }
 
-export function formatDateYMD(s: string) {
+export function formatDateYYMMDD(s: string) {
   const t = String(s || '').replace(/[^0-9]/g, '')
   if (t.length === 8) {
-    return `${t.slice(0, 4)}-${t.slice(4, 6)}-${t.slice(6, 8)}`
+    return `${t.slice(2, 4)}-${t.slice(4, 6)}-${t.slice(6, 8)}`
   }
   if (t.length === 6) {
-    return `${t.slice(0, 4)}-${t.slice(4, 6)}`
+    return `${t.slice(2, 4)}-${t.slice(4, 6)}`
   }
-  if (t.length === 4) return t
+  if (t.length === 4) return t.slice(2, 4)
   return s
 }
diff --git a/migration/sync-ts/vn-sync/upload.ts b/migration/sync-ts/vn-sync/upload.ts
@@ -2,32 +2,13 @@ import { stat } from 'fs/promises'
 import { uploadSmallFileToS3 } from '../../../lib/s3/uploadSmallFileToS3'
 import { uploadLargeFileToS3 } from '../../../lib/s3/uploadLargeFileToS3'
 import { MAX_SMALL_FILE_SIZE } from '../../../config/upload'
-import sharp from 'sharp'
-import { uploadImageToS3 } from '../../../lib/s3/uploadImageToS3'
-import { checkBufferSize } from '../../../app/api/utils/checkBufferSize'
-import { MAX_SIZE, COMPRESS_QUALITY } from '../../../app/api/utils/constants'
+import { uploadPatchBanner as _uploadPatchBanner } from '../../../app/api/utils/uploadPatchBanner'
+import { sanitizeFileName } from '../../../utils/sanitizeFileName'
 
-export async function uploadBannerForPatchWebp(
-  patchId: number,
-  image: ArrayBuffer
-) {
-  const banner = await sharp(image)
-    .resize(1920, 1080, { fit: 'inside', withoutEnlargement: true })
-    .webp({ quality: COMPRESS_QUALITY })
-    .toBuffer()
-  const miniBanner = await sharp(image)
-    .resize(460, 259, { fit: 'inside', withoutEnlargement: true })
-    .webp({ quality: COMPRESS_QUALITY })
-    .toBuffer()
-
-  if (!checkBufferSize(miniBanner, MAX_SIZE)) {
-    return '图片太大'
-  }
-
-  const bucketKey = `patch/${patchId}/banner`
-  await uploadImageToS3(`${bucketKey}/banner.webp`, banner)
-  await uploadImageToS3(`${bucketKey}/banner-mini.webp`, miniBanner)
-  const link = `${process.env.KUN_VISUAL_NOVEL_IMAGE_BED_URL}/patch/${patchId}/banner/banner.webp`
+export async function uploadBannerForPatch(patchId: number, image: ArrayBuffer) {
+  const res = await _uploadPatchBanner(image, patchId)
+  if (typeof res === 'string') return res
+  const link = `${process.env.KUN_VISUAL_NOVEL_IMAGE_BED_URL}/patch/${patchId}/banner/banner.avif`
   return { link }
 }
 
@@ -36,7 +17,8 @@ export async function uploadPatchFileToS3(
   hash: string,
   filePath: string
 ) {
-  const fileName = filePath.split(/[\\/]/).pop()!
+  const rawName = filePath.split(/[\\/]/).pop()!
+  const fileName = sanitizeFileName(rawName)
   const s3Key = `patch/${patchId}/${hash}/${fileName}`
   const st = await stat(filePath)
   if (st.size < MAX_SMALL_FILE_SIZE) {
