fix: address all 6 Copilot review comments on PR #4

1. sslmode=disable → sslmode=prefer (tries TLS, graceful fallback for local dev)
2. grpc bump is intentional (CVE-2026-33186) — already noted in PR description
3. Table scroll comment updated to match unconditional behavior
4. isDocs derived from pathname (/docs prefix) not !isHome — fixes 404 page
5. 404 page gets pb-20 for mobile bottom nav clearance
6. Broken /configuration link → /docs/configuration in getting-started

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

Author: michaelmcnees

internal/config/config.go

@@ -129,7 +129,7 @@ func Load(cmd *cobra.Command) (*Config, error) {
 	v := viper.New()
 
 	// Defaults
-	v.SetDefault("database.url", "postgres://mantle:mantle@localhost:5432/mantle?sslmode=disable")
+	v.SetDefault("database.url", "postgres://mantle:mantle@localhost:5432/mantle?sslmode=prefer")
 	v.SetDefault("database.max_open_conns", 25)
 	v.SetDefault("database.max_idle_conns", 25)
 	v.SetDefault("database.conn_max_lifetime", 5*time.Minute)

internal/config/config_test.go

@@ -29,7 +29,7 @@ func TestLoad_Defaults(t *testing.T) {
 		t.Fatalf("Load() error = %v", err)
 	}
 
-	if cfg.Database.URL != "postgres://mantle:mantle@localhost:5432/mantle?sslmode=disable" {
+	if cfg.Database.URL != "postgres://mantle:mantle@localhost:5432/mantle?sslmode=prefer" {
 		t.Errorf("Database.URL = %q, want default", cfg.Database.URL)
 	}
 	if cfg.API.Address != ":8080" {
@@ -97,7 +97,7 @@ func TestLoad_ImplicitConfigMissing_UsesDefaults(t *testing.T) {
 		t.Fatalf("Load() error = %v, want nil (silent fallback)", err)
 	}
 
-	if cfg.Database.URL != "postgres://mantle:mantle@localhost:5432/mantle?sslmode=disable" {
+	if cfg.Database.URL != "postgres://mantle:mantle@localhost:5432/mantle?sslmode=prefer" {
 		t.Errorf("Database.URL = %q, want default", cfg.Database.URL)
 	}
 }

site/src/components/Nav.astro

@@ -4,7 +4,7 @@ interface Props {
 }
 const { isHome = false } = Astro.props;
 const prefix = isHome ? '' : '/';
-const isDocs = !isHome;
+const isDocs = Astro.url.pathname.startsWith('/docs');
 ---
 <nav class="fixed top-0 left-0 right-0 z-50 border-b border-outline-variant bg-surface/80 backdrop-blur-md">
   <div class={`nav-inner mx-auto px-6 flex items-center justify-between h-16 ${isHome ? 'max-w-[1200px]' : 'max-w-[1280px]'}`}>

site/src/content/docs/getting-started/index.md

@@ -40,7 +40,7 @@ The default database URL uses `sslmode=disable`, which is correct for local deve
 export MANTLE_DATABASE_URL="postgres://mantle:secret@db.example.com:5432/mantle?sslmode=require"
 ```
 
-See [Configuration](/configuration) for all database options.
+See [Configuration](/docs/configuration) for all database options.
 
 You should see:
 

site/src/pages/404.astro

@@ -4,7 +4,7 @@ import Nav from '../components/Nav.astro';
 ---
 <Base title="404 — Route Not Found | Mantle">
   <Nav />
-  <main class="min-h-screen flex items-center justify-center pt-16">
+  <main class="min-h-screen flex items-center justify-center pt-16 pb-20 md:pb-0">
     <div class="mx-auto max-w-[700px] px-6 py-24">
       <h1 class="font-headline text-4xl sm:text-5xl font-bold tracking-tight mb-8">
         404: Route Not Found

site/src/styles/global.css

@@ -317,7 +317,7 @@ body {
   margin-top: 0;
 }
 
-/* Tables — horizontally scrollable on small screens */
+/* Tables — display:block enables horizontal scroll when content overflows */
 .prose table {
   display: block;
   width: 100%;