[FIX] Config: check stylesheets scope

The style that we define in our component can sometimes interfere within
Odoo. This revision adds a step to force the style to be scoped to
'o-spreadsheet'.

Task: 4629441

Author: rrahir

.husky/pre-commit

@@ -25,15 +25,15 @@ if [ "$HUSKY_PRE_COMMIT" != 0 ]; then
             exec git diff --cached --color | grep -nE $consoleregexp -A 2 -B 2
             read -p "There are some occurrences of forbidden patterns at your modification. Are you sure want to continue? (y/n)" yn
             echo $yn | grep ^[Yy]$
-            if [ $? -eq 0 ]
-            then
-                exit 0; #THE USER WANTS TO CONTINUE
-            else
+            if [ $? -ne 0 ]; then
                 exit 1; # THE USER DONT WANT TO CONTINUE SO ROLLBACK
             fi
         else
             echo -e "\033[1;31mWARNING: Some occurences of forbidden patterns were detected \033[0m"
             exec git diff --cached --color | grep -nE $consoleregexp -A 2 -B 2
         fi
     fi
+
+    # Css wrap check
+    node .husky/scss_check.js
 fi

.husky/scss_check.js

@@ -0,0 +1,46 @@
+import { execSync } from "child_process";
+import { existsSync, readFileSync } from "fs";
+import process from "process";
+
+const commentPattern = /\/\/.*|\/\*[\s\S]*?\*\//g;
+// const firstLevelSelectorPattern = /^(\.(?!o-spreadsheet\s)|[a-zA-Z]+).*\{/gm;
+const firstLevelSelectorPattern = /^(?!(\s|\$|\})).+/gm;
+
+// Get scss files in diff
+const files = execSync("git diff --name-only --cached")
+  .toString()
+  .split("\n")
+  .filter((file) => file.endsWith(".scss"));
+
+const faultyFiles = [];
+
+for (const file of files) {
+  if (!existsSync(file)) continue;
+  let content = readFileSync(file, "utf8");
+
+  // Remove content of comments
+  content = content.replace(commentPattern, "");
+  // look for first level selectors
+  const firstLevelSelectors = content.match(firstLevelSelectorPattern);
+  if (!firstLevelSelectors.every((line) => line.startsWith(".o-spreadsheet "))) {
+    faultyFiles.push(file);
+  }
+}
+
+if (faultyFiles.length > 0) {
+  const fileList = " - " + faultyFiles.join("\n - ");
+  console.log(`\x1b[31m
+Some scss files are not scoped to .o-spreadsheet. Please fix them before committing.
+Faulty files:
+
+${fileList}
+
+Your file should start with:
+.o-spreadsheet {
+    ...
+}
+
+and the selector should encompass all the style.\x1b[0m
+`);
+  process.exit(1);
+}

src/variables.scss

@@ -1 +1 @@
-$os-text-body: #374151;
+$os-text-body: #1869eb;