@atscript/db-mongo 0.1.88 → 0.1.89

package/dist/index.cjs

@@ -919,16 +919,20 @@ async function syncColumnsImpl(host, diff) {
 	if (diff.renamed.length > 0) {
 		const renameSpec = {};
 		for (const r of diff.renamed) {
+			if (pathCrossesArray(host, r.field.path)) {
+				host._log("syncColumns: skipping $rename for array-element field", r.oldName, "→", r.field.physicalName, "(Mongo $rename cannot traverse arrays)");
+				continue;
+			}
 			renameSpec[r.oldName] = r.field.physicalName;
 			renamed.push(r.field.physicalName);
 		}
-		update.$rename = renameSpec;
+		if (Object.keys(renameSpec).length > 0) update.$rename = renameSpec;
 	}
 	if (diff.added.length > 0) {
 		const setSpec = {};
 		for (const field of diff.added) {
 			const defaultVal = resolveSyncDefault(field);
-			if (defaultVal !== void 0) setSpec[field.physicalName] = defaultVal;
+			if (defaultVal !== void 0) setSpec[arraySafePath(host, field.physicalName, field.path)] = defaultVal;
 			added.push(field.physicalName);
 		}
 		if (Object.keys(setSpec).length > 0) update.$set = setSpec;
@@ -942,13 +946,52 @@ async function syncColumnsImpl(host, diff) {
 async function dropColumnsImpl(host, columns) {
 	if (columns.length === 0) return;
 	const unsetSpec = {};
-	for (const col of columns) unsetSpec[col] = "";
+	for (const col of columns) unsetSpec[arraySafePath(host, col, col)] = "";
 	await host.collection.updateMany({}, { $unset: unsetSpec }, host._getSessionOpts());
 }
+/**
+* Rewrites a dotted physical path to use Mongo's all-positional $[] operator
+* at every segment that's typed as an array in the table's flatMap. Returns
+* the input unchanged when no segment crosses an array boundary.
+*
+* `logicalPath` drives the array-boundary walk (flatMap is keyed by logical
+* path); the leaf of `physicalPath` is preserved so any `@db.column` rename
+* on the leaf still applies.
+*/
+function arraySafePath(host, physicalPath, logicalPath) {
+	const logicalSegments = logicalPath.split(".");
+	if (logicalSegments.length < 2) return physicalPath;
+	const physicalSegments = physicalPath.split(".");
+	const physicalLeaf = physicalSegments[physicalSegments.length - 1];
+	const out = [];
+	let prefix = "";
+	for (let i = 0; i < logicalSegments.length; i++) {
+		const isLeaf = i === logicalSegments.length - 1;
+		out.push(isLeaf ? physicalLeaf : logicalSegments[i]);
+		prefix = prefix ? `${prefix}.${logicalSegments[i]}` : logicalSegments[i];
+		if (!isLeaf) {
+			const type = host._table.flatMap.get(prefix);
+			if (type && (0, _atscript_db.resolveDesignType)(type) === "array") out.push("$[]");
+		}
+	}
+	return out.join(".");
+}
+/** Returns true if any non-leaf segment of the path is typed as an array. */
+function pathCrossesArray(host, logicalPath) {
+	const segments = logicalPath.split(".");
+	if (segments.length < 2) return false;
+	let prefix = "";
+	for (let i = 0; i < segments.length - 1; i++) {
+		prefix = prefix ? `${prefix}.${segments[i]}` : segments[i];
+		const type = host._table.flatMap.get(prefix);
+		if (type && (0, _atscript_db.resolveDesignType)(type) === "array") return true;
+	}
+	return false;
+}
 /** Resolves a field's default value for bulk $set during column sync. */
 function resolveSyncDefault(field) {
-	if (!field.defaultValue) return field.optional ? null : void 0;
-	if (field.defaultValue.kind === "value") return field.defaultValue.value;
+	if (!field.defaultValue) return;
+	if (field.defaultValue.kind === "value") return field.designType === "string" ? field.defaultValue.value : JSON.parse(field.defaultValue.value);
 }
 async function syncIndexesImpl(host) {
 	await host.ensureCollectionExists();

package/dist/index.mjs

@@ -1,5 +1,5 @@
 import { t as buildMongoFilter } from "./mongo-filter-BsocUQG3.mjs";
-import { AtscriptDbView, BaseDbAdapter, DbError, DbSpace, computeInsights, getDbFieldOp, getKeyProps } from "@atscript/db";
+import { AtscriptDbView, BaseDbAdapter, DbError, DbSpace, computeInsights, getDbFieldOp, getKeyProps, resolveDesignType } from "@atscript/db";
 import { MongoClient, MongoServerError, ObjectId } from "mongodb";
 //#region src/lib/collection-patcher.ts
 /**
@@ -918,16 +918,20 @@ async function syncColumnsImpl(host, diff) {
 	if (diff.renamed.length > 0) {
 		const renameSpec = {};
 		for (const r of diff.renamed) {
+			if (pathCrossesArray(host, r.field.path)) {
+				host._log("syncColumns: skipping $rename for array-element field", r.oldName, "→", r.field.physicalName, "(Mongo $rename cannot traverse arrays)");
+				continue;
+			}
 			renameSpec[r.oldName] = r.field.physicalName;
 			renamed.push(r.field.physicalName);
 		}
-		update.$rename = renameSpec;
+		if (Object.keys(renameSpec).length > 0) update.$rename = renameSpec;
 	}
 	if (diff.added.length > 0) {
 		const setSpec = {};
 		for (const field of diff.added) {
 			const defaultVal = resolveSyncDefault(field);
-			if (defaultVal !== void 0) setSpec[field.physicalName] = defaultVal;
+			if (defaultVal !== void 0) setSpec[arraySafePath(host, field.physicalName, field.path)] = defaultVal;
 			added.push(field.physicalName);
 		}
 		if (Object.keys(setSpec).length > 0) update.$set = setSpec;
@@ -941,13 +945,52 @@ async function syncColumnsImpl(host, diff) {
 async function dropColumnsImpl(host, columns) {
 	if (columns.length === 0) return;
 	const unsetSpec = {};
-	for (const col of columns) unsetSpec[col] = "";
+	for (const col of columns) unsetSpec[arraySafePath(host, col, col)] = "";
 	await host.collection.updateMany({}, { $unset: unsetSpec }, host._getSessionOpts());
 }
+/**
+* Rewrites a dotted physical path to use Mongo's all-positional $[] operator
+* at every segment that's typed as an array in the table's flatMap. Returns
+* the input unchanged when no segment crosses an array boundary.
+*
+* `logicalPath` drives the array-boundary walk (flatMap is keyed by logical
+* path); the leaf of `physicalPath` is preserved so any `@db.column` rename
+* on the leaf still applies.
+*/
+function arraySafePath(host, physicalPath, logicalPath) {
+	const logicalSegments = logicalPath.split(".");
+	if (logicalSegments.length < 2) return physicalPath;
+	const physicalSegments = physicalPath.split(".");
+	const physicalLeaf = physicalSegments[physicalSegments.length - 1];
+	const out = [];
+	let prefix = "";
+	for (let i = 0; i < logicalSegments.length; i++) {
+		const isLeaf = i === logicalSegments.length - 1;
+		out.push(isLeaf ? physicalLeaf : logicalSegments[i]);
+		prefix = prefix ? `${prefix}.${logicalSegments[i]}` : logicalSegments[i];
+		if (!isLeaf) {
+			const type = host._table.flatMap.get(prefix);
+			if (type && resolveDesignType(type) === "array") out.push("$[]");
+		}
+	}
+	return out.join(".");
+}
+/** Returns true if any non-leaf segment of the path is typed as an array. */
+function pathCrossesArray(host, logicalPath) {
+	const segments = logicalPath.split(".");
+	if (segments.length < 2) return false;
+	let prefix = "";
+	for (let i = 0; i < segments.length - 1; i++) {
+		prefix = prefix ? `${prefix}.${segments[i]}` : segments[i];
+		const type = host._table.flatMap.get(prefix);
+		if (type && resolveDesignType(type) === "array") return true;
+	}
+	return false;
+}
 /** Resolves a field's default value for bulk $set during column sync. */
 function resolveSyncDefault(field) {
-	if (!field.defaultValue) return field.optional ? null : void 0;
-	if (field.defaultValue.kind === "value") return field.defaultValue.value;
+	if (!field.defaultValue) return;
+	if (field.defaultValue.kind === "value") return field.designType === "string" ? field.defaultValue.value : JSON.parse(field.defaultValue.value);
 }
 async function syncIndexesImpl(host) {
 	await host.ensureCollectionExists();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atscript/db-mongo",
-  "version": "0.1.88",
+  "version": "0.1.89",
   "description": "Mongodb plugin for atscript.",
   "keywords": [
     "atscript",
@@ -46,17 +46,17 @@
     "access": "public"
   },
   "devDependencies": {
-    "@atscript/core": "^0.1.62",
-    "@atscript/typescript": "^0.1.62",
+    "@atscript/core": "^0.1.63",
+    "@atscript/typescript": "^0.1.63",
     "mongodb": "^6.17.0",
     "mongodb-memory-server-core": "^10.0.0",
-    "unplugin-atscript": "^0.1.62"
+    "unplugin-atscript": "^0.1.63"
   },
   "peerDependencies": {
-    "@atscript/core": "^0.1.62",
-    "@atscript/typescript": "^0.1.62",
+    "@atscript/core": "^0.1.63",
+    "@atscript/typescript": "^0.1.63",
     "mongodb": "^6.17.0",
-    "@atscript/db": "^0.1.88"
+    "@atscript/db": "^0.1.89"
   },
   "scripts": {
     "postinstall": "asc -f dts",