fix: avoid constructing unused service instances

Author: mdevils

src/schema-to-typescript/common/client.ts

@@ -18,6 +18,7 @@ import {
     objectExpression,
     objectProperty,
     program,
+    returnStatement,
     spreadElement,
     Statement,
     stringLiteral,
@@ -166,9 +167,17 @@ export function generateClient({
             });
             clientClassBody.body.push(
                 attachJsDocComment(
-                    classProperty(
+                    classMethod(
+                        'get',
                         identifier(applyEntityNameCase(tag, 'camelCase')),
-                        newExpression(identifier(name), [memberExpression(thisExpression(), identifier('getClient'))])
+                        [],
+                        blockStatement([
+                            returnStatement(
+                                callExpression(memberExpression(thisExpression(), identifier('getServiceInstance')), [
+                                    identifier(name)
+                                ])
+                            )
+                        ])
                     ),
                     renderJsDoc(jsdoc, jsDocRenderConfig)
                 )
@@ -246,14 +255,6 @@ export function generateClient({
                     )
                 )
             ])
-        ),
-        classProperty(
-            identifier('createClientWithServices'),
-            memberExpression(identifier(commonHttpClientImportName), identifier('createClientWithServices')),
-            null,
-            null,
-            false,
-            true
         )
     );
 

src/schema-to-typescript/common/core/common-http-service.ts

@@ -1,6 +1,22 @@
 import * as commonHttpClient from './common-http-client';
 
 export class CommonHttpService {
+    protected serviceInstancesMap: Map<
+        new (getClientInstance: () => commonHttpClient.CommonHttpClient) => CommonHttpService,
+        CommonHttpService
+    > = new Map();
+
+    protected getServiceInstance<T extends CommonHttpService>(
+        serviceClass: new (getClientInstance: () => commonHttpClient.CommonHttpClient) => T
+    ): T {
+        let serviceInstance = this.serviceInstancesMap.get(serviceClass);
+        if (!serviceInstance) {
+            serviceInstance = new serviceClass(this.getClientInstance);
+            this.serviceInstancesMap.set(serviceClass, serviceInstance);
+        }
+        return serviceInstance as T;
+    }
+
     protected getClientInstance: () => commonHttpClient.CommonHttpClient;
     constructor(getClientInstance: () => commonHttpClient.CommonHttpClient) {
         this.getClientInstance = () => {