fix-lint

Author: JayT106

cmd/cronosd/cmd/patch_db.go

@@ -229,15 +229,15 @@ Examples:
 				} else {
 					// For multiple databases, validate that targetPath is a directory, not a *.db file
 					cleanedTargetPath := filepath.Clean(targetPath)
-					if filepath.Ext(cleanedTargetPath) == ".db" {
+					if filepath.Ext(cleanedTargetPath) == dbmigrate.DbExtension {
 						return fmt.Errorf("when patching multiple databases, --target-path must be a data directory (e.g., ~/.cronos/data), not a *.db file path (got %q); remove the .db suffix", targetPath)
 					}
 					// Treat targetPath as data directory
-					dbTargetPath = filepath.Join(targetPath, dbName+".db")
+					dbTargetPath = filepath.Join(targetPath, dbName+dbmigrate.DbExtension)
 				}
 
 				cleanTargetPath := filepath.Clean(dbTargetPath)
-				if filepath.Ext(cleanTargetPath) != ".db" {
+				if filepath.Ext(cleanTargetPath) != dbmigrate.DbExtension {
 					return fmt.Errorf("--target-path must reference a *.db directory (got %q)", dbTargetPath)
 				}
 

cmd/cronosd/cmd/patch_db_test.go

@@ -4,6 +4,7 @@ import (
 	"path/filepath"
 	"testing"
 
+	"github.com/crypto-org-chain/cronos/cmd/cronosd/dbmigrate"
 	"github.com/stretchr/testify/require"
 )
 
@@ -65,7 +66,7 @@ func TestTargetPathValidation(t *testing.T) {
 			} else {
 				// Multiple DBs: validate targetPath is not a *.db file
 				cleanedTargetPath := filepath.Clean(tt.targetPath)
-				if filepath.Ext(cleanedTargetPath) == ".db" {
+				if filepath.Ext(cleanedTargetPath) == dbmigrate.DbExtension {
 					err = &targetPathError{path: tt.targetPath}
 				}
 			}

cmd/cronosd/dbmigrate/migrate.go

@@ -117,7 +117,7 @@ func Migrate(opts MigrateOptions) (*MigrationStats, error) {
 	// For migration, we need to ensure we don't accidentally overwrite an existing DB
 	// Unified path format for all backends: <dbName>.migrate-temp.db
 	tempTargetDir := filepath.Join(targetDataDir, opts.DBName+".migrate-temp.db")
-	finalTargetDir := filepath.Join(targetDataDir, opts.DBName+".db")
+	finalTargetDir := filepath.Join(targetDataDir, opts.DBName+DbExtension)
 
 	var targetDB dbm.DB
 	if opts.TargetBackend == dbm.RocksDBBackend {

cmd/cronosd/dbmigrate/patch.go

@@ -18,7 +18,7 @@ import (
 )
 
 const (
-	dbExtension = ".db"
+	DbExtension = ".db"
 )
 
 // EthTxInfo stores information needed to search for event-indexed keys in source DB
@@ -70,7 +70,7 @@ func validatePatchOptions(opts PatchOptions) error {
 	}
 
 	// Construct and validate source database path
-	sourceDBPath := filepath.Join(opts.SourceHome, "data", opts.DBName+".db")
+	sourceDBPath := filepath.Join(opts.SourceHome, "data", opts.DBName+DbExtension)
 	if _, err := os.Stat(sourceDBPath); os.IsNotExist(err) {
 		return fmt.Errorf("source database does not exist: %s", sourceDBPath)
 	}
@@ -85,11 +85,11 @@ func validatePatchOptions(opts PatchOptions) error {
 
 // openSourceDatabase opens the source database for reading
 func openSourceDatabase(opts PatchOptions) (dbm.DB, string, error) {
-	sourceDBPath := filepath.Join(opts.SourceHome, "data", opts.DBName+".db")
+	sourceDBPath := filepath.Join(opts.SourceHome, "data", opts.DBName+DbExtension)
 	sourceDir := filepath.Dir(sourceDBPath)
 	sourceName := filepath.Base(sourceDBPath)
-	if len(sourceName) > len(dbExtension) && sourceName[len(sourceName)-len(dbExtension):] == dbExtension {
-		sourceName = sourceName[:len(sourceName)-len(dbExtension)]
+	if len(sourceName) > len(DbExtension) && sourceName[len(sourceName)-len(DbExtension):] == DbExtension {
+		sourceName = sourceName[:len(sourceName)-len(DbExtension)]
 	}
 
 	sourceDB, err := dbm.NewDB(sourceName, opts.SourceBackend, sourceDir)
@@ -109,7 +109,7 @@ func openTargetDatabase(opts PatchOptions) (dbm.DB, error) {
 	} else {
 		targetDir := filepath.Dir(opts.TargetPath)
 		targetName := filepath.Base(opts.TargetPath)
-		targetName = strings.TrimSuffix(targetName, dbExtension)
+		targetName = strings.TrimSuffix(targetName, DbExtension)
 		targetDB, err = dbm.NewDB(targetName, opts.TargetBackend, targetDir)
 	}
 	if err != nil {