From 4da6fb117b5b93f8c1be51b9a6c35e668110adbd Mon Sep 17 00:00:00 2001
From: Andrey Avtomonov <andreybavt@gmail.com>
Date: Thu, 14 May 2026 16:12:30 +0200
Subject: [PATCH] fix(cli): read metabase status url from api_url

`ktx status` was probing `url` / `base_url` on metabase connections, but
ktx.yaml stores it as `api_url`, so the field always reported "url not
set". Read `api_url` directly and align the warning text with the actual
key.
---
 packages/cli/src/status-project.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/cli/src/status-project.ts b/packages/cli/src/status-project.ts
index 08686355..3e44dba9 100644
--- a/packages/cli/src/status-project.ts
+++ b/packages/cli/src/status-project.ts
@@ -281,9 +281,9 @@ function buildConnectionStatus(
       return warn('repoUrl not set', 'Rerun `ktx setup`');
     }
     case 'metabase': {
-      const url = (conn as Record<string, unknown>).url ?? (conn as Record<string, unknown>).base_url;
+      const url = (conn as Record<string, unknown>).api_url;
       if (typeof url === 'string' && url.length > 0) return ok(`url: ${url}`);
-      return warn('url not set', 'Rerun `ktx setup`');
+      return warn('api_url not set', 'Rerun `ktx setup`');
     }
     case 'looker':
     case 'lookml': {