src: convert project from CJS to ESM

This is a substantial change that moves the release-1.x branch from
CommonJS to ESM as discussed in
kubernetes-client#2014. Please
review/test carefully.

This commit does the following things:

- Updates the tsconfig to generate ESM code. The formatter also
  automatically formatted this file.
- Add `"type": "module"` to the `package.json` so that consumers
  treat this as ESM instead of CJS.
- Updates all of the imports to be valid ESM imports. ESM requires
  that file extensions are provided. TypeScript is smart enough to
  handle the `.js` extensions, but TypeScript does not support
  adding the extensions at build time. See
  microsoft/TypeScript#16577 for more
  details about this. The `tsc-esm-fix` utility on npm was used to
  automatically add the extensions. The code generator probably
  needs to add support for this too if it doesn't already.
- Fixup the `jsonpath` imports, as this module does not have a
  default ESM export. This just means adding `* as` to the
  existing imports.
- Remove the `AbortError` export originating from `node-fetch`.
  Apparently this is not actually there.

I was able to successfully import the transpiled code using
`import * as k8s from './dist/index.js';`. This should hopefully
unblock running tests against the transpiled code as well (it
currently does not work because the openid-client and chai
dependencies have both moved to ESM only).

package.json

@@ -2,6 +2,7 @@
     "name": "@kubernetes/client-node",
     "version": "1.0.0-rc7",
     "description": "NodeJS client for kubernetes",
+    "type": "module",
     "repository": {
         "type": "git",
         "url": "https://github.com/kubernetes-client/javascript.git"

src/api.ts

@@ -1 +1 @@
-export * from './gen';
+export * from './gen/index.js';

src/attach.ts

@@ -2,9 +2,9 @@ import WebSocket from 'isomorphic-ws';
 import querystring from 'node:querystring';
 import stream from 'node:stream';
 
-import { KubeConfig } from './config';
-import { isResizable, ResizableStream, TerminalSizeQueue } from './terminal-size-queue';
-import { WebSocketHandler, WebSocketInterface } from './web-socket-handler';
+import { KubeConfig } from './config.js';
+import { isResizable, ResizableStream, TerminalSizeQueue } from './terminal-size-queue.js';
+import { WebSocketHandler, WebSocketInterface } from './web-socket-handler.js';
 
 export class Attach {
     public 'handler': WebSocketInterface;

src/attach_test.ts

@@ -4,10 +4,10 @@ import { ReadableStreamBuffer, WritableStreamBuffer } from 'stream-buffers';
 import { anyFunction, anything, capture, instance, mock, verify, when } from 'ts-mockito';
 
 import { CallAwaiter, matchBuffer, ResizableWriteableStreamBuffer } from '../test';
-import { Attach } from './attach';
-import { KubeConfig } from './config';
-import { TerminalSize } from './terminal-size-queue';
-import { WebSocketHandler, WebSocketInterface } from './web-socket-handler';
+import { Attach } from './attach.js';
+import { KubeConfig } from './config.js';
+import { TerminalSize } from './terminal-size-queue.js';
+import { WebSocketHandler, WebSocketInterface } from './web-socket-handler.js';
 
 describe('Attach', () => {
     describe('basic', () => {

src/auth.ts

@@ -1,6 +1,6 @@
 import https from 'node:https';
 
-import { User } from './config_types';
+import { User } from './config_types.js';
 import WebSocket from 'isomorphic-ws';
 
 export interface Authenticator {

src/azure_auth.ts

@@ -1,9 +1,9 @@
 import proc from 'node:child_process';
 import https from 'node:https';
-import jsonpath from 'jsonpath-plus';
+import * as jsonpath from 'jsonpath-plus';
 
-import { Authenticator } from './auth';
-import { User } from './config_types';
+import { Authenticator } from './auth.js';
+import { User } from './config_types.js';
 
 /* FIXME: maybe we can extend the User and User.authProvider type to have a proper type.
 Currently user.authProvider has `any` type and so we don't have a type for user.authProvider.config.

src/azure_auth_test.ts

@@ -2,10 +2,10 @@ import { use, expect } from 'chai';
 import chaiAsPromised from 'chai-as-promised';
 import { join } from 'node:path';
 
-import { User, Cluster } from './config_types';
-import { AzureAuth } from './azure_auth';
-import { KubeConfig } from './config';
-import { HttpMethod, RequestContext } from '.';
+import { User, Cluster } from './config_types.js';
+import { AzureAuth } from './azure_auth.js';
+import { KubeConfig } from './config.js';
+import { HttpMethod, RequestContext } from './index.js';
 
 use(chaiAsPromised);
 

src/cache.ts

@@ -9,10 +9,10 @@ import {
     ListPromise,
     ObjectCallback,
     UPDATE,
-} from './informer';
-import { KubernetesObject } from './types';
-import { ObjectSerializer } from './serializer';
-import { Watch } from './watch';
+} from './informer.js';
+import { KubernetesObject } from './types.js';
+import { ObjectSerializer } from './serializer.js';
+import { Watch } from './watch.js';
 
 export interface ObjectCache<T> {
     get(name: string, namespace?: string): T | undefined;

src/cache_test.ts

@@ -3,16 +3,23 @@ import chaiAsPromised from 'chai-as-promised';
 
 import mock from 'ts-mockito';
 
-import { V1Namespace, V1NamespaceList, V1ObjectMeta, V1Pod, V1PodList, V1ListMeta } from './api';
-import { deleteObject, ListWatch, deleteItems, CacheMap, cacheMapFromList, addOrUpdateObject } from './cache';
-import { KubeConfig } from './config';
-import { Cluster, Context, User } from './config_types';
-import { ListPromise } from './informer';
+import { V1Namespace, V1NamespaceList, V1ObjectMeta, V1Pod, V1PodList, V1ListMeta } from './api.js';
+import {
+    deleteObject,
+    ListWatch,
+    deleteItems,
+    CacheMap,
+    cacheMapFromList,
+    addOrUpdateObject,
+} from './cache.js';
+import { KubeConfig } from './config.js';
+import { Cluster, Context, User } from './config_types.js';
+import { ListPromise } from './informer.js';
 
 use(chaiAsPromised);
 
 import nock from 'nock';
-import { Watch } from './watch';
+import { Watch } from './watch.js';
 
 const server = 'http://foo.company.com';
 

src/config.ts

@@ -5,9 +5,9 @@ import net from 'node:net';
 import path from 'node:path';
 
 import { Headers, RequestInit } from 'node-fetch';
-import { RequestContext } from './api';
-import { Authenticator } from './auth';
-import { AzureAuth } from './azure_auth';
+import { RequestContext } from './api.js';
+import { Authenticator } from './auth.js';
+import { AzureAuth } from './azure_auth.js';
 import {
     Cluster,
     ConfigOptions,
@@ -19,18 +19,18 @@ import {
     newContexts,
     newUsers,
     User,
-} from './config_types';
-import { ExecAuth } from './exec_auth';
-import { FileAuth } from './file_auth';
-import { GoogleCloudPlatformAuth } from './gcp_auth';
+} from './config_types.js';
+import { ExecAuth } from './exec_auth.js';
+import { FileAuth } from './file_auth.js';
+import { GoogleCloudPlatformAuth } from './gcp_auth.js';
 import {
     AuthMethodsConfiguration,
     Configuration,
     createConfiguration,
     SecurityAuthentication,
     ServerConfiguration,
-} from './gen';
-import { OpenIDConnectAuth } from './oidc_auth';
+} from './gen/index.js';
+import { OpenIDConnectAuth } from './oidc_auth.js';
 import WebSocket from 'isomorphic-ws';
 import child_process from 'node:child_process';
 

src/config_test.ts

@@ -8,12 +8,12 @@ import chaiAsPromised from 'chai-as-promised';
 import mockfs from 'mock-fs';
 
 import { Headers } from 'node-fetch';
-import { HttpMethod } from '.';
+import { HttpMethod } from './index.js';
 import { assertRequestAgentsEqual, assertRequestOptionsEqual } from '../test/match-buffer';
-import { CoreV1Api, RequestContext } from './api';
-import { bufferFromFileOrString, findHomeDir, findObject, KubeConfig, makeAbsolutePath } from './config';
-import { ActionOnInvalid, Cluster, newClusters, newContexts, newUsers, User } from './config_types';
-import { ExecAuth } from './exec_auth';
+import { CoreV1Api, RequestContext } from './api.js';
+import { bufferFromFileOrString, findHomeDir, findObject, KubeConfig, makeAbsolutePath } from './config.js';
+import { ActionOnInvalid, Cluster, newClusters, newContexts, newUsers, User } from './config_types.js';
+import { ExecAuth } from './exec_auth.js';
 
 const kcFileName = 'testdata/kubeconfig.yaml';
 const kc2FileName = 'testdata/kubeconfig-2.yaml';

src/cp.ts

@@ -3,8 +3,8 @@ import { WritableStreamBuffer } from 'stream-buffers';
 import * as tar from 'tar';
 import tmp from 'tmp-promise';
 
-import { KubeConfig } from './config';
-import { Exec } from './exec';
+import { KubeConfig } from './config.js';
+import { Exec } from './exec.js';
 
 export class Cp {
     public execInstance: Exec;

src/cp_test.ts

@@ -3,10 +3,10 @@ import querystring from 'node:querystring';
 import WebSocket from 'isomorphic-ws';
 
 import { CallAwaiter } from '../test';
-import { KubeConfig } from './config';
-import { Exec } from './exec';
-import { Cp } from './cp';
-import { WebSocketHandler, WebSocketInterface } from './web-socket-handler';
+import { KubeConfig } from './config.js';
+import { Exec } from './exec.js';
+import { Cp } from './cp.js';
+import { WebSocketHandler, WebSocketInterface } from './web-socket-handler.js';
 
 describe('Cp', () => {
     describe('cpFromPod', () => {

src/exec.ts

@@ -2,10 +2,10 @@ import WebSocket from 'isomorphic-ws';
 import querystring from 'node:querystring';
 import stream from 'stream';
 
-import { V1Status } from './api';
-import { KubeConfig } from './config';
-import { isResizable, ResizableStream, TerminalSizeQueue } from './terminal-size-queue';
-import { WebSocketHandler, WebSocketInterface } from './web-socket-handler';
+import { V1Status } from './api.js';
+import { KubeConfig } from './config.js';
+import { isResizable, ResizableStream, TerminalSizeQueue } from './terminal-size-queue.js';
+import { WebSocketHandler, WebSocketInterface } from './web-socket-handler.js';
 
 export class Exec {
     public 'handler': WebSocketInterface;

src/exec_auth.ts

@@ -1,8 +1,8 @@
 import { OutgoingHttpHeaders } from 'node:http';
 import https from 'node:https';
 
-import { Authenticator } from './auth';
-import { User } from './config_types';
+import { Authenticator } from './auth.js';
+import { User } from './config_types.js';
 
 import child_process from 'node:child_process';
 

src/exec_auth_test.ts

@@ -5,8 +5,8 @@ use(chaiAsPromised);
 import https from 'node:https';
 import { OutgoingHttpHeaders } from 'node:http';
 
-import { ExecAuth } from './exec_auth';
-import { User } from './config_types';
+import { ExecAuth } from './exec_auth.js';
+import { User } from './config_types.js';
 
 import child_process from 'node:child_process';
 

src/exec_test.ts

@@ -4,11 +4,11 @@ import { ReadableStreamBuffer, WritableStreamBuffer } from 'stream-buffers';
 import { anyFunction, anything, capture, instance, mock, verify, when } from 'ts-mockito';
 
 import { CallAwaiter, matchBuffer, ResizableWriteableStreamBuffer } from '../test';
-import { V1Status } from './api';
-import { KubeConfig } from './config';
-import { Exec } from './exec';
-import { TerminalSize } from './terminal-size-queue';
-import { WebSocketHandler, WebSocketInterface } from './web-socket-handler';
+import { V1Status } from './api.js';
+import { KubeConfig } from './config.js';
+import { Exec } from './exec.js';
+import { TerminalSize } from './terminal-size-queue.js';
+import { WebSocketHandler, WebSocketInterface } from './web-socket-handler.js';
 
 describe('Exec', () => {
     describe('basic', () => {

src/file_auth.ts

@@ -1,8 +1,8 @@
 import fs from 'node:fs';
 import https from 'node:https';
 
-import { Authenticator } from './auth';
-import { User } from './config_types';
+import { Authenticator } from './auth.js';
+import { User } from './config_types.js';
 
 export class FileAuth implements Authenticator {
     private token: string | null = null;

src/file_auth_test.ts

@@ -3,8 +3,8 @@ import { OutgoingHttpHeaders } from 'node:http';
 import https from 'node:https';
 import mockfs from 'mock-fs';
 
-import { User } from './config_types';
-import { FileAuth } from './file_auth';
+import { User } from './config_types.js';
+import { FileAuth } from './file_auth.js';
 
 describe('FileAuth', () => {
     it('should refresh when null', async () => {

src/gcp_auth.ts

@@ -1,9 +1,9 @@
 import proc from 'node:child_process';
 import https from 'node:https';
-import jsonpath from 'jsonpath-plus';
+import * as jsonpath from 'jsonpath-plus';
 
-import { Authenticator } from './auth';
-import { User } from './config_types';
+import { Authenticator } from './auth.js';
+import { User } from './config_types.js';
 
 /* FIXME: maybe we can extend the User and User.authProvider type to have a proper type.
 Currently user.authProvider has `any` type and so we don't have a type for user.authProvider.config.

src/gcp_auth_test.ts

@@ -1,10 +1,10 @@
 import { expect } from 'chai';
 import { join } from 'node:path';
 
-import { User, Cluster } from './config_types';
-import { GoogleCloudPlatformAuth } from './gcp_auth';
-import { KubeConfig } from './config';
-import { HttpMethod, RequestContext } from './gen';
+import { User, Cluster } from './config_types.js';
+import { GoogleCloudPlatformAuth } from './gcp_auth.js';
+import { KubeConfig } from './config.js';
+import { HttpMethod, RequestContext } from './gen/index.js';
 import { Agent } from 'node:https';
 
 describe('GoogleCloudPlatformAuth', () => {

src/gen/apis/AdmissionregistrationApi.ts

@@ -1,16 +1,16 @@
 // TODO: better import syntax?
-import {BaseAPIRequestFactory, RequiredError, COLLECTION_FORMATS} from './baseapi';
-import {Configuration} from '../configuration';
-import {RequestContext, HttpMethod, ResponseContext, HttpFile, HttpInfo} from '../http/http';
+import {BaseAPIRequestFactory, RequiredError, COLLECTION_FORMATS} from './baseapi.js';
+import {Configuration} from '../configuration.js';
+import {RequestContext, HttpMethod, ResponseContext, HttpFile, HttpInfo} from '../http/http.js';
 import  FormData from "form-data";
 import { URLSearchParams } from 'url';
-import {ObjectSerializer} from '../models/ObjectSerializer';
-import {ApiException} from './exception';
-import {canConsumeForm, isCodeInRange} from '../util';
-import {SecurityAuthentication} from '../auth/auth';
+import {ObjectSerializer} from '../models/ObjectSerializer.js';
+import {ApiException} from './exception.js';
+import {canConsumeForm, isCodeInRange} from '../util.js';
+import {SecurityAuthentication} from '../auth/auth.js';
 
 
-import { V1APIGroup } from '../models/V1APIGroup';
+import { V1APIGroup } from '../models/V1APIGroup.js';
 
 /**
  * no description

src/gen/apis/AdmissionregistrationV1Api.ts

@@ -1,26 +1,26 @@
 // TODO: better import syntax?
-import {BaseAPIRequestFactory, RequiredError, COLLECTION_FORMATS} from './baseapi';
-import {Configuration} from '../configuration';
-import {RequestContext, HttpMethod, ResponseContext, HttpFile, HttpInfo} from '../http/http';
+import {BaseAPIRequestFactory, RequiredError, COLLECTION_FORMATS} from './baseapi.js';
+import {Configuration} from '../configuration.js';
+import {RequestContext, HttpMethod, ResponseContext, HttpFile, HttpInfo} from '../http/http.js';
 import  FormData from "form-data";
 import { URLSearchParams } from 'url';
-import {ObjectSerializer} from '../models/ObjectSerializer';
-import {ApiException} from './exception';
-import {canConsumeForm, isCodeInRange} from '../util';
-import {SecurityAuthentication} from '../auth/auth';
-
-
-import { V1APIResourceList } from '../models/V1APIResourceList';
-import { V1DeleteOptions } from '../models/V1DeleteOptions';
-import { V1MutatingWebhookConfiguration } from '../models/V1MutatingWebhookConfiguration';
-import { V1MutatingWebhookConfigurationList } from '../models/V1MutatingWebhookConfigurationList';
-import { V1Status } from '../models/V1Status';
-import { V1ValidatingAdmissionPolicy } from '../models/V1ValidatingAdmissionPolicy';
-import { V1ValidatingAdmissionPolicyBinding } from '../models/V1ValidatingAdmissionPolicyBinding';
-import { V1ValidatingAdmissionPolicyBindingList } from '../models/V1ValidatingAdmissionPolicyBindingList';
-import { V1ValidatingAdmissionPolicyList } from '../models/V1ValidatingAdmissionPolicyList';
-import { V1ValidatingWebhookConfiguration } from '../models/V1ValidatingWebhookConfiguration';
-import { V1ValidatingWebhookConfigurationList } from '../models/V1ValidatingWebhookConfigurationList';
+import {ObjectSerializer} from '../models/ObjectSerializer.js';
+import {ApiException} from './exception.js';
+import {canConsumeForm, isCodeInRange} from '../util.js';
+import {SecurityAuthentication} from '../auth/auth.js';
+
+
+import { V1APIResourceList } from '../models/V1APIResourceList.js';
+import { V1DeleteOptions } from '../models/V1DeleteOptions.js';
+import { V1MutatingWebhookConfiguration } from '../models/V1MutatingWebhookConfiguration.js';
+import { V1MutatingWebhookConfigurationList } from '../models/V1MutatingWebhookConfigurationList.js';
+import { V1Status } from '../models/V1Status.js';
+import { V1ValidatingAdmissionPolicy } from '../models/V1ValidatingAdmissionPolicy.js';
+import { V1ValidatingAdmissionPolicyBinding } from '../models/V1ValidatingAdmissionPolicyBinding.js';
+import { V1ValidatingAdmissionPolicyBindingList } from '../models/V1ValidatingAdmissionPolicyBindingList.js';
+import { V1ValidatingAdmissionPolicyList } from '../models/V1ValidatingAdmissionPolicyList.js';
+import { V1ValidatingWebhookConfiguration } from '../models/V1ValidatingWebhookConfiguration.js';
+import { V1ValidatingWebhookConfigurationList } from '../models/V1ValidatingWebhookConfigurationList.js';
 
 /**
  * no description