@atlashub/smartstack-mcp 1.2.0 → 1.2.2

package/dist/index.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-#!/usr/bin/env node
 
 // src/server.ts
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
@@ -149,7 +148,7 @@ var defaultConfig = {
       "loc_",
       "lic_"
     ],
-    migrationFormat: "YYYYMMDD_NNN_{Description}",
+    migrationFormat: "{context}_v{version}_{sequence}_{Description}",
     namespaces: {
       domain: "SmartStack.Domain",
       application: "SmartStack.Application",
@@ -281,7 +280,7 @@ var ConventionsConfigSchema = z.object({
     "loc_",
     "lic_"
   ]),
-  migrationFormat: z.string().default("YYYYMMDD_NNN_{Description}"),
+  migrationFormat: z.string().default("{context}_v{version}_{sequence}_{Description}"),
   namespaces: z.object({
     domain: z.string(),
     application: z.string(),
@@ -533,7 +532,7 @@ async function findControllerFiles(apiPath) {
 import path5 from "path";
 var validateConventionsTool = {
   name: "validate_conventions",
-  description: "Validate AtlasHub/SmartStack conventions: SQL schemas (core/extensions), domain table prefixes (auth_, nav_, ai_, etc.), migration naming (YYYYMMDD_NNN_*), service interfaces (I*Service), namespace structure",
+  description: "Validate AtlasHub/SmartStack conventions: SQL schemas (core/extensions), domain table prefixes (auth_, nav_, ai_, etc.), migration naming ({context}_v{version}_{sequence}_*), service interfaces (I*Service), namespace structure",
   inputSchema: {
     type: "object",
     properties: {
@@ -675,7 +674,7 @@ async function validateMigrationNaming(structure, _config, result) {
     return;
   }
   const migrationFiles = await findFiles("*.cs", { cwd: structure.migrations });
-  const migrationPattern = /^(\d{8})_(\d{3})_(.+)\.cs$/;
+  const migrationPattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d{3})_(.+)\.cs$/;
   const designerPattern = /\.Designer\.cs$/;
   for (const file of migrationFiles) {
     const fileName = path5.basename(file);
@@ -688,7 +687,7 @@ async function validateMigrationNaming(structure, _config, result) {
         category: "migrations",
         message: `Migration "${fileName}" does not follow naming convention`,
         file: path5.relative(structure.root, file),
-        suggestion: `Expected format: YYYYMMDD_NNN_Description.cs (e.g., 20260115_001_InitialCreate.cs)`
+        suggestion: `Expected format: {context}_v{version}_{sequence}_{Description}.cs (e.g., core_v1.0.0_001_CreateAuthUsers.cs)`
       });
     }
   }
@@ -696,14 +695,18 @@ async function validateMigrationNaming(structure, _config, result) {
   for (let i = 1; i < orderedMigrations.length; i++) {
     const prev = orderedMigrations[i - 1];
     const curr = orderedMigrations[i];
-    const prevDate = prev.substring(0, 8);
-    const currDate = curr.substring(0, 8);
-    if (currDate < prevDate) {
-      result.warnings.push({
-        type: "warning",
-        category: "migrations",
-        message: `Migration order issue: "${curr}" dated before "${prev}"`
-      });
+    const prevMatch = migrationPattern.exec(prev);
+    const currMatch = migrationPattern.exec(curr);
+    if (prevMatch && currMatch) {
+      const prevVersion = prevMatch[2];
+      const currVersion = currMatch[2];
+      if (currVersion < prevVersion) {
+        result.warnings.push({
+          type: "warning",
+          category: "migrations",
+          message: `Migration order issue: "${curr}" (v${currVersion}) comes before "${prev}" (v${prevVersion})`
+        });
+      }
     }
   }
 }
@@ -871,7 +874,7 @@ async function handleCheckMigrations(args, config) {
 async function parseMigrations(migrationsPath, rootPath) {
   const files = await findFiles("*.cs", { cwd: migrationsPath });
   const migrations = [];
-  const pattern = /^(\d{8})_(\d{3})_(.+)\.cs$/;
+  const pattern = /^(\w+)_v(\d+\.\d+\.\d+)_(\d{3})_(.+)\.cs$/;
   for (const file of files) {
     const fileName = path6.basename(file);
     if (fileName.includes(".Designer.") || fileName.includes("ModelSnapshot")) {
@@ -881,10 +884,13 @@ async function parseMigrations(migrationsPath, rootPath) {
     if (match) {
       migrations.push({
         name: fileName.replace(".cs", ""),
-        timestamp: match[1],
-        prefix: match[2],
-        // Now this is the sequence number (NNN)
-        description: match[3],
+        context: match[1],
+        // DbContext (core, extensions, etc.)
+        version: match[2],
+        // Semver version (1.0.0, 1.2.0, etc.)
+        sequence: match[3],
+        // Sequence number (001, 002, etc.)
+        description: match[4],
         file: path6.relative(rootPath, file),
         applied: true
         // We'd need DB connection to check this
@@ -892,8 +898,9 @@ async function parseMigrations(migrationsPath, rootPath) {
     } else {
       migrations.push({
         name: fileName.replace(".cs", ""),
-        timestamp: "",
-        prefix: "Unknown",
+        context: "Unknown",
+        version: "0.0.0",
+        sequence: "000",
         description: fileName.replace(".cs", ""),
         file: path6.relative(rootPath, file),
         applied: true
@@ -901,51 +908,63 @@ async function parseMigrations(migrationsPath, rootPath) {
     }
   }
   return migrations.sort((a, b) => {
-    const dateCompare = a.timestamp.localeCompare(b.timestamp);
-    if (dateCompare !== 0) return dateCompare;
-    return a.prefix.localeCompare(b.prefix);
+    const versionCompare = compareVersions(a.version, b.version);
+    if (versionCompare !== 0) return versionCompare;
+    return a.sequence.localeCompare(b.sequence);
   });
 }
+function compareVersions(a, b) {
+  const partsA = a.split(".").map(Number);
+  const partsB = b.split(".").map(Number);
+  for (let i = 0; i < 3; i++) {
+    if (partsA[i] > partsB[i]) return 1;
+    if (partsA[i] < partsB[i]) return -1;
+  }
+  return 0;
+}
 function checkNamingConventions(result, _config) {
   for (const migration of result.migrations) {
-    if (!migration.timestamp) {
+    if (migration.context === "Unknown") {
       result.conflicts.push({
         type: "naming",
         description: `Migration "${migration.name}" does not follow naming convention`,
         files: [migration.file],
-        resolution: `Rename to format: YYYYMMDD_NNN_Description (e.g., 20260115_001_InitialCreate)`
+        resolution: `Rename to format: {context}_v{version}_{sequence}_{Description} (e.g., core_v1.0.0_001_CreateAuthUsers)`
       });
     }
-    if (migration.prefix === "Unknown") {
+    if (migration.version === "0.0.0") {
       result.conflicts.push({
         type: "naming",
-        description: `Migration "${migration.name}" missing sequence number`,
+        description: `Migration "${migration.name}" missing version number`,
         files: [migration.file],
-        resolution: `Use format: YYYYMMDD_NNN_Description where NNN is a 3-digit sequence (001, 002, etc.)`
+        resolution: `Use format: {context}_v{version}_{sequence}_{Description} where version is semver (1.0.0, 1.2.0, etc.)`
       });
     }
   }
 }
 function checkChronologicalOrder(result) {
-  const migrations = result.migrations.filter((m) => m.timestamp);
+  const migrations = result.migrations.filter((m) => m.context !== "Unknown");
   for (let i = 1; i < migrations.length; i++) {
     const prev = migrations[i - 1];
     const curr = migrations[i];
-    if (curr.timestamp < prev.timestamp) {
-      result.conflicts.push({
-        type: "order",
-        description: `Migration "${curr.name}" (${curr.timestamp}) is dated before "${prev.name}" (${prev.timestamp})`,
-        files: [curr.file, prev.file],
-        resolution: "Reorder migrations or update timestamps"
-      });
-    }
-    if (curr.timestamp === prev.timestamp && curr.prefix === prev.prefix) {
-      result.conflicts.push({
-        type: "order",
-        description: `Migrations "${curr.name}" and "${prev.name}" have same timestamp`,
-        files: [curr.file, prev.file],
-        resolution: "Use different sequence numbers (NNN) or different dates"
-      });
+    if (curr.context === prev.context) {
+      const versionCompare = compareVersions(curr.version, prev.version);
+      if (versionCompare < 0) {
+        result.conflicts.push({
+          type: "order",
+          description: `Migration "${curr.name}" (v${curr.version}) is versioned before "${prev.name}" (v${prev.version})`,
+          files: [curr.file, prev.file],
+          resolution: "Reorder migrations or update version numbers"
+        });
+      }
+      if (curr.version === prev.version && curr.sequence === prev.sequence) {
+        result.conflicts.push({
+          type: "order",
+          description: `Migrations "${curr.name}" and "${prev.name}" have same version and sequence`,
+          files: [curr.file, prev.file],
+          resolution: "Use different sequence numbers (001, 002, etc.) for migrations in the same version"
+        });
+      }
     }
   }
 }
@@ -1000,7 +1019,7 @@ async function checkModelSnapshot(result, structure) {
   }
   const snapshotContent = await readText(snapshotFiles[0]);
   for (const migration of result.migrations) {
-    if (migration.timestamp && !snapshotContent.includes(migration.name)) {
+    if (migration.context !== "Unknown" && !snapshotContent.includes(migration.name)) {
       result.conflicts.push({
         type: "dependency",
         description: `Migration "${migration.name}" not referenced in ModelSnapshot`,
@@ -1018,12 +1037,12 @@ function generateSuggestions(result) {
   }
   if (result.conflicts.some((c) => c.type === "naming")) {
     result.suggestions.push(
-      "Use convention: YYYYMMDD_NNN_Description for migration naming (e.g., 20260115_001_InitialCreate)"
+      "Use convention: {context}_v{version}_{sequence}_{Description} for migration naming (e.g., core_v1.0.0_001_CreateAuthUsers)"
     );
   }
   if (result.conflicts.some((c) => c.type === "order")) {
     result.suggestions.push(
-      "Ensure migrations are created in chronological order to avoid conflicts"
+      "Ensure migrations are created in version order to avoid conflicts"
     );
   }
   if (result.migrations.length > 20) {
@@ -1047,11 +1066,11 @@ function formatResult2(result, currentBranch, compareBranch) {
   lines.push("");
   lines.push("## Migrations");
   lines.push("");
-  lines.push("| Name | Timestamp | Prefix | Description |");
-  lines.push("|------|-----------|--------|-------------|");
+  lines.push("| Name | Context | Version | Sequence | Description |");
+  lines.push("|------|---------|---------|----------|-------------|");
   for (const migration of result.migrations) {
     lines.push(
-      `| ${migration.name} | ${migration.timestamp || "N/A"} | ${migration.prefix} | ${migration.description} |`
+      `| ${migration.name} | ${migration.context} | ${migration.version} | ${migration.sequence} | ${migration.description} |`
     );
   }
   lines.push("");
@@ -1978,28 +1997,37 @@ Migrations MUST follow this naming pattern:
 ${migrationFormat}
 \`\`\`
 
+| Part | Description | Example |
+|------|-------------|---------|
+| \`{context}\` | DbContext name | \`core\`, \`extensions\` |
+| \`{version}\` | Semver version | \`v1.0.0\`, \`v1.2.0\` |
+| \`{sequence}\` | Order in version | \`001\`, \`002\` |
+| \`{Description}\` | Action (PascalCase) | \`CreateAuthUsers\` |
+
 **Examples:**
-- \`20260115_001_InitialSchema.cs\`
-- \`20260120_002_AddUserProfiles.cs\`
-- \`20260125_003_AddSupportTickets.cs\`
+- \`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\`
 
 ### Creating Migrations
 
 \`\`\`bash
 # Create a new migration
-dotnet ef migrations add 20260115_001_InitialSchema
+dotnet ef migrations add core_v1.0.0_001_InitialSchema
 
 # With context specified
-dotnet ef migrations add 20260115_001_InitialSchema --context ApplicationDbContext
+dotnet ef migrations add core_v1.2.0_001_AddUserProfiles --context ApplicationDbContext
 \`\`\`
 
 ### Migration Rules
 
 1. **One migration per feature** - Group related changes in a single migration
-2. **Timestamps ensure order** - Use YYYYMMDD format for chronological sorting
-3. **Sequence numbers** - Use NNN (001, 002, etc.) for migrations on the same day
-4. **Descriptive names** - Use clear descriptions (InitialSchema, AddUserProfiles, etc.)
-5. **Schema must be specified** - All tables must specify their schema in ToTable()
+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
+4. **Context prefix** - Use \`core_\` for platform tables, \`extensions_\` for client extensions
+5. **Descriptive names** - Use clear PascalCase descriptions (CreateAuthUsers, AddUserProfiles, etc.)
+6. **Schema must be specified** - All tables must specify their schema in ToTable()
 
 ---
 
@@ -2166,7 +2194,7 @@ public interface IUserServiceHooks
 | Platform schema | \`${schemas.platform}\` | \`.ToTable("auth_Users", "${schemas.platform}")\` |
 | Extensions schema | \`${schemas.extensions}\` | \`.ToTable("client_Custom", "${schemas.extensions}")\` |
 | Table prefixes | \`${tablePrefixes.slice(0, 5).join(", ")}\`, etc. | \`auth_Users\`, \`nav_Modules\` |
-| Migration | \`YYYYMMDD_NNN_Description\` | \`20260115_001_InitialCreate\` |
+| Migration | \`{context}_v{version}_{seq}_{Desc}\` | \`core_v1.0.0_001_CreateAuthUsers\` |
 | Interface | \`I<Name>Service\` | \`IUserService\` |
 | Implementation | \`<Name>Service\` | \`UserService\` |
 | Domain namespace | \`${namespaces.domain}\` | - |