@aoxborrow/registrar-client 0.2.0 → 0.3.0

package/dist/index.d.cts

@@ -181,6 +181,7 @@ interface Registrar {
     readonly name: string;
     readonly environment: RegistrarEnvironment;
     readonly features: readonly RegistrarFeature[];
+    readonly requiresNameserversFetch: boolean;
     supports(feature: RegistrarFeature): boolean;
     testConnection(opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
@@ -361,6 +362,7 @@ declare abstract class BaseRegistrar implements Registrar {
     static get features(): readonly RegistrarFeature[];
     get features(): readonly RegistrarFeature[];
     supports(feature: RegistrarFeature): boolean;
+    readonly requiresNameserversFetch: boolean;
     constructor(credentials: RegistrarCredentials, httpConfig: Omit<HttpClientConfig, 'options'>, options?: RegistrarOptions);
     testConnection(_opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(_opts?: ListDomainsOptions): Promise<Domain[]>;
@@ -450,6 +452,7 @@ declare class DynadotRegistrar extends BaseRegistrar {
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
     private domainInfo;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     getContacts(domainName: string, opts?: RequestOptions): Promise<ContactSet>;
     getDnsRecords(domainName: string, opts?: RequestOptions): Promise<DnsRecord[]>;
@@ -532,6 +535,7 @@ declare class GoDaddyRegistrar extends BaseRegistrar {
     private listDomainsV3;
     private listDomainsV1;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     checkAvailability(domainNames: string[], opts?: RequestOptions): Promise<DomainAvailability[]>;
     getPricing(tldOrDomain: string, opts?: RequestOptions): Promise<TldPricing>;
@@ -647,6 +651,7 @@ declare class NameSiloRegistrar extends BaseRegistrar {
     testConnection(opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     getContacts(domainName: string, opts?: RequestOptions): Promise<ContactSet>;
     getDnsRecords(domainName: string, opts?: RequestOptions): Promise<DnsRecord[]>;
@@ -716,6 +721,7 @@ declare class SpaceshipRegistrar extends BaseRegistrar {
     testConnection(opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     getAuthCode(domainName: string, opts?: RequestOptions): Promise<string>;
     checkAvailability(domainNames: string[], opts?: RequestOptions): Promise<DomainAvailability[]>;

package/dist/index.d.ts

@@ -181,6 +181,7 @@ interface Registrar {
     readonly name: string;
     readonly environment: RegistrarEnvironment;
     readonly features: readonly RegistrarFeature[];
+    readonly requiresNameserversFetch: boolean;
     supports(feature: RegistrarFeature): boolean;
     testConnection(opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
@@ -361,6 +362,7 @@ declare abstract class BaseRegistrar implements Registrar {
     static get features(): readonly RegistrarFeature[];
     get features(): readonly RegistrarFeature[];
     supports(feature: RegistrarFeature): boolean;
+    readonly requiresNameserversFetch: boolean;
     constructor(credentials: RegistrarCredentials, httpConfig: Omit<HttpClientConfig, 'options'>, options?: RegistrarOptions);
     testConnection(_opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(_opts?: ListDomainsOptions): Promise<Domain[]>;
@@ -450,6 +452,7 @@ declare class DynadotRegistrar extends BaseRegistrar {
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
     private domainInfo;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     getContacts(domainName: string, opts?: RequestOptions): Promise<ContactSet>;
     getDnsRecords(domainName: string, opts?: RequestOptions): Promise<DnsRecord[]>;
@@ -532,6 +535,7 @@ declare class GoDaddyRegistrar extends BaseRegistrar {
     private listDomainsV3;
     private listDomainsV1;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     checkAvailability(domainNames: string[], opts?: RequestOptions): Promise<DomainAvailability[]>;
     getPricing(tldOrDomain: string, opts?: RequestOptions): Promise<TldPricing>;
@@ -647,6 +651,7 @@ declare class NameSiloRegistrar extends BaseRegistrar {
     testConnection(opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     getContacts(domainName: string, opts?: RequestOptions): Promise<ContactSet>;
     getDnsRecords(domainName: string, opts?: RequestOptions): Promise<DnsRecord[]>;
@@ -716,6 +721,7 @@ declare class SpaceshipRegistrar extends BaseRegistrar {
     testConnection(opts?: RequestOptions): Promise<ConnectionResult>;
     listDomains(opts?: ListDomainsOptions): Promise<Domain[]>;
     getDomain(domainName: string, opts?: RequestOptions): Promise<Domain>;
+    readonly requiresNameserversFetch = false;
     getNameservers(domainName: string, opts?: RequestOptions): Promise<string[]>;
     getAuthCode(domainName: string, opts?: RequestOptions): Promise<string>;
     checkAvailability(domainNames: string[], opts?: RequestOptions): Promise<DomainAvailability[]>;

package/dist/index.js

@@ -256,7 +256,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)
@@ -644,6 +644,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;
@@ -1714,6 +1719,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;
@@ -3157,6 +3165,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;
@@ -5058,6 +5069,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;
@@ -6299,6 +6313,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 ?? [];