adds an importExtension option

.circleci/config.yml

@@ -1,7 +1,7 @@
 version: 2.1
 
 orbs:
-  codacy: codacy/base@7.0.0
+  codacy: codacy/base@10.8.0
 
 references:
   release_filter: &release_filter
@@ -10,11 +10,10 @@ references:
         only:
           - master
 
-
 jobs:
   build_package:
     docker:
-      - image: cimg/node:20.8
+      - image: cimg/node:20.1.0
     working_directory: ~/workdir
     steps:
       - attach_workspace:
@@ -27,7 +26,7 @@ jobs:
       - run:
           name: Install dependencies and generate
           command: |
-            npm install
+            PUPPETEER_SKIP_DOWNLOAD=true npm install --unsafe-perm=true
       - run:
           name: Build library
           command: |
@@ -44,7 +43,7 @@ jobs:
 
   publish_packages:
     docker:
-      - image: cimg/node:20.8
+      - image: cimg/node:20.1.0
     working_directory: ~/workdir
     steps:
       - attach_workspace:

bin/index.js

@@ -16,6 +16,7 @@ const params = program
     .option('--name <value>', 'Custom client class name')
     .option('--useOptions', 'Use options instead of arguments')
     .option('--splitParameters', 'User arguments for path parameters and options for query parameters')
+    .option('--importExtension', 'Add .js extension to import statements')
     .option('--useUnionTypes', 'Use union types instead of enums')
     .option('--exportCore <value>', 'Write core files to disk', true)
     .option('--exportServices <value>', 'Write services to disk', true)
@@ -38,6 +39,7 @@ if (OpenAPI) {
         clientName: params.name,
         useOptions: params.useOptions,
         splitParameters: params.splitParameters,
+        importExtension: params.importExtension,
         useUnionTypes: params.useUnionTypes,
         exportCore: JSON.parse(params.exportCore) === true,
         exportServices: JSON.parse(params.exportServices) === true,

package.json

@@ -47,7 +47,8 @@
         "eslint:fix": "eslint . --fix",
         "prepare": "npm run clean && npm run release",
         "generate-spa-codacy": "npm run build && rimraf ./generated && node ./bin/index.js --input ./samples/spec/codacy-api.yaml --output ./generated --useUnionTypes --indent 2 --client fetch --splitParameters",
-        "generate-ide-codacy": "npm run build && rimraf ./generated && node ./bin/index.js --input ./samples/spec/codacy-api.yaml --output ./generated --useUnionTypes --indent 2 --client axios"
+        "generate-ide-codacy": "npm run build && rimraf ./generated && node ./bin/index.js --input ./samples/spec/codacy-api.yaml --output ./generated --useUnionTypes --indent 2 --client axios",
+        "generate-mcp-codacy": "npm run build && rimraf ./generated && node ./bin/index.js --input ./samples/spec/codacy-api.yaml --output ./generated --useUnionTypes --indent 2 --client fetch --importExtension"
     },
     "dependencies": {
         "@apidevtools/json-schema-ref-parser": "^10.1.0",

src/index.ts

@@ -19,6 +19,7 @@ export type Options = {
     clientName?: string;
     useOptions?: boolean;
     splitParameters?: boolean;
+    importExtension?: boolean;
     useUnionTypes?: boolean;
     exportCore?: boolean;
     exportServices?: boolean;
@@ -41,6 +42,7 @@ export type Options = {
  * @param clientName Custom client class name
  * @param useOptions Use options or arguments functions
  * @param splitParameters User arguments for path parameters and options for query parameters
+ * @param importExtension Add .js extension to import statements
  * @param useUnionTypes Use union types instead of enums
  * @param exportCore Generate core client classes
  * @param exportServices Generate services
@@ -59,6 +61,7 @@ export const generate = async ({
     clientName,
     useOptions = false,
     splitParameters = false,
+    importExtension = false,
     useUnionTypes = false,
     exportCore = true,
     exportServices = true,
@@ -90,6 +93,7 @@ export const generate = async ({
                 httpClient,
                 useOptions,
                 splitParameters,
+                importExtension,
                 useUnionTypes,
                 exportCore,
                 exportServices,
@@ -115,6 +119,7 @@ export const generate = async ({
                 httpClient,
                 useOptions,
                 splitParameters,
+                importExtension,
                 useUnionTypes,
                 exportCore,
                 exportServices,

src/templates/core/ApiError.hbs

@@ -1,7 +1,7 @@
 {{>header}}
 
-import type { ApiRequestOptions } from './ApiRequestOptions';
-import type { ApiResult } from './ApiResult';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiResult } from './ApiResult{{#if @root.importExtension}}.js{{/if}}';
 
 export class ApiError extends Error {
 	public readonly url: string;

src/templates/core/BaseHttpRequest.hbs

@@ -4,12 +4,12 @@
 import type { HttpClient } from '@angular/common/http';
 import type { Observable } from 'rxjs';
 
-import type { ApiRequestOptions } from './ApiRequestOptions';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
 import type { OpenAPIConfig } from './OpenAPI';
 {{else}}
-import type { ApiRequestOptions } from './ApiRequestOptions';
-import type { CancelablePromise } from './CancelablePromise';
-import type { OpenAPIConfig } from './OpenAPI';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
+import type { CancelablePromise } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OpenAPIConfig } from './OpenAPI{{#if @root.importExtension}}.js{{/if}}';
 {{/equals}}
 
 export abstract class BaseHttpRequest {

src/templates/core/HttpRequest.hbs

@@ -5,17 +5,17 @@ import { Inject, Injectable } from '@angular/core';
 import { HttpClient } from '@angular/common/http';
 import type { Observable } from 'rxjs';
 
-import type { ApiRequestOptions } from './ApiRequestOptions';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
 import { BaseHttpRequest } from './BaseHttpRequest';
 import type { OpenAPIConfig } from './OpenAPI';
 import { OpenAPI } from './OpenAPI';
 import { request as __request } from './request';
 {{else}}
-import type { ApiRequestOptions } from './ApiRequestOptions';
-import { BaseHttpRequest } from './BaseHttpRequest';
-import type { CancelablePromise } from './CancelablePromise';
-import type { OpenAPIConfig } from './OpenAPI';
-import { request as __request } from './request';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
+import { BaseHttpRequest } from './BaseHttpRequest{{#if @root.importExtension}}.js{{/if}}';
+import type { CancelablePromise } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OpenAPIConfig } from './OpenAPI{{#if @root.importExtension}}.js{{/if}}';
+import { request as __request } from './request{{#if @root.importExtension}}.js{{/if}}';
 {{/equals}}
 
 {{#equals @root.httpClient 'angular'}}

src/templates/core/OpenAPI.hbs

@@ -1,6 +1,6 @@
 {{>header}}
 
-import type { ApiRequestOptions } from './ApiRequestOptions';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
 
 type Resolver<T> = (options: ApiRequestOptions) => Promise<T | undefined>;
 type Headers = Record<string, string>;

src/templates/core/angular/request.hbs

@@ -7,7 +7,7 @@ import { catchError, map, switchMap } from 'rxjs/operators';
 import type { Observable } from 'rxjs';
 
 import { ApiError } from './ApiError';
-import type { ApiRequestOptions } from './ApiRequestOptions';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
 import type { ApiResult } from './ApiResult';
 import type { OpenAPIConfig } from './OpenAPI';
 

src/templates/core/axios/request.hbs

@@ -4,12 +4,12 @@ import axios from 'axios';
 import type { AxiosError, AxiosRequestConfig, AxiosResponse, AxiosInstance } from 'axios';
 import FormData from 'form-data';
 
-import { ApiError } from './ApiError';
-import type { ApiRequestOptions } from './ApiRequestOptions';
-import type { ApiResult } from './ApiResult';
-import { CancelablePromise } from './CancelablePromise';
-import type { OnCancel } from './CancelablePromise';
-import type { OpenAPIConfig } from './OpenAPI';
+import { ApiError } from './ApiError{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiResult } from './ApiResult{{#if @root.importExtension}}.js{{/if}}';
+import { CancelablePromise } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OnCancel } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OpenAPIConfig } from './OpenAPI{{#if @root.importExtension}}.js{{/if}}';
 
 {{>functions/isDefined}}
 

src/templates/core/fetch/request.hbs

@@ -1,11 +1,11 @@
 {{>header}}
 
-import { ApiError } from './ApiError';
-import type { ApiRequestOptions } from './ApiRequestOptions';
-import type { ApiResult } from './ApiResult';
-import { CancelablePromise } from './CancelablePromise';
-import type { OnCancel } from './CancelablePromise';
-import type { OpenAPIConfig } from './OpenAPI';
+import { ApiError } from './ApiError{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiResult } from './ApiResult{{#if @root.importExtension}}.js{{/if}}';
+import { CancelablePromise } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OnCancel } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OpenAPIConfig } from './OpenAPI{{#if @root.importExtension}}.js{{/if}}';
 
 {{>functions/isDefined}}
 

src/templates/core/node/request.hbs

@@ -5,12 +5,12 @@ import fetch, { Headers } from 'node-fetch';
 import type { RequestInit, Response } from 'node-fetch';
 import type { AbortSignal } from 'node-fetch/externals';
 
-import { ApiError } from './ApiError';
-import type { ApiRequestOptions } from './ApiRequestOptions';
-import type { ApiResult } from './ApiResult';
-import { CancelablePromise } from './CancelablePromise';
-import type { OnCancel } from './CancelablePromise';
-import type { OpenAPIConfig } from './OpenAPI';
+import { ApiError } from './ApiError{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiResult } from './ApiResult{{#if @root.importExtension}}.js{{/if}}';
+import { CancelablePromise } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OnCancel } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OpenAPIConfig } from './OpenAPI{{#if @root.importExtension}}.js{{/if}}';
 
 {{>functions/isDefined}}
 

src/templates/core/xhr/request.hbs

@@ -1,11 +1,11 @@
 {{>header}}
 
-import { ApiError } from './ApiError';
-import type { ApiRequestOptions } from './ApiRequestOptions';
-import type { ApiResult } from './ApiResult';
-import { CancelablePromise } from './CancelablePromise';
-import type { OnCancel } from './CancelablePromise';
-import type { OpenAPIConfig } from './OpenAPI';
+import { ApiError } from './ApiError{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiRequestOptions } from './ApiRequestOptions{{#if @root.importExtension}}.js{{/if}}';
+import type { ApiResult } from './ApiResult{{#if @root.importExtension}}.js{{/if}}';
+import { CancelablePromise } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OnCancel } from './CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+import type { OpenAPIConfig } from './OpenAPI{{#if @root.importExtension}}.js{{/if}}';
 
 {{>functions/isDefined}}
 

src/templates/exportModel.hbs

@@ -3,7 +3,7 @@
 {{#if imports}}
 
 {{#each imports}}
-import type { {{{this}}} } from './{{{this}}}';
+import type { {{{this}}} } from './{{{this}}}{{#if @root.importExtension}}.js{{/if}}';
 {{/each}}
 {{/if}}
 

src/templates/exportService.hbs

@@ -13,22 +13,22 @@ import type { Observable } from 'rxjs';
 {{/equals}}
 {{#if imports}}
 {{#each imports}}
-import type { {{{this}}} } from '../models/{{{this}}}';
+import type { {{{this}}} } from '../models/{{{this}}}{{#if @root.importExtension}}.js{{/if}}';
 {{/each}}
 
 {{/if}}
 {{#notEquals @root.httpClient 'angular'}}
-import type { CancelablePromise } from '../core/CancelablePromise';
+import type { CancelablePromise } from '../core/CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
 {{/notEquals}}
 {{#if @root.exportClient}}
 {{#equals @root.httpClient 'angular'}}
-import { BaseHttpRequest } from '../core/BaseHttpRequest';
+import { BaseHttpRequest } from '../core/BaseHttpRequest{{#if @root.importExtension}}.js{{/if}}';
 {{else}}
-import type { BaseHttpRequest } from '../core/BaseHttpRequest';
+import type { BaseHttpRequest } from '../core/BaseHttpRequest{{#if @root.importExtension}}.js{{/if}}';
 {{/equals}}
 {{else}}
-import { OpenAPI } from '../core/OpenAPI';
-import { request as __request } from '../core/request';
+import { OpenAPI } from '../core/OpenAPI{{#if @root.importExtension}}.js{{/if}}';
+import { request as __request } from '../core/request{{#if @root.importExtension}}.js{{/if}}';
 {{/if}}
 
 {{#equals @root.httpClient 'angular'}}

src/templates/index.hbs

@@ -1,30 +1,30 @@
 {{>header}}
 
 {{#if @root.exportClient}}
-export { {{{clientName}}} } from './{{{clientName}}}';
+export { {{{clientName}}} } from './{{{clientName}}}{{#if @root.importExtension}}.js{{/if}}';
 
 {{/if}}
 {{#if @root.exportCore}}
-export { ApiError as OpenAPIError } from './core/ApiError';
+export { ApiError as OpenAPIError } from './core/ApiError{{#if @root.importExtension}}.js{{/if}}';
 {{#if @root.exportClient}}
-export { BaseHttpRequest } from './core/BaseHttpRequest';
+export { BaseHttpRequest } from './core/BaseHttpRequest{{#if @root.importExtension}}.js{{/if}}';
 {{/if}}
-export { CancelablePromise, CancelError } from './core/CancelablePromise';
-export { OpenAPI } from './core/OpenAPI';
-export type { OpenAPIConfig } from './core/OpenAPI';
+export { CancelablePromise, CancelError } from './core/CancelablePromise{{#if @root.importExtension}}.js{{/if}}';
+export { OpenAPI } from './core/OpenAPI{{#if @root.importExtension}}.js{{/if}}';
+export type { OpenAPIConfig } from './core/OpenAPI{{#if @root.importExtension}}.js{{/if}}';
 {{/if}}
 {{#if @root.exportModels}}
 {{#if models}}
 
 {{#each models}}
 {{#if @root.useUnionTypes}}
-export type { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}';
+export type { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}{{#if @root.importExtension}}.js{{/if}}';
 {{else if enum}}
-export { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}';
+export { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}{{#if @root.importExtension}}.js{{/if}}';
 {{else if enums}}
-export { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}';
+export { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}{{#if @root.importExtension}}.js{{/if}}';
 {{else}}
-export type { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}';
+export type { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfixModels}}}{{/if}} } from './models/{{{name}}}{{#if @root.importExtension}}.js{{/if}}';
 {{/if}}
 {{/each}}
 {{/if}}
@@ -33,15 +33,15 @@ export type { {{{name}}}{{#if @root.postfixModels}} as {{{name}}}{{{@root.postfi
 {{#if models}}
 
 {{#each models}}
-export { ${{{name}}} } from './schemas/${{{name}}}';
+export { ${{{name}}} } from './schemas/${{{name}}}{{#if @root.importExtension}}.js{{/if}}';
 {{/each}}
 {{/if}}
 {{/if}}
 {{#if @root.exportServices}}
 {{#if services}}
 
 {{#each services}}
-export { {{{name}}}{{{@root.postfixServices}}} } from './services/{{{name}}}{{{@root.postfixServices}}}';
+export { {{{name}}}{{{@root.postfixServices}}} } from './services/{{{name}}}{{{@root.postfixServices}}}{{#if @root.importExtension}}.js{{/if}}';
 {{/each}}
 {{/if}}
 {{/if}}

src/utils/writeClient.spec.ts

@@ -44,6 +44,7 @@ describe('writeClient', () => {
             false,
             false,
             false,
+            false,
             true,
             true,
             true,