@geekmidas/cli 10.0.0-alpha.5 → 10.0.0-alpha.6

package/dist/{openapi-kkQbsRVj.mjs → openapi-Bw8DWFWz.mjs}

@@ -1,6 +1,7 @@
 import { ConstructGenerator, isPartitionedRoutes } from "./discover-CAlFgYec.mjs";
-import { loadWorkspaceConfig } from "./config-WRdsQYZK.mjs";
-import { dirname, join, relative } from "node:path";
+import { loadWorkspaceConfig } from "./config-CxP7P-qG.mjs";
+import { dirname, join, relative, resolve } from "node:path";
+import { kebabCase } from "@geekmidas/manifest";
 import { mkdir, writeFile } from "node:fs/promises";
 import { spawn } from "node:child_process";
 import { Endpoint } from "@geekmidas/constructs/endpoints";
@@ -602,24 +603,21 @@ export async function setupEndpoints(
 *
 * Falls back to the configured paths for an app that declares no surface.
 */
-function runtimeFor(context, fromDir) {
-	const module = context.surface?.module;
+function runtimeFor(context, fromDir, owner) {
+	const module = owner ? context.owners?.[owner] ?? context.surface?.module : context.surface?.module;
 	if (module) {
 		const specifier = importSpecifier(fromDir, module.specifier);
 		return {
 			imports: `import { ${module.exportName} as __surface } from '${specifier}';`,
 			bindings: [
-				"// What this surface was declared with. Objects, not module paths:",
-				"// the entry imports the surface, so there is nothing to print.",
+				"// What this construct was declared with. Objects, not module paths:",
+				"// the entry imports the owner, so there is nothing to print.",
 				"const envParser = __surface.envParser;",
 				"const logger = __surface.logger;"
 			].join("\n")
 		};
 	}
-	return {
-		imports: [`import ${context.envParserImportPattern} from '${importSpecifier(fromDir, context.envParserPath)}';`, `import ${context.loggerImportPattern} from '${importSpecifier(fromDir, context.loggerPath)}';`].join("\n"),
-		bindings: ""
-	};
+	throw new Error(`No owner to take a logger and an environment parser from${owner ? ` for "${owner}"` : ""}. Build it from a \`RestApi\` or a \`Worker\`.`);
 }
 /** A relative specifier ESM will accept: always prefixed, never bare. */
 function importSpecifier(fromDir, target) {
@@ -992,6 +990,7 @@ import type { Hono as HonoType } from 'hono';
 import { setupEndpoints } from './endpoints.js';
 import { setupSubscribers } from './subscribers.js';
 import { setupQueues } from './queues.js';
+import { setupCrons } from './crons.js';
 ${runtime.imports}
 ${telescopeImports}
 ${studioImports}
@@ -1079,6 +1078,12 @@ ${afterSetupCall}
         logger.error({ error }, 'Failed to start queue workers');
       });
 
+      // Schedule this app's crons. Caught like the others: a scheduler that
+      // cannot start is not a reason for the HTTP server not to.
+      await setupCrons(envParser, logger).catch((error) => {
+        logger.error({ error }, 'Failed to schedule crons');
+      });
+
       logger.info({ port }, 'Starting server');
 
       // Start HTTP server using provided serve function
@@ -1149,8 +1154,7 @@ export const handler = ${exportName};
 	async generateProductionAppFile(outputDir, context) {
 		const appFileName = "app.ts";
 		const appPath = join(outputDir, appFileName);
-		const relativeLoggerPath = relative(dirname(appPath), context.loggerPath);
-		const relativeEnvParserPath = relative(dirname(appPath), context.envParserPath);
+		const runtime = runtimeFor(context, dirname(appPath));
 		const production = context.production;
 		const healthCheckPath = production.healthCheck;
 		const enableGracefulShutdown = production.gracefulShutdown;
@@ -1186,9 +1190,16 @@ export const handler = ${exportName};
       await setupQueues(envParser, logger).catch((error) => {
         logger.error({ error }, 'Failed to start queue workers');
       });
+
+      // Schedule this app's crons. Caught like the others: a scheduler that
+      // cannot start is not a reason for the HTTP server not to.
+      await setupCrons(envParser, logger).catch((error) => {
+        logger.error({ error }, 'Failed to schedule crons');
+      });
 ` : "";
 		const subscriberImport = includeSubscribers ? `import { setupSubscribers } from './subscribers.js';
-import { setupQueues } from './queues.js';` : "";
+import { setupQueues } from './queues.js';
+import { setupCrons } from './crons.js';` : "";
 		const gracefulShutdownCode = enableGracefulShutdown ? `
   // Graceful shutdown handling
   let isShuttingDown = false;
@@ -1218,8 +1229,8 @@ import { Hono } from 'hono';
 import type { Hono as HonoType } from 'hono';
 import { setupEndpoints } from '${endpointsImportPath}';
 ${subscriberImport}
-import ${context.envParserImportPattern} from '${relativeEnvParserPath}';
-import ${context.loggerImportPattern} from '${relativeLoggerPath}';
+${runtime.imports}
+${runtime.bindings}
 ${hooksImports}
 ${cors.imports}
 ${context.storageDrivers?.imports ?? ""}
@@ -1827,6 +1838,16 @@ function normalizeRoutes(routes) {
 */
 const OPENAPI_OUTPUT_PATH = "./.gkm/openapi.ts";
 /**
+* Where a surface's generated client lands.
+*
+* Named from the surface, so `new RestApi('Webhooks')` writes
+* `.gkm/openapi/webhooks.ts` — the same kebab form that gives it its container
+* name and its `WEBHOOKS_URL`.
+*/
+function openApiPathFor(surfaceId) {
+	return `./.gkm/openapi/${kebabCase(surfaceId)}.ts`;
+}
+/**
 * Resolve OpenAPI config from GkmConfig
 */
 function resolveOpenApiConfig(config) {
@@ -1845,33 +1866,66 @@ function resolveOpenApiConfig(config) {
 	};
 }
 /**
-* Generate OpenAPI spec from endpoints
-* @returns Object with output path and endpoint count, or null if disabled
+* Load endpoints from a glob, then generate.
+*
+* For the callers that have no endpoints in hand — `gkm openapi`, and `gkm dev`
+* regenerating on reload. The glob is the app's derived one, off `AppSpec.code`,
+* not a `routes` field anybody wrote.
 */
-async function generateOpenApi(config, options = {}) {
+async function generateOpenApiFrom(routes, options = {}) {
+	const loaded = await new EndpointGenerator().load(routes, void 0, options.bustCache);
+	return generateOpenApi(loaded.map(({ construct }) => construct), options);
+}
+/**
+* Generate an OpenAPI spec for each surface these endpoints name.
+*
+* Takes the endpoints, not a glob to find them with. The build has already
+* loaded every one of them to generate handlers, so globbing again was a second
+* discovery pass over the same files — and the reason this function needed a
+* `routes` to be told where to look, which is how a retired config field stayed
+* load-bearing.
+*
+* @returns What was written, or null if disabled or there were none
+*/
+async function generateOpenApi(endpoints, options = {}) {
 	const logger = options.silent ? { log: () => {} } : console;
-	const openApiConfig = resolveOpenApiConfig(config);
+	const openApiConfig = resolveOpenApiConfig({ openapi: options.openapi });
 	if (!openApiConfig.enabled) return null;
-	const endpointGenerator = new EndpointGenerator();
-	const loadedEndpoints = await endpointGenerator.load(config.routes, void 0, options.bustCache);
-	if (loadedEndpoints.length === 0) {
+	if (endpoints.length === 0) {
 		logger.log("No valid endpoints found for OpenAPI generation");
 		return null;
 	}
-	const endpoints = loadedEndpoints.map(({ construct }) => construct);
-	const outputPath = join(process.cwd(), OPENAPI_OUTPUT_PATH);
-	await mkdir(dirname(outputPath), { recursive: true });
+	const bySurface = /* @__PURE__ */ new Map();
+	for (const endpoint of endpoints) {
+		const owner = endpoint.owner ?? endpoint.surface?.id;
+		if (!owner) continue;
+		const existing = bySurface.get(owner);
+		if (existing) existing.push(endpoint);
+		else bySurface.set(owner, [endpoint]);
+	}
+	if (bySurface.size === 0) {
+		logger.log("No endpoints named a surface, so no OpenAPI was generated");
+		return null;
+	}
 	const tsGenerator = new OpenApiTsGenerator();
-	const tsContent = await tsGenerator.generate(endpoints, {
-		title: openApiConfig.title,
-		version: openApiConfig.version,
-		description: openApiConfig.description
-	});
-	await writeFile(outputPath, tsContent);
-	logger.log(`📄 OpenAPI client generated: ${OPENAPI_OUTPUT_PATH}`);
+	const written = [];
+	for (const [surfaceId, surfaceEndpoints] of bySurface) {
+		const relative$1 = openApiPathFor(surfaceId);
+		const outputPath = join(process.cwd(), relative$1);
+		await mkdir(dirname(outputPath), { recursive: true });
+		const tsContent = await tsGenerator.generate(surfaceEndpoints, {
+			title: openApiConfig.title ?? surfaceId,
+			version: openApiConfig.version,
+			description: openApiConfig.description
+		});
+		await writeFile(outputPath, tsContent);
+		written.push(relative$1);
+		logger.log(`📄 OpenAPI client generated: ${relative$1} (${surfaceEndpoints.length} endpoints)`);
+	}
 	return {
-		outputPath,
-		endpointCount: loadedEndpoints.length
+		outputPath: join(process.cwd(), written[0]),
+		outputPaths: written.map((r) => join(process.cwd(), r)),
+		endpointCount: endpoints.length
 	};
 }
 async function openapiCommand(options = {}) {
@@ -1881,12 +1935,12 @@ async function openapiCommand(options = {}) {
 		if (loadedConfig.type === "single") {
 			const config = loadedConfig.raw;
 			if (!config.openapi) config.openapi = { enabled: true };
-			const result = await generateOpenApi(config);
+			const result = await generateOpenApiFrom(config.constructs, { openapi: config.openapi });
 			if (result) logger.log(`Found ${result.endpointCount} endpoints`);
 		} else {
 			const { workspace } = loadedConfig;
 			const workspaceRoot = workspace.root;
-			const backendApps = Object.entries(workspace.apps).filter(([_, app]) => app.type === "backend" && app.routes !== void 0 && app.openapi !== false && (typeof app.openapi !== "object" || app.openapi.enabled !== false));
+			const backendApps = Object.entries(workspace.apps).filter(([_, app]) => app.type === "backend" && app.openapi !== false && (typeof app.openapi !== "object" || app.openapi.enabled !== false));
 			if (backendApps.length === 0) {
 				logger.log("No backend apps with OpenAPI enabled found");
 				return;
@@ -1900,8 +1954,12 @@ async function openapiCommand(options = {}) {
 				return;
 			}
 			for (const [appName, app] of backendApps) {
-				if (app.type !== "backend" || !app.routes) continue;
 				const appPath = join(workspaceRoot, app.path);
+				if (resolve(appPath) === resolve(workspaceRoot)) {
+					const result = await generateOpenApiForApp(workspaceRoot, appName, app, false);
+					if (result) logger.log(`Found ${result.endpointCount} endpoints`);
+					continue;
+				}
 				await runOpenApiInSubprocess(appPath, appName);
 			}
 		}
@@ -1914,18 +1972,14 @@ async function openapiCommand(options = {}) {
 * Runs in-process. The caller is responsible for ensuring `process.cwd()`
 * is the app's directory so tsx loads the app's tsconfig path aliases.
 */
-async function generateOpenApiForApp(workspaceRoot, _appName, app) {
-	if (app.type !== "backend" || !app.routes) return null;
+async function generateOpenApiForApp(workspaceRoot, _appName, app, silent = true) {
+	if (app.type !== "backend") return null;
 	const appPath = join(workspaceRoot, app.path);
-	const routes = normalizeRoutes(app.routes);
-	const routesGlob = routes.map((r) => join(appPath, r));
-	const gkmConfig = {
-		routes: routesGlob,
-		envParser: app.envParser || "",
-		logger: app.logger || "",
-		openapi: app.openapi
-	};
-	return generateOpenApi(gkmConfig, { silent: true });
+	const globs = [join(appPath, "**/*.ts")];
+	return generateOpenApiFrom(globs, {
+		openapi: app.openapi ?? { enabled: true },
+		silent
+	});
 }
 /**
 * Resolve the gkm bin path. Tests can override via `GKM_BIN_PATH` env var
@@ -1963,5 +2017,5 @@ async function runOpenApiInSubprocess(appCwd, appName) {
 }
 
 //#endregion
-export { EndpointGenerator, OPENAPI_OUTPUT_PATH, generateOpenApi, normalizeRoutes, openapiCommand, resolveOpenApiConfig };
-//# sourceMappingURL=openapi-kkQbsRVj.mjs.map
+export { EndpointGenerator, OPENAPI_OUTPUT_PATH, generateOpenApi, generateOpenApiFrom, normalizeRoutes, openApiPathFor, openapiCommand, resolveOpenApiConfig, runtimeFor };
+//# sourceMappingURL=openapi-Bw8DWFWz.mjs.map