build: update to latest sass version (#20984)

Bumps to the latest version of Sass and fixes up our custom patch so that it accounts
for some changes in the published files.

Author: crisbeto

WORKSPACE

@@ -19,10 +19,10 @@ http_archive(
     # printed in worker mode: https://github.com/bazelbuild/rules_sass/issues/96.
     # TODO(devversion): remove this patch once the Sass Node entry-point returns a `Promise`.
     patches = ["//tools/postinstall:sass_worker_async.patch"],
-    sha256 = "c78be58f5e0a29a04686b628cf54faaee0094322ae0ac99da5a8a8afca59a647",
-    strip_prefix = "rules_sass-1.25.0",
+    sha256 = "cf28ff1bcfafb3c97f138bbc8ca9fe386e968ed3faaa9f8e6214abb5e88a2ecd",
+    strip_prefix = "rules_sass-1.29.0",
     urls = [
-        "https://github.com/bazelbuild/rules_sass/archive/1.25.0.zip",
+        "https://github.com/bazelbuild/rules_sass/archive/1.29.0.zip",
     ],
 )
 

package.json

@@ -158,7 +158,7 @@
     "rollup-plugin-node-resolve": "^5.2.0",
     "rollup-plugin-sourcemaps": "^0.4.2",
     "run-sequence": "^1.2.2",
-    "sass": "^1.24.4",
+    "sass": "^1.29.0",
     "scss-bundle": "^3.1.2",
     "selenium-webdriver": "^3.6.0",
     "semver": "^6.3.0",

tools/postinstall/sass_worker_async.patch

@@ -1,15 +1,21 @@
 diff --git sass/sass_wrapper.js sass/sass_wrapper.js
-index 21abb8f..9c750a3 100644
+index ad1f7a6..957d56e 100644
 --- sass/sass_wrapper.js
 +++ sass/sass_wrapper.js
-@@ -17,7 +17,11 @@ const fs = require('fs');
+@@ -17,7 +17,17 @@ const fs = require('fs');
  const args = process.argv.slice(2);
  if (runAsWorker(args)) {
    debug('Starting Sass compiler persistent worker...');
--  runWorkerLoop(args => sass.run_(args));
+-  runWorkerLoop(args => sass.cli_pkg_main_0_(args));
 +  runWorkerLoop(args => {
 +    return new Promise(resolve => {
-+      sass.run_(args)['then$1$2$onError'](() => resolve(true), () => resolve(false));
++      const result = sass.cli_pkg_main_0_(args);
++      const originalThen = result.then$1$2$onError;
++      let dynamicType;
++      result.then$1$2$onError = (receiver, done, error, type) => dynamicType = type;
++      result.then$1();
++      result.then$1$2$onError = originalThen;
++      result.then$1$2$onError(null, () => resolve(true), () => resolve(false), dynamicType);
 +    });
 +  });
    // Note: intentionally don't process.exit() here, because runWorkerLoop

yarn.lock

@@ -10846,16 +10846,16 @@ sass-lookup@^3.0.0:
     commander "^2.16.0"
 
 sass@^1.23.7:
-  version "1.29.0"
-  resolved "https://registry.yarnpkg.com/sass/-/sass-1.29.0.tgz#ec4e1842c146d8ea9258c28c141b8c2b7c6ab7f1"
-  integrity sha512-ZpwAUFgnvAUCdkjwPREny+17BpUj8nh5Yr6zKPGtLNTLrmtoRYIjm7njP24COhjJldjwW1dcv52Lpf4tNZVVRA==
+  version "1.27.0"
+  resolved "https://registry.yarnpkg.com/sass/-/sass-1.27.0.tgz#0657ff674206b95ec20dc638a93e179c78f6ada2"
+  integrity sha512-0gcrER56OkzotK/GGwgg4fPrKuiFlPNitO7eUJ18Bs+/NBlofJfMxmxqpqJxjae9vu0Wq8TZzrSyxZal00WDig==
   dependencies:
     chokidar ">=2.0.0 <4.0.0"
 
-sass@^1.24.4:
-  version "1.24.4"
-  resolved "https://registry.yarnpkg.com/sass/-/sass-1.24.4.tgz#aa50575a9ed2b9e9645b5599156fd149bdad9eaa"
-  integrity sha512-SqizkIEEcVPzmK1tYdlNRl/RSXMEwGcifL9GD+S2p9rEPdj6ycRbk4PWZs0jwlajNSyBPo/SXRB81i22SG0jmw==
+sass@^1.29.0:
+  version "1.29.0"
+  resolved "https://registry.yarnpkg.com/sass/-/sass-1.29.0.tgz#ec4e1842c146d8ea9258c28c141b8c2b7c6ab7f1"
+  integrity sha512-ZpwAUFgnvAUCdkjwPREny+17BpUj8nh5Yr6zKPGtLNTLrmtoRYIjm7njP24COhjJldjwW1dcv52Lpf4tNZVVRA==
   dependencies:
     chokidar ">=2.0.0 <4.0.0"