@keq-request/cli 5.0.0-alpha.12 → 5.0.0-alpha.14

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 ## 5.0.0-alpha.6 (2025-09-17)
 
+## 5.0.0-alpha.14
+
+### Patch Changes
+
+- 0459484: the type exported add type prefix
+- 0459484: unabled add ignore rule by cli
+- 0459484: triggers type coercion when form-data requestBody filed is not string or file
+  - keq@5.0.0-alpha.14
+
+## 5.0.0-alpha.13
+
+### Patch Changes
+
+- 9d481d8: Generation will no longer be skipped when the object schema contains only the `additionalProperties` key.
+- 5150d57: The generated type file has an error when no requestBody definition in swagger.
+  - keq@5.0.0-alpha.13
+
 ## 5.0.0-alpha.12
 
 ### Patch Changes

package/dist/cli.cjs

@@ -1100,7 +1100,10 @@ function createPersistArtifactTask() {
     title: "Write files",
     task: async (context, task) => {
       if (!context.setup) throw new Error("Please run setup task first.");
-      if (!context.compiled) throw new Error("Please run compile task first.");
+      if (!context.compiled) {
+        task.skip("No compiled artifacts to persist.");
+        return;
+      }
       const rc = context.setup.rc;
       const artifacts = context.compiled.artifacts;
       for (const artifact of artifacts) {
@@ -1231,11 +1234,12 @@ function generateArray(schema, alias) {
   return "any[]";
 }
 function indent(space, text) {
+  if (text === "") return "";
   const indentation = " ".repeat(space);
   return text.split("\n").map((line) => `${indentation}${line}`).join("\n");
 }
 function generateObject(schema, alias) {
-  if (!schema.properties || !Object.keys(schema.properties).length) {
+  if ((!schema.properties || R11.isEmpty(schema.properties)) && (!schema.additionalProperties || R11.isEmpty(schema.additionalProperties))) {
     return "object";
   }
   const $properties = Object.entries(schema.properties || {}).map(([propertyName, propertySchema]) => {
@@ -1405,15 +1409,17 @@ function toComment(msg) {
 
 // src/tasks/utils/dependency.ts
 var DependencyIdentifier = class {
-  constructor(name, alias) {
+  constructor(name, alias, type = false) {
     this.name = name;
     this.alias = alias;
+    this.type = type;
   }
   toCode() {
+    const $type = this.type ? "type " : "";
     if (this.alias) {
-      return `${this.name} as ${this.alias}`;
+      return `${$type}${this.name} as ${this.alias}`;
     }
-    return this.name;
+    return `${$type}${this.name}`;
   }
 };
 var Dependency = class {
@@ -1424,6 +1430,11 @@ var Dependency = class {
   constructor(source, identifiers, belongTo, options) {
     this.source = source;
     this.identifiers = identifiers.map((i) => typeof i === "string" ? new DependencyIdentifier(i) : i);
+    if (options?.type) {
+      for (const identifier of this.identifiers) {
+        identifier.type = true;
+      }
+    }
     this.export = !!options?.export;
     this.belongTo = belongTo;
   }
@@ -1595,7 +1606,10 @@ async function compileSchemaDefinition(options) {
     const artifact = new Artifact({
       id: filepath,
       filepath,
-      content: ""
+      content: [
+        "/* @anchor:file:start */",
+        "/* @anchor:file:end */"
+      ].join("\n")
     });
     for (const schemaDefinition of schemaDefinitions2 || []) {
       const dependentArtifact = artifacts.find(isArtifactCompiledBy(schemaDefinition));
@@ -1611,7 +1625,7 @@ async function compileSchemaDefinition(options) {
 }
 
 // src/tasks/compile/utils/compile-operation-definition.ts
-var R17 = __toESM(require("ramda"), 1);
+var R18 = __toESM(require("ramda"), 1);
 
 // src/renderer/operation-type/index.ts
 var R15 = __toESM(require("ramda"), 1);
@@ -1708,7 +1722,7 @@ async function operationTypeRenderer(operationDefinition, alias = R15.identity)
     $parameterBodies || void 0,
     $requestParameters,
     "",
-    `export interface Operation<STATUS extends keyof ${typeName("ResponseBodies")}, CONTENT_TYPE extends keyof ${typeName("ParameterBodies")}> extends KeqOperation {`,
+    `export interface Operation<STATUS extends keyof ${typeName("ResponseBodies")}, CONTENT_TYPE extends ${$parameterBodies ? `keyof ${typeName("ParameterBodies")}` : "string"} > extends KeqOperation {`,
     `  requestParams: ${typeName("RouteParameters")} & { [key: string]: KeqPathParameterInit }`,
     `  requestQuery: ${typeName("RequestQuery")} & { [key: string]: KeqQueryInit }`,
     `  requestHeaders: ${typeName("RequestHeaders")} & { [key: string]: string | number }`,
@@ -1737,7 +1751,12 @@ function generateParameters(name, parameters, alias) {
 }
 
 // src/renderer/operation-request/index.ts
+var R17 = __toESM(require("ramda"), 1);
+
+// src/renderer/operation-request/request-body.ts
 var R16 = __toESM(require("ramda"), 1);
+
+// src/renderer/operation-request/error-to-comment.ts
 function errorToComment(err, mediaType) {
   const $err = String(err).split("\n").map(((line) => ` * ${line}`)).join("\n");
   return [
@@ -1747,6 +1766,34 @@ function errorToComment(err, mediaType) {
     " */"
   ].join("\n");
 }
+
+// src/renderer/operation-request/request-body.ts
+function requestBodyFormDataPropertyRenderer(propertyName, propertySchema, mediaType, operationDefinition) {
+  try {
+    const $propertyName = JSON.stringify(propertyName);
+    const schema = JsonSchemaUtils.isRef(propertySchema) ? SwaggerUtils.dereferenceDeep(propertySchema.$ref, operationDefinition.document.swagger) : propertySchema;
+    if (schema.type === "string" && schema.format === "binary" || schema.contentMediaType === "application/octet-stream") {
+      return `if (args && ${$propertyName} in args && args[${$propertyName}]) req.attach(${$propertyName}, args[${$propertyName}])`;
+    } else if (schema.type === "string" || schema.type === "array" && schema.items && schema.items.type === "string") {
+      return `if (args && ${$propertyName} in args && args[${$propertyName}] !== undefined) req.field(${$propertyName}, args[${$propertyName}])`;
+    } else if (schema.type === "number" || schema.type === "integer") {
+      return `if (args && ${$propertyName} in args && args[${$propertyName}] !== undefined) req.field(${$propertyName}, String(args[${$propertyName}]))`;
+    }
+    return `if (args && ${$propertyName} in args && args[${$propertyName}] !== undefined) req.field(${$propertyName}, String(args[${$propertyName}]) /* type is non-string in schema; triggers type coercion here */)`;
+  } catch (err) {
+    return errorToComment(err, mediaType);
+  }
+}
+function requestBodyPropertyRenderer(propertyName, propertySchema, mediaType, operationDefinition) {
+  if (mediaType === "application/json") {
+    const $propertyName = JSON.stringify(propertyName);
+    return `if (args && ${$propertyName} in args) req.send({ ${$propertyName}: args[${$propertyName}] })`;
+  } else if (mediaType === "multipart/form-data") {
+    return requestBodyFormDataPropertyRenderer(propertyName, propertySchema, mediaType, operationDefinition);
+  } else {
+    throw new Error(`Unsupported media type: ${mediaType}`);
+  }
+}
 function requestBodyRenderer(operationDefinition, typeName) {
   const { operation } = operationDefinition;
   const requestBodyContent = operation.requestBody?.content || {};
@@ -1757,22 +1804,10 @@ function requestBodyRenderer(operationDefinition, typeName) {
       if (schema.type !== "object") return;
       const properties = schema.properties || {};
       return Object.entries(properties).map(([propertyName, propertySchema]) => {
-        const $propertyName = JSON.stringify(propertyName);
-        if (mediaType === "application/json") {
-          return `  if (args && ${$propertyName} in args) req.send({ ${$propertyName}: args[${$propertyName}] })`;
-        } else if (mediaType === "multipart/form-data") {
-          try {
-            const schema2 = JsonSchemaUtils.isRef(propertySchema) ? SwaggerUtils.dereferenceDeep(propertySchema.$ref, operationDefinition.document.swagger) : propertySchema;
-            if (schema2.type === "string" && schema2.format === "binary" || schema2.contentMediaType === "application/octet-stream") {
-              return `  if (args && ${$propertyName} in args && args[${$propertyName}]) req.attach(${$propertyName}, args[${$propertyName}])`;
-            }
-            return `  if (args && ${$propertyName} in args && args[${$propertyName}] !== undefined) req.field(${$propertyName}, args[${$propertyName}])`;
-          } catch (err) {
-            return indent(2, errorToComment(err, mediaType));
-          }
-        } else {
-          throw new Error(`Unsupported media type: ${mediaType}`);
-        }
+        return indent(
+          2,
+          requestBodyPropertyRenderer(propertyName, propertySchema, mediaType, operationDefinition)
+        );
       }).join("\n");
     } catch (err) {
       return indent(2, errorToComment(err, mediaType));
@@ -1780,6 +1815,8 @@ function requestBodyRenderer(operationDefinition, typeName) {
   }).filter(R16.isNotNil).join("\n");
   return $requestBody;
 }
+
+// src/renderer/operation-request/index.ts
 function requestHeadersRenderer(operationDefinition, typeName) {
   const { operation } = operationDefinition;
   const $headers = (operation.parameters || []).filter((p) => !JsonSchemaUtils.isRef(p)).filter((p) => p.in === "header").map((p) => `  if (args && ${JSON.stringify(p.name)} in args) req.header(${JSON.stringify(p.name)}, args[${JSON.stringify(p.name)}])`).concat("").join("\n");
@@ -1789,7 +1826,7 @@ function requestQueryRenderer(operationDefinition, qs, typeName) {
   const { operation } = operationDefinition;
   const $query = (operation.parameters || []).filter((p) => !JsonSchemaUtils.isRef(p)).filter((p) => p.in === "query").map((p) => {
     const option = qs(p);
-    const $option = !option || R16.isEmpty(option) ? "" : `, ${JSON.stringify(option)}`;
+    const $option = !option || R17.isEmpty(option) ? "" : `, ${JSON.stringify(option)}`;
     return `  if (args && ${JSON.stringify(p.name)} in args) req.query(${JSON.stringify(p.name)}, args[${JSON.stringify(p.name)}]${$option})`;
   }).concat("").join("\n");
   return $query;
@@ -1821,9 +1858,9 @@ function operationDeclarationRenderer(operationDefinition, typeName) {
   const { operationId } = operationDefinition;
   const mediaTypes = getRequestMediaTypes(operationDefinition);
   if (mediaTypes.length === 0) {
-    return `function ${operationId}<STATUS extends keyof ${typeName("ResponseBodies")}>(args?: ${typeName("RequestParameters")}): Keq<Operation<STATUS, never>>`;
+    return `function ${operationId}<STATUS extends keyof ${typeName("ResponseBodies")}, CONTENT_TYPE extends never = never>(args?: ${typeName("RequestParameters")}): Keq<Operation<STATUS, CONTENT_TYPE>>`;
   } else if (mediaTypes.length === 1) {
-    return `function ${operationId}<STATUS extends keyof ${typeName("ResponseBodies")}>(args?: ${typeName("RequestParameters")}): Keq<Operation<STATUS, ${JSON.stringify(mediaTypes[0])}>>`;
+    return `function ${operationId}<STATUS extends keyof ${typeName("ResponseBodies")}, CONTENT_TYPE extends ${JSON.stringify(mediaTypes[0])} = ${JSON.stringify(mediaTypes[0])}>(args?: ${typeName("RequestParameters")}): Keq<Operation<STATUS, CONTENT_TYPE>>`;
   } else if (mediaTypes.length > 1) {
     return `function ${operationId}<STATUS extends keyof ${typeName("ResponseBodies")}, CONTENT_TYPE extends ${typeName("RequestParameters")}["content-type"]>(args?: Extract<${typeName("RequestParameters")}, { "content-type": CONTENT_TYPE }>): Keq<Operation<STATUS, CONTENT_TYPE>>`;
   }
@@ -1835,6 +1872,7 @@ async function operationRequestRenderer(operationDefinition, options) {
   if (!operation.responses) return "";
   const typeName = typeNameFactory(operationDefinition);
   const moduleName = operationDefinition.module.name;
+  const $method = method.toLowerCase();
   const $queryParameters = requestQueryRenderer(operationDefinition, qs, typeName);
   const $headerParameters = requestHeadersRenderer(operationDefinition, typeName);
   const $pathParameters = requestPathParametersRenderer(operationDefinition, typeName);
@@ -1850,7 +1888,7 @@ async function operationRequestRenderer(operationDefinition, options) {
     "",
     "/* @anchor:operation-declaration */",
     `export ${$operationDeclaration} {`,
-    `  const req = request.post<${typeName("ResponseBodies")}[STATUS]>("${pathname}")`,
+    `  const req = request.${$method}<${typeName("ResponseBodies")}[STATUS]>("${pathname}")`,
     "    .option('module', { name: moduleName, pathname, method })",
     "",
     $mediaType || void 0,
@@ -1871,13 +1909,13 @@ async function operationRequestRenderer(operationDefinition, options) {
     "  /* @anchor:body:end */",
     "",
     "  /* @anchor:operation-return */",
-    `  return req as ReturnType<typeof ${operationId}>`,
+    `  return req as ReturnType<typeof ${operationId}<STATUS${$operationDeclaration.includes("CONTENT_TYPE") ? ", CONTENT_TYPE" : ""}>>`,
     "}",
     "",
     `${operationId}.pathname = pathname`,
     `${operationId}.method = method`,
     "/* @anchor:file:end */"
-  ].filter(R16.isNotNil).join("\n");
+  ].filter(R17.isNotNil).join("\n");
 }
 
 // src/tasks/compile/utils/compile-operation-definition.ts
@@ -1974,7 +2012,8 @@ async function compileOperationDefinition(options) {
         "Operation",
         typeName("ResponseBodies"),
         typeName("RequestParameters")
-      ]
+      ],
+      { type: true }
     );
     artifact.addDependence(
       typeArtifact,
@@ -1983,11 +2022,11 @@ async function compileOperationDefinition(options) {
         `${typeName("RequestHeaders")}`,
         `${typeName("RequestBodies")}`
       ],
-      { export: true }
+      { export: true, type: true }
     );
     return await compiler.hooks.afterCompileOperationRequest.promise(artifact, operationDefinition, task);
   }
-  const artifacts = R17.unnest(
+  const artifacts = R18.unnest(
     await Promise.all(
       operationDefinitions.map(async (operationDefinition) => {
         const typeArtifact = await createTypeArtifact(operationDefinition);
@@ -1996,7 +2035,7 @@ async function compileOperationDefinition(options) {
       })
     )
   );
-  const operationDefinitionsGroupByModuleName = R17.groupBy(
+  const operationDefinitionsGroupByModuleName = R18.groupBy(
     (operationDefinition) => operationDefinition.module.name,
     operationDefinitions
   );
@@ -2005,7 +2044,10 @@ async function compileOperationDefinition(options) {
     const artifact = new Artifact({
       id: filepath,
       filepath,
-      content: ""
+      content: [
+        "/* @anchor:file:start */",
+        "/* @anchor:file:end */"
+      ].join("\n")
     });
     for (const operationDefinition of operationDefinitions2 || []) {
       const dependentArtifact = artifacts.find((artifact2) => artifact2.filepath === genOperationRequestFilepath(operationDefinition));
@@ -2023,14 +2065,12 @@ async function compileOperationDefinition(options) {
 // src/renderer/request/index.ts
 async function requestRenderer() {
   return [
-    "import { KeqRequest } from 'keq'",
-    "",
     "/* @anchor:file:start */",
     "",
     "/* @anchor:request-declaration */",
     "export const request = new KeqRequest()",
     "",
-    "/* @anchor:file:end"
+    "/* @anchor:file:end */"
   ].join("\n");
 }
 
@@ -2043,13 +2083,15 @@ function main6(compiler) {
       const rc = context.setup.rc;
       const matcher = context.setup.matcher;
       const documents = context.shaken.documents.filter((document) => !matcher.isModuleIgnored(document.module));
-      const requestArtifact = await compiler.hooks.afterCompileKeqRequest.promise(
-        new Artifact({
-          id: "request",
-          filepath: "request",
-          content: await requestRenderer(),
-          extensionName: ".ts"
-        }),
+      let requestArtifact = new Artifact({
+        id: "request",
+        filepath: "request",
+        content: await requestRenderer(),
+        extensionName: ".ts"
+      });
+      requestArtifact.addDependence("keq", ["KeqRequest"]);
+      requestArtifact = await compiler.hooks.afterCompileKeqRequest.promise(
+        requestArtifact,
         task
       );
       const schemaDefinitions = documents.flatMap((document) => document.schemas);
@@ -2140,7 +2182,7 @@ function createInteractiveTask(options) {
 }
 
 // src/compiler/intercepter/perfect-error-message.ts
-var R18 = __toESM(require("ramda"), 1);
+var R19 = __toESM(require("ramda"), 1);
 function perfectErrorMessage() {
   return {
     register: (tap) => {
@@ -2172,8 +2214,8 @@ function perfectErrorMessage() {
       }
       if (tap.type === "async") {
         tap.fn = (...args) => {
-          const callback = R18.last(args);
-          return fn(...R18.init(args), (err, result) => {
+          const callback = R19.last(args);
+          return fn(...R19.init(args), (err, result) => {
             prefix(err);
             return callback(err, result);
           });
@@ -2264,11 +2306,11 @@ var Compiler = class {
       [
         createSetupTask(this, options),
         createDownloadTask(this, { skipIgnoredModules: !options.interactive }),
-        createValidateTask(this, { enabled: !!options.build }),
+        createValidateTask(this),
         createInteractiveTask({ enabled: !!options.interactive, ...typeof options.interactive === "object" ? options.interactive : { mode: "except" } }),
         createShakingTask(this, { enabled: !!options.build, ...typeof options.build === "object" ? options.build.shaking : void 0 }),
         createCompileTask(this, { enabled: !!options.build }),
-        createPersistTask(this, { enabled: !!options.build })
+        createPersistTask(this)
       ],
       {
         concurrent: false,
@@ -2354,8 +2396,8 @@ program.command("ignore").addArgument(
       }
     });
   } else {
-    if (!options.module && !options.method && !options.pathname) {
-      throw new Error("at least one of '--module', '--method' or '--pathname' must be specified");
+    if (!options.method && !options.pathname) {
+      throw new Error("at least one of '-i --interactive', '--method' or '--pathname' must be specified");
     }
     const moduleNames = options.module || ["*"];
     compiler = new Compiler({