lib: fix internalBinding type definitions

Author: anonrig

lib/internal/bootstrap/realm.js

@@ -170,8 +170,8 @@ const experimentalModuleList = new SafeSet();
   };
 }
 
-// Set up internalBinding() in the closure.
 /**
+ * Set up internalBinding() in the closure.
  * @type {InternalBinding}
  */
 let internalBinding;
@@ -439,3 +439,4 @@ setInternalLoaders(internalBinding, requireBuiltin);
 
 // Setup per-realm bindings.
 setupPrepareStackTrace();
+setInternalLoaders(internalBinding, requireBuiltin);

tsconfig.json

@@ -2,22 +2,8 @@
   "include": ["lib", "doc"],
   "exclude": ["src", "tools", "out"],
   "files": [
-    "./typings/internalBinding/async_wrap.d.ts",
-    "./typings/internalBinding/blob.d.ts",
-    "./typings/internalBinding/config.d.ts",
-    "./typings/internalBinding/constants.d.ts",
-    "./typings/internalBinding/fs.d.ts",
-    "./typings/internalBinding/http_parser.d.ts",
-    "./typings/internalBinding/messaging.d.ts",
-    "./typings/internalBinding/options.d.ts",
-    "./typings/internalBinding/os.d.ts",
-    "./typings/internalBinding/serdes.d.ts",
-    "./typings/internalBinding/symbols.d.ts",
-    "./typings/internalBinding/timers.d.ts",
-    "./typings/internalBinding/types.d.ts",
-    "./typings/internalBinding/url.d.ts",
-    "./typings/internalBinding/util.d.ts",
-    "./typings/internalBinding/worker.d.ts",
+    "./typings/internalBinding/*.d.ts",
+    "./typings/bindings.d.ts",
     "./typings/globals.d.ts",
     "./typings/primordials.d.ts"
   ],

typings/bindings.d.ts

@@ -0,0 +1,37 @@
+import type { AsyncWrapBinding } from './internalBinding/async_wrap';
+import type { BlobBinding } from "./internalBinding/blob";
+import type { ConfigBinding } from "./internalBinding/config";
+import type { ConstantsBinding } from "./internalBinding/constants";
+import type { FSBinding } from "./internalBinding/fs";
+import type { HttpParserBinding } from "./internalBinding/http_parser";
+import type { MessagingBinding } from "./internalBinding/messaging";
+import type { OptionsBinding } from "./internalBinding/options";
+import type { OSBinding } from "./internalBinding/os";
+import type { SerdesBinding } from "./internalBinding/serdes";
+import type { SymbolsBinding } from "./internalBinding/symbols";
+import type { TimersBinding } from "./internalBinding/timers";
+import type { TypesBinding } from "./internalBinding/types";
+import type { URLBinding } from "./internalBinding/url";
+import type { UtilBindings } from "./internalBinding/util";
+import type { WorkerBinding } from "./internalBinding/worker";
+
+export type InternalBindingMap = {
+  async_wrap: AsyncWrapBinding;
+  blob: BlobBinding;
+  config: ConfigBinding;
+  constants: ConstantsBinding;
+  fs: FSBinding;
+  http_parser: HttpParserBinding;
+  messaging: MessagingBinding;
+  options: OptionsBinding;
+  os: OSBinding;
+  serdes: SerdesBinding;
+  symbols: SymbolsBinding;
+  timers: TimersBinding;
+  types: TypesBinding;
+  url: URLBinding;
+  util: UtilBindings;
+  worker: WorkerBinding;
+}
+
+export type InternalBinding<T extends keyof InternalBindingMap> = (binding: T) => InternalBindingMap[T];

typings/internalBinding/async_wrap.d.ts

@@ -72,7 +72,7 @@ declare namespace InternalAsyncWrapBinding {
   }
 }
 
-declare function InternalBinding(binding: 'async_wrap'): {
+export type AsyncWrapBinding = {
   setupHooks(): {
     init: (
       asyncId: number,
@@ -129,4 +129,4 @@ declare function InternalBinding(binding: 'async_wrap'): {
     kDefaultTriggerAsyncId: 3;
   };
   Providers: InternalAsyncWrapBinding.Providers;
-};
+}

typings/internalBinding/blob.d.ts

@@ -10,7 +10,7 @@ declare namespace InternalBlobBinding {
   }
 }
 
-declare function InternalBinding(binding: 'blob'): {
+export type BlobBinding = {
   createBlob(sources: Array<Uint8Array | InternalBlobBinding.BlobHandle>, length: number): InternalBlobBinding.BlobHandle;
   FixedSizeBlobCopyJob: typeof InternalBlobBinding.FixedSizeBlobCopyJob;
   getDataObject(id: string): [handle: InternalBlobBinding.BlobHandle | undefined, length: number, type: string] | undefined;

typings/internalBinding/config.d.ts

@@ -1,4 +1,4 @@
-declare function InternalBinding(binding: 'config'): {
+export type ConfigBinding = {
   isDebugBuild: boolean;
   hasOpenSSL: boolean;
   fipsMode: boolean;

typings/internalBinding/constants.d.ts

@@ -1,4 +1,4 @@
-declare function InternalBinding(binding: 'constants'): {
+export type ConstantsBinding = {
   os: {
     UV_UDP_REUSEADDR: 4;
     dlopen: {

typings/internalBinding/fs.d.ts

@@ -109,7 +109,7 @@ declare namespace InternalFSBinding {
 
   function internalModuleReadJSON(path: string): [] | [string, boolean];
   function internalModuleStat(path: string): number;
-  
+
   function lchown(path: string, uid: number, gid: number, req: FSReqCallback): void;
   function lchown(path: string, uid: number, gid: number, req: undefined, ctx: FSSyncContext): void;
   function lchown(path: string, uid: number, gid: number, usePromises: typeof kUsePromises): Promise<void>;
@@ -198,7 +198,7 @@ declare namespace InternalFSBinding {
   function symlink(target: StringOrBuffer, path: StringOrBuffer, type: number, req: FSReqCallback): void;
   function symlink(target: StringOrBuffer, path: StringOrBuffer, type: number, req: undefined, ctx: FSSyncContext): void;
   function symlink(target: StringOrBuffer, path: StringOrBuffer, type: number, usePromises: typeof kUsePromises): Promise<void>;
-  
+
   function unlink(path: string, req: FSReqCallback): void;
   function unlink(path: string, req: undefined, ctx: FSSyncContext): void;
   function unlink(path: string, usePromises: typeof kUsePromises): Promise<void>;
@@ -220,7 +220,7 @@ declare namespace InternalFSBinding {
   function writeString(fd: number, value: string, pos: unknown, encoding: unknown, usePromises: typeof kUsePromises): Promise<number>;
 }
 
-declare function InternalBinding(binding: 'fs'): {
+export type FSBinding = {
   FSReqCallback: typeof InternalFSBinding.FSReqCallback;
 
   FileHandle: typeof InternalFSBinding.FileHandle;

typings/internalBinding/http_parser.d.ts

@@ -39,7 +39,7 @@ declare namespace InternalHttpParserBinding {
   }
 }
 
-declare function InternalBinding(binding: 'http_parser'): {
+export type HttpParserBinding = {
   methods: string[];
   HTTPParser: typeof InternalHttpParserBinding.HTTPParser;
 };

typings/internalBinding/messaging.d.ts

@@ -1,3 +1,5 @@
+import type { DOMException } from 'internal/per_context/domexception';
+
 declare namespace InternalMessagingBinding {
   class MessageChannel {
     port1: MessagePort;
@@ -17,8 +19,8 @@ declare namespace InternalMessagingBinding {
 }
 
 
-declare function InternalBinding(binding: 'messaging'): {
-  DOMException: typeof import('internal/per_context/domexception').DOMException;
+export type MessagingBinding = {
+  DOMException: typeof DOMException;
   MessageChannel: typeof InternalMessagingBinding.MessageChannel;
   MessagePort: typeof InternalMessagingBinding.MessagePort;
   JSTransferable: typeof InternalMessagingBinding.JSTransferable;