fix: handle --limit 0 for schema inspection and safe dispose() cleanup

- `--limit 0` now correctly calls `immediatelyExecuteSQLWithLimit` (valid for schema inspection)
- `--limit foo` (NaN) is rejected and falls through to unlimited execution
- Wrap `adapter.dispose()` in try-catch to prevent masking command errors

Author: suryaiyer95

packages/dbt-tools/src/commands/execute.ts

@@ -5,8 +5,8 @@ export async function execute(adapter: DBTProjectIntegrationAdapter, args: strin
   if (!sql) return { error: "Missing --query" }
   const model = flag(args, "model") ?? ""
   const raw = flag(args, "limit")
-  const limit = raw ? parseInt(raw, 10) : undefined
-  if (limit) return adapter.immediatelyExecuteSQLWithLimit(sql, model, limit)
+  const limit = raw !== undefined ? parseInt(raw, 10) : undefined
+  if (limit !== undefined && !Number.isNaN(limit)) return adapter.immediatelyExecuteSQLWithLimit(sql, model, limit)
   return adapter.immediatelyExecuteSQL(sql, model)
 }
 

packages/dbt-tools/src/index.ts

@@ -187,7 +187,7 @@ async function main() {
         result = { error: `Unknown command: ${cmd}`, usage: USAGE }
     }
   } finally {
-    await adapter.dispose()
+    try { await adapter.dispose() } catch {}
   }
 
   return result