After upgrading to version 6.0.0, source file URLs are not generated correctly

[ldu1991]

In version 5.1.0 the source file was shown correctly, but after switching to version 6 sourcemap behaves incomprehensibly
Here is the build https://github.com/ldu1991/acf-blocks-theme/blob/master/sources/gulpfile.js
this happens when building css files in scssBlocks function (line 230)

[wkillerud]

Hi. In #846 (comment) it's mentioned that gulp-sourcemaps is deprecated and should be replaced with some sort of built-in sourcemaps in Gulp. I'm guessing this. Can you try that?

[Alecto]

in 6.0 gulp-sass this options doesnt work correctly.

I have a problem: when generating a source map using gulp options, instead of the file name, I see "data:; charset" in the devtools instead of the scss file name. I reverted to gulp-sass version 5.1 and everything works correctly.

[wkillerud]

Seems to work for me 🤷

If it doesn't work for you, you can help by creating a minimal repository that demonstrates a bug.

import gulp from "gulp";
import gulpSass from "gulp-sass";
import * as dartSass from "sass";

const sass = gulpSass(dartSass);

export default (cb) => {
  gulp
    .src("./src/**/*.scss", { sourcemaps: true })
    .pipe(sass().on("error", sass.logError))
    .pipe(gulp.dest("./public", { sourcemaps: "." }));
  cb();
};

[TalantRys]

@wkillerud, this issue happens when your forder has deep level folders more than just one with gulp-sass v6. Maybe it's sass new Api problem🤔.

To avoid this bug for now, i desided to use lower version.

I've tried to recreate it in stackblitz. You can switch versions in terminal by npm and see differents of *.map in dipper folder (Change sass API in gulpfile also).

https://stackblitz.com/edit/stackblitz-starters-mmhrqprq?view=editor

[TalantRys]

gulp-sass injects content into the path rather than a file in *.map

[wkillerud]

Thanks for the Stackblitz @TalantRys, I managed to reproduce the error now 🙌

#876 should fix it once it's released.

If anyone wants to help test, you can try replacing the contents of node_modules/gulp-sass/index.js with the version from the pull-request and see if the source maps look correct.

[xzyfer]

@wkillerud fix has been merged and released in v6.0.1. Apologies for the disruption.

[TalantRys]

Hello, @wkillerud. Looks like there is still some bug with mapping.
When I navigate to sourse in DevTools I see that link:
https://localhost/bitrix/templates/default/css/bitrix/templates/default/css/scss/header/_default.scss

but I expect to see this:
https://localhost/bitrix/templates/default/css/scss/header/_default.scss

Sourses tab shows deep folder:

My index.scss:

Scss partials:

[TalantRys]

Sorry, I had to add options to the gulp source maps.
No issue for now 👍
sourcemaps.write('.', { includeContent: false, sourceRoot: './' }
https://stackblitz.com/edit/stackblitz-starters-mmhrqprq?view=editor