chore: increase the cache for mutable manifest

apps/api/src/routes/hls/hls.ts

@@ -41,6 +41,15 @@ function contentTypeFor(path: string): string {
   return HLS_CONTENT_TYPES[ext] ?? 'application/octet-stream';
 }
 
+/**
+ * Only `master.m3u8` is mutable — it gets patched when a p1080 rendition is
+ * added later, so it stays short-lived. Variant/media playlists and segments
+ * are write-once per assetId/path and never change, so cache them immutably.
+ */
+function cacheControlFor(relativePath: string): string {
+  return relativePath === 'master.m3u8' ? 'private, max-age=10' : 'private, max-age=31536000, immutable';
+}
+
 export const hlsRouter = new Hono()
   .options('/:assetId/*', (c) => {
     const origin = c.req.header('origin');
@@ -144,7 +153,7 @@ export const hlsRouter = new Hono()
         const head = await client.send(new HeadObjectCommand({ Bucket: config.bucketVideos, Key: objectKey }));
         const headHeaders: Record<string, string> = {
           'Content-Type': contentTypeFor(relative),
-          'Cache-Control': relative.endsWith('.m3u8') ? 'private, max-age=10' : 'private, max-age=31536000, immutable'
+          'Cache-Control': cacheControlFor(relative)
         };
         if (head.ContentLength != null) headHeaders['Content-Length'] = String(head.ContentLength);
         return withCors(null, 200, headHeaders);
@@ -157,7 +166,7 @@ export const hlsRouter = new Hono()
 
       const headers: Record<string, string> = {
         'Content-Type': contentTypeFor(relative),
-        'Cache-Control': relative.endsWith('.m3u8') ? 'private, max-age=10' : 'private, max-age=31536000, immutable'
+        'Cache-Control': cacheControlFor(relative)
       };
       if (result.ContentLength != null) headers['Content-Length'] = String(result.ContentLength);
 

apps/tenant-router/src/index.ts

@@ -187,9 +187,11 @@ async function handleHlsRequest(request: Request, env: Env): Promise<Response> {
 
   const headers = new Headers();
   headers.set('Content-Type', contentTypeFor(relativePath));
-  // Playlists must not be cached aggressively (small + may regenerate); segments are immutable per assetId/path.
+  // Only `master.m3u8` is mutable — it gets patched when a p1080 rendition is
-  const isManifest = relativePath.endsWith('.m3u8');
+  // added later, so it stays short-lived. Variant/media playlists and segments
-  headers.set('Cache-Control', isManifest ? 'private, max-age=10' : 'private, max-age=31536000, immutable');
+  // are write-once per assetId/path and never change, so cache them immutably.
+  const isMutableManifest = relativePath === 'master.m3u8';
+  headers.set('Cache-Control', isMutableManifest ? 'private, max-age=10' : 'private, max-age=31536000, immutable');
   if ('writeHttpMetadata' in object) {
     object.writeHttpMetadata(headers);
   }