refactor: use destructuring arguments in client constructor and respe…

…ct false values (#89)

src/core.ts

@@ -769,12 +769,12 @@ export const ensurePresent = <T>(value: T | null | undefined): T => {
  *
  * Will return an empty string if the environment variable doesn't exist or cannot be accessed.
  */
-export const readEnv = (env: string): string => {
+export const readEnv = (env: string): string | undefined => {
   if (typeof process === 'undefined') {
-    return '';
+    return undefined;
   }
 
-  return process.env[env] ?? '';
+  return process.env[env] ?? undefined;
 };
 
 export const coerceInteger = (value: unknown): number => {
@@ -797,6 +797,27 @@ export const coerceBoolean = (value: unknown): boolean => {
   return Boolean(value);
 };
 
+export const maybeCoerceInteger = (value: unknown): number | undefined => {
+  if (value === undefined) {
+    return undefined;
+  }
+  return coerceInteger(value);
+};
+
+export const maybeCoerceFloat = (value: unknown): number | undefined => {
+  if (value === undefined) {
+    return undefined;
+  }
+  return coerceFloat(value);
+};
+
+export const maybeCoerceBoolean = (value: unknown): boolean | undefined => {
+  if (value === undefined) {
+    return undefined;
+  }
+  return coerceBoolean(value);
+};
+
 // https://stackoverflow.com/a/34491287
 export function isEmptyObj(obj: Object | null | undefined): boolean {
   if (!obj) return true;

src/index.ts

@@ -76,14 +76,18 @@ export class Anthropic extends Core.APIClient {
 
   private _options: ClientOptions;
 
-  constructor(opts: ClientOptions = {}) {
-    const authToken = opts.authToken || Core.readEnv('ANTHROPIC_AUTH_TOKEN') || null;
+  constructor({
+    apiKey = Core.readEnv('ANTHROPIC_API_KEY') ?? null,
+    authToken = Core.readEnv('ANTHROPIC_AUTH_TOKEN') ?? null,
+    ...opts
+  }: ClientOptions = {}) {
+    undefined;
 
     const options: ClientOptions = {
-      apiKey: typeof process === 'undefined' ? '' : process.env['ANTHROPIC_API_KEY'] || '',
+      apiKey,
+      authToken,
       baseURL: `https://api.anthropic.com`,
       ...opts,
-      authToken,
     };
 
     super({
@@ -93,9 +97,9 @@ export class Anthropic extends Core.APIClient {
       maxRetries: options.maxRetries,
       fetch: options.fetch,
     });
-    this.apiKey = options.apiKey || null;
     this._options = options;
 
+    this.apiKey = apiKey;
     this.authToken = authToken;
   }