fix: astro-purgecss is modifying files keeping old hash (#564)

* fix: astro-purgecss is modifying files keeping the hash fix #450 * x * x * Address the-dijkstra comments * missing utils on build command
codiume · Aug 3, 2024 · 26df36e · 26df36e
1 parent 8f5ed26
commit 26df36e
Show file tree

Hide file tree

Showing 3 changed files with 79 additions and 16 deletions.
diff --git a/packages/astro-purgecss/package.json b/packages/astro-purgecss/package.json
@@ -3,7 +3,7 @@
   "description": "Remove unused CSS rules from your final Astro bundle",
   "version": "4.1.1",
   "scripts": {
-    "build": "astro-build --src src/index.ts",
+    "build": "astro-build --src src/index.ts src/utils.ts",
     "typecheck": "tsc --declaration --emitDeclarationOnly"
   },
   "type": "module",

diff --git a/packages/astro-purgecss/src/index.ts b/packages/astro-purgecss/src/index.ts
@@ -1,21 +1,13 @@
 import type { AstroIntegration } from 'astro';
-import { PurgeCSS, type UserDefinedOptions } from 'purgecss';
-import { writeFile } from 'node:fs/promises';
+import { createHash } from 'crypto';
+import { rename, writeFile } from 'node:fs/promises';
+import path from 'node:path';
 import { fileURLToPath } from 'node:url';
+import { PurgeCSS, type UserDefinedOptions } from 'purgecss';
+import { handleWindowsPath, replaceStringInDirectory } from './utils';
 
 export interface PurgeCSSOptions extends Partial<UserDefinedOptions> {}
 
-function handleWindowsPath(outputPath: string): string {
-  if (process.platform !== 'win32') return outputPath;
-
-  if (outputPath.endsWith('\\')) {
-    outputPath = outputPath.substring(0, outputPath.length - 1);
-  }
-  outputPath = outputPath.replaceAll('\\', '/');
-
-  return outputPath;
-}
-
 export default function (options: PurgeCSSOptions = {}): AstroIntegration {
   return {
     name: 'astro-purgecss',
@@ -30,12 +22,33 @@ export default function (options: PurgeCSSOptions = {}): AstroIntegration {
             `${outDir}/**/*.html`,
             `${outDir}/**/*.js`,
             ...(options.content || [])
-          ],
+          ]
         });
         await Promise.all(
           purged
             .filter(({ file }) => file?.endsWith('.css'))
-            .map(async ({ css, file }) => await writeFile(file!, css))
+            .map(async ({ css, file }) => {
+              if (!file) return;
+
+              await writeFile(file, css);
+
+              // Get content hash
+              const hash = await createHash('sha256')
+                .update(css)
+                .digest('hex')
+                .substring(0, 7);
+
+              // Rename file
+              const newPath = file.slice(0, -12) + hash + '.css';
+              await rename(file, newPath);
+
+              // Replace old name references by newPath
+              await replaceStringInDirectory(
+                outDir + '../',
+                path.basename(file),
+                path.basename(newPath)
+              );
+            })
         );
       }
     }

diff --git a/packages/astro-purgecss/src/utils.ts b/packages/astro-purgecss/src/utils.ts
@@ -0,0 +1,50 @@
+import { readdir, readFile, writeFile } from 'node:fs/promises';
+import path from 'node:path';
+
+export function handleWindowsPath(outputPath: string): string {
+  if (process.platform !== 'win32') return outputPath;
+
+  if (outputPath.endsWith('\\')) {
+    outputPath = outputPath.substring(0, outputPath.length - 1);
+  }
+  outputPath = outputPath.replaceAll('\\', '/');
+
+  return outputPath;
+}
+
+export async function replaceStringInFile(
+  filePath: string,
+  searchValue: string,
+  replaceValue: string
+) {
+  try {
+    const fileContent = await readFile(filePath, 'utf8');
+    if (fileContent.includes(searchValue)) {
+      const re = new RegExp(searchValue, 'g');
+      const newContent = fileContent.replace(re, replaceValue);
+      await writeFile(filePath, newContent, 'utf8');
+    }
+  } catch (err) {
+    console.error(`Error processing file ${filePath}: ${err}`);
+  }
+}
+
+export async function replaceStringInDirectory(
+  directory: string,
+  searchValue: string,
+  replaceValue: string
+) {
+  try {
+    const files = await readdir(directory, { withFileTypes: true });
+    for (const file of files) {
+      const fullPath = path.join(directory, file.name);
+      if (file.isDirectory()) {
+        await replaceStringInDirectory(fullPath, searchValue, replaceValue);
+      } else if (file.isFile()) {
+        await replaceStringInFile(fullPath, searchValue, replaceValue);
+      }
+    }
+  } catch (err) {
+    console.error(`Error processing directory ${directory}: ${err}`);
+  }
+}