@o-lang/olang 1.1.1 → 1.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@o-lang/olang",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "author": "Olalekan Ogundipe <info@workfily.com>",
   "description": "O-Lang: A governance language for user-directed, rule-enforced agent workflows",
   "main": "./src/index.js",

package/src/runtime.js

@@ -343,9 +343,8 @@ class RuntimeAPI {
       }
     };
 
+    // ✅ CORRECTED: Strict safety WITH proper resolver chaining
     const runResolvers = async (action) => {
-      const outputs = [];
-
       const mathPattern =
         /^(Add|Subtract|Multiply|Divide|Sum|Avg|Min|Max|Round|Floor|Ceil|Abs)\b/i;
 
@@ -371,13 +370,13 @@ class RuntimeAPI {
         resolversToRun = [agentResolver];
       }
 
-      // ✅ Return the FIRST resolver that returns a non-undefined result
+      // ✅ CORRECTED SAFETY: Try ALL resolvers before halting
       for (let idx = 0; idx < resolversToRun.length; idx++) {
         const resolver = resolversToRun[idx];
-        enforceResolverPolicy(resolver, step); // ✅ Use new policy enforcement
+        enforceResolverPolicy(resolver, step);
 
         try {
-          let result; // ✅ ADDITION 4 — External Resolver Execution Path
+          let result;
 
           if (this._isExternalResolver(resolver)) {
             result = await this._callExternalResolver(
@@ -389,22 +388,27 @@ class RuntimeAPI {
             result = await resolver(action, this.context);
           }
 
-          if (result !== undefined) {
+          // ✅ ACCEPT valid result immediately (non-null/non-undefined)
+          if (result !== undefined && result !== null) {
             this.context[`__resolver_${idx}`] = result;
             return result;
           }
+          // ❌ Resolver skipped this action (returned undefined/null) — continue to next resolver
+          // This is NORMAL resolver chaining behavior — NOT an error
 
-          outputs.push(result);
-          this.context[`__resolver_${idx}`] = result;
         } catch (e) {
-          this.addWarning(`Resolver ${resolver?.resolverName || resolver?.name || idx} failed for action "${action}": ${e.message}`);
-          outputs.push(null);
-          this.context[`__resolver_${idx}`] = null;
+          // ❌ Resolver threw an error — log warning but CONTINUE to next resolver
+          this.addWarning(`Resolver "${resolver?.resolverName || resolver?.name || idx}" threw error for action "${action}": ${e.message}`);
+          // Continue loop to try next resolver (don't halt yet)
         }
       }
 
-      // If no resolver handled the action, return undefined
-      return undefined;
+      // ✅ SAFETY GUARD: ALL resolvers skipped/failed — HALT workflow
+      throw new Error(
+        `[O-Lang SAFETY] No resolver successfully handled action: "${action}". ` +
+        `Attempted resolvers: ${resolversToRun.map(r => r.resolverName || r.name || 'anonymous').join(', ')}. ` +
+        `Workflow execution halted to prevent unsafe data propagation.`
+      );
     };
 
     switch (stepType) {