Fix: @import url() error in dependency resolver (#2632)

* Skip url() imports from deps list * Tests for url() imports in deps resolver
stylus · Sep 22, 2021 · d2cddcf · d2cddcf
1 parent 9cb7635
commit d2cddcf
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 0 deletions.
diff --git a/lib/visitor/deps-resolver.js b/lib/visitor/deps-resolver.js
@@ -110,6 +110,9 @@ DepsResolver.prototype.visitCall = function(call) {
  */
 
 DepsResolver.prototype.visitImport = function(node) {
+  // If it's a url() call, skip
+  if (node.path.first.name === 'url') return;
+
   var path = !node.path.first.val.isNull && node.path.first.val || node.path.first.name
     , literal, found, oldPath;
 

diff --git a/test/deps-resolver/url.deps b/test/deps-resolver/url.deps
@@ -0,0 +1,2 @@
+test/deps-resolver/url/a.styl
+test/deps-resolver/url/b.styl
diff --git a/test/deps-resolver/url.styl b/test/deps-resolver/url.styl
@@ -0,0 +1 @@
+@import 'url/a'
diff --git a/test/deps-resolver/url/a.styl b/test/deps-resolver/url/a.styl
@@ -0,0 +1,2 @@
+@import url('https://example.com')
+@import 'b'
diff --git a/test/deps-resolver/url/b.styl b/test/deps-resolver/url/b.styl
@@ -0,0 +1 @@
+@import url('http://foo.com/some.styl')