From 0470afa5114d963141ad540634c02765c621f74f Mon Sep 17 00:00:00 2001
From: Tom Limoncelli <tal@whatexit.org>
Date: Thu, 1 Oct 2026 10:15:42 -0400
Subject: [PATCH] refactor: Reduce unused parameters

---
 commands/provider_syntax_test.go                   | 4 ++--
 providers/akamaiedgedns/akamaiEdgeDnsProvider.go   | 2 +-
 providers/alidns/aliDnsProvider.go                 | 2 +-
 providers/cloudns/cloudnsProvider.go               | 2 +-
 providers/desec/desecProvider.go                   | 2 +-
 providers/digitalocean/digitaloceanProvider.go     | 2 +-
 providers/domainnameshop/domainnameshopProvider.go | 2 +-
 providers/dynu/dynuProvider.go                     | 2 +-
 providers/gcore/gcoreProvider.go                   | 2 +-
 providers/gidinet/gidinetProvider.go               | 2 +-
 providers/huaweicloud/huaweicloudProvider.go       | 2 +-
 providers/infoblox/infobloxProvider.go             | 2 +-
 providers/inwx/inwxProvider.go                     | 2 +-
 providers/joker/jokerProvider.go                   | 2 +-
 providers/linode/linodeProvider.go                 | 2 +-
 providers/luadns/luadnsProvider.go                 | 2 +-
 providers/mythicbeasts/mythicbeastsProvider.go     | 2 +-
 providers/netbird/netbirdProvider.go               | 2 +-
 providers/opensrs/opensrsProvider.go               | 2 +-
 providers/packetframe/packetframeProvider.go       | 2 +-
 providers/tencentdns/tencentdnsProvider.go         | 2 +-
 providers/transip/transipProvider.go               | 2 +-
 providers/vultr/vultrProvider.go                   | 2 +-
 23 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/commands/provider_syntax_test.go b/commands/provider_syntax_test.go
index 5e052fb848..d5d1e7ffb0 100644
--- a/commands/provider_syntax_test.go
+++ b/commands/provider_syntax_test.go
@@ -227,9 +227,9 @@ func TestProviderConversionGuide(t *testing.T) {
 	guide, err := os.ReadFile("../documentation/getting-started/converting-dnsconfig.md")
 	require.NoError(t, err)
 	examples := regexp.MustCompile("(?s)```javascript\\n(.*?)```").FindAllStringSubmatch(string(guide), -1)
-	require.Len(t, examples, 4, "two complete before/after pairs")
+	require.Len(t, examples, 2, "one complete before/after pair")
 	credsPath := filepath.Join(t.TempDir(), "creds.json")
-	require.NoError(t, os.WriteFile(credsPath, []byte(`{"dnsimple":{"TYPE":"DNSIMPLE"}}`), 0o600))
+	require.NoError(t, os.WriteFile(credsPath, []byte(`{"gandi_main":{"TYPE":"GANDI_V5"}}`), 0o600))
 	creds, err := credsfile.LoadProviderConfigs(credsPath)
 	require.NoError(t, err)
 	for i := 0; i < len(examples); i += 2 {
diff --git a/providers/akamaiedgedns/akamaiEdgeDnsProvider.go b/providers/akamaiedgedns/akamaiEdgeDnsProvider.go
index 565948bd43..9429c8b80f 100644
--- a/providers/akamaiedgedns/akamaiEdgeDnsProvider.go
+++ b/providers/akamaiedgedns/akamaiEdgeDnsProvider.go
@@ -119,7 +119,7 @@ func init() {
 }
 
 // DnsServiceProvider.
-func newEdgeDNSDSP(config map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newEdgeDNSDSP(config map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	clientSecret := config["client_secret"]
 	host := config["host"]
 	accessToken := config["access_token"]
diff --git a/providers/alidns/aliDnsProvider.go b/providers/alidns/aliDnsProvider.go
index c83be2732e..ebaca84947 100644
--- a/providers/alidns/aliDnsProvider.go
+++ b/providers/alidns/aliDnsProvider.go
@@ -79,7 +79,7 @@ type domainVersionInfo struct {
 	maxTTL      uint32
 }
 
-func newAliDNSDsp(config map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newAliDNSDsp(config map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	accessKeyID := config["access_key_id"]
 	if accessKeyID == "" {
 		return nil, errors.New("creds.json: access_key_id must not be empty")
diff --git a/providers/cloudns/cloudnsProvider.go b/providers/cloudns/cloudnsProvider.go
index 05eaf44f31..fa331283b9 100644
--- a/providers/cloudns/cloudnsProvider.go
+++ b/providers/cloudns/cloudnsProvider.go
@@ -37,7 +37,7 @@ func newCloudns(m map[string]string) (*cloudnsProvider, error) {
 	return c, nil
 }
 
-func newDsp(conf map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newDsp(conf map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	return newCloudns(conf)
 }
 
diff --git a/providers/desec/desecProvider.go b/providers/desec/desecProvider.go
index c8c8c65c8f..dacdd306a1 100644
--- a/providers/desec/desecProvider.go
+++ b/providers/desec/desecProvider.go
@@ -20,7 +20,7 @@ Info required in `creds.json`:
 */
 
 // NewDeSec creates the provider.
-func NewDeSec(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewDeSec(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	c := &desecProvider{}
 	c.token = strings.TrimSpace(m["auth-token"])
 	if c.token == "" {
diff --git a/providers/digitalocean/digitaloceanProvider.go b/providers/digitalocean/digitaloceanProvider.go
index 13d4f9137a..dc7fd60710 100644
--- a/providers/digitalocean/digitaloceanProvider.go
+++ b/providers/digitalocean/digitaloceanProvider.go
@@ -47,7 +47,7 @@ var defaultNameServerNames = []string{
 const perPageSize = 100
 
 // NewDo creates a DO-specific DNS provider.
-func NewDo(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewDo(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["token"] == "" {
 		return nil, errors.New("no DigitalOcean token provided")
 	}
diff --git a/providers/domainnameshop/domainnameshopProvider.go b/providers/domainnameshop/domainnameshopProvider.go
index 95e04715b1..2d4ed71722 100644
--- a/providers/domainnameshop/domainnameshopProvider.go
+++ b/providers/domainnameshop/domainnameshopProvider.go
@@ -60,7 +60,7 @@ func init() {
 }
 
 // newDomainNameShopProvider creates a Domainnameshop specific DNS provider.
-func newDomainNameShopProvider(conf map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newDomainNameShopProvider(conf map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if conf["token"] == "" {
 		return nil, errors.New("no Domainnameshop token provided")
 	} else if conf["secret"] == "" {
diff --git a/providers/dynu/dynuProvider.go b/providers/dynu/dynuProvider.go
index 6b749cdd72..603e520961 100644
--- a/providers/dynu/dynuProvider.go
+++ b/providers/dynu/dynuProvider.go
@@ -58,7 +58,7 @@ func init() {
 }
 
 // New creates a Dynu provider from credentials.
-func New(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func New(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	apiKey := m["api_key"]
 	if apiKey == "" {
 		return nil, errors.New("missing Dynu API key")
diff --git a/providers/gcore/gcoreProvider.go b/providers/gcore/gcoreProvider.go
index 3632b6eeba..f4c9cbc966 100644
--- a/providers/gcore/gcoreProvider.go
+++ b/providers/gcore/gcoreProvider.go
@@ -32,7 +32,7 @@ func (c *gcoreProvider) SetConversionObserver(observer providers.ConversionObser
 }
 
 // NewGCore creates the provider.
-func NewGCore(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewGCore(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["api-key"] == "" {
 		return nil, errors.New("missing G-Core API key")
 	}
diff --git a/providers/gidinet/gidinetProvider.go b/providers/gidinet/gidinetProvider.go
index 788f742875..5fa509bfa5 100644
--- a/providers/gidinet/gidinetProvider.go
+++ b/providers/gidinet/gidinetProvider.go
@@ -107,7 +107,7 @@ func newRegistrar(m map[string]string) (providers.Registrar, error) {
 }
 
 // NewGidinet creates a new Gidinet DNS provider.
-func NewGidinet(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewGidinet(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["username"] == "" {
 		return nil, errors.New("missing Gidinet username")
 	}
diff --git a/providers/huaweicloud/huaweicloudProvider.go b/providers/huaweicloud/huaweicloudProvider.go
index 8d3f623c93..2393f776c7 100644
--- a/providers/huaweicloud/huaweicloudProvider.go
+++ b/providers/huaweicloud/huaweicloudProvider.go
@@ -50,7 +50,7 @@ const (
 )
 
 // newHuaweicloud creates the provider.
-func newHuaweicloud(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newHuaweicloud(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	auth, err := basic.NewCredentialsBuilder().
 		WithAk(m["KeyId"]).
 		WithSk(m["SecretKey"]).
diff --git a/providers/infoblox/infobloxProvider.go b/providers/infoblox/infobloxProvider.go
index aad1916a85..75a27ffa86 100644
--- a/providers/infoblox/infobloxProvider.go
+++ b/providers/infoblox/infobloxProvider.go
@@ -36,7 +36,7 @@ type infobloxProvider struct {
 	api *infobloxAPI
 }
 
-func newInfobloxDsp(conf map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newInfobloxDsp(conf map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	return newInfoblox(conf)
 }
 
diff --git a/providers/inwx/inwxProvider.go b/providers/inwx/inwxProvider.go
index edeb9955dd..c6b23bd900 100644
--- a/providers/inwx/inwxProvider.go
+++ b/providers/inwx/inwxProvider.go
@@ -195,7 +195,7 @@ func newInwxReg(m map[string]string) (providers.Registrar, error) {
 }
 
 // new InwxDsp is called to initialize the INWX domain service provider.
-func newInwxDsp(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newInwxDsp(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	return newInwx(m)
 }
 
diff --git a/providers/joker/jokerProvider.go b/providers/joker/jokerProvider.go
index 31ff5e8876..2b4c345c63 100644
--- a/providers/joker/jokerProvider.go
+++ b/providers/joker/jokerProvider.go
@@ -89,7 +89,7 @@ type jokerProvider struct {
 }
 
 // newJoker creates a new Joker DMAPI provider.
-func newJoker(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newJoker(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	api := &jokerProvider{
 		apiURL:     "https://dmapi.joker.com/request/",
 		httpClient: &http.Client{Timeout: 30 * time.Second},
diff --git a/providers/linode/linodeProvider.go b/providers/linode/linodeProvider.go
index ad2bcff8e4..aeff55e205 100644
--- a/providers/linode/linodeProvider.go
+++ b/providers/linode/linodeProvider.go
@@ -62,7 +62,7 @@ var defaultNameServerNames = []string{
 }
 
 // NewLinode creates the provider.
-func NewLinode(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewLinode(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["token"] == "" {
 		return nil, errors.New("missing Linode token")
 	}
diff --git a/providers/luadns/luadnsProvider.go b/providers/luadns/luadnsProvider.go
index 1570597af5..a6dbab7582 100644
--- a/providers/luadns/luadnsProvider.go
+++ b/providers/luadns/luadnsProvider.go
@@ -90,7 +90,7 @@ func (l *luadnsProvider) SetConversionObserver(observer providers.ConversionObse
 }
 
 // NewLuaDNS creates the provider.
-func NewLuaDNS(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewLuaDNS(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["email"] == "" || m["apikey"] == "" {
 		return nil, errors.New("missing LuaDNS email or apikey")
 	}
diff --git a/providers/mythicbeasts/mythicbeastsProvider.go b/providers/mythicbeasts/mythicbeastsProvider.go
index 33df09be52..d249598fbf 100644
--- a/providers/mythicbeasts/mythicbeastsProvider.go
+++ b/providers/mythicbeasts/mythicbeastsProvider.go
@@ -81,7 +81,7 @@ func init() {
 	})
 }
 
-func newDsp(conf map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newDsp(conf map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if conf["keyID"] == "" {
 		return nil, errors.New("missing Mythic Beasts auth keyID")
 	}
diff --git a/providers/netbird/netbirdProvider.go b/providers/netbird/netbirdProvider.go
index 55c6a3ecfc..a7213ecaf5 100644
--- a/providers/netbird/netbirdProvider.go
+++ b/providers/netbird/netbirdProvider.go
@@ -45,7 +45,7 @@ type netbirdProvider struct {
 }
 
 // NewNetbird creates a NetBird-specific DNS provider.
-func NewNetbird(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewNetbird(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["token"] == "" {
 		return nil, errors.New("no NetBird token provided")
 	}
diff --git a/providers/opensrs/opensrsProvider.go b/providers/opensrs/opensrsProvider.go
index 9bec2af13b..63ed727eae 100644
--- a/providers/opensrs/opensrsProvider.go
+++ b/providers/opensrs/opensrsProvider.go
@@ -122,7 +122,7 @@ func newReg(conf map[string]string) (providers.Registrar, error) {
 	return newProvider(conf, nil)
 }
 
-func newProvider(m map[string]string, metadata json.RawMessage) (*opensrsProvider, error) {
+func newProvider(m map[string]string, _ json.RawMessage) (*opensrsProvider, error) {
 	api := &opensrsProvider{}
 	api.APIKey = m["apikey"]
 
diff --git a/providers/packetframe/packetframeProvider.go b/providers/packetframe/packetframeProvider.go
index 3cee9ef54d..08e42ce215 100644
--- a/providers/packetframe/packetframeProvider.go
+++ b/providers/packetframe/packetframeProvider.go
@@ -28,7 +28,7 @@ func (api *packetframeProvider) SetConversionObserver(observer providers.Convers
 }
 
 // newPacketframe creates the provider.
-func newPacketframe(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newPacketframe(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["token"] == "" {
 		return nil, errors.New("missing Packetframe token")
 	}
diff --git a/providers/tencentdns/tencentdnsProvider.go b/providers/tencentdns/tencentdnsProvider.go
index 5e4853ba99..668a5cfca1 100644
--- a/providers/tencentdns/tencentdnsProvider.go
+++ b/providers/tencentdns/tencentdnsProvider.go
@@ -86,7 +86,7 @@ type tencentdnsProvider struct {
 	client *tencentCloudClient
 }
 
-func newTencentDNSDsp(config map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func newTencentDNSDsp(config map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	return newTencentDNS(config)
 }
 
diff --git a/providers/transip/transipProvider.go b/providers/transip/transipProvider.go
index f38c120c77..650f603d83 100644
--- a/providers/transip/transipProvider.go
+++ b/providers/transip/transipProvider.go
@@ -68,7 +68,7 @@ var features = providers.DocumentationNotes{
 }
 
 // NewTransip creates a new TransIP provider.
-func NewTransip(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewTransip(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	if m["AccessToken"] == "" && m["PrivateKey"] == "" {
 		return nil, errors.New("no TransIP AccessToken or PrivateKey provided")
 	}
diff --git a/providers/vultr/vultrProvider.go b/providers/vultr/vultrProvider.go
index fb7243fa3c..d1fc950de0 100644
--- a/providers/vultr/vultrProvider.go
+++ b/providers/vultr/vultrProvider.go
@@ -66,7 +66,7 @@ var defaultNS = []string{
 }
 
 // NewProvider initializes a Vultr DNSServiceProvider.
-func NewProvider(m map[string]string, metadata json.RawMessage) (providers.DNSServiceProvider, error) {
+func NewProvider(m map[string]string, _ json.RawMessage) (providers.DNSServiceProvider, error) {
 	token := m["token"]
 	if token == "" {
 		return nil, errors.New("missing Vultr API token")