domma-cms 0.34.0 → 0.34.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "domma-cms",
-  "version": "0.34.0",
+  "version": "0.34.1",
   "description": "File-based CMS powered by Domma and Fastify. Run npx domma-cms my-site to create a new project.",
   "type": "module",
   "main": "server/server.js",

package/server/services/adapters/MongoAdapter.js

@@ -198,6 +198,27 @@ export class MongoAdapter {
         await col.deleteMany({});
     }
 
+    /**
+     * Drop the backing MongoDB collection entirely (documents, indexes and
+     * the namespace). Used when the CMS collection itself is deleted —
+     * clear() only removes documents and would leave `cms_<slug>` behind.
+     *
+     * Deliberately bypasses _col(): that helper ensures the unique index,
+     * which would recreate the very collection being removed.
+     *
+     * @param {string} slug
+     * @returns {Promise<void>}
+     */
+    async drop(slug) {
+        try {
+            await this._db.collection(`${PREFIX}${slug}`).drop();
+        } catch (err) {
+            // Already absent — nothing to drop.
+            if (err.codeName !== 'NamespaceNotFound') throw err;
+        }
+        this._ensured.delete(slug);
+    }
+
     /**
      * Return all entries (used for export).
      *

package/server/services/collections.js

@@ -92,7 +92,10 @@ export async function listCollections() {
     return results
         .filter(r => r.status === 'fulfilled')
         .map(r => r.value)
-        .sort((a, b) => a.title.localeCompare(b.title));
+        // Hand-rolled schemas can lack `title` — fall back to slug rather
+        // than crashing the whole listing (seen in the wild: a schema with
+        // `name` instead of `title` 500'd /api/collections permanently).
+        .sort((a, b) => String(a.title || a.slug || '').localeCompare(String(b.title || b.slug || '')));
 }
 
 /**
@@ -210,11 +213,18 @@ export async function deleteCollection(slug) {
     const schema = await getCollection(slug);
     if (!schema) throw new Error(`Collection "${slug}" not found`);
 
-    // Clear adapter data first (handles MongoDB collections on non-file adapters).
+    // Remove adapter-side data first (handles MongoDB collections on
+    // non-file adapters). Prefer drop() — clear() only deletes documents
+    // and leaves the backing Mongo collection + its unique index behind
+    // forever.
     try {
         const adapter = await getAdapter(slug);
         if (adapter.constructor.name !== 'FileAdapter') {
-            await adapter.clear(slug);
+            if (typeof adapter.drop === 'function') {
+                await adapter.drop(slug);
+            } else {
+                await adapter.clear(slug);
+            }
         }
     } catch {
         // Ignore errors — directory removal handles file-backed data.