@nordcraft/search 1.0.45 → 1.0.47

package/dist/util/helpers.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"helpers.test.js","sourceRoot":"","sources":["../../src/util/helpers.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,UAAU,CAAA;AACjD,OAAO,EAAE,qBAAqB,EAAE,eAAe,EAAE,MAAM,WAAW,CAAA;AAElE,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,IAAI,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACxD,MAAM,YAAY,GAAG;YACnB,CAAC,YAAY,EAAE,QAAQ,CAAC;YACxB,CAAC,YAAY,EAAE,OAAO,CAAC;SACxB,CAAA;QACD,MAAM,CACJ,eAAe,CAAC;YACd,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,CAAC;YAC7D,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,eAAe,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,OAAO,CAAC,EAAE,YAAY,EAAE,CAAC,CACjE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,eAAe,CAAC;YACd,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,EAAE,UAAU,CAAC;YAC1C,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,eAAe,CAAC;YACd,IAAI,EAAE,CAAC,YAAY,EAAE,sBAAsB,CAAC;YAC5C,YAAY,EAAE;gBACZ;oBACE,YAAY;oBACZ,sBAAsB;oBACtB,UAAU;oBACV,uBAAuB;oBACvB,SAAS;iBACV;aACF;SACF,CAAC,CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACd,CAAC,CAAC,CAAA;IAEF,IAAI,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC7D,MAAM,YAAY,GAAG;YACnB,CAAC,YAAY,EAAE,QAAQ,CAAC;YACxB,CAAC,YAAY,EAAE,OAAO,CAAC;SACxB,CAAA;QACD,MAAM,CACJ,eAAe,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,MAAM,CAAC,EAAE,YAAY,EAAE,CAAC,CAChE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACb,MAAM,CACJ,eAAe,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,MAAM,CAAC,EAAE,YAAY,EAAE,CAAC,CAChE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACf,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,IAAI,CAAC,IAAI,CAAC,2CAA2C,EAAE,GAAG,EAAE;QAC1D,MAAM,YAAY,GAAG;YACnB,CAAC,YAAY,EAAE,QAAQ,CAAC;YACxB,CAAC,YAAY,EAAE,OAAO,CAAC;SACxB,CAAA;QACD,MAAM,CACJ,qBAAqB,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,CAAC,EAAE,YAAY,EAAE,CAAC,CACxE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,qBAAqB,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,OAAO,CAAC,EAAE,YAAY,EAAE,CAAC,CACvE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,qBAAqB,CAAC;YACpB,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,EAAE,UAAU,CAAC;YAC1C,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACb,MAAM,CACJ,qBAAqB,CAAC;YACpB,IAAI,EAAE,CAAC,YAAY,EAAE,sBAAsB,CAAC;YAC5C,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACf,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}
+{"version":3,"file":"helpers.test.js","sourceRoot":"","sources":["../../src/util/helpers.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,UAAU,CAAA;AACjD,OAAO,EAAE,qBAAqB,EAAE,eAAe,EAAE,MAAM,WAAW,CAAA;AAElE,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,IAAI,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACxD,MAAM,YAAY,GAAG;YACnB,CAAC,YAAY,EAAE,QAAQ,CAAC;YACxB,CAAC,YAAY,EAAE,OAAO,CAAC;SACxB,CAAA;QACD,MAAM,CACJ,eAAe,CAAC;YACd,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,EAAE,WAAW,CAAC;YAC7D,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,eAAe,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,OAAO,CAAC,EAAE,YAAY,EAAE,CAAC,CACjE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,eAAe,CAAC;YACd,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,EAAE,UAAU,CAAC;YAC1C,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,eAAe,CAAC;YACd,IAAI,EAAE,CAAC,YAAY,EAAE,sBAAsB,CAAC;YAC5C,YAAY,EAAE;gBACZ;oBACE,YAAY;oBACZ,sBAAsB;oBACtB,UAAU;oBACV,uBAAuB;oBACvB,SAAS;iBACV;aACF;SACF,CAAC,CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACd,CAAC,CAAC,CAAA;IAEF,IAAI,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC7D,MAAM,YAAY,GAAG;YACnB,CAAC,YAAY,EAAE,QAAQ,CAAC;YACxB,CAAC,YAAY,EAAE,OAAO,CAAC;SACxB,CAAA;QACD,MAAM,CACJ,eAAe,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,MAAM,CAAC,EAAE,YAAY,EAAE,CAAC,CAChE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACb,MAAM,CACJ,eAAe,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,MAAM,CAAC,EAAE,YAAY,EAAE,CAAC,CAChE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACf,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,IAAI,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACrD,MAAM,YAAY,GAAG;YACnB,CAAC,YAAY,EAAE,QAAQ,CAAC;YACxB,CAAC,YAAY,EAAE,OAAO,CAAC;SACxB,CAAA;QACD,MAAM,CACJ,qBAAqB,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,CAAC,EAAE,YAAY,EAAE,CAAC,CACxE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,qBAAqB,CAAC,EAAE,IAAI,EAAE,CAAC,YAAY,EAAE,OAAO,CAAC,EAAE,YAAY,EAAE,CAAC,CACvE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACZ,MAAM,CACJ,qBAAqB,CAAC;YACpB,IAAI,EAAE,CAAC,YAAY,EAAE,QAAQ,EAAE,UAAU,CAAC;YAC1C,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACb,MAAM,CACJ,qBAAqB,CAAC;YACpB,IAAI,EAAE,CAAC,YAAY,EAAE,sBAAsB,CAAC;YAC5C,YAAY;SACb,CAAC,CACH,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACf,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/util/removeUnused.fix.js

@@ -1,3 +1,20 @@
-import { omit } from '@nordcraft/core/dist/utils/collections';
+import { get, omit, set } from '@nordcraft/core/dist/utils/collections';
 export const removeFromPathFix = ({ path, files }) => omit(files, path);
+/**
+ * Same as removeFromPathFix, but also removes the node from its parent's children.
+ */
+export const removeNodeFromPathFix = (data) => {
+    if (data.nodeType !== 'component-node') {
+        throw new Error('removeNodeFromPathFix can only be used on component nodes');
+    }
+    const componentNodesPath = data.path.slice(0, -1).map(String);
+    const filesWithoutNode = removeFromPathFix(data);
+    const nodes = get(filesWithoutNode, componentNodesPath);
+    for (const key in nodes) {
+        if (nodes[key].children?.includes(data.path[data.path.length - 1])) {
+            nodes[key].children = nodes[key].children.filter((p) => p !== data.path[data.path.length - 1]);
+        }
+    }
+    return set(filesWithoutNode, componentNodesPath, nodes);
+};
 //# sourceMappingURL=removeUnused.fix.js.map

package/dist/util/removeUnused.fix.js.map

@@ -1 +1 @@
-{"version":3,"file":"removeUnused.fix.js","sourceRoot":"","sources":["../../src/util/removeUnused.fix.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,wCAAwC,CAAA;AAG7D,MAAM,CAAC,MAAM,iBAAiB,GAA0B,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,EAAE,EAAE,CAC1E,IAAI,CAAC,KAAK,EAAE,IAAI,CAAC,CAAA"}
+{"version":3,"file":"removeUnused.fix.js","sourceRoot":"","sources":["../../src/util/removeUnused.fix.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,MAAM,wCAAwC,CAAA;AAGvE,MAAM,CAAC,MAAM,iBAAiB,GAA0B,CAAC,EAAE,IAAI,EAAE,KAAK,EAAE,EAAE,EAAE,CAC1E,IAAI,CAAC,KAAK,EAAE,IAAI,CAAC,CAAA;AAEnB;;GAEG;AACH,MAAM,CAAC,MAAM,qBAAqB,GAA0B,CAAC,IAAI,EAAE,EAAE;IACnE,IAAI,IAAI,CAAC,QAAQ,KAAK,gBAAgB,EAAE,CAAC;QACvC,MAAM,IAAI,KAAK,CAAC,2DAA2D,CAAC,CAAA;IAC9E,CAAC;IAED,MAAM,kBAAkB,GAAG,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,CAAA;IAC7D,MAAM,gBAAgB,GAAG,iBAAiB,CAAC,IAAI,CAAC,CAAA;IAChD,MAAM,KAAK,GAAG,GAAG,CAAC,gBAAgB,EAAE,kBAAkB,CAGrD,CAAA;IACD,KAAK,MAAM,GAAG,IAAI,KAAK,EAAE,CAAC;QACxB,IACE,KAAK,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAW,CAAC,EACxE,CAAC;YACD,KAAK,CAAC,GAAG,CAAC,CAAC,QAAQ,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,QAAQ,CAAC,MAAM,CAC9C,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,CAC7C,CAAA;QACH,CAAC;IACH,CAAC;IAED,OAAO,GAAG,CAAC,gBAAgB,EAAE,kBAAkB,EAAE,KAAK,CAAC,CAAA;AACzD,CAAC,CAAA"}

package/package.json

@@ -5,7 +5,7 @@
 "license": "Apache-2.0",
 "homepage": "https://github.com/nordcraftengine/nordcraft",
 "dependencies": {
-"@nordcraft/ssr": "1.0.45",
+"@nordcraft/ssr": "1.0.47",
 "jsondiffpatch": "0.7.3",
 "postcss": "8.5.6"
 },
@@ -19,5 +19,5 @@
 "test:watch:only": "bun test --watch --only"
 },
 "files": ["dist", "src"],
-"version": "1.0.45"
+"version": "1.0.47"
 }

package/src/problems.worker.ts

@@ -12,6 +12,7 @@ import { unknownApiInputRule } from './rules/apis/unknownApiInputRule'
 import { unknownApiRule } from './rules/apis/unknownApiRule'
 import { noReferenceAttributeRule } from './rules/attributes/noReferenceAttributeRule'
 import { unknownAttributeRule } from './rules/attributes/unknownAttributeRule'
+import { unknownComponentAttributeRule } from './rules/attributes/unknownComponentAttributeRule'
 import { noReferenceComponentRule } from './rules/components/noReferenceComponentRule'
 import { unknownComponentRule } from './rules/components/unknownComponentRule'
 import { noContextConsumersRule } from './rules/context/noContextConsumersRule'
@@ -37,6 +38,7 @@ import { unknownFormulaRule } from './rules/formulas/unknownFormulaRule'
 import { unknownProjectFormulaRule } from './rules/formulas/unknownProjectFormulaRule'
 import { unknownRepeatIndexFormulaRule } from './rules/formulas/unknownRepeatIndexFormulaRule'
 import { unknownRepeatItemFormulaRule } from './rules/formulas/unknownRepeatItemFormulaRule'
+import { noStaticNodeCondition } from './rules/logic/noStaticNodeCondition'
 import { noUnnecessaryConditionFalsy } from './rules/logic/noUnnecessaryConditionFalsy'
 import { noUnnecessaryConditionTruthy } from './rules/logic/noUnnecessaryConditionTruthy'
 import { noReferenceNodeRule } from './rules/noReferenceNodeRule'
@@ -162,6 +164,7 @@ const RULES = [
   noReferenceProjectActionRule,
   noReferenceProjectFormulaRule,
   noReferenceVariableRule,
+  noStaticNodeCondition,
   noUnnecessaryConditionFalsy,
   noUnnecessaryConditionTruthy,
   requireExtensionRule,
@@ -178,6 +181,7 @@ const RULES = [
   unknownContextProviderWorkflowRule,
   unknownContextWorkflowRule,
   unknownCookieRule,
+  unknownComponentAttributeRule,
   unknownEventRule,
   unknownFormulaRule,
   unknownProjectActionRule,
@@ -196,21 +200,25 @@ const RULES = [
 ]
 
 interface FindProblemsArgs {
+  id: string
   files: ProjectFiles
   options?: Options
 }
 
 interface FixProblemsArgs {
+  id: string
   files: ProjectFiles
   options?: Options
   fixRule: Code
   fixType: FixType
-  id: string
 }
 
 type Message = FindProblemsArgs | FixProblemsArgs
 
-type FindProblemsResponse = Result[]
+interface FindProblemsResponse {
+  id: string
+  results: Result[]
+}
 
 interface FixProblemsResponse {
   id: string
@@ -225,6 +233,11 @@ const respond = (data: Response) => postMessage(data)
 
 const findProblems = (data: FindProblemsArgs) => {
   const { files, options = {} } = data
+  const idRespond = (results: Result[]) =>
+    respond({
+      id: data.id,
+      results,
+    })
   const rules = RULES.filter(
     (rule) =>
       (!options.categories || options.categories.includes(rule.category)) &&
@@ -250,7 +263,7 @@ const findProblems = (data: FindProblemsArgs) => {
       case 'per-file': {
         if (fileType !== problem.path[0] || fileName !== problem.path[1]) {
           if (batch.length > 0) {
-            respond(batch)
+            idRespond(batch)
           }
           batch = []
           fileType = problem.path[0]
@@ -264,7 +277,7 @@ const findProblems = (data: FindProblemsArgs) => {
       default: {
         batch.push(problem)
         if (batch.length >= (options.batchSize ?? 1)) {
-          respond(batch)
+          idRespond(batch)
           batch = []
         }
         break
@@ -273,7 +286,7 @@ const findProblems = (data: FindProblemsArgs) => {
   }
 
   // Send the remaining results
-  respond(batch)
+  idRespond(batch)
 }
 
 const fixProblems = (data: FixProblemsArgs) => {

package/src/rules/attributes/unknownComponentAttributeRule.test.ts

@@ -0,0 +1,156 @@
+import type { ComponentNodeModel } from '@nordcraft/core/dist/component/component.types'
+import { valueFormula } from '@nordcraft/core/dist/formula/formulaUtils'
+import type { ProjectFiles } from '@nordcraft/ssr/dist/ssr.types'
+import { describe, expect, test } from 'bun:test'
+import { fixProject } from '../../fixProject'
+import { searchProject } from '../../searchProject'
+import { unknownComponentAttributeRule } from './unknownComponentAttributeRule'
+
+describe('find unknownComponentAttribute', () => {
+  test('should report unknown component attributes', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          components: {
+            referenced: {
+              name: 'referenced',
+              nodes: {},
+              attributes: {
+                first: {
+                  name: 'first',
+                  testValue: valueFormula(null),
+                },
+                second: {
+                  name: 'second',
+                  testValue: valueFormula(null),
+                },
+              },
+              apis: {},
+              formulas: {},
+              variables: {},
+            },
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  type: 'component',
+                  name: 'referenced',
+                  events: {},
+                  children: [],
+                  style: {},
+                  attrs: {
+                    first: valueFormula('exists'),
+                    third: valueFormula('does not exist'),
+                  },
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [unknownComponentAttributeRule],
+      }),
+    )
+
+    expect(problems).toHaveLength(1)
+    expect(problems[0].code).toBe('unknown component attribute')
+    expect(problems[0].details).toEqual({ name: 'third' })
+  })
+
+  test('should not report component attributes that exist', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  type: 'element',
+                  attrs: {
+                    known: valueFormula('exists'),
+                  },
+                  classes: {},
+                  events: {},
+                  tag: 'div',
+                  children: [],
+                  style: {},
+                },
+              },
+              formulas: {},
+              apis: {},
+              variables: {},
+              attributes: {
+                known: {
+                  name: 'known',
+                  testValue: { type: 'value', value: null },
+                },
+              },
+            },
+          },
+        },
+        rules: [unknownComponentAttributeRule],
+      }),
+    )
+    expect(problems).toEqual([])
+  })
+})
+
+describe('fix unknownComponentAttribute', () => {
+  test('should remove unknown component attributes', () => {
+    const files: ProjectFiles = {
+      components: {
+        referenced: {
+          name: 'referenced',
+          nodes: {},
+          attributes: {
+            first: {
+              name: 'first',
+              testValue: valueFormula(null),
+            },
+            second: {
+              name: 'second',
+              testValue: valueFormula(null),
+            },
+          },
+          apis: {},
+          formulas: {},
+          variables: {},
+        },
+        test: {
+          name: 'test',
+          nodes: {
+            root: {
+              type: 'component',
+              name: 'referenced',
+              events: {},
+              children: [],
+              style: {},
+              attrs: {
+                first: valueFormula('exists'),
+                third: valueFormula('does not exist'),
+              },
+            },
+          },
+          formulas: {},
+          apis: {},
+          attributes: {},
+          variables: {},
+        },
+      },
+    }
+    const fixedProject = fixProject({
+      files,
+      rule: unknownComponentAttributeRule,
+      fixType: 'delete-component-attribute',
+    })
+    expect(
+      (fixedProject.components.test!.nodes.root as ComponentNodeModel).attrs,
+    ).toEqual({
+      first: valueFormula('exists'),
+    })
+  })
+})

package/src/rules/attributes/unknownComponentAttributeRule.ts

@@ -0,0 +1,40 @@
+import { isDefined } from '@nordcraft/core/dist/utils/util'
+import type { Rule } from '../../types'
+import { removeFromPathFix } from '../../util/removeUnused.fix'
+
+export const unknownComponentAttributeRule: Rule<{
+  name: string
+}> = {
+  code: 'unknown component attribute',
+  level: 'error',
+  category: 'Unknown Reference',
+  visit: (report, { path, files, value, nodeType }) => {
+    if (
+      nodeType !== 'component-node' ||
+      value.type !== 'component' ||
+      // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+      Object.keys(value.attrs ?? {}).length === 0
+    ) {
+      return
+    }
+    const component = value.package
+      ? files.packages?.[value.package].components?.[value.name]
+      : files.components[value.name]
+    if (!component) {
+      return
+    }
+    for (const attrKey of Object.keys(value.attrs)) {
+      // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+      if (!isDefined(component.attributes?.[attrKey])) {
+        report([...path, 'attrs', attrKey], { name: attrKey }, [
+          'delete-component-attribute',
+        ])
+      }
+    }
+  },
+  fixes: {
+    'delete-component-attribute': removeFromPathFix,
+  },
+}
+
+export type UnknownComponentAttributeRuleFix = 'delete-component-attribute'

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

@@ -0,0 +1,290 @@
+import type { ProjectFiles } from '@nordcraft/ssr/dist/ssr.types'
+import { describe, expect, test } from 'bun:test'
+import { fixProject } from '../../fixProject'
+import { searchProject } from '../../searchProject'
+import { noStaticNodeCondition } from './noStaticNodeCondition'
+
+describe('noStaticNodeCondition', () => {
+  test('should report node condition that is always truthy', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  type: 'element',
+                  attrs: {},
+                  classes: {},
+                  events: {},
+                  tag: 'div',
+                  children: [],
+                  style: {},
+                  condition: {
+                    type: 'value',
+                    value: true,
+                  },
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [noStaticNodeCondition],
+      }),
+    )
+
+    expect(problems).toHaveLength(1)
+    expect(problems[0].code).toBe('no-static-node-condition')
+    expect(problems[0].details).toEqual({
+      result: true,
+    })
+    expect(problems[0].fixes).toEqual(['remove-condition'])
+    expect(problems[0].path).toEqual([
+      'components',
+      'test',
+      'nodes',
+      'root',
+      'condition',
+    ])
+  })
+
+  test('should report node condition that is always falsy', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  type: 'element',
+                  attrs: {},
+                  classes: {},
+                  events: {},
+                  tag: 'div',
+                  children: [],
+                  style: {},
+                  condition: {
+                    type: 'value',
+                    value: false,
+                  },
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [noStaticNodeCondition],
+      }),
+    )
+
+    expect(problems).toHaveLength(1)
+    expect(problems[0].code).toBe('no-static-node-condition')
+    expect(problems[0].details).toEqual({
+      result: false,
+    })
+  })
+
+  test('should not report node condition that is dynamic', () => {
+    const problems = Array.from(
+      searchProject({
+        files: {
+          components: {
+            test: {
+              name: 'test',
+              nodes: {
+                root: {
+                  type: 'element',
+                  attrs: {},
+                  classes: {},
+                  events: {},
+                  tag: 'div',
+                  children: [],
+                  style: {},
+                  condition: {
+                    type: 'apply',
+                    name: 'randomNumber',
+                    arguments: [],
+                  },
+                },
+              },
+              formulas: {},
+              apis: {},
+              attributes: {},
+              variables: {},
+            },
+          },
+        },
+        rules: [noStaticNodeCondition],
+      }),
+    )
+
+    expect(problems).toHaveLength(0)
+  })
+
+  test('should fix static truthy condition by removing the condition', () => {
+    const files: ProjectFiles = {
+      components: {
+        test: {
+          name: 'test',
+          nodes: {
+            root: {
+              type: 'element',
+              attrs: {},
+              classes: {},
+              events: {},
+              tag: 'div',
+              children: [],
+              style: {},
+              condition: {
+                type: 'value',
+                value: true,
+              },
+            },
+          },
+          formulas: {},
+          apis: {},
+          attributes: {},
+          variables: {},
+        },
+      },
+    }
+    const fixedFiles = fixProject({
+      files,
+      rule: noStaticNodeCondition,
+      fixType: 'remove-condition',
+    })
+    expect(fixedFiles).toMatchInlineSnapshot(`
+      {
+        "components": {
+          "test": {
+            "apis": {},
+            "attributes": {},
+            "formulas": {},
+            "name": "test",
+            "nodes": {
+              "root": {
+                "attrs": {},
+                "children": [],
+                "classes": {},
+                "events": {},
+                "style": {},
+                "tag": "div",
+                "type": "element",
+              },
+            },
+            "variables": {},
+          },
+        },
+      }
+    `)
+  })
+
+  test("should fix static falsy condition by removing the node and any parents' references to it", () => {
+    const files: ProjectFiles = {
+      components: {
+        test: {
+          name: 'test',
+          nodes: {
+            root: {
+              type: 'element',
+              attrs: {},
+              classes: {},
+              events: {},
+              tag: 'div',
+              children: ['alwaysHiddenElement', 'sometimesVisibleElement'],
+              style: {},
+            },
+            alwaysHiddenElement: {
+              type: 'element',
+              attrs: {},
+              classes: {},
+              events: {},
+              tag: 'div',
+              children: [],
+              style: {},
+              condition: {
+                type: 'value',
+                value: false,
+              },
+            },
+            sometimesVisibleElement: {
+              type: 'element',
+              attrs: {},
+              classes: {},
+              events: {},
+              tag: 'div',
+              children: [],
+              style: {},
+              condition: {
+                type: 'path',
+                path: ['Variables', 'maybe'],
+              },
+            },
+          },
+          formulas: {},
+          apis: {},
+          attributes: {},
+          variables: {},
+        },
+      },
+    }
+
+    const fixedFiles = fixProject({
+      files,
+      rule: noStaticNodeCondition,
+      fixType: 'remove-node',
+    })
+
+    expect(fixedFiles).toMatchInlineSnapshot(`
+      {
+        "components": {
+          "test": {
+            "apis": {},
+            "attributes": {},
+            "formulas": {},
+            "name": "test",
+            "nodes": {
+              "root": {
+                "attrs": {},
+                "children": [
+                  "sometimesVisibleElement",
+                ],
+                "classes": {},
+                "events": {},
+                "style": {},
+                "tag": "div",
+                "type": "element",
+              },
+              "sometimesVisibleElement": {
+                "attrs": {},
+                "children": [],
+                "classes": {},
+                "condition": {
+                  "path": [
+                    "Variables",
+                    "maybe",
+                  ],
+                  "type": "path",
+                },
+                "events": {},
+                "style": {},
+                "tag": "div",
+                "type": "element",
+              },
+            },
+            "variables": {},
+          },
+        },
+      }
+    `)
+  })
+})

package/src/rules/logic/noStaticNodeCondition.ts

@@ -0,0 +1,43 @@
+import type { FixFunction, NodeType, Rule } from '../../types'
+import { contextlessEvaluateFormula } from '../../util/contextlessEvaluateFormula'
+import {
+  removeFromPathFix,
+  removeNodeFromPathFix,
+} from '../../util/removeUnused.fix'
+
+export const noStaticNodeCondition: Rule<{
+  result: ReturnType<typeof contextlessEvaluateFormula>['result']
+}> = {
+  code: 'no-static-node-condition',
+  level: 'warning',
+  category: 'Quality',
+  visit: (report, { path: nodePath, value, nodeType }) => {
+    if (nodeType !== 'component-node' || !value.condition) {
+      return
+    }
+
+    const { isStatic, result } = contextlessEvaluateFormula(value.condition)
+    if (isStatic) {
+      const conditionPath = [...nodePath, 'condition']
+      // - if truthy: "Condition is always true, you can safely remove the condition as it will always be rendered."
+      // - if falsy: "Condition is always false, you can safely remove the entire node as it will never be rendered."
+      report(
+        conditionPath,
+        {
+          result,
+        },
+        Boolean(result) === true ? ['remove-condition'] : ['remove-node'],
+      )
+    }
+  },
+  fixes: {
+    'remove-condition': removeFromPathFix,
+    'remove-node': (({ path, ...data }) =>
+      removeNodeFromPathFix({
+        path: path.slice(0, -1),
+        ...data,
+      })) as FixFunction<NodeType>,
+  },
+}
+
+export type NoStaticNodeConditionRuleFix = 'remove-condition' | 'remove-node'