@exaudeus/workrail 3.17.0 → 3.18.1

package/README.md

@@ -150,6 +150,19 @@ Then prompt your agent:
 
 The agent will find the workflow, start at step 1, and proceed systematically.
 
+### Troubleshooting: "Permission denied" on startup
+
+Versions before 3.19.0 were published without the execute bit set on the binary.
+If you see `Permission denied` when WorkRail starts, reinstall or fix it in place:
+
+```sh
+# Option A: reinstall (recommended)
+npm install -g @exaudeus/workrail
+
+# Option B: fix in place without reinstalling
+chmod +x $(npm root -g)/@exaudeus/workrail/dist/mcp-server.js
+```
+
 ---
 
 ## CI & Releases

package/dist/application/services/validation-engine.js

@@ -685,24 +685,24 @@ let ValidationEngine = ValidationEngine_1 = class ValidationEngine {
                 issues.push(`${stepLabel}: assessmentConsequences must not be empty when declared`);
                 return;
             }
-            if (!step.assessmentRefs || step.assessmentRefs.length !== 1) {
-                issues.push(`${stepLabel}: assessmentConsequences require exactly one assessmentRef on the same step`);
-                suggestions.push(`Add exactly one assessmentRef to ${stepLabel} before declaring assessmentConsequences`);
+            if (!step.assessmentRefs || step.assessmentRefs.length === 0) {
+                issues.push(`${stepLabel}: assessmentConsequences require at least one assessmentRef on the same step`);
+                suggestions.push(`Add at least one assessmentRef to ${stepLabel} before declaring assessmentConsequences`);
                 return;
             }
             if (step.assessmentConsequences.length > 1) {
                 issues.push(`${stepLabel}: v1 assessment support allows exactly one assessment consequence per step`);
                 suggestions.push(`Reduce assessmentConsequences on ${stepLabel} to a single declaration`);
             }
-            const assessmentDefinition = assessments.find(assessment => assessment.id === step.assessmentRefs?.[0]);
-            if (!assessmentDefinition)
+            const referencedDefinitions = assessments.filter(assessment => step.assessmentRefs.includes(assessment.id));
+            if (referencedDefinitions.length === 0)
                 return;
             for (const consequence of step.assessmentConsequences) {
                 const trigger = consequence.when;
                 const effect = consequence.effect;
-                const allLevels = assessmentDefinition.dimensions.flatMap(d => d.levels);
+                const allLevels = referencedDefinitions.flatMap(def => def.dimensions.flatMap(d => d.levels));
                 if (!allLevels.includes(trigger.anyEqualsLevel)) {
-                    issues.push(`${stepLabel}: assessment consequence anyEqualsLevel '${trigger.anyEqualsLevel}' is not declared in any dimension of assessment '${assessmentDefinition.id}'`);
+                    issues.push(`${stepLabel}: assessment consequence anyEqualsLevel '${trigger.anyEqualsLevel}' is not declared in any dimension of any referenced assessment`);
                     suggestions.push(`Use a level declared in one of the dimensions: ${[...new Set(allLevels)].join(', ')}`);
                 }
                 if (effect.kind !== 'require_followup') {
@@ -776,10 +776,6 @@ let ValidationEngine = ValidationEngine_1 = class ValidationEngine {
                     }
                 }
                 validateAssessmentRefsForStep(typedStep, `Step '${step.id}'`);
-                if (typedStep.assessmentRefs !== undefined && typedStep.assessmentRefs.length > 1) {
-                    issues.push(`Step '${step.id}': v1 assessment support allows exactly one assessmentRef per step`);
-                    suggestions.push(`Reduce assessmentRefs on step '${step.id}' to a single assessment id`);
-                }
                 validateAssessmentConsequencesForStep(typedStep, `Step '${step.id}'`);
                 const callValidation = this.validateStepFunctionCalls(step, workflow.definition.functionDefinitions || []);
                 if (!callValidation.valid) {

package/dist/application/services/workflow-compiler.js

@@ -140,21 +140,19 @@ let WorkflowCompiler = class WorkflowCompiler {
             const assessmentConsequences = typedStep.assessmentConsequences;
             if (!assessmentConsequences)
                 continue;
-            if (!typedStep.assessmentRefs || typedStep.assessmentRefs.length !== 1) {
-                return (0, neverthrow_1.err)(error_1.Err.invalidState(`Step '${step.id}' declares assessmentConsequences but does not declare exactly one assessmentRef`));
-            }
-            const assessment = (workflow.definition.assessments ?? []).find(candidate => candidate.id === typedStep.assessmentRefs?.[0]);
-            if (!assessment) {
-                return (0, neverthrow_1.err)(error_1.Err.invalidState(`Step '${step.id}' declares assessmentConsequences for unknown assessmentRef '${typedStep.assessmentRefs[0]}'`));
+            if (!typedStep.assessmentRefs || typedStep.assessmentRefs.length === 0) {
+                return (0, neverthrow_1.err)(error_1.Err.invalidState(`Step '${step.id}' declares assessmentConsequences but declares no assessmentRefs`));
             }
             if (assessmentConsequences.length > 1) {
                 return (0, neverthrow_1.err)(error_1.Err.invalidState(`Step '${step.id}' declares ${assessmentConsequences.length} assessment consequences. V1 supports exactly one assessment consequence per step.`));
             }
+            const allLevelsAcrossRefs = (workflow.definition.assessments ?? [])
+                .filter(candidate => typedStep.assessmentRefs.includes(candidate.id))
+                .flatMap(assessment => assessment.dimensions.flatMap(d => d.levels));
             for (const consequence of assessmentConsequences) {
                 const trigger = consequence.when;
-                const allLevels = assessment.dimensions.flatMap(d => d.levels);
-                if (!allLevels.includes(trigger.anyEqualsLevel)) {
-                    return (0, neverthrow_1.err)(error_1.Err.invalidState(`Step '${step.id}' declares consequence with anyEqualsLevel '${trigger.anyEqualsLevel}' that is not declared in any dimension of assessment '${assessment.id}'`));
+                if (!allLevelsAcrossRefs.includes(trigger.anyEqualsLevel)) {
+                    return (0, neverthrow_1.err)(error_1.Err.invalidState(`Step '${step.id}' declares consequence with anyEqualsLevel '${trigger.anyEqualsLevel}' that is not declared in any dimension of any referenced assessment`));
                 }
                 if (consequence.effect.kind !== 'require_followup') {
                     return (0, neverthrow_1.err)(error_1.Err.invalidState(`Step '${step.id}' declares unsupported assessment consequence effect '${String(consequence.effect.kind)}'`));
@@ -187,8 +185,8 @@ let WorkflowCompiler = class WorkflowCompiler {
                     }
                 }
                 if (bodyStep.assessmentConsequences) {
-                    if (!bodyStep.assessmentRefs || bodyStep.assessmentRefs.length !== 1) {
-                        return (0, neverthrow_1.err)(error_1.Err.invalidState(`Loop body step '${bodyStep.id}' in loop '${loop.id}' declares assessmentConsequences but does not declare exactly one assessmentRef`));
+                    if (!bodyStep.assessmentRefs || bodyStep.assessmentRefs.length === 0) {
+                        return (0, neverthrow_1.err)(error_1.Err.invalidState(`Loop body step '${bodyStep.id}' in loop '${loop.id}' declares assessmentConsequences but declares no assessmentRefs`));
                     }
                 }
             }