@salesforce-ux/eslint-plugin-slds 1.0.3-internal → 1.0.4-internal

package/build/index.js

@@ -45,9 +45,9 @@ var init_node = __esm({
   }
 });
 
-// yaml-file:/Users/ritesh.kumar2/Documents/projects/stylelint-sds/packages/eslint-plugin-slds/src/config/rule-messages.yml
+// yaml-file:rule-messages.yml
 var require_rule_messages = __commonJS({
-  "yaml-file:/Users/ritesh.kumar2/Documents/projects/stylelint-sds/packages/eslint-plugin-slds/src/config/rule-messages.yml"(exports2, module2) {
+  "yaml-file:rule-messages.yml"(exports2, module2) {
     module2.exports = {
       "no-slds-class-overrides": {
         "description": "Create new custom CSS classes instead of overriding SLDS selectors",
@@ -1056,7 +1056,7 @@ function forEachFontValue(valueText, callback) {
 }
 
 // src/utils/css-utils.ts
-function extractSldsVariable(node) {
+function extractCssVariable(node, filter) {
   if (!node || node.type !== "Function" || node.name !== "var") {
     return null;
   }
@@ -1072,19 +1072,31 @@ function extractSldsVariable(node) {
     return null;
   }
   const variableName = firstChild.name;
-  if (!variableName || !variableName.startsWith("--slds-")) {
+  if (!variableName) {
     return null;
   }
-  const hasFallback = childrenArray.some(
-    (child) => child.type === "Operator" && child.value === ","
-  );
-  return {
-    name: variableName,
-    hasFallback
-  };
+  return filter(variableName, childrenArray);
 }
 function forEachSldsVariable(valueText, callback) {
-  forEachValue(valueText, extractSldsVariable, () => false, callback);
+  const extractor = (node) => extractCssVariable(node, (variableName, childrenArray) => {
+    if (!variableName.startsWith("--slds-")) {
+      return null;
+    }
+    const hasFallback = childrenArray.some(
+      (child) => child.type === "Operator" && child.value === ","
+    );
+    return { name: variableName, hasFallback };
+  });
+  forEachValue(valueText, extractor, () => false, callback);
+}
+function forEachNamespacedVariable(valueText, callback) {
+  const extractor = (node) => extractCssVariable(node, (variableName) => {
+    if (variableName.startsWith("--slds-") || variableName.startsWith("--sds-")) {
+      return { name: variableName, hasFallback: false };
+    }
+    return null;
+  });
+  forEachValue(valueText, extractor, () => false, callback);
 }
 function formatSuggestionHooks(hooks) {
   if (hooks.length === 1) {
@@ -1503,7 +1515,7 @@ var import_rule_messages10 = __toESM(require_rule_messages());
 var ruleConfig8 = import_rule_messages10.default["no-slds-namespace-for-custom-hooks"];
 var { type: type9, description: description9, url: url9, messages: messages9 } = ruleConfig8;
 var sldsPlusStylingHooks3 = import_next10.default.sldsPlusStylingHooks;
-var allSldsHooks3 = [...sldsPlusStylingHooks3.global, ...sldsPlusStylingHooks3.component];
+var allSldsHooks3 = [...sldsPlusStylingHooks3.global, ...sldsPlusStylingHooks3.component, ...sldsPlusStylingHooks3.shared];
 var toSldsToken2 = (sdsToken) => sdsToken.replace("--sds-", "--slds-");
 function shouldIgnoreDetection4(sdsToken) {
   if (sdsToken.startsWith("--sds-") || sdsToken.startsWith("--slds-")) {
@@ -1522,33 +1534,54 @@ var no_slds_namespace_for_custom_hooks_default = {
     messages: messages9
   },
   create(context) {
-    function reportViolation(node, token) {
-      const tokenWithoutNamespace = token.replace("--slds-", "").replace("--sds-", "");
-      context.report({
-        node,
-        messageId: "customHookNamespace",
-        data: {
-          token,
-          tokenWithoutNamespace
-        }
-      });
-    }
     return {
-      // CSS custom property declarations: --slds-* and --sds-* properties
-      "Declaration[property=/^--s(lds|ds)-/]"(node) {
+      "Declaration"(node) {
         const property = node.property;
-        if (shouldIgnoreDetection4(property)) {
-          return;
+        if (property && (property.startsWith("--slds-") || property.startsWith("--sds-")) && !shouldIgnoreDetection4(property)) {
+          const tokenWithoutNamespace = property.replace("--slds-", "").replace("--sds-", "");
+          context.report({
+            node,
+            loc: node.loc,
+            // Use full declaration loc which includes the property
+            messageId: "customHookNamespace",
+            data: {
+              token: property,
+              tokenWithoutNamespace
+            }
+          });
         }
-        reportViolation(node, property);
-      },
-      // SLDS/SDS tokens inside var() functions: var(--slds-*) or var(--sds-*)
-      "Function[name='var'] Identifier[name=/^--s(lds|ds)-/]"(node) {
-        const tokenName = node.name;
-        if (shouldIgnoreDetection4(tokenName)) {
-          return;
+        const valueText = context.sourceCode.getText(node.value);
+        if (valueText) {
+          forEachNamespacedVariable(valueText, (variableInfo, positionInfo) => {
+            const { name: tokenName } = variableInfo;
+            if (!shouldIgnoreDetection4(tokenName)) {
+              const tokenWithoutNamespace = tokenName.replace("--slds-", "").replace("--sds-", "");
+              if (positionInfo.start && positionInfo.end && node.value.loc) {
+                context.report({
+                  node,
+                  loc: {
+                    start: {
+                      line: node.value.loc.start.line + positionInfo.start.line - 1,
+                      column: node.value.loc.start.column + positionInfo.start.column - 1
+                    },
+                    end: {
+                      line: node.value.loc.start.line + positionInfo.end.line - 1,
+                      column: node.value.loc.start.column + positionInfo.end.column - 1
+                    }
+                  },
+                  messageId: "customHookNamespace",
+                  data: { token: tokenName, tokenWithoutNamespace }
+                });
+              } else {
+                context.report({
+                  node,
+                  messageId: "customHookNamespace",
+                  data: { token: tokenName, tokenWithoutNamespace }
+                });
+              }
+            }
+          });
         }
-        reportViolation(node, tokenName);
       }
     };
   }
@@ -2352,7 +2385,7 @@ var rules = {
 var plugin = {
   meta: {
     name: "@salesforce-ux/eslint-plugin-slds",
-    version: "1.0.3-internal"
+    version: "1.0.4-internal"
   },
   rules,
   configs: {}