@atlashub/smartstack-mcp 1.1.0 → 1.2.1

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";
@@ -132,11 +131,24 @@ var defaultConfig = {
     apiEnabled: process.env.SMARTSTACK_API_ENABLED !== "false"
   },
   conventions: {
-    tablePrefix: {
-      core: "Core_",
-      client: "Client_"
+    schemas: {
+      platform: "core",
+      extensions: "extensions"
     },
-    migrationFormat: "YYYYMMDD_{Prefix}_NNN_{Description}",
+    tablePrefixes: [
+      "auth_",
+      "nav_",
+      "usr_",
+      "ai_",
+      "cfg_",
+      "wkf_",
+      "support_",
+      "entra_",
+      "ref_",
+      "loc_",
+      "lic_"
+    ],
+    migrationFormat: "{context}_v{version}_{sequence}_{Description}",
     namespaces: {
       domain: "SmartStack.Domain",
       application: "SmartStack.Application",
@@ -210,9 +222,9 @@ function mergeConfig(base, override) {
     conventions: {
       ...base.conventions,
       ...override.conventions,
-      tablePrefix: {
-        ...base.conventions.tablePrefix,
-        ...override.conventions?.tablePrefix
+      schemas: {
+        ...base.conventions.schemas,
+        ...override.conventions?.schemas
       },
       namespaces: {
         ...base.conventions.namespaces,
@@ -520,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: {
@@ -662,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);
@@ -675,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)`
       });
     }
   }
@@ -683,18 +695,22 @@ 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})`
+        });
+      }
     }
   }
 }
-async function validateServiceInterfaces(structure, config, result) {
+async function validateServiceInterfaces(structure, _config, result) {
   if (!structure.application) {
     result.warnings.push({
       type: "warning",
@@ -858,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")) {
@@ -868,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
@@ -879,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
@@ -888,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"
+        });
+      }
     }
   }
 }
@@ -987,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`,
@@ -1005,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) {
@@ -1034,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("");
@@ -1269,7 +1301,7 @@ public class {{name}}Configuration : IEntityTypeConfiguration<{{domainNamespace}
     baseEntity,
     infrastructureNamespace: config.conventions.namespaces.infrastructure,
     domainNamespace: config.conventions.namespaces.domain,
-    tablePrefix: config.conventions.tablePrefix.core
+    schema: config.conventions.schemas.platform
   };
   const entityContent = Handlebars.compile(entityTemplate)(context);
   const configContent = Handlebars.compile(configTemplate)(context);
@@ -1965,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()
 
 ---
 
@@ -2153,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}\` | - |
@@ -2256,7 +2297,8 @@ async function getProjectInfoResource(config) {
   lines.push(JSON.stringify({
     smartstack: config.smartstack,
     conventions: {
-      tablePrefix: config.conventions.tablePrefix,
+      schemas: config.conventions.schemas,
+      tablePrefixes: config.conventions.tablePrefixes,
       migrationFormat: config.conventions.migrationFormat
     }
   }, null, 2));
@@ -2319,7 +2361,7 @@ async function getApiEndpointsResource(config, endpointFilter) {
   ) : allEndpoints;
   return formatEndpoints(endpoints, endpointFilter);
 }
-async function parseController(filePath, rootPath) {
+async function parseController(filePath, _rootPath) {
   const content = await readText(filePath);
   const fileName = path10.basename(filePath, ".cs");
   const controllerName = fileName.replace("Controller", "");
@@ -2327,10 +2369,8 @@ async function parseController(filePath, rootPath) {
   const routeMatch = content.match(/\[Route\s*\(\s*"([^"]+)"\s*\)\]/);
   const baseRoute = routeMatch ? routeMatch[1].replace("[controller]", controllerName.toLowerCase()) : `/api/${controllerName.toLowerCase()}`;
   const classAuthorize = /\[Authorize\s*(?:\([^)]*\))?\s*\]\s*(?:\[.*\]\s*)*public\s+class/.test(content);
-  const methodPattern = /\/\/\/\s*<summary>\s*\n\s*\/\/\/\s*([^\n]+)\s*\n\s*\/\/\/\s*<\/summary>[\s\S]*?(?=\[Http)|(\[Http(Get|Post|Put|Patch|Delete)(?:\s*\(\s*"([^"]*)"\s*\))?\][\s\S]*?public\s+(?:async\s+)?(?:Task<)?(?:ActionResult<)?(\w+)(?:[<>[\],\s\w]*)?\s+(\w+)\s*\(([^)]*)\))/g;
-  let lastSummary = "";
-  let match;
   const httpMethods = ["HttpGet", "HttpPost", "HttpPut", "HttpPatch", "HttpDelete"];
+  let match;
   for (const httpMethod of httpMethods) {
     const regex = new RegExp(
       `\\[${httpMethod}(?:\\s*\\(\\s*"([^"]*)"\\s*\\))?\\]\\s*(?:\\[.*?\\]\\s*)*public\\s+(?:async\\s+)?(?:Task<)?(?:ActionResult<)?(\\w+)(?:[<>\\[\\],\\s\\w]*)?\\s+(\\w+)\\s*\\(([^)]*)\\)`,
@@ -2500,13 +2540,11 @@ async function getDbSchemaResource(config, tableFilter) {
   ) : entities;
   return formatSchema(filteredEntities, tableFilter, config);
 }
-async function parseEntity(filePath, rootPath, config) {
+async function parseEntity(filePath, rootPath, _config) {
   const content = await readText(filePath);
-  const fileName = path11.basename(filePath, ".cs");
   const classMatch = content.match(/public\s+(?:class|record)\s+(\w+)(?:\s*:\s*(\w+))?/);
   if (!classMatch) return null;
   const entityName = classMatch[1];
-  const baseClass = classMatch[2];
   if (entityName.endsWith("Dto") || entityName.endsWith("Command") || entityName.endsWith("Query") || entityName.endsWith("Handler")) {
     return null;
   }
@@ -2556,7 +2594,7 @@ async function parseEntity(filePath, rootPath, config) {
       isPrimaryKey: propertyName === "Id" || propertyName === `${entityName}Id`
     });
   }
-  const tableName = `${config.conventions.tablePrefix.core}${entityName}s`;
+  const tableName = `cfg_${entityName}s`;
   return {
     name: entityName,
     tableName,
@@ -2565,7 +2603,7 @@ async function parseEntity(filePath, rootPath, config) {
     file: path11.relative(rootPath, filePath)
   };
 }
-async function enrichFromConfigurations(entities, infrastructurePath, config) {
+async function enrichFromConfigurations(entities, infrastructurePath, _config) {
   const configFiles = await findFiles("**/Configurations/**/*.cs", {
     cwd: infrastructurePath
   });
@@ -2619,7 +2657,7 @@ function mapCSharpType(csharpType) {
   const baseType = csharpType.replace("?", "");
   return typeMap[baseType] || "nvarchar";
 }
-function formatSchema(entities, filter, config) {
+function formatSchema(entities, filter, _config) {
   const lines = [];
   lines.push("# SmartStack Database Schema");
   lines.push("");