@momentumcms/migrations 0.5.6 → 0.5.8

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@momentumcms/migrations",
-	"version": "0.5.6",
+	"version": "0.5.8",
 	"description": "Database migration system for Momentum CMS",
 	"license": "MIT",
 	"main": "./src/index.cjs",

package/src/cli/generate.cjs

@@ -1621,7 +1621,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -1664,6 +1666,25 @@ function parseMigrationArgs(args) {
     skipCloneTest: args.includes("--skip-clone-test")
   };
 }
+function mergePluginCollections(config) {
+  const collections = [...config.collections];
+  const plugins = config.plugins ?? [];
+  const slugs = new Set(collections.map((c) => c.slug));
+  for (const plugin of plugins) {
+    if (plugin.collections) {
+      for (const col of plugin.collections) {
+        if (!slugs.has(col.slug)) {
+          collections.push(col);
+          slugs.add(col.slug);
+        }
+      }
+    }
+    if (plugin.modifyCollections) {
+      plugin.modifyCollections(collections);
+    }
+  }
+  return collections;
+}
 
 // libs/migrations/src/cli/generate.ts
 async function main() {
@@ -1677,7 +1698,8 @@ async function main() {
     process.exit(1);
   }
   const directory = (0, import_node_path2.resolve)(migrationConfig.directory);
-  const desired = collectionsToSchema(config.collections, dialect);
+  const allCollections = mergePluginCollections(config);
+  const desired = collectionsToSchema(allCollections, dialect);
   let previous = readSnapshot(directory);
   if (!previous) {
     try {

package/src/cli/generate.js

@@ -1619,7 +1619,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -1662,6 +1664,25 @@ function parseMigrationArgs(args) {
     skipCloneTest: args.includes("--skip-clone-test")
   };
 }
+function mergePluginCollections(config) {
+  const collections = [...config.collections];
+  const plugins = config.plugins ?? [];
+  const slugs = new Set(collections.map((c) => c.slug));
+  for (const plugin of plugins) {
+    if (plugin.collections) {
+      for (const col of plugin.collections) {
+        if (!slugs.has(col.slug)) {
+          collections.push(col);
+          slugs.add(col.slug);
+        }
+      }
+    }
+    if (plugin.modifyCollections) {
+      plugin.modifyCollections(collections);
+    }
+  }
+  return collections;
+}
 
 // libs/migrations/src/cli/generate.ts
 async function main() {
@@ -1675,7 +1696,8 @@ async function main() {
     process.exit(1);
   }
   const directory = resolve(migrationConfig.directory);
-  const desired = collectionsToSchema(config.collections, dialect);
+  const allCollections = mergePluginCollections(config);
+  const desired = collectionsToSchema(allCollections, dialect);
   let previous = readSnapshot(directory);
   if (!previous) {
     try {

package/src/cli/rollback.cjs

@@ -540,7 +540,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -555,7 +557,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);

package/src/cli/rollback.js

@@ -538,7 +538,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -553,7 +555,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);

package/src/cli/run.cjs

@@ -950,7 +950,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -965,7 +967,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);
@@ -1014,7 +1018,9 @@ function buildContextFromAdapter(adapter, dialect) {
 }
 function buildCloneDbFromAdapter(adapter) {
   if (!adapter.cloneDatabase || !adapter.dropClone) {
-    throw new Error("DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply");
+    throw new Error(
+      "DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply"
+    );
   }
   return {
     cloneDatabase: adapter.cloneDatabase.bind(adapter),

package/src/cli/run.js

@@ -956,7 +956,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -971,7 +973,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);
@@ -1020,7 +1024,9 @@ function buildContextFromAdapter(adapter, dialect) {
 }
 function buildCloneDbFromAdapter(adapter) {
   if (!adapter.cloneDatabase || !adapter.dropClone) {
-    throw new Error("DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply");
+    throw new Error(
+      "DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply"
+    );
   }
   return {
     cloneDatabase: adapter.cloneDatabase.bind(adapter),

package/src/cli/shared.d.ts

@@ -1,4 +1,4 @@
-import type { ResolvedMomentumConfig, DatabaseAdapter } from '@momentumcms/core';
+import type { ResolvedMomentumConfig, DatabaseAdapter, CollectionConfig } from '@momentumcms/core';
 import type { DatabaseDialect } from '../lib/schema/column-type-map';
 import type { TrackerQueryFn } from '../lib/tracking/migration-tracker';
 import type { MigrationContext } from '../lib/migration.types';
@@ -55,3 +55,9 @@ export interface MigrationCliArgs {
  * Usage: npx tsx <command>.ts <configPath> [--name <name>] [--dry-run] [--test-only] [--skip-clone-test]
  */
 export declare function parseMigrationArgs(args: string[]): MigrationCliArgs;
+/**
+ * Merge plugin-declared collections into the config collections array.
+ * Reads each plugin's static `collections` property and calls `modifyCollections()`
+ * to pick up field injections — all without running async `onInit`.
+ */
+export declare function mergePluginCollections(config: ResolvedMomentumConfig): CollectionConfig[];

package/src/cli/status.cjs

@@ -301,7 +301,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -316,7 +318,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);

package/src/cli/status.js

@@ -299,7 +299,9 @@ async function loadMomentumConfig(configPath) {
   if (!raw.db?.adapter) {
     throw new Error(`Config at ${configPath} is missing db.adapter`);
   }
-  if (!raw.collections || raw.collections.length === 0) {
+  const hasExplicitCollections = raw.collections && raw.collections.length > 0;
+  const hasPluginCollections = Array.isArray(raw.plugins) && raw.plugins.some((p) => Array.isArray(p.collections) && p.collections.length > 0);
+  if (!hasExplicitCollections && !hasPluginCollections) {
     throw new Error(`Config at ${configPath} has no collections`);
   }
   return raw;
@@ -314,7 +316,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);

package/src/index.cjs

@@ -2421,7 +2421,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);
@@ -2481,7 +2483,9 @@ function buildPushDbFromAdapter(adapter) {
 }
 function buildCloneDbFromAdapter(adapter) {
   if (!adapter.cloneDatabase || !adapter.dropClone) {
-    throw new Error("DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply");
+    throw new Error(
+      "DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply"
+    );
   }
   return {
     cloneDatabase: adapter.cloneDatabase.bind(adapter),

package/src/index.js

@@ -2356,7 +2356,9 @@ function resolveDialect(adapter) {
 }
 function buildTrackerFromAdapter(adapter) {
   if (!adapter.queryRaw || !adapter.executeRaw) {
-    throw new Error("DatabaseAdapter must implement queryRaw and executeRaw for migration tracking");
+    throw new Error(
+      "DatabaseAdapter must implement queryRaw and executeRaw for migration tracking"
+    );
   }
   const queryRaw = adapter.queryRaw.bind(adapter);
   const executeRaw = adapter.executeRaw.bind(adapter);
@@ -2416,7 +2418,9 @@ function buildPushDbFromAdapter(adapter) {
 }
 function buildCloneDbFromAdapter(adapter) {
   if (!adapter.cloneDatabase || !adapter.dropClone) {
-    throw new Error("DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply");
+    throw new Error(
+      "DatabaseAdapter must implement cloneDatabase and dropClone for clone-test-apply"
+    );
   }
   return {
     cloneDatabase: adapter.cloneDatabase.bind(adapter),