@kolatts/pncli 1.23.0 → 1.25.0

package/dist/cli.js

@@ -8,7 +8,7 @@ import {
   setRepoConfigValue,
   writeGlobalConfig,
   writeRepoConfig
-} from "./chunk-KRDKHGYS.js";
+} from "./chunk-JOAWUILY.js";
 import {
   ExitCode,
   PncliError,
@@ -20,13 +20,29 @@ import {
   success,
   warn,
   writeRawOutput
-} from "./chunk-XYHZMXJU.js";
+} from "./chunk-ABHLES5M.js";
 import {
   AdoCoreClient
 } from "./chunk-VDL32PSG.js";
 
 // src/cli.ts
 import { Command as Command3 } from "commander";
+
+// src/lib/proxyFetch.ts
+async function configureProxy() {
+  try {
+    const { setGlobalDispatcher, EnvHttpProxyAgent } = await import("undici");
+    setGlobalDispatcher(new EnvHttpProxyAgent());
+  } catch (err) {
+    const isModuleNotFound = err instanceof Error && "code" in err && err.code === "ERR_MODULE_NOT_FOUND";
+    if (!isModuleNotFound) {
+      process.stderr.write(`[pncli] Warning: failed to configure proxy: ${String(err)}
+`);
+    }
+  }
+}
+
+// src/cli.ts
 import { createRequire } from "module";
 
 // src/services/git/client.ts
@@ -778,6 +794,17 @@ var BitbucketClient = class {
       }
     );
   }
+  /**
+   * Returns true if `from` has commits not yet in `to` (i.e. there is something
+   * to promote), false if the branches are at the same point.
+   */
+  async compareCommits(project, repo, from, to) {
+    const result = await this.http.bitbucket(
+      `${API}/projects/${project}/repos/${repo}/commits`,
+      { params: { until: from, since: to, limit: 1 } }
+    );
+    return (result.values?.length ?? 0) > 0;
+  }
 };
 
 // src/services/git/commands.ts
@@ -1830,6 +1857,50 @@ function registerBitbucketCommands(program2) {
       fail(err, "bitbucket", "create-pr", start);
     }
   });
+  bb.command("create-prs").description("Create pull requests for multiple repositories in a project").requiredOption("--from <branch>", "Source branch").requiredOption("--to <branch>", "Target branch").requiredOption("--repos <slugs>", "Comma-separated repository slugs").option("--title <title>", 'PR title (defaults to "Promote <from> to <to>")').option("--description <desc>", "PR description").option("--reviewers <users>", "Comma-separated reviewer usernames").option("--project <key>", "Bitbucket project key (overrides parent --project)").action(async (opts) => {
+    const start = Date.now();
+    try {
+      const globalOpts = bb.optsWithGlobals();
+      const config = loadConfig({ configPath: globalOpts.config });
+      const http = createHttpClient(config, Boolean(globalOpts.dryRun));
+      const client = new BitbucketClient(http);
+      const ctx = getGitContext(config);
+      const project = opts.project ?? globalOpts.project ?? ctx?.project ?? config.defaults.bitbucket?.project ?? "";
+      if (!project) {
+        throw new PncliError("Could not determine Bitbucket project. Pass --project, or run pncli config init.", 1);
+      }
+      const repos = opts.repos.split(",").map((s) => s.trim()).filter(Boolean);
+      if (repos.length === 0) {
+        throw new PncliError("--repos must contain at least one repository slug", 1);
+      }
+      const title = opts.title ?? `Promote ${opts.from} to ${opts.to}`;
+      const reviewers = opts.reviewers ? opts.reviewers.split(",").map((s) => s.trim()) : [];
+      const results = [];
+      let created = 0;
+      let nothingToPromote = 0;
+      let errors = 0;
+      for (const repo of repos) {
+        try {
+          const hasCommits = await client.compareCommits(project, repo, opts.from, opts.to);
+          if (!hasCommits) {
+            results.push({ repo, status: "nothing_to_promote" });
+            nothingToPromote++;
+            continue;
+          }
+          const pr = await client.createPR({ project, repo, title, source: opts.from, target: opts.to, description: opts.description, reviewers });
+          results.push({ repo, status: "created", pr });
+          created++;
+        } catch (err) {
+          const message = err instanceof Error ? err.message : String(err);
+          results.push({ repo, status: "error", error: message });
+          errors++;
+        }
+      }
+      success({ results, summary: { created, nothing_to_promote: nothingToPromote, error: errors } }, "bitbucket", "create-prs", start);
+    } catch (err) {
+      fail(err, "bitbucket", "create-prs", start);
+    }
+  });
   bb.command("update-pr").description("Update a pull request").requiredOption("--id <pr-id>", "Pull request ID").option("--title <title>", "New title").option("--description <desc>", "New description").option("--reviewers <users>", "Comma-separated reviewer usernames").action(async (opts) => {
     const start = Date.now();
     try {
@@ -2235,12 +2306,84 @@ var GitHubClient = class {
       }
     });
   }
+  async listReviewThreads(owner, repo, pullNumber) {
+    const query = `
+      query($owner: String!, $repo: String!, $number: Int!) {
+        repository(owner: $owner, name: $repo) {
+          pullRequest(number: $number) {
+            reviewThreads(first: 100) {
+              pageInfo { hasNextPage }
+              nodes {
+                id
+                isResolved
+                isOutdated
+                path
+                line
+                comments(first: 10) {
+                  nodes {
+                    id
+                    databaseId
+                    body
+                    author { login }
+                    createdAt
+                  }
+                }
+              }
+            }
+          }
+        }
+      }
+    `;
+    const data = await this.http.githubGraphQL(query, { owner, repo, number: pullNumber });
+    const threads = data.repository.pullRequest.reviewThreads;
+    if (threads.pageInfo?.hasNextPage) {
+      process.stderr.write(`warning: more than ${threads.nodes.length} review threads on PR #${pullNumber}; only showing the first ${threads.nodes.length}
+`);
+    }
+    return threads.nodes;
+  }
+  async resolveReviewThread(threadId) {
+    const mutation = `
+      mutation($threadId: ID!) {
+        resolveReviewThread(input: { threadId: $threadId }) {
+          thread {
+            id
+            isResolved
+            isOutdated
+            path
+            line
+            comments(first: 10) {
+              nodes {
+                id
+                databaseId
+                body
+                author { login }
+                createdAt
+              }
+            }
+          }
+        }
+      }
+    `;
+    const data = await this.http.githubGraphQL(mutation, { threadId });
+    return data.resolveReviewThread.thread;
+  }
 };
 
 // src/services/github/commands.ts
 function getClient3(program2, overrides) {
   const opts = program2.optsWithGlobals();
   const config = loadConfig({ configPath: opts.config });
+  const missing = [];
+  if (!config.github.baseUrl) missing.push("github.baseUrl  (e.g. pncli config set github.baseUrl https://api.github.com)");
+  if (!config.github.token) missing.push("github.token    (e.g. pncli config set github.token <your-PAT>)");
+  if (missing.length) {
+    throw new PncliError(
+      `GitHub not configured. Missing required settings:
+  ${missing.join("\n  ")}`,
+      1
+    );
+  }
   const http = createHttpClient(config, Boolean(opts.dryRun));
   const client = new GitHubClient(http);
   const ctx = getGitContext(config);
@@ -2541,6 +2684,26 @@ function registerGitHubCommands(program2) {
       fail(err, "github", "list-reviewers", start);
     }
   });
+  gh.command("list-review-threads").description("List review threads on a pull request (includes thread node IDs for resolve-thread)").requiredOption("--number <n>", "Pull request number").action(async (opts) => {
+    const start = Date.now();
+    try {
+      const { client, owner, repo } = getClient3(gh);
+      const data = await client.listReviewThreads(owner, repo, parseInt(opts.number, 10));
+      success(data, "github", "list-review-threads", start);
+    } catch (err) {
+      fail(err, "github", "list-review-threads", start);
+    }
+  });
+  gh.command("resolve-thread").description("Mark a review thread as resolved (use list-review-threads to get the thread id)").requiredOption("--thread-id <id>", "GraphQL node ID of the review thread (e.g. PRT_kwDO...)").action(async (opts) => {
+    const start = Date.now();
+    try {
+      const { client } = getClient3(gh);
+      const data = await client.resolveReviewThread(opts.threadId);
+      success(data, "github", "resolve-thread", start);
+    } catch (err) {
+      fail(err, "github", "resolve-thread", start);
+    }
+  });
   gh.command("get-status").description("Get combined commit status for a ref (branch, tag, or SHA)").requiredOption("--ref <ref>", "Git ref (branch, tag, or commit SHA)").action(async (opts) => {
     const start = Date.now();
     try {
@@ -2778,6 +2941,28 @@ function xmlParseHint(err, bodyContent, convertedFromMarkdown = false) {
   return `Offending line ${row}${suffix}: ${line}
 ${pointer}`;
 }
+function storageFormatHint(err, bodyContent) {
+  if (!(err instanceof PncliError) || err.status !== 500) return null;
+  const hints = [];
+  if (/<ac:parameter[^>]*(?<![:\w])name\s*=/i.test(bodyContent)) {
+    hints.push(
+      '<ac:parameter name="..."> found \u2014 use ac:name="..." instead (e.g. <ac:parameter ac:name="language">text</ac:parameter>).'
+    );
+  }
+  const macroTagRe = /<ac:structured-macro(?:\s[^>]*)?\s*>/gi;
+  let macroMatch;
+  while ((macroMatch = macroTagRe.exec(bodyContent)) !== null) {
+    if (!/\bac:macro-id\b/.test(macroMatch[0])) {
+      hints.push(
+        '<ac:structured-macro> without ac:macro-id found \u2014 add a unique UUID: ac:macro-id="<uuid>" (e.g. ac:macro-id="a1b2c3d4-...").'
+      );
+      break;
+    }
+  }
+  if (hints.length === 0) return null;
+  return `HTTP 500 \u2014 possible storage-format causes:
+${hints.map((h) => `  \u2022 ${h}`).join("\n")}`;
+}
 function getClient4(program2) {
   const opts = program2.optsWithGlobals();
   const config = loadConfig({ configPath: opts.config });
@@ -2873,7 +3058,9 @@ function registerConfluenceCommands(program2) {
       });
       success(data, "confluence", "create-page", start);
     } catch (err) {
-      const hint = bodyContent ? xmlParseHint(err, bodyContent, Boolean(opts.markdown)) : null;
+      const xmlHint = bodyContent ? xmlParseHint(err, bodyContent, Boolean(opts.markdown)) : null;
+      const sfHint = bodyContent ? storageFormatHint(err, bodyContent) : null;
+      const hint = [xmlHint, sfHint].filter(Boolean).join("\n\n") || null;
       if (hint) {
         const msg = err instanceof Error ? err.message : String(err);
         const status = err instanceof PncliError ? err.status : 1;
@@ -2905,7 +3092,9 @@ ${hint}`, status, url), "confluence", "create-page", start);
       });
       success(data, "confluence", "update-page", start);
     } catch (err) {
-      const hint = bodyContent ? xmlParseHint(err, bodyContent, Boolean(opts.markdown)) : null;
+      const xmlHint = bodyContent ? xmlParseHint(err, bodyContent, Boolean(opts.markdown)) : null;
+      const sfHint = bodyContent ? storageFormatHint(err, bodyContent) : null;
+      const hint = [xmlHint, sfHint].filter(Boolean).join("\n\n") || null;
       if (hint) {
         const msg = err instanceof Error ? err.message : String(err);
         const status = err instanceof PncliError ? err.status : 1;
@@ -5724,7 +5913,7 @@ function registerConfigCommands(program2) {
       }
       if (cfg.checkmarx.baseUrl && cfg.checkmarx.tenantName && (cfg.checkmarx.apiKey || cfg.checkmarx.clientId && cfg.checkmarx.clientSecret)) {
         try {
-          await http.checkmarx("/api/projects", { params: { limit: 1 } });
+          await http.checkmarx("projects", { params: { limit: 1 } });
           results.checkmarx = { ok: true, message: "connected" };
         } catch (err) {
           results.checkmarx = { ok: false, message: err instanceof Error ? err.message : String(err) };
@@ -5966,7 +6155,7 @@ function registerConfigCommands(program2) {
         results.checkmarx = { status: "error", message: "baseUrl not configured" };
       } else {
         try {
-          await http.checkmarx("/api/projects", { params: { limit: 1 }, timeoutMs: 1e4 });
+          await http.checkmarx("projects", { params: { limit: 1 }, timeoutMs: 1e4 });
           results.checkmarx = { status: "valid", message: "ok" };
         } catch (err) {
           results.checkmarx = categorize2(err);
@@ -7979,11 +8168,11 @@ function registerAdoRepoCommands(ado) {
       const start = Date.now();
       try {
         const globalOpts = ado.optsWithGlobals();
-        const { loadConfig: lc } = await import("./config-QN5FFAZY.js");
+        const { loadConfig: lc } = await import("./config-SQ3YHJHI.js");
         const config = lc({ configPath: globalOpts.config });
         const { collection, project, repo: repo2, gitClient } = getAdoContext(ado, true);
         const { AdoCoreClient: AdoCoreClient2 } = await import("./core-SWYVXOQ4.js");
-        const { createHttpClient: createHttpClient2 } = await import("./http-MCIU2IV6.js");
+        const { createHttpClient: createHttpClient2 } = await import("./http-WCK6AZW6.js");
         const http = createHttpClient2(config, Boolean(globalOpts.dryRun));
         const coreClient = new AdoCoreClient2(http);
         const me = await coreClient.getConnectionData(collection);
@@ -10752,6 +10941,7 @@ function registerLogscaleCommands(program2) {
 if (!process.env.PNCLI_VERIFY_TLS) {
   process.env.NODE_TLS_REJECT_UNAUTHORIZED = "0";
 }
+await configureProxy();
 var require2 = createRequire(import.meta.url);
 var pkg = require2("../package.json");
 var TAGLINE = "One command does what three meetings couldn't.";