@zaininnari/postcss-cachebuster 0.3.0 → 0.4.1

package/.github/workflows/lint.yml

@@ -0,0 +1,28 @@
+name: Lint
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [20, 22, 24]
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Use Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run tests
+        run: npm run lint

package/.github/workflows/test-latest.yml

@@ -10,7 +10,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18, 20, 22, 24]
+        node-version: [20, 22, 24]
 
     steps:
       - uses: actions/checkout@v4

package/.github/workflows/test.yml

@@ -10,7 +10,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18, 20, 22, 24]
+        node-version: [20, 22, 24]
 
     steps:
       - uses: actions/checkout@v4

package/.idea/jsLibraryMappings.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project version="4">
+  <component name="JavaScriptLibraryMappings">
+    <includedPredefinedLibrary name="Node.js Core" />
+  </component>
+</project>

package/CHANGELOG.md

@@ -1,3 +1,27 @@
+# Changelog
+
+## 0.4.1 (2026-01-02)
+- Add ESLint
+  - @eslint/js recommended
+  - eslint-plugin-n recommended
+
+## 0.4.0 (2025-12-30)
+- update dependencies
+- update support node version 20+
+
+## 0.3.0 (2025-12-29)
+- update npm package
+- update support node version 18+
+
+## Before fork
+
+## 0.1.6 (2018-12-07)
+
+## 0.1.5 (2018-12-07)
+
+## 0.1.4 (2017-01-07)
+Fix multiple css imports in single css file.
+
 ## 0.1.3 (2016-05-27)
 
 - `type` can be defined as function. 

package/eslint.config.js

@@ -0,0 +1,22 @@
+import js from "@eslint/js";
+import globals from "globals";
+import n from "eslint-plugin-n";
+import {defineConfig} from "eslint/config";
+
+export default defineConfig([
+    {
+        files: ["**/*.{js,mjs,cjs}"],
+        languageOptions: {
+            ecmaVersion: "latest",
+            sourceType: "module",
+            globals: {
+                ...globals.node,
+                ...globals.mocha,
+                myCustomGlobal: "readonly",
+            },
+        },
+        plugins: {js, n},
+        extends: ["js/recommended", "n/recommended"],
+        ignores: ['eslint.config.js'],
+    },
+]);

package/index.mjs

@@ -1,4 +1,3 @@
-import url from 'url';
 import fs from 'fs';
 import crypto from 'crypto';
 import chalk from 'chalk';
@@ -7,9 +6,18 @@ import path from 'canonical-path';
 
 const checksums = {};
 
+/**
+ * parsed
+ * @typedef {Object} parsed
+ * @property {URL} u
+ * @property {boolean} isAbsolute
+ * @property {boolean} isRootRelativeButNotProtocolRelative
+ * @property {string} originalUrl
+ */
+
 const plugin = (opts = {}) => {
   const pattern = /url\((['"])?([^'")]+)(['"])?\)/g;
-  let supportedProps = [
+  const supportedPropsDefault = [
     'background',
     'background-image',
     'border-image',
@@ -22,8 +30,9 @@ const plugin = (opts = {}) => {
   opts.type = opts.type || 'mtime';
   opts.paramName = opts.paramName || 'v';
   opts.hashAlgorithm = opts.hashAlgorithm || 'md5';
-  supportedProps = opts.supportedProps || supportedProps;
-  supportedProps = supportedProps.concat(opts.additionalProps || []);
+  opts.supportedProps = opts.supportedProps || supportedPropsDefault;
+  opts.additionalProps = opts.additionalProps || [];
+  const supportedProps = opts.supportedProps.concat(opts.additionalProps);
 
   function createCachebuster(assetPath, origPath, type) {
     let cachebuster;
@@ -55,10 +64,17 @@ const plugin = (opts = {}) => {
     return cachebuster;
   }
 
-  function resolveUrl(assetUrl, file, imagesPath) {
+  /**
+   * @param assetUrl {URL}
+   * @param file {string}
+   * @param imagesPath {string}
+   * @param isRootRelativeButNotProtocolRelative {boolean}
+   * @returns {string}
+   */
+  function resolveUrl(assetUrl, file, imagesPath, isRootRelativeButNotProtocolRelative) {
     let assetPath = decodeURI(assetUrl.pathname);
 
-    if (/^\//.test(assetUrl.pathname)) {
+    if (isRootRelativeButNotProtocolRelative) {
       assetPath = path.join(imagesPath, assetPath);
     } else {
       assetPath = path.join(opts.cssPath || path.dirname(file), assetPath);
@@ -66,14 +82,22 @@ const plugin = (opts = {}) => {
     return assetPath;
   }
 
-  function updateAssetUrl(assetUrl, inputFile) {
-    const assetPath = resolveUrl(assetUrl, inputFile, opts.imagesPath);
+  /**
+   * @param assetUrl {URL}
+   * @param inputFile {string}
+   * @param isRootRelativeButNotProtocolRelative {boolean}
+   */
+  function updateAssetUrl(assetUrl, inputFile, isRootRelativeButNotProtocolRelative) {
+    const assetPath = resolveUrl(assetUrl, inputFile, opts.imagesPath, isRootRelativeButNotProtocolRelative);
 
     // complete url with cachebuster
-    const cachebuster = createCachebuster(assetPath, assetUrl.pathname, opts.type);
+    const originPath = isRootRelativeButNotProtocolRelative ? assetUrl.pathname : assetUrl.pathname.substring(1);
+    const cachebuster = createCachebuster(assetPath, originPath, opts.type);
     if (!cachebuster) {
       return;
-    } else if (typeof opts.type === 'function') {
+    }
+
+    if (typeof opts.type === 'function') {
       assetUrl.pathname = cachebuster;
     } else if (assetUrl.search && assetUrl.search.length > 1) {
       assetUrl.search = assetUrl.search + '&' + opts.paramName + cachebuster;
@@ -82,6 +106,39 @@ const plugin = (opts = {}) => {
     }
   }
 
+  const DUMMY_BASE = 'http://localhost';
+
+  /**
+   * @param input {string}
+   * @returns {parsed}
+   */
+  function parseUrlPreserveRelative(input) {
+    const isAbsolute = /^[a-zA-Z][a-zA-Z\d+\-.]*:/.test(input);
+    const isRootRelativeButNotProtocolRelative = /^\/(?!\/)/.test(input);
+    const u = isAbsolute ? new URL(input) : new URL(input, DUMMY_BASE);
+    return {u, isAbsolute, isRootRelativeButNotProtocolRelative, originalUrl: input};
+  }
+
+  /**
+   * @param parsed {parsed}
+   * @returns {string|string}
+   */
+  function formatUrlPreserveRelative(parsed) {
+    const u = parsed.u;
+    const isAbsolute = parsed.isAbsolute;
+    const isRootRelativeButNotProtocolRelative = parsed.isRootRelativeButNotProtocolRelative;
+
+    if (isAbsolute) {
+      return u.toString();
+    }
+    const url = `${u.pathname}${u.search}${u.hash}`;
+    if (!isRootRelativeButNotProtocolRelative) {
+      // remove start slash
+      return  url.substring(1);
+    }
+    return url;
+  }
+
   return {
     postcssPlugin: 'postcss-cachebuster',
     Once(root) {
@@ -95,10 +152,10 @@ const plugin = (opts = {}) => {
         const quote = results[1] || '"';
         const originalUrl = results[2];
 
-        const assetUrl = url.parse(originalUrl);
-        updateAssetUrl(assetUrl, inputFile);
+        const parsed = parseUrlPreserveRelative(originalUrl);
+        updateAssetUrl(parsed.u, inputFile, parsed.isRootRelativeButNotProtocolRelative, parsed.originalUrl);
 
-        atrule.params = 'url(' + quote + url.format(assetUrl) + quote + ')';
+        atrule.params = 'url(' + quote + formatUrlPreserveRelative(parsed) + quote + ')';
       });
 
       root.walkDecls(function walkThroughtDeclarations(declaration) {
@@ -110,20 +167,22 @@ const plugin = (opts = {}) => {
         declaration.value = declaration.value.replace(pattern, function (match, quote, originalUrl) {
           quote = quote || '"';
 
-          const assetUrl = url.parse(originalUrl);
+          const parsed = parseUrlPreserveRelative(originalUrl);
+          const assetUrl = parsed.u;
 
           // only locals
           if (
-            assetUrl.host ||
+            assetUrl.toString().indexOf(DUMMY_BASE) !== 0 ||
+            parsed.isAbsolute ||
             assetUrl.pathname.indexOf('//') === 0 ||
             assetUrl.pathname.indexOf(';base64') !== -1
           ) {
             return match;
           }
 
-          updateAssetUrl(assetUrl, inputFile);
+          updateAssetUrl(assetUrl, inputFile, parsed.isRootRelativeButNotProtocolRelative);
 
-          return 'url(' + quote + url.format(assetUrl) + quote + ')';
+          return 'url(' + quote + formatUrlPreserveRelative(parsed) + quote + ')';
         });
       });
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zaininnari/postcss-cachebuster",
-  "version": "0.3.0",
+  "version": "0.4.1",
   "description": "Fork of postcss-cachebuster. Cachebusting all local files in css",
   "keywords": [
     "postcss",
@@ -38,7 +38,11 @@
     "path": "^0.12.7"
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.2",
     "chai": "^6.2.2",
+    "eslint": "^9.39.2",
+    "eslint-plugin-n": "^17.23.1",
+    "globals": "^16.5.0",
     "gulp": "^5.0.1",
     "gulp-mocha": "^10.0.1",
     "gulp-postcss": "^10.0.0",
@@ -46,8 +50,11 @@
     "sinon": "^21.0.1",
     "sinon-chai": "^4.0.1"
   },
+  "overrides": {
+    "glob": "9.3.5"
+  },
   "scripts": {
-    "test": "gulp --gulpfile gulpfile.mjs",
-    "lint:watch": "esw --fix --watch index.mjs"
+    "lint": "eslint .",
+    "test": "gulp --gulpfile gulpfile.mjs"
   }
 }

package/.eslintrc.json

@@ -1,13 +0,0 @@
-
-{
-  "rules": {
-    "max-len": 0,
-    "indent": ["error", 2, { "VariableDeclarator": 1 }],
-    "global-require": 0,
-    "func-names": 0,
-    "prefer-rest-params": 1,
-    "strict": 1,
-    "no-param-reassign": 0
-  }
-}
-