fix: address PR review — paused container, quit option, dbdefaults in config, test names

- docker.go: handle "paused" container state in dockerStartPostgres
- init.go: add q/quit/back escape from connection string prompt
- config.go: build default DB URL from dbdefaults constants
- loopback_test.go: add 127.0.0.2 case and named subtests

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: michaelmcnees

internal/cli/docker.go

@@ -68,7 +68,7 @@ func dockerStartPostgres(cfg config.DatabaseConfig) error {
 	case "running":
 		// Already running — just wait for readiness.
 		return waitForPostgres(cfg)
-	case "exited", "created", "dead":
+	case "exited", "created", "dead", "paused":
 		_ = dockerRemoveContainer()
 	}
 

internal/cli/init.go

@@ -139,6 +139,11 @@ func promptConnectionString(cmd *cobra.Command, cfg *config.Config) (*sql.DB, er
 			continue
 		}
 
+		lower := strings.ToLower(connStr)
+		if lower == "q" || lower == "quit" || lower == "back" {
+			return nil, fmt.Errorf("connection string entry cancelled")
+		}
+
 		cfg.Database.URL = connStr
 		database, err := db.Open(cfg.Database)
 		if err != nil {

internal/config/config.go

@@ -11,6 +11,7 @@ import (
 	"time"
 
 	"github.com/dvflw/mantle/internal/budget"
+	"github.com/dvflw/mantle/internal/dbdefaults"
 	"github.com/dvflw/mantle/internal/netutil"
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
@@ -142,7 +143,7 @@ func Load(cmd *cobra.Command) (*Config, error) {
 	v := viper.New()
 
 	// Defaults
-	v.SetDefault("database.url", "postgres://mantle:mantle@localhost:5432/mantle?sslmode=prefer")
+	v.SetDefault("database.url", fmt.Sprintf("postgres://%s:%s@localhost:5432/%s?sslmode=prefer", dbdefaults.User, dbdefaults.Password, dbdefaults.Database))
 	v.SetDefault("database.max_open_conns", 25)
 	v.SetDefault("database.max_idle_conns", 25)
 	v.SetDefault("database.conn_max_lifetime", 5*time.Minute)

internal/netutil/loopback_test.go

@@ -9,21 +9,23 @@ import (
 
 func TestIsLoopback(t *testing.T) {
 	tests := []struct {
+		name     string
 		host     string
 		expected bool
 	}{
-		{"localhost", true},
-		{"LOCALHOST", true},
-		{"Localhost", true},
-		{"127.0.0.1", true},
-		{"::1", true},
-		{"db.example.com", false},
-		{"10.0.0.1", false},
-		{"192.168.1.1", false},
-		{"", false},
+		{"localhost", "localhost", true},
+		{"localhost-upper", "LOCALHOST", true},
+		{"localhost-mixed", "Localhost", true},
+		{"ipv4-loopback", "127.0.0.1", true},
+		{"ipv4-loopback-range", "127.0.0.2", true},
+		{"ipv6-loopback", "::1", true},
+		{"remote-host", "db.example.com", false},
+		{"private-ipv4", "10.0.0.1", false},
+		{"lan-ipv4", "192.168.1.1", false},
+		{"empty", "", false},
 	}
 	for _, tt := range tests {
-		t.Run(tt.host, func(t *testing.T) {
+		t.Run(tt.name, func(t *testing.T) {
 			assert.Equal(t, tt.expected, netutil.IsLoopback(tt.host))
 		})
 	}