@weborigami/language 0.6.15 → 0.6.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@weborigami/language",
-  "version": "0.6.15",
+  "version": "0.6.17",
   "description": "Web Origami expression language compiler and runtime",
   "type": "module",
   "main": "./main.js",
@@ -11,7 +11,7 @@
     "typescript": "5.9.3"
   },
   "dependencies": {
-    "@weborigami/async-tree": "0.6.15",
+    "@weborigami/async-tree": "0.6.17",
     "exif-parser": "0.1.12",
     "watcher": "2.3.1",
     "yaml": "2.8.2"

package/src/handlers/oridocument_handler.js

@@ -40,7 +40,7 @@ export default {
 
 // Return a function that adds the given extension
 function addExtension(resultExtension) {
-  return (sourceValue, sourceKey) => {
+  const keyFn = (sourceValue, sourceKey) => {
     if (sourceKey === undefined) {
       return undefined;
     }
@@ -51,4 +51,6 @@ function addExtension(resultExtension) {
       : normalizedKey + resultExtension;
     return resultKey;
   };
+  keyFn.needsSourceValue = false;
+  return keyFn;
 }

package/src/runtime/ops.js

@@ -92,7 +92,7 @@ export async function cache(cache, path, code) {
   }
 
   // Don't await: might get another request for this before promise resolves
-  const promise = await execute(code);
+  const promise = execute(code);
 
   // Save promise so another request will get the same promise
   cache[path] = promise;
@@ -178,7 +178,10 @@ addOpLabel(exponentiation, "«ops.exponentiation»");
  * @param {...any} args
  */
 export async function flat(...args) {
-  return Tree.flat(args);
+  // Unpack packed arguments so they can be flattened
+  const unpacked = args.map((arg) => (isUnpackable(arg) ? arg.unpack() : arg));
+  // Flatten to depth 2: 1 for args array, 1 for flattening
+  return Tree.flat(unpacked, 2);
 }
 addOpLabel(flat, "«ops.flat»");
 
@@ -258,7 +261,8 @@ export function lambda(length, parameters, code, state = {}) {
       // interim stack frame.
       const interimStack = stack.slice();
       interimStack.push(args);
-      const frame = await expressionObject(parameters, { stack: interimStack });
+      const paramState = { ...state, stack: interimStack };
+      const frame = await expressionObject(parameters, paramState);
       // Record which code this stack frame is associated with
       Object.defineProperty(frame, codeSymbol, {
         value: code,

package/test/runtime/errors.test.js

@@ -249,19 +249,19 @@ evaluating: \x1B[31mfile.foo/\x1B[0m`,
       );
     });
 
-    test("identifies when a value didn't need to be unpacked", async () => {
+    test("identifies when data was already unpacked", async () => {
       const parent = {
         a: {
-          b: 1,
+          "b.json": 1,
         },
       };
       await assertError(
-        `a/b/`,
+        `a/b.json/`,
         `TraverseError: A path tried to unpack data that's already unpacked.
-Tried to traverse path:  a/b/
-Stopped unexpectedly at: b/
-You can drop the trailing slash and just use: b
-evaluating: \x1B[31mb/\x1B[0m`,
+Tried to traverse path:  a/b.json/
+Stopped unexpectedly at: b.json/
+You can drop the trailing slash and just use: b.json
+evaluating: \x1B[31mb.json/\x1B[0m`,
         { parent },
       );
     });

package/test/runtime/ops.test.js

@@ -158,7 +158,8 @@ describe("ops", () => {
         d: 4,
       });
       const array = [5, 6];
-      const result = await ops.flat(object, tree, array);
+      const set = new Set([7, 8]);
+      const result = await ops.flat(object, tree, array, set);
       assert.deepEqual(await Tree.plain(result), {
         a: 1,
         b: 2,
@@ -166,6 +167,8 @@ describe("ops", () => {
         d: 4,
         0: 5,
         1: 6,
+        2: 7,
+        3: 8,
       });
     });
   });