@abraca/dabra 2.7.0 → 2.8.0

package/dist/abracadabra-provider.cjs

@@ -15715,6 +15715,19 @@ function makeEntryMap(fields) {
 	return m;
 }
 /**
+* A label is a "placeholder" — i.e. carries no real user title — when it is
+* empty/whitespace, null/undefined, or the literal `"Untitled"` sentinel
+* (case-insensitive, so `"untitled"` from a `labelToFilename` round-trip is
+* caught too). The whole tree-label corruption class boils down to a
+* placeholder being allowed to overwrite a real title; `patchEntry` refuses
+* exactly that. Mirrors cou-sh's `isEmptyTreeLabel`.
+*/
+function isPlaceholderLabel(label) {
+	if (typeof label !== "string") return label == null;
+	const t = label.trim();
+	return t === "" || t.toLowerCase() === "untitled";
+}
+/**
 * Patch an EXISTING entry's fields per-key on its nested `Y.Map`, so a
 * concurrent edit to a *different* field by a peer is preserved instead
 * of being clobbered by a whole-entry write — the whole-entry-LWW fix
@@ -15729,21 +15742,44 @@ function makeEntryMap(fields) {
 * Self-transacting: it batches its writes in one `Y.Doc` transaction
 * (a safe reentrant no-op join when already inside one), so callers
 * don't need to pass or own a transaction.
+*
+* ── NO-DESTROY LABEL INVARIANT ──────────────────────────────────────────
+* A `label` patch that is a placeholder (empty/whitespace/"Untitled") is
+* DROPPED when the entry already holds a real (non-placeholder) label —
+* regardless of which consumer (cou-sh title-sync, fs-sync rename
+* detection, MCP, table renderers, a stale snapshot) tried it. This is the
+* source-of-truth guard against the "card title silently becomes Untitled
+* / files renamed to untitled.md" corruption: a placeholder must never win
+* over a real title. Creating a brand-new entry with an empty label (e.g.
+* a fresh kanban card) is still allowed — the guard only fires when a real
+* label already exists. Pass `{ allowLabelClear: true }` to override (the
+* single legitimate "user explicitly cleared it" path).
 */
-function patchEntry(treeMap, id, patch, removeKeys = []) {
+function patchEntry(treeMap, id, patch, removeKeys = [], opts = {}) {
 	const apply = () => {
 		const raw = treeMap.get(id);
+		let effectivePatch = patch;
+		if (!opts.allowLabelClear && Object.hasOwn(patch, "label") && isPlaceholderLabel(patch.label)) {
+			let existingLabel;
+			if (raw != null && typeof raw.get === "function") existingLabel = raw.get("label");
+			else if (raw != null && typeof raw.toJSON === "function") existingLabel = raw.toJSON()?.label;
+			else if (raw != null && typeof raw === "object") existingLabel = raw.label;
+			if (!isPlaceholderLabel(existingLabel)) {
+				const { label: _dropped, ...rest } = patch;
+				effectivePatch = rest;
+			}
+		}
 		if (raw instanceof yjs.Map) {
-			for (const [k, v] of Object.entries(patch)) if (v === void 0) raw.delete(k);
+			for (const [k, v] of Object.entries(effectivePatch)) if (v === void 0) raw.delete(k);
 			else raw.set(k, v);
 			for (const k of removeKeys) raw.delete(k);
 			return;
 		}
 		const merged = {
 			...raw == null ? {} : toPlain(raw),
-			...patch
+			...effectivePatch
 		};
-		for (const [k, v] of Object.entries(patch)) if (v === void 0) delete merged[k];
+		for (const [k, v] of Object.entries(effectivePatch)) if (v === void 0) delete merged[k];
 		for (const k of removeKeys) delete merged[k];
 		treeMap.set(id, makeEntryMap(merged));
 	};
@@ -20900,6 +20936,7 @@ exports.filenameToLabel = filenameToLabel;
 exports.foldRecords = foldRecords;
 exports.generateMnemonic = generateMnemonic;
 exports.isEncryptedContent = isEncryptedContent;
+exports.isPlaceholderLabel = isPlaceholderLabel;
 exports.makeEncryptedYMap = makeEncryptedYMap;
 exports.makeEncryptedYText = makeEncryptedYText;
 exports.makeEntryMap = makeEntryMap;