Allow urls that contain fragments, fixes #560 (#561)

Author: romainmenke

lib/parse-styles.js

@@ -227,11 +227,6 @@ function isProcessableURL(uri) {
   try {
     // needs a base to parse properly
     const url = new URL(uri, "https://example.com")
-
-    if (url.hash) {
-      return false
-    }
-
     if (url.search) {
       return false
     }

test/fixtures/filter-ignore.css

@@ -13,5 +13,4 @@
 @import url('//css');
 @import url(//css);
 @import url('foo.css?query=string');
-@import url('foo.css#hash');
 content{}

test/fixtures/filter-ignore.expected.css

@@ -14,7 +14,6 @@
 @import url('//css');
 @import url(//css);
 @import url('foo.css?query=string');
-@import url('foo.css#hash');
 @media (min-width: 25em){
 ignore{}
 }

test/fixtures/imports/modules/subpath/a.css

@@ -0,0 +1,3 @@
+.foo {
+  color: green;
+}

test/fixtures/imports/modules/subpath/package.json

@@ -0,0 +1,6 @@
+{
+  "style": "style.css",
+  "imports": {
+    "#a.css": "./a.css"
+  }
+}

test/fixtures/imports/modules/subpath/style.css

@@ -0,0 +1 @@
+@import '#a.css';

test/fixtures/subpath.css

@@ -0,0 +1 @@
+@import "subpath";

test/fixtures/subpath.expected.css

@@ -0,0 +1,3 @@
+.foo {
+  color: green;
+}

test/resolve.js

@@ -1,6 +1,8 @@
 "use strict"
 // external tooling
 const test = require("ava")
+const fs = require("fs")
+const path = require("path")
 
 // internal tooling
 const checkFixture = require("./helpers/check-fixture")
@@ -72,3 +74,24 @@ test(
   { path: null, addModulesDirectories: ["shared_modules"] },
   { from: "test/fixtures/imports/foo.css" },
 )
+
+test(
+  "should resolve modules with node subpath imports with a custom resolver",
+  checkFixture,
+  "subpath",
+  {
+    resolve: (id, basedir) => {
+      // see: https://nodejs.org/api/packages.html#subpath-imports
+      if (id.startsWith("#")) {
+        const pkgJSON = JSON.parse(
+          fs.readFileSync(path.join(basedir, "package.json")),
+        )
+
+        return path.join(basedir, pkgJSON.imports[id])
+      }
+
+      return id
+    },
+    path: "test/fixtures/imports/modules",
+  },
+)