fix: make types compile under TypeScript 7.0

packages/client/src/client.d.ts

@@ -51,8 +51,23 @@ declare class Client {
   request(data: ClientRequest, cb?: (err: ResponseError, response: [ClientResponse, any]) => void): Promise<[ClientResponse, any]>;
 }
 
-declare const client: Client;
-// @ts-ignore
-export = client
+/**
+ * The module exports the singleton, with the class attached as
+ * `module.exports.Client` (see index.js), so both meanings of `Client` —
+ * the value and the instance type — stay importable by name.
+ */
+declare namespace client {
+  export {Client};
 
-export {Client};
+  export function setApiKey(apiKey: string): void;
+  export function setTwilioEmailAuth(username: string, password: string): void;
+  export function setImpersonateSubuser(subuser: string): void;
+  export function setDefaultHeader(key: string | { [s: string]: string }, value ?: string): Client;
+  export function setDefaultRequest<K extends keyof ClientRequest>(key: K | ClientRequest, value ?: ClientRequest[K]): Client;
+  export function setDataResidency(region: string): Client;
+  export function createHeaders(data: { [key: string]: string }): { [key: string]: string };
+  export function createRequest(data: ClientRequest): ClientRequest;
+  export function request(data: ClientRequest, cb?: (err: ResponseError, response: [ClientResponse, any]) => void): Promise<[ClientResponse, any]>;
+}
+
+export = client;

packages/mail/src/mail.d.ts

@@ -40,11 +40,23 @@ declare class MailService {
   sendMultiple(data: MailDataRequired, cb?: (error: Error | ResponseError, result: [ClientResponse, {}]) => void): Promise<[ClientResponse, {}]>;
 }
 
-declare const mail: MailService;
-// @ts-ignore
-export = mail;
+/**
+ * The module exports the singleton, with the class attached as
+ * `module.exports.MailService` (see index.js).
+ */
+declare namespace mail {
+  export {MailService};
+  export {MailDataRequired};
+  export {ClientResponse};
+  export {ResponseError};
+
+  export function setApiKey(apiKey: string): void;
+  export function setClient(client: Client): void;
+  export function setTwilioEmailAuth(username: string, password: string): void;
+  export function setTimeout(timeout: number): void;
+  export function setSubstitutionWrappers(left: string, right: string): void;
+  export function send(data: MailDataRequired | MailDataRequired[], isMultiple?: boolean, cb?: (err: Error | ResponseError, result: [ClientResponse, {}]) => void): Promise<[ClientResponse, {}]>;
+  export function sendMultiple(data: MailDataRequired, cb?: (error: Error | ResponseError, result: [ClientResponse, {}]) => void): Promise<[ClientResponse, {}]>;
+}
 
-export {MailService};
-export {MailDataRequired};
-export {ClientResponse};
-export {ResponseError};
+export = mail;

test/typescript/client.ts

@@ -31,3 +31,7 @@ Client.request({
 }).then(res => {
   res[0].statusCode;
 });
+
+// Test Client class attached to the module
+const instance: Client.Client = new Client.Client();
+instance.setApiKey("MY_SENDGRID_API_KEY");

test/typescript/mail.ts

@@ -1,9 +1,19 @@
 import { Client } from "@sendgrid/client";
+import { MailDataRequired, ClientResponse, ResponseError } from "@sendgrid/mail";
 import sgMail = require("@sendgrid/mail");
 
 // Test setClient() method
 sgMail.setClient(new Client());
 
+// Test Client used as a type
+const client: Client = new Client();
+sgMail.setClient(client);
+
+// Test types re-exported from the package root
+declare const data: MailDataRequired;
+declare const onSent: (err: Error | ResponseError, result: [ClientResponse, {}]) => void;
+sgMail.send(data, false, onSent);
+
 // Test setApiKey() method
 sgMail.setApiKey("MY_SENDGRID_API_KEY");
 

tsconfig.json

@@ -3,9 +3,9 @@
         "noEmit": true,
         "module": "commonjs",
         "target": "es6",
-        "baseUrl": ".",
+        "types": ["node"],
         "paths": {
-            "@sendgrid/*": ["packages/*"]
+            "@sendgrid/*": ["./packages/*"]
         }
     },
     "include": [