@nordcraft/search 1.0.93 → 1.0.94

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

@@ -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

@@ -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

@@ -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

@@ -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

@@ -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

@@ -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

@@ -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

@@ -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

@@ -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

@@ -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',