@nordcraft/search 1.0.79 → 1.0.81

package/src/rules/issues/context/noContextConsumersRule.test.ts

@@ -122,7 +122,7 @@ describe('noContextConsumersRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
   test('should not detect component with exposed workflows/formulas when a slot exists', () => {
     const problems = Array.from(
@@ -177,7 +177,7 @@ describe('noContextConsumersRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
   test('should not detect component with exposed workflows/formulas when a child component exists', () => {
     const problems = Array.from(
@@ -235,6 +235,6 @@ describe('noContextConsumersRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/context/unknownContextFormulaRule.test.ts

@@ -89,6 +89,6 @@ describe('unknownContextFormulaRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/context/unknownContextProviderFormulaRule.test.ts

@@ -170,6 +170,6 @@ describe('unknownContextFormulaProviderRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/context/unknownContextProviderRule.test.ts

@@ -75,6 +75,6 @@ describe('unknownContextProviderRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/dom/createRequiredDirectChildRule.test.ts

@@ -115,6 +115,6 @@ describe('createRequiredDirectChildRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/dom/createRequiredDirectParentRule.test.ts

@@ -120,6 +120,6 @@ describe('createRequiredDirectParentRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/dom/createRequiredElementAttributeRule.test.ts

@@ -119,7 +119,7 @@ describe('requiredElementAttributeRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
   test('should not detect issues when elements have all required attributes', () => {
     const problems = Array.from(
@@ -155,7 +155,7 @@ describe('requiredElementAttributeRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should report if attribute is false or null, as such attributes are not rendered', () => {
@@ -256,6 +256,6 @@ describe('requiredElementAttributeRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/dom/elementWithoutInteractiveContentRule.test.ts

@@ -216,7 +216,7 @@ describe('elementWithoutInteractiveContentRule', () => {
         rules: [elementWithoutInteractiveContentRule],
       }),
     )
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
   test('should detect invalid children when negative attribute requirement is met', () => {
     const problems = Array.from(
@@ -265,7 +265,7 @@ describe('elementWithoutInteractiveContentRule', () => {
         rules: [elementWithoutInteractiveContentRule],
       }),
     )
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
   test('should detect invalid label in button element in nested component', () => {
     const problems = Array.from(
@@ -396,6 +396,6 @@ describe('elementWithoutInteractiveContentRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/dom/imageWithoutDimensionRule.test.ts

@@ -77,7 +77,7 @@ describe('imageWithoutDimensionRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should report if image has width but no height', () => {
@@ -246,7 +246,7 @@ describe('imageWithoutDimensionRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should not report if dimensions are dynamic', () => {
@@ -288,7 +288,7 @@ describe('imageWithoutDimensionRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should not report non-image elements', () => {
@@ -321,7 +321,7 @@ describe('imageWithoutDimensionRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should report if dimensions are set to non-static keywords (auto | empty string)', () => {

package/src/rules/issues/dom/nonEmptyVoidElementRule.test.ts

@@ -78,6 +78,6 @@ describe('voidElementRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/events/duplicateEventTriggerRule.test.ts

@@ -93,6 +93,6 @@ describe('duplicateEventTriggerRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/formulas/duplicateFormulaArgumentNameRule.test.ts

@@ -106,7 +106,7 @@ describe('duplicateFormulaArgumentNameRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should detect multiple duplicates', () => {
@@ -175,6 +175,6 @@ describe('duplicateFormulaArgumentNameRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/formulas/noReferenceProjectFormulaRule.test.ts

@@ -288,7 +288,7 @@ describe('find noReferenceFormulaRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })
 

package/src/rules/issues/formulas/unknownComponentFormulaInputRule.test.ts

@@ -89,7 +89,7 @@ describe('unknownComponentFormulaInput', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
   test('should not detect @toddle.parent/item path formulas', () => {
     const problems = Array.from(
@@ -136,6 +136,6 @@ describe('unknownComponentFormulaInput', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/formulas/unknownProjectFormulaInputRule.test.ts

@@ -70,7 +70,7 @@ describe('unknownProjectFormulaInput', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
   test('should not detect @toddle.parent/item path formulas', () => {
     const problems = Array.from(
@@ -107,6 +107,6 @@ describe('unknownProjectFormulaInput', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/formulas/unknownProjectFormulaRule.test.ts

@@ -121,6 +121,64 @@ describe('unknownProjectFormulaRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
+  })
+
+  test('should not report unknown formulas from within packages', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          formulas: {},
+          packages: {
+            'my-package': {
+              components: {},
+              actions: {},
+              formulas: {
+                'package-formula': {
+                  formula: {
+                    type: 'function',
+                    name: 'non-existent-formula',
+                    arguments: [],
+                  },
+                  name: 'package-formula',
+                  arguments: [],
+                },
+              },
+              manifest: {
+                name: 'my-package',
+                commit: '123',
+              },
+            },
+          },
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  type: 'element',
+                  attrs: {
+                    test: functionFormula('package-formula', {
+                      package: 'my-package',
+                    }),
+                  },
+                  classes: {},
+                  events: {},
+                  tag: 'div',
+                  children: [],
+                  style: {},
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [unknownProjectFormulaRule],
+      }),
+    )
+
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/formulas/unknownRepeatIndexFormulaRule.test.ts

@@ -262,6 +262,6 @@ describe('unknownRepeatIndexFormulaRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/formulas/unknownRepeatItemFormulaRule.test.ts

@@ -277,6 +277,6 @@ describe('unknownRepeatItemFormulaRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/logic/noStaticNodeCondition.test.ts

@@ -127,7 +127,7 @@ describe('noStaticNodeCondition', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should fix static truthy condition by removing the condition', () => {

package/src/rules/issues/logic/noUnnecessaryConditionFalsy.test.ts

@@ -97,6 +97,6 @@ describe('noUnnecessaryConditionFalsy', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/logic/noUnnecessaryConditionTruthy.test.ts

@@ -154,6 +154,6 @@ describe('noUnnecessaryConditionTruthy', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/miscellaneous/requireExtensionRule.test.ts

@@ -69,6 +69,6 @@ describe('requireExtension', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/miscellaneous/unknownCookieRule.test.ts

@@ -107,6 +107,6 @@ describe('unknownCookie', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/routing/duplicateRouteRule.test.ts

@@ -142,6 +142,6 @@ describe('duplicateRouteRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/routing/duplicateUrlParameterRule.test.ts

@@ -130,6 +130,6 @@ describe('duplicateUrlParameterRule', () => {
         rules: [duplicateUrlParameterRule],
       }),
     )
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 })

package/src/rules/issues/style/noReferenceGlobalCSSVariable.test.ts

@@ -57,7 +57,7 @@ describe('noReferenceGlobalCSSVariableRule', () => {
       }),
     )
 
-    expect(problems).toHaveLength(0)
+    expect(problems).toBeEmpty()
   })
 
   test('should report global CSS variables that are never used', () => {
@@ -243,4 +243,62 @@ describe('noReferenceGlobalCSSVariableRule', () => {
     expect(problems[0].details).toEqual({ name: '--local-color' })
     expect(problems[1].details).toEqual({ name: '--a-third-local-color' })
   })
+
+  test('should not report when variable is used in package components', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          themes: {
+            Default: {
+              fonts: [],
+              propertyDefinitions: {
+                '--global-color': {
+                  syntax: { type: 'primitive', name: 'color' },
+                  description: 'A global color',
+                  inherits: true,
+                  initialValue: 'blue',
+                  values: {},
+                },
+              },
+            },
+          },
+          formulas: {},
+          components: {},
+          packages: [
+            {
+              manifest: {
+                name: 'Test Package',
+                commit: '',
+              },
+              components: {
+                PackageComponent: {
+                  name: 'PackageComponent',
+                  nodes: {
+                    root: {
+                      tag: 'div',
+                      type: 'element',
+                      attrs: {},
+                      style: {
+                        color: 'var(--global-color)',
+                      },
+                      events: {},
+                      classes: {},
+                      children: [],
+                    },
+                  },
+                  formulas: {},
+                  apis: {},
+                  attributes: {},
+                  variables: {},
+                },
+              },
+            },
+          ] as any,
+        },
+        rules: [noReferenceGlobalCSSVariableRule],
+      }),
+    )
+
+    expect(problems).toBeEmpty()
+  })
 })

package/src/rules/issues/style/noReferenceGlobalCSSVariable.ts

@@ -28,12 +28,11 @@ export const noReferenceGlobalCSSVariableRule: Rule<{
               ;[{ style: node.style }, ...(node.variants ?? [])].forEach(
                 ({ style }) => {
                   Object.values(style ?? {}).forEach((styleValue) => {
-                    if (typeof styleValue !== 'string') {
-                      return
+                    if (typeof styleValue === 'string') {
+                      styleValue.matchAll(REGEX).forEach(([_, varName]) => {
+                        vars.add(varName)
+                      })
                     }
-                    styleValue.matchAll(REGEX).forEach(([_, varName]) => {
-                      vars.add(varName)
-                    })
                   })
                 },
               )
@@ -45,6 +44,34 @@ export const noReferenceGlobalCSSVariableRule: Rule<{
       },
     )
 
+    const usedCSSVariablesInPackageComponents = memo(
+      'css-variables-used-in-package-components',
+      () => {
+        const vars = new Set<string>()
+        Object.values(files.packages ?? {}).forEach((pkg) => {
+          Object.values(pkg?.components ?? {}).forEach((component) => {
+            Object.values(component?.nodes ?? {}).forEach((node) => {
+              if (node.type === 'element' || node.type === 'component') {
+                ;[{ style: node.style }, ...(node.variants ?? [])].forEach(
+                  ({ style }) => {
+                    Object.values(style ?? {}).forEach((styleValue) => {
+                      if (typeof styleValue === 'string') {
+                        styleValue.matchAll(REGEX).forEach(([_, varName]) => {
+                          vars.add(varName)
+                        })
+                      }
+                    })
+                  },
+                )
+              }
+            })
+          })
+        })
+
+        return vars
+      },
+    )
+
     const usedCSSVariablesInCSSVariables = memo(
       'css-variables-used-in-css-variables',
       () => {
@@ -52,13 +79,11 @@ export const noReferenceGlobalCSSVariableRule: Rule<{
         Object.values(theme.propertyDefinitions ?? {}).forEach((propDef) => {
           ;[...Object.values(propDef.values), propDef.initialValue].forEach(
             (val) => {
-              if (typeof val !== 'string') {
-                return
+              if (typeof val === 'string') {
+                val.matchAll(REGEX).forEach(([_, varName]) => {
+                  vars.add(varName)
+                })
               }
-
-              val.matchAll(REGEX).forEach(([_, varName]) => {
-                vars.add(varName)
-              })
             },
           )
         })
@@ -69,7 +94,8 @@ export const noReferenceGlobalCSSVariableRule: Rule<{
 
     if (
       usedCSSVariablesInCSSVariables.has(value.key) ||
-      usedCSSVariablesInComponents.has(value.key)
+      usedCSSVariablesInComponents.has(value.key) ||
+      usedCSSVariablesInPackageComponents.has(value.key)
     ) {
       return
     }

package/src/rules/issues/style/styleRules.index.ts

@@ -1,9 +1,11 @@
 import { invalidStyleSyntaxRule } from './invalidStyleSyntaxRule'
 import { noReferenceGlobalCSSVariableRule } from './noReferenceGlobalCSSVariable'
 import { unknownClassnameRule } from './unknownClassnameRule'
+import { unknownCSSVariableRule } from './unknownCSSVariable'
 
 export default [
   invalidStyleSyntaxRule,
   unknownClassnameRule,
+  unknownCSSVariableRule,
   noReferenceGlobalCSSVariableRule,
 ]