postgresai 0.12.0-beta.6 → 0.14.0-dev.10

package/dist/bin/postgres-ai.js

@@ -49,6 +49,7 @@ const url_1 = require("url");
 const mcp_server_1 = require("../lib/mcp-server");
 const issues_1 = require("../lib/issues");
 const util_2 = require("../lib/util");
+const init_1 = require("../lib/init");
 const execPromise = (0, util_1.promisify)(child_process_1.exec);
 const execFilePromise = (0, util_1.promisify)(child_process_1.execFile);
 /**
@@ -71,6 +72,25 @@ function getConfig(opts) {
     }
     return { apiKey };
 }
+// Human-friendly output helper: YAML for TTY by default, JSON when --json or non-TTY
+function printResult(result, json) {
+    if (typeof result === "string") {
+        process.stdout.write(result);
+        if (!/\n$/.test(result))
+            console.log();
+        return;
+    }
+    if (json || !process.stdout.isTTY) {
+        console.log(JSON.stringify(result, null, 2));
+    }
+    else {
+        let text = yaml.dump(result);
+        if (Array.isArray(result)) {
+            text = text.replace(/\n- /g, "\n\n- ");
+        }
+        console.log(text);
+    }
+}
 const program = new commander_1.Command();
 program
     .name("postgres-ai")
@@ -79,6 +99,163 @@ program
     .option("--api-key <key>", "API key (overrides PGAI_API_KEY)")
     .option("--api-base-url <url>", "API base URL for backend RPC (overrides PGAI_API_BASE_URL)")
     .option("--ui-base-url <url>", "UI base URL for browser routes (overrides PGAI_UI_BASE_URL)");
+program
+    .command("init [conn]")
+    .description("Create a monitoring user and grant all required permissions (idempotent)")
+    .option("--db-url <url>", "PostgreSQL connection URL (admin) to run the setup against (deprecated; pass it as positional arg)")
+    .option("-h, --host <host>", "PostgreSQL host (psql-like)")
+    .option("-p, --port <port>", "PostgreSQL port (psql-like)")
+    .option("-U, --username <username>", "PostgreSQL user (psql-like)")
+    .option("-d, --dbname <dbname>", "PostgreSQL database name (psql-like)")
+    .option("--admin-password <password>", "Admin connection password (otherwise uses PGPASSWORD if set)")
+    .option("--monitoring-user <name>", "Monitoring role name to create/update", "postgres_ai_mon")
+    .option("--password <password>", "Monitoring role password (overrides PGAI_MON_PASSWORD)")
+    .option("--skip-optional-permissions", "Skip optional permissions (RDS/self-managed extras)", false)
+    .option("--print-sql", "Print SQL steps before applying (passwords redacted by default)", false)
+    .option("--show-secrets", "When printing SQL, do not redact secrets (DANGEROUS)", false)
+    .option("--dry-run", "Print SQL steps and exit without applying changes", false)
+    .action(async (conn, opts) => {
+    let adminConn;
+    try {
+        adminConn = (0, init_1.resolveAdminConnection)({
+            conn,
+            dbUrlFlag: opts.dbUrl,
+            host: opts.host,
+            port: opts.port,
+            username: opts.username,
+            dbname: opts.dbname,
+            adminPassword: opts.adminPassword,
+            envPassword: process.env.PGPASSWORD,
+        });
+    }
+    catch (e) {
+        const msg = e instanceof Error ? e.message : String(e);
+        console.error(`✗ ${msg}`);
+        process.exitCode = 1;
+        return;
+    }
+    const includeOptionalPermissions = !opts.skipOptionalPermissions;
+    console.log(`Connecting to: ${adminConn.display}`);
+    console.log(`Monitoring user: ${opts.monitoringUser}`);
+    console.log(`Optional permissions: ${includeOptionalPermissions ? "enabled" : "skipped"}`);
+    const shouldPrintSql = !!opts.printSql || !!opts.dryRun;
+    // Use native pg client instead of requiring psql to be installed
+    const { Client } = require("pg");
+    const client = new Client(adminConn.clientConfig);
+    try {
+        await client.connect();
+        const roleRes = await client.query("select 1 from pg_catalog.pg_roles where rolname = $1", [
+            opts.monitoringUser,
+        ]);
+        const roleExists = roleRes.rowCount > 0;
+        const dbRes = await client.query("select current_database() as db");
+        const database = dbRes.rows?.[0]?.db;
+        if (typeof database !== "string" || !database) {
+            throw new Error("Failed to resolve current database name");
+        }
+        let monPassword;
+        try {
+            const resolved = await (0, init_1.resolveMonitoringPassword)({
+                passwordFlag: opts.password,
+                passwordEnv: process.env.PGAI_MON_PASSWORD,
+                monitoringUser: opts.monitoringUser,
+            });
+            monPassword = resolved.password;
+            if (resolved.generated) {
+                console.log(`Generated password for monitoring user ${opts.monitoringUser}: ${monPassword}`);
+                console.log("Store it securely (or rerun with --password / PGAI_MON_PASSWORD to set your own).");
+            }
+        }
+        catch (e) {
+            const msg = e instanceof Error ? e.message : String(e);
+            console.error(`✗ ${msg}`);
+            process.exitCode = 1;
+            return;
+        }
+        const plan = await (0, init_1.buildInitPlan)({
+            database,
+            monitoringUser: opts.monitoringUser,
+            monitoringPassword: monPassword,
+            includeOptionalPermissions,
+            roleExists,
+        });
+        if (shouldPrintSql) {
+            const redact = !opts.showSecrets;
+            const redactPasswords = (sql) => {
+                if (!redact)
+                    return sql;
+                // Replace PASSWORD '<literal>' (handles doubled quotes inside).
+                return sql.replace(/password\s+'(?:''|[^'])*'/gi, "password '<redacted>'");
+            };
+            console.log("\n--- SQL plan ---");
+            for (const step of plan.steps) {
+                console.log(`\n-- ${step.name}${step.optional ? " (optional)" : ""}`);
+                console.log(redactPasswords(step.sql));
+            }
+            console.log("\n--- end SQL plan ---\n");
+            if (redact) {
+                console.log("Note: passwords are redacted in the printed SQL (use --show-secrets to print them).");
+            }
+        }
+        if (opts.dryRun) {
+            console.log("✓ dry-run completed (no changes were applied)");
+            return;
+        }
+        const { applied, skippedOptional } = await (0, init_1.applyInitPlan)({ client, plan });
+        console.log("✓ init completed");
+        if (skippedOptional.length > 0) {
+            console.log("⚠ Some optional steps were skipped (not supported or insufficient privileges):");
+            for (const s of skippedOptional)
+                console.log(`- ${s}`);
+        }
+        // Keep output compact but still useful
+        if (process.stdout.isTTY) {
+            console.log(`Applied ${applied.length} steps`);
+        }
+    }
+    catch (error) {
+        const errAny = error;
+        let message = "";
+        if (error instanceof Error && error.message) {
+            message = error.message;
+        }
+        else if (errAny && typeof errAny === "object" && typeof errAny.message === "string" && errAny.message) {
+            message = errAny.message;
+        }
+        else {
+            message = String(error);
+        }
+        if (!message || message === "[object Object]") {
+            message = "Unknown error";
+        }
+        console.error(`✗ init failed: ${message}`);
+        if (errAny && typeof errAny === "object") {
+            if (typeof errAny.code === "string" && errAny.code) {
+                console.error(`Error code: ${errAny.code}`);
+            }
+            if (typeof errAny.detail === "string" && errAny.detail) {
+                console.error(`Detail: ${errAny.detail}`);
+            }
+            if (typeof errAny.hint === "string" && errAny.hint) {
+                console.error(`Hint: ${errAny.hint}`);
+            }
+        }
+        if (errAny && typeof errAny === "object" && typeof errAny.code === "string") {
+            if (errAny.code === "42501") {
+                console.error("Hint: connect as a superuser (or a role with CREATEROLE and sufficient GRANT privileges).");
+            }
+        }
+        process.exitCode = 1;
+    }
+    finally {
+        try {
+            await client.end();
+        }
+        catch {
+            // ignore
+        }
+    }
+});
 /**
  * Stub function for not implemented commands
  */
@@ -206,23 +383,284 @@ const mon = program.command("mon").description("monitoring services management")
 mon
     .command("quickstart")
     .description("complete setup (generate config, start monitoring services)")
-    .option("--demo", "demo mode", false)
-    .action(async () => {
+    .option("--demo", "demo mode with sample database", false)
+    .option("--api-key <key>", "Postgres AI API key for automated report uploads")
+    .option("--db-url <url>", "PostgreSQL connection URL to monitor")
+    .option("-y, --yes", "accept all defaults and skip interactive prompts", false)
+    .action(async (opts) => {
+    console.log("\n=================================");
+    console.log("  PostgresAI Monitoring Quickstart");
+    console.log("=================================\n");
+    console.log("This will install, configure, and start the monitoring system\n");
+    // Validate conflicting options
+    if (opts.demo && opts.dbUrl) {
+        console.log("⚠ Both --demo and --db-url provided. Demo mode includes its own database.");
+        console.log("⚠ The --db-url will be ignored in demo mode.\n");
+        opts.dbUrl = undefined;
+    }
+    if (opts.demo && opts.apiKey) {
+        console.error("✗ Cannot use --api-key with --demo mode");
+        console.error("✗ Demo mode is for testing only and does not support API key integration");
+        console.error("\nUse demo mode without API key: postgres-ai mon quickstart --demo");
+        console.error("Or use production mode with API key: postgres-ai mon quickstart --api-key=your_key");
+        process.exitCode = 1;
+        return;
+    }
     // Check if containers are already running
     const { running, containers } = checkRunningContainers();
     if (running) {
-        console.log(`Monitoring services are already running: ${containers.join(", ")}`);
-        console.log("Use 'postgres-ai mon restart' to restart them");
+        console.log(`⚠ Monitoring services are already running: ${containers.join(", ")}`);
+        console.log("Use 'postgres-ai mon restart' to restart them\n");
         return;
     }
+    // Step 1: API key configuration (only in production mode)
+    if (!opts.demo) {
+        console.log("Step 1: Postgres AI API Configuration (Optional)");
+        console.log("An API key enables automatic upload of PostgreSQL reports to Postgres AI\n");
+        if (opts.apiKey) {
+            console.log("Using API key provided via --api-key parameter");
+            config.writeConfig({ apiKey: opts.apiKey });
+            console.log("✓ API key saved\n");
+        }
+        else if (opts.yes) {
+            // Auto-yes mode without API key - skip API key setup
+            console.log("Auto-yes mode: no API key provided, skipping API key setup");
+            console.log("⚠ Reports will be generated locally only");
+            console.log("You can add an API key later with: postgres-ai add-key <api_key>\n");
+        }
+        else {
+            const rl = readline.createInterface({
+                input: process.stdin,
+                output: process.stdout
+            });
+            const question = (prompt) => new Promise((resolve) => rl.question(prompt, resolve));
+            try {
+                const answer = await question("Do you have a Postgres AI API key? (Y/n): ");
+                const proceedWithApiKey = !answer || answer.toLowerCase() === "y";
+                if (proceedWithApiKey) {
+                    while (true) {
+                        const inputApiKey = await question("Enter your Postgres AI API key: ");
+                        const trimmedKey = inputApiKey.trim();
+                        if (trimmedKey) {
+                            config.writeConfig({ apiKey: trimmedKey });
+                            console.log("✓ API key saved\n");
+                            break;
+                        }
+                        console.log("⚠ API key cannot be empty");
+                        const retry = await question("Try again or skip API key setup, retry? (Y/n): ");
+                        if (retry.toLowerCase() === "n") {
+                            console.log("⚠ Skipping API key setup - reports will be generated locally only");
+                            console.log("You can add an API key later with: postgres-ai add-key <api_key>\n");
+                            break;
+                        }
+                    }
+                }
+                else {
+                    console.log("⚠ Skipping API key setup - reports will be generated locally only");
+                    console.log("You can add an API key later with: postgres-ai add-key <api_key>\n");
+                }
+            }
+            finally {
+                rl.close();
+            }
+        }
+    }
+    else {
+        console.log("Step 1: Demo mode - API key configuration skipped");
+        console.log("Demo mode is for testing only and does not support API key integration\n");
+    }
+    // Step 2: Add PostgreSQL instance (if not demo mode)
+    if (!opts.demo) {
+        console.log("Step 2: Add PostgreSQL Instance to Monitor\n");
+        // Clear instances.yml in production mode (start fresh)
+        const instancesPath = path.resolve(process.cwd(), "instances.yml");
+        const emptyInstancesContent = "# PostgreSQL instances to monitor\n# Add your instances using: postgres-ai mon targets add\n\n";
+        fs.writeFileSync(instancesPath, emptyInstancesContent, "utf8");
+        if (opts.dbUrl) {
+            console.log("Using database URL provided via --db-url parameter");
+            console.log(`Adding PostgreSQL instance from: ${opts.dbUrl}\n`);
+            const match = opts.dbUrl.match(/^postgresql:\/\/[^@]+@([^:/]+)/);
+            const autoInstanceName = match ? match[1] : "db-instance";
+            const connStr = opts.dbUrl;
+            const m = connStr.match(/^postgresql:\/\/([^:]+):([^@]+)@([^:\/]+)(?::(\d+))?\/(.+)$/);
+            if (!m) {
+                console.error("✗ Invalid connection string format");
+                process.exitCode = 1;
+                return;
+            }
+            const host = m[3];
+            const db = m[5];
+            const instanceName = `${host}-${db}`.replace(/[^a-zA-Z0-9-]/g, "-");
+            const body = `- name: ${instanceName}\n  conn_str: ${connStr}\n  preset_metrics: full\n  custom_metrics:\n  is_enabled: true\n  group: default\n  custom_tags:\n    env: production\n    cluster: default\n    node_name: ${instanceName}\n    sink_type: ~sink_type~\n`;
+            fs.appendFileSync(instancesPath, body, "utf8");
+            console.log(`✓ Monitoring target '${instanceName}' added\n`);
+            // Test connection
+            console.log("Testing connection to the added instance...");
+            try {
+                const { Client } = require("pg");
+                const client = new Client({ connectionString: connStr });
+                await client.connect();
+                const result = await client.query("select version();");
+                console.log("✓ Connection successful");
+                console.log(`${result.rows[0].version}\n`);
+                await client.end();
+            }
+            catch (error) {
+                const message = error instanceof Error ? error.message : String(error);
+                console.error(`✗ Connection failed: ${message}\n`);
+            }
+        }
+        else if (opts.yes) {
+            // Auto-yes mode without database URL - skip database setup
+            console.log("Auto-yes mode: no database URL provided, skipping database setup");
+            console.log("⚠ No PostgreSQL instance added");
+            console.log("You can add one later with: postgres-ai mon targets add\n");
+        }
+        else {
+            const rl = readline.createInterface({
+                input: process.stdin,
+                output: process.stdout
+            });
+            const question = (prompt) => new Promise((resolve) => rl.question(prompt, resolve));
+            try {
+                console.log("You need to add at least one PostgreSQL instance to monitor");
+                const answer = await question("Do you want to add a PostgreSQL instance now? (Y/n): ");
+                const proceedWithInstance = !answer || answer.toLowerCase() === "y";
+                if (proceedWithInstance) {
+                    console.log("\nYou can provide either:");
+                    console.log("  1. A full connection string: postgresql://user:pass@host:port/database");
+                    console.log("  2. Press Enter to skip for now\n");
+                    const connStr = await question("Enter connection string (or press Enter to skip): ");
+                    if (connStr.trim()) {
+                        const m = connStr.match(/^postgresql:\/\/([^:]+):([^@]+)@([^:\/]+)(?::(\d+))?\/(.+)$/);
+                        if (!m) {
+                            console.error("✗ Invalid connection string format");
+                            console.log("⚠ Continuing without adding instance\n");
+                        }
+                        else {
+                            const host = m[3];
+                            const db = m[5];
+                            const instanceName = `${host}-${db}`.replace(/[^a-zA-Z0-9-]/g, "-");
+                            const body = `- name: ${instanceName}\n  conn_str: ${connStr}\n  preset_metrics: full\n  custom_metrics:\n  is_enabled: true\n  group: default\n  custom_tags:\n    env: production\n    cluster: default\n    node_name: ${instanceName}\n    sink_type: ~sink_type~\n`;
+                            fs.appendFileSync(instancesPath, body, "utf8");
+                            console.log(`✓ Monitoring target '${instanceName}' added\n`);
+                            // Test connection
+                            console.log("Testing connection to the added instance...");
+                            try {
+                                const { Client } = require("pg");
+                                const client = new Client({ connectionString: connStr });
+                                await client.connect();
+                                const result = await client.query("select version();");
+                                console.log("✓ Connection successful");
+                                console.log(`${result.rows[0].version}\n`);
+                                await client.end();
+                            }
+                            catch (error) {
+                                const message = error instanceof Error ? error.message : String(error);
+                                console.error(`✗ Connection failed: ${message}\n`);
+                            }
+                        }
+                    }
+                    else {
+                        console.log("⚠ No PostgreSQL instance added - you can add one later with: postgres-ai mon targets add\n");
+                    }
+                }
+                else {
+                    console.log("⚠ No PostgreSQL instance added - you can add one later with: postgres-ai mon targets add\n");
+                }
+            }
+            finally {
+                rl.close();
+            }
+        }
+    }
+    else {
+        console.log("Step 2: Demo mode enabled - using included demo PostgreSQL database\n");
+    }
+    // Step 3: Update configuration
+    console.log(opts.demo ? "Step 3: Updating configuration..." : "Step 3: Updating configuration...");
     const code1 = await runCompose(["run", "--rm", "sources-generator"]);
     if (code1 !== 0) {
         process.exitCode = code1;
         return;
     }
-    const code2 = await runCompose(["up", "-d"]);
-    if (code2 !== 0)
+    console.log("✓ Configuration updated\n");
+    // Step 4: Ensure Grafana password is configured
+    console.log(opts.demo ? "Step 4: Configuring Grafana security..." : "Step 4: Configuring Grafana security...");
+    const cfgPath = path.resolve(process.cwd(), ".pgwatch-config");
+    let grafanaPassword = "";
+    try {
+        if (fs.existsSync(cfgPath)) {
+            const stats = fs.statSync(cfgPath);
+            if (!stats.isDirectory()) {
+                const content = fs.readFileSync(cfgPath, "utf8");
+                const match = content.match(/^grafana_password=([^\r\n]+)/m);
+                if (match) {
+                    grafanaPassword = match[1].trim();
+                }
+            }
+        }
+        if (!grafanaPassword) {
+            console.log("Generating secure Grafana password...");
+            const { stdout: password } = await execPromise("openssl rand -base64 12 | tr -d '\n'");
+            grafanaPassword = password.trim();
+            let configContent = "";
+            if (fs.existsSync(cfgPath)) {
+                const stats = fs.statSync(cfgPath);
+                if (!stats.isDirectory()) {
+                    configContent = fs.readFileSync(cfgPath, "utf8");
+                }
+            }
+            const lines = configContent.split(/\r?\n/).filter((l) => !/^grafana_password=/.test(l));
+            lines.push(`grafana_password=${grafanaPassword}`);
+            fs.writeFileSync(cfgPath, lines.filter(Boolean).join("\n") + "\n", "utf8");
+        }
+        console.log("✓ Grafana password configured\n");
+    }
+    catch (error) {
+        console.log("⚠ Could not generate Grafana password automatically");
+        console.log("Using default password: demo\n");
+        grafanaPassword = "demo";
+    }
+    // Step 5: Start services
+    console.log(opts.demo ? "Step 5: Starting monitoring services..." : "Step 5: Starting monitoring services...");
+    const code2 = await runCompose(["up", "-d", "--force-recreate"]);
+    if (code2 !== 0) {
         process.exitCode = code2;
+        return;
+    }
+    console.log("✓ Services started\n");
+    // Final summary
+    console.log("=================================");
+    console.log("  🎉 Quickstart setup completed!");
+    console.log("=================================\n");
+    console.log("What's running:");
+    if (opts.demo) {
+        console.log("  ✅ Demo PostgreSQL database (monitoring target)");
+    }
+    console.log("  ✅ PostgreSQL monitoring infrastructure");
+    console.log("  ✅ Grafana dashboards (with secure password)");
+    console.log("  ✅ Prometheus metrics storage");
+    console.log("  ✅ Flask API backend");
+    console.log("  ✅ Automated report generation (every 24h)");
+    console.log("  ✅ Host stats monitoring (CPU, memory, disk, I/O)\n");
+    if (!opts.demo) {
+        console.log("Next steps:");
+        console.log("  • Add more PostgreSQL instances: postgres-ai mon targets add");
+        console.log("  • View configured instances: postgres-ai mon targets list");
+        console.log("  • Check service health: postgres-ai mon health\n");
+    }
+    else {
+        console.log("Demo mode next steps:");
+        console.log("  • Explore Grafana dashboards at http://localhost:3000");
+        console.log("  • Connect to demo database: postgresql://postgres:postgres@localhost:55432/target_database");
+        console.log("  • Generate some load on the demo database to see metrics\n");
+    }
+    console.log("━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━");
+    console.log("🚀 MAIN ACCESS POINT - Start here:");
+    console.log("   Grafana Dashboard: http://localhost:3000");
+    console.log(`   Login: monitor / ${grafanaPassword}`);
+    console.log("━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n");
 });
 mon
     .command("start")
@@ -289,10 +727,12 @@ mon
     .option("--wait <seconds>", "wait time in seconds for services to become healthy", parseInt, 0)
     .action(async (opts) => {
     const services = [
-        { name: "Grafana", url: "http://localhost:3000/api/health" },
-        { name: "Prometheus", url: "http://localhost:59090/-/healthy" },
-        { name: "PGWatch (Postgres)", url: "http://localhost:58080/health" },
-        { name: "PGWatch (Prometheus)", url: "http://localhost:58089/health" },
+        { name: "Grafana", container: "grafana-with-datasources" },
+        { name: "Prometheus", container: "sink-prometheus" },
+        { name: "PGWatch (Postgres)", container: "pgwatch-postgres" },
+        { name: "PGWatch (Prometheus)", container: "pgwatch-prometheus" },
+        { name: "Target DB", container: "target-db" },
+        { name: "Sink Postgres", container: "sink-postgres" },
     ];
     const waitTime = opts.wait || 0;
     const maxAttempts = waitTime > 0 ? Math.ceil(waitTime / 5) : 1;
@@ -306,19 +746,16 @@ mon
         allHealthy = true;
         for (const service of services) {
             try {
-                // Use native fetch instead of requiring curl to be installed
-                const controller = new AbortController();
-                const timeoutId = setTimeout(() => controller.abort(), 5000);
-                const response = await fetch(service.url, {
-                    signal: controller.signal,
-                    method: 'GET',
-                });
-                clearTimeout(timeoutId);
-                if (response.status === 200) {
+                const { execSync } = require("child_process");
+                const status = execSync(`docker inspect -f '{{.State.Status}}' ${service.container} 2>/dev/null`, {
+                    encoding: 'utf8',
+                    stdio: ['pipe', 'pipe', 'pipe']
+                }).trim();
+                if (status === 'running') {
                     console.log(`✓ ${service.name}: healthy`);
                 }
                 else {
-                    console.log(`✗ ${service.name}: unhealthy (HTTP ${response.status})`);
+                    console.log(`✗ ${service.name}: unhealthy (status: ${status})`);
                     allHealthy = false;
                 }
             }
@@ -1041,12 +1478,30 @@ mon
     console.log(`  Password: ${password}`);
     console.log("");
 });
+/**
+ * Interpret escape sequences in a string (e.g., \n -> newline)
+ * Note: In regex, to match literal backslash-n, we need \\n in the pattern
+ * which requires \\\\n in the JavaScript string literal
+ */
+function interpretEscapes(str) {
+    // First handle double backslashes by temporarily replacing them
+    // Then handle other escapes, then restore double backslashes as single
+    return str
+        .replace(/\\\\/g, '\x00') // Temporarily mark double backslashes
+        .replace(/\\n/g, '\n') // Match literal backslash-n (\\\\n in JS string -> \\n in regex -> matches \n)
+        .replace(/\\t/g, '\t')
+        .replace(/\\r/g, '\r')
+        .replace(/\\"/g, '"')
+        .replace(/\\'/g, "'")
+        .replace(/\x00/g, '\\'); // Restore double backslashes as single
+}
 // Issues management
 const issues = program.command("issues").description("issues management");
 issues
     .command("list")
     .description("list issues")
     .option("--debug", "enable debug output")
+    .option("--json", "output raw JSON")
     .action(async (opts) => {
     try {
         const rootOpts = program.opts();
@@ -1059,14 +1514,90 @@ issues
         }
         const { apiBaseUrl } = (0, util_2.resolveBaseUrls)(rootOpts, cfg);
         const result = await (0, issues_1.fetchIssues)({ apiKey, apiBaseUrl, debug: !!opts.debug });
-        if (typeof result === "string") {
-            process.stdout.write(result);
-            if (!/\n$/.test(result))
-                console.log();
+        const trimmed = Array.isArray(result)
+            ? result.map((r) => ({
+                id: r.id,
+                title: r.title,
+                status: r.status,
+                created_at: r.created_at,
+            }))
+            : result;
+        printResult(trimmed, opts.json);
+    }
+    catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        console.error(message);
+        process.exitCode = 1;
+    }
+});
+issues
+    .command("view <issueId>")
+    .description("view issue details and comments")
+    .option("--debug", "enable debug output")
+    .option("--json", "output raw JSON")
+    .action(async (issueId, opts) => {
+    try {
+        const rootOpts = program.opts();
+        const cfg = config.readConfig();
+        const { apiKey } = getConfig(rootOpts);
+        if (!apiKey) {
+            console.error("API key is required. Run 'pgai auth' first or set --api-key.");
+            process.exitCode = 1;
+            return;
         }
-        else {
-            console.log(JSON.stringify(result, null, 2));
+        const { apiBaseUrl } = (0, util_2.resolveBaseUrls)(rootOpts, cfg);
+        const issue = await (0, issues_1.fetchIssue)({ apiKey, apiBaseUrl, issueId, debug: !!opts.debug });
+        if (!issue) {
+            console.error("Issue not found");
+            process.exitCode = 1;
+            return;
+        }
+        const comments = await (0, issues_1.fetchIssueComments)({ apiKey, apiBaseUrl, issueId, debug: !!opts.debug });
+        const combined = { issue, comments };
+        printResult(combined, opts.json);
+    }
+    catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        console.error(message);
+        process.exitCode = 1;
+    }
+});
+issues
+    .command("post_comment <issueId> <content>")
+    .description("post a new comment to an issue")
+    .option("--parent <uuid>", "parent comment id")
+    .option("--debug", "enable debug output")
+    .option("--json", "output raw JSON")
+    .action(async (issueId, content, opts) => {
+    try {
+        // Interpret escape sequences in content (e.g., \n -> newline)
+        if (opts.debug) {
+            // eslint-disable-next-line no-console
+            console.log(`Debug: Original content: ${JSON.stringify(content)}`);
+        }
+        content = interpretEscapes(content);
+        if (opts.debug) {
+            // eslint-disable-next-line no-console
+            console.log(`Debug: Interpreted content: ${JSON.stringify(content)}`);
         }
+        const rootOpts = program.opts();
+        const cfg = config.readConfig();
+        const { apiKey } = getConfig(rootOpts);
+        if (!apiKey) {
+            console.error("API key is required. Run 'pgai auth' first or set --api-key.");
+            process.exitCode = 1;
+            return;
+        }
+        const { apiBaseUrl } = (0, util_2.resolveBaseUrls)(rootOpts, cfg);
+        const result = await (0, issues_1.createIssueComment)({
+            apiKey,
+            apiBaseUrl,
+            issueId,
+            content,
+            parentCommentId: opts.parent,
+            debug: !!opts.debug,
+        });
+        printResult(result, opts.json);
     }
     catch (err) {
         const message = err instanceof Error ? err.message : String(err);