@aoxborrow/registrar-client 0.2.0 → 0.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aoxborrow/registrar-client",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "One consistent, fully-typed TypeScript interface to many domain registrar APIs (GoDaddy, Cloudflare, Namecheap, Spaceship, Dynadot, Porkbun, and more) behind a single pluggable multi-provider client",
   "author": {
     "name": "Aaron Oxborrow",
@@ -11,6 +11,7 @@
     "access": "public"
   },
   "homepage": "https://github.com/aoxborrow/registrar-client",
+  "funding": "https://github.com/sponsors/aoxborrow",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/aoxborrow/registrar-client.git"

package/src/client.ts

@@ -50,8 +50,10 @@ export class RegistrarClient {
           const detail = await this.provider.getDomain(name, opts);
           let merged = { ...domains[index], ...detail };
           // Some providers expose nameservers only via a dedicated endpoint, not
-          // on the domain detail — fetch them separately when still empty.
-          if (merged.nameservers.length === 0) {
+          // on the domain detail — fetch them separately when still empty. Skip
+          // providers whose getNameservers just re-reads getDomain: it would
+          // repeat the same call and return the same empty result.
+          if (merged.nameservers.length === 0 && this.provider.requiresNameserversFetch) {
             try {
               const nameservers = normalizeNameservers(
                 await this.provider.getNameservers(name, opts)

package/src/registrar.ts

@@ -75,6 +75,12 @@ export abstract class BaseRegistrar implements Registrar {
     return this.features.includes(feature);
   }
 
+  // See Registrar.requiresNameserversFetch. Defaults true, so a provider
+  // whose getNameservers hits a real endpoint keeps the enrichment fallback
+  // without opting in; the providers whose getNameservers just re-reads getDomain
+  // override this to false to drop the redundant call.
+  readonly requiresNameserversFetch: boolean = true;
+
   constructor(
     credentials: RegistrarCredentials,
     httpConfig: Omit<HttpClientConfig, 'options'>,

package/src/registrars/dynadot.ts

@@ -321,6 +321,10 @@ export class DynadotRegistrar extends BaseRegistrar {
     return data.domain_info;
   }
 
+  // getNameservers re-reads getDomain (domain_info carries NS inline; no separate
+  // endpoint), so listing enrichment gains nothing from the fallback — skip it.
+  override readonly requiresNameserversFetch = false;
+
   override async getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]> {
     // domain_info carries the nameservers inline (empty for Dynadot-DNS-hosted
     // domains, which have no external NS)

package/src/registrars/godaddy.ts

@@ -421,6 +421,10 @@ export class GoDaddyRegistrar extends BaseRegistrar {
     return this.toDomain(d);
   }
 
+  // getNameservers re-reads getDomain (GoDaddy has no dedicated NS endpoint), so
+  // listing enrichment gains nothing from the fallback — skip it.
+  override readonly requiresNameserversFetch = false;
+
   override async getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]> {
     const domain = await this.getDomain(domainName, opts);
     return domain.nameservers;

package/src/registrars/namesilo.ts

@@ -286,6 +286,10 @@ export class NameSiloRegistrar extends BaseRegistrar {
     });
   }
 
+  // getNameservers re-reads getDomain (getDomainInfo carries NS; no separate
+  // endpoint), so listing enrichment gains nothing from the fallback — skip it.
+  override readonly requiresNameserversFetch = false;
+
   override async getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]> {
     const domain = await this.getDomain(domainName, opts);
     return domain.nameservers;

package/src/registrars/spaceship.ts

@@ -204,6 +204,10 @@ export class SpaceshipRegistrar extends BaseRegistrar {
     return this.toDomain(await this.getRaw(domainName, opts));
   }
 
+  // getNameservers reads the same getRaw payload as getDomain (no dedicated NS
+  // endpoint), so listing enrichment gains nothing from the fallback — skip it.
+  override readonly requiresNameserversFetch = false;
+
   override async getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]> {
     const d = await this.getRaw(domainName, opts);
     return d.nameservers?.hosts ?? [];

package/src/types.ts

@@ -307,6 +307,13 @@ export interface Registrar {
   // the capabilities this provider supports (core contract + its extended features)
   readonly features: readonly RegistrarFeature[];
 
+  // Whether reading a domain's nameservers needs a getNameservers call beyond
+  // getDomain. True when getNameservers hits its own endpoint that can surface
+  // nameservers getDomain omits; false when it just re-reads getDomain, so
+  // listing enrichment skips the redundant fetch. See BaseRegistrar for the
+  // default.
+  readonly requiresNameserversFetch: boolean;
+
   // whether this provider supports a given capability
   supports(feature: RegistrarFeature): boolean;