@yawlabs/postgres-mcp 0.6.0 → 0.6.1

package/CHANGELOG.md

@@ -7,6 +7,41 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- `pg_advisor` `tables_without_primary_key` now includes partitioned-table
+  parents (`relkind='p'`) alongside plain heap tables. A partitioned table
+  with no PK is a real design-drift signal -- and the neighboring
+  `public_tables_without_rls` check already covered both relkinds, so the
+  inconsistency was an oversight. Partition children still inherit the
+  parent's PK as an `indisprimary` index, so they remain filtered out by
+  the existing `NOT EXISTS` clause.
+- `pg_replication_status` now surfaces partial failures via a top-level
+  `_warnings` array instead of short-circuiting on the first sub-query
+  failure. Matches the convention already used by `pg_health`,
+  `pg_describe_table`, and `pg_advisor`. When the WAL position lookup
+  fails, `is_replica` is now `null` rather than `false` so a permission
+  error can't be mistaken for "this is a primary."
+- `identSchema` (shared schema/table/column-name validator) now enforces
+  postgres's 63-byte `NAMEDATALEN` limit on byte length, not JS char
+  length. A multi-byte identifier like 32x `é` (64 UTF-8 bytes) used to
+  pass validation but postgres would silently truncate it; now it fails
+  at the call boundary with a clear message. Centralized in `params.ts`
+  so `schemas.ts`, `stats.ts`, `admin.ts`, and `explain.ts` share one
+  definition. `pg_explain.hypothetical_indexes` validates the same byte
+  limit per-piece on the `schema.table` form and per-column inside
+  `validateHypoIndex`, so direct handler calls that bypass Zod still
+  get the protection.
+- `getSslConfig` now logs a one-shot stderr warning when
+  `POSTGRES_SSL_REJECT_UNAUTHORIZED` is set to an unrecognized value
+  (typo, empty string, ...). Previously the typo silently fell through
+  to pg's default, indistinguishable from "env var unset" -- a connection
+  with unintended TLS posture could land without any signal.
+
+### Changed
+- `pg_top_queries` extension-presence and version probes consolidated
+  into a single catalog round-trip (was two). The actual stats query
+  remains a second round-trip since the column names are version-dynamic.
+
 ## [0.6.0] - 2026-05-14
 
 ### Added

package/dist/index.js

@@ -36117,6 +36117,9 @@ function getSslConfig() {
   if (raw === void 0) return void 0;
   if (raw === "0" || raw === "false") return { rejectUnauthorized: false };
   if (raw === "1" || raw === "true") return { rejectUnauthorized: true };
+  console.error(
+    `[postgres-mcp] POSTGRES_SSL_REJECT_UNAUTHORIZED=${JSON.stringify(raw)} not recognized; expected "0", "false", "1", or "true". Deferring to the pg driver / connection-string default.`
+  );
   return void 0;
 }
 function getPool() {
@@ -36346,6 +36349,14 @@ async function shutdown() {
   }
 }
 
+// src/tools/params.ts
+var paramValue = external_exports.lazy(
+  () => external_exports.union([external_exports.string(), external_exports.number(), external_exports.boolean(), external_exports.null(), external_exports.array(paramValue), external_exports.record(external_exports.string(), paramValue)])
+);
+var identSchema = external_exports.string().min(1).max(63).refine((v) => Buffer.byteLength(v, "utf8") <= 63, {
+  message: "Identifier exceeds PostgreSQL's 63-byte NAMEDATALEN limit (multi-byte characters count as multiple bytes)."
+});
+
 // src/tools/admin.ts
 var adminTools = [
   {
@@ -36445,8 +36456,8 @@ var adminTools = [
       openWorldHint: true
     },
     inputSchema: external_exports.object({
-      schema: external_exports.string().min(1).max(63).default("public").describe("Schema name (defaults to 'public')."),
-      table: external_exports.string().min(1).max(63).optional().describe("Table name. Omit to list privileges for all tables in the schema.")
+      schema: identSchema.default("public").describe("Schema name (defaults to 'public')."),
+      table: identSchema.optional().describe("Table name. Omit to list privileges for all tables in the schema.")
     }),
     handler: async (input) => {
       const { schema, table } = input;
@@ -36548,16 +36559,18 @@ var adminTools = [
                END AS wal_position`
           )
         ]);
-        if (!slotsRes.ok) return slotsRes;
-        if (!replicasRes.ok) return replicasRes;
-        if (!walRes.ok) return walRes;
+        const warnings = [];
+        if (!slotsRes.ok) warnings.push(`slots fetch failed: ${slotsRes.error}`);
+        if (!replicasRes.ok) warnings.push(`replicas fetch failed: ${replicasRes.error}`);
+        if (!walRes.ok) warnings.push(`wal_position fetch failed: ${walRes.error}`);
         return {
           ok: true,
           data: {
-            is_replica: walRes.data?.[0]?.is_in_recovery ?? false,
-            wal_position: walRes.data?.[0]?.wal_position ?? null,
-            slots: slotsRes.data ?? [],
-            replicas: replicasRes.data ?? []
+            is_replica: walRes.ok ? walRes.data?.[0]?.is_in_recovery ?? false : null,
+            wal_position: walRes.ok ? walRes.data?.[0]?.wal_position ?? null : null,
+            slots: slotsRes.ok ? slotsRes.data ?? [] : [],
+            replicas: replicasRes.ok ? replicasRes.data ?? [] : [],
+            ...warnings.length > 0 ? { _warnings: warnings } : {}
           }
         };
       });
@@ -36575,7 +36588,7 @@ var adminTools = [
     },
     inputSchema: external_exports.object({
       seqExhaustionThreshold: external_exports.number().min(0).max(1).default(0.5).describe("Minimum used-fraction (last_value / max_value) to flag a sequence (default 0.5 = 50%)."),
-      rlsSchemas: external_exports.array(external_exports.string().min(1).max(63)).default(["public"]).describe("Schemas where RLS-missing should be flagged. Defaults to ['public']."),
+      rlsSchemas: external_exports.array(identSchema).default(["public"]).describe("Schemas where RLS-missing should be flagged. Defaults to ['public']."),
       limit: external_exports.number().int().min(1).max(500).default(50).describe("Max rows per category (default 50).")
     }),
     handler: async (input) => {
@@ -36603,15 +36616,22 @@ var adminTools = [
             [seqExhaustionThreshold, limit]
           ),
           run(
-            // Declarative-partition children inherit the parent's primary key
-            // as an indisprimary index, so the NOT EXISTS clause already
-            // excludes them. Nothing extra needed for partitioned schemas.
+            // Includes partitioned parents (relkind='p') alongside plain heap
+            // tables ('r'). A partitioned table with no PK is a real design-
+            // drift signal -- if a PK exists on a partitioned table it must
+            // include the partition key columns, but having no PK at all is
+            // legal and usually unintended. Matches the relkind filter used
+            // by public_tables_without_rls below.
+            //
+            // Partition children (relkind='r') inherit the parent's PK as an
+            // indisprimary index on the child, so the NOT EXISTS clause keeps
+            // already filtering them out.
             `SELECT
                n.nspname AS schema,
                c.relname AS "table"
              FROM pg_catalog.pg_class c
              JOIN pg_catalog.pg_namespace n ON n.oid = c.relnamespace
-             WHERE c.relkind = 'r'
+             WHERE c.relkind IN ('r', 'p')
                AND n.nspname NOT IN ('pg_catalog', 'information_schema')
                AND n.nspname NOT LIKE 'pg_%'
                AND NOT EXISTS (
@@ -36663,7 +36683,7 @@ var adminTools = [
       openWorldHint: true
     },
     inputSchema: external_exports.object({
-      schema: external_exports.string().min(1).max(63).optional().describe("Limit to one schema. If omitted, all user schemas are included."),
+      schema: identSchema.optional().describe("Limit to one schema. If omitted, all user schemas are included."),
       minDeadRatio: external_exports.number().min(0).max(1).default(0.1).describe("Minimum dead-tuple fraction to include - dead / (live + dead). Default 0.1 = 10%."),
       limit: external_exports.number().int().min(1).max(200).default(50).describe("Max rows to return (default 50).")
     }),
@@ -36699,16 +36719,15 @@ var adminTools = [
   }
 ];
 
-// src/tools/params.ts
-var paramValue = external_exports.lazy(
-  () => external_exports.union([external_exports.string(), external_exports.number(), external_exports.boolean(), external_exports.null(), external_exports.array(paramValue), external_exports.record(external_exports.string(), paramValue)])
-);
-
 // src/tools/explain.ts
 var indexAccessMethod = external_exports.enum(["btree", "hash", "gin", "gist", "brin", "spgist"]);
 var hypotheticalIndex = external_exports.object({
+  // `table` is `schema.table` or `table`. The 127-char ceiling is a generous
+  // upper bound on the combined form -- the actual NAMEDATALEN (63-byte)
+  // limit on each piece after the split is enforced in validateHypoIndex,
+  // since splitting and per-piece byte-checking is awkward in Zod.
   table: external_exports.string().min(1).max(127).describe("Target table. Use `schema.table` (e.g. `public.users`) or just `table` for the search_path."),
-  columns: external_exports.array(external_exports.string().min(1).max(63)).min(1).describe("Column names in index order. Quoted identifiers are not supported here -- pass plain names."),
+  columns: external_exports.array(identSchema).min(1).describe("Column names in index order. Quoted identifiers are not supported here -- pass plain names."),
   using: indexAccessMethod.default("btree").describe("Index access method. btree is the right answer for almost every query.")
 });
 function quoteIdent(name) {
@@ -36722,11 +36741,17 @@ function validateHypoIndex(idx) {
     if (piece.includes('"')) {
       return `Hypothetical index table ${JSON.stringify(idx.table)} contains a double-quote; pass plain identifier names without pre-quoting.`;
     }
+    if (Buffer.byteLength(piece, "utf8") > 63) {
+      return `Hypothetical index table piece ${JSON.stringify(piece)} exceeds PostgreSQL's 63-byte NAMEDATALEN limit (multi-byte characters count as multiple bytes).`;
+    }
   }
   for (const col of idx.columns) {
     if (col.includes('"')) {
       return `Hypothetical index column ${JSON.stringify(col)} contains a double-quote; pass plain identifier names without pre-quoting.`;
     }
+    if (Buffer.byteLength(col, "utf8") > 63) {
+      return `Hypothetical index column ${JSON.stringify(col)} exceeds PostgreSQL's 63-byte NAMEDATALEN limit (multi-byte characters count as multiple bytes).`;
+    }
   }
   return null;
 }
@@ -36949,7 +36974,6 @@ var queryTools = [
 ];
 
 // src/tools/schemas.ts
-var identSchema = external_exports.string().min(1).max(63);
 var schemaTools = [
   {
     name: "pg_list_schemas",
@@ -37365,7 +37389,6 @@ var schemaTools = [
 ];
 
 // src/tools/stats.ts
-var identSchema2 = external_exports.string().min(1).max(63);
 var statsTools = [
   {
     name: "pg_top_queries",
@@ -37383,22 +37406,17 @@ var statsTools = [
     }),
     handler: async (input) => {
       const { orderBy, limit } = input;
-      const check2 = await runInternal(
-        `SELECT EXISTS (
-           SELECT 1 FROM pg_catalog.pg_extension WHERE extname = 'pg_stat_statements'
-         ) AS installed`
+      const versionRes = await runInternal(
+        `SELECT extversion AS version FROM pg_catalog.pg_extension WHERE extname = 'pg_stat_statements'`
       );
-      if (!check2.ok) return check2;
-      if (!check2.data?.[0]?.installed) {
+      if (!versionRes.ok) return versionRes;
+      if (!versionRes.data || versionRes.data.length === 0) {
         return {
           ok: false,
           error: "pg_stat_statements extension is not installed. Install it with `CREATE EXTENSION pg_stat_statements;` (may require superuser) and add `pg_stat_statements` to `shared_preload_libraries` in postgresql.conf, then restart."
         };
       }
-      const versionRes = await runInternal(
-        `SELECT extversion AS version FROM pg_catalog.pg_extension WHERE extname = 'pg_stat_statements'`
-      );
-      const extVersion = versionRes.ok ? versionRes.data?.[0]?.version ?? "0" : "0";
+      const extVersion = versionRes.data[0]?.version ?? "0";
       const useExecSuffix = compareVersions(extVersion, "1.8") >= 0;
       const totalCol = useExecSuffix ? "total_exec_time" : "total_time";
       const meanCol = useExecSuffix ? "mean_exec_time" : "mean_time";
@@ -37441,7 +37459,7 @@ var statsTools = [
       openWorldHint: true
     },
     inputSchema: external_exports.object({
-      schema: identSchema2.optional().describe("Limit to one schema. If omitted, all user schemas are included."),
+      schema: identSchema.optional().describe("Limit to one schema. If omitted, all user schemas are included."),
       minSize: external_exports.number().int().min(0).default(1e3).describe("Minimum live tuple count to include (default 1000, filters out tiny/empty tables)."),
       limit: external_exports.number().int().min(1).max(100).default(20).describe("Max rows to return (default 20).")
     }),
@@ -37482,7 +37500,7 @@ var statsTools = [
       openWorldHint: true
     },
     inputSchema: external_exports.object({
-      schema: identSchema2.optional().describe("Limit to one schema. If omitted, all user schemas are included."),
+      schema: identSchema.optional().describe("Limit to one schema. If omitted, all user schemas are included."),
       maxScans: external_exports.number().int().min(0).default(10).describe("Include indexes with scan count <= this (default 10). Use 0 for 'never scanned'."),
       limit: external_exports.number().int().min(1).max(200).default(50).describe("Max rows to return (default 50).")
     }),
@@ -37529,7 +37547,7 @@ function compareVersions(a, b) {
 }
 
 // src/index.ts
-var version2 = true ? "0.6.0" : (await null).createRequire(import.meta.url)("../package.json").version;
+var version2 = true ? "0.6.1" : (await null).createRequire(import.meta.url)("../package.json").version;
 var subcommand = process.argv[2];
 if (subcommand === "version" || subcommand === "--version") {
   console.log(version2);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yawlabs/postgres-mcp",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "PostgreSQL MCP server - query, schema introspection, explain, and health checks for AI assistants",
   "license": "MIT",
   "author": "YawLabs <contact@yaw.sh>",