tdsql-mcp 1.3.0__tar.gz → 1.3.2__tar.gz

{tdsql_mcp-1.3.0 → tdsql_mcp-1.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tdsql-mcp
-Version: 1.3.0
+Version: 1.3.2
 Summary: MCP server for Teradata Vantage — SQL execution and native analytics function reference for AI agents
 Project-URL: Homepage, https://github.com/ksturgeon-td/tdsql-mcp
 Project-URL: Repository, https://github.com/ksturgeon-td/tdsql-mcp

{tdsql_mcp-1.3.0 → tdsql_mcp-1.3.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "tdsql-mcp"
-version = "1.3.0"
+version = "1.3.2"
 description = "MCP server for Teradata Vantage — SQL execution and native analytics function reference for AI agents"
 readme = "README.md"
 requires-python = ">=3.10"

{tdsql_mcp-1.3.0 → tdsql_mcp-1.3.2}/skills/teradata-sql-analytics/syntax/catalog-views.md

@@ -98,15 +98,31 @@ WHERE DatabaseName = 'mydb' AND TableName = 'mytable'
 ORDER BY ColumnName;
 ```
 
-## Access Rights
+## Listing Accessible Databases
+
+Prefer the `list_databases` MCP tool — it calls `DBC.DatabasesV` which already filters to databases visible to the current session.
+
+`DBC.DatabasesV` answers "what databases exist and are visible to me?"  
+`DBC.AllRightsV` answers "what databases do I have explicit rights on?" (includes role-inherited grants)
+
+Use `DBC.AllRightsV` when you need to know what rights a specific user holds:
+
 ```sql
--- What access does the current user have on a database?
+-- Databases a user has rights on (includes role-inherited grants)
+SELECT DISTINCT DatabaseName
+FROM DBC.AllRightsV
+WHERE UserName = 'some_user'
+ORDER BY DatabaseName;
+
+-- Full rights detail for a user
 SELECT AccessRight, DatabaseName, TableName
-FROM DBC.UserRightsV
-WHERE UserName = USER
+FROM DBC.AllRightsV
+WHERE UserName = 'some_user'
 ORDER BY DatabaseName, TableName;
 ```
 
+> **Do not use `DBC.UserRightsV` for database enumeration** — it shows only directly granted rights and misses role-inherited access. Use `DBC.AllRightsV` for a complete picture.
+
 ## Common Lookup Patterns
 ```sql
 -- Fully qualified table info in one query

{tdsql_mcp-1.3.0 → tdsql_mcp-1.3.2}/skills/teradata-sql-analytics/syntax/guidelines.md

@@ -353,7 +353,7 @@ Native functions do not cover everything. Use hand-written SQL for:
 - Date/time arithmetic (`date-time`)
 - CASE expressions and NULL handling (`conditional`)
 - Window functions for lag/lead features, running totals (`window-functions`)
-- Schema discovery queries against DBC.* views (`catalog-views`)
+- Schema discovery via MCP tools first — `list_databases`, `list_tables`, `describe_table` cover the common cases; fall back to manual DBC.* queries only for capabilities not covered by those tools (`catalog-views`)
 - Bit/byte manipulation — `BITAND`, `BITOR`, `BITXOR`, `BITNOT`, `SHIFTLEFT`/`SHIFTRIGHT`, `ROTATELEFT`/`ROTATERIGHT`, `GETBIT`, `SETBIT`, `COUNTSET`, `SUBBITSTR`, `TO_BYTE` — all Teradata-specific functions, **no ANSI equivalents**; do not use `&`, `|`, `^`, `~` operators (`bit-byte-functions`)
 - JSON data — native `JSON` type with BSON/UBJSON binary formats; JSONPath extraction; shredding (JSON→relational) and publishing (relational→JSON) — all in-database (`json-functions`)
 - One-off computations not covered by any native function

{tdsql_mcp-1.3.0 → tdsql_mcp-1.3.2}/src/tdsql_mcp/server.py

@@ -81,6 +81,18 @@ def _require_write() -> None:
         raise PermissionError("Server is running in read-only mode; write operations are disabled.")
 
 
+def _maybe_parse_json(value: Any) -> Any:
+    """If a value is a string that looks like a JSON object or array, parse it."""
+    if isinstance(value, str):
+        s = value.strip()
+        if s and s[0] in ('{', '['):
+            try:
+                return json.loads(s)
+            except Exception:
+                pass
+    return value
+
+
 def _execute_query_internal(sql: str, params: list | None = None) -> list[dict]:
     with _conn_lock:
         conn = _reconnect_if_needed()
@@ -120,7 +132,10 @@ def execute_query(sql: str, max_rows: int = 100) -> str:
                 return json.dumps({"rows": [], "row_count": 0, "truncated": False})
             columns = [desc[0] for desc in cur.description]
             rows = cur.fetchmany(max_rows)
-            result = [dict(zip(columns, row)) for row in rows]
+            result = [
+                dict(zip(columns, (_maybe_parse_json(v) for v in row)))
+                for row in rows
+            ]
             # Peek to detect truncation without fetching everything
             truncated = cur.fetchone() is not None
             return json.dumps(