fix: address CLI UX feedback — file input, region hints, comment handling

- Add --file flag and stdin support to `dremio query run` so multi-line
  SQL no longer requires shell escaping (supports --file path, piped
  stdin, and '-' argument)
- Enrich 401 errors with a region hint when the default US endpoint is
  used, saving EU users from a confusing debugging loop
- Put breadcrumb comment on its own line so SQL with -- inline comments
  is no longer swallowed by the prepended comment
- Show config file path and region info in `dremio setup --help`

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

Author: rahim-bhojani

src/drs/client.py

@@ -164,7 +164,7 @@ async def delete_project(self, project_id: str) -> dict:
 
     async def submit_sql(self, sql: str, context: list[str] | None = None) -> dict:
         """Submit a SQL query. Returns job metadata including job_id."""
-        body: dict[str, Any] = {"sql": f"/* dremio-cli: submitter=cli */ {sql}"}
+        body: dict[str, Any] = {"sql": f"/* dremio-cli: submitter=cli */\n{sql}"}
         if context:
             body["context"] = context
         return await self._post(self._v0("/sql"), json=body)

src/drs/commands/query.py

@@ -18,6 +18,8 @@
 from __future__ import annotations
 
 import asyncio
+import sys
+from pathlib import Path
 from typing import Any
 
 import httpx
@@ -131,7 +133,8 @@ async def _execute():
 
 @app.command("run")
 def cli_run(
-    sql: str = typer.Argument(help="SQL query to execute"),
+    sql: str | None = typer.Argument(None, help="SQL query to execute (use '-' to read from stdin)"),
+    file: Path | None = typer.Option(None, "--file", help="Path to a SQL file to execute"),
     context: str = typer.Option(None, help="Dot-separated default schema context (e.g., myspace.folder)"),
     fmt: OutputFormat = typer.Option(OutputFormat.json, "--output", "-o", help="Output format"),
     fields: str = typer.Option(
@@ -140,10 +143,37 @@ def cli_run(
 ) -> None:
     """Execute a SQL query, wait for completion, and return results.
 
+    SQL can be provided as:
+
+      dremio query run "SELECT 1"          # inline argument
+
+      dremio query run --file query.sql    # from a file
+
+      cat query.sql | dremio query run -   # from stdin
+
     Submits the query, polls until the job completes, then fetches all result
     rows. For long-running queries, use 'drs query run' to submit and
     'drs query status' to check progress separately.
     """
+    # Resolve SQL from argument, --file, or stdin
+    if file and sql:
+        error("Cannot specify both a SQL argument and --file.")
+        raise typer.Exit(1)
+    if file:
+        if not file.exists():
+            error(f"File not found: {file}")
+            raise typer.Exit(1)
+        sql = file.read_text()
+    elif sql == "-" or (sql is None and not sys.stdin.isatty()):
+        sql = sys.stdin.read()
+    elif sql is None:
+        error("Provide a SQL query as an argument, --file path, or pipe via stdin (use '-' for stdin).")
+        raise typer.Exit(1)
+
+    if not sql.strip():
+        error("SQL query is empty.")
+        raise typer.Exit(1)
+
     client = _get_client()
     ctx = context.split(".") if context else None
 

src/drs/commands/setup.py

@@ -147,7 +147,11 @@ def _prompt_project_id(app_url: str) -> str:
 def setup_command(
     ctx: typer.Context,
 ) -> None:
-    """Interactive setup wizard — configure credentials for Dremio Cloud."""
+    """Interactive setup wizard — configure credentials for Dremio Cloud.
+
+    Writes configuration to ~/.config/dremioai/config.yaml (or the path
+    specified with --config). Prompts for region (US/EU), PAT, and project ID.
+    """
     if not sys.stdin.isatty():
         err_console.print(
             "[bold]dremio setup[/bold] requires an interactive terminal.\n\n"

src/drs/utils.py

@@ -202,6 +202,8 @@ def handle_api_error(exc: httpx.HTTPStatusError) -> DremioAPIError:
 
     if status == 401:
         hint = "Authentication failed — check your PAT token"
+        if "api.dremio.cloud" in url and "api.eu.dremio.cloud" not in url:
+            hint += ". If you are on Dremio Cloud EU, set uri: https://api.eu.dremio.cloud in your config"
     elif status == 403:
         hint = "Permission denied — insufficient privileges for this operation"
     elif status == 404:

tests/test_client.py

@@ -119,7 +119,7 @@ async def _capture(request: httpx.Request) -> httpx.Response:
 
         await client.submit_sql("SELECT 1")
 
-        assert captured["body"]["sql"] == "/* dremio-cli: submitter=cli */ SELECT 1"
+        assert captured["body"]["sql"] == "/* dremio-cli: submitter=cli */\nSELECT 1"
 
     @pytest.mark.asyncio
     async def test_submit_sql_breadcrumb_with_context(self, client: DremioClient) -> None: