@zhin.js/config-yaml 1.0.2 → 1.0.4

package/lib/yaml-config-document.js

@@ -2,7 +2,7 @@ import { createHash, randomUUID } from 'node:crypto';
 import { readFile, rename, rm, stat, writeFile } from 'node:fs/promises';
 import { basename, dirname, resolve } from 'node:path';
 import { ConfigPatchPathError, } from '@zhin.js/runtime';
-import { parseDocument } from 'yaml';
+import { isMap, isSeq, parseDocument } from 'yaml';
 export class YamlConfigDocumentError extends Error {
     constructor(message, options) {
         super(message, options);
@@ -125,9 +125,51 @@ function applyPatch(document, patch) {
         return;
     }
     if (patch.op === 'set')
-        document.setIn(patch.path, structuredClone(patch.value));
+        document.setIn(resolvePath(document, patch.path), structuredClone(patch.value));
     else
-        document.deleteIn(patch.path);
+        document.deleteIn(resolvePath(document, patch.path));
+}
+/**
+ * Walks the AST so numeric segments address sequence elements (`endpoints.0.url`).
+ * Non-numeric segments on a sequence and out-of-bounds indexes are rejected
+ * with the same errors as the Runtime planner; missing intermediates are left
+ * for `setIn` to create as mappings (matching planner semantics).
+ */
+function resolvePath(document, path) {
+    const resolved = [];
+    let node = document.contents;
+    for (const [index, segment] of path.entries()) {
+        const isLast = index === path.length - 1;
+        if (isSeq(node)) {
+            if (!/^(0|[1-9]\d*)$/.test(segment)) {
+                throw new ConfigPatchPathError(`Config path ${pointer(path.slice(0, index + 1))} requires an array index, got "${segment}"`);
+            }
+            const arrayIndex = Number(segment);
+            if (arrayIndex >= node.items.length) {
+                throw new ConfigPatchPathError(`Config path ${pointer(path.slice(0, index + 1))} is out of bounds (array length ${node.items.length})`);
+            }
+            resolved.push(arrayIndex);
+            node = isLast ? undefined : node.get(arrayIndex, false);
+        }
+        else if (isMap(node)) {
+            resolved.push(segment);
+            node = isLast ? undefined : node.get(segment, false);
+        }
+        else if (node == null) {
+            // Missing intermediate mapping (setIn creates it).
+            resolved.push(segment);
+            node = undefined;
+        }
+        else {
+            throw new ConfigPatchPathError(`Config path ${pointer(path.slice(0, index + 1))} is not an object`);
+        }
+    }
+    return resolved;
+}
+function pointer(path) {
+    if (path.length === 0)
+        return '/';
+    return `/${path.map((segment) => segment.replaceAll('~', '~0').replaceAll('/', '~1')).join('/')}`;
 }
 function assertPath(path) {
     for (const segment of path) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhin.js/config-yaml",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Comment-preserving YAML ConfigDocument adapter for the Zhin Plugin Runtime",
   "type": "module",
   "main": "./lib/index.js",
@@ -17,12 +17,12 @@
   ],
   "dependencies": {
     "yaml": "2.9.0",
-    "@zhin.js/runtime": "1.0.2"
+    "@zhin.js/runtime": "1.0.4"
   },
   "devDependencies": {
     "@types/node": "^26.1.0",
     "typescript": "^6.0.3",
-    "@zhin.js/plugin-runtime": "1.1.0"
+    "@zhin.js/plugin-runtime": "1.1.1"
   },
   "engines": {
     "node": "^20.19.0 || >=22.12.0"