[v2.1.1] breaks `resolve-url-loader`

[jussikinnula]

When upgrading to 2.1.1 version of postcss-loader, file-loader and url-loader functionality is broken (e.g. using require() in JavaScript files or url() in CSS files).

The error message:

[0] WARNING in ./node_modules/css-loader??ref--3-1!./node_modules/resolve-url-loader??ref--3-2!./node_modules/postcss-loader/lib??ref--3-3!./node_modules/stylus-loader??ref--3-4!./app/App/index.styl
[0] (Emitted value instead of an instance of Error)   resolve-url-loader cannot operate: CSS error
[0]   ENOENT: no such file or directory, scandir '/Users/xxx/dev/app/App/app/App'
[0]   at Object.fs.readdirSync (fs.js:904:18)
[0]  @ ./app/App/index.styl 4:14-254
[0]  @ ./app/App/index.tsx
[0]  @ ./app/index.tsx

The problem seems to be that stylus-loader the path wrong. It should be /Users/xxx/dev/app/App (not /Users/xxx/dev/app/App/app/App).

On this case the index.styl contains:

.App
  background-image url('./foo.png')

On Webpack configuration, context is set like:

context: path.resolve(__dirname, './'),

Downgrading to postcss-loader 2.1.0 gets everything working again.

[jussikinnula]

Manually readding to: file, to options (ultimately reverting the PR #339 fix) resolves the issue:
https://github.com/postcss/postcss-loader/blob/master/lib/index.js#L105-L112

[Demivan]

Have same issue with resolve-url-loader.
resolve-url-loader uses SourceMapConsumer.originalPositionFor from mozilla/source-map. Path resolved by that method is now wrong:

{ source: 'D:/Projects/test/node_modules/video.js/dist/D:/Projects/test/node_modules/video.js/dist/video-js.css',
  line: 17,
  column: 2,
  name: null }

with version 2.1.0:

{ source: 'D:/Projects/test/node_modules/video.js/dist/video-js.css',
  line: 17,
  column: 2,
  name: null }

[alexander-akait]

Move postcss-loader above resolve-url-loader

[alexander-akait]

Also resolve-url-loader use rework (which deprecated and very buggy)

[Demivan]

Yeah, I know :). But it does not change that mozilla/source-map is now returning wrong path. It could break other loaders too.

[alexander-akait]

/cc @michael-ciniawsky

[jussikinnula]

After changing the order (moving postcss-loader above resolve-url-loader) gets rid of the error.

I understand the fact that many webpack loaders are buggy, but for now I haven't been able to resolve resolve importing/requiring a Stylus file inside a Stylus file - without using resolve-url-loader. Without it, the imported Stylus files are looking url() inside the same directory where the file requiring another is.

[alexander-akait]

@jussikinnula we can just rewrite resolve-url-loader on postcss and release this to avoid problem in url for scss/stylus and etc

[jussikinnula]

@evilebottnawi, that would be superb. It would simplify our life of bundling a lot of reusable React components in component libraries :-)

[Demivan]

There is already pending pull request to use postcss in resolve-url-loader

[Demivan]

but resolve-url-loader as postcss plugin would be just great

[michael-ciniawsky]

Sry, I didn't think of resolve-url-loader when merging #339 :). The fact is that

1. resolve-url-loader is basically a hack in general
2. This should be fixed in/guarded by resolve-url-loader itself
3. The use case for resolve-url-loader can be implemented in a cleaner way
4. It forces other loaders like e.g postcss-loader to produce malformed source maps then

The change should not have landed this way (as a patch) in retro spec, but since resolve-url-loader should/could be fixed and changing the loader order seems to work aswell, I'm not going to revert this change for now