perf: tweak server function caching

Author: danielroe

server/api/jsr/[...pkg].get.ts

@@ -20,6 +20,7 @@ export default defineCachedEventHandler<Promise<JsrPackageInfo>>(
   },
   {
     maxAge: 60 * 60, // 1 hour
+    swr: true,
     name: 'api-jsr-package',
     getKey: event => getRouterParam(event, 'pkg') ?? '',
   },

server/api/registry/[...pkg].get.ts

@@ -18,7 +18,8 @@ export default defineCachedEventHandler(
     }
   },
   {
-    maxAge: 60 * 5,
+    maxAge: 60 * 60, // 1 hour
+    swr: true,
     getKey: event => getRouterParam(event, 'pkg') ?? '',
   },
 )

server/api/registry/file/[...pkg].get.ts

@@ -175,7 +175,8 @@ export default defineCachedEventHandler(
     }
   },
   {
-    maxAge: 60 * 60, // Cache for 1 hour (files don't change for a given version)
+    // File content for a specific version never changes - cache permanently
+    maxAge: 60 * 60 * 24 * 365, // 1 year
     getKey: event => {
       const pkg = getRouterParam(event, 'pkg') ?? ''
       return `file:v${CACHE_VERSION}:${pkg}`

server/api/registry/files/[...pkg].get.ts

@@ -47,7 +47,8 @@ export default defineCachedEventHandler(
     }
   },
   {
-    maxAge: 60 * 60, // Cache for 1 hour (files don't change for a given version)
+    // Files for a specific version never change - cache permanently
+    maxAge: 60 * 60 * 24 * 365, // 1 year
     getKey: event => {
       const pkg = getRouterParam(event, 'pkg') ?? ''
       return `files:${pkg}`

server/api/registry/install-size/[...pkg].get.ts

@@ -66,7 +66,8 @@ export default defineCachedEventHandler(
     }
   },
   {
-    maxAge: 60 * 60, // Cache for 1 hour
+    maxAge: 60 * 60, // 1 hour
+    swr: true,
     getKey: event => getRouterParam(event, 'pkg') ?? '',
   },
 )

server/api/registry/readme/[...pkg].get.ts

@@ -98,7 +98,8 @@ export default defineCachedEventHandler(
     }
   },
   {
-    maxAge: 60 * 10,
+    maxAge: 60 * 60, // 1 hour
+    swr: true,
     getKey: event => {
       const pkg = getRouterParam(event, 'pkg') ?? ''
       return `readme:${pkg}`

server/utils/install-size.ts

@@ -50,6 +50,7 @@ const fetchPackument = defineCachedFunction(
   },
   {
     maxAge: 60 * 60, // 1 hour
+    swr: true,
     name: 'packument',
     getKey: (name: string) => name,
   },
@@ -263,6 +264,7 @@ export const calculateInstallSize = defineCachedFunction(
   {
     // Cache for 1 hour - dependency resolutions can change with new releases
     maxAge: 60 * 60,
+    swr: true,
     name: 'install-size',
     getKey: (name: string, version: string) => `${name}@${version}`,
   },

server/utils/jsr.ts

@@ -58,8 +58,8 @@ export const fetchJsrPackageInfo = defineCachedFunction(
     }
   },
   {
-    // Cache for 1 hour - JSR info doesn't change often
-    maxAge: 60 * 60,
+    maxAge: 60 * 60 * 24, // 1 day
+    swr: true,
     name: 'jsr-package-info',
     getKey: (name: string) => name,
   },

server/utils/npm.ts

@@ -1,8 +1,7 @@
-import type { Packument, NpmSearchResponse, NpmDownloadCount } from '#shared/types'
+import type { Packument } from '#shared/types'
 import { maxSatisfying, prerelease } from 'semver'
 
 const NPM_REGISTRY = 'https://registry.npmjs.org'
-const NPM_API = 'https://api.npmjs.org'
 
 function encodePackageName(name: string): string {
   if (name.startsWith('@')) {
@@ -18,39 +17,12 @@ export const fetchNpmPackage = defineCachedFunction(
   },
   {
     maxAge: 60 * 5,
+    swr: true,
     name: 'npm-package',
     getKey: (name: string) => name,
   },
 )
 
-export const fetchNpmSearch = defineCachedFunction(
-  async (query: string, size: number = 20, from: number = 0): Promise<NpmSearchResponse> => {
-    const params = new URLSearchParams({
-      text: query,
-      size: String(size),
-      from: String(from),
-    })
-    return await $fetch<NpmSearchResponse>(`${NPM_REGISTRY}/-/v1/search?${params.toString()}`)
-  },
-  {
-    maxAge: 60 * 2,
-    name: 'npm-search',
-    getKey: (query: string, size: number, from: number) => `${query}:${size}:${from}`,
-  },
-)
-
-export const fetchNpmDownloads = defineCachedFunction(
-  async (name: string, period: string = 'last-week'): Promise<NpmDownloadCount> => {
-    const encodedName = encodePackageName(name)
-    return await $fetch<NpmDownloadCount>(`${NPM_API}/downloads/point/${period}/${encodedName}`)
-  },
-  {
-    maxAge: 60 * 60,
-    name: 'npm-downloads',
-    getKey: (name: string, period: string) => `${name}:${period}`,
-  },
-)
-
 /**
  * Check if a version constraint explicitly includes a prerelease tag.
  * e.g., "^1.0.0-alpha" or ">=2.0.0-beta.1" include prereleases