Enable and fix unused issues

Author: jshufro

.github/workflows/lint.yml

@@ -35,7 +35,7 @@ jobs:
           # args: --issues-exit-code=0
 
           # For now, Smart Node will only enforce goimports linting
-          args: --disable-all --enable goimports --enable staticcheck --enable ineffassign --enable gosimple
+          args: --disable-all --enable goimports --enable staticcheck --enable ineffassign --enable gosimple --enable unused
 
           # Optional: show only new issues if it's a pull request. The default value is `false`.
           # only-new-issues: true

rocketpool-cli/client/utils.go

@@ -9,7 +9,6 @@ import (
 	"path/filepath"
 
 	"github.com/alessio/shellescape"
-	"github.com/rocket-pool/node-manager-core/api/types"
 	"github.com/rocket-pool/smartnode/v2/shared/config"
 	"gopkg.in/yaml.v2"
 )
@@ -98,16 +97,3 @@ func SaveConfig(cfg *config.SmartNodeConfig, directory string, filename string)
 
 	return nil
 }
-
-// Parse and augment the status of a client into a human-readable format
-func getClientStatusString(clientStatus types.ClientStatus) string {
-	if clientStatus.IsSynced {
-		return "synced and ready"
-	}
-
-	if clientStatus.IsWorking {
-		return fmt.Sprintf("syncing (%.2f%%)", SyncRatioToPercent(clientStatus.SyncProgress))
-	}
-
-	return fmt.Sprintf("unavailable (%s)", clientStatus.Error)
-}

rocketpool-cli/commands/node/utils.go

@@ -15,19 +15,13 @@ import (
 
 	"github.com/ethereum/go-ethereum/common"
 	"github.com/goccy/go-json"
-	"github.com/mitchellh/go-homedir"
-	"github.com/rocket-pool/node-manager-core/beacon"
 	"github.com/urfave/cli/v2"
-	"gopkg.in/yaml.v2"
 
 	"github.com/rocket-pool/node-manager-core/eth"
-	nmc_utils "github.com/rocket-pool/node-manager-core/utils"
 	"github.com/rocket-pool/node-manager-core/utils/math"
 	"github.com/rocket-pool/smartnode/v2/rocketpool-cli/client"
 	"github.com/rocket-pool/smartnode/v2/rocketpool-cli/utils"
 	"github.com/rocket-pool/smartnode/v2/rocketpool-cli/utils/tx"
-	"github.com/rocket-pool/smartnode/v2/shared/config"
-	"github.com/rocket-pool/smartnode/v2/shared/types/api"
 )
 
 const (
@@ -324,77 +318,6 @@ func promptMinNodeFee(networkCurrentNodeFee, networkMinNodeFee float64) float64
 
 }
 
-// Prompt for the password to a solo validator key as part of migration
-func promptForSoloKeyPassword(cfg *config.SmartNodeConfig, pubkey beacon.ValidatorPubkey) (string, error) {
-
-	// Check for the custom key directory
-	customKeyDir, err := homedir.Expand(cfg.GetCustomKeyPath())
-	if err != nil {
-		return "", fmt.Errorf("error expanding custom keys directory: %w", err)
-	}
-	info, err := os.Stat(customKeyDir)
-	if os.IsNotExist(err) || !info.IsDir() {
-		return "", nil
-	}
-
-	// Get the custom keystore files
-	files, err := os.ReadDir(customKeyDir)
-	if err != nil {
-		return "", fmt.Errorf("error enumerating custom keystores: %w", err)
-	}
-	if len(files) == 0 {
-		return "", nil
-	}
-
-	// Get the pubkeys for the custom keystores
-	pubkeyPasswords := map[string]string{}
-	for _, file := range files {
-		// Read the file
-		bytes, err := os.ReadFile(filepath.Join(customKeyDir, file.Name()))
-		if err != nil {
-			return "", fmt.Errorf("error reading custom keystore %s: %w", file.Name(), err)
-		}
-
-		// Deserialize it
-		keystore := api.ValidatorKeystore{}
-		err = json.Unmarshal(bytes, &keystore)
-		if err != nil {
-			return "", fmt.Errorf("error deserializing custom keystore %s: %w", file.Name(), err)
-		}
-
-		if keystore.Pubkey == pubkey {
-			// Found it, prompt for the password
-			password := utils.PromptPassword(
-				fmt.Sprintf("Please enter the password that the keystore for %s was encrypted with:", pubkey.Hex()), "^.*$", "",
-			)
-
-			formattedPubkey := strings.ToUpper(nmc_utils.RemovePrefix(pubkey.Hex()))
-			pubkeyPasswords[formattedPubkey] = password
-
-			fmt.Println()
-			break
-		}
-	}
-
-	if len(pubkeyPasswords) == 0 {
-		return "", fmt.Errorf("couldn't find the keystore for validator %s in the custom-keys directory; if you want to import this key into the Smartnode stack, you will need to put its keystore file into custom-keys first", pubkey.HexWithPrefix())
-	}
-
-	// Store it in the file
-	fileBytes, err := yaml.Marshal(pubkeyPasswords)
-	if err != nil {
-		return "", fmt.Errorf("error serializing keystore passwords file: %w", err)
-	}
-	passwordFile := cfg.GetCustomKeyPasswordFilePath()
-	err = os.WriteFile(passwordFile, fileBytes, 0600)
-	if err != nil {
-		return "", fmt.Errorf("error writing keystore passwords file: %w", err)
-	}
-
-	return passwordFile, nil
-
-}
-
 func SwapRpl(c *cli.Context, rp *client.Client, amountWei *big.Int) error {
 	// Get the TX
 	response, err := rp.Api.Node.SwapRpl(amountWei)

rocketpool-cli/commands/service/config/cfg-form.go

@@ -16,20 +16,6 @@ type parameterizedFormItem struct {
 	item      tview.FormItem
 }
 
-func registerEnableCheckbox(param *config.Parameter[bool], checkbox *tview.Checkbox, form *Form, items []*parameterizedFormItem) {
-	checkbox.SetChangedFunc(func(checked bool) {
-		param.Value = checked
-		if !checked {
-			form.Clear(true)
-			form.AddFormItem(checkbox)
-		} else {
-			for _, item := range items {
-				form.AddFormItem(item.item)
-			}
-		}
-	})
-}
-
 // Create a list of form items based on a set of parameters
 func createParameterizedFormItems(params []config.IParameter, descriptionBox *tview.TextView) []*parameterizedFormItem {
 	formItems := []*parameterizedFormItem{}

rocketpool-cli/commands/service/config/pseudomodal.go

@@ -36,12 +36,6 @@ type Pseudomodal struct {
 
 	// The currently selected form (for vertical layouts)
 	selected int
-
-	// A fixed width for the description box (0 for auto)
-	descriptionWidth int
-
-	// The collection of descriptions for each button, to be displayed in the description box
-	buttonDescriptions []string
 }
 
 // NewPseudomodal returns a new modal message window.

rocketpool-cli/commands/service/config/review-native-page.go

@@ -13,13 +13,6 @@ import (
 // Constants
 const reviewNativePageID string = "review-native-settings"
 
-// The changed settings review page
-type ReviewNativePage struct {
-	md              *mainDisplay
-	changedSettings []*config.ChangedSection
-	page            *page
-}
-
 // Create a page to review any changes
 func NewReviewNativePage(md *mainDisplay, oldConfig *snCfg.SmartNodeConfig, newConfig *snCfg.SmartNodeConfig) *ReviewPage {
 	var changedSettings []*config.ChangedSection

rocketpool-cli/commands/service/config/step-random-bn.go

@@ -9,43 +9,6 @@ import (
 const randomBnPrysmID string = "step-random-bn-prysm"
 const randomBnID string = "step-random-bn"
 
-func createRandomPrysmStep(wiz *wizard, currentStep int, totalSteps int, goodOptions []*config.ParameterOption[config.BeaconNode]) *choiceWizardStep {
-	helperText := "You have been randomly assigned to Prysm for your Beacon Node.\n\n[orange]NOTE: Prysm currently has a very high representation of the Beacon Chain. For the health of the network and the overall safety of your funds, please consider choosing a client with a lower representation. Please visit https://clientdiversity.org to learn more."
-
-	show := func(modal *choiceModalLayout) {
-		wiz.md.setPage(modal.page)
-		modal.focus(0)
-	}
-
-	done := func(buttonIndex int, buttonLabel string) {
-		if buttonIndex == 0 {
-			selectRandomBn(goodOptions, false, wiz, currentStep, totalSteps)
-		} else {
-			wiz.checkpointSyncProviderModal.show()
-		}
-	}
-
-	back := func() {
-		wiz.localBnModal.show()
-	}
-
-	return newChoiceStep(
-		wiz,
-		currentStep,
-		totalSteps,
-		helperText,
-		[]string{"Choose Another Random Client", "Keep Prysm"},
-		[]string{},
-		76,
-		"Beacon Node > Selection",
-		DirectionalModalHorizontal,
-		show,
-		done,
-		back,
-		randomBnPrysmID,
-	)
-}
-
 func createRandomBnStep(wiz *wizard, currentStep int, totalSteps int, goodOptions []*config.ParameterOption[config.BeaconNode]) *choiceWizardStep {
 	var selectedClientName string
 	selectedClient := wiz.md.Config.LocalBeaconClient.BeaconNode.Value

rocketpool-cli/commands/service/config/types.go

@@ -4,7 +4,3 @@ type settingsPage interface {
 	handleLayoutChanged()
 	getPage() *page
 }
-
-type wizardStep interface {
-	show()
-}

rocketpool-cli/commands/service/config/wizard.go

@@ -25,7 +25,6 @@ type wizard struct {
 	// Step 5 - BN settings
 	localBnModal                *choiceWizardStep
 	localBnRandomModal          *choiceWizardStep
-	localBnRandomPrysmModal     *choiceWizardStep
 	localBnPrysmWarning         *choiceWizardStep
 	localBnTekuWarning          *choiceWizardStep
 	checkpointSyncProviderModal *textBoxWizardStep

rocketpool-cli/commands/service/sync.go

@@ -4,7 +4,6 @@ import (
 	"fmt"
 	"math"
 	"strings"
-	"time"
 
 	"github.com/urfave/cli/v2"
 
@@ -22,11 +21,6 @@ func SyncRatioToPercent(in float64) float64 {
 	// TODO: INCORPORATE THIS
 }
 
-// Settings
-const (
-	ethClientRecentBlockThreshold time.Duration = 5 * time.Minute
-)
-
 func printClientStatus(status *types.ClientStatus, name string) {
 
 	if status.Error != "" {