Port dynamic-assets.js to TypeScript (#51436)

github · Jun 26, 2024 · 53b5f97 · 53b5f97
1 parent 0e2fe51
commit 53b5f97
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 9 deletions.
diff --git a/src/assets/middleware/dynamic-assets.js → src/assets/middleware/dynamic-assets.ts b/src/assets/middleware/dynamic-assets.js → src/assets/middleware/dynamic-assets.ts
@@ -1,12 +1,14 @@
 import fs from 'fs/promises'
 
+import type { Response, NextFunction } from 'express'
 import sharp from 'sharp'
 
-import { assetCacheControl, defaultCacheControl } from '#src/frame/middleware/cache-control.js'
+import type { ExtendedRequest } from '@/types'
+import { assetCacheControl, defaultCacheControl } from '@/frame/middleware/cache-control.js'
 import {
   setFastlySurrogateKey,
   SURROGATE_ENUMS,
-} from '#src/frame/middleware/set-fastly-surrogate-key.js'
+} from '@/frame/middleware/set-fastly-surrogate-key.js'
 
 /**
  * This is the indicator that is a virtual part of the URL.
@@ -37,7 +39,11 @@ const maxWidthPathPartRegex = /\/mw-(\d+)\//
  */
 const VALID_MAX_WIDTHS = [1440, 1000]
 
-export default async function dynamicAssets(req, res, next) {
+export default async function dynamicAssets(
+  req: ExtendedRequest,
+  res: Response,
+  next: NextFunction,
+) {
   if (!req.url.startsWith('/assets/')) return next()
 
   if (!(req.method === 'GET' || req.method === 'HEAD')) {
@@ -88,6 +94,7 @@ export default async function dynamicAssets(req, res, next) {
 
       if (maxWidth) {
         const { width } = await image.metadata()
+        if (width === undefined) throw new Error('image metadata does not have a width')
         if (width > maxWidth) {
           image.resize({ width: maxWidth })
         }
@@ -140,7 +147,7 @@ export default async function dynamicAssets(req, res, next) {
       assetCacheControl(res)
       return res.type('image/webp').send(buffer)
     } catch (error) {
-      if (error.code !== 'ENOENT') {
+      if (error instanceof Error && (error as any).code !== 'ENOENT') {
         throw error
       }
     }
@@ -166,7 +173,7 @@ export default async function dynamicAssets(req, res, next) {
   res.status(404).type('text/plain').send('Asset not found')
 }
 
-function deconstructImageURL(url) {
+function deconstructImageURL(url: string) {
   let error
   let maxWidth
   const match = url.match(maxWidthPathPartRegex)

diff --git a/src/assets/tests/dynamic-assets.js → src/assets/tests/dynamic-assets.ts b/src/assets/tests/dynamic-assets.js → src/assets/tests/dynamic-assets.ts
@@ -3,8 +3,8 @@ import { describe, expect, test, vi } from 'vitest'
 import sharp from 'sharp'
 import { fileTypeFromBuffer } from 'file-type'
 
-import { SURROGATE_ENUMS } from '#src/frame/middleware/set-fastly-surrogate-key.js'
-import { get, head } from '#src/tests/helpers/e2etest.js'
+import { SURROGATE_ENUMS } from '@/frame/middleware/set-fastly-surrogate-key.js'
+import { get, head } from '@/tests/helpers/e2etest.js'
 
 describe('dynamic assets', () => {
   vi.setConfig({ testTimeout: 3 * 60 * 1000 })
@@ -15,7 +15,10 @@ describe('dynamic assets', () => {
     })
     expect(res.statusCode).toBe(200)
     expect(res.headers['content-type']).toBe('image/webp')
-    const { mime } = await fileTypeFromBuffer(res.body)
+
+    const fileTypeResult = await fileTypeFromBuffer(res.body)
+    if (!fileTypeResult) throw new Error('fileTypeFromBuffer failed')
+    const { mime } = fileTypeResult
     expect(mime).toBe('image/webp')
   })
 

diff --git a/src/frame/middleware/index.ts b/src/frame/middleware/index.ts
@@ -60,7 +60,7 @@ import fastHead from './fast-head'
 import fastlyCacheTest from './fastly-cache-test'
 import trailingSlashes from './trailing-slashes'
 import mockVaPortal from './mock-va-portal.js'
-import dynamicAssets from '@/assets/middleware/dynamic-assets.js'
+import dynamicAssets from '@/assets/middleware/dynamic-assets'
 import contextualizeSearch from '@/search/middleware/contextualize.js'
 import shielding from '@/shielding/middleware'
 import tracking from '@/tracking/middleware'