@remit/web-client 0.0.201 → 0.0.202

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/web-client",
-  "version": "0.0.201",
+  "version": "0.0.202",
   "type": "module",
   "description": "Remit web client, published as composable primitives — the app shell, auth shells, and runtime config. A distributor imports what it composes and bundles it.",
   "exports": {

package/src/lib/delete-folder.test.ts

@@ -285,3 +285,20 @@ describe("initialStage", () => {
 		assert.equal(initialStage(3), "choose-fate");
 	});
 });
+
+describe("hasChildFolders in a flat namespace", () => {
+	it("does not read a shared prefix as a child", () => {
+		const target = folder({
+			mailboxId: "work",
+			fullPath: "Work",
+			hierarchyDelimiter: "",
+		});
+		assert.equal(
+			hasChildFolders(target, [
+				target,
+				{ mailboxId: "shop", fullPath: "Workshop" },
+			]),
+			false,
+		);
+	});
+});

package/src/lib/delete-folder.ts

@@ -34,12 +34,14 @@ const isInboxPath = (fullPath: string): boolean =>
 /**
  * A folder has children when another mailbox's path is nested under it — the
  * folder's own path followed by the hierarchy delimiter. The delimiter matters:
- * `Work` is not a parent of `Workshop`, only of `Work/Shop`.
+ * `Work` is not a parent of `Workshop`, only of `Work/Shop`. A server that
+ * reports no delimiter has a flat namespace, where nothing nests at all.
  */
 export const hasChildFolders = (
 	folder: Pick<FolderNode, "mailboxId" | "fullPath" | "hierarchyDelimiter">,
 	all: readonly Pick<FolderNode, "mailboxId" | "fullPath">[],
 ): boolean => {
+	if (folder.hierarchyDelimiter.length === 0) return false;
 	const prefix = `${folder.fullPath}${folder.hierarchyDelimiter}`;
 	return all.some(
 		(other) =>

package/src/lib/new-folder.test.ts

@@ -107,3 +107,21 @@ describe("validateNewFolderName", () => {
 		);
 	});
 });
+
+describe("validateNewFolderName in a flat namespace", () => {
+	const flat = { delimiter: "", existingPaths: ["INBOX", "Archive"] };
+
+	it("accepts a name when the server reports no delimiter", () => {
+		assert.equal(
+			validateNewFolderName({ ...flat, name: "Receipts" }),
+			undefined,
+		);
+	});
+
+	it("still catches a collision", () => {
+		assert.match(
+			validateNewFolderName({ ...flat, name: "inbox" }) ?? "",
+			/already exists/,
+		);
+	});
+});

package/src/lib/new-folder.ts

@@ -19,7 +19,7 @@ export function composeFolderPath(name: string, parent?: FolderTarget): string {
  * match `inbox/x` against `INBOX/x` without folding case anywhere else.
  */
 const canonicalizePath = (path: string, delimiter: string): string => {
-	const parts = path.split(delimiter);
+	const parts = delimiter.length === 0 ? [path] : path.split(delimiter);
 	if (parts[0]?.toLowerCase() === "inbox") parts[0] = "INBOX";
 	return parts.join(delimiter);
 };
@@ -29,7 +29,8 @@ const canonicalizePath = (path: string, delimiter: string): string => {
  * name must be non-empty, must not contain the hierarchy delimiter (nesting is
  * expressed by the parent select, not by typing a path), and must not collide
  * with an existing folder — INBOX compared case-insensitively, everything else
- * exactly.
+ * exactly. A server that reports no delimiter has a flat namespace, so there is
+ * no separator to forbid and every name would otherwise contain it.
  */
 export function validateNewFolderName({
 	name,
@@ -44,7 +45,7 @@ export function validateNewFolderName({
 }): string | undefined {
 	const trimmed = name.trim();
 	if (trimmed === "") return "Enter a folder name.";
-	if (trimmed.includes(delimiter))
+	if (delimiter.length > 0 && trimmed.includes(delimiter))
 		return `A folder name can't contain "${delimiter}".`;
 	const candidate = canonicalizePath(
 		composeFolderPath(trimmed, parent),