npm - @nordcraft/search - Versions diffs - 1.0.93 → 1.0.95 - Mend

@nordcraft/search 1.0.93 → 1.0.95

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 (306) hide show

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

@@ -1,8 +1,8 @@
 import { parse } from 'postcss'
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
 import { removeFromPathFix } from '../../../util/removeUnused.fix'
-export const invalidStyleSyntaxRule: Rule<{
+export const invalidStyleSyntaxRule: IssueRule<{
   property: string
 }> = {
   code: 'invalid style syntax',

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

@@ -8,13 +8,13 @@ import type { Nullable } from '@nordcraft/core/dist/types'
 import { get, set } from '@nordcraft/core/dist/utils/collections'
 import type {
   FixFunction,
+  IssueRule,
   NodeType,
-  Rule,
   StyleVariableNode,
 } from '../../../types'
 import { removeFromPathFix } from '../../../util/removeUnused.fix'
-export const legacyStyleVariableRule: Rule<
+export const legacyStyleVariableRule: IssueRule<
   {
     name: string
   },

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

@@ -1,7 +1,7 @@
 import { isDefined } from '@nordcraft/core/dist/utils/util'
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const legacyThemeRule: Rule = {
+export const legacyThemeRule: IssueRule = {
   code: 'legacy theme',
   level: 'warning',
   category: 'Deprecation',

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

@@ -1,9 +1,9 @@
 import type { NodeStyleModel } from '@nordcraft/core/dist/component/component.types'
 import type { Nullable } from '@nordcraft/core/dist/types'
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
 import { removeFromPathFix } from '../../../util/removeUnused.fix'
-export const noReferenceAnimationRule: Rule = {
+export const noReferenceAnimationRule: IssueRule = {
   code: 'no-reference animation',
   level: 'warning',
   category: 'No References',

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

@@ -1,8 +1,8 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
 const REGEX = /var\(\s*(--[\w-]+)/g
-export const noReferenceGlobalCSSVariableRule: Rule<{
+export const noReferenceGlobalCSSVariableRule: IssueRule<{
   name: string
 }> = {
   code: 'no-reference global css variable',

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

@@ -1,9 +1,9 @@
 import type { CustomPropertyDefinition } from '@nordcraft/core/dist/styling/theme'
-import type { FixFunction, Rule, StyleNode } from '../../../types'
+import type { FixFunction, IssueRule, StyleNode } from '../../../types'
 const REGEX = /var\(\s*(--[\w-]+)/g
-export const unknownCSSVariableRule: Rule<
+export const unknownCSSVariableRule: IssueRule<
   {
     name: string
   },

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

@@ -1,7 +1,7 @@
 import { isDefined } from '@nordcraft/core/dist/utils/util'
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownClassnameRule: Rule<{
+export const unknownClassnameRule: IssueRule<{
   name: string
 }> = {
   code: 'unknown classname',

package/src/rules/issues/variables/noReferenceVariableRule.ts CHANGED Viewed

@@ -1,7 +1,7 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
 import { removeFromPathFix } from '../../../util/removeUnused.fix'
-export const noReferenceVariableRule: Rule<void> = {
+export const noReferenceVariableRule: IssueRule<void> = {
   code: 'no-reference variable',
   level: 'warning',
   category: 'No References',

package/src/rules/issues/variables/unknownVariableRule.ts CHANGED Viewed

@@ -1,6 +1,6 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownVariableRule: Rule<{
+export const unknownVariableRule: IssueRule<{
   name: string
 }> = {
   code: 'unknown variable',

package/src/rules/issues/variables/unknownVariableSetterRule.ts CHANGED Viewed

@@ -1,6 +1,6 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownVariableSetterRule: Rule<{
+export const unknownVariableSetterRule: IssueRule<{
   name: string
 }> = {
   code: 'unknown variable setter',

package/src/rules/issues/workflows/duplicateWorkflowParameterRule.ts CHANGED Viewed

@@ -1,30 +1,31 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const duplicateWorkflowParameterRule: Rule<{ parameter: string }> = {
-  code: 'duplicate workflow parameter',
-  level: 'warning',
-  category: 'Quality',
-  visit: (report, { nodeType, path, value }) => {
-    if (
-      nodeType !== 'component-workflow' ||
-      !value.parameters ||
-      (value.parameters ?? []).length === 0
-    ) {
-      return
-    }
-    const parameterNames = new Set<string>()
-    value.parameters.forEach((p, i) => {
-      if (parameterNames.has(p.name)) {
-        report({
-          path: [...path, 'parameters', i],
-          info: {
-            title: 'Duplicate workflow parameter',
-            description: `Multiple parameters with the name **${p.name}** exist. Ensure parameter names are unique.`,
-          },
-          details: { parameter: p.name },
-        })
+export const duplicateWorkflowParameterRule: IssueRule<{ parameter: string }> =
+  {
+    code: 'duplicate workflow parameter',
+    level: 'warning',
+    category: 'Quality',
+    visit: (report, { nodeType, path, value }) => {
+      if (
+        nodeType !== 'component-workflow' ||
+        !value.parameters ||
+        (value.parameters ?? []).length === 0
+      ) {
+        return
       }
-      parameterNames.add(p.name)
-    })
-  },
-}
+      const parameterNames = new Set<string>()
+      value.parameters.forEach((p, i) => {
+        if (parameterNames.has(p.name)) {
+          report({
+            path: [...path, 'parameters', i],
+            info: {
+              title: 'Duplicate workflow parameter',
+              description: `Multiple parameters with the name **${p.name}** exist. Ensure parameter names are unique.`,
+            },
+            details: { parameter: p.name },
+          })
+        }
+        parameterNames.add(p.name)
+      })
+    },
+  }

package/src/rules/issues/workflows/noPostNavigateAction.ts CHANGED Viewed

@@ -2,11 +2,11 @@ import { get, set } from '@nordcraft/core/dist/utils/collections'
 import type {
   ActionModelNode,
   FixFunctionArgs,
+  IssueRule,
   NodeType,
-  Rule,
 } from '../../../types'
-export const noPostNavigateAction: Rule<
+export const noPostNavigateAction: IssueRule<
   { parameter: string },
   NodeType,
   ActionModelNode

package/src/rules/issues/workflows/noReferenceComponentWorkflowRule.ts CHANGED Viewed

@@ -1,7 +1,7 @@
 import { ToddleComponent } from '@nordcraft/core/dist/component/ToddleComponent'
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const noReferenceComponentWorkflowRule: Rule<{
+export const noReferenceComponentWorkflowRule: IssueRule<{
   name: string
   contextSubscribers: string[]
 }> = {

package/src/rules/issues/workflows/unknownContextProviderWorkflowRule.ts CHANGED Viewed

@@ -1,6 +1,6 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownContextProviderWorkflowRule: Rule<{
+export const unknownContextProviderWorkflowRule: IssueRule<{
   providerName: string
   workflowName: string
 }> = {

package/src/rules/issues/workflows/unknownContextWorkflowRule.ts CHANGED Viewed

@@ -1,6 +1,6 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownContextWorkflowRule: Rule<{
+export const unknownContextWorkflowRule: IssueRule<{
   providerName: string
   workflowName: string
 }> = {

package/src/rules/issues/workflows/unknownTriggerWorkflowParameterRule.ts CHANGED Viewed

@@ -1,61 +1,62 @@
 import type { ComponentWorkflow } from '@nordcraft/core/dist/component/component.types'
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownTriggerWorkflowParameterRule: Rule<{ parameter: string }> =
-  {
-    code: 'unknown trigger workflow parameter',
-    level: 'error',
-    category: 'Unknown Reference',
-    visit: (report, args) => {
-      const { path, files, value, nodeType } = args
-      if (
-        nodeType !== 'action-model' ||
-        value.type !== 'TriggerWorkflow' ||
-        // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
-        Object.entries(value?.parameters ?? {}).length === 0
-      ) {
-        return
-      }
+export const unknownTriggerWorkflowParameterRule: IssueRule<{
+  parameter: string
+}> = {
+  code: 'unknown trigger workflow parameter',
+  level: 'error',
+  category: 'Unknown Reference',
+  visit: (report, args) => {
+    const { path, files, value, nodeType } = args
+    if (
+      nodeType !== 'action-model' ||
+      value.type !== 'TriggerWorkflow' ||
+      // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+      Object.entries(value?.parameters ?? {}).length === 0
+    ) {
+      return
+    }
-      let workflow: ComponentWorkflow | undefined
-      if (typeof value.contextProvider === 'string') {
-        const subscription = args.component.contexts?.[value.contextProvider]
-        const isSubscribed = subscription?.workflows?.includes(value.workflow)
-        if (!isSubscribed || typeof subscription?.componentName !== 'string') {
-          return
-        }
-        workflow = subscription?.package
-          ? files.packages?.[subscription.package]?.components[
-              subscription.componentName
-            ]?.workflows?.[value.workflow]
-          : files.components[subscription.componentName]?.workflows?.[
-              value.workflow
-            ]
-        if (!workflow) {
-          return
-        }
-      } else {
-        workflow = args.component.workflows?.[value.workflow]
+    let workflow: ComponentWorkflow | undefined
+    if (typeof value.contextProvider === 'string') {
+      const subscription = args.component.contexts?.[value.contextProvider]
+      const isSubscribed = subscription?.workflows?.includes(value.workflow)
+      if (!isSubscribed || typeof subscription?.componentName !== 'string') {
+        return
       }
+      workflow = subscription?.package
+        ? files.packages?.[subscription.package]?.components[
+            subscription.componentName
+          ]?.workflows?.[value.workflow]
+        : files.components[subscription.componentName]?.workflows?.[
+            value.workflow
+          ]
       if (!workflow) {
         return
       }
-      const workflowParameters = new Set(
-        Object.values(workflow.parameters).map((p) => p.name),
-      )
-      Object.keys(value.parameters ?? {}).forEach((parameterKey) => {
-        if (!workflowParameters.has(parameterKey)) {
-          report({
-            path: [...path, 'parameters', parameterKey],
-            info: {
-              title: `Unknown parameter in workflow`,
-              description: `The workflow parameter "${parameterKey}" is not a valid. It may have been removed from the workflow after it was set. This parameter can safely be removed.`,
-            },
-            details: {
-              parameter: parameterKey,
-            },
-          })
-        }
-      })
-    },
-  }
+    } else {
+      workflow = args.component.workflows?.[value.workflow]
+    }
+    if (!workflow) {
+      return
+    }
+    const workflowParameters = new Set(
+      Object.values(workflow.parameters).map((p) => p.name),
+    )
+    Object.keys(value.parameters ?? {}).forEach((parameterKey) => {
+      if (!workflowParameters.has(parameterKey)) {
+        report({
+          path: [...path, 'parameters', parameterKey],
+          info: {
+            title: `Unknown parameter in workflow`,
+            description: `The workflow parameter "${parameterKey}" is not a valid. It may have been removed from the workflow after it was set. This parameter can safely be removed.`,
+          },
+          details: {
+            parameter: parameterKey,
+          },
+        })
+      }
+    })
+  },
+}

package/src/rules/issues/workflows/unknownTriggerWorkflowRule.ts CHANGED Viewed

@@ -1,6 +1,6 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownTriggerWorkflowRule: Rule<{ workflow: string }> = {
+export const unknownTriggerWorkflowRule: IssueRule<{ workflow: string }> = {
   code: 'unknown trigger workflow',
   level: 'error',
   category: 'Unknown Reference',

package/src/rules/issues/workflows/unknownWorkflowParameterRule.ts CHANGED Viewed

@@ -1,6 +1,6 @@
-import type { Rule } from '../../../types'
+import type { IssueRule } from '../../../types'
-export const unknownWorkflowParameterRule: Rule<{ parameter: string }> = {
+export const unknownWorkflowParameterRule: IssueRule<{ parameter: string }> = {
   code: 'unknown workflow parameter',
   level: 'error',
   category: 'Unknown Reference',