dreamteamer 0.6.2 → 0.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "dreamteamer",
-	"version": "0.6.2",
+	"version": "0.6.3",
 	"description": "A workspace compiler for coding agents — schema-validated records as plain files over git, compiled into every harness",
 	"license": "Apache-2.0",
 	"author": "Gilad Khen <giladkhen@gmail.com>",

package/src/collections-cli.js

@@ -295,15 +295,32 @@ function parseViewValue(raw) {
 	return raw;
 }
 
-/** Assign `a.b.c` into a nested object, creating plain objects on the way down. */
+/**
+ * Assign `a.b.c` into a nested object, creating plain objects on the way down.
+ *
+ * An empty or null value REMOVES the key rather than writing `''` — the same convention
+ * `store.set` has always used for top-level fields (`if (v === null || v === '') delete next[k]`),
+ * extended to the nested paths the meta verbs address. Without it there was no way to take a key
+ * back out of a `ui-view`'s `options`, so a superseded option lingered as `provider: ''` and the
+ * only cure was `rm` + `add`.
+ *
+ * Intermediate objects are not created on the way to a delete: unsetting `a.b` on a record with no
+ * `a` should leave the record alone, not grow an empty `a: {}`.
+ */
 function assignPath(target, dotted, value) {
 	const keys = dotted.split('.');
+	const leaf = keys[keys.length - 1];
+	const unset = value === null || value === '';
 	let node = target;
 	for (const k of keys.slice(0, -1)) {
-		if (node[k] == null || typeof node[k] !== 'object' || Array.isArray(node[k])) node[k] = {};
+		if (node[k] == null || typeof node[k] !== 'object' || Array.isArray(node[k])) {
+			if (unset) return;
+			node[k] = {};
+		}
 		node = node[k];
 	}
-	node[keys[keys.length - 1]] = value;
+	if (unset) delete node[leaf];
+	else node[leaf] = value;
 }
 
 const VIEW_META_FLAGS = new Set(['id', 'json', 'force']);