Broken sourcemap path

[kyle-ssg]

• Operating System: OSX High Sierra
• Node Version: 10.x
• NPM Version: v9.8.0
• webpack Version: 6.4.1
• sass-loader Version: 7.1.0

Expected Behavior

Given a file located at /Users/kyle/project/web/styles/components/_stat-card.scss I would expect the sourcemap path to be maintained.

Actual Behavior

Sourcemap outputs a path of /Users/kyle/project/web/styles/web/styles/components/_stat-card.scss

https://github.com/kyle-ssg/sass-loader-bug

Results in:

I was able to temporarily get around this by doing the following in loader.js:

 result.map.sources = result.map.sources.map((p)=>{
                return "file://"+process.cwd()+"/"+path.normalize(p)
            });

[kyle-ssg]

Seems similar to #460

[alexander-akait]

@kyle-ssg don't use gist for reproducible test repo, please create repo with source code, because in some places node-sass have bugs with source maps and it is hard to reproduce

[kyle-ssg]

@evilebottnawi Sure. Created a minimal repo that replicates the issue.

[alexander-akait]

@kyle-ssg what is command you use when you have invalid source map? I see you use Extract Text Webpack Plugin, he is outdated, you need use mini-css-webpack-plugin for extract css.

[kyle-ssg]

npm run dev

The sourcemap if you hover over the path in chrome points to /web/web

[alexander-akait]

@kyle-ssg do you test this on linux?

[kyle-ssg]

Nope, osx. The source map does show the scss but the path is wrong, so you can’t edit and save file from chrome.

[alexander-akait]

@kyle-ssg can't reproduce on osx, what do you mean "path is wrong"? Anyway i will look this problem in near future, feel free to investigate and send a PR

[kyle-ssg]

Ok, I’ll also try it on a different machine / send a replication video

On Mon, 4 Mar 2019 at 09:32, Evilebot Tnawi ***@***.***> wrote: @kyle-ssg <https://github.com/kyle-ssg> can't reproduce on osx, what do you mean "path is wrong"? Anyway i will look this problem in near future, feel free to investigate and send a PR — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#666 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AINaOi0RY0y80cS0b0XGIRIQ9vtoSi4Zks5vTOgggaJpZM4bBYrs> .

-- Kyle Johnson Head of Mobile Development Solid thinking. State of the art. Sign up <https://www.getrevue.co/profile/solidstategroup/> for our monthly take on tech and business

[alexander-akait]

Reproduce bug, similar problem webpack-contrib/css-loader#652, investigate who is broken

[alexander-akait]

Yep, duplicate webpack-contrib/css-loader#652, sass-loader output right source map