From 486a369f05125a3b86d663ea94684466e0658099 Mon Sep 17 00:00:00 2001
From: s <s@db.org.ai>
Date: Fri, 7 Nov 2025 18:13:02 -0500
Subject: fix: prevent empty string from overriding environment variable api
 key

---
 cmd/root.go               | 2 +-
 internal/config/config.go | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/cmd/root.go b/cmd/root.go
index 0f29583..cb6a676 100644
--- a/cmd/root.go
+++ b/cmd/root.go
@@ -28,5 +28,5 @@ func Execute() {
 }
 
 func init() {
-	rootCmd.PersistentFlags().StringP("api-key", "k", os.Getenv("DBORG_API_KEY"), "API key for authentication")
+	rootCmd.PersistentFlags().StringP("api-key", "k", "", "API key for authentication (or set DBORG_API_KEY env var)")
 }
diff --git a/internal/config/config.go b/internal/config/config.go
index 44ca7e6..e1da2c2 100644
--- a/internal/config/config.go
+++ b/internal/config/config.go
@@ -24,7 +24,9 @@ func New() *Config {
 }
 
 func (c *Config) WithAPIKey(key string) *Config {
-	c.APIKey = key
+	if key != "" {
+		c.APIKey = key
+	}
 	return c
 }
 
-- 
cgit v1.2.3