@happyvertical/smrt-core 0.40.42 → 0.40.44

package/dist/vite-plugin/sveltekit-generator.js

@@ -19,6 +19,9 @@ var STANDARD_API_ACTIONS = [
 	"update",
 	"delete"
 ];
+var GITIGNORE_MANAGED_BLOCK_START = "# BEGIN SMRT auto-generated routes (Vite plugin)";
+var GITIGNORE_MANAGED_BLOCK_END = "# END SMRT auto-generated routes (Vite plugin)";
+var LEGACY_GITIGNORE_HEADER = "# SMRT auto-generated routes (from Vite plugin)";
 /**
 * Extract simple class name from potentially qualified name.
 * Qualified names follow the pattern: @scope/package:ClassName
@@ -635,25 +638,28 @@ async function generateSvelteKitRoutes(projectRoot, manifest, options) {
 	clearGeneratedRouteFiles(join(projectRoot, options.routesDir));
 	clearGeneratedKnowledgeRoute(projectRoot, options);
 	await generateSmrtConfigFile(projectRoot, manifest, options);
+	const generatedRoutePaths = [];
 	let generatedCount = 0;
 	let skippedCollections = 0;
 	for (const [className, objectDef] of Object.entries(manifest.objects)) {
 		if (isCollectionManifestClass(manifest, objectDef)) {
-			if (await generateCollectionRoutesForObject(projectRoot, className, objectDef, manifest, options)) generatedCount++;
+			const collectionRoutePaths = await generateCollectionRoutesForObject(projectRoot, className, objectDef, manifest, options);
+			generatedRoutePaths.push(...collectionRoutePaths);
+			if (collectionRoutePaths.length > 0) generatedCount++;
 			else {
 				console.log(`[smrt] Skipping ${className} - no collection API routes to generate`);
 				skippedCollections++;
 			}
 			continue;
 		}
-		await generateRoutesForObject(projectRoot, className, objectDef, manifest, options);
+		generatedRoutePaths.push(...await generateRoutesForObject(projectRoot, className, objectDef, manifest, options));
 		generatedCount++;
 	}
-	if (options.knowledge?.api?.enabled) generateKnowledgeRoute(projectRoot, options);
-	generateSyncApplyRoute(projectRoot, manifest, options);
-	generateChangesRoute(projectRoot, manifest, options);
-	generateEventsRoute(projectRoot, manifest, options, computeWebManifestHash(manifest));
-	updateGitignore(projectRoot, options);
+	if (options.knowledge?.api?.enabled) generatedRoutePaths.push(generateKnowledgeRoute(projectRoot, options));
+	if (generateSyncApplyRoute(projectRoot, manifest, options)) generatedRoutePaths.push(join(projectRoot, options.routesDir, "sync", "apply", "+server.ts"));
+	if (generateChangesRoute(projectRoot, manifest, options)) generatedRoutePaths.push(join(projectRoot, options.routesDir, "_changes", "+server.ts"));
+	if (generateEventsRoute(projectRoot, manifest, options, computeWebManifestHash(manifest))) generatedRoutePaths.push(join(projectRoot, options.routesDir, "_events", "+server.ts"));
+	updateGitignore(projectRoot, options, generatedRoutePaths);
 	const skippedMsg = skippedCollections > 0 ? ` (skipped ${skippedCollections} collection classes)` : "";
 	console.log(`[smrt] Generated routes for ${generatedCount} SMRT objects${skippedMsg}`);
 }
@@ -686,7 +692,7 @@ function clearGeneratedKnowledgeRoute(projectRoot, options) {
 	if (readFileSync(routePath, "utf-8").startsWith("// Auto-generated by @smrt/core vite plugin")) unlinkSync(routePath);
 }
 function generateKnowledgeRoute(projectRoot, options) {
-	writeRoute(knowledgeRouteDir(projectRoot, options), "+server.ts", generateKnowledgeRouteTemplate(options.knowledge ?? {}, readKnowledgeRouteArtifact(projectRoot)));
+	return writeRoute(knowledgeRouteDir(projectRoot, options), "+server.ts", generateKnowledgeRouteTemplate(options.knowledge ?? {}, readKnowledgeRouteArtifact(projectRoot)));
 }
 function readKnowledgeRouteArtifact(projectRoot) {
 	for (const relativePath of [".smrt/smrt-knowledge.json", "dist/smrt-knowledge.json"]) {
@@ -938,16 +944,20 @@ export async function getCollection<
 async function generateRoutesForObject(projectRoot, className, objectDef, manifest, options) {
 	const collectionName = objectDef.collection;
 	const routeDir = join(projectRoot, options.routesDir, collectionName);
+	const generatedRoutePaths = [];
 	const apiConfig = objectDef.decoratorConfig?.api;
 	if (apiConfig === false) {
 		console.log(`[smrt] Skipping ${className} - API disabled`);
-		return;
+		return generatedRoutePaths;
 	}
 	const includedActions = resolveStandardCrudActions(apiConfig);
-	if (includedActions.includes("list") || includedActions.includes("create")) writeRoute(routeDir, "+server.ts", generateCollectionRouteTemplate(projectRoot, className, objectDef, manifest, includedActions, options, routeDir));
+	if (includedActions.includes("list") || includedActions.includes("create")) {
+		const collectionRoute = generateCollectionRouteTemplate(projectRoot, className, objectDef, manifest, includedActions, options, routeDir);
+		generatedRoutePaths.push(writeRoute(routeDir, "+server.ts", collectionRoute));
+	}
 	if (includedActions.includes("get") || includedActions.includes("update") || includedActions.includes("delete")) {
 		const itemRoute = generateItemRouteTemplate(projectRoot, className, objectDef, manifest, includedActions, options, join(routeDir, "[id]"));
-		writeRoute(join(routeDir, "[id]"), "+server.ts", itemRoute);
+		generatedRoutePaths.push(writeRoute(join(routeDir, "[id]"), "+server.ts", itemRoute));
 	}
 	const customActions = Object.entries(objectDef.methods).filter(([name, method]) => !STANDARD_API_ACTIONS.includes(name) && method.isPublic && shouldIncludeInApi(name, apiConfig));
 	const actionSpecs = [];
@@ -971,20 +981,24 @@ async function generateRoutesForObject(projectRoot, className, objectDef, manife
 			}
 		});
 	}
-	for (const [actionRouteDir, routeSpecs] of groupCustomActionRoutes(actionSpecs)) writeRoute(actionRouteDir, "+server.ts", generateActionRouteTemplate(projectRoot, actionRouteDir, routeSpecs, objectDef, manifest, options));
+	for (const [actionRouteDir, routeSpecs] of groupCustomActionRoutes(actionSpecs)) {
+		const actionRoute = generateActionRouteTemplate(projectRoot, actionRouteDir, routeSpecs, objectDef, manifest, options);
+		generatedRoutePaths.push(writeRoute(actionRouteDir, "+server.ts", actionRoute));
+	}
+	return generatedRoutePaths;
 }
 async function generateCollectionRoutesForObject(projectRoot, className, objectDef, manifest, options) {
+	const generatedRoutePaths = [];
 	const apiConfig = objectDef.decoratorConfig?.api;
 	if (apiConfig === false) {
 		console.log(`[smrt] Skipping ${className} - API disabled`);
-		return false;
+		return generatedRoutePaths;
 	}
 	const routeDir = join(projectRoot, options.routesDir, objectDef.collection);
 	const lookupClassName = findItemClassRegistryKey(className, objectDef, manifest);
 	const lookupObjectDef = findObjectDefByRegistryKey(manifest, lookupClassName);
 	const customActions = Object.entries(objectDef.methods).filter(([name, method]) => !STANDARD_API_ACTIONS.includes(name) && method.isPublic && shouldIncludeInApi(name, apiConfig));
-	if (customActions.length === 0) return false;
-	let generatedAnyRoutes = false;
+	if (customActions.length === 0) return generatedRoutePaths;
 	const actionSpecs = [];
 	for (const [actionName, actionDef] of customActions) {
 		const routeConfig = resolveApiActionRouteConfig(actionName, actionDef, apiConfig, { kebabRoutes: options.kebabRoutes }, "collection");
@@ -1006,10 +1020,10 @@ async function generateCollectionRoutesForObject(projectRoot, className, objectD
 		});
 	}
 	for (const [actionRouteDir, routeSpecs] of groupCustomActionRoutes(actionSpecs)) {
-		writeRoute(actionRouteDir, "+server.ts", generateActionRouteTemplate(projectRoot, actionRouteDir, routeSpecs, objectDef, manifest, options));
-		generatedAnyRoutes = true;
+		const actionRoute = generateActionRouteTemplate(projectRoot, actionRouteDir, routeSpecs, objectDef, manifest, options);
+		generatedRoutePaths.push(writeRoute(actionRouteDir, "+server.ts", actionRoute));
 	}
-	return generatedAnyRoutes;
+	return generatedRoutePaths;
 }
 /**
 * Resolve the list of standard CRUD actions exposed by the API for a given
@@ -1120,6 +1134,7 @@ function writeRoute(dir, filename, content) {
 		const filePath = join(dir, filename);
 		writeFileSync(filePath, content, "utf-8");
 		console.log(`[smrt] Generated: ${filePath}`);
+		return filePath;
 	} catch (error) {
 		console.error(`[smrt] [ERROR] Failed to write route file:`);
 		console.error(`[smrt] [ERROR]   Directory: ${dir}`);
@@ -1582,28 +1597,42 @@ function rolesFrom(value: unknown): string[] {
 /**
 * Updates .gitignore to exclude auto-generated routes
 */
-function updateGitignore(projectRoot, options) {
+function updateGitignore(projectRoot, options, generatedRoutePaths) {
 	const gitignorePath = join(projectRoot, ".gitignore");
-	const patternsToAdd = ["# SMRT auto-generated routes (from Vite plugin)", `${options.routesDir}/**/+server.ts`];
-	if (options.knowledge?.api?.enabled) {
-		const knowledgeRoute = relative(projectRoot, join(knowledgeRouteDir(projectRoot, options), "+server.ts")).replaceAll("\\", "/");
-		patternsToAdd.push(knowledgeRoute);
-	}
 	let gitignoreContent = "";
 	if (existsSync(gitignorePath)) gitignoreContent = readFileSync(gitignorePath, "utf-8");
-	let needsUpdate = false;
-	const linesToAdd = [];
-	for (const pattern of patternsToAdd) if (!gitignoreContent.includes(pattern)) {
-		linesToAdd.push(pattern);
-		needsUpdate = true;
+	const generatedPaths = [...new Set(generatedRoutePaths.map((path) => gitignorePatternForPath(relative(projectRoot, path))))].sort((a, b) => a.localeCompare(b));
+	const updatedContent = updateGeneratedRouteIgnoreBlock(gitignoreContent, options.routesDir, generatedPaths);
+	if (updatedContent !== gitignoreContent) {
+		writeFileSync(gitignorePath, updatedContent, "utf-8");
+		console.log("[smrt] Updated .gitignore with generated route paths");
 	}
-	if (needsUpdate) {
-		if (gitignoreContent.length > 0 && !gitignoreContent.endsWith("\n")) gitignoreContent += "\n";
-		if (gitignoreContent.length > 0 && !gitignoreContent.endsWith("\n\n")) gitignoreContent += "\n";
-		gitignoreContent += `${linesToAdd.join("\n")}\n`;
-		writeFileSync(gitignorePath, gitignoreContent, "utf-8");
-		console.log("[smrt] Updated .gitignore with generated route patterns");
+}
+function gitignorePatternForPath(path) {
+	return path.replaceAll("\\", "/").replaceAll(/([*?[\]\\!#])/g, "\\$1");
+}
+function updateGeneratedRouteIgnoreBlock(gitignoreContent, routesDir, generatedPaths) {
+	const lines = gitignoreContent.split("\n");
+	const startIndex = lines.indexOf(GITIGNORE_MANAGED_BLOCK_START);
+	const endIndex = lines.indexOf(GITIGNORE_MANAGED_BLOCK_END);
+	const managedBlock = [
+		GITIGNORE_MANAGED_BLOCK_START,
+		...generatedPaths,
+		GITIGNORE_MANAGED_BLOCK_END
+	];
+	if (startIndex !== -1 || endIndex !== -1) {
+		if (startIndex === -1 || endIndex < startIndex) return gitignoreContent;
+		lines.splice(startIndex, endIndex - startIndex + 1, ...generatedPaths.length > 0 ? managedBlock : []);
+		return lines.join("\n");
 	}
+	const legacyPattern = `${routesDir.replaceAll("\\", "/")}/**/+server.ts`;
+	const legacyIndex = lines.findIndex((line, index) => line === LEGACY_GITIGNORE_HEADER && lines[index + 1] === legacyPattern);
+	if (legacyIndex !== -1) lines.splice(legacyIndex, 2);
+	if (generatedPaths.length === 0) return lines.join("\n");
+	let updatedContent = lines.join("\n");
+	if (updatedContent.length > 0 && !updatedContent.endsWith("\n")) updatedContent += "\n";
+	if (updatedContent.length > 0 && !updatedContent.endsWith("\n\n")) updatedContent += "\n";
+	return `${updatedContent}${managedBlock.join("\n")}\n`;
 }
 //#endregion
 export { findCliApiCoherenceViolations, generateSvelteKitRoutes, methodNameToKebab, resolveApiActionRouteConfig, resolveApiActionSet, validateCliIncludeAgainstApi };