@topogram/cli 0.3.44 → 0.3.45

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@topogram/cli",
-  "version": "0.3.44",
+  "version": "0.3.45",
   "description": "Topogram CLI for checking Topogram workspaces and generating app bundles.",
   "license": "Apache-2.0",
   "repository": {

package/src/cli.js

@@ -595,9 +595,9 @@ function printGeneratorHelp() {
   console.log("Inspects generator manifests and checks generator pack conformance.");
   console.log("");
   console.log("Notes:");
-  console.log("  - list shows bundled generators plus installed package-backed generators declared in package.json.");
-  console.log("  - show accepts an installed package name or a bundled fallback generator id.");
-  console.log("  - check validates a local generator package path or an already installed package.");
+  console.log("  - list shows bundled generators plus installed package-backed generators declared in package.json; it reads manifests only.");
+  console.log("  - show accepts an installed package name or a bundled fallback generator id; it does not load adapter code.");
+  console.log("  - check validates a local generator package path or an already installed package by loading the adapter and running smoke generation.");
   console.log("  - Topogram does not install generator packages during show or check.");
   console.log(`  - package-backed project generators are governed by ${GENERATOR_POLICY_FILE}; bundled topogram/* generators are allowed.`);
   console.log("");
@@ -1250,6 +1250,7 @@ function printGeneratorCheck(payload) {
     console.log(`Projection platforms: ${payload.manifest.projectionPlatforms.join(", ")}`);
     console.log(`Source mode: ${payload.manifest.source}`);
   }
+  console.log("Executes package code: yes (loads adapter and runs smoke generate)");
   console.log("");
   console.log("Checks:");
   for (const check of payload.checks || []) {
@@ -1285,6 +1286,8 @@ function generatorManifestSummary(manifest, metadata = {}) {
     stack: manifest.stack || {},
     capabilities: manifest.capabilities || {},
     source: manifest.source,
+    loadsAdapter: false,
+    executesPackageCode: false,
     ...(manifest.profile ? { profile: manifest.profile } : {}),
     ...(manifest.package ? { package: manifest.package } : {}),
     ...(installCommand ? { installCommand } : {}),
@@ -1496,6 +1499,8 @@ function printGeneratorList(payload) {
     const stack = Object.values(generator.stack || {}).join(" + ") || "not declared";
     console.log(`- ${id}${generator.version ? `@${generator.version}` : ""} (${generator.surface || "unknown"}, ${status})`);
     console.log(`  Source: ${generator.source}`);
+    console.log("  Adapter loaded: no");
+    console.log("  Executes package code: no");
     if (generator.source === "package") {
       console.log(`  Installed: ${generator.installed ? "yes" : "no"}`);
     }
@@ -1529,6 +1534,8 @@ function printGeneratorShow(payload) {
   console.log(`Generator: ${generator.id}@${generator.version}`);
   console.log(`Surface: ${generator.surface}`);
   console.log(`Source: ${generator.source}${generator.planned ? " (planned)" : ""}`);
+  console.log("Adapter loaded: no");
+  console.log("Executes package code: no");
   if (generator.source === "package") {
     console.log(`Installed: ${generator.installed ? "yes" : "no"}`);
   }

package/src/generator/adapters.js

@@ -10,6 +10,10 @@ import {
   resolveGeneratorManifestForBinding,
   validateGeneratorManifest
 } from "./registry.js";
+import {
+  generatorPolicyDiagnosticsForBindings,
+  loadGeneratorPolicy
+} from "../generator-policy.js";
 import { generateDbContractGraph } from "./surfaces/databases/contract.js";
 import { generateDbLifecyclePlan } from "./surfaces/databases/lifecycle-shared.js";
 import {
@@ -252,6 +256,26 @@ function loadPackageGeneratorAdapter(manifest, component, options = {}) {
     throw new Error(`Component '${component?.id || "unknown"}' generator '${manifest.id}@${manifest.version}' is package-backed but does not declare a package.`);
   }
   const rootDir = options.configDir || options.rootDir || process.cwd();
+  const diagnostics = generatorPolicyDiagnosticsForBindings(
+    loadGeneratorPolicy(rootDir),
+    [{
+      componentId: String(component?.id || "unknown"),
+      componentType: String(component?.type || manifest.surface || "unknown"),
+      projection: String(component?.projection?.id || component?.projection || "unknown"),
+      generatorId: String(component?.generator?.id || manifest.id),
+      version: String(component?.generator?.version || manifest.version),
+      packageName
+    }],
+    "generator-adapter"
+  );
+  const errors = diagnostics.filter((diagnostic) => diagnostic.severity === "error");
+  if (errors.length > 0) {
+    throw new Error(errors.map((diagnostic) =>
+      diagnostic.suggestedFix
+        ? `${diagnostic.message} Suggested fix: ${diagnostic.suggestedFix}`
+        : diagnostic.message
+    ).join("\n"));
+  }
   let moduleValue;
   try {
     moduleValue = requireFromProject(rootDir)(packageName);

package/src/generator/check.js

@@ -25,6 +25,7 @@ import {
  * @property {Array<{ name: string, ok: boolean, message: string }>} checks
  * @property {string[]} errors
  * @property {{ files: number, artifacts: number, diagnostics: number }|null} smoke
+ * @property {boolean} executesPackageCode
  */
 
 /**
@@ -238,7 +239,8 @@ export function checkGeneratorPack(sourceSpec, options = {}) {
     manifest: null,
     checks: [],
     errors: [],
-    smoke: null
+    smoke: null,
+    executesPackageCode: true
   };
 
   /** @type {any|null} */

package/src/generator/surfaces/services/stateless.js

@@ -1,6 +1,7 @@
 // @ts-check
 
 import { getProjection } from "../shared.js";
+import { generateServerContract } from "./server-contract.js";
 
 function renderPackageJson(profile) {
   const dependencies = profile === "express"
@@ -41,10 +42,10 @@ function routePath(path) {
   return String(path || "/").replace(/:([A-Za-z0-9_]+)/g, ":$1");
 }
 
-function renderHonoIndex(projection) {
-  const routes = (projection.http || []).map((route) => {
+function renderHonoIndex(projection, contract) {
+  const routes = (contract.routes || []).map((route) => {
     const method = String(route.method || "GET").toLowerCase();
-    return `app.${method}("${routePath(route.path)}", (c) => c.json({ ok: true, capability: "${route.capabilityId}", input: { params: c.req.param(), query: c.req.query() } }, ${route.success || 200} as any));`;
+    return `app.${method}("${routePath(route.path)}", (c) => c.json({ ok: true, capability: "${route.capabilityId}", input: { params: c.req.param(), query: c.req.query() } }, ${route.successStatus || 200} as any));`;
   }).join("\n");
   return `import { serve } from "@hono/node-server";
 import { Hono } from "hono";
@@ -65,10 +66,10 @@ function expressPath(path) {
   return routePath(path);
 }
 
-function renderExpressIndex(projection) {
-  const routes = (projection.http || []).map((route) => {
+function renderExpressIndex(projection, contract) {
+  const routes = (contract.routes || []).map((route) => {
     const method = String(route.method || "GET").toLowerCase();
-    return `app.${method}("${expressPath(route.path)}", (req, res) => res.status(${route.success || 200}).json({ ok: true, capability: "${route.capabilityId}", input: { params: req.params, query: req.query } }));`;
+    return `app.${method}("${expressPath(route.path)}", (req, res) => res.status(${route.successStatus || 200}).json({ ok: true, capability: "${route.capabilityId}", input: { params: req.params, query: req.query } }));`;
   }).join("\n");
   return `import express from "express";
 
@@ -88,10 +89,11 @@ app.listen(port, () => {
 
 export function generateStatelessServer(graph, options = {}) {
   const projection = getProjection(graph, options.projectionId);
+  const contract = generateServerContract(graph, { ...options, projectionId: projection.id });
   const profile = options.profile === "express" ? "express" : "hono";
   return {
     "package.json": renderPackageJson(profile),
     "tsconfig.json": renderTsconfig(),
-    "src/index.ts": profile === "express" ? renderExpressIndex(projection) : renderHonoIndex(projection)
+    "src/index.ts": profile === "express" ? renderExpressIndex(projection, contract) : renderHonoIndex(projection, contract)
   };
 }