@happyvertical/smrt-core 0.40.31 → 0.40.32

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

@@ -3,7 +3,7 @@ import { AUTO_GENERATED_ROUTE_HEADER } from "./route-header.js";
 import { generateChangesRoute } from "./changes-route.js";
 import { generateEventsRoute } from "./events-route.js";
 import { generateSyncApplyRoute } from "./sync-apply-route.js";
-import { computeWebManifestHash } from "./web-collections.js";
+import { computeWebManifestHash, isCollectionManifestClass, resolveCollectionItemObject, resolveCollectionItemTypeName } from "./web-collections.js";
 import { existsSync, mkdirSync, readFileSync, readdirSync, unlinkSync, writeFileSync } from "node:fs";
 import { join, relative } from "node:path";
 //#region src/vite-plugin/sveltekit-generator.ts
@@ -611,9 +611,9 @@ function buildScopedOptionsForTenantRead(actionDef, routeConfig, tenantScoped) {
 	};
 }
 function findItemClassRegistryKey(className, objectDef, manifest) {
-	const itemClassName = objectDef.extendsTypeArg || (className.endsWith("Collection") ? className.slice(0, -10) : void 0);
-	if (!itemClassName) return className;
-	return Object.entries(manifest.objects).find(([manifestKey, candidate]) => manifestKey === itemClassName || candidate.className === itemClassName)?.[0] || itemClassName;
+	const itemObject = resolveCollectionItemObject(manifest, objectDef);
+	if (!itemObject) return resolveCollectionItemTypeName(manifest, objectDef) || className;
+	return Object.entries(manifest.objects).find(([, manifestObject]) => manifestObject === itemObject)?.[0] || itemObject.className;
 }
 function groupCustomActionRoutes(actionSpecs) {
 	const groupedRoutes = /* @__PURE__ */ new Map();
@@ -638,7 +638,7 @@ async function generateSvelteKitRoutes(projectRoot, manifest, options) {
 	let generatedCount = 0;
 	let skippedCollections = 0;
 	for (const [className, objectDef] of Object.entries(manifest.objects)) {
-		if (isCollectionClass(objectDef)) {
+		if (isCollectionManifestClass(manifest, objectDef)) {
 			if (await generateCollectionRoutesForObject(projectRoot, className, objectDef, manifest, options)) generatedCount++;
 			else {
 				console.log(`[smrt] Skipping ${className} - no collection API routes to generate`);
@@ -715,7 +715,7 @@ async function generateRegistrationFile(projectRoot, manifest, options) {
 			classNames: [],
 			hasCollectionImport: false
 		};
-		if (isCollectionClass(objectDef)) packageEntry.hasCollectionImport = true;
+		if (isCollectionManifestClass(manifest, objectDef)) packageEntry.hasCollectionImport = true;
 		else packageEntry.classNames.push(className);
 		packageObjects.set(objectDef.packageName, packageEntry);
 	} else localObjects.push([className, objectDef]);
@@ -728,7 +728,7 @@ async function generateRegistrationFile(projectRoot, manifest, options) {
 			const normalized = relative(configDir, objectDef.filePath).replace(/\\/g, "/").replace(/\.(ts|js|tsx|jsx)$/, "");
 			importPath = normalized.startsWith(".") ? normalized : `./${normalized}`;
 		} else importPath = getSvelteKitImportPath(projectRoot, void 0, options.objectsDir, simpleClassName);
-		if (isCollectionClass(objectDef)) {
+		if (isCollectionManifestClass(manifest, objectDef)) {
 			localSideEffectImports.add(importPath);
 			continue;
 		}
@@ -762,7 +762,7 @@ ${[Array.from(packageObjects.entries()).sort(([left], [right]) => left.localeCom
 
 // Re-register imported objects with explicit package names for bundled runtimes
 ${Object.entries(manifest.objects).map(([className, objectDef]) => {
-		if (isCollectionClass(objectDef)) return null;
+		if (isCollectionManifestClass(manifest, objectDef)) return null;
 		const simpleClassName = extractSimpleClassName(className);
 		const packageName = getRegistrationPackageName(manifest, objectDef, isLocalObject(projectRoot, objectDef));
 		if (!packageName) return null;
@@ -1052,11 +1052,11 @@ function shouldIncludeInApi(actionName, apiConfig) {
 * to drive route generation, exposed so coherence checks (e.g. CLI vs API)
 * can ask "what does the API expose?" without re-implementing the logic.
 */
-function resolveApiActionSet(objectDef) {
+function resolveApiActionSet(objectDef, manifest) {
 	const apiConfig = objectDef.decoratorConfig?.api;
 	if (apiConfig === false) return /* @__PURE__ */ new Set();
-	const actions = new Set(resolveStandardCrudActions(apiConfig));
-	const objectIsCollectionClass = isCollectionClass(objectDef);
+	const objectIsCollectionClass = manifest ? isCollectionManifestClass(manifest, objectDef) : isCollectionClass(objectDef);
+	const actions = objectIsCollectionClass ? /* @__PURE__ */ new Set() : new Set(resolveStandardCrudActions(apiConfig));
 	const config = getApiConfigObject(apiConfig);
 	for (const [name, method] of Object.entries(objectDef.methods || {})) {
 		if (STANDARD_API_ACTIONS.includes(name)) continue;
@@ -1090,7 +1090,7 @@ function findCliApiCoherenceViolations(manifest) {
 		const cliExclude = Array.isArray(cliConfig.exclude) ? cliConfig.exclude : [];
 		const effectiveCliCommands = cliConfig.include.filter((cmd) => !cliExclude.includes(cmd));
 		if (effectiveCliCommands.length === 0) continue;
-		const apiActionSet = resolveApiActionSet(objectDef);
+		const apiActionSet = resolveApiActionSet(objectDef, manifest);
 		const unreachable = effectiveCliCommands.filter((action) => !apiActionSet.has(action));
 		if (unreachable.length > 0) violations.push({
 			className,