@beignet/cli 0.0.26 → 0.0.28

package/dist/inspect.js

@@ -4,6 +4,7 @@ import { parseProviderPackageMetadata } from "@beignet/core/providers";
 import { createPainter } from "./ansi.js";
 import { clientDirPath, clientFormsPath, clientIndexPath, defaultBeignetConfig, directoryPath, loadBeignetConfig, normalizePath, resolveConfig, } from "./config.js";
 import { formatGithubAnnotation, } from "./github-annotations.js";
+import { providersFilePath, wireListenersProviderSource, } from "./make/shared.js";
 import { detectedProviderVariants, diagnosticPackageJsonFile, formatProviderMetadataIssues, installedPackageNames, providerDoctorRulesForInstalledPackages, providerOperationalConfigFiles, providerRequiredEnvExists, providerRequiredTableExists, providerVariantRegistrationSeverity, readProviderListEntries, readProviderPackageMetadataSource, registrationTokenHint, serverProviderFiles, } from "./provider-audit.js";
 import { appendToArrayExpression, appendToNamedArray, appendToOutboxRegistryArray, arrayInitializerInfo, identifiersFromArrayExpression, insertAfterImports, matchingDelimiterIndex, outboxRegistryValueInfo, } from "./registry-edits.js";
 import { getCliVersion } from "./version.js";
@@ -61,6 +62,9 @@ export async function applyDoctorFixes(options = {}) {
     if (taskFix)
         fixes.push(taskFix);
     fixes.push(...(await fixUnregisteredOutboxEntries(targetDir, files, drift, config)));
+    const listenerFix = await fixUnregisteredListeners(targetDir, files, drift, config);
+    if (listenerFix)
+        fixes.push(listenerFix);
     const openApiFix = await fixDirectOpenApiArrayDrift(targetDir, files, config, convention, contracts, matchedRoutes);
     if (openApiFix)
         fixes.push(openApiFix);
@@ -595,6 +599,22 @@ function parseRouteExports(source, handlerFile, routePath, config) {
     const exports = [];
     const imports = parseNamedImports(source, config);
     const exportRegex = /export const\s+(GET|POST|PUT|PATCH|DELETE|HEAD|OPTIONS)\s*=\s*([^;\n]+)/g;
+    const apiRouteExportRegex = /export const\s*\{([^}]+)\}\s*=\s*createApiRoute\s*\(/g;
+    for (const match of source.matchAll(apiRouteExportRegex)) {
+        for (const member of match[1].split(",")) {
+            const parts = member.split(":");
+            const method = (parts[1] ?? parts[0])?.trim();
+            if (!method || !isHttpMethod(method))
+                continue;
+            exports.push({
+                method,
+                handlerFile,
+                contractRef: routePath,
+                catchAllPrefix: catchAllRoutePrefix(routePath),
+                source: "next-route",
+            });
+        }
+    }
     for (const match of source.matchAll(exportRegex)) {
         const method = match[1];
         const expression = match[2];
@@ -822,7 +842,7 @@ async function inspectDiagnostics(targetDir, files, config, convention, contract
     for (const routeHandler of matchedRoutes.unmatchedRouteHandlers) {
         diagnostics.push(unmatchedRouteDiagnostic(routeHandler));
     }
-    diagnostics.push(...(await inspectPackageScripts(targetDir, files, convention)), ...(await inspectOpenApiDrift(targetDir, files, config, convention, contracts, matchedRoutes)), ...(await inspectFeatureRouteRegistration(targetDir, files, config, convention, contracts)), ...(await inspectRouteErrorCatalogDrift(targetDir, files, config, convention, contracts, strict)), ...(await inspectCanonicalConformance(targetDir, files, config, convention, strict)), ...(await inspectVersionSkew(targetDir, files)), ...(await inspectProviderMetadataDrift(targetDir, files, convention)), ...(await inspectPortProviderDrift(targetDir, files, config, convention)), ...(await inspectProviderRegistrationDrift(targetDir, files, config, convention)), ...(await inspectDatabaseLifecycleDrift(targetDir, files, config, convention, strict)), ...(await inspectAuthzAuditDrift(targetDir, files, config, convention, contracts)), ...(await inspectServerlessFootguns(targetDir, files)), ...(await inspectProductionReadiness(targetDir, files, config, convention)), ...(await inspectFeatureArtifactDrift(targetDir, files, config, convention)), ...(await inspectWorkflowRegistrationDrift(targetDir, files, config, convention)), ...(await inspectResourceSlices(targetDir, files, config, convention, strict)));
+    diagnostics.push(...(await inspectPackageScripts(targetDir, files, convention)), ...(await inspectOpenApiDrift(targetDir, files, config, convention, contracts, matchedRoutes)), ...(await inspectFeatureRouteRegistration(targetDir, files, config, convention, contracts)), ...(await inspectRouteErrorCatalogDrift(targetDir, files, config, convention, contracts, strict)), ...(await inspectCanonicalConformance(targetDir, files, config, convention, strict)), ...(await inspectVersionSkew(targetDir, files)), ...(await inspectProviderMetadataDrift(targetDir, files, convention)), ...(await inspectPortProviderDrift(targetDir, files, config, convention)), ...(await inspectProviderRegistrationDrift(targetDir, files, config, convention)), ...(await inspectDatabaseLifecycleDrift(targetDir, files, config, convention, strict)), ...(await inspectAuthzAuditDrift(targetDir, files, config, convention, contracts)), ...(await inspectServerlessFootguns(targetDir, files, config, convention)), ...(await inspectProductionReadiness(targetDir, files, config, convention)), ...(await inspectFeatureArtifactDrift(targetDir, files, config, convention)), ...(await inspectWorkflowRegistrationDrift(targetDir, files, config, convention)), ...(await inspectResourceSlices(targetDir, files, config, convention, strict)));
     return dedupeDiagnostics(diagnostics);
 }
 async function inspectCanonicalConformance(targetDir, files, config, convention, strict) {
@@ -2314,7 +2334,7 @@ function isWorkflowTestFile(file) {
         file.startsWith("tests/") ||
         file.includes("/tests/"));
 }
-async function inspectServerlessFootguns(targetDir, files) {
+async function inspectServerlessFootguns(targetDir, files, config, convention) {
     const diagnostics = [];
     for (const file of files) {
         if (!file.endsWith(".ts") ||
@@ -2326,6 +2346,16 @@ async function inspectServerlessFootguns(targetDir, files) {
         const isProviderFile = /(^|\/)provider(s)?\.ts$/.test(file) ||
             /(^|\/)[^/]*provider[^/]*\.ts$/.test(file);
         const source = await readFile(path.join(targetDir, file), "utf8");
+        if (convention.resourceGenerator &&
+            isNextRouteFile(file, config) &&
+            importsEagerServer(source, file, files, config)) {
+            diagnostics.push({
+                severity: "warning",
+                code: "BEIGNET_NEXT_ROUTE_EAGER_SERVER_IMPORT",
+                file,
+                message: `${file} imports the eager server singleton. Next.js imports route modules during production builds, so use createNextServerLoader(...) in ${config.paths.server} and pass getServer to createApiRoute(...), createHealthRoute(...), or the focused route helper instead.`,
+            });
+        }
         if (isInfraOrServerFile &&
             isProviderFile &&
             /\bset(?:Interval|Timeout)\s*\(/.test(source)) {
@@ -2370,6 +2400,26 @@ async function inspectServerlessFootguns(targetDir, files) {
     }
     return diagnostics;
 }
+function isNextRouteFile(file, config) {
+    const routesPath = directoryPath(config.paths.routes);
+    return (file.startsWith(`${routesPath}/`) &&
+        file.endsWith("/route.ts") &&
+        !file.endsWith(".test.ts"));
+}
+function importsEagerServer(source, importerFile, files, config) {
+    const importRegex = /import\s+\{([^}]+)\}\s+from\s+["']([^"']+)["']/g;
+    for (const match of source.matchAll(importRegex)) {
+        const members = match[1]
+            .split(",")
+            .map((member) => parseImportMember(member));
+        if (!members.some((member) => member?.importedName === "server"))
+            continue;
+        const sourceFile = sourceFileFromImport(match[2], importerFile, files);
+        if (sourceFile === config.paths.server)
+            return true;
+    }
+    return false;
+}
 function hasOutboxRegistry(files) {
     return files.some(isOutboxRegistryFile);
 }
@@ -2533,7 +2583,9 @@ async function inspectDatabaseLifecycleDrift(targetDir, files, config, conventio
         {
             script: "db:seed",
             command: "seed",
-            file: `${infrastructurePath}/db/seed.ts`,
+            // The seed entrypoint lives beside the server runtime because it boots
+            // the app through getServer(), which infra/ must not import.
+            file: `${directoryPath(path.dirname(config.paths.server))}/seed.ts`,
             code: "BEIGNET_DB_SEED_ENTRYPOINT_MISSING",
         },
         {
@@ -3341,6 +3393,72 @@ async function fixUnregisteredTasks(targetDir, files, drift, config) {
         append: (source, entry, importLine) => appendToNamedArray(source, "tasks", entry, importLine),
     });
 }
+/**
+ * Wire fully unregistered feature listener registries into the providers
+ * file with the same `<registry>Provider` block `beignet make listener`
+ * generates, so both write paths stay byte-compatible.
+ *
+ * Bails without writing when the providers file or its exported array is
+ * missing or customized, when the app has no eventBus port to register
+ * against, or when the registry name is already imported from another file.
+ * The diagnostic stays in every bail case.
+ */
+async function fixUnregisteredListeners(targetDir, files, drift, config) {
+    const candidates = drift.listeners.unregistered.filter((entry) => entry.fullyUnregistered);
+    if (candidates.length === 0)
+        return undefined;
+    const providersFile = providersFilePath(config);
+    const contextFile = normalizePath(path.join(path.dirname(config.paths.server), "context.ts"));
+    const wiringDependencies = [
+        providersFile,
+        contextFile,
+        config.paths.appContext,
+        config.paths.ports,
+    ];
+    if (wiringDependencies.some((file) => !files.includes(file))) {
+        return undefined;
+    }
+    // The generated wiring registers listeners on ports.eventBus; without the
+    // port the fix would produce code that cannot typecheck.
+    const portsSource = await readFile(path.join(targetDir, config.paths.ports), "utf8");
+    if (!/\beventBus\s*[:?]/.test(portsSource))
+        return undefined;
+    const providersPath = path.join(targetDir, providersFile);
+    const original = await readFile(providersPath, "utf8");
+    let next = original;
+    const registeredNames = [];
+    for (const { registry } of candidates) {
+        const imported = parseNamedImportSources(next).get(registry.registryName);
+        if (imported) {
+            const importedFile = sourceFileFromImport(imported.sourcePath, providersFile, files);
+            if (importedFile !== registry.indexFile)
+                return undefined;
+        }
+        const result = wireListenersProviderSource(next, {
+            registryName: registry.registryName,
+            featureKebab: listenerFeatureKebab(registry.indexFile),
+            listenerModule: `@/${modulePath(registry.indexFile)}`,
+            config,
+        });
+        if (result.kind === "missing")
+            return undefined;
+        if (result.kind === "updated") {
+            next = result.source;
+            registeredNames.push(registry.registryName);
+        }
+    }
+    if (next === original || registeredNames.length === 0)
+        return undefined;
+    await writeFile(providersPath, next);
+    return {
+        code: "BEIGNET_LISTENER_UNREGISTERED",
+        file: providersFile,
+        message: `Registered ${registeredNames.join(", ")} with registerListeners(...) in ${providersFile}.`,
+    };
+}
+function listenerFeatureKebab(indexFile) {
+    return path.posix.basename(path.posix.dirname(path.posix.dirname(normalizePath(indexFile))));
+}
 async function fixUnregisteredOutboxEntries(targetDir, files, drift, config) {
     if (!drift.outbox.exists)
         return [];