@shipfox/api-definitions 12.3.0 → 14.0.0

package/dist/core/validate-definition.js

@@ -5,13 +5,13 @@ import { InvalidWorkflowYamlError, parseWorkflowYamlWithLocations } from './work
 export function validateDefinition(yamlContent, options) {
     try {
         const { document, stepSourceLocations } = parseWorkflowYamlWithLocations(yamlContent);
-        const warnings = [];
+        const diagnostics = [];
         const model = normalizeWorkflowDocument(document, {
             defaultRunnerLabels: options.defaultRunnerLabels ?? definitionDefaultRunnerLabels,
             agentValidationCatalog: options.agentValidationCatalog,
             integrationValidationContext: options.integrationValidationContext,
             stepSourceLocations,
-            warnings
+            diagnostics
         });
         return {
             valid: true,
@@ -19,7 +19,7 @@ export function validateDefinition(yamlContent, options) {
                 document,
                 model
             },
-            warnings: validationWarningsFor(warnings)
+            diagnostics: validationDiagnosticsFor(diagnostics)
         };
     } catch (error) {
         return {
@@ -28,11 +28,12 @@ export function validateDefinition(yamlContent, options) {
         };
     }
 }
-function validationWarningsFor(issues) {
-    return issues.map((issue)=>validationWarning({
+function validationDiagnosticsFor(issues) {
+    return issues.map((issue)=>validationDiagnostic({
             code: issue.code,
             message: issue.message,
-            path: issue.path.join('.') || undefined
+            path: issue.path.join('.') || undefined,
+            severity: issue.severity
         }));
 }
 function validationErrorsFor(error) {
@@ -67,15 +68,19 @@ function validationError(params) {
         path: params.path
     };
 }
-function validationWarning(params) {
-    if (params.path === undefined) return {
-        code: params.code,
-        message: params.message
-    };
+function validationDiagnostic(params) {
+    if (params.path === undefined) {
+        return {
+            code: params.code,
+            message: params.message,
+            severity: params.severity
+        };
+    }
     return {
         code: params.code,
         message: params.message,
-        path: params.path
+        path: params.path,
+        severity: params.severity
     };
 }
 

package/dist/core/validate-definition.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../src/core/validate-definition.ts"],"sourcesContent":["import type {AgentValidationCatalog} from '@shipfox/api-agent-dto/inter-module';\nimport {InvalidWorkflowDocumentError} from '@shipfox/workflow-document';\nimport {definitionDefaultRunnerLabels} from '../config.js';\nimport type {IntegrationValidationContext} from './entities/integration-context.js';\nimport type {ValidationWarning} from './entities/validation-warning.js';\nimport type {WorkflowDefinitionPayload} from './entities/workflow-definition.js';\nimport {\n  InvalidWorkflowModelError,\n  normalizeWorkflowDocument,\n  type WorkflowModelValidationIssue,\n} from './workflow-model/index.js';\nimport {InvalidWorkflowYamlError, parseWorkflowYamlWithLocations} from './workflow-yaml/index.js';\n\nexport type ValidationError = {message: string; path?: string | undefined};\nexport type {ValidationWarning} from './entities/validation-warning.js';\n\nexport interface DefinitionValidationOptions {\n  defaultRunnerLabels?: readonly string[];\n  agentValidationCatalog: AgentValidationCatalog;\n  integrationValidationContext?: IntegrationValidationContext;\n}\n\nexport type ValidationResult =\n  | {valid: true; definition: WorkflowDefinitionPayload; warnings: ValidationWarning[]}\n  | {valid: false; errors: ValidationError[]};\n\nexport function validateDefinition(\n  yamlContent: string,\n  options: DefinitionValidationOptions,\n): ValidationResult {\n  try {\n    const {document, stepSourceLocations} = parseWorkflowYamlWithLocations(yamlContent);\n    const warnings: WorkflowModelValidationIssue[] = [];\n    const model = normalizeWorkflowDocument(document, {\n      defaultRunnerLabels: options.defaultRunnerLabels ?? definitionDefaultRunnerLabels,\n      agentValidationCatalog: options.agentValidationCatalog,\n      integrationValidationContext: options.integrationValidationContext,\n      stepSourceLocations,\n      warnings,\n    });\n    return {valid: true, definition: {document, model}, warnings: validationWarningsFor(warnings)};\n  } catch (error) {\n    return {valid: false, errors: validationErrorsFor(error)};\n  }\n}\n\nfunction validationWarningsFor(\n  issues: readonly WorkflowModelValidationIssue[],\n): ValidationWarning[] {\n  return issues.map((issue) =>\n    validationWarning({\n      code: issue.code,\n      message: issue.message,\n      path: issue.path.join('.') || undefined,\n    }),\n  );\n}\n\nfunction validationErrorsFor(error: unknown): ValidationError[] {\n  if (error instanceof InvalidWorkflowYamlError) {\n    return [\n      validationError({\n        message: error.message,\n        path:\n          error.location === undefined\n            ? undefined\n            : `${error.location.line}:${error.location.column}`,\n      }),\n    ];\n  }\n\n  if (error instanceof InvalidWorkflowDocumentError) {\n    return error.validationError.issues.map((issue) =>\n      validationError({\n        message: issue.message,\n        path: issue.path.join('.') || undefined,\n      }),\n    );\n  }\n\n  if (error instanceof InvalidWorkflowModelError) {\n    return error.issues.map((issue) =>\n      validationError({message: issue.message, path: issue.path.join('.')}),\n    );\n  }\n\n  throw error;\n}\n\nfunction validationError(params: {message: string; path?: string | undefined}): ValidationError {\n  if (params.path === undefined) return {message: params.message};\n  return {message: params.message, path: params.path};\n}\n\nfunction validationWarning(params: {\n  code: string;\n  message: string;\n  path?: string | undefined;\n}): ValidationWarning {\n  if (params.path === undefined) return {code: params.code, message: params.message};\n  return {code: params.code, message: params.message, path: params.path};\n}\n"],"names":["InvalidWorkflowDocumentError","definitionDefaultRunnerLabels","InvalidWorkflowModelError","normalizeWorkflowDocument","InvalidWorkflowYamlError","parseWorkflowYamlWithLocations","validateDefinition","yamlContent","options","document","stepSourceLocations","warnings","model","defaultRunnerLabels","agentValidationCatalog","integrationValidationContext","valid","definition","validationWarningsFor","error","errors","validationErrorsFor","issues","map","issue","validationWarning","code","message","path","join","undefined","validationError","location","line","column","params"],"mappings":"AACA,SAAQA,4BAA4B,QAAO,6BAA6B;AACxE,SAAQC,6BAA6B,QAAO,eAAe;AAI3D,SACEC,yBAAyB,EACzBC,yBAAyB,QAEpB,4BAA4B;AACnC,SAAQC,wBAAwB,EAAEC,8BAA8B,QAAO,2BAA2B;AAelG,OAAO,SAASC,mBACdC,WAAmB,EACnBC,OAAoC;IAEpC,IAAI;QACF,MAAM,EAACC,QAAQ,EAAEC,mBAAmB,EAAC,GAAGL,+BAA+BE;QACvE,MAAMI,WAA2C,EAAE;QACnD,MAAMC,QAAQT,0BAA0BM,UAAU;YAChDI,qBAAqBL,QAAQK,mBAAmB,IAAIZ;YACpDa,wBAAwBN,QAAQM,sBAAsB;YACtDC,8BAA8BP,QAAQO,4BAA4B;YAClEL;YACAC;QACF;QACA,OAAO;YAACK,OAAO;YAAMC,YAAY;gBAACR;gBAAUG;YAAK;YAAGD,UAAUO,sBAAsBP;QAAS;IAC/F,EAAE,OAAOQ,OAAO;QACd,OAAO;YAACH,OAAO;YAAOI,QAAQC,oBAAoBF;QAAM;IAC1D;AACF;AAEA,SAASD,sBACPI,MAA+C;IAE/C,OAAOA,OAAOC,GAAG,CAAC,CAACC,QACjBC,kBAAkB;YAChBC,MAAMF,MAAME,IAAI;YAChBC,SAASH,MAAMG,OAAO;YACtBC,MAAMJ,MAAMI,IAAI,CAACC,IAAI,CAAC,QAAQC;QAChC;AAEJ;AAEA,SAAST,oBAAoBF,KAAc;IACzC,IAAIA,iBAAiBf,0BAA0B;QAC7C,OAAO;YACL2B,gBAAgB;gBACdJ,SAASR,MAAMQ,OAAO;gBACtBC,MACET,MAAMa,QAAQ,KAAKF,YACfA,YACA,GAAGX,MAAMa,QAAQ,CAACC,IAAI,CAAC,CAAC,EAAEd,MAAMa,QAAQ,CAACE,MAAM,EAAE;YACzD;SACD;IACH;IAEA,IAAIf,iBAAiBnB,8BAA8B;QACjD,OAAOmB,MAAMY,eAAe,CAACT,MAAM,CAACC,GAAG,CAAC,CAACC,QACvCO,gBAAgB;gBACdJ,SAASH,MAAMG,OAAO;gBACtBC,MAAMJ,MAAMI,IAAI,CAACC,IAAI,CAAC,QAAQC;YAChC;IAEJ;IAEA,IAAIX,iBAAiBjB,2BAA2B;QAC9C,OAAOiB,MAAMG,MAAM,CAACC,GAAG,CAAC,CAACC,QACvBO,gBAAgB;gBAACJ,SAASH,MAAMG,OAAO;gBAAEC,MAAMJ,MAAMI,IAAI,CAACC,IAAI,CAAC;YAAI;IAEvE;IAEA,MAAMV;AACR;AAEA,SAASY,gBAAgBI,MAAoD;IAC3E,IAAIA,OAAOP,IAAI,KAAKE,WAAW,OAAO;QAACH,SAASQ,OAAOR,OAAO;IAAA;IAC9D,OAAO;QAACA,SAASQ,OAAOR,OAAO;QAAEC,MAAMO,OAAOP,IAAI;IAAA;AACpD;AAEA,SAASH,kBAAkBU,MAI1B;IACC,IAAIA,OAAOP,IAAI,KAAKE,WAAW,OAAO;QAACJ,MAAMS,OAAOT,IAAI;QAAEC,SAASQ,OAAOR,OAAO;IAAA;IACjF,OAAO;QAACD,MAAMS,OAAOT,IAAI;QAAEC,SAASQ,OAAOR,OAAO;QAAEC,MAAMO,OAAOP,IAAI;IAAA;AACvE"}
+{"version":3,"sources":["../../src/core/validate-definition.ts"],"sourcesContent":["import type {AgentValidationCatalog} from '@shipfox/api-agent-dto/inter-module';\nimport {InvalidWorkflowDocumentError} from '@shipfox/workflow-document';\nimport {definitionDefaultRunnerLabels} from '../config.js';\nimport type {IntegrationValidationContext} from './entities/integration-context.js';\nimport type {ValidationDiagnostic} from './entities/validation-diagnostic.js';\nimport type {WorkflowDefinitionPayload} from './entities/workflow-definition.js';\nimport {\n  InvalidWorkflowModelError,\n  normalizeWorkflowDocument,\n  type WorkflowModelValidationIssue,\n} from './workflow-model/index.js';\nimport {InvalidWorkflowYamlError, parseWorkflowYamlWithLocations} from './workflow-yaml/index.js';\n\nexport type ValidationError = {message: string; path?: string | undefined};\nexport type {ValidationDiagnostic} from './entities/validation-diagnostic.js';\n\nexport interface DefinitionValidationOptions {\n  defaultRunnerLabels?: readonly string[];\n  agentValidationCatalog: AgentValidationCatalog;\n  integrationValidationContext?: IntegrationValidationContext;\n}\n\nexport type ValidationResult =\n  | {valid: true; definition: WorkflowDefinitionPayload; diagnostics: ValidationDiagnostic[]}\n  | {valid: false; errors: ValidationError[]};\n\nexport function validateDefinition(\n  yamlContent: string,\n  options: DefinitionValidationOptions,\n): ValidationResult {\n  try {\n    const {document, stepSourceLocations} = parseWorkflowYamlWithLocations(yamlContent);\n    const diagnostics: WorkflowModelValidationIssue[] = [];\n    const model = normalizeWorkflowDocument(document, {\n      defaultRunnerLabels: options.defaultRunnerLabels ?? definitionDefaultRunnerLabels,\n      agentValidationCatalog: options.agentValidationCatalog,\n      integrationValidationContext: options.integrationValidationContext,\n      stepSourceLocations,\n      diagnostics,\n    });\n    return {\n      valid: true,\n      definition: {document, model},\n      diagnostics: validationDiagnosticsFor(diagnostics),\n    };\n  } catch (error) {\n    return {valid: false, errors: validationErrorsFor(error)};\n  }\n}\n\nfunction validationDiagnosticsFor(\n  issues: readonly WorkflowModelValidationIssue[],\n): ValidationDiagnostic[] {\n  return issues.map((issue) =>\n    validationDiagnostic({\n      code: issue.code,\n      message: issue.message,\n      path: issue.path.join('.') || undefined,\n      severity: issue.severity,\n    }),\n  );\n}\n\nfunction validationErrorsFor(error: unknown): ValidationError[] {\n  if (error instanceof InvalidWorkflowYamlError) {\n    return [\n      validationError({\n        message: error.message,\n        path:\n          error.location === undefined\n            ? undefined\n            : `${error.location.line}:${error.location.column}`,\n      }),\n    ];\n  }\n\n  if (error instanceof InvalidWorkflowDocumentError) {\n    return error.validationError.issues.map((issue) =>\n      validationError({\n        message: issue.message,\n        path: issue.path.join('.') || undefined,\n      }),\n    );\n  }\n\n  if (error instanceof InvalidWorkflowModelError) {\n    return error.issues.map((issue) =>\n      validationError({message: issue.message, path: issue.path.join('.')}),\n    );\n  }\n\n  throw error;\n}\n\nfunction validationError(params: {message: string; path?: string | undefined}): ValidationError {\n  if (params.path === undefined) return {message: params.message};\n  return {message: params.message, path: params.path};\n}\n\nfunction validationDiagnostic(params: {\n  code: string;\n  message: string;\n  path?: string | undefined;\n  severity: 'error' | 'warning';\n}): ValidationDiagnostic {\n  if (params.path === undefined) {\n    return {code: params.code, message: params.message, severity: params.severity};\n  }\n  return {code: params.code, message: params.message, path: params.path, severity: params.severity};\n}\n"],"names":["InvalidWorkflowDocumentError","definitionDefaultRunnerLabels","InvalidWorkflowModelError","normalizeWorkflowDocument","InvalidWorkflowYamlError","parseWorkflowYamlWithLocations","validateDefinition","yamlContent","options","document","stepSourceLocations","diagnostics","model","defaultRunnerLabels","agentValidationCatalog","integrationValidationContext","valid","definition","validationDiagnosticsFor","error","errors","validationErrorsFor","issues","map","issue","validationDiagnostic","code","message","path","join","undefined","severity","validationError","location","line","column","params"],"mappings":"AACA,SAAQA,4BAA4B,QAAO,6BAA6B;AACxE,SAAQC,6BAA6B,QAAO,eAAe;AAI3D,SACEC,yBAAyB,EACzBC,yBAAyB,QAEpB,4BAA4B;AACnC,SAAQC,wBAAwB,EAAEC,8BAA8B,QAAO,2BAA2B;AAelG,OAAO,SAASC,mBACdC,WAAmB,EACnBC,OAAoC;IAEpC,IAAI;QACF,MAAM,EAACC,QAAQ,EAAEC,mBAAmB,EAAC,GAAGL,+BAA+BE;QACvE,MAAMI,cAA8C,EAAE;QACtD,MAAMC,QAAQT,0BAA0BM,UAAU;YAChDI,qBAAqBL,QAAQK,mBAAmB,IAAIZ;YACpDa,wBAAwBN,QAAQM,sBAAsB;YACtDC,8BAA8BP,QAAQO,4BAA4B;YAClEL;YACAC;QACF;QACA,OAAO;YACLK,OAAO;YACPC,YAAY;gBAACR;gBAAUG;YAAK;YAC5BD,aAAaO,yBAAyBP;QACxC;IACF,EAAE,OAAOQ,OAAO;QACd,OAAO;YAACH,OAAO;YAAOI,QAAQC,oBAAoBF;QAAM;IAC1D;AACF;AAEA,SAASD,yBACPI,MAA+C;IAE/C,OAAOA,OAAOC,GAAG,CAAC,CAACC,QACjBC,qBAAqB;YACnBC,MAAMF,MAAME,IAAI;YAChBC,SAASH,MAAMG,OAAO;YACtBC,MAAMJ,MAAMI,IAAI,CAACC,IAAI,CAAC,QAAQC;YAC9BC,UAAUP,MAAMO,QAAQ;QAC1B;AAEJ;AAEA,SAASV,oBAAoBF,KAAc;IACzC,IAAIA,iBAAiBf,0BAA0B;QAC7C,OAAO;YACL4B,gBAAgB;gBACdL,SAASR,MAAMQ,OAAO;gBACtBC,MACET,MAAMc,QAAQ,KAAKH,YACfA,YACA,GAAGX,MAAMc,QAAQ,CAACC,IAAI,CAAC,CAAC,EAAEf,MAAMc,QAAQ,CAACE,MAAM,EAAE;YACzD;SACD;IACH;IAEA,IAAIhB,iBAAiBnB,8BAA8B;QACjD,OAAOmB,MAAMa,eAAe,CAACV,MAAM,CAACC,GAAG,CAAC,CAACC,QACvCQ,gBAAgB;gBACdL,SAASH,MAAMG,OAAO;gBACtBC,MAAMJ,MAAMI,IAAI,CAACC,IAAI,CAAC,QAAQC;YAChC;IAEJ;IAEA,IAAIX,iBAAiBjB,2BAA2B;QAC9C,OAAOiB,MAAMG,MAAM,CAACC,GAAG,CAAC,CAACC,QACvBQ,gBAAgB;gBAACL,SAASH,MAAMG,OAAO;gBAAEC,MAAMJ,MAAMI,IAAI,CAACC,IAAI,CAAC;YAAI;IAEvE;IAEA,MAAMV;AACR;AAEA,SAASa,gBAAgBI,MAAoD;IAC3E,IAAIA,OAAOR,IAAI,KAAKE,WAAW,OAAO;QAACH,SAASS,OAAOT,OAAO;IAAA;IAC9D,OAAO;QAACA,SAASS,OAAOT,OAAO;QAAEC,MAAMQ,OAAOR,IAAI;IAAA;AACpD;AAEA,SAASH,qBAAqBW,MAK7B;IACC,IAAIA,OAAOR,IAAI,KAAKE,WAAW;QAC7B,OAAO;YAACJ,MAAMU,OAAOV,IAAI;YAAEC,SAASS,OAAOT,OAAO;YAAEI,UAAUK,OAAOL,QAAQ;QAAA;IAC/E;IACA,OAAO;QAACL,MAAMU,OAAOV,IAAI;QAAEC,SAASS,OAAOT,OAAO;QAAEC,MAAMQ,OAAOR,IAAI;QAAEG,UAAUK,OAAOL,QAAQ;IAAA;AAClG"}

package/dist/core/workflow-model/cron-trigger.d.ts

@@ -3,7 +3,7 @@ export declare const cronTriggerDefaultTimezone = "UTC";
 export declare function validateCronTrigger(params: {
     readonly sourceKey: string;
     readonly trigger: {
-        readonly event: string;
+        readonly event?: string | undefined;
     };
     readonly config: {
         readonly schedule?: string | undefined;

package/dist/core/workflow-model/cron-trigger.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cron-trigger.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/cron-trigger.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAC,4BAA4B,EAAC,MAAM,mCAAmC,CAAC;AAGpF,eAAO,MAAM,0BAA0B,QAAQ,CAAC;AAGhD,wBAAgB,mBAAmB,CAAC,MAAM,EAAE;IAC1C,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,OAAO,EAAE;QAChB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;KACxB,CAAC;IACF,QAAQ,CAAC,MAAM,EAAE;QACf,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;QACvC,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;KACxC,CAAC;IACF,QAAQ,CAAC,MAAM,EAAE,4BAA4B,EAAE,CAAC;CACjD,GAAG,IAAI,CA2CP;AAED,wBAAgB,qBAAqB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAWjE;AAED,wBAAgB,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAOzD"}
+{"version":3,"file":"cron-trigger.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/cron-trigger.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAC,4BAA4B,EAAC,MAAM,mCAAmC,CAAC;AAGpF,eAAO,MAAM,0BAA0B,QAAQ,CAAC;AAGhD,wBAAgB,mBAAmB,CAAC,MAAM,EAAE;IAC1C,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,OAAO,EAAE;QAChB,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;KACrC,CAAC;IACF,QAAQ,CAAC,MAAM,EAAE;QACf,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;QACvC,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;KACxC,CAAC;IACF,QAAQ,CAAC,MAAM,EAAE,4BAA4B,EAAE,CAAC;CACjD,GAAG,IAAI,CA+CP;AAED,wBAAgB,qBAAqB,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAWjE;AAED,wBAAgB,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAOzD"}

package/dist/core/workflow-model/cron-trigger.js

@@ -4,7 +4,7 @@ export const cronTriggerDefaultTimezone = 'UTC';
 const cronFieldSeparatorRegex = /\s+/;
 export function validateCronTrigger(params) {
     const { sourceKey, trigger, config, issues } = params;
-    if (trigger.event !== 'tick') {
+    if (trigger.event !== undefined && trigger.event !== 'tick') {
         issues.push(issue({
             code: 'invalid-cron-event',
             message: `A cron trigger must use event "tick"; found "${trigger.event}".`,
@@ -15,7 +15,8 @@ export function validateCronTrigger(params) {
             ],
             details: {
                 event: trigger.event
-            }
+            },
+            scope: 'trigger'
         }));
     }
     if (config.schedule === undefined) {
@@ -27,7 +28,8 @@ export function validateCronTrigger(params) {
                 sourceKey,
                 'config',
                 'schedule'
-            ]
+            ],
+            scope: 'trigger'
         }));
     } else if (!isValidCronExpression(config.schedule)) {
         issues.push(issue({
@@ -41,7 +43,8 @@ export function validateCronTrigger(params) {
             ],
             details: {
                 schedule: config.schedule
-            }
+            },
+            scope: 'trigger'
         }));
     }
     if (config.timezone !== undefined && !isValidTimezone(config.timezone)) {
@@ -56,7 +59,8 @@ export function validateCronTrigger(params) {
             ],
             details: {
                 timezone: config.timezone
-            }
+            },
+            scope: 'trigger'
         }));
     }
 }

package/dist/core/workflow-model/cron-trigger.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../../src/core/workflow-model/cron-trigger.ts"],"sourcesContent":["import {CronExpressionParser} from 'cron-parser';\nimport type {WorkflowModelValidationIssue} from './invalid-workflow-model-error.js';\nimport {issue} from './validation-issue.js';\n\nexport const cronTriggerDefaultTimezone = 'UTC';\nconst cronFieldSeparatorRegex = /\\s+/;\n\nexport function validateCronTrigger(params: {\n  readonly sourceKey: string;\n  readonly trigger: {\n    readonly event: string;\n  };\n  readonly config: {\n    readonly schedule?: string | undefined;\n    readonly timezone?: string | undefined;\n  };\n  readonly issues: WorkflowModelValidationIssue[];\n}): void {\n  const {sourceKey, trigger, config, issues} = params;\n\n  if (trigger.event !== 'tick') {\n    issues.push(\n      issue({\n        code: 'invalid-cron-event',\n        message: `A cron trigger must use event \"tick\"; found \"${trigger.event}\".`,\n        path: ['triggers', sourceKey, 'event'],\n        details: {event: trigger.event},\n      }),\n    );\n  }\n\n  if (config.schedule === undefined) {\n    issues.push(\n      issue({\n        code: 'missing-cron-schedule',\n        message: 'A cron trigger requires a schedule.',\n        path: ['triggers', sourceKey, 'config', 'schedule'],\n      }),\n    );\n  } else if (!isValidCronExpression(config.schedule)) {\n    issues.push(\n      issue({\n        code: 'invalid-cron-schedule',\n        message: 'Cron trigger schedule must be a valid 5-field cron expression.',\n        path: ['triggers', sourceKey, 'config', 'schedule'],\n        details: {schedule: config.schedule},\n      }),\n    );\n  }\n\n  if (config.timezone !== undefined && !isValidTimezone(config.timezone)) {\n    issues.push(\n      issue({\n        code: 'invalid-cron-timezone',\n        message: 'Cron trigger timezone must be a valid IANA time zone.',\n        path: ['triggers', sourceKey, 'config', 'timezone'],\n        details: {timezone: config.timezone},\n      }),\n    );\n  }\n}\n\nexport function isValidCronExpression(expression: string): boolean {\n  const trimmedExpression = expression.trim();\n  if (trimmedExpression.length === 0) return false;\n  if (trimmedExpression.split(cronFieldSeparatorRegex).length !== 5) return false;\n\n  try {\n    CronExpressionParser.parse(trimmedExpression);\n    return true;\n  } catch {\n    return false;\n  }\n}\n\nexport function isValidTimezone(timezone: string): boolean {\n  try {\n    new Intl.DateTimeFormat('en-US', {timeZone: timezone});\n    return true;\n  } catch {\n    return false;\n  }\n}\n"],"names":["CronExpressionParser","issue","cronTriggerDefaultTimezone","cronFieldSeparatorRegex","validateCronTrigger","params","sourceKey","trigger","config","issues","event","push","code","message","path","details","schedule","undefined","isValidCronExpression","timezone","isValidTimezone","expression","trimmedExpression","trim","length","split","parse","Intl","DateTimeFormat","timeZone"],"mappings":"AAAA,SAAQA,oBAAoB,QAAO,cAAc;AAEjD,SAAQC,KAAK,QAAO,wBAAwB;AAE5C,OAAO,MAAMC,6BAA6B,MAAM;AAChD,MAAMC,0BAA0B;AAEhC,OAAO,SAASC,oBAAoBC,MAUnC;IACC,MAAM,EAACC,SAAS,EAAEC,OAAO,EAAEC,MAAM,EAAEC,MAAM,EAAC,GAAGJ;IAE7C,IAAIE,QAAQG,KAAK,KAAK,QAAQ;QAC5BD,OAAOE,IAAI,CACTV,MAAM;YACJW,MAAM;YACNC,SAAS,CAAC,6CAA6C,EAAEN,QAAQG,KAAK,CAAC,EAAE,CAAC;YAC1EI,MAAM;gBAAC;gBAAYR;gBAAW;aAAQ;YACtCS,SAAS;gBAACL,OAAOH,QAAQG,KAAK;YAAA;QAChC;IAEJ;IAEA,IAAIF,OAAOQ,QAAQ,KAAKC,WAAW;QACjCR,OAAOE,IAAI,CACTV,MAAM;YACJW,MAAM;YACNC,SAAS;YACTC,MAAM;gBAAC;gBAAYR;gBAAW;gBAAU;aAAW;QACrD;IAEJ,OAAO,IAAI,CAACY,sBAAsBV,OAAOQ,QAAQ,GAAG;QAClDP,OAAOE,IAAI,CACTV,MAAM;YACJW,MAAM;YACNC,SAAS;YACTC,MAAM;gBAAC;gBAAYR;gBAAW;gBAAU;aAAW;YACnDS,SAAS;gBAACC,UAAUR,OAAOQ,QAAQ;YAAA;QACrC;IAEJ;IAEA,IAAIR,OAAOW,QAAQ,KAAKF,aAAa,CAACG,gBAAgBZ,OAAOW,QAAQ,GAAG;QACtEV,OAAOE,IAAI,CACTV,MAAM;YACJW,MAAM;YACNC,SAAS;YACTC,MAAM;gBAAC;gBAAYR;gBAAW;gBAAU;aAAW;YACnDS,SAAS;gBAACI,UAAUX,OAAOW,QAAQ;YAAA;QACrC;IAEJ;AACF;AAEA,OAAO,SAASD,sBAAsBG,UAAkB;IACtD,MAAMC,oBAAoBD,WAAWE,IAAI;IACzC,IAAID,kBAAkBE,MAAM,KAAK,GAAG,OAAO;IAC3C,IAAIF,kBAAkBG,KAAK,CAACtB,yBAAyBqB,MAAM,KAAK,GAAG,OAAO;IAE1E,IAAI;QACFxB,qBAAqB0B,KAAK,CAACJ;QAC3B,OAAO;IACT,EAAE,OAAM;QACN,OAAO;IACT;AACF;AAEA,OAAO,SAASF,gBAAgBD,QAAgB;IAC9C,IAAI;QACF,IAAIQ,KAAKC,cAAc,CAAC,SAAS;YAACC,UAAUV;QAAQ;QACpD,OAAO;IACT,EAAE,OAAM;QACN,OAAO;IACT;AACF"}
+{"version":3,"sources":["../../../src/core/workflow-model/cron-trigger.ts"],"sourcesContent":["import {CronExpressionParser} from 'cron-parser';\nimport type {WorkflowModelValidationIssue} from './invalid-workflow-model-error.js';\nimport {issue} from './validation-issue.js';\n\nexport const cronTriggerDefaultTimezone = 'UTC';\nconst cronFieldSeparatorRegex = /\\s+/;\n\nexport function validateCronTrigger(params: {\n  readonly sourceKey: string;\n  readonly trigger: {\n    readonly event?: string | undefined;\n  };\n  readonly config: {\n    readonly schedule?: string | undefined;\n    readonly timezone?: string | undefined;\n  };\n  readonly issues: WorkflowModelValidationIssue[];\n}): void {\n  const {sourceKey, trigger, config, issues} = params;\n\n  if (trigger.event !== undefined && trigger.event !== 'tick') {\n    issues.push(\n      issue({\n        code: 'invalid-cron-event',\n        message: `A cron trigger must use event \"tick\"; found \"${trigger.event}\".`,\n        path: ['triggers', sourceKey, 'event'],\n        details: {event: trigger.event},\n        scope: 'trigger',\n      }),\n    );\n  }\n\n  if (config.schedule === undefined) {\n    issues.push(\n      issue({\n        code: 'missing-cron-schedule',\n        message: 'A cron trigger requires a schedule.',\n        path: ['triggers', sourceKey, 'config', 'schedule'],\n        scope: 'trigger',\n      }),\n    );\n  } else if (!isValidCronExpression(config.schedule)) {\n    issues.push(\n      issue({\n        code: 'invalid-cron-schedule',\n        message: 'Cron trigger schedule must be a valid 5-field cron expression.',\n        path: ['triggers', sourceKey, 'config', 'schedule'],\n        details: {schedule: config.schedule},\n        scope: 'trigger',\n      }),\n    );\n  }\n\n  if (config.timezone !== undefined && !isValidTimezone(config.timezone)) {\n    issues.push(\n      issue({\n        code: 'invalid-cron-timezone',\n        message: 'Cron trigger timezone must be a valid IANA time zone.',\n        path: ['triggers', sourceKey, 'config', 'timezone'],\n        details: {timezone: config.timezone},\n        scope: 'trigger',\n      }),\n    );\n  }\n}\n\nexport function isValidCronExpression(expression: string): boolean {\n  const trimmedExpression = expression.trim();\n  if (trimmedExpression.length === 0) return false;\n  if (trimmedExpression.split(cronFieldSeparatorRegex).length !== 5) return false;\n\n  try {\n    CronExpressionParser.parse(trimmedExpression);\n    return true;\n  } catch {\n    return false;\n  }\n}\n\nexport function isValidTimezone(timezone: string): boolean {\n  try {\n    new Intl.DateTimeFormat('en-US', {timeZone: timezone});\n    return true;\n  } catch {\n    return false;\n  }\n}\n"],"names":["CronExpressionParser","issue","cronTriggerDefaultTimezone","cronFieldSeparatorRegex","validateCronTrigger","params","sourceKey","trigger","config","issues","event","undefined","push","code","message","path","details","scope","schedule","isValidCronExpression","timezone","isValidTimezone","expression","trimmedExpression","trim","length","split","parse","Intl","DateTimeFormat","timeZone"],"mappings":"AAAA,SAAQA,oBAAoB,QAAO,cAAc;AAEjD,SAAQC,KAAK,QAAO,wBAAwB;AAE5C,OAAO,MAAMC,6BAA6B,MAAM;AAChD,MAAMC,0BAA0B;AAEhC,OAAO,SAASC,oBAAoBC,MAUnC;IACC,MAAM,EAACC,SAAS,EAAEC,OAAO,EAAEC,MAAM,EAAEC,MAAM,EAAC,GAAGJ;IAE7C,IAAIE,QAAQG,KAAK,KAAKC,aAAaJ,QAAQG,KAAK,KAAK,QAAQ;QAC3DD,OAAOG,IAAI,CACTX,MAAM;YACJY,MAAM;YACNC,SAAS,CAAC,6CAA6C,EAAEP,QAAQG,KAAK,CAAC,EAAE,CAAC;YAC1EK,MAAM;gBAAC;gBAAYT;gBAAW;aAAQ;YACtCU,SAAS;gBAACN,OAAOH,QAAQG,KAAK;YAAA;YAC9BO,OAAO;QACT;IAEJ;IAEA,IAAIT,OAAOU,QAAQ,KAAKP,WAAW;QACjCF,OAAOG,IAAI,CACTX,MAAM;YACJY,MAAM;YACNC,SAAS;YACTC,MAAM;gBAAC;gBAAYT;gBAAW;gBAAU;aAAW;YACnDW,OAAO;QACT;IAEJ,OAAO,IAAI,CAACE,sBAAsBX,OAAOU,QAAQ,GAAG;QAClDT,OAAOG,IAAI,CACTX,MAAM;YACJY,MAAM;YACNC,SAAS;YACTC,MAAM;gBAAC;gBAAYT;gBAAW;gBAAU;aAAW;YACnDU,SAAS;gBAACE,UAAUV,OAAOU,QAAQ;YAAA;YACnCD,OAAO;QACT;IAEJ;IAEA,IAAIT,OAAOY,QAAQ,KAAKT,aAAa,CAACU,gBAAgBb,OAAOY,QAAQ,GAAG;QACtEX,OAAOG,IAAI,CACTX,MAAM;YACJY,MAAM;YACNC,SAAS;YACTC,MAAM;gBAAC;gBAAYT;gBAAW;gBAAU;aAAW;YACnDU,SAAS;gBAACI,UAAUZ,OAAOY,QAAQ;YAAA;YACnCH,OAAO;QACT;IAEJ;AACF;AAEA,OAAO,SAASE,sBAAsBG,UAAkB;IACtD,MAAMC,oBAAoBD,WAAWE,IAAI;IACzC,IAAID,kBAAkBE,MAAM,KAAK,GAAG,OAAO;IAC3C,IAAIF,kBAAkBG,KAAK,CAACvB,yBAAyBsB,MAAM,KAAK,GAAG,OAAO;IAE1E,IAAI;QACFzB,qBAAqB2B,KAAK,CAACJ;QAC3B,OAAO;IACT,EAAE,OAAM;QACN,OAAO;IACT;AACF;AAEA,OAAO,SAASF,gBAAgBD,QAAgB;IAC9C,IAAI;QACF,IAAIQ,KAAKC,cAAc,CAAC,SAAS;YAACC,UAAUV;QAAQ;QACpD,OAAO;IACT,EAAE,OAAM;QACN,OAAO;IACT;AACF"}

package/dist/core/workflow-model/index.d.ts

@@ -1,4 +1,4 @@
 export { DEFAULT_RUN_TIMEOUT_MS } from './constants.js';
-export { InvalidWorkflowModelError, invalidWorkflowModelErrorCode, type WorkflowModelValidationIssue, type WorkflowModelValidationIssueCode, type WorkflowModelValidationIssuePathSegment, type WorkflowModelValidationIssueSeverity, } from './invalid-workflow-model-error.js';
+export { InvalidWorkflowModelError, invalidWorkflowModelErrorCode, type WorkflowModelValidationIssue, type WorkflowModelValidationIssueCode, type WorkflowModelValidationIssuePathSegment, type WorkflowModelValidationIssueScope, type WorkflowModelValidationIssueSeverity, } from './invalid-workflow-model-error.js';
 export { normalizeWorkflowDocument } from './normalize-workflow-document.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/core/workflow-model/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,sBAAsB,EAAC,MAAM,gBAAgB,CAAC;AACtD,OAAO,EACL,yBAAyB,EACzB,6BAA6B,EAC7B,KAAK,4BAA4B,EACjC,KAAK,gCAAgC,EACrC,KAAK,uCAAuC,EAC5C,KAAK,oCAAoC,GAC1C,MAAM,mCAAmC,CAAC;AAC3C,OAAO,EAAC,yBAAyB,EAAC,MAAM,kCAAkC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,sBAAsB,EAAC,MAAM,gBAAgB,CAAC;AACtD,OAAO,EACL,yBAAyB,EACzB,6BAA6B,EAC7B,KAAK,4BAA4B,EACjC,KAAK,gCAAgC,EACrC,KAAK,uCAAuC,EAC5C,KAAK,iCAAiC,EACtC,KAAK,oCAAoC,GAC1C,MAAM,mCAAmC,CAAC;AAC3C,OAAO,EAAC,yBAAyB,EAAC,MAAM,kCAAkC,CAAC"}

package/dist/core/workflow-model/index.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../../src/core/workflow-model/index.ts"],"sourcesContent":["export {DEFAULT_RUN_TIMEOUT_MS} from './constants.js';\nexport {\n  InvalidWorkflowModelError,\n  invalidWorkflowModelErrorCode,\n  type WorkflowModelValidationIssue,\n  type WorkflowModelValidationIssueCode,\n  type WorkflowModelValidationIssuePathSegment,\n  type WorkflowModelValidationIssueSeverity,\n} from './invalid-workflow-model-error.js';\nexport {normalizeWorkflowDocument} from './normalize-workflow-document.js';\n"],"names":["DEFAULT_RUN_TIMEOUT_MS","InvalidWorkflowModelError","invalidWorkflowModelErrorCode","normalizeWorkflowDocument"],"mappings":"AAAA,SAAQA,sBAAsB,QAAO,iBAAiB;AACtD,SACEC,yBAAyB,EACzBC,6BAA6B,QAKxB,oCAAoC;AAC3C,SAAQC,yBAAyB,QAAO,mCAAmC"}
+{"version":3,"sources":["../../../src/core/workflow-model/index.ts"],"sourcesContent":["export {DEFAULT_RUN_TIMEOUT_MS} from './constants.js';\nexport {\n  InvalidWorkflowModelError,\n  invalidWorkflowModelErrorCode,\n  type WorkflowModelValidationIssue,\n  type WorkflowModelValidationIssueCode,\n  type WorkflowModelValidationIssuePathSegment,\n  type WorkflowModelValidationIssueScope,\n  type WorkflowModelValidationIssueSeverity,\n} from './invalid-workflow-model-error.js';\nexport {normalizeWorkflowDocument} from './normalize-workflow-document.js';\n"],"names":["DEFAULT_RUN_TIMEOUT_MS","InvalidWorkflowModelError","invalidWorkflowModelErrorCode","normalizeWorkflowDocument"],"mappings":"AAAA,SAAQA,sBAAsB,QAAO,iBAAiB;AACtD,SACEC,yBAAyB,EACzBC,6BAA6B,QAMxB,oCAAoC;AAC3C,SAAQC,yBAAyB,QAAO,mCAAmC"}

package/dist/core/workflow-model/invalid-workflow-model-error.d.ts

@@ -1,13 +1,15 @@
 export declare const invalidWorkflowModelErrorCode = "invalid-workflow-model";
-export type WorkflowModelValidationIssueCode = 'context-unavailable-at-fill-site' | 'dynamic-name-self-reference' | 'context-unavailable-at-predicate-site' | 'computed-context-key' | 'checkout-target-invalid' | 'duplicate-job-id' | 'duplicate-step-id' | 'duplicate-trigger-id' | 'harness-provider-incompatible' | 'harness-thinking-incompatible' | 'harness-tool-incompatible' | 'integration-connection-not-capable' | 'integration-connection-not-found' | 'integration-write-not-allowed' | 'invalid-cron-event' | 'invalid-cron-schedule' | 'invalid-cron-timezone' | 'invalid-provider' | 'invalid-trigger-filter' | 'invalid-interpolation-expression' | 'invalid-interpolation-template' | 'invalid-duration' | 'invalid-listener-filter' | 'invalid-model' | 'invalid-job-if' | 'invalid-job-success' | 'invalid-output-schema' | 'invalid-runner-label' | 'invalid-step-gate-restart-from' | 'invalid-step-gate-success' | 'invalid-step-if' | 'job-dependency-cycle' | 'listening-job-missing-resolution-source' | 'listening-timeout-exceeds-run-timeout' | 'missing-harness-for-tools' | 'missing-connection-for-integration' | 'missing-job-needs-edge' | 'missing-cron-schedule' | 'missing-runner-label' | 'multiple-manual-triggers' | 'runner-context-not-bare' | 'runner-context-in-field' | 'runner-context-in-server-predicate' | 're-evaluating-command' | 'self-job-dependency' | 'too-many-runner-labels' | 'unknown-secret-store' | 'unknown-interpolation-context' | 'unknown-integration-method' | 'unknown-integration-tool' | 'unknown-job-dependency' | 'untrusted-agent-selection-context';
+export type WorkflowModelValidationIssueCode = 'context-unavailable-at-fill-site' | 'dynamic-name-self-reference' | 'context-unavailable-at-predicate-site' | 'computed-context-key' | 'checkout-target-invalid' | 'duplicate-job-id' | 'duplicate-step-id' | 'duplicate-trigger-id' | 'harness-provider-incompatible' | 'harness-thinking-incompatible' | 'harness-tool-incompatible' | 'integration-connection-not-capable' | 'integration-connection-not-found' | 'integration-write-not-allowed' | 'invalid-cron-event' | 'invalid-cron-schedule' | 'invalid-cron-timezone' | 'invalid-provider' | 'invalid-trigger-filter' | 'invalid-interpolation-expression' | 'invalid-interpolation-template' | 'invalid-duration' | 'invalid-listener-filter' | 'invalid-model' | 'invalid-job-if' | 'invalid-job-success' | 'invalid-output-schema' | 'invalid-runner-label' | 'invalid-step-gate-restart-from' | 'invalid-step-gate-success' | 'invalid-step-if' | 'job-dependency-cycle' | 'listening-job-missing-resolution-source' | 'listening-job-no-active-matcher' | 'listening-timeout-exceeds-run-timeout' | 'missing-harness-for-tools' | 'missing-connection-for-integration' | 'missing-job-needs-edge' | 'missing-cron-schedule' | 'missing-runner-label' | 'multiple-manual-triggers' | 'runner-context-not-bare' | 'runner-context-in-field' | 'runner-context-in-server-predicate' | 're-evaluating-command' | 'self-job-dependency' | 'too-many-runner-labels' | 'unknown-secret-store' | 'unknown-interpolation-context' | 'unknown-integration-method' | 'unknown-integration-tool' | 'unknown-job-dependency' | 'untrusted-agent-selection-context';
 export type WorkflowModelValidationIssuePathSegment = string | number;
 export type WorkflowModelValidationIssueSeverity = 'error' | 'warning';
+export type WorkflowModelValidationIssueScope = 'definition' | 'trigger';
 export interface WorkflowModelValidationIssue {
     readonly code: WorkflowModelValidationIssueCode;
     readonly message: string;
     readonly path: readonly WorkflowModelValidationIssuePathSegment[];
     readonly details?: Readonly<Record<string, unknown>>;
-    readonly severity?: WorkflowModelValidationIssueSeverity;
+    readonly severity: WorkflowModelValidationIssueSeverity;
+    readonly scope: WorkflowModelValidationIssueScope;
 }
 export declare class InvalidWorkflowModelError extends Error {
     readonly issues: readonly WorkflowModelValidationIssue[];

package/dist/core/workflow-model/invalid-workflow-model-error.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"invalid-workflow-model-error.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/invalid-workflow-model-error.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,6BAA6B,2BAA2B,CAAC;AAEtE,MAAM,MAAM,gCAAgC,GACxC,kCAAkC,GAClC,6BAA6B,GAC7B,uCAAuC,GACvC,sBAAsB,GACtB,yBAAyB,GACzB,kBAAkB,GAClB,mBAAmB,GACnB,sBAAsB,GACtB,+BAA+B,GAC/B,+BAA+B,GAC/B,2BAA2B,GAC3B,oCAAoC,GACpC,kCAAkC,GAClC,+BAA+B,GAC/B,oBAAoB,GACpB,uBAAuB,GACvB,uBAAuB,GACvB,kBAAkB,GAClB,wBAAwB,GACxB,kCAAkC,GAClC,gCAAgC,GAChC,kBAAkB,GAClB,yBAAyB,GACzB,eAAe,GACf,gBAAgB,GAChB,qBAAqB,GACrB,uBAAuB,GACvB,sBAAsB,GACtB,gCAAgC,GAChC,2BAA2B,GAC3B,iBAAiB,GACjB,sBAAsB,GACtB,yCAAyC,GACzC,uCAAuC,GACvC,2BAA2B,GAC3B,oCAAoC,GACpC,wBAAwB,GACxB,uBAAuB,GACvB,sBAAsB,GACtB,0BAA0B,GAC1B,yBAAyB,GACzB,yBAAyB,GACzB,oCAAoC,GACpC,uBAAuB,GACvB,qBAAqB,GACrB,wBAAwB,GACxB,sBAAsB,GACtB,+BAA+B,GAC/B,4BAA4B,GAC5B,0BAA0B,GAC1B,wBAAwB,GACxB,mCAAmC,CAAC;AAExC,MAAM,MAAM,uCAAuC,GAAG,MAAM,GAAG,MAAM,CAAC;AAEtE,MAAM,MAAM,oCAAoC,GAAG,OAAO,GAAG,SAAS,CAAC;AAEvE,MAAM,WAAW,4BAA4B;IAC3C,QAAQ,CAAC,IAAI,EAAE,gCAAgC,CAAC;IAChD,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,IAAI,EAAE,SAAS,uCAAuC,EAAE,CAAC;IAClE,QAAQ,CAAC,OAAO,CAAC,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC;IACrD,QAAQ,CAAC,QAAQ,CAAC,EAAE,oCAAoC,CAAC;CAC1D;AAED,qBAAa,yBAA0B,SAAQ,KAAK;IAGtC,QAAQ,CAAC,MAAM,EAAE,SAAS,4BAA4B,EAAE;IAFpE,QAAQ,CAAC,IAAI,4BAAiC;IAE9C,YAAqB,MAAM,EAAE,SAAS,4BAA4B,EAAE,EAGnE;CACF"}
+{"version":3,"file":"invalid-workflow-model-error.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/invalid-workflow-model-error.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,6BAA6B,2BAA2B,CAAC;AAEtE,MAAM,MAAM,gCAAgC,GACxC,kCAAkC,GAClC,6BAA6B,GAC7B,uCAAuC,GACvC,sBAAsB,GACtB,yBAAyB,GACzB,kBAAkB,GAClB,mBAAmB,GACnB,sBAAsB,GACtB,+BAA+B,GAC/B,+BAA+B,GAC/B,2BAA2B,GAC3B,oCAAoC,GACpC,kCAAkC,GAClC,+BAA+B,GAC/B,oBAAoB,GACpB,uBAAuB,GACvB,uBAAuB,GACvB,kBAAkB,GAClB,wBAAwB,GACxB,kCAAkC,GAClC,gCAAgC,GAChC,kBAAkB,GAClB,yBAAyB,GACzB,eAAe,GACf,gBAAgB,GAChB,qBAAqB,GACrB,uBAAuB,GACvB,sBAAsB,GACtB,gCAAgC,GAChC,2BAA2B,GAC3B,iBAAiB,GACjB,sBAAsB,GACtB,yCAAyC,GACzC,iCAAiC,GACjC,uCAAuC,GACvC,2BAA2B,GAC3B,oCAAoC,GACpC,wBAAwB,GACxB,uBAAuB,GACvB,sBAAsB,GACtB,0BAA0B,GAC1B,yBAAyB,GACzB,yBAAyB,GACzB,oCAAoC,GACpC,uBAAuB,GACvB,qBAAqB,GACrB,wBAAwB,GACxB,sBAAsB,GACtB,+BAA+B,GAC/B,4BAA4B,GAC5B,0BAA0B,GAC1B,wBAAwB,GACxB,mCAAmC,CAAC;AAExC,MAAM,MAAM,uCAAuC,GAAG,MAAM,GAAG,MAAM,CAAC;AAEtE,MAAM,MAAM,oCAAoC,GAAG,OAAO,GAAG,SAAS,CAAC;AAEvE,MAAM,MAAM,iCAAiC,GAAG,YAAY,GAAG,SAAS,CAAC;AAEzE,MAAM,WAAW,4BAA4B;IAC3C,QAAQ,CAAC,IAAI,EAAE,gCAAgC,CAAC;IAChD,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,IAAI,EAAE,SAAS,uCAAuC,EAAE,CAAC;IAClE,QAAQ,CAAC,OAAO,CAAC,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC;IACrD,QAAQ,CAAC,QAAQ,EAAE,oCAAoC,CAAC;IACxD,QAAQ,CAAC,KAAK,EAAE,iCAAiC,CAAC;CACnD;AAED,qBAAa,yBAA0B,SAAQ,KAAK;IAGtC,QAAQ,CAAC,MAAM,EAAE,SAAS,4BAA4B,EAAE;IAFpE,QAAQ,CAAC,IAAI,4BAAiC;IAE9C,YAAqB,MAAM,EAAE,SAAS,4BAA4B,EAAE,EAGnE;CACF"}

package/dist/core/workflow-model/invalid-workflow-model-error.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../../src/core/workflow-model/invalid-workflow-model-error.ts"],"sourcesContent":["export const invalidWorkflowModelErrorCode = 'invalid-workflow-model';\n\nexport type WorkflowModelValidationIssueCode =\n  | 'context-unavailable-at-fill-site'\n  | 'dynamic-name-self-reference'\n  | 'context-unavailable-at-predicate-site'\n  | 'computed-context-key'\n  | 'checkout-target-invalid'\n  | 'duplicate-job-id'\n  | 'duplicate-step-id'\n  | 'duplicate-trigger-id'\n  | 'harness-provider-incompatible'\n  | 'harness-thinking-incompatible'\n  | 'harness-tool-incompatible'\n  | 'integration-connection-not-capable'\n  | 'integration-connection-not-found'\n  | 'integration-write-not-allowed'\n  | 'invalid-cron-event'\n  | 'invalid-cron-schedule'\n  | 'invalid-cron-timezone'\n  | 'invalid-provider'\n  | 'invalid-trigger-filter'\n  | 'invalid-interpolation-expression'\n  | 'invalid-interpolation-template'\n  | 'invalid-duration'\n  | 'invalid-listener-filter'\n  | 'invalid-model'\n  | 'invalid-job-if'\n  | 'invalid-job-success'\n  | 'invalid-output-schema'\n  | 'invalid-runner-label'\n  | 'invalid-step-gate-restart-from'\n  | 'invalid-step-gate-success'\n  | 'invalid-step-if'\n  | 'job-dependency-cycle'\n  | 'listening-job-missing-resolution-source'\n  | 'listening-timeout-exceeds-run-timeout'\n  | 'missing-harness-for-tools'\n  | 'missing-connection-for-integration'\n  | 'missing-job-needs-edge'\n  | 'missing-cron-schedule'\n  | 'missing-runner-label'\n  | 'multiple-manual-triggers'\n  | 'runner-context-not-bare'\n  | 'runner-context-in-field'\n  | 'runner-context-in-server-predicate'\n  | 're-evaluating-command'\n  | 'self-job-dependency'\n  | 'too-many-runner-labels'\n  | 'unknown-secret-store'\n  | 'unknown-interpolation-context'\n  | 'unknown-integration-method'\n  | 'unknown-integration-tool'\n  | 'unknown-job-dependency'\n  | 'untrusted-agent-selection-context';\n\nexport type WorkflowModelValidationIssuePathSegment = string | number;\n\nexport type WorkflowModelValidationIssueSeverity = 'error' | 'warning';\n\nexport interface WorkflowModelValidationIssue {\n  readonly code: WorkflowModelValidationIssueCode;\n  readonly message: string;\n  readonly path: readonly WorkflowModelValidationIssuePathSegment[];\n  readonly details?: Readonly<Record<string, unknown>>;\n  readonly severity?: WorkflowModelValidationIssueSeverity;\n}\n\nexport class InvalidWorkflowModelError extends Error {\n  readonly code = invalidWorkflowModelErrorCode;\n\n  constructor(readonly issues: readonly WorkflowModelValidationIssue[]) {\n    super('Invalid workflow model');\n    this.name = 'InvalidWorkflowModelError';\n  }\n}\n"],"names":["invalidWorkflowModelErrorCode","InvalidWorkflowModelError","Error","issues","code","name"],"mappings":"AAAA,OAAO,MAAMA,gCAAgC,yBAAyB;AAoEtE,OAAO,MAAMC,kCAAkCC;IAG7C,YAAY,AAASC,MAA+C,CAAE;QACpE,KAAK,CAAC,gCADaA,SAAAA,aAFZC,OAAOJ;QAId,IAAI,CAACK,IAAI,GAAG;IACd;AACF"}
+{"version":3,"sources":["../../../src/core/workflow-model/invalid-workflow-model-error.ts"],"sourcesContent":["export const invalidWorkflowModelErrorCode = 'invalid-workflow-model';\n\nexport type WorkflowModelValidationIssueCode =\n  | 'context-unavailable-at-fill-site'\n  | 'dynamic-name-self-reference'\n  | 'context-unavailable-at-predicate-site'\n  | 'computed-context-key'\n  | 'checkout-target-invalid'\n  | 'duplicate-job-id'\n  | 'duplicate-step-id'\n  | 'duplicate-trigger-id'\n  | 'harness-provider-incompatible'\n  | 'harness-thinking-incompatible'\n  | 'harness-tool-incompatible'\n  | 'integration-connection-not-capable'\n  | 'integration-connection-not-found'\n  | 'integration-write-not-allowed'\n  | 'invalid-cron-event'\n  | 'invalid-cron-schedule'\n  | 'invalid-cron-timezone'\n  | 'invalid-provider'\n  | 'invalid-trigger-filter'\n  | 'invalid-interpolation-expression'\n  | 'invalid-interpolation-template'\n  | 'invalid-duration'\n  | 'invalid-listener-filter'\n  | 'invalid-model'\n  | 'invalid-job-if'\n  | 'invalid-job-success'\n  | 'invalid-output-schema'\n  | 'invalid-runner-label'\n  | 'invalid-step-gate-restart-from'\n  | 'invalid-step-gate-success'\n  | 'invalid-step-if'\n  | 'job-dependency-cycle'\n  | 'listening-job-missing-resolution-source'\n  | 'listening-job-no-active-matcher'\n  | 'listening-timeout-exceeds-run-timeout'\n  | 'missing-harness-for-tools'\n  | 'missing-connection-for-integration'\n  | 'missing-job-needs-edge'\n  | 'missing-cron-schedule'\n  | 'missing-runner-label'\n  | 'multiple-manual-triggers'\n  | 'runner-context-not-bare'\n  | 'runner-context-in-field'\n  | 'runner-context-in-server-predicate'\n  | 're-evaluating-command'\n  | 'self-job-dependency'\n  | 'too-many-runner-labels'\n  | 'unknown-secret-store'\n  | 'unknown-interpolation-context'\n  | 'unknown-integration-method'\n  | 'unknown-integration-tool'\n  | 'unknown-job-dependency'\n  | 'untrusted-agent-selection-context';\n\nexport type WorkflowModelValidationIssuePathSegment = string | number;\n\nexport type WorkflowModelValidationIssueSeverity = 'error' | 'warning';\n\nexport type WorkflowModelValidationIssueScope = 'definition' | 'trigger';\n\nexport interface WorkflowModelValidationIssue {\n  readonly code: WorkflowModelValidationIssueCode;\n  readonly message: string;\n  readonly path: readonly WorkflowModelValidationIssuePathSegment[];\n  readonly details?: Readonly<Record<string, unknown>>;\n  readonly severity: WorkflowModelValidationIssueSeverity;\n  readonly scope: WorkflowModelValidationIssueScope;\n}\n\nexport class InvalidWorkflowModelError extends Error {\n  readonly code = invalidWorkflowModelErrorCode;\n\n  constructor(readonly issues: readonly WorkflowModelValidationIssue[]) {\n    super('Invalid workflow model');\n    this.name = 'InvalidWorkflowModelError';\n  }\n}\n"],"names":["invalidWorkflowModelErrorCode","InvalidWorkflowModelError","Error","issues","code","name"],"mappings":"AAAA,OAAO,MAAMA,gCAAgC,yBAAyB;AAwEtE,OAAO,MAAMC,kCAAkCC;IAG7C,YAAY,AAASC,MAA+C,CAAE;QACpE,KAAK,CAAC,gCADaA,SAAAA,aAFZC,OAAOJ;QAId,IAAI,CAACK,IAAI,GAAG;IACd;AACF"}

package/dist/core/workflow-model/normalize-job-listening.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"normalize-job-listening.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/normalize-job-listening.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAC,mBAAmB,EAAC,MAAM,4BAA4B,CAAC;AACpE,OAAO,KAAK,EAAC,yBAAyB,EAAC,MAAM,+BAA+B,CAAC;AAE7E,OAAO,KAAK,EAAC,4BAA4B,EAAC,MAAM,mCAAmC,CAAC;AAMpF,wBAAgB,qBAAqB,CAAC,MAAM,EAAE;IAC5C,GAAG,EAAE,mBAAmB,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,4BAA4B,EAAE,CAAC;IACvC,oBAAoB,EAAE,WAAW,CAAC,MAAM,CAAC,CAAC;CAC3C,GAAG,yBAAyB,GAAG,SAAS,CA8ExC"}
+{"version":3,"file":"normalize-job-listening.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/normalize-job-listening.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAC,mBAAmB,EAAC,MAAM,4BAA4B,CAAC;AACpE,OAAO,KAAK,EAAC,yBAAyB,EAAC,MAAM,+BAA+B,CAAC;AAE7E,OAAO,KAAK,EAAC,4BAA4B,EAAC,MAAM,mCAAmC,CAAC;AAMpF,wBAAgB,qBAAqB,CAAC,MAAM,EAAE;IAC5C,GAAG,EAAE,mBAAmB,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,4BAA4B,EAAE,CAAC;IACvC,oBAAoB,EAAE,WAAW,CAAC,MAAM,CAAC,CAAC;CAC3C,GAAG,yBAAyB,GAAG,SAAS,CAuGxC"}

package/dist/core/workflow-model/normalize-job-listening.js

@@ -44,7 +44,46 @@ export function normalizeJobListening(params) {
         ],
         issues: params.issues
     });
-    if (listening.until === undefined && listening.timeout === undefined && listening.max_executions === undefined) {
+    const on = listening.on.map((trigger, index)=>normalizeListeningTrigger({
+            trigger,
+            field: 'listener.on',
+            path: [
+                ...path,
+                'listening',
+                'on',
+                index,
+                'filter'
+            ],
+            issues: params.issues,
+            allowedJobReferences: params.allowedJobReferences
+        })).filter((matcher)=>matcher !== undefined);
+    if (on.length === 0) {
+        params.issues.push(issue({
+            code: 'listening-job-no-active-matcher',
+            message: `Listening job "${params.sourceName}" has no active "on" matcher; it can never execute.`,
+            path: [
+                ...path,
+                'listening',
+                'on'
+            ]
+        }));
+    }
+    const until = listening.until === undefined ? undefined : listening.until.map((trigger, index)=>normalizeListeningTrigger({
+            trigger,
+            field: 'listener.until',
+            path: [
+                ...path,
+                'listening',
+                'until',
+                index,
+                'filter'
+            ],
+            issues: params.issues,
+            allowedJobReferences: params.allowedJobReferences
+        })).filter((matcher)=>matcher !== undefined);
+    // Inert `until` matchers do not resolve the listening job; the check runs on
+    // the matchers that stay active.
+    if ((until === undefined || until.length === 0) && listening.timeout === undefined && listening.max_executions === undefined) {
         params.issues.push(issue({
             code: 'listening-job-missing-resolution-source',
             message: `Listening job "${params.sourceName}" must declare until, timeout, or max_executions.`,
@@ -66,33 +105,9 @@ export function normalizeJobListening(params) {
         }
     };
     return {
-        on: listening.on.map((trigger, index)=>normalizeListeningTrigger({
-                trigger,
-                field: 'listener.on',
-                path: [
-                    ...path,
-                    'listening',
-                    'on',
-                    index,
-                    'filter'
-                ],
-                issues: params.issues,
-                allowedJobReferences: params.allowedJobReferences
-            })),
-        ...listening.until === undefined ? {} : {
-            until: listening.until.map((trigger, index)=>normalizeListeningTrigger({
-                    trigger,
-                    field: 'listener.until',
-                    path: [
-                        ...path,
-                        'listening',
-                        'until',
-                        index,
-                        'filter'
-                    ],
-                    issues: params.issues,
-                    allowedJobReferences: params.allowedJobReferences
-                }))
+        on,
+        ...until === undefined || until.length === 0 ? {} : {
+            until
         },
         ...timeoutMs === undefined ? {} : {
             timeoutMs
@@ -107,6 +122,7 @@ export function normalizeJobListening(params) {
     };
 }
 function normalizeListeningTrigger(params) {
+    const matcherIssues = [];
     if (params.trigger.filter !== undefined) {
         validatePredicateExpression({
             field: params.field,
@@ -114,10 +130,15 @@ function normalizeListeningTrigger(params) {
             path: params.path,
             invalidCode: 'invalid-listener-filter',
             invalidMessage: `${params.field === 'listener.on' ? 'Listener on' : 'Listener until'} filter must be a valid CEL boolean expression.`,
-            issues: params.issues,
-            allowedJobReferences: params.allowedJobReferences
+            issues: matcherIssues,
+            allowedJobReferences: params.allowedJobReferences,
+            scope: 'trigger'
         });
     }
+    params.issues.push(...matcherIssues);
+    // A matcher with a trigger-scoped issue is inert: excluded from the matcher
+    // list while the authored document entry stays untouched.
+    if (matcherIssues.some((candidate)=>candidate.scope === 'trigger')) return undefined;
     return normalizeTriggerEntry(params.trigger);
 }
 

package/dist/core/workflow-model/normalize-job-listening.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../../src/core/workflow-model/normalize-job-listening.ts"],"sourcesContent":["import type {WorkflowDocumentJob} from '@shipfox/workflow-document';\nimport type {WorkflowModelJobListening} from '../entities/workflow-model.js';\nimport {DEFAULT_RUN_TIMEOUT_MS} from './constants.js';\nimport type {WorkflowModelValidationIssue} from './invalid-workflow-model-error.js';\nimport {normalizeTriggerEntry} from './normalize-triggers.js';\nimport {parseDurationMs} from './parse-duration-ms.js';\nimport {validatePredicateExpression} from './validate-predicate-expression.js';\nimport {issue} from './validation-issue.js';\n\nexport function normalizeJobListening(params: {\n  job: WorkflowDocumentJob;\n  sourceName: string;\n  issues: WorkflowModelValidationIssue[];\n  allowedJobReferences: ReadonlySet<string>;\n}): WorkflowModelJobListening | undefined {\n  const path = ['jobs', params.sourceName] as const;\n  const listening = params.job.listening;\n\n  if (listening === undefined) {\n    return undefined;\n  }\n\n  const timeoutMs = parseDurationMs({\n    source: listening.timeout,\n    path: [...path, 'listening', 'timeout'],\n    issues: params.issues,\n    maxMs: DEFAULT_RUN_TIMEOUT_MS,\n    outOfRangeCode: 'listening-timeout-exceeds-run-timeout',\n    outOfRangeMessage: 'Listening job timeout must be between 1s and the workflow run timeout.',\n  });\n  const debounceMs = parseDurationMs({\n    source: listening.batch?.debounce,\n    path: [...path, 'listening', 'batch', 'debounce'],\n    issues: params.issues,\n  });\n  const maxWaitMs = parseDurationMs({\n    source: listening.batch?.max_wait,\n    path: [...path, 'listening', 'batch', 'max_wait'],\n    issues: params.issues,\n  });\n\n  if (\n    listening.until === undefined &&\n    listening.timeout === undefined &&\n    listening.max_executions === undefined\n  ) {\n    params.issues.push(\n      issue({\n        code: 'listening-job-missing-resolution-source',\n        message: `Listening job \"${params.sourceName}\" must declare until, timeout, or max_executions.`,\n        path: [...path, 'listening'],\n      }),\n    );\n  }\n\n  const batch =\n    debounceMs === undefined && listening.batch?.max_size === undefined && maxWaitMs === undefined\n      ? undefined\n      : {\n          ...(debounceMs === undefined ? {} : {debounceMs}),\n          ...(listening.batch?.max_size === undefined ? {} : {maxSize: listening.batch.max_size}),\n          ...(maxWaitMs === undefined ? {} : {maxWaitMs}),\n        };\n\n  return {\n    on: listening.on.map((trigger, index) =>\n      normalizeListeningTrigger({\n        trigger,\n        field: 'listener.on',\n        path: [...path, 'listening', 'on', index, 'filter'],\n        issues: params.issues,\n        allowedJobReferences: params.allowedJobReferences,\n      }),\n    ),\n    ...(listening.until === undefined\n      ? {}\n      : {\n          until: listening.until.map((trigger, index) =>\n            normalizeListeningTrigger({\n              trigger,\n              field: 'listener.until',\n              path: [...path, 'listening', 'until', index, 'filter'],\n              issues: params.issues,\n              allowedJobReferences: params.allowedJobReferences,\n            }),\n          ),\n        }),\n    ...(timeoutMs === undefined ? {} : {timeoutMs}),\n    ...(listening.max_executions === undefined ? {} : {maxExecutions: listening.max_executions}),\n    ...(batch === undefined ? {} : {batch}),\n    onResolve: listening.on_resolve ?? 'finish',\n  };\n}\n\nfunction normalizeListeningTrigger(params: {\n  trigger: {\n    readonly source: string;\n    readonly event: string;\n    readonly with?: Readonly<Record<string, unknown>> | undefined;\n    readonly filter?: string | undefined;\n  };\n  field: 'listener.on' | 'listener.until';\n  path: readonly (string | number)[];\n  issues: WorkflowModelValidationIssue[];\n  allowedJobReferences: ReadonlySet<string>;\n}): WorkflowModelJobListening['on'][number] {\n  if (params.trigger.filter !== undefined) {\n    validatePredicateExpression({\n      field: params.field,\n      source: params.trigger.filter,\n      path: params.path,\n      invalidCode: 'invalid-listener-filter',\n      invalidMessage: `${params.field === 'listener.on' ? 'Listener on' : 'Listener until'} filter must be a valid CEL boolean expression.`,\n      issues: params.issues,\n      allowedJobReferences: params.allowedJobReferences,\n    });\n  }\n\n  return normalizeTriggerEntry(params.trigger);\n}\n"],"names":["DEFAULT_RUN_TIMEOUT_MS","normalizeTriggerEntry","parseDurationMs","validatePredicateExpression","issue","normalizeJobListening","params","path","sourceName","listening","job","undefined","timeoutMs","source","timeout","issues","maxMs","outOfRangeCode","outOfRangeMessage","debounceMs","batch","debounce","maxWaitMs","max_wait","until","max_executions","push","code","message","max_size","maxSize","on","map","trigger","index","normalizeListeningTrigger","field","allowedJobReferences","maxExecutions","onResolve","on_resolve","filter","invalidCode","invalidMessage"],"mappings":"AAEA,SAAQA,sBAAsB,QAAO,iBAAiB;AAEtD,SAAQC,qBAAqB,QAAO,0BAA0B;AAC9D,SAAQC,eAAe,QAAO,yBAAyB;AACvD,SAAQC,2BAA2B,QAAO,qCAAqC;AAC/E,SAAQC,KAAK,QAAO,wBAAwB;AAE5C,OAAO,SAASC,sBAAsBC,MAKrC;IACC,MAAMC,OAAO;QAAC;QAAQD,OAAOE,UAAU;KAAC;IACxC,MAAMC,YAAYH,OAAOI,GAAG,CAACD,SAAS;IAEtC,IAAIA,cAAcE,WAAW;QAC3B,OAAOA;IACT;IAEA,MAAMC,YAAYV,gBAAgB;QAChCW,QAAQJ,UAAUK,OAAO;QACzBP,MAAM;eAAIA;YAAM;YAAa;SAAU;QACvCQ,QAAQT,OAAOS,MAAM;QACrBC,OAAOhB;QACPiB,gBAAgB;QAChBC,mBAAmB;IACrB;IACA,MAAMC,aAAajB,gBAAgB;QACjCW,QAAQJ,UAAUW,KAAK,EAAEC;QACzBd,MAAM;eAAIA;YAAM;YAAa;YAAS;SAAW;QACjDQ,QAAQT,OAAOS,MAAM;IACvB;IACA,MAAMO,YAAYpB,gBAAgB;QAChCW,QAAQJ,UAAUW,KAAK,EAAEG;QACzBhB,MAAM;eAAIA;YAAM;YAAa;YAAS;SAAW;QACjDQ,QAAQT,OAAOS,MAAM;IACvB;IAEA,IACEN,UAAUe,KAAK,KAAKb,aACpBF,UAAUK,OAAO,KAAKH,aACtBF,UAAUgB,cAAc,KAAKd,WAC7B;QACAL,OAAOS,MAAM,CAACW,IAAI,CAChBtB,MAAM;YACJuB,MAAM;YACNC,SAAS,CAAC,eAAe,EAAEtB,OAAOE,UAAU,CAAC,iDAAiD,CAAC;YAC/FD,MAAM;mBAAIA;gBAAM;aAAY;QAC9B;IAEJ;IAEA,MAAMa,QACJD,eAAeR,aAAaF,UAAUW,KAAK,EAAES,aAAalB,aAAaW,cAAcX,YACjFA,YACA;QACE,GAAIQ,eAAeR,YAAY,CAAC,IAAI;YAACQ;QAAU,CAAC;QAChD,GAAIV,UAAUW,KAAK,EAAES,aAAalB,YAAY,CAAC,IAAI;YAACmB,SAASrB,UAAUW,KAAK,CAACS,QAAQ;QAAA,CAAC;QACtF,GAAIP,cAAcX,YAAY,CAAC,IAAI;YAACW;QAAS,CAAC;IAChD;IAEN,OAAO;QACLS,IAAItB,UAAUsB,EAAE,CAACC,GAAG,CAAC,CAACC,SAASC,QAC7BC,0BAA0B;gBACxBF;gBACAG,OAAO;gBACP7B,MAAM;uBAAIA;oBAAM;oBAAa;oBAAM2B;oBAAO;iBAAS;gBACnDnB,QAAQT,OAAOS,MAAM;gBACrBsB,sBAAsB/B,OAAO+B,oBAAoB;YACnD;QAEF,GAAI5B,UAAUe,KAAK,KAAKb,YACpB,CAAC,IACD;YACEa,OAAOf,UAAUe,KAAK,CAACQ,GAAG,CAAC,CAACC,SAASC,QACnCC,0BAA0B;oBACxBF;oBACAG,OAAO;oBACP7B,MAAM;2BAAIA;wBAAM;wBAAa;wBAAS2B;wBAAO;qBAAS;oBACtDnB,QAAQT,OAAOS,MAAM;oBACrBsB,sBAAsB/B,OAAO+B,oBAAoB;gBACnD;QAEJ,CAAC;QACL,GAAIzB,cAAcD,YAAY,CAAC,IAAI;YAACC;QAAS,CAAC;QAC9C,GAAIH,UAAUgB,cAAc,KAAKd,YAAY,CAAC,IAAI;YAAC2B,eAAe7B,UAAUgB,cAAc;QAAA,CAAC;QAC3F,GAAIL,UAAUT,YAAY,CAAC,IAAI;YAACS;QAAK,CAAC;QACtCmB,WAAW9B,UAAU+B,UAAU,IAAI;IACrC;AACF;AAEA,SAASL,0BAA0B7B,MAWlC;IACC,IAAIA,OAAO2B,OAAO,CAACQ,MAAM,KAAK9B,WAAW;QACvCR,4BAA4B;YAC1BiC,OAAO9B,OAAO8B,KAAK;YACnBvB,QAAQP,OAAO2B,OAAO,CAACQ,MAAM;YAC7BlC,MAAMD,OAAOC,IAAI;YACjBmC,aAAa;YACbC,gBAAgB,GAAGrC,OAAO8B,KAAK,KAAK,gBAAgB,gBAAgB,iBAAiB,+CAA+C,CAAC;YACrIrB,QAAQT,OAAOS,MAAM;YACrBsB,sBAAsB/B,OAAO+B,oBAAoB;QACnD;IACF;IAEA,OAAOpC,sBAAsBK,OAAO2B,OAAO;AAC7C"}
+{"version":3,"sources":["../../../src/core/workflow-model/normalize-job-listening.ts"],"sourcesContent":["import type {WorkflowDocumentJob} from '@shipfox/workflow-document';\nimport type {WorkflowModelJobListening} from '../entities/workflow-model.js';\nimport {DEFAULT_RUN_TIMEOUT_MS} from './constants.js';\nimport type {WorkflowModelValidationIssue} from './invalid-workflow-model-error.js';\nimport {normalizeTriggerEntry} from './normalize-triggers.js';\nimport {parseDurationMs} from './parse-duration-ms.js';\nimport {validatePredicateExpression} from './validate-predicate-expression.js';\nimport {issue} from './validation-issue.js';\n\nexport function normalizeJobListening(params: {\n  job: WorkflowDocumentJob;\n  sourceName: string;\n  issues: WorkflowModelValidationIssue[];\n  allowedJobReferences: ReadonlySet<string>;\n}): WorkflowModelJobListening | undefined {\n  const path = ['jobs', params.sourceName] as const;\n  const listening = params.job.listening;\n\n  if (listening === undefined) {\n    return undefined;\n  }\n\n  const timeoutMs = parseDurationMs({\n    source: listening.timeout,\n    path: [...path, 'listening', 'timeout'],\n    issues: params.issues,\n    maxMs: DEFAULT_RUN_TIMEOUT_MS,\n    outOfRangeCode: 'listening-timeout-exceeds-run-timeout',\n    outOfRangeMessage: 'Listening job timeout must be between 1s and the workflow run timeout.',\n  });\n  const debounceMs = parseDurationMs({\n    source: listening.batch?.debounce,\n    path: [...path, 'listening', 'batch', 'debounce'],\n    issues: params.issues,\n  });\n  const maxWaitMs = parseDurationMs({\n    source: listening.batch?.max_wait,\n    path: [...path, 'listening', 'batch', 'max_wait'],\n    issues: params.issues,\n  });\n\n  const on = listening.on\n    .map((trigger, index) =>\n      normalizeListeningTrigger({\n        trigger,\n        field: 'listener.on',\n        path: [...path, 'listening', 'on', index, 'filter'],\n        issues: params.issues,\n        allowedJobReferences: params.allowedJobReferences,\n      }),\n    )\n    .filter((matcher): matcher is WorkflowModelJobListening['on'][number] => matcher !== undefined);\n\n  if (on.length === 0) {\n    params.issues.push(\n      issue({\n        code: 'listening-job-no-active-matcher',\n        message: `Listening job \"${params.sourceName}\" has no active \"on\" matcher; it can never execute.`,\n        path: [...path, 'listening', 'on'],\n      }),\n    );\n  }\n\n  const until =\n    listening.until === undefined\n      ? undefined\n      : listening.until\n          .map((trigger, index) =>\n            normalizeListeningTrigger({\n              trigger,\n              field: 'listener.until',\n              path: [...path, 'listening', 'until', index, 'filter'],\n              issues: params.issues,\n              allowedJobReferences: params.allowedJobReferences,\n            }),\n          )\n          .filter(\n            (matcher): matcher is WorkflowModelJobListening['on'][number] => matcher !== undefined,\n          );\n\n  // Inert `until` matchers do not resolve the listening job; the check runs on\n  // the matchers that stay active.\n  if (\n    (until === undefined || until.length === 0) &&\n    listening.timeout === undefined &&\n    listening.max_executions === undefined\n  ) {\n    params.issues.push(\n      issue({\n        code: 'listening-job-missing-resolution-source',\n        message: `Listening job \"${params.sourceName}\" must declare until, timeout, or max_executions.`,\n        path: [...path, 'listening'],\n      }),\n    );\n  }\n\n  const batch =\n    debounceMs === undefined && listening.batch?.max_size === undefined && maxWaitMs === undefined\n      ? undefined\n      : {\n          ...(debounceMs === undefined ? {} : {debounceMs}),\n          ...(listening.batch?.max_size === undefined ? {} : {maxSize: listening.batch.max_size}),\n          ...(maxWaitMs === undefined ? {} : {maxWaitMs}),\n        };\n\n  return {\n    on,\n    ...(until === undefined || until.length === 0\n      ? {}\n      : {\n          until,\n        }),\n    ...(timeoutMs === undefined ? {} : {timeoutMs}),\n    ...(listening.max_executions === undefined ? {} : {maxExecutions: listening.max_executions}),\n    ...(batch === undefined ? {} : {batch}),\n    onResolve: listening.on_resolve ?? 'finish',\n  };\n}\n\nfunction normalizeListeningTrigger(params: {\n  trigger: {\n    readonly source: string;\n    readonly event?: string | undefined;\n    readonly with?: Readonly<Record<string, unknown>> | undefined;\n    readonly filter?: string | undefined;\n  };\n  field: 'listener.on' | 'listener.until';\n  path: readonly (string | number)[];\n  issues: WorkflowModelValidationIssue[];\n  allowedJobReferences: ReadonlySet<string>;\n}): WorkflowModelJobListening['on'][number] | undefined {\n  const matcherIssues: WorkflowModelValidationIssue[] = [];\n  if (params.trigger.filter !== undefined) {\n    validatePredicateExpression({\n      field: params.field,\n      source: params.trigger.filter,\n      path: params.path,\n      invalidCode: 'invalid-listener-filter',\n      invalidMessage: `${params.field === 'listener.on' ? 'Listener on' : 'Listener until'} filter must be a valid CEL boolean expression.`,\n      issues: matcherIssues,\n      allowedJobReferences: params.allowedJobReferences,\n      scope: 'trigger',\n    });\n  }\n  params.issues.push(...matcherIssues);\n\n  // A matcher with a trigger-scoped issue is inert: excluded from the matcher\n  // list while the authored document entry stays untouched.\n  if (matcherIssues.some((candidate) => candidate.scope === 'trigger')) return undefined;\n\n  return normalizeTriggerEntry(params.trigger);\n}\n"],"names":["DEFAULT_RUN_TIMEOUT_MS","normalizeTriggerEntry","parseDurationMs","validatePredicateExpression","issue","normalizeJobListening","params","path","sourceName","listening","job","undefined","timeoutMs","source","timeout","issues","maxMs","outOfRangeCode","outOfRangeMessage","debounceMs","batch","debounce","maxWaitMs","max_wait","on","map","trigger","index","normalizeListeningTrigger","field","allowedJobReferences","filter","matcher","length","push","code","message","until","max_executions","max_size","maxSize","maxExecutions","onResolve","on_resolve","matcherIssues","invalidCode","invalidMessage","scope","some","candidate"],"mappings":"AAEA,SAAQA,sBAAsB,QAAO,iBAAiB;AAEtD,SAAQC,qBAAqB,QAAO,0BAA0B;AAC9D,SAAQC,eAAe,QAAO,yBAAyB;AACvD,SAAQC,2BAA2B,QAAO,qCAAqC;AAC/E,SAAQC,KAAK,QAAO,wBAAwB;AAE5C,OAAO,SAASC,sBAAsBC,MAKrC;IACC,MAAMC,OAAO;QAAC;QAAQD,OAAOE,UAAU;KAAC;IACxC,MAAMC,YAAYH,OAAOI,GAAG,CAACD,SAAS;IAEtC,IAAIA,cAAcE,WAAW;QAC3B,OAAOA;IACT;IAEA,MAAMC,YAAYV,gBAAgB;QAChCW,QAAQJ,UAAUK,OAAO;QACzBP,MAAM;eAAIA;YAAM;YAAa;SAAU;QACvCQ,QAAQT,OAAOS,MAAM;QACrBC,OAAOhB;QACPiB,gBAAgB;QAChBC,mBAAmB;IACrB;IACA,MAAMC,aAAajB,gBAAgB;QACjCW,QAAQJ,UAAUW,KAAK,EAAEC;QACzBd,MAAM;eAAIA;YAAM;YAAa;YAAS;SAAW;QACjDQ,QAAQT,OAAOS,MAAM;IACvB;IACA,MAAMO,YAAYpB,gBAAgB;QAChCW,QAAQJ,UAAUW,KAAK,EAAEG;QACzBhB,MAAM;eAAIA;YAAM;YAAa;YAAS;SAAW;QACjDQ,QAAQT,OAAOS,MAAM;IACvB;IAEA,MAAMS,KAAKf,UAAUe,EAAE,CACpBC,GAAG,CAAC,CAACC,SAASC,QACbC,0BAA0B;YACxBF;YACAG,OAAO;YACPtB,MAAM;mBAAIA;gBAAM;gBAAa;gBAAMoB;gBAAO;aAAS;YACnDZ,QAAQT,OAAOS,MAAM;YACrBe,sBAAsBxB,OAAOwB,oBAAoB;QACnD,IAEDC,MAAM,CAAC,CAACC,UAAgEA,YAAYrB;IAEvF,IAAIa,GAAGS,MAAM,KAAK,GAAG;QACnB3B,OAAOS,MAAM,CAACmB,IAAI,CAChB9B,MAAM;YACJ+B,MAAM;YACNC,SAAS,CAAC,eAAe,EAAE9B,OAAOE,UAAU,CAAC,mDAAmD,CAAC;YACjGD,MAAM;mBAAIA;gBAAM;gBAAa;aAAK;QACpC;IAEJ;IAEA,MAAM8B,QACJ5B,UAAU4B,KAAK,KAAK1B,YAChBA,YACAF,UAAU4B,KAAK,CACZZ,GAAG,CAAC,CAACC,SAASC,QACbC,0BAA0B;YACxBF;YACAG,OAAO;YACPtB,MAAM;mBAAIA;gBAAM;gBAAa;gBAASoB;gBAAO;aAAS;YACtDZ,QAAQT,OAAOS,MAAM;YACrBe,sBAAsBxB,OAAOwB,oBAAoB;QACnD,IAEDC,MAAM,CACL,CAACC,UAAgEA,YAAYrB;IAGvF,6EAA6E;IAC7E,iCAAiC;IACjC,IACE,AAAC0B,CAAAA,UAAU1B,aAAa0B,MAAMJ,MAAM,KAAK,CAAA,KACzCxB,UAAUK,OAAO,KAAKH,aACtBF,UAAU6B,cAAc,KAAK3B,WAC7B;QACAL,OAAOS,MAAM,CAACmB,IAAI,CAChB9B,MAAM;YACJ+B,MAAM;YACNC,SAAS,CAAC,eAAe,EAAE9B,OAAOE,UAAU,CAAC,iDAAiD,CAAC;YAC/FD,MAAM;mBAAIA;gBAAM;aAAY;QAC9B;IAEJ;IAEA,MAAMa,QACJD,eAAeR,aAAaF,UAAUW,KAAK,EAAEmB,aAAa5B,aAAaW,cAAcX,YACjFA,YACA;QACE,GAAIQ,eAAeR,YAAY,CAAC,IAAI;YAACQ;QAAU,CAAC;QAChD,GAAIV,UAAUW,KAAK,EAAEmB,aAAa5B,YAAY,CAAC,IAAI;YAAC6B,SAAS/B,UAAUW,KAAK,CAACmB,QAAQ;QAAA,CAAC;QACtF,GAAIjB,cAAcX,YAAY,CAAC,IAAI;YAACW;QAAS,CAAC;IAChD;IAEN,OAAO;QACLE;QACA,GAAIa,UAAU1B,aAAa0B,MAAMJ,MAAM,KAAK,IACxC,CAAC,IACD;YACEI;QACF,CAAC;QACL,GAAIzB,cAAcD,YAAY,CAAC,IAAI;YAACC;QAAS,CAAC;QAC9C,GAAIH,UAAU6B,cAAc,KAAK3B,YAAY,CAAC,IAAI;YAAC8B,eAAehC,UAAU6B,cAAc;QAAA,CAAC;QAC3F,GAAIlB,UAAUT,YAAY,CAAC,IAAI;YAACS;QAAK,CAAC;QACtCsB,WAAWjC,UAAUkC,UAAU,IAAI;IACrC;AACF;AAEA,SAASf,0BAA0BtB,MAWlC;IACC,MAAMsC,gBAAgD,EAAE;IACxD,IAAItC,OAAOoB,OAAO,CAACK,MAAM,KAAKpB,WAAW;QACvCR,4BAA4B;YAC1B0B,OAAOvB,OAAOuB,KAAK;YACnBhB,QAAQP,OAAOoB,OAAO,CAACK,MAAM;YAC7BxB,MAAMD,OAAOC,IAAI;YACjBsC,aAAa;YACbC,gBAAgB,GAAGxC,OAAOuB,KAAK,KAAK,gBAAgB,gBAAgB,iBAAiB,+CAA+C,CAAC;YACrId,QAAQ6B;YACRd,sBAAsBxB,OAAOwB,oBAAoB;YACjDiB,OAAO;QACT;IACF;IACAzC,OAAOS,MAAM,CAACmB,IAAI,IAAIU;IAEtB,4EAA4E;IAC5E,0DAA0D;IAC1D,IAAIA,cAAcI,IAAI,CAAC,CAACC,YAAcA,UAAUF,KAAK,KAAK,YAAY,OAAOpC;IAE7E,OAAOV,sBAAsBK,OAAOoB,OAAO;AAC7C"}

package/dist/core/workflow-model/normalize-triggers.d.ts

@@ -4,7 +4,7 @@ import type { WorkflowModelValidationIssue } from './invalid-workflow-model-erro
 export declare function normalizeTriggers(document: WorkflowDocument, issues: WorkflowModelValidationIssue[]): readonly WorkflowModelTrigger[];
 export declare function normalizeTriggerEntry(trigger: {
     readonly source: string;
-    readonly event: string;
+    readonly event?: string | undefined;
     readonly with?: Readonly<Record<string, unknown>> | undefined;
     readonly filter?: string | undefined;
 }): WorkflowModelListeningTrigger;

package/dist/core/workflow-model/normalize-triggers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"normalize-triggers.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/normalize-triggers.ts"],"names":[],"mappings":"AAAA,OAAO,EAA6B,KAAK,gBAAgB,EAAC,MAAM,4BAA4B,CAAC;AAC7F,OAAO,KAAK,EACV,6BAA6B,EAC7B,oBAAoB,EACrB,MAAM,+BAA+B,CAAC;AAEvC,OAAO,KAAK,EAAC,4BAA4B,EAAC,MAAM,mCAAmC,CAAC;AASpF,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,gBAAgB,EAC1B,MAAM,EAAE,4BAA4B,EAAE,GACrC,SAAS,oBAAoB,EAAE,CAiEjC;AAiCD,wBAAgB,qBAAqB,CAAC,OAAO,EAAE;IAC7C,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,IAAI,CAAC,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GAAG,SAAS,CAAC;IAC9D,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CACtC,GAAG,6BAA6B,CAOhC"}
+{"version":3,"file":"normalize-triggers.d.ts","sourceRoot":"","sources":["../../../src/core/workflow-model/normalize-triggers.ts"],"names":[],"mappings":"AAAA,OAAO,EAA6B,KAAK,gBAAgB,EAAC,MAAM,4BAA4B,CAAC;AAC7F,OAAO,KAAK,EACV,6BAA6B,EAC7B,oBAAoB,EACrB,MAAM,+BAA+B,CAAC;AAEvC,OAAO,KAAK,EAAC,4BAA4B,EAAC,MAAM,mCAAmC,CAAC;AASpF,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,gBAAgB,EAC1B,MAAM,EAAE,4BAA4B,EAAE,GACrC,SAAS,oBAAoB,EAAE,CAiFjC;AAmCD,wBAAgB,qBAAqB,CAAC,OAAO,EAAE;IAC7C,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACpC,QAAQ,CAAC,IAAI,CAAC,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,GAAG,SAAS,CAAC;IAC9D,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CACtC,GAAG,6BAA6B,CAYhC"}

package/dist/core/workflow-model/normalize-triggers.js

@@ -9,18 +9,7 @@ export function normalizeTriggers(document, issues) {
     const triggers = document.triggers ?? {};
     const manualTriggerKeys = Object.entries(triggers).filter(([, trigger])=>trigger.source === manualTriggerSource).map(([sourceKey])=>sourceKey);
     const usedTriggerIds = new Map();
-    if (manualTriggerKeys.length > 1) {
-        issues.push(issue({
-            code: 'multiple-manual-triggers',
-            message: `A workflow may declare at most one manual trigger; found ${manualTriggerKeys.length}: ${manualTriggerKeys.join(', ')}.`,
-            path: [
-                'triggers'
-            ],
-            details: {
-                manualTriggerKeys
-            }
-        }));
-    }
+    let activeManualTriggerSeen = false;
     return Object.entries(triggers).flatMap(([sourceKey, trigger])=>{
         const id = stableId(sourceKey);
         const existingSourceKey = usedTriggerIds.get(id);
@@ -38,18 +27,41 @@ export function normalizeTriggers(document, issues) {
                         existingSourceKey,
                         sourceKey
                     ]
-                }
+                },
+                scope: 'trigger'
             }));
             return [];
         }
         usedTriggerIds.set(id, sourceKey);
+        if (trigger.source === manualTriggerSource) {
+            if (activeManualTriggerSeen) {
+                issues.push(issue({
+                    code: 'multiple-manual-triggers',
+                    message: `A workflow may declare at most one manual trigger; found ${manualTriggerKeys.length}: ${manualTriggerKeys.join(', ')}. This trigger is inert because it is not the first manual trigger in document order.`,
+                    path: [
+                        'triggers',
+                        sourceKey
+                    ],
+                    details: {
+                        manualTriggerKeys
+                    },
+                    scope: 'trigger'
+                }));
+                return [];
+            }
+        }
+        const triggerIssues = [];
         validateTriggerFilter({
             sourceKey,
             trigger,
-            issues
+            issues: triggerIssues
         });
+        issues.push(...triggerIssues);
         const normalizedTrigger = normalizeTriggerEntry(trigger);
+        const triggerIsInert = triggerIssues.some((candidate)=>candidate.scope === 'trigger');
         if (trigger.source !== cronTriggerSource) {
+            if (triggerIsInert) return [];
+            if (trigger.source === manualTriggerSource) activeManualTriggerSeen = true;
             return [
                 {
                     id,
@@ -66,12 +78,16 @@ export function normalizeTriggers(document, issues) {
             ...cronConfig,
             timezone: cronConfig.timezone ?? cronTriggerDefaultTimezone
         };
+        const cronIssues = [];
         validateCronTrigger({
             trigger,
             config: cronConfig,
             sourceKey,
-            issues
+            issues: cronIssues
         });
+        issues.push(...cronIssues);
+        const cronIsInert = cronIssues.some((candidate)=>candidate.scope === 'trigger');
+        if (triggerIsInert || cronIsInert) return [];
         return [
             {
                 id,
@@ -98,7 +114,8 @@ function validateTriggerFilter(params) {
             details: {
                 source: trigger.filter,
                 triggerSource: trigger.source
-            }
+            },
+            scope: 'trigger'
         }));
         return;
     }
@@ -108,13 +125,21 @@ function validateTriggerFilter(params) {
         path,
         invalidCode: 'invalid-trigger-filter',
         invalidMessage: 'Trigger filter must be a valid boolean predicate.',
-        issues
+        issues,
+        scope: 'trigger'
     });
 }
 export function normalizeTriggerEntry(trigger) {
+    // `manual` and `cron` have no delivered event to resolve against at fire
+    // time, so materialize their built-in name when the author omitted it.
+    // Integration sources keep the event absent, which becomes a source
+    // subscription (NULL row) on the write path.
+    const event = builtinEventForSource(trigger.source, trigger.event);
     return {
         source: trigger.source,
-        event: trigger.event,
+        ...event === undefined ? {} : {
+            event
+        },
         ...trigger.with === undefined ? {} : {
             inputs: trigger.with
         },
@@ -123,5 +148,10 @@ export function normalizeTriggerEntry(trigger) {
         }
     };
 }
+function builtinEventForSource(source, event) {
+    if (source === manualTriggerSource) return event ?? 'fire';
+    if (source === cronTriggerSource) return event ?? 'tick';
+    return event;
+}
 
 //# sourceMappingURL=normalize-triggers.js.map