npm - @nordcraft/search - Versions diffs - 1.0.87 → 1.0.89 - Mend

@nordcraft/search 1.0.87 → 1.0.89

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (23) hide show

package/dist/rules/issues/dom/nonEmptyVoidElementRule.js CHANGED Viewed

@@ -1,4 +1,4 @@
-import { VOID_HTML_ELEMENTS } from '@nordcraft/ssr/dist/const';
+import { VOID_HTML_ELEMENTS } from '@nordcraft/core/dist/utils/html';
 /**
  * See full list here
  * https://developer.mozilla.org/en-US/docs/Glossary/Void_element

package/dist/rules/issues/dom/nonEmptyVoidElementRule.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"nonEmptyVoidElementRule.js","sourceRoot":"","sources":["../../../../src/rules/issues/dom/nonEmptyVoidElementRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAAE,MAAM,~~2BAA2B~~,CAAA;~~AAE9D~~;;;GAGG;AACH,MAAM,CAAC,MAAM,uBAAuB,GAA0B;IAC5D,IAAI,EAAE,wBAAwB;IAC9B,KAAK,EAAE,SAAS;IAChB,QAAQ,EAAE,SAAS;IACnB,KAAK,EAAE,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC;QAC5C,IACE,QAAQ,KAAK,gBAAgB;YAC7B,KAAK,CAAC,IAAI,KAAK,SAAS;YACxB,KAAK,CAAC,QAAQ,CAAC,MAAM,IAAI,CAAC;YAC1B,CAAC,kBAAkB,CAAC,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,EACvC,CAAC;YACD,OAAM;QACR,CAAC;QACD,MAAM,CAAC;YACL,IAAI;YACJ,IAAI,EAAE;gBACJ,KAAK,EAAE,wBAAwB;gBAC/B,WAAW,EAAE,SAAS,KAAK,CAAC,GAAG,wCAAwC,KAAK,CAAC,GAAG,4GAA4G;aAC7L;YACD,OAAO,EAAE,EAAE,GAAG,EAAE,KAAK,CAAC,GAAG,EAAE;SAC5B,CAAC,CAAA;IAAA,CACH;CACF,CAAA"}
1	+ {"version":3,"file":"nonEmptyVoidElementRule.js","sourceRoot":"","sources":["../../../../src/rules/issues/dom/nonEmptyVoidElementRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAAE,MAAM,iCAAiC,CAAA;AAEpE;;;GAGG;AACH,MAAM,CAAC,MAAM,uBAAuB,GAA0B;IAC5D,IAAI,EAAE,wBAAwB;IAC9B,KAAK,EAAE,SAAS;IAChB,QAAQ,EAAE,SAAS;IACnB,KAAK,EAAE,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC;QAC5C,IACE,QAAQ,KAAK,gBAAgB;YAC7B,KAAK,CAAC,IAAI,KAAK,SAAS;YACxB,KAAK,CAAC,QAAQ,CAAC,MAAM,IAAI,CAAC;YAC1B,CAAC,kBAAkB,CAAC,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,EACvC,CAAC;YACD,OAAM;QACR,CAAC;QACD,MAAM,CAAC;YACL,IAAI;YACJ,IAAI,EAAE;gBACJ,KAAK,EAAE,wBAAwB;gBAC/B,WAAW,EAAE,SAAS,KAAK,CAAC,GAAG,wCAAwC,KAAK,CAAC,GAAG,4GAA4G;aAC7L;YACD,OAAO,EAAE,EAAE,GAAG,EAAE,KAAK,CAAC,GAAG,EAAE;SAC5B,CAAC,CAAA;IAAA,CACH;CACF,CAAA"}

package/dist/rules/issues/miscellaneous/createStaticSizeConstraintRule.js CHANGED Viewed

@@ -1,5 +1,5 @@
+import { VOID_HTML_ELEMENTS } from '@nordcraft/core/dist/utils/html';
 import { isDefined } from '@nordcraft/core/dist/utils/util';
-import { VOID_HTML_ELEMENTS } from '@nordcraft/ssr/dist/const';
 export function createStaticSizeConstraintRule(tag, maxSize, level = 'info') {
     return {
         code: 'size constraint',

package/dist/rules/issues/miscellaneous/createStaticSizeConstraintRule.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"createStaticSizeConstraintRule.js","sourceRoot":"","sources":["../../../../src/rules/issues/miscellaneous/createStaticSizeConstraintRule.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,~~SAAS~~,EAAE,MAAM,iCAAiC,CAAA;~~AAC3D~~,OAAO,EAAE,~~kBAAkB~~,EAAE,MAAM,~~2BAA2B~~,CAAA;~~AAG9D~~,MAAM,UAAU,8BAA8B,CAC5C,GAAW,EACX,OAAe,EACf,KAAK,GAAU,MAAM,EAIpB;IACD,OAAO;QACL,IAAI,EAAE,iBAAiB;QACvB,QAAQ,EAAE,aAAa;QACvB,KAAK,EAAE,KAAK;QACZ,KAAK,EAAE,CAAC,MAAM,EAAE,IAAI,EAAE,EAAE,CAAC;YACvB,IACE,IAAI,CAAC,QAAQ,KAAK,gBAAgB;gBAClC,IAAI,CAAC,KAAK,CAAC,IAAI,KAAK,SAAS;gBAC7B,IAAI,CAAC,KAAK,CAAC,GAAG,KAAK,GAAG,EACtB,CAAC;gBACD,IAAI,IAAI,GAAG,CAAC,CAAA;gBACZ,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAA;gBAChC,MAAM,eAAe,GAAG,CAAC,OAAmB,EAAU,EAAE,CAAC;oBACvD,IACE,CAAC,OAAO;wBACR,CAAC,SAAS,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,CAAC;4BAC7D,KAAK,EACP,CAAC;wBACD,OAAO,EAAE,CAAA;oBACX,CAAC;oBACD,MAAM,QAAQ,GAAG,CAAC,OAAO,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CACtD,eAAe,CAAC,SAAS,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,CAAC,CAC1C,CAAA;oBACD,MAAM,GAAG,GAAG,OAAO,CAAC,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,MAAM,CAAA;oBAC7D,MAAM,UAAU,GAA2B,EAAE,CAAA;oBAC7C,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC;wBAC/D,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC;4BAC5D,IAAI,KAAK,EAAE,IAAI,KAAK,OAAO,IAAI,SAAS,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC;gCACtD,UAAU,CAAC,GAAG,CAAC,GAAG,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAA;4BACvC,CAAC;wBAAA,CACF,CAAC,CAAA;oBACJ,CAAC;oBACD,MAAM,eAAe,GAAG,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC;yBAC/C,GAAG,CAAC,CAAC,CAAC,GAAG,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,KAAK,KAAK,GAAG,CAAC;yBAC1C,IAAI,CAAC,GAAG,CAAC,CAAA;oBACZ,IAAI,kBAAkB,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;wBACrC,OAAO,eAAe;4BACpB,CAAC,CAAC,IAAI,GAAG,IAAI,eAAe,KAAK;4BACjC,CAAC,CAAC,IAAI,GAAG,KAAK,CAAA;oBAClB,CAAC;yBAAM,CAAC;wBACN,MAAM,UAAU,GAAG,eAAe;4BAChC,CAAC,CAAC,IAAI,GAAG,IAAI,eAAe,GAAG;4BAC/B,CAAC,CAAC,IAAI,GAAG,GAAG,CAAA;wBACd,OAAO,GAAG,UAAU,GAAG,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC,KAAK,GAAG,GAAG,CAAA;oBACrD,CAAC;gBAAA,CACF,CAAA;gBACD,MAAM,aAAa,GAAG,eAAe,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;gBACjD,IAAI,GAAG,IAAI,IAAI,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAA;gBACrC,IAAI,IAAI,GAAG,OAAO,EAAE,CAAC;oBACnB,MAAM,CAAC;wBACL,IAAI,EAAE,IAAI,CAAC,IAAI;wBACf,IAAI,EAAE;4BACJ,KAAK,EAAE,8DAA8D,aAAa,CAAC,OAAO,CAAC,EAAE;4BAC7F,WAAW,EAAE,QAAQ,GAAG,2BAA2B,aAAa,CAAC,IAAI,CAAC,0CAA0C,aAAa,CAAC,OAAO,CAAC,6GAA6G;yBACpP;wBACD,OAAO,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE;qBACvB,CAAC,CAAA;gBACJ,CAAC;YACH,CAAC;QAAA,CACF;KACF,CAAA;AAAA,CACF;AAED,MAAM,aAAa,GAAG,CAAC,WAAmB,EAAE,EAAE,CAAC;IAC7C,MAAM,KAAK,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,CAAA;IAC9C,IAAI,SAAS,GAAG,CAAC,CAAA;IACjB,IAAI,IAAI,GAAG,WAAW,CAAA;IAEtB,OAAO,IAAI,IAAI,IAAI,IAAI,SAAS,GAAG,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACpD,IAAI,IAAI,IAAI,CAAA;QACZ,SAAS,EAAE,CAAA;IACb,CAAC;IACD,MAAM,IAAI,GAAG,SAAS,KAAK,CAAC,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAA;IACvE,OAAO,GAAG,IAAI,IAAI,CAAC,YAAY,CAAC,SAAS,EAAE;QACzC,qBAAqB,EAAE,SAAS,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;KAC/C,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,IAAI,EAAE,CAAA;AAAA,CAC1B,CAAA"}
1	+ {"version":3,"file":"createStaticSizeConstraintRule.js","sourceRoot":"","sources":["../../../../src/rules/issues/miscellaneous/createStaticSizeConstraintRule.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,kBAAkB,EAAE,MAAM,iCAAiC,CAAA;AACpE,OAAO,EAAE,SAAS,EAAE,MAAM,iCAAiC,CAAA;AAG3D,MAAM,UAAU,8BAA8B,CAC5C,GAAW,EACX,OAAe,EACf,KAAK,GAAU,MAAM,EAIpB;IACD,OAAO;QACL,IAAI,EAAE,iBAAiB;QACvB,QAAQ,EAAE,aAAa;QACvB,KAAK,EAAE,KAAK;QACZ,KAAK,EAAE,CAAC,MAAM,EAAE,IAAI,EAAE,EAAE,CAAC;YACvB,IACE,IAAI,CAAC,QAAQ,KAAK,gBAAgB;gBAClC,IAAI,CAAC,KAAK,CAAC,IAAI,KAAK,SAAS;gBAC7B,IAAI,CAAC,KAAK,CAAC,GAAG,KAAK,GAAG,EACtB,CAAC;gBACD,IAAI,IAAI,GAAG,CAAC,CAAA;gBACZ,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAA;gBAChC,MAAM,eAAe,GAAG,CAAC,OAAmB,EAAU,EAAE,CAAC;oBACvD,IACE,CAAC,OAAO;wBACR,CAAC,SAAS,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,CAAC;4BAC7D,KAAK,EACP,CAAC;wBACD,OAAO,EAAE,CAAA;oBACX,CAAC;oBACD,MAAM,QAAQ,GAAG,CAAC,OAAO,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CACtD,eAAe,CAAC,SAAS,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,CAAC,CAC1C,CAAA;oBACD,MAAM,GAAG,GAAG,OAAO,CAAC,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,MAAM,CAAA;oBAC7D,MAAM,UAAU,GAA2B,EAAE,CAAA;oBAC7C,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC;wBAC/D,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC;4BAC5D,IAAI,KAAK,EAAE,IAAI,KAAK,OAAO,IAAI,SAAS,CAAC,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC;gCACtD,UAAU,CAAC,GAAG,CAAC,GAAG,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAA;4BACvC,CAAC;wBAAA,CACF,CAAC,CAAA;oBACJ,CAAC;oBACD,MAAM,eAAe,GAAG,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC;yBAC/C,GAAG,CAAC,CAAC,CAAC,GAAG,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,KAAK,KAAK,GAAG,CAAC;yBAC1C,IAAI,CAAC,GAAG,CAAC,CAAA;oBACZ,IAAI,kBAAkB,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;wBACrC,OAAO,eAAe;4BACpB,CAAC,CAAC,IAAI,GAAG,IAAI,eAAe,KAAK;4BACjC,CAAC,CAAC,IAAI,GAAG,KAAK,CAAA;oBAClB,CAAC;yBAAM,CAAC;wBACN,MAAM,UAAU,GAAG,eAAe;4BAChC,CAAC,CAAC,IAAI,GAAG,IAAI,eAAe,GAAG;4BAC/B,CAAC,CAAC,IAAI,GAAG,GAAG,CAAA;wBACd,OAAO,GAAG,UAAU,GAAG,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC,KAAK,GAAG,GAAG,CAAA;oBACrD,CAAC;gBAAA,CACF,CAAA;gBACD,MAAM,aAAa,GAAG,eAAe,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;gBACjD,IAAI,GAAG,IAAI,IAAI,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAA;gBACrC,IAAI,IAAI,GAAG,OAAO,EAAE,CAAC;oBACnB,MAAM,CAAC;wBACL,IAAI,EAAE,IAAI,CAAC,IAAI;wBACf,IAAI,EAAE;4BACJ,KAAK,EAAE,8DAA8D,aAAa,CAAC,OAAO,CAAC,EAAE;4BAC7F,WAAW,EAAE,QAAQ,GAAG,2BAA2B,aAAa,CAAC,IAAI,CAAC,0CAA0C,aAAa,CAAC,OAAO,CAAC,6GAA6G;yBACpP;wBACD,OAAO,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE;qBACvB,CAAC,CAAA;gBACJ,CAAC;YACH,CAAC;QAAA,CACF;KACF,CAAA;AAAA,CACF;AAED,MAAM,aAAa,GAAG,CAAC,WAAmB,EAAE,EAAE,CAAC;IAC7C,MAAM,KAAK,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,CAAA;IAC9C,IAAI,SAAS,GAAG,CAAC,CAAA;IACjB,IAAI,IAAI,GAAG,WAAW,CAAA;IAEtB,OAAO,IAAI,IAAI,IAAI,IAAI,SAAS,GAAG,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACpD,IAAI,IAAI,IAAI,CAAA;QACZ,SAAS,EAAE,CAAA;IACb,CAAC;IACD,MAAM,IAAI,GAAG,SAAS,KAAK,CAAC,IAAI,IAAI,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAA;IACvE,OAAO,GAAG,IAAI,IAAI,CAAC,YAAY,CAAC,SAAS,EAAE;QACzC,qBAAqB,EAAE,SAAS,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;KAC/C,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,IAAI,EAAE,CAAA;AAAA,CAC1B,CAAA"}

package/dist/rules/issues/style/invalidStyleSyntaxRule.js CHANGED Viewed

@@ -8,6 +8,22 @@ export const invalidStyleSyntaxRule = {
         if (nodeType !== 'style-declaration') {
             return;
         }
+        // Check for variable/formula references: Variables., Formulas., Event., Attributes., Apis., Parameters., ListItem., URLParameters.
+        if (typeof value.styleValue === 'string') {
+            const hasVariableReference = /\b(Variables|Formulas|Event|Attributes|Apis|Parameters|ListItem|URLParameters)\.\w+/i.test(value.styleValue);
+            if (hasVariableReference) {
+                report({
+                    path,
+                    info: {
+                        title: `Formulas detected in style declaration`,
+                        description: `The style declaration for the property "${value.styleProperty}" contains Nordcraft formula syntax (e.g., references like "Variables.xxx", "Event.xxx", "Attributes.xxx", etc.). Formulas should not be used directly in CSS style values. Use style-variables or computed styles instead.`,
+                    },
+                    details: { property: value.styleProperty },
+                    fixes: ['delete-style-property'],
+                });
+                return;
+            }
+        }
         const valid = memo(`valid-style-${value.styleProperty}:${value.styleValue}`, () => {
             try {
                 parse(`${value.styleProperty}: ${value.styleValue}`);

package/dist/rules/issues/style/invalidStyleSyntaxRule.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"invalidStyleSyntaxRule.js","sourceRoot":"","sources":["../../../../src/rules/issues/style/invalidStyleSyntaxRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,SAAS,CAAA;AAE/B,OAAO,EAAE,iBAAiB,EAAE,MAAM,gCAAgC,CAAA;AAElE,MAAM,CAAC,MAAM,sBAAsB,GAE9B;IACH,IAAI,EAAE,sBAAsB;IAC5B,KAAK,EAAE,OAAO;IACd,QAAQ,EAAE,SAAS;IACnB,KAAK,EAAE,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC;QAClD,IAAI,QAAQ,KAAK,mBAAmB,EAAE,CAAC;YACrC,OAAM;QACR,CAAC;~~QACD~~,MAAM,KAAK,GAAG,IAAI,CAChB,eAAe,KAAK,CAAC,aAAa,IAAI,KAAK,CAAC,UAAU,EAAE,EACxD,GAAG,EAAE,CAAC;YACJ,IAAI,CAAC;gBACH,KAAK,CAAC,GAAG,KAAK,CAAC,aAAa,KAAK,KAAK,CAAC,UAAU,EAAE,CAAC,CAAA;gBACpD,OAAO,IAAI,CAAA;YACb,CAAC;YAAC,MAAM,CAAC;gBACP,OAAO,KAAK,CAAA;YACd,CAAC;QAAA,CACF,CACF,CAAA;QACD,IAAI,CAAC,KAAK,EAAE,CAAC;YACX,MAAM,CAAC;gBACL,IAAI;gBACJ,IAAI,EAAE;oBACJ,KAAK,EAAE,2BAA2B;oBAClC,WAAW,EAAE,2CAA2C,KAAK,CAAC,aAAa,kIAAkI;iBAC9M;gBACD,OAAO,EAAE,EAAE,QAAQ,EAAE,KAAK,CAAC,aAAa,EAAE;gBAC1C,KAAK,EAAE,CAAC,uBAAuB,CAAC;aACjC,CAAC,CAAA;QACJ,CAAC;IAAA,CACF;IACD,KAAK,EAAE;QACL,uBAAuB,EAAE,iBAAiB;KAC3C;CACF,CAAA"}
1	+ {"version":3,"file":"invalidStyleSyntaxRule.js","sourceRoot":"","sources":["../../../../src/rules/issues/style/invalidStyleSyntaxRule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,SAAS,CAAA;AAE/B,OAAO,EAAE,iBAAiB,EAAE,MAAM,gCAAgC,CAAA;AAElE,MAAM,CAAC,MAAM,sBAAsB,GAE9B;IACH,IAAI,EAAE,sBAAsB;IAC5B,KAAK,EAAE,OAAO;IACd,QAAQ,EAAE,SAAS;IACnB,KAAK,EAAE,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC;QAClD,IAAI,QAAQ,KAAK,mBAAmB,EAAE,CAAC;YACrC,OAAM;QACR,CAAC;QAED,mIAAmI;QACnI,IAAI,OAAO,KAAK,CAAC,UAAU,KAAK,QAAQ,EAAE,CAAC;YACzC,MAAM,oBAAoB,GACxB,sFAAsF,CAAC,IAAI,CACzF,KAAK,CAAC,UAAU,CACjB,CAAA;YACH,IAAI,oBAAoB,EAAE,CAAC;gBACzB,MAAM,CAAC;oBACL,IAAI;oBACJ,IAAI,EAAE;wBACJ,KAAK,EAAE,wCAAwC;wBAC/C,WAAW,EAAE,2CAA2C,KAAK,CAAC,aAAa,6NAA6N;qBACzS;oBACD,OAAO,EAAE,EAAE,QAAQ,EAAE,KAAK,CAAC,aAAa,EAAE;oBAC1C,KAAK,EAAE,CAAC,uBAAuB,CAAC;iBACjC,CAAC,CAAA;gBACF,OAAM;YACR,CAAC;QACH,CAAC;QAED,MAAM,KAAK,GAAG,IAAI,CAChB,eAAe,KAAK,CAAC,aAAa,IAAI,KAAK,CAAC,UAAU,EAAE,EACxD,GAAG,EAAE,CAAC;YACJ,IAAI,CAAC;gBACH,KAAK,CAAC,GAAG,KAAK,CAAC,aAAa,KAAK,KAAK,CAAC,UAAU,EAAE,CAAC,CAAA;gBACpD,OAAO,IAAI,CAAA;YACb,CAAC;YAAC,MAAM,CAAC;gBACP,OAAO,KAAK,CAAA;YACd,CAAC;QAAA,CACF,CACF,CAAA;QACD,IAAI,CAAC,KAAK,EAAE,CAAC;YACX,MAAM,CAAC;gBACL,IAAI;gBACJ,IAAI,EAAE;oBACJ,KAAK,EAAE,2BAA2B;oBAClC,WAAW,EAAE,2CAA2C,KAAK,CAAC,aAAa,kIAAkI;iBAC9M;gBACD,OAAO,EAAE,EAAE,QAAQ,EAAE,KAAK,CAAC,aAAa,EAAE;gBAC1C,KAAK,EAAE,CAAC,uBAAuB,CAAC;aACjC,CAAC,CAAA;QACJ,CAAC;IAAA,CACF;IACD,KAAK,EAAE;QACL,uBAAuB,EAAE,iBAAiB;KAC3C;CACF,CAAA"}

package/dist/rules/issues/style/styleRules.index.d.ts CHANGED Viewed

@@ -2,5 +2,7 @@ declare const _default: (import("../../../types").Rule<unknown, import("../../..
     property: string;
 }, import("../../../types").NodeType, import("../../../types").NodeType> | import("../../../types").Rule<{
     name: string;
-}, import("../../../types").NodeType, import("../../../types").StyleVariableNode>)[];
+}, import("../../../types").NodeType, import("../../../types").StyleVariableNode> | import("../../../types").Rule<{
+    name: string;
+}, import("../../../types").StyleNode, import("../../../types").StyleNode>)[];
 export default _default;

package/dist/rules/issues/style/unknownCSSVariable.d.ts CHANGED Viewed

@@ -1,4 +1,5 @@
-import type { Rule } from '../../../types';
+import type { Rule, StyleNode } from '../../../types';
 export declare const unknownCSSVariableRule: Rule<{
     name: string;
-}>;
+}, StyleNode>;
+export type AddToThemeFix = 'add-to-theme';

package/dist/rules/issues/style/unknownCSSVariable.js CHANGED Viewed

@@ -18,59 +18,102 @@ export const unknownCSSVariableRule = {
         if (vars.length === 0) {
             return;
         }
-        const allCssVariableDeclarations = memo(`all-css-variables-declarations`, () => {
-            const cssVariableKeys = new Set(Object.keys(theme.propertyDefinitions ?? {}));
-            Object.values(files.components ?? {}).forEach((component) => {
-                Object.keys(component?.nodes ?? {}).forEach((nodeKey) => {
-                    const node = component?.nodes?.[nodeKey];
-                    if (!node) {
-                        return;
-                    }
-                    if (node.type === 'component' || node.type === 'element') {
-                        Object.keys(node.customProperties ?? {}).forEach((varName) => {
-                            cssVariableKeys.add(varName);
+        const themeCssVariables = theme.propertyDefinitions;
+        const [_fileType, componentName, _nodes, nodeName] = path;
+        const localCssVariables = memo(`component-css-variables-${componentName}-${nodeName}`, () => {
+            const vars = new Set();
+            const component = files.components[componentName];
+            if (!component) {
+                return vars;
+            }
+            const visitVars = (nodeName) => {
+                const node = component?.nodes?.[nodeName];
+                if (!node) {
+                    return;
+                }
+                if (node.type === 'component' || node.type === 'element') {
+                    Object.keys(node.customProperties ?? {}).forEach((varName) => {
+                        vars.add(varName);
+                    });
+                    Object.values(node.variants ?? {}).forEach((variant) => {
+                        Object.keys(variant.customProperties ?? {}).forEach((varName) => {
+                            vars.add(varName);
                         });
-                        Object.values(node.variants ?? {}).forEach((variant) => {
-                            Object.keys(variant.customProperties ?? {}).forEach((varName) => {
-                                cssVariableKeys.add(varName);
-                            });
+                    });
+                    // Also add legacy style variables
+                    if (node.type === 'element' && node['style-variables']) {
+                        node['style-variables'].forEach((styleVar) => {
+                            vars.add(`--${styleVar.name}`);
                         });
-                        // Also add legacy style variables
-                        if (node.type === 'element' && node['style-variables']) {
-                            node['style-variables'].forEach((styleVar) => {
-                                cssVariableKeys.add(`--${styleVar.name}`);
-                            });
+                    }
+                    // Add if declared in any parent styles object
+                    Object.keys(node.style ?? {}).forEach((styleKey) => {
+                        if (styleKey.startsWith('--')) {
+                            vars.add(styleKey);
                         }
-                        // Add if declared in any parent styles object
-                        Object.keys(node.style ?? {}).forEach((styleKey) => {
+                    });
+                    Object.values(node.variants ?? {}).forEach((variant) => {
+                        Object.keys(variant.style ?? {}).forEach((styleKey) => {
                             if (styleKey.startsWith('--')) {
-                                cssVariableKeys.add(styleKey);
+                                vars.add(styleKey);
                             }
                         });
-                        Object.values(node.variants ?? {}).forEach((variant) => {
-                            Object.keys(variant.style ?? {}).forEach((styleKey) => {
-                                if (styleKey.startsWith('--')) {
-                                    cssVariableKeys.add(styleKey);
-                                }
-                            });
-                        });
-                    }
-                });
-            });
-            return cssVariableKeys;
+                    });
+                }
+                const parent = Object.entries(component.nodes ?? {}).find(([_, n]) => n.children?.includes(nodeName));
+                if (parent) {
+                    visitVars(parent[0]);
+                }
+            };
+            visitVars(nodeName.toString());
+            return vars;
         });
         for (const varName of vars) {
-            if (allCssVariableDeclarations.has(varName) === false) {
+            if (!(varName in themeCssVariables) && !localCssVariables.has(varName)) {
                 report({
                     path,
                     info: {
                         title: `Unknown CSS variable`,
-                        description: `The CSS variable **${varName}** is not declared in any parent element or in your theme. The CSS variable must be declared in an ancestor element in its component or in your global theme.`,
+                        description: `The CSS variable **${varName}** is not declared in any parent element or theme. CSS variables must be declared in an ancestor element or in your global theme.`,
                     },
                     details: { name: varName },
+                    fixes: ['add-to-theme'],
                 });
             }
         }
     },
+    fixes: {
+        'add-to-theme': addToThemeFix,
+    },
 };
+function addToThemeFix(args) {
+    const varName = args.details?.name;
+    if (typeof varName !== 'string' || !args.data.files.themes?.Default) {
+        return args.data.files;
+    }
+    const definition = {
+        syntax: {
+            type: 'primitive',
+            name: '*',
+        },
+        inherits: true,
+        initialValue: '',
+        values: {},
+        description: '',
+    };
+    // Add the variable to the theme with Any (*) syntax type
+    return {
+        ...args.data.files,
+        themes: {
+            ...args.data.files.themes,
+            Default: {
+                ...args.data.files.themes.Default,
+                propertyDefinitions: {
+                    ...args.data.files.themes.Default.propertyDefinitions,
+                    [varName]: definition,
+                },
+            },
+        },
+    };
+}
 //# sourceMappingURL=unknownCSSVariable.js.map

package/dist/rules/issues/style/unknownCSSVariable.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"unknownCSSVariable.js","sourceRoot":"","sources":["../../../../src/rules/issues/style/unknownCSSVariable.ts"],"names":[],"mappings":"~~AAEA~~,MAAM,KAAK,GAAG,qBAAqB,CAAA;AAEnC,MAAM,CAAC,MAAM,sBAAsB,~~GAE9B~~;~~IACH~~,IAAI,EAAE,sBAAsB;IAC5B,KAAK,EAAE,SAAS;IAChB,QAAQ,EAAE,mBAAmB;IAC7B,KAAK,EAAE,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC;QACzD,IACE,QAAQ,KAAK,mBAAmB;YAChC,OAAO,KAAK,CAAC,UAAU,KAAK,QAAQ,EACpC,CAAC;YACD,OAAM;QACR,CAAC;QAED,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,EAAE,OAAO,CAAA;QACnC,yDAAyD;QACzD,IAAI,KAAK,EAAE,mBAAmB,KAAK,SAAS,EAAE,CAAC;YAC7C,OAAM;QACR,CAAC;QAED,kCAAkC;QAClC,MAAM,IAAI,GAAG,CAAC,GAAG,KAAK,CAAC,UAAU,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,CAC/D,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,EAAE,EAAE,CAAC,OAAO,CAC1B,CAAA;QACD,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACtB,OAAM;QACR,CAAC;QAED,MAAM,~~0BAA0B~~,GAAG,~~IAAI~~,~~CACrC~~,~~gCAAgC~~,~~EAChC~~,~~GAAG~~,EAAE,~~CAAC;YACJ~~,MAAM,~~eAAe~~,GAAG,IAAI,~~GAAG~~,~~CAC7B,~~MAAM,~~CAAC~~,IAAI,~~CAAC~~,~~KAAK~~,~~CAAC~~,~~mBAAmB,~~IAAI,EAAE,~~CAAC~~,~~CAC7C~~,~~CAAA;YACD~~,~~MAAM,~~CAAC,MAAM,~~CAAC~~,~~KAAK~~,~~CAAC~~,~~UAAU~~,~~IAAI~~,~~EAAE~~,~~CAAC~~,~~CAAC~~,~~OAAO~~,~~CAAC~~,CAAC,~~SAAS~~,~~EAAE~~,~~EAAE~~,CAAC;~~gBAC3D~~,~~MAAM,CAAC,~~IAAI,CAAC,SAAS,EAAE,~~KAAK~~,IAAI,~~EAAE~~,CAAC,~~CAAC~~,~~OAAO~~,~~CAAC~~,CAAC,~~OAAO~~,EAAE,EAAE,CAAC;~~oBACvD~~,MAAM,IAAI,GAAG,SAAS,EAAE,KAAK,EAAE,CAAC,~~OAAO~~,CAAC,CAAA;~~oBACxC~~,IAAI,CAAC,IAAI,EAAE,CAAC;~~wBACV~~,OAAM;~~oBACR~~,CAAC;~~oBAED~~,IAAI,IAAI,CAAC,IAAI,KAAK,WAAW,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;~~wBACzD~~,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;~~4BAC5D~~,~~eAAe~~,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;~~wBAAA~~,~~CAC7B~~,CAAC,CAAA;~~wBACF~~,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;~~4BACtD~~,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,OAAO,~~CACjD~~,CAAC,OAAO,EAAE,EAAE,CAAC;~~gCACX~~,~~eAAe~~,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;~~4BAAA~~,~~CAC7B~~,~~CACF~~,CAAA;~~wBAAA~~,~~CACF~~,CAAC,CAAA;~~wBAEF~~,kCAAkC;~~wBAClC~~,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,IAAI,IAAI,CAAC,iBAAiB,CAAC,EAAE,CAAC;~~4BACvD~~,IAAI,CAAC,iBAAiB,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC;~~gCAC5C~~,~~eAAe~~,CAAC,GAAG,CAAC,KAAK,QAAQ,CAAC,IAAI,EAAE,CAAC,CAAA;~~4BAAA~~,~~CAC1C~~,CAAC,CAAA;~~wBACJ~~,CAAC;~~wBAED~~,8CAA8C;~~wBAC9C~~,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC;~~4BAClD~~,IAAI,QAAQ,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;~~gCAC9B~~,~~eAAe~~,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAA;~~4BAC/B~~,CAAC;~~wBAAA~~,CACF,CAAC,CAAA;~~wBACF~~,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;~~4BACtD~~,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC;~~gCACrD~~,IAAI,QAAQ,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;~~oCAC9B~~,~~eAAe~~,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAA;~~gCAC/B~~,CAAC;~~4BAAA~~,CACF,CAAC,CAAA;~~wBAAA~~,CACH,CAAC,CAAA;~~oBACJ~~,CAAC;~~gBAAA~~,~~CACF~~,CAAC,CAAA;YAAA,~~CACH~~,CAAC,CAAA;~~YACF~~,OAAO,~~eAAe~~,CAAA;QAAA,~~CACvB~~,CACF,CAAA;QAED,KAAK,MAAM,OAAO,IAAI,IAAI,EAAE,CAAC;YAC3B,IAAI,~~0BAA0B~~,CAAC,GAAG,CAAC,OAAO,CAAC,~~KAAK,KAAK,~~EAAE,CAAC;~~gBACtD~~,MAAM,CAAC;oBACL,IAAI;oBACJ,IAAI,EAAE;wBACJ,KAAK,EAAE,sBAAsB;wBAC7B,WAAW,EAAE,sBAAsB,OAAO~~,+JAA+J~~;~~qBAC1M~~;oBACD,OAAO,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE;~~iBAC3B~~,CAAC,CAAA;YACJ,CAAC;QACH,CAAC;IAAA,CACF;CACF,CAAA"}
1	+ {"version":3,"file":"unknownCSSVariable.js","sourceRoot":"","sources":["../../../../src/rules/issues/style/unknownCSSVariable.ts"],"names":[],"mappings":"AAGA,MAAM,KAAK,GAAG,qBAAqB,CAAA;AAEnC,MAAM,CAAC,MAAM,sBAAsB,GAK/B;IACF,IAAI,EAAE,sBAAsB;IAC5B,KAAK,EAAE,SAAS;IAChB,QAAQ,EAAE,mBAAmB;IAC7B,KAAK,EAAE,CAAC,MAAM,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC;QACzD,IACE,QAAQ,KAAK,mBAAmB;YAChC,OAAO,KAAK,CAAC,UAAU,KAAK,QAAQ,EACpC,CAAC;YACD,OAAM;QACR,CAAC;QAED,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,EAAE,OAAO,CAAA;QACnC,yDAAyD;QACzD,IAAI,KAAK,EAAE,mBAAmB,KAAK,SAAS,EAAE,CAAC;YAC7C,OAAM;QACR,CAAC;QAED,kCAAkC;QAClC,MAAM,IAAI,GAAG,CAAC,GAAG,KAAK,CAAC,UAAU,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,CAC/D,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,EAAE,EAAE,CAAC,OAAO,CAC1B,CAAA;QACD,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACtB,OAAM;QACR,CAAC;QAED,MAAM,iBAAiB,GAAG,KAAK,CAAC,mBAAmB,CAAA;QACnD,MAAM,CAAC,SAAS,EAAE,aAAa,EAAE,MAAM,EAAE,QAAQ,CAAC,GAAG,IAAI,CAAA;QACzD,MAAM,iBAAiB,GAAG,IAAI,CAC5B,2BAA2B,aAAa,IAAI,QAAQ,EAAE,EACtD,GAAG,EAAE,CAAC;YACJ,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAA;YAC9B,MAAM,SAAS,GAAG,KAAK,CAAC,UAAU,CAAC,aAAa,CAAC,CAAA;YACjD,IAAI,CAAC,SAAS,EAAE,CAAC;gBACf,OAAO,IAAI,CAAA;YACb,CAAC;YAED,MAAM,SAAS,GAAG,CAAC,QAAgB,EAAE,EAAE,CAAC;gBACtC,MAAM,IAAI,GAAG,SAAS,EAAE,KAAK,EAAE,CAAC,QAAQ,CAAC,CAAA;gBACzC,IAAI,CAAC,IAAI,EAAE,CAAC;oBACV,OAAM;gBACR,CAAC;gBAED,IAAI,IAAI,CAAC,IAAI,KAAK,WAAW,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;oBACzD,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;wBAC5D,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;oBAAA,CAClB,CAAC,CAAA;oBACF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;wBACtD,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;4BAC/D,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAA;wBAAA,CAClB,CAAC,CAAA;oBAAA,CACH,CAAC,CAAA;oBAEF,kCAAkC;oBAClC,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,IAAI,IAAI,CAAC,iBAAiB,CAAC,EAAE,CAAC;wBACvD,IAAI,CAAC,iBAAiB,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC;4BAC5C,IAAI,CAAC,GAAG,CAAC,KAAK,QAAQ,CAAC,IAAI,EAAE,CAAC,CAAA;wBAAA,CAC/B,CAAC,CAAA;oBACJ,CAAC;oBAED,8CAA8C;oBAC9C,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC;wBAClD,IAAI,QAAQ,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;4BAC9B,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAA;wBACpB,CAAC;oBAAA,CACF,CAAC,CAAA;oBACF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;wBACtD,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC;4BACrD,IAAI,QAAQ,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;gCAC9B,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAA;4BACpB,CAAC;wBAAA,CACF,CAAC,CAAA;oBAAA,CACH,CAAC,CAAA;gBACJ,CAAC;gBAED,MAAM,MAAM,GAAG,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CACnE,CAAC,CAAC,QAAQ,EAAE,QAAQ,CAAC,QAAQ,CAAC,CAC/B,CAAA;gBACD,IAAI,MAAM,EAAE,CAAC;oBACX,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAA;gBACtB,CAAC;YAAA,CACF,CAAA;YAED,SAAS,CAAC,QAAQ,CAAC,QAAQ,EAAE,CAAC,CAAA;YAC9B,OAAO,IAAI,CAAA;QAAA,CACZ,CACF,CAAA;QAED,KAAK,MAAM,OAAO,IAAI,IAAI,EAAE,CAAC;YAC3B,IAAI,CAAC,CAAC,OAAO,IAAI,iBAAiB,CAAC,IAAI,CAAC,iBAAiB,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;gBACvE,MAAM,CAAC;oBACL,IAAI;oBACJ,IAAI,EAAE;wBACJ,KAAK,EAAE,sBAAsB;wBAC7B,WAAW,EAAE,sBAAsB,OAAO,mIAAmI;qBAC9K;oBACD,OAAO,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE;oBAC1B,KAAK,EAAE,CAAC,cAAc,CAAC;iBACxB,CAAC,CAAA;YACJ,CAAC;QACH,CAAC;IAAA,CACF;IACD,KAAK,EAAE;QACL,cAAc,EAAE,aAAa;KAC9B;CACF,CAAA;AAID,SAAS,aAAa,CACpB,IAA6D,EACP;IACtD,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,EAAE,IAAI,CAAA;IAClC,IAAI,OAAO,OAAO,KAAK,QAAQ,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,OAAO,EAAE,CAAC;QACpE,OAAO,IAAI,CAAC,IAAI,CAAC,KAAK,CAAA;IACxB,CAAC;IAED,MAAM,UAAU,GAA6B;QAC3C,MAAM,EAAE;YACN,IAAI,EAAE,WAAW;YACjB,IAAI,EAAE,GAAG;SACV;QACD,QAAQ,EAAE,IAAI;QACd,YAAY,EAAE,EAAE;QAChB,MAAM,EAAE,EAAE;QACV,WAAW,EAAE,EAAE;KACP,CAAA;IAEV,yDAAyD;IACzD,OAAO;QACL,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK;QAClB,MAAM,EAAE;YACN,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM;YACzB,OAAO,EAAE;gBACP,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,OAAO;gBACjC,mBAAmB,EAAE;oBACnB,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,OAAO,CAAC,mBAAmB;oBACrD,CAAC,OAAO,CAAC,EAAE,UAAU;iBACtB;aACF;SACF;KACF,CAAA;AAAA,CACF"}

package/dist/types.d.ts CHANGED Viewed

@@ -29,6 +29,7 @@ import type { NoStaticNodeConditionRuleFix } from './rules/issues/logic/noStatic
 import type { NoReferenceNodeRuleFix } from './rules/issues/miscellaneous/noReferenceNodeRule';
 import type { InvalidStyleSyntaxRuleFix } from './rules/issues/style/invalidStyleSyntaxRule';
 import type { LegacyStyleVariableRuleFix } from './rules/issues/style/legacyStyleVariableRule';
+import type { AddToThemeFix } from './rules/issues/style/unknownCSSVariable';
 import type { NoReferenceVariableRuleFix } from './rules/issues/variables/noReferenceVariableRule';
 import type { NoPostNavigateActionRuleFix } from './rules/issues/workflows/noPostNavigateAction';
 export type Code = 'duplicate action argument name' | 'duplicate event trigger' | 'duplicate formula argument name' | 'duplicate route' | 'duplicate url parameter' | 'duplicate workflow parameter' | 'image without dimension' | 'invalid api parser mode' | 'invalid api proxy body setting' | 'invalid api proxy cookie setting' | 'invalid component structure' | 'invalid element child' | 'invalid style syntax' | 'legacy action' | 'legacy api' | 'legacy formula' | 'legacy style variable' | 'legacy theme' | 'no context consumers' | 'no post navigate action' | 'no-console' | 'no-reference api input' | 'no-reference api service' | 'no-reference api' | 'no-reference attribute in instance' | 'no-reference attribute' | 'no-reference component formula' | 'no-reference component workflow' | 'no-reference component' | 'no-reference event' | 'no-reference global css variable' | 'no-reference node' | 'no-reference project action' | 'no-reference project formula' | 'no-reference variable' | 'no-static-node-condition' | 'no-unnecessary-condition-falsy' | 'no-unnecessary-condition-truthy' | 'non-empty void element' | 'required api autofetch' | 'required direct child' | 'required direct parent' | 'required element attribute' | 'required extension' | 'required meta tag' | 'size constraint' | 'unknown action argument' | 'unknown action event' | 'unknown api input' | 'unknown api service' | 'unknown api' | 'unknown attribute' | 'unknown classname' | 'unknown component attribute' | 'unknown component formula input' | 'unknown component slot' | 'unknown component' | 'unknown context formula' | 'unknown context provider formula' | 'unknown context provider workflow' | 'unknown context provider' | 'unknown context workflow' | 'unknown cookie' | 'unknown css variable' | 'unknown event' | 'unknown fetch input' | 'unknown formula' | 'unknown project action' | 'unknown project formula input' | 'unknown project formula' | 'unknown repeat index formula' | 'unknown repeat item formula' | 'unknown set url parameter' | 'unknown set url parameters' | 'unknown trigger event' | 'unknown trigger workflow parameter' | 'unknown trigger workflow' | 'unknown url parameter' | 'unknown variable setter' | 'unknown variable' | 'unknown workflow parameter';
@@ -261,7 +262,7 @@ export type StyleNode = {
     };
 } & Base;
 export type NodeType = ActionModelNode | ComponentAPIInputNode | ComponentAPINode | ComponentAttributeNode | ComponentContext | ComponentEvent | ComponentFormulaNode | ComponentNode | ComponentNodeAttributeNode | ComponentNodeNode | ComponentVariableNode | ComponentWorkflowNode | CustomActionModelArgumentNode | CustomActionModelEventNode | CustomPropertyNode | FormulaNode | ProjectActionNode | ProjectApiService | ProjectConfigNode | ProjectFormulaNode | ProjectRoute | ProjectThemeNode | ProjectThemePropertyNode | StyleNode | StyleVariableNode | StyleVariantNode;
-export type FixType = ChangeDataTypeFix | DeleteFetchInputFix | InvalidStyleSyntaxRuleFix | LegacyActionRuleFix | LegacyFormulaRuleFix | LegacyStyleVariableRuleFix | NoPostNavigateActionRuleFix | NoReferenceApiRuleFix | NoReferenceApiServiceRuleFix | NoReferenceAttributeRuleFix | NoReferenceComponentFormulaRuleFix | NoReferenceComponentRuleFix | NoReferenceEventRuleFix | NoReferenceNodeRuleFix | NoReferenceProjectActionRuleFix | NoReferenceProjectFormulaRuleFix | NoReferenceVariableRuleFix | NoStaticNodeConditionRuleFix | UnknownActionArgumentRuleFix | UnknownActionEventRuleFix | UnknownApiServiceRuleFix | UnknownComponentAttributeRuleFix;
+export type FixType = AddToThemeFix | ChangeDataTypeFix | DeleteFetchInputFix | InvalidStyleSyntaxRuleFix | LegacyActionRuleFix | LegacyFormulaRuleFix | LegacyStyleVariableRuleFix | NoPostNavigateActionRuleFix | NoReferenceApiRuleFix | NoReferenceApiServiceRuleFix | NoReferenceAttributeRuleFix | NoReferenceComponentFormulaRuleFix | NoReferenceComponentRuleFix | NoReferenceEventRuleFix | NoReferenceNodeRuleFix | NoReferenceProjectActionRuleFix | NoReferenceProjectFormulaRuleFix | NoReferenceVariableRuleFix | NoStaticNodeConditionRuleFix | UnknownActionArgumentRuleFix | UnknownActionEventRuleFix | UnknownApiServiceRuleFix | UnknownComponentAttributeRuleFix;
 interface ReportedIssueInfo {
     title: string;
     description: string;

package/package.json CHANGED Viewed

@@ -10,8 +10,8 @@
 "directory": "packages/search"
 },
 "dependencies": {
-"@nordcraft/ssr": "1.0.87",
-"@nordcraft/core": "1.0.87",
+"@nordcraft/ssr": "1.0.89",
+"@nordcraft/core": "1.0.89",
 "jsondiffpatch": "0.7.3",
 "postcss": "8.5.6",
 "zod": "4.2.1"
@@ -26,5 +26,5 @@
 "test:watch:only": "bun test --watch --only"
 },
 "files": ["dist", "src"],
-"version": "1.0.87"
+"version": "1.0.89"
 }

package/src/rules/issues/actions/unknownActionEventRule.test.ts CHANGED Viewed

@@ -1,4 +1,3 @@
-/* eslint-disable inclusive-language/use-inclusive-words */
 import type { CustomActionModel } from '@nordcraft/core/dist/component/component.types'
 import { describe, expect, test } from 'bun:test'
 import { fixProject } from '../../../fixProject'

package/src/rules/issues/dom/nonEmptyVoidElementRule.ts CHANGED Viewed

@@ -1,4 +1,4 @@
-import { VOID_HTML_ELEMENTS } from '@nordcraft/ssr/dist/const'
+import { VOID_HTML_ELEMENTS } from '@nordcraft/core/dist/utils/html'
 import type { Rule } from '../../../types'
 /**
  * See full list here

package/src/rules/issues/events/noReferenceEventRule.test.ts CHANGED Viewed

@@ -42,7 +42,6 @@ describe('find noReferenceEventRule', () => {
               events: [
                 {
                   name: 'unused-event',
-                  // eslint-disable-next-line inclusive-language/use-inclusive-words
                   dummyEvent: {
                     name: 'Name',
                   },
@@ -120,7 +119,6 @@ describe('find noReferenceEventRule', () => {
               events: [
                 {
                   name: 'known-event',
-                  // eslint-disable-next-line inclusive-language/use-inclusive-words
                   dummyEvent: {
                     name: 'Name',
                   },
@@ -130,7 +128,6 @@ describe('find noReferenceEventRule', () => {
                 },
                 {
                   name: 'used-event',
-                  // eslint-disable-next-line inclusive-language/use-inclusive-words
                   dummyEvent: {
                     name: 'Name',
                   },
@@ -184,7 +181,6 @@ describe('fix noReferenceEventRule', () => {
           events: [
             {
               name: 'unused-event',
-              // eslint-disable-next-line inclusive-language/use-inclusive-words
               dummyEvent: {
                 name: 'Name',
               },

package/src/rules/issues/events/unknownEventRule.test.ts CHANGED Viewed

@@ -66,7 +66,6 @@ describe('unknownEvent', () => {
               events: [
                 {
                   name: 'known-event',
-                  // eslint-disable-next-line inclusive-language/use-inclusive-words
                   dummyEvent: {
                     name: 'Name',
                   },
@@ -94,7 +93,6 @@ describe('unknownEvent', () => {
                   events: [
                     {
                       name: 'known-event',
-                      // eslint-disable-next-line inclusive-language/use-inclusive-words
                       dummyEvent: {
                         name: 'Name',
                       },
@@ -187,7 +185,6 @@ describe('unknownEvent', () => {
               events: [
                 {
                   name: 'known-event',
-                  // eslint-disable-next-line inclusive-language/use-inclusive-words
                   dummyEvent: {
                     name: 'Name',
                   },
@@ -215,7 +212,6 @@ describe('unknownEvent', () => {
                   events: [
                     {
                       name: 'known-event',
-                      // eslint-disable-next-line inclusive-language/use-inclusive-words
                       dummyEvent: {
                         name: 'Name',
                       },

package/src/rules/issues/events/unknownTriggerEventRule.test.ts CHANGED Viewed

@@ -40,7 +40,6 @@ describe('unknownTriggerEventRule', () => {
               events: [
                 {
                   name: 'known-event',
-                  // eslint-disable-next-line inclusive-language/use-inclusive-words
                   dummyEvent: {
                     name: 'Name',
                   },
@@ -97,7 +96,6 @@ describe('unknownTriggerEventRule', () => {
               events: [
                 {
                   name: 'known-event',
-                  // eslint-disable-next-line inclusive-language/use-inclusive-words
                   dummyEvent: {
                     name: 'Name',
                   },

package/src/rules/issues/miscellaneous/createStaticSizeConstraintRule.ts CHANGED Viewed

@@ -1,6 +1,6 @@
 import type { NodeModel } from '@nordcraft/core/dist/component/component.types'
+import { VOID_HTML_ELEMENTS } from '@nordcraft/core/dist/utils/html'
 import { isDefined } from '@nordcraft/core/dist/utils/util'
-import { VOID_HTML_ELEMENTS } from '@nordcraft/ssr/dist/const'
 import type { Level, Rule } from '../../../types'
 export function createStaticSizeConstraintRule(

package/src/rules/issues/style/invalidStyleSyntaxRule.test.ts CHANGED Viewed

@@ -203,6 +203,298 @@ describe('find invalidStyleSyntaxRule', () => {
   })
 })
+describe('find formulas in style syntax', () => {
+  test('should find Variables. references in style syntax', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          formulas: {},
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  tag: 'div',
+                  type: 'element',
+                  attrs: {},
+                  style: {
+                    transform: 'translateX(Variables.offsetX)',
+                    width: '100px',
+                    height: 'Variables.height',
+                  },
+                  events: {},
+                  classes: {},
+                  children: [],
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [invalidStyleSyntaxRule],
+      }),
+    )
+    expect(problems).toMatchObject([
+      {
+        code: 'invalid style syntax',
+        path: ['components', 'test', 'nodes', 'root', 'style', 'transform'],
+        details: { property: 'transform' },
+      },
+      {
+        code: 'invalid style syntax',
+        path: ['components', 'test', 'nodes', 'root', 'style', 'height'],
+        details: { property: 'height' },
+      },
+    ])
+  })
+  test('should find Formulas., Event., Attributes., Apis., Parameters., ListItem., URLParameters. references', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          formulas: {},
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  tag: 'div',
+                  type: 'element',
+                  attrs: {},
+                  style: {
+                    color: 'Formulas.getColor()',
+                    top: 'Event.clientY',
+                    margin: 'Attributes.margin',
+                    background: 'Apis.getBackground()',
+                    fontSize: 'Parameters.size',
+                    backgroundColor: 'ListItem.color',
+                    width: 'URLParameters.width',
+                  },
+                  events: {},
+                  classes: {},
+                  children: [],
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [invalidStyleSyntaxRule],
+      }),
+    )
+    expect(problems).toHaveLength(7)
+    expect(problems.map((p) => p.details.property)).toEqual([
+      'color',
+      'top',
+      'margin',
+      'background',
+      'fontSize',
+      'backgroundColor',
+      'width',
+    ])
+  })
+  test('should not find formulas in valid CSS values', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          formulas: {},
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  tag: 'div',
+                  type: 'element',
+                  attrs: {},
+                  style: {
+                    width: '100px',
+                    height: '50%',
+                    color: '#ffffff',
+                    backgroundColor: 'var(--my-var)',
+                    transform: 'translateX(10px)',
+                    margin: '10px 20px',
+                  },
+                  events: {},
+                  classes: {},
+                  children: [],
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [invalidStyleSyntaxRule],
+      }),
+    )
+    expect(problems).toHaveLength(0)
+  })
+  test('should not find formulas in numeric style values', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          formulas: {},
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  tag: 'div',
+                  type: 'element',
+                  attrs: {},
+                  style: {
+                    opacity: 0.5,
+                    zIndex: 10,
+                    flex: 1,
+                  },
+                  events: {},
+                  classes: {},
+                  children: [],
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [invalidStyleSyntaxRule],
+      }),
+    )
+    expect(problems).toHaveLength(0)
+  })
+  test('should find formulas in variant styles', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          formulas: {},
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  tag: 'div',
+                  type: 'element',
+                  attrs: {},
+                  style: {
+                    width: '100px',
+                  },
+                  events: {},
+                  classes: {},
+                  children: [],
+                  variants: [
+                    {
+                      style: {
+                        transform: 'translateX(Variables.offsetX)',
+                        color: 'Event.color',
+                      },
+                      hover: true,
+                    },
+                  ],
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [invalidStyleSyntaxRule],
+      }),
+    )
+    expect(problems).toMatchObject([
+      {
+        code: 'invalid style syntax',
+        path: [
+          'components',
+          'test',
+          'nodes',
+          'root',
+          'variants',
+          0,
+          'style',
+          'transform',
+        ],
+        details: { property: 'transform' },
+      },
+      {
+        code: 'invalid style syntax',
+        path: [
+          'components',
+          'test',
+          'nodes',
+          'root',
+          'variants',
+          0,
+          'style',
+          'color',
+        ],
+        details: { property: 'color' },
+      },
+    ])
+  })
+  test('should handle case-insensitive matching for formulas', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          formulas: {},
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  tag: 'div',
+                  type: 'element',
+                  attrs: {},
+                  style: {
+                    width: 'VARIABLES.offsetX',
+                    height: 'variables.height',
+                    color: 'FORMULAS.getColor()',
+                  },
+                  events: {},
+                  classes: {},
+                  children: [],
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [invalidStyleSyntaxRule],
+      }),
+    )
+    expect(problems).toHaveLength(3)
+    expect(problems.map((p) => p.details.property)).toEqual([
+      'width',
+      'height',
+      'color',
+    ])
+  })
+})
 describe('fix invalidStyleSyntaxRule', () => {
   test('should remove an invalid style property', () => {
     const files: ProjectFiles = {
@@ -248,6 +540,49 @@ describe('fix invalidStyleSyntaxRule', () => {
       }
     `)
   })
+  test('should remove a style property with formula syntax', () => {
+    const files: ProjectFiles = {
+      formulas: {},
+      components: {
+        test: {
+          name: 'test',
+          nodes: {
+            root: {
+              tag: 'div',
+              type: 'element',
+              attrs: {},
+              style: {
+                width: '100px',
+                transform: 'translateX(Variables.offsetX)',
+                height: '50px',
+                color: 'Variables.color',
+              },
+              events: {},
+              classes: {},
+              children: [],
+            },
+          },
+          formulas: {},
+          apis: {},
+          attributes: {},
+          variables: {},
+        },
+      },
+    }
+    const fixedFiles = fixProject({
+      files,
+      rule: invalidStyleSyntaxRule,
+      fixType: 'delete-style-property',
+    })
+    expect((fixedFiles.components.test!.nodes?.root as ElementNodeModel).style)
+      .toMatchInlineSnapshot(`
+      {
+        "height": "50px",
+        "width": "100px",
+      }
+    `)
+  })
   test('should remove an invalid style variant style property', () => {
     const files: ProjectFiles = {
       formulas: {},
@@ -300,4 +635,55 @@ describe('fix invalidStyleSyntaxRule', () => {
       }
     `)
   })
+  test('should remove a variant style property with formula syntax', () => {
+    const files: ProjectFiles = {
+      formulas: {},
+      components: {
+        test: {
+          name: 'test',
+          nodes: {
+            root: {
+              tag: 'div',
+              type: 'element',
+              attrs: {},
+              style: {},
+              events: {},
+              classes: {},
+              children: [],
+              variants: [
+                {
+                  style: {
+                    width: '100px',
+                    transform: 'translateX(Variables.offsetX)',
+                    height: '50px',
+                    color: 'Event.color',
+                  },
+                  hover: true,
+                },
+              ],
+            },
+          },
+          formulas: {},
+          apis: {},
+          attributes: {},
+          variables: {},
+        },
+      },
+    }
+    const fixedFiles = fixProject({
+      files,
+      rule: invalidStyleSyntaxRule,
+      fixType: 'delete-style-property',
+    })
+    expect(
+      (fixedFiles.components.test!.nodes?.root as ElementNodeModel)
+        .variants?.[0].style,
+    ).toMatchInlineSnapshot(`
+      {
+        "height": "50px",
+        "width": "100px",
+      }
+    `)
+  })
 })

package/src/rules/issues/style/invalidStyleSyntaxRule.ts CHANGED Viewed

@@ -12,6 +12,27 @@ export const invalidStyleSyntaxRule: Rule<{
     if (nodeType !== 'style-declaration') {
       return
     }
+    // Check for variable/formula references: Variables., Formulas., Event., Attributes., Apis., Parameters., ListItem., URLParameters.
+    if (typeof value.styleValue === 'string') {
+      const hasVariableReference =
+        /\b(Variables|Formulas|Event|Attributes|Apis|Parameters|ListItem|URLParameters)\.\w+/i.test(
+          value.styleValue,
+        )
+      if (hasVariableReference) {
+        report({
+          path,
+          info: {
+            title: `Formulas detected in style declaration`,
+            description: `The style declaration for the property "${value.styleProperty}" contains Nordcraft formula syntax (e.g., references like "Variables.xxx", "Event.xxx", "Attributes.xxx", etc.). Formulas should not be used directly in CSS style values. Use style-variables or computed styles instead.`,
+          },
+          details: { property: value.styleProperty },
+          fixes: ['delete-style-property'],
+        })
+        return
+      }
+    }
     const valid = memo(
       `valid-style-${value.styleProperty}:${value.styleValue}`,
       () => {

package/src/rules/issues/style/unknownCSSVariable.test.ts CHANGED Viewed

@@ -1,4 +1,6 @@
+import type { ProjectFiles } from '@nordcraft/ssr/dist/src/ssr.types'
 import { describe, expect, test } from 'bun:test'
+import { fixProject } from '../../../fixProject'
 import { searchProject } from '../../../searchProject'
 import { unknownCSSVariableRule } from './unknownCSSVariable'
@@ -171,7 +173,7 @@ describe('unknownCSSVariableRule', () => {
     expect(problems[1].details).toEqual({ name: '--undefined-local' })
   })
-  test('should not report when legacy style variables are used', () => {
+  test('should report problem if local variable is used in parent but only defined in child', () => {
     const problems = Array.from(
       searchProject({
         files: {
@@ -190,19 +192,26 @@ describe('unknownCSSVariableRule', () => {
                   tag: 'div',
                   type: 'element',
                   attrs: {},
-                  'style-variables': [
-                    {
-                      name: 'legacy-variable',
+                  style: {
+                    color: 'var(--child-only-variable)',
+                  },
+                  events: {},
+                  classes: {},
+                  children: ['child'],
+                },
+                child: {
+                  tag: 'div',
+                  type: 'element',
+                  attrs: {},
+                  customProperties: {
+                    '--child-only-variable': {
                       formula: {
                         type: 'value',
-                        value: '10px',
+                        value: 'red',
                       },
-                      category: 'spacing',
                     },
-                  ],
-                  style: {
-                    margin: 'var(--legacy-variable)',
                   },
+                  style: {},
                   events: {},
                   classes: {},
                   children: [],
@@ -219,10 +228,10 @@ describe('unknownCSSVariableRule', () => {
       }),
     )
-    expect(problems).toBeEmpty()
+    expect(problems).toHaveLength(1)
   })
-  test('should not report when declared in a style', () => {
+  test('should not report when legacy style variables are used', () => {
     const problems = Array.from(
       searchProject({
         files: {
@@ -241,9 +250,18 @@ describe('unknownCSSVariableRule', () => {
                   tag: 'div',
                   type: 'element',
                   attrs: {},
+                  'style-variables': [
+                    {
+                      name: 'legacy-variable',
+                      formula: {
+                        type: 'value',
+                        value: '10px',
+                      },
+                      category: 'spacing',
+                    },
+                  ],
                   style: {
-                    '--declared-in-style': '10px',
-                    margin: 'var(--declared-in-style)',
+                    margin: 'var(--legacy-variable)',
                   },
                   events: {},
                   classes: {},
@@ -264,13 +282,14 @@ describe('unknownCSSVariableRule', () => {
     expect(problems).toBeEmpty()
   })
-  test('should not report when a legacy theme exists (no propertyDefinitions)', () => {
+  test('should not report when declared in a style', () => {
     const problems = Array.from(
       searchProject({
         files: {
           themes: {
             Default: {
               fonts: [],
+              propertyDefinitions: {},
             },
           },
           formulas: {},
@@ -283,7 +302,8 @@ describe('unknownCSSVariableRule', () => {
                   type: 'element',
                   attrs: {},
                   style: {
-                    color: 'var(--legacy-theme-variable)',
+                    '--declared-in-style': '10px',
+                    margin: 'var(--declared-in-style)',
                   },
                   events: {},
                   classes: {},
@@ -304,32 +324,26 @@ describe('unknownCSSVariableRule', () => {
     expect(problems).toBeEmpty()
   })
-  test('should not report if CSS variable is defined in any other component or self', () => {
+  test('should not report when a legacy theme exists (no propertyDefinitions)', () => {
     const problems = Array.from(
       searchProject({
         files: {
           themes: {
             Default: {
               fonts: [],
-              propertyDefinitions: {},
             },
           },
           formulas: {},
           components: {
-            componentA: {
-              name: 'componentA',
+            test: {
+              name: 'test',
               nodes: {
                 root: {
                   tag: 'div',
                   type: 'element',
-                  customProperties: {
-                    '--variable-in-a': {
-                      formula: { type: 'value', value: 'red' },
-                    },
-                  },
                   attrs: {},
                   style: {
-                    color: 'var(--variable-in-a)',
+                    color: 'var(--legacy-theme-variable)',
                   },
                   events: {},
                   classes: {},
@@ -341,32 +355,81 @@ describe('unknownCSSVariableRule', () => {
               attributes: {},
               variables: {},
             },
-            componentB: {
-              name: 'componentB',
-              nodes: {
-                root: {
-                  tag: 'div',
-                  type: 'element',
-                  attrs: {},
-                  style: {
-                    color: 'var(--variable-in-a)',
-                  },
-                  events: {},
-                  classes: {},
-                  children: [],
-                },
+          },
+        },
+        rules: [unknownCSSVariableRule],
+      }),
+    )
+    expect(problems).toBeEmpty()
+  })
+  test('fix should add variable to theme', () => {
+    const files: ProjectFiles = {
+      themes: {
+        Default: {
+          fonts: [],
+          propertyDefinitions: {},
+        },
+      },
+      formulas: {},
+      components: {
+        test: {
+          name: 'test',
+          nodes: {
+            root: {
+              tag: 'div',
+              type: 'element',
+              attrs: {},
+              style: {
+                color: 'var(--new-theme-variable)',
               },
-              formulas: {},
-              apis: {},
-              attributes: {},
-              variables: {},
+              events: {},
+              classes: {},
+              children: [],
             },
           },
+          formulas: {},
+          apis: {},
+          attributes: {},
+          variables: {},
         },
+      },
+    }
+    const problems = Array.from(
+      searchProject({
+        files,
         rules: [unknownCSSVariableRule],
       }),
     )
-    expect(problems).toBeEmpty()
+    expect(problems).toHaveLength(1)
+    const fixedProject = fixProject({
+      files,
+      rule: unknownCSSVariableRule,
+      fixType: 'add-to-theme',
+    })
+    expect(
+      Object.keys(fixedProject.themes?.Default.propertyDefinitions ?? {}),
+    ).toHaveLength(1)
+    expect(fixedProject.themes?.Default.propertyDefinitions).toHaveProperty(
+      '--new-theme-variable',
+    )
+    expect(
+      fixedProject.themes?.Default.propertyDefinitions?.[
+        '--new-theme-variable'
+      ],
+    ).toEqual({
+      syntax: {
+        type: 'primitive',
+        name: '*',
+      },
+      inherits: true,
+      initialValue: '',
+      values: {},
+      description: '',
+    })
   })
 })

package/src/rules/issues/style/unknownCSSVariable.ts CHANGED Viewed

@@ -1,10 +1,14 @@
-import type { Rule } from '../../../types'
+import type { CustomPropertyDefinition } from '@nordcraft/core/dist/styling/theme'
+import type { FixFunction, Rule, StyleNode } from '../../../types'
 const REGEX = /var\(\s*(--[\w-]+)/g
-export const unknownCSSVariableRule: Rule<{
-  name: string
-}> = {
+export const unknownCSSVariableRule: Rule<
+  {
+    name: string
+  },
+  StyleNode
+> = {
   code: 'unknown css variable',
   level: 'warning',
   category: 'Unknown Reference',
@@ -30,69 +34,120 @@ export const unknownCSSVariableRule: Rule<{
       return
     }
-    const allCssVariableDeclarations = memo(
-      `all-css-variables-declarations`,
+    const themeCssVariables = theme.propertyDefinitions
+    const [_fileType, componentName, _nodes, nodeName] = path
+    const localCssVariables = memo(
+      `component-css-variables-${componentName}-${nodeName}`,
       () => {
-        const cssVariableKeys = new Set(
-          Object.keys(theme.propertyDefinitions ?? {}),
-        )
-        Object.values(files.components ?? {}).forEach((component) => {
-          Object.keys(component?.nodes ?? {}).forEach((nodeKey) => {
-            const node = component?.nodes?.[nodeKey]
-            if (!node) {
-              return
-            }
+        const vars = new Set<string>()
+        const component = files.components[componentName]
+        if (!component) {
+          return vars
+        }
+        const visitVars = (nodeName: string) => {
+          const node = component?.nodes?.[nodeName]
+          if (!node) {
+            return
+          }
-            if (node.type === 'component' || node.type === 'element') {
-              Object.keys(node.customProperties ?? {}).forEach((varName) => {
-                cssVariableKeys.add(varName)
+          if (node.type === 'component' || node.type === 'element') {
+            Object.keys(node.customProperties ?? {}).forEach((varName) => {
+              vars.add(varName)
+            })
+            Object.values(node.variants ?? {}).forEach((variant) => {
+              Object.keys(variant.customProperties ?? {}).forEach((varName) => {
+                vars.add(varName)
               })
-              Object.values(node.variants ?? {}).forEach((variant) => {
-                Object.keys(variant.customProperties ?? {}).forEach(
-                  (varName) => {
-                    cssVariableKeys.add(varName)
-                  },
-                )
+            })
+            // Also add legacy style variables
+            if (node.type === 'element' && node['style-variables']) {
+              node['style-variables'].forEach((styleVar) => {
+                vars.add(`--${styleVar.name}`)
               })
+            }
-              // Also add legacy style variables
-              if (node.type === 'element' && node['style-variables']) {
-                node['style-variables'].forEach((styleVar) => {
-                  cssVariableKeys.add(`--${styleVar.name}`)
-                })
+            // Add if declared in any parent styles object
+            Object.keys(node.style ?? {}).forEach((styleKey) => {
+              if (styleKey.startsWith('--')) {
+                vars.add(styleKey)
               }
-              // Add if declared in any parent styles object
-              Object.keys(node.style ?? {}).forEach((styleKey) => {
+            })
+            Object.values(node.variants ?? {}).forEach((variant) => {
+              Object.keys(variant.style ?? {}).forEach((styleKey) => {
                 if (styleKey.startsWith('--')) {
-                  cssVariableKeys.add(styleKey)
+                  vars.add(styleKey)
                 }
               })
-              Object.values(node.variants ?? {}).forEach((variant) => {
-                Object.keys(variant.style ?? {}).forEach((styleKey) => {
-                  if (styleKey.startsWith('--')) {
-                    cssVariableKeys.add(styleKey)
-                  }
-                })
-              })
-            }
-          })
-        })
-        return cssVariableKeys
+            })
+          }
+          const parent = Object.entries(component.nodes ?? {}).find(([_, n]) =>
+            n.children?.includes(nodeName),
+          )
+          if (parent) {
+            visitVars(parent[0])
+          }
+        }
+        visitVars(nodeName.toString())
+        return vars
       },
     )
     for (const varName of vars) {
-      if (allCssVariableDeclarations.has(varName) === false) {
+      if (!(varName in themeCssVariables) && !localCssVariables.has(varName)) {
         report({
           path,
           info: {
             title: `Unknown CSS variable`,
-            description: `The CSS variable **${varName}** is not declared in any parent element or in your theme. The CSS variable must be declared in an ancestor element in its component or in your global theme.`,
+            description: `The CSS variable **${varName}** is not declared in any parent element or theme. CSS variables must be declared in an ancestor element or in your global theme.`,
           },
           details: { name: varName },
+          fixes: ['add-to-theme'],
         })
       }
     }
   },
+  fixes: {
+    'add-to-theme': addToThemeFix,
+  },
+}
+export type AddToThemeFix = 'add-to-theme'
+function addToThemeFix(
+  args: Parameters<FixFunction<StyleNode, { name: string }>>[0],
+): ReturnType<FixFunction<StyleNode, { name: string }>> {
+  const varName = args.details?.name
+  if (typeof varName !== 'string' || !args.data.files.themes?.Default) {
+    return args.data.files
+  }
+  const definition: CustomPropertyDefinition = {
+    syntax: {
+      type: 'primitive',
+      name: '*',
+    },
+    inherits: true,
+    initialValue: '',
+    values: {},
+    description: '',
+  } as const
+  // Add the variable to the theme with Any (*) syntax type
+  return {
+    ...args.data.files,
+    themes: {
+      ...args.data.files.themes,
+      Default: {
+        ...args.data.files.themes.Default,
+        propertyDefinitions: {
+          ...args.data.files.themes.Default.propertyDefinitions,
+          [varName]: definition,
+        },
+      },
+    },
+  }
 }

package/src/types.ts CHANGED Viewed

@@ -52,6 +52,7 @@ import type { NoStaticNodeConditionRuleFix } from './rules/issues/logic/noStatic
 import type { NoReferenceNodeRuleFix } from './rules/issues/miscellaneous/noReferenceNodeRule'
 import type { InvalidStyleSyntaxRuleFix } from './rules/issues/style/invalidStyleSyntaxRule'
 import type { LegacyStyleVariableRuleFix } from './rules/issues/style/legacyStyleVariableRule'
+import type { AddToThemeFix } from './rules/issues/style/unknownCSSVariable'
 import type { NoReferenceVariableRuleFix } from './rules/issues/variables/noReferenceVariableRule'
 import type { NoPostNavigateActionRuleFix } from './rules/issues/workflows/noPostNavigateAction'
@@ -439,6 +440,7 @@ export type NodeType =
   | StyleVariantNode
 export type FixType =
+  | AddToThemeFix
   | ChangeDataTypeFix
   | DeleteFetchInputFix
   | InvalidStyleSyntaxRuleFix