@rebasepro/common 0.19.2-canary.g9b599d4 → 0.20.0

package/dist/index.es.js

@@ -10,10 +10,28 @@ var DEFAULT_ONE_OF_VALUE = "value";
 function isPropertyBuilder(property) {
 	return typeof property?.dynamicProps === "function";
 }
+/**
+* What a form opens with: a value for every property it can write.
+*
+* `excludeFromApi` columns are left out, and that is the whole of the rule —
+* they are not part of the API surface in either direction, so there is nothing
+* for a form to open showing and nothing it may send back. Including them was
+* not cosmetic: the baseline is what gets submitted, so a new record carried
+* `passwordHash: null` and `emailVerificationToken: null` into the create, and
+* the server refused the whole write with "these columns are the server's to
+* set" — the users collection could not be added to from the panel at all. The
+* fields were invisible on screen (`admin.disabled.hidden`), which is what made
+* the error read as being about the roles the operator *had* just edited.
+*
+* Server-side defaulting does not come through here: `applyDefaultValuesOnCreate`
+* asks each property for its own default, so an excluded column with a declared
+* `defaultValue` is still filled in on an in-process write.
+*/
 function getDefaultValuesFor(properties) {
 	if (!properties) return {};
 	return Object.entries(properties).map(([key, property]) => {
 		if (!property) return {};
+		if (property.excludeFromApi) return {};
 		const value = getDefaultValueFor(property);
 		return value === void 0 ? {} : { [key]: value };
 	}).reduce((a, b) => ({
@@ -124,18 +142,18 @@ function updateUserAutoValues({ inputValues, properties, status, uid }) {
 function applyDefaultValuesOnCreate(values, properties) {
 	if (!properties) return values ?? {};
 	const result = { ...values ?? {} };
-	const defaults = getDefaultValuesFor(properties);
 	for (const [key, property] of Object.entries(properties)) {
 		if (!property) continue;
 		if (!declaresDefault(property)) continue;
+		const defaultValue = getDefaultValueFor(property);
 		if (result[key] !== void 0) {
 			if (property.type === "map" && property.defaultValue === void 0 && isPlainObject(result[key])) result[key] = {
-				...defaults[key] ?? {},
+				...defaultValue ?? {},
 				...result[key]
 			};
 			continue;
 		}
-		if (defaults[key] !== void 0) result[key] = defaults[key];
+		if (defaultValue !== void 0) result[key] = defaultValue;
 	}
 	return result;
 }