@atlashub/smartstack-cli 3.34.0 → 3.36.0

package/dist/mcp-entry.mjs

@@ -26396,6 +26396,7 @@ var init_config = __esm({
         ],
         customTablePrefixes: [],
         scopeTypes: ["Core", "Extension", "Partner", "Community"],
+        // Incremental: {context}_v{version}_{sequence}_{Description} | Squash: {context}_v{version}
         migrationFormat: "{context}_v{version}_{sequence}_{Description}",
         namespaces: {
           // Empty = auto-detect from .csproj files
@@ -26847,32 +26848,39 @@ async function validateMigrationNaming(structure, _config, result) {
     return;
   }
   const migrationFiles = await findFiles("*.cs", { cwd: structure.migrations });
-  const migrationPattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d{3})_(.+)\.cs$/;
+  const incrementalPattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d{3})_(.+)\.cs$/;
+  const squashPattern = /^(\w+)_v(\d+\.\d+\.\d+)\.cs$/;
   const designerPattern = /\.Designer\.cs$/;
   for (const file of migrationFiles) {
     const fileName = path8.basename(file);
     if (designerPattern.test(fileName) || fileName.includes("ModelSnapshot")) {
       continue;
     }
-    if (!migrationPattern.test(fileName)) {
+    if (!incrementalPattern.test(fileName) && !squashPattern.test(fileName)) {
       result.errors.push({
         type: "error",
         category: "migrations",
         message: `Migration "${fileName}" does not follow naming convention`,
         file: path8.relative(structure.root, file),
-        suggestion: `Expected format: {context}_v{version}_{sequence}_{Description}.cs (e.g., core_v1.0.0_001_CreateAuthUsers.cs)`
+        suggestion: `Expected format: {context}_v{version}_{sequence}_{Description}.cs (incremental) or {context}_v{version}.cs (squash)`
       });
     }
   }
-  const orderedMigrations = migrationFiles.map((f) => path8.basename(f)).filter((f) => migrationPattern.test(f) && !f.includes("Designer")).sort();
+  const isValidMigration = (f) => (incrementalPattern.test(f) || squashPattern.test(f)) && !f.includes("Designer");
+  const getVersion = (f) => {
+    const inc = incrementalPattern.exec(f);
+    if (inc) return inc[2];
+    const sq = squashPattern.exec(f);
+    if (sq) return sq[2];
+    return "0.0.0";
+  };
+  const orderedMigrations = migrationFiles.map((f) => path8.basename(f)).filter(isValidMigration).sort();
   for (let i = 1; i < orderedMigrations.length; i++) {
     const prev = orderedMigrations[i - 1];
     const curr = orderedMigrations[i];
-    const prevMatch = migrationPattern.exec(prev);
-    const currMatch = migrationPattern.exec(curr);
-    if (prevMatch && currMatch) {
-      const prevVersion = prevMatch[2];
-      const currVersion = currMatch[2];
+    const prevVersion = getVersion(prev);
+    const currVersion = getVersion(curr);
+    if (prevVersion !== "0.0.0" && currVersion !== "0.0.0") {
       if (currVersion < prevVersion) {
         result.warnings.push({
           type: "warning",
@@ -27176,7 +27184,6 @@ async function validateControllerRoutes(structure, _config, result) {
       const navRouteMatch = content.match(/\[NavRoute\s*\(\s*"([^"]+)"(?:\s*,\s*Suffix\s*=\s*"([^"]+)")?\s*\)\]/);
       if (navRouteMatch) {
         const routePath = navRouteMatch[1];
-        const suffix = navRouteMatch[2];
         const parts = routePath.split(".");
         if (parts.length < 2) {
           result.warnings.push({
@@ -27197,26 +27204,14 @@ async function validateControllerRoutes(structure, _config, result) {
             suggestion: 'NavRoute paths must be lowercase (e.g., "platform.administration.users")'
           });
         }
-        const expectedRoute = `api/${routePath.replace(/\./g, "/")}${suffix ? `/${suffix}` : ""}`;
         const routeAttrMatch = content.match(/\[Route\s*\(\s*"([^"]+)"\s*\)\]/);
         if (routeAttrMatch) {
-          const actualRoute = routeAttrMatch[1];
-          if (actualRoute !== expectedRoute) {
-            result.errors.push({
-              type: "error",
-              category: "controllers",
-              message: `Controller "${fileName}" has [Route("${actualRoute}")] that doesn't match NavRoute "${routePath}". Expected [Route("${expectedRoute}")]`,
-              file: path8.relative(structure.root, file),
-              suggestion: `Change [Route] to [Route("${expectedRoute}")] to match the NavRoute convention`
-            });
-          }
-        } else {
-          result.warnings.push({
-            type: "warning",
+          result.errors.push({
+            type: "error",
             category: "controllers",
-            message: `Controller "${fileName}" has [NavRoute] but no explicit [Route] attribute`,
+            message: `Controller "${fileName}" has BOTH [Route("${routeAttrMatch[1]}")] and [NavRoute("${routePath}")]. Only [NavRoute] should be used.`,
             file: path8.relative(structure.root, file),
-            suggestion: `Add [Route("${expectedRoute}")] for deterministic routing`
+            suggestion: `Remove [Route("${routeAttrMatch[1]}")] \u2014 [NavRoute("${routePath}")] resolves the route dynamically from the database at startup`
           });
         }
       }
@@ -28276,26 +28271,34 @@ async function handleCheckMigrations(args, config2) {
 async function parseMigrations(migrationsPath, rootPath) {
   const files = await findFiles("*.cs", { cwd: migrationsPath });
   const migrations = [];
-  const pattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d{3})_(.+)\.cs$/;
+  const incrementalPattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d{3})_(.+)\.cs$/;
+  const squashPattern = /^(\w+)_v(\d+\.\d+\.\d+)\.cs$/;
   for (const file of files) {
     const fileName = path9.basename(file);
     if (fileName.includes(".Designer.") || fileName.includes("ModelSnapshot")) {
       continue;
     }
-    const match2 = pattern.exec(fileName);
-    if (match2) {
+    const incrementalMatch = incrementalPattern.exec(fileName);
+    const squashMatch = squashPattern.exec(fileName);
+    if (incrementalMatch) {
       migrations.push({
         name: fileName.replace(".cs", ""),
-        context: match2[1],
-        // DbContext (core, extensions, etc.)
-        version: match2[2],
-        // Semver version (1.0.0, 1.2.0, etc.)
-        sequence: match2[3],
-        // Sequence number (001, 002, etc.)
-        description: match2[4],
+        context: incrementalMatch[1],
+        version: incrementalMatch[2],
+        sequence: incrementalMatch[3],
+        description: incrementalMatch[4],
+        file: path9.relative(rootPath, file),
+        applied: true
+      });
+    } else if (squashMatch) {
+      migrations.push({
+        name: fileName.replace(".cs", ""),
+        context: squashMatch[1],
+        version: squashMatch[2],
+        sequence: "000",
+        description: "Squash",
         file: path9.relative(rootPath, file),
         applied: true
-        // We'd need DB connection to check this
       });
     } else {
       migrations.push({
@@ -28322,7 +28325,7 @@ function checkNamingConventions(result, _config) {
         type: "naming",
         description: `Migration "${migration.name}" does not follow naming convention`,
         files: [migration.file],
-        resolution: `Rename to format: {context}_v{version}_{sequence}_{Description} (e.g., core_v1.0.0_001_CreateAuthUsers)`
+        resolution: `Rename to format: {context}_v{version}_{sequence}_{Description} (incremental) or {context}_v{version} (squash)`
       });
     }
     if (migration.version === "0.0.0") {
@@ -28330,7 +28333,7 @@ function checkNamingConventions(result, _config) {
         type: "naming",
         description: `Migration "${migration.name}" missing version number`,
         files: [migration.file],
-        resolution: `Use format: {context}_v{version}_{sequence}_{Description} where version is semver (1.0.0, 1.2.0, etc.)`
+        resolution: `Use format: {context}_v{version}_{sequence}_{Description} (incremental) or {context}_v{version} (squash) where version is semver (1.0.0, 1.2.0, etc.)`
       });
     }
     if (migration.version !== "0.0.0" && !parseSemver(migration.version)) {
@@ -28438,7 +28441,7 @@ function generateSuggestions(result) {
   }
   if (result.conflicts.some((c) => c.type === "naming")) {
     result.suggestions.push(
-      "Use convention: {context}_v{version}_{sequence}_{Description} for migration naming (e.g., core_v1.0.0_001_CreateAuthUsers)"
+      "Use convention: {context}_v{version}_{sequence}_{Description} for incremental migrations (e.g., core_v1.0.0_001_CreateAuthUsers) or {context}_v{version} for squash migrations (e.g., core_v1.0.0)"
     );
   }
   if (result.conflicts.some((c) => c.type === "order")) {
@@ -52792,19 +52795,24 @@ async function handleSuggestMigration(args, config2) {
   } else {
     version2 = version2 || "1.0.0";
   }
-  const pascalDescription = toPascalCase(sanitizedDescription);
-  if (!pascalDescription || !/^[A-Z][a-zA-Z0-9]*$/.test(pascalDescription)) {
-    throw new Error(`Invalid migration description after PascalCase conversion: "${pascalDescription}"`);
+  let migrationName;
+  if (input.squash) {
+    migrationName = `${context}_v${version2}`;
+  } else {
+    const pascalDescription = toPascalCase(sanitizedDescription);
+    if (!pascalDescription || !/^[A-Z][a-zA-Z0-9]*$/.test(pascalDescription)) {
+      throw new Error(`Invalid migration description after PascalCase conversion: "${pascalDescription}"`);
+    }
+    const sequenceStr = sequence.toString().padStart(3, "0");
+    migrationName = `${context}_v${version2}_${sequenceStr}_${pascalDescription}`;
   }
-  const sequenceStr = sequence.toString().padStart(3, "0");
-  const migrationName = `${context}_v${version2}_${sequenceStr}_${pascalDescription}`;
   const dbContextName = context === "core" ? "CoreDbContext" : "ExtensionsDbContext";
   const outputPath = context === "extensions" ? "Persistence/Migrations/Extensions" : "Persistence/Migrations";
   const command = `dotnet ef migrations add ${migrationName} --context ${dbContextName} --project ../SmartStack.Infrastructure -o ${outputPath}`;
   const lines = [];
   lines.push("# Migration Name Suggestion");
   lines.push("");
-  lines.push("## Suggested Name");
+  lines.push(`## Suggested Name (${input.squash ? "Squash" : "Incremental"})`);
   lines.push("```");
   lines.push(migrationName);
   lines.push("```");
@@ -52822,8 +52830,13 @@ async function handleSuggestMigration(args, config2) {
   lines.push(`| DbContext | \`${dbContextName}\` | EF Core DbContext to use |`);
   lines.push(`| Schema | \`${context}\` | Database schema for tables |`);
   lines.push(`| Version | \`v${version2}\` | Semver version |`);
-  lines.push(`| Sequence | \`${sequenceStr}\` | Order in version |`);
-  lines.push(`| Description | \`${pascalDescription}\` | Migration description |`);
+  if (!input.squash) {
+    const sequenceStr = sequence.toString().padStart(3, "0");
+    lines.push(`| Sequence | \`${sequenceStr}\` | Order in version |`);
+    lines.push(`| Description | \`${toPascalCase(sanitizedDescription)}\` | Migration description |`);
+  } else {
+    lines.push(`| Mode | \`squash\` | Consolidated baseline (no sequence/description) |`);
+  }
   lines.push("");
   lines.push("> **Note**: Migrations are stored in separate history tables:");
   lines.push(`> - Core: \`core.__EFMigrationsHistory\``);
@@ -52845,15 +52858,17 @@ async function findExistingMigrations(structure, config2, context) {
   const migrationsPath = path12.join(infraPath, "Persistence", "Migrations");
   try {
     const migrationFiles = await findFiles("*.cs", { cwd: migrationsPath });
-    const migrationPattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d+)_(\w+)\.cs$/;
+    const incrementalPattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d+)_(\w+)\.cs$/;
+    const squashPattern = /^(\w+)_v(\d+\.\d+\.\d+)\.cs$/;
     for (const file of migrationFiles) {
       const fileName = path12.basename(file);
       if (fileName.includes(".Designer.") || fileName.includes("ModelSnapshot")) {
         continue;
       }
-      const match2 = fileName.match(migrationPattern);
-      if (match2) {
-        const [, ctx, ver, seq, desc] = match2;
+      const incrementalMatch = fileName.match(incrementalPattern);
+      const squashMatch = fileName.match(squashPattern);
+      if (incrementalMatch) {
+        const [, ctx, ver, seq, desc] = incrementalMatch;
         if (ctx === context || !context) {
           migrations.push({
             name: fileName.replace(".cs", ""),
@@ -52863,6 +52878,17 @@ async function findExistingMigrations(structure, config2, context) {
             description: desc
           });
         }
+      } else if (squashMatch) {
+        const [, ctx, ver] = squashMatch;
+        if (ctx === context || !context) {
+          migrations.push({
+            name: fileName.replace(".cs", ""),
+            context: ctx,
+            version: ver,
+            sequence: 0,
+            description: "Squash"
+          });
+        }
       }
     }
     migrations.sort((a, b) => {
@@ -52895,7 +52921,7 @@ var init_suggest_migration = __esm({
     init_logger();
     suggestMigrationTool = {
       name: "suggest_migration",
-      description: "Suggest a migration name following SmartStack conventions ({context}_v{version}_{sequence}_{Description})",
+      description: "Suggest a migration name following SmartStack conventions ({context}_v{version}_{sequence}_{Description} for incremental, {context}_v{version} for squash)",
       inputSchema: {
         type: "object",
         properties: {
@@ -52911,6 +52937,10 @@ var init_suggest_migration = __esm({
           version: {
             type: "string",
             description: 'Semver version (e.g., "1.0.0", "1.2.0"). If not provided, uses latest from existing migrations.'
+          },
+          squash: {
+            type: "boolean",
+            description: "If true, generates squash format: {context}_v{version} (no sequence/description). Used before merge to consolidate feature migrations."
           }
         },
         required: ["description"]
@@ -52919,7 +52949,8 @@ var init_suggest_migration = __esm({
     SuggestMigrationInputSchema2 = external_exports.object({
       description: external_exports.string().min(3, "Migration description must be at least 3 characters").max(100, "Migration description must be at most 100 characters").describe("Description of what the migration does"),
       context: external_exports.enum(["core", "extensions"]).optional().describe("DbContext name (default: auto-detected from project config)"),
-      version: external_exports.string().regex(/^\d+\.\d+\.\d+$/, 'Version must be semver format (e.g., "1.0.0")').optional().describe('Semver version (e.g., "1.0.0")')
+      version: external_exports.string().regex(/^\d+\.\d+\.\d+$/, 'Version must be semver format (e.g., "1.0.0")').optional().describe('Semver version (e.g., "1.0.0")'),
+      squash: external_exports.boolean().optional().describe("If true, generates squash format: {context}_v{version} (no sequence/description)")
     });
   }
 });
@@ -57631,7 +57662,6 @@ async function discoverNavRoutes(structure, scope, warnings) {
           permissions.push(match2[1]);
         }
         const fullNavRoute = suffix ? `${navRoute}.${suffix}` : navRoute;
-        const expectedRoute = `api/${navRouteToUrlPath(navRoute)}${suffix ? `/${toKebabCase(suffix)}` : ""}`;
         routes.push({
           navRoute: fullNavRoute,
           apiPath: `/api/${navRouteToUrlPath(navRoute)}${suffix ? `/${toKebabCase(suffix)}` : ""}`,
@@ -57642,12 +57672,9 @@ async function discoverNavRoutes(structure, scope, warnings) {
         });
         const routeAttrMatch = content.match(/\[Route\s*\(\s*"([^"]+)"\s*\)\]/);
         if (routeAttrMatch && warnings) {
-          const actualRoute = routeAttrMatch[1];
-          if (actualRoute !== expectedRoute && actualRoute !== "api/[controller]") {
-            warnings.push(
-              `WARNING: ${controllerName}Controller has [Route("${actualRoute}")] that doesn't match NavRoute "${navRoute}". Expected [Route("${expectedRoute}")]`
-            );
-          }
+          warnings.push(
+            `WARNING: ${controllerName}Controller has [Route("${routeAttrMatch[1]}")] alongside [NavRoute]. Remove [Route] \u2014 NavRoute resolves routes dynamically from the database.`
+          );
         }
       }
     } catch {
@@ -64918,8 +64945,9 @@ builder.HasOne(e => e.User)
 
 ### Naming Format
 
-Migrations MUST follow this naming pattern:
+Migrations follow two naming patterns depending on context:
 
+**Incremental (during development):**
 \`\`\`
 ${migrationFormat}
 \`\`\`
@@ -64931,30 +64959,50 @@ ${migrationFormat}
 | \`{sequence}\` | Order in version | \`001\`, \`002\` |
 | \`{Description}\` | Action (PascalCase) | \`CreateAuthUsers\` |
 
+**Squash (before merge, consolidated baseline):**
+\`\`\`
+{context}_v{version}
+\`\`\`
+
+The squash format has no sequence or description \u2014 it represents the complete state at a version.
+
 **Examples:**
-- \`core_v1.0.0_001_InitialSchema.cs\`
-- \`core_v1.0.0_002_CreateAuthUsers.cs\`
-- \`core_v1.2.0_001_AddUserProfiles.cs\`
-- \`extensions_v1.0.0_001_AddClientFeatures.cs\`
+- \`core_v1.0.0_001_InitialSchema.cs\` (incremental)
+- \`core_v1.0.0_002_CreateAuthUsers.cs\` (incremental)
+- \`core_v1.2.0_001_AddUserProfiles.cs\` (incremental)
+- \`extensions_v1.0.0_001_AddClientFeatures.cs\` (incremental)
+- \`core_v1.0.0.cs\` (squash \u2014 consolidated baseline)
 
 ### Creating Migrations
 
 \`\`\`bash
-# Create a new migration
+# Create an incremental migration
 dotnet ef migrations add core_v1.0.0_001_InitialSchema
 
 # With context specified
 dotnet ef migrations add core_v1.2.0_001_AddUserProfiles --context ApplicationDbContext
 \`\`\`
 
+### Squash Workflow
+
+Before creating a PR, squash all feature migrations into one:
+
+\`\`\`bash
+# Run /efcore squash \u2014 produces: core_v1.0.0 (no sequence/description)
+dotnet ef migrations add core_v1.0.0 --context CoreDbContext
+\`\`\`
+
+After a squash at \`core_v1.0.0\`, the next incremental migration starts at \`core_v1.0.0_001_*\`.
+
 ### Migration Rules
 
-1. **One migration per feature** - Group related changes in a single migration
+1. **One migration per feature** - Squash before merge to ensure a single migration per feature branch
 2. **Version-based naming** - Use semver (v1.0.0, v1.2.0) to link migrations to releases
-3. **Sequence numbers** - Use NNN (001, 002, etc.) for migrations in the same version
+3. **Sequence numbers** - Use NNN (001, 002, etc.) for incremental migrations in the same version
 4. **Context prefix** - Use \`core_\` for platform tables, \`extensions_\` for client extensions
-5. **Descriptive names** - Use clear PascalCase descriptions (CreateAuthUsers, AddUserProfiles, etc.)
+5. **Descriptive names** - Use clear PascalCase descriptions for incremental (CreateAuthUsers, AddUserProfiles, etc.)
 6. **Schema must be specified** - All tables must specify their schema in ToTable()
+7. **Squash before PR** - Feature branches must have exactly 1 migration (squashed) before creating a PR
 
 ---