postgresai 0.14.0-dev.79 → 0.14.0-dev.80

package/bin/postgres-ai.ts

@@ -20,7 +20,7 @@ import { maskSecret } from "../lib/util";
 import { createInterface } from "readline";
 import * as childProcess from "child_process";
 import { REPORT_GENERATORS, CHECK_INFO, generateAllReports } from "../lib/checkup";
-import { getAllCheckupEntries, getCheckupEntry, isValidCheckupCode } from "../lib/checkup-dictionary";
+import { getCheckupEntry } from "../lib/checkup-dictionary";
 import { createCheckupReport, uploadCheckupReportJson, RpcError, formatRpcErrorForDisplay, withRetry } from "../lib/checkup-api";
 
 // Singleton readline interface for stdin prompts
@@ -1642,14 +1642,6 @@ program
     }
   });
 
-// Build help text showing all checks from dictionary with express-mode indicators
-const expressCheckIds = new Set(Object.keys(REPORT_GENERATORS));
-const allChecks = getAllCheckupEntries();
-const checkHelpLines = allChecks.map((entry) => {
-  const isExpress = expressCheckIds.has(entry.code);
-  return `  ${entry.code}: ${entry.title}${isExpress ? "" : " (*)"}`;
-});
-
 program
   .command("checkup [conn]")
   .description("generate health check reports directly from PostgreSQL (express mode)")
@@ -1667,18 +1659,13 @@ program
     [
       "",
       "Available checks:",
-      ...checkHelpLines,
-      "",
-      "  (*) = not yet available in express mode (coming soon)",
-      "  ALL = run all express-mode checks",
+      ...Object.entries(CHECK_INFO).map(([id, title]) => `  ${id}: ${title}`),
       "",
       "Examples:",
       "  postgresai checkup postgresql://user:pass@host:5432/db",
-      "  postgresai checkup postgresql://user:pass@host:5432/db --check-id A003",
+      "  postgresai checkup postgresql://user:pass@host:5432/db --check-id D001",
       "  postgresai checkup postgresql://user:pass@host:5432/db --output ./reports",
       "  postgresai checkup postgresql://user:pass@host:5432/db --project my_project",
-      "  postgresai set-default-project my_project",
-      "  postgresai checkup postgresql://user:pass@host:5432/db",
       "  postgresai checkup postgresql://user:pass@host:5432/db --no-upload --json",
     ].join("\n")
   )

package/dist/bin/postgres-ai.js

@@ -13064,7 +13064,7 @@ var {
 // package.json
 var package_default = {
   name: "postgresai",
-  version: "0.14.0-dev.79",
+  version: "0.14.0-dev.80",
   description: "postgres_ai CLI",
   license: "Apache-2.0",
   private: false,
@@ -15889,7 +15889,7 @@ var Result = import_lib.default.Result;
 var TypeOverrides = import_lib.default.TypeOverrides;
 var defaults = import_lib.default.defaults;
 // package.json
-var version = "0.14.0-dev.79";
+var version = "0.14.0-dev.80";
 var package_default2 = {
   name: "postgresai",
   version,
@@ -27755,6 +27755,23 @@ async function generateD004(client, nodeName) {
   };
   return report;
 }
+async function generateD001(client, nodeName) {
+  const report = createBaseReport("D001", "Logging settings", nodeName);
+  const postgresVersion = await getPostgresVersion(client);
+  const pgMajorVersion = parseInt(postgresVersion.server_major_ver, 10) || 16;
+  const allSettings = await getSettings(client, pgMajorVersion);
+  const loggingSettings = {};
+  for (const [name, setting] of Object.entries(allSettings)) {
+    if (name.startsWith("log_") || name.startsWith("logging_")) {
+      loggingSettings[name] = setting;
+    }
+  }
+  report.results[nodeName] = {
+    data: loggingSettings,
+    postgres_version: postgresVersion
+  };
+  return report;
+}
 async function generateF001(client, nodeName) {
   const report = createBaseReport("F001", "Autovacuum: current settings", nodeName);
   const postgresVersion = await getPostgresVersion(client);
@@ -27861,6 +27878,7 @@ var REPORT_GENERATORS = {
   A004: generateA004,
   A007: generateA007,
   A013: generateA013,
+  D001: generateD001,
   D004: generateD004,
   F001: generateF001,
   G001: generateG001,
@@ -27897,9 +27915,6 @@ async function generateAllReports(client, nodeName = "node-01", onProgress) {
 
 // lib/checkup-dictionary.ts
 var dictionaryByCode2 = new Map(CHECKUP_DICTIONARY_DATA.map((entry) => [entry.code, entry]));
-function getAllCheckupEntries() {
-  return CHECKUP_DICTIONARY_DATA;
-}
 function getCheckupEntry(code) {
   return dictionaryByCode2.get(code.toUpperCase()) ?? null;
 }
@@ -29410,27 +29425,16 @@ program2.command("unprepare-db [conn]").description("remove monitoring setup: dr
     closeReadline();
   }
 });
-var expressCheckIds = new Set(Object.keys(REPORT_GENERATORS));
-var allChecks = getAllCheckupEntries();
-var checkHelpLines = allChecks.map((entry) => {
-  const isExpress = expressCheckIds.has(entry.code);
-  return `  ${entry.code}: ${entry.title}${isExpress ? "" : " (*)"}`;
-});
 program2.command("checkup [conn]").description("generate health check reports directly from PostgreSQL (express mode)").option("--check-id <id>", `specific check to run (see list below), or ALL`, "ALL").option("--node-name <name>", "node name for reports", "node-01").option("--output <path>", "output directory for JSON files").option("--[no-]upload", "upload JSON results to PostgresAI (default: enabled; requires API key)", undefined).option("--project <project>", "project name or ID for remote upload (used with --upload; defaults to config defaultProject; auto-generated on first run)").option("--json", "output JSON to stdout (implies --no-upload)").addHelpText("after", [
   "",
   "Available checks:",
-  ...checkHelpLines,
-  "",
-  "  (*) = not yet available in express mode (coming soon)",
-  "  ALL = run all express-mode checks",
+  ...Object.entries(CHECK_INFO).map(([id, title]) => `  ${id}: ${title}`),
   "",
   "Examples:",
   "  postgresai checkup postgresql://user:pass@host:5432/db",
-  "  postgresai checkup postgresql://user:pass@host:5432/db --check-id A003",
+  "  postgresai checkup postgresql://user:pass@host:5432/db --check-id D001",
   "  postgresai checkup postgresql://user:pass@host:5432/db --output ./reports",
   "  postgresai checkup postgresql://user:pass@host:5432/db --project my_project",
-  "  postgresai set-default-project my_project",
-  "  postgresai checkup postgresql://user:pass@host:5432/db",
   "  postgresai checkup postgresql://user:pass@host:5432/db --no-upload --json"
 ].join(`
 `)).action(async (conn, opts, cmd) => {

package/lib/checkup.ts

@@ -1186,6 +1186,37 @@ async function generateD004(client: Client, nodeName: string): Promise<Report> {
   return report;
 }
 
+/**
+ * Generate D001 report - Logging settings
+ *
+ * Collects all PostgreSQL logging-related settings including:
+ * - Log destination and collector settings
+ * - Log file rotation and naming
+ * - Log verbosity and filtering
+ * - Statement and duration logging
+ */
+async function generateD001(client: Client, nodeName: string): Promise<Report> {
+  const report = createBaseReport("D001", "Logging settings", nodeName);
+  const postgresVersion = await getPostgresVersion(client);
+  const pgMajorVersion = parseInt(postgresVersion.server_major_ver, 10) || 16;
+  const allSettings = await getSettings(client, pgMajorVersion);
+
+  // Filter logging-related settings (log_* and logging_*)
+  const loggingSettings: Record<string, SettingInfo> = {};
+  for (const [name, setting] of Object.entries(allSettings)) {
+    if (name.startsWith("log_") || name.startsWith("logging_")) {
+      loggingSettings[name] = setting;
+    }
+  }
+
+  report.results[nodeName] = {
+    data: loggingSettings,
+    postgres_version: postgresVersion,
+  };
+
+  return report;
+}
+
 /**
  * Generate F001 report - Autovacuum: current settings
  */
@@ -1336,6 +1367,7 @@ export const REPORT_GENERATORS: Record<string, (client: Client, nodeName: string
   A004: generateA004,
   A007: generateA007,
   A013: generateA013,
+  D001: generateD001,
   D004: generateD004,
   F001: generateF001,
   G001: generateG001,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postgresai",
-  "version": "0.14.0-dev.79",
+  "version": "0.14.0-dev.80",
   "description": "postgres_ai CLI",
   "license": "Apache-2.0",
   "private": false,

package/test/checkup.test.ts

@@ -86,7 +86,7 @@ describe("createBaseReport", () => {
 // Tests for CHECK_INFO
 describe("CHECK_INFO and REPORT_GENERATORS", () => {
   // Express-mode checks that have generators
-  const expressCheckIds = ["A002", "A003", "A004", "A007", "A013", "D004", "F001", "G001", "H001", "H002", "H004"];
+  const expressCheckIds = ["A002", "A003", "A004", "A007", "A013", "D001", "D004", "F001", "G001", "H001", "H002", "H004"];
 
   test("CHECK_INFO contains all express-mode checks", () => {
     for (const checkId of expressCheckIds) {