@bigcommerce/stencil-cli 7.2.1 → 7.2.3

package/lib/ScssValidator.js

@@ -65,6 +65,7 @@ class ScssValidator {
                     const filePath = item[1].slice(1, -1);
                     const fileName = this.tryToResolveCssFileLocation(filePath);
                     if (
+                        fileName &&
                         !this.isStyleSheetAComment(content, filePath) &&
                         !cssFiles.includes(fileName)
                     ) {
@@ -107,6 +108,9 @@ class ScssValidator {
         for (const location of possibleLocations) {
             const fullFilePath = path.join(this.themePath, location);
             if (fs.existsSync(fullFilePath)) {
+                if (fullFilePath.endsWith('.css')) {
+                    return null;
+                }
                 if (!this.isRootCssFile(location)) {
                     return this.getCssFileWithoutRootFolder(location);
                 }

package/lib/nodeSass/AutoFixer.js

@@ -109,7 +109,7 @@ class AutoFixer {
             }
             if (
                 err.formatted.includes('Invalid CSS after') &&
-                err.formatted.includes('expected selector, was ', '')
+                err.formatted.includes('expected selector, was ",')
             ) {
                 return POSSIBLE_WRONG_COMMA;
             }

package/lib/nodeSass/BaseRulesFixer.js

@@ -13,19 +13,28 @@ class BaseRulesFixer extends BaseFixer {
     }
 
     transform() {
+        const self = this;
         return {
             postcssPlugin: 'Transform Base Rules Issues into Comments',
             Rule(rule, { Comment }) {
                 if (
                     rule.parent.type === 'root' &&
-                    (rule.selector.startsWith('&--') || rule.selector.startsWith('&.'))
+                    (rule.selector.startsWith('&') ||
+                        rule.selector.startsWith(':not(&)') ||
+                        rule.selector.startsWith('* &'))
                 ) {
-                    const comment = new Comment({ text: rule.toString() });
+                    const comment = new Comment({ text: self.replaceInnerComments(rule) });
                     rule.replaceWith(comment);
                 }
             },
         };
     }
+
+    // when we replace rule with comment, there might be a case when comment is inside another rule
+    // which breaks the commenting root rule
+    replaceInnerComments(rule) {
+        return rule.toString().replace(/\/\*(.*)\*\//g, '//$1');
+    }
 }
 
 module.exports = BaseRulesFixer;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bigcommerce/stencil-cli",
-  "version": "7.2.1",
+  "version": "7.2.3",
   "description": "CLI tool to run BigCommerce Stores locally for theme development.",
   "main": "index.js",
   "engines": {
@@ -48,7 +48,7 @@
     "package-lock.json"
   ],
   "dependencies": {
-    "@bigcommerce/stencil-paper": "4.2.1",
+    "@bigcommerce/stencil-paper": "4.10.4",
     "@bigcommerce/stencil-styles": "5.1.0",
     "@hapi/boom": "^10.0.0",
     "@hapi/glue": "^8.0.0",
@@ -79,6 +79,7 @@
     "object-to-spawn-args": "^2.0.0",
     "parse-json": "^5.2.0",
     "postcss": "^8.4.21",
+    "postcss-safe-parser": "^6.0.0",
     "postcss-scss": "^4.0.6",
     "progress": "^2.0.3",
     "recursive-readdir": "^2.2.2",