Delete source maps when source files are deleted in watch mode

Closes #2613

Author: nex3

CHANGELOG.md

@@ -1,3 +1,10 @@
+## 1.91.1
+
+### Command-Line Interface
+
+* In `--watch` mode, delete the source map when the associated source file is
+  deleted.
+
 ## 1.91.0
 
 * **Potentially breaking change:** `meta.inspect()` (as well as other systems

lib/src/executable/watch.dart

@@ -166,7 +166,10 @@ final class _Watcher {
     var url = _canonicalize(path);
 
     if (_graph.nodes.containsKey(url)) {
-      if (_destinationFor(path) case var destination?) _delete(destination);
+      if (_destinationFor(path) case var destination?) {
+        _delete(destination);
+        _delete("$destination.map");
+      }
     }
 
     var downstream = _graph.remove(FilesystemImporter.cwd, url);

pkg/sass-parser/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.4.27
+
+* No user-visible changes.
+
 ## 0.4.26
 
 * No user-visible changes.

pkg/sass-parser/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sass-parser",
-  "version": "0.4.26",
+  "version": "0.4.27-dev",
   "description": "A PostCSS-compatible wrapper of the official Sass parser",
   "repository": "sass/sass",
   "author": "Google Inc.",

pkg/sass_api/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 15.9.1
+
+* No user-visible changes.
+
 ## 15.9.0
 
 * No user-visible changes.

pkg/sass_api/pubspec.yaml

@@ -2,15 +2,15 @@ name: sass_api
 # Note: Every time we add a new Sass AST node, we need to bump the *major*
 # version because it's a breaking change for anyone who's implementing the
 # visitor interface(s).
-version: 15.9.0
+version: 15.9.1-dev
 description: Additional APIs for Dart Sass.
 homepage: https://github.com/sass/dart-sass
 
 environment:
   sdk: ">=3.6.0 <4.0.0"
 
 dependencies:
-  sass: 1.91.0
+  sass: 1.91.1-dev
 
 dev_dependencies:
   dartdoc: ^8.0.14

pubspec.yaml

@@ -1,5 +1,5 @@
 name: sass
-version: 1.91.0
+version: 1.91.1-dev
 description: A Sass implementation in Dart.
 homepage: https://github.com/sass/dart-sass
 

test/cli/shared/watch.dart

@@ -976,6 +976,29 @@ void sharedTests(Future<TestProcess> runSass(Iterable<String> arguments)) {
 
           await d.dir("dir", [d.nothing("out.css")]).validate();
         });
+
+        test("and deletes the source map", () async {
+          await d.file("test.scss", "a {b: c}").create();
+
+          var sass = await watch(["--source-map", "test.scss:out.css"]);
+          await expectLater(
+            sass.stdout,
+            emits(endsWith('Compiled test.scss to out.css.')),
+          );
+          await expectLater(sass.stdout, _watchingForChanges);
+          await tickIfPoll();
+
+          d.file("test.scss").io.deleteSync();
+          await expectLater(
+              sass.stdout,
+              emitsInOrder([
+                'Deleted out.css.',
+                'Deleted out.css.map.',
+              ]));
+          await sass.kill();
+
+          await d.nothing("out.css").validate();
+        });
       });
 
       test("creates a new CSS file when a Sass file is added", () async {