@aoxborrow/registrar-client 0.2.0 → 0.3.0

package/README.md

@@ -4,6 +4,7 @@
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Node.js](https://img.shields.io/badge/node-%3E%3D20.0.0-brightgreen.svg)](https://nodejs.org)
 [![TypeScript](https://img.shields.io/badge/TypeScript-5.9-blue.svg)](https://www.typescriptlang.org)
+[![Sponsor](https://img.shields.io/badge/Sponsor-%E2%9D%A4-ea4aaa?logo=githubsponsors&logoColor=white)](https://github.com/sponsors/aoxborrow)
 
 > One consistent, fully-typed TypeScript interface to many domain registrar APIs.
 

package/dist/index.cjs

@@ -258,7 +258,7 @@ var RegistrarClient = class {
         try {
           const detail = await this.provider.getDomain(name, opts);
           let merged = { ...domains[index], ...detail };
-          if (merged.nameservers.length === 0) {
+          if (merged.nameservers.length === 0 && this.provider.requiresNameserversFetch) {
             try {
               const nameservers = normalizeNameservers(
                 await this.provider.getNameservers(name, opts)
@@ -646,6 +646,11 @@ var BaseRegistrar = class {
   supports(feature) {
     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.
+  requiresNameserversFetch = true;
   constructor(credentials, httpConfig, options) {
     const { environment = "production", ...clientOptions } = options ?? {};
     this.credentials = credentials;
@@ -1716,6 +1721,9 @@ ${bodyStr}`;
     if (!data.domain_info) throw new NotFoundError(`Dynadot: domain '${domainName}' not found`);
     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.
+  requiresNameserversFetch = false;
   async getNameservers(domainName, opts) {
     const domain = await this.getDomain(domainName, opts);
     return domain.nameservers;
@@ -3159,6 +3167,9 @@ var GoDaddyRegistrar = 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.
+  requiresNameserversFetch = false;
   async getNameservers(domainName, opts) {
     const domain = await this.getDomain(domainName, opts);
     return domain.nameservers;
@@ -5060,6 +5071,9 @@ var NameSiloRegistrar = class extends BaseRegistrar {
       nameservers: extractNsHosts(r.nameservers)
     });
   }
+  // getNameservers re-reads getDomain (getDomainInfo carries NS; no separate
+  // endpoint), so listing enrichment gains nothing from the fallback — skip it.
+  requiresNameserversFetch = false;
   async getNameservers(domainName, opts) {
     const domain = await this.getDomain(domainName, opts);
     return domain.nameservers;
@@ -6301,6 +6315,9 @@ var SpaceshipRegistrar = class extends BaseRegistrar {
   async getDomain(domainName, opts) {
     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.
+  requiresNameserversFetch = false;
   async getNameservers(domainName, opts) {
     const d = await this.getRaw(domainName, opts);
     return d.nameservers?.hosts ?? [];