perf: improve static builds cache

Signed-off-by: Andres Correa Casablanca <andreu@kindspells.dev>
kindspells · Mar 15, 2024 · 611e9bc · 611e9bc
1 parent b7a9f6e
commit 611e9bc
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 18 deletions.
diff --git a/src/core.mjs b/src/core.mjs
@@ -168,24 +168,31 @@ export const updateStaticPageSriHashes = async (
 				}
 
 				if (src) {
-					/** @type {string | ArrayBuffer | Buffer} */
-					let resourceContent
-					if (src.startsWith('/')) {
-						const resourcePath = resolve(distDir, `.${src}`)
-						resourceContent = await readFile(resourcePath)
-					} else if (src.startsWith('http')) {
-						setCrossorigin = true
-						const resourceResponse = await fetch(src, { method: 'GET' })
-						resourceContent = await resourceResponse.arrayBuffer()
+					const cachedHash = h.perResourceSriHashes[t2].get(src)
+					if (cachedHash) {
+						sriHash = cachedHash
+						h[`ext${t}Hashes`].add(sriHash)
+						pageHashes[t2].add(sriHash)
 					} else {
-						logger.warn(`Unable to process external resource: "${src}"`)
-						continue
-					}
+						/** @type {string | ArrayBuffer | Buffer} */
+						let resourceContent
+						if (src.startsWith('/')) {
+							const resourcePath = resolve(distDir, `.${src}`)
+							resourceContent = await readFile(resourcePath)
+						} else if (src.startsWith('http')) {
+							setCrossorigin = true
+							const resourceResponse = await fetch(src, { method: 'GET' })
+							resourceContent = await resourceResponse.arrayBuffer()
+						} else {
+							logger.warn(`Unable to process external resource: "${src}"`)
+							continue
+						}
 
-					sriHash = generateSRIHash(resourceContent)
-					h[`ext${t}Hashes`].add(sriHash)
-					pageHashes[t2].add(sriHash)
-					h.perResourceSriHashes[t2].set(src, sriHash)
+						sriHash = generateSRIHash(resourceContent)
+						h[`ext${t}Hashes`].add(sriHash)
+						pageHashes[t2].add(sriHash)
+						h.perResourceSriHashes[t2].set(src, sriHash)
+					}
 				}
 			}
 

diff --git a/tests/core.test.mts b/tests/core.test.mts
@@ -362,7 +362,7 @@ describe('updateStaticPageSriHashes', () => {
 				<title>My Test Page</title>
 			</head>
 			<body>
-				<script type="module" src="/core.mjs" integrity="sha256-mBEQGpYqqmUThJOhZ7SIAP3agaHSLAyV/38of2lYhnI="></script>
+				<script type="module" src="/core.mjs" integrity="sha256-etOR/kKV9aCSESe7t5JeBixVQA1DjUU2Zxk13wsPU8M="></script>
 			</body>
 		</html>`
 
@@ -379,7 +379,7 @@ describe('updateStaticPageSriHashes', () => {
 		expect(h.extScriptHashes.size).toBe(1)
 		expect(
 			h.extScriptHashes.has(
-				'sha256-mBEQGpYqqmUThJOhZ7SIAP3agaHSLAyV/38of2lYhnI=',
+				'sha256-etOR/kKV9aCSESe7t5JeBixVQA1DjUU2Zxk13wsPU8M=',
 			),
 		).toBe(true)
 		expect(h.inlineScriptHashes.size).toBe(0)