chore: set esModuleInterop to true in packages (#1516)

Author: trivikr

packages/client-documentation-generator/src/sdk-client-source-update.ts

@@ -3,7 +3,7 @@ import { Converter } from "typedoc/dist/lib/converter";
 import { Component, ConverterComponent } from "typedoc/dist/lib/converter/components";
 import { Context } from "typedoc/dist/lib/converter/context";
 import { SourceFile } from "typedoc/dist/lib/models";
-import * as ts from "typescript";
+import ts from "typescript";
 
 const basePathsToIgnore = ["model"];
 

packages/credential-provider-imds/src/remoteProvider/httpRequest.spec.ts

@@ -1,6 +1,6 @@
 import { ProviderError } from "@aws-sdk/property-provider";
 import { createServer } from "http";
-import * as nock from "nock";
+import nock from "nock";
 
 import { httpRequest } from "./httpRequest";
 

packages/credential-provider-ini/src/index.spec.ts

@@ -33,7 +33,7 @@ jest.mock("fs", () => {
 
   return fs;
 });
-import * as fs from "fs";
+import fs from "fs";
 const { __addMatcher, __clearMatchers } = fs as any;
 
 jest.mock("os", () => {

packages/credential-provider-process/src/index.spec.ts

@@ -32,7 +32,7 @@ jest.mock("fs", () => {
 
   return fs;
 });
-import * as fs from "fs";
+import fs from "fs";
 const { __addFsMatcher, __clearFsMatchers } = fs as any;
 
 jest.mock("os", () => {
@@ -79,7 +79,7 @@ jest.mock("child_process", () => {
 
   return child_process;
 });
-import * as child_process from "child_process";
+import child_process from "child_process";
 const { __addChildProcessMatcher, __clearChildProcessMatchers } = child_process as any;
 
 const DEFAULT_CREDS = {

packages/fetch-http-handler/src/fetch-http-handler.spec.ts

@@ -2,7 +2,7 @@ import { AbortController } from "@aws-sdk/abort-controller";
 import { HttpRequest } from "@aws-sdk/protocol-http";
 
 import { FetchHttpHandler } from "./fetch-http-handler";
-import * as timeouts from "./request-timeout";
+import { requestTimeout } from "./request-timeout";
 
 const mockRequest = jest.fn();
 let mockResponse: any;
@@ -150,7 +150,7 @@ describe.skip(FetchHttpHandler.name, () => {
     const mockFetch = jest.fn().mockResolvedValue(mockResponse);
     (global as any).fetch = mockFetch;
 
-    timeoutSpy = jest.spyOn(timeouts, "requestTimeout");
+    timeoutSpy = jest.spyOn({ requestTimeout }, "requestTimeout");
     const fetchHttpHandler = new FetchHttpHandler({
       requestTimeout: 500,
     });

packages/fetch-http-handler/src/index.spec.ts

@@ -1,7 +1,7 @@
-import * as index from "./index";
+import { FetchHttpHandler } from "./index";
 
 describe("index", () => {
   it("exports FetchHttpHandler", () => {
-    expect(typeof index.FetchHttpHandler).toBe("function");
+    expect(typeof FetchHttpHandler).toBe("function");
   });
 });

packages/node-http-handler/src/index.spec.ts

@@ -1,7 +1,7 @@
-import * as index from "./index";
+import { NodeHttpHandler } from "./index";
 
 describe("index", () => {
   it("exports NodeHttpHandler", () => {
-    expect(typeof index.NodeHttpHandler).toBe("function");
+    expect(typeof NodeHttpHandler).toBe("function");
   });
 });

packages/node-http-handler/src/node-http-handler.spec.ts

@@ -1,9 +1,9 @@
 import { AbortController } from "@aws-sdk/abort-controller";
 import { HttpRequest } from "@aws-sdk/protocol-http";
 import { Server as HttpServer } from "http";
-import * as http from "http";
+import http from "http";
 import { Server as HttpsServer } from "https";
-import * as https from "https";
+import https from "https";
 import { AddressInfo } from "net";
 
 import { NodeHttpHandler } from "./node-http-handler";

packages/node-http-handler/src/node-http-handler.ts

@@ -1,8 +1,8 @@
 import { HttpHandler, HttpRequest, HttpResponse } from "@aws-sdk/protocol-http";
 import { buildQueryString } from "@aws-sdk/querystring-builder";
 import { HttpHandlerOptions } from "@aws-sdk/types";
-import * as http from "http";
-import * as https from "https";
+import { Agent as hAgent, request as hRequest } from "http";
+import { Agent as hsAgent, request as hsRequest, RequestOptions } from "https";
 
 import { getTransformedHeaders } from "./get-transformed-headers";
 import { setConnectionTimeout } from "./set-connection-timeout";
@@ -25,13 +25,13 @@ export interface NodeHttpOptions {
    */
   socketTimeout?: number;
 
-  httpAgent?: http.Agent;
-  httpsAgent?: https.Agent;
+  httpAgent?: hAgent;
+  httpsAgent?: hsAgent;
 }
 
 export class NodeHttpHandler implements HttpHandler {
-  private readonly httpAgent: http.Agent;
-  private readonly httpsAgent: https.Agent;
+  private readonly httpAgent: hAgent;
+  private readonly httpsAgent: hsAgent;
   private readonly connectionTimeout?: number;
   private readonly socketTimeout?: number;
   // Node http handler is hard-coded to http/1.1: https://github.com/nodejs/node/blob/ff5664b83b89c55e4ab5d5f60068fb457f1f5872/lib/_http_server.js#L286
@@ -41,8 +41,8 @@ export class NodeHttpHandler implements HttpHandler {
     this.connectionTimeout = connectionTimeout;
     this.socketTimeout = socketTimeout;
     const keepAlive = true;
-    this.httpAgent = httpAgent || new http.Agent({ keepAlive });
-    this.httpsAgent = httpsAgent || new https.Agent({ keepAlive });
+    this.httpAgent = httpAgent || new hAgent({ keepAlive });
+    this.httpsAgent = httpsAgent || new hsAgent({ keepAlive });
   }
 
   destroy(): void {
@@ -63,7 +63,7 @@ export class NodeHttpHandler implements HttpHandler {
       // determine which http(s) client to use
       const isSSL = request.protocol === "https:";
       const queryString = buildQueryString(request.query || {});
-      const nodeHttpsOptions: https.RequestOptions = {
+      const nodeHttpsOptions: RequestOptions = {
         headers: request.headers,
         host: request.hostname,
         method: request.method,
@@ -73,7 +73,8 @@ export class NodeHttpHandler implements HttpHandler {
       };
 
       // create the http request
-      const req = (isSSL ? https : http).request(nodeHttpsOptions, (res) => {
+      const requestFunc = isSSL ? hsRequest : hRequest;
+      const req = requestFunc(nodeHttpsOptions, (res) => {
         const httpResponse = new HttpResponse({
           statusCode: res.statusCode || -1,
           headers: getTransformedHeaders(res.headers),

packages/shared-ini-file-loader/src/index.spec.ts

@@ -31,7 +31,7 @@ jest.mock("fs", () => {
 
   return fs;
 });
-import * as fs from "fs";
+import fs from "fs";
 const { __addMatcher, __clearMatchers } = fs as any;
 
 jest.mock("os", () => {