@kosdev-code/kos-codegen-core 3.0.21 → 3.0.23

package/index.js

@@ -1013,7 +1013,8 @@ function normalizeOptions(codegenFs, options, projects) {
   const booleanDefaults = {
     companion: false,
     skipRegistration: false,
-    futureAware: "none"
+    futureAware: "none",
+    singleton: false
   };
   return {
     ...booleanDefaults,
@@ -1105,6 +1106,15 @@ function readSourceFile(codegenFs, filePath, inspect) {
     project.createSourceFile(filePath, content, { overwrite: true })
   );
 }
+function decoratorConfigText(sourceFile, cls, decoratorName) {
+  const arg = cls.getDecorator(decoratorName)?.getArguments()[0];
+  if (!arg) return void 0;
+  if (arg.getKindName() === "ObjectLiteralExpression") return arg.getText();
+  const text = arg.getText().trim();
+  if (!/^[A-Za-z_$][\w$]*$/.test(text)) return text;
+  const initializer = sourceFile.getVariableDeclaration(text)?.getInitializer()?.getText();
+  return initializer ? initializer.replace(/\s+as\s+const\s*$/, "") : text;
+}
 function ensureNamedImport(sourceFile, moduleSpecifier, names) {
   const decls = sourceFile.getImportDeclarations().filter((d) => d.getModuleSpecifierValue() === moduleSpecifier);
   const existing = /* @__PURE__ */ new Set();
@@ -1903,9 +1913,9 @@ function readDeclaredModelType(codegenFs, filePath) {
   try {
     return readSourceFile(codegenFs, filePath, (sf) => {
       const cls = sf.getClasses().find((c) => c.getDecorator("kosModel"));
-      const arg = cls?.getDecorator("kosModel")?.getArguments()[0];
-      if (!arg) return void 0;
-      const inner = arg.getKindName() === "ObjectLiteralExpression" ? modelTypeIdProperty(arg.getText()) : arg.getText();
+      if (!cls) return void 0;
+      const config = decoratorConfigText(sf, cls, "kosModel");
+      const inner = config?.startsWith("{") ? modelTypeIdProperty(config) : config;
       if (!inner) return void 0;
       return resolveToStringLiteral(inner.trim(), sf.getFullText());
     });
@@ -2823,7 +2833,7 @@ function describeModel(codegenFs, options, projects) {
     };
   }
   const classDecorators = cls.getDecorators().map((d) => d.getName());
-  const kosModelArg = cls.getDecorator("kosModel") ? firstDecoratorArg(cls.getDecorator("kosModel").getText()) : void 0;
+  const kosModelArg = decoratorConfigText(sf, cls, "kosModel");
   const singleton = /\bsingleton\s*:\s*true\b/.test(kosModelArg ?? "");
   return {
     modelFilePath,
@@ -3254,6 +3264,13 @@ function assertServiceModuleAcceptsTransform(codegenFs, serviceModuleFile, model
     `${file} predates the transform-aware service layer (EndpointCtx takes ${typeParams} type parameter). Run \`kosui api:generate --project ${modelProject} --helpers-only\` to refresh the helper layer in place (no spec fetch, openapi.d.ts untouched), then add the service request.`
   );
 }
+const MUTATING_METHODS = /* @__PURE__ */ new Set(["post", "put", "delete", "patch"]);
+function assertLifecycleSuitsMethod(method, mode, methodName) {
+  if (mode !== "lifecycle" || !MUTATING_METHODS.has(method)) return;
+  throw new Error(
+    `${methodName} is a ${method.toUpperCase()} with a lifecycle, so it would run every time the model reaches that phase rather than when something calls it — a mutation on every load. Drop \`lifecycle\` to get the method-driven form (the default for ${method.toUpperCase()}), or pass \`mode: "method"\`. If the phase really is the trigger, the model calls the method from its own lifecycle hook, where the call is visible.`
+  );
+}
 function assertServiceModuleAcceptsProvisional(codegenFs, serviceModuleFile, modelProject) {
   const file = `${serviceModuleFile}.ts`;
   if (!codegenFs.exists(file)) return;
@@ -3435,6 +3452,7 @@ function addServiceRequestToModel(codegenFs, options, projects) {
   );
   const method = (options.method || "get").toLowerCase();
   const mode = options.mode ?? (options.lifecycle ? "lifecycle" : "method");
+  assertLifecycleSuitsMethod(method, mode, options.methodName);
   const lifecycle = options.lifecycle || "LOAD";
   const catalogName = `${pascalCase(modelBase)}Endpoints`;
   const mockMode = options.mock ?? "auto";