docs: Fixed npm publish issues with @deephaven/jsapi-nodejs (#2265)

Fixed a couple of issues with the @deephaven/jsapi-nodejs npm package
publish
- Added .js extensions to relative imports to make compatible with
NodeNext consumers (still works for CommonJS as well)
- Fixed issue where types didn't get built
- Fixed package title in README

packages/jsapi-nodejs/README.md

@@ -1,4 +1,4 @@
-# @deephaven/jsapi-components
+# @deephaven/jsapi-nodejs
 
 Deephaven utils for consuming Jsapi from a server from a nodejs app. It can 
 optionally convert the server module format from `ESM` -> `CJS` or `CJS` -> `ESM` 

packages/jsapi-nodejs/src/index.ts

@@ -1,5 +1,5 @@
-export * from './errorUtils';
-export * from './fsUtils';
-export * from './loaderUtils';
-export * from './polyfillWs';
-export * from './serverUtils';
+export * from './errorUtils.js';
+export * from './fsUtils.js';
+export * from './loaderUtils.js';
+export * from './polyfillWs.js';
+export * from './serverUtils.js';

packages/jsapi-nodejs/src/loaderUtils.ts

@@ -2,9 +2,9 @@ import fs from 'node:fs';
 import path from 'node:path';
 import esbuild from 'esbuild';
 
-import { downloadFromURL, urlToDirectoryName } from './serverUtils';
-import { polyfillWs } from './polyfillWs';
-import { ensureDirectoriesExist, getDownloadPaths } from './fsUtils';
+import { downloadFromURL, urlToDirectoryName } from './serverUtils.js';
+import { polyfillWs } from './polyfillWs.js';
+import { ensureDirectoriesExist, getDownloadPaths } from './fsUtils.js';
 
 type NonEmptyArray<T> = [T, ...T[]];
 

packages/jsapi-nodejs/src/serverUtils.ts

@@ -1,6 +1,6 @@
 import * as http from 'node:http';
 import * as https from 'node:https';
-import { hasErrorCode, isAggregateError } from './errorUtils';
+import { hasErrorCode, isAggregateError } from './errorUtils.js';
 
 export const SERVER_STATUS_CHECK_TIMEOUT = 3000;
 

packages/jsapi-nodejs/tsconfig.json

@@ -1,6 +1,13 @@
 {
   "extends": "../../tsconfig.json",
   "compilerOptions": {
+    // NodeNext is the recommended module setting for Node.js v12 or later. This
+    // allows CommonJS and ESM consumers to use this package.
+    "module": "NodeNext",
+    // This is the required setting when `module` is set to `NodeNext`. One side
+    // effect of this setting is that relative imports require explicit `.js`
+    // extensions.
+    "moduleResolution": "NodeNext",
     "rootDir": "src/",
     "outDir": "dist/"
   },

tsconfig.json

@@ -17,7 +17,12 @@
     "excludeDirectories": ["**/node_modules", "**/dist"]
   },
   "references": [
+    // `npm run types` relies on project references to build types. All packages
+    // must be included in the reference tree somewhere. Top-level packages need
+    // to be included here, and their respective `tsconfig.json` files should
+    // reference their dependencies.
     { "path": "./packages/code-studio" },
-    { "path": "./packages/embed-widget" }
+    { "path": "./packages/embed-widget" },
+    { "path": "./packages/jsapi-nodejs" }
   ]
 }