jinzd-ai-cli 0.4.215 → 0.4.217

package/dist/pr-KPQ5RPKB.js

@@ -0,0 +1,267 @@
+#!/usr/bin/env node
+import {
+  countSeverity,
+  runCi
+} from "./chunk-KOPUCJXM.js";
+import "./chunk-WZ3VKLF3.js";
+import {
+  buildReviewPrompt,
+  buildSecurityReviewPrompt,
+  truncateDiff
+} from "./chunk-HLWUDRBO.js";
+import {
+  ProviderRegistry
+} from "./chunk-QMXC327F.js";
+import "./chunk-XPBEJB27.js";
+import {
+  ConfigManager
+} from "./chunk-WKOQ5CYC.js";
+import "./chunk-TZQHYZKT.js";
+import {
+  VERSION
+} from "./chunk-XJGEQIYS.js";
+import "./chunk-IW3Q7AE5.js";
+
+// src/cli/pr.ts
+import { execFileSync } from "child_process";
+var ALLOWED_AGENTS = /* @__PURE__ */ new Set(["security", "bugs", "tests", "maintainability"]);
+function parsePrTarget(input) {
+  if (!input) return {};
+  const s = input.trim();
+  if (/^https:\/\/github\.com\/[^/]+\/[^/]+\/pull\/\d+(?:[/?#].*)?$/.test(s)) return { pr: s };
+  if (/^#?\d+$/.test(s)) return { pr: parseInt(s.replace(/^#/, ""), 10) };
+  const range = s.match(/^([^\s.][^\s]*?)\.\.\.([^\s]+)$/);
+  if (range) return { base: range[1], head: range[2] };
+  return { base: s, head: "HEAD" };
+}
+function parseAgentList(input) {
+  if (!input) return [];
+  const agents = input.split(",").map((s) => s.trim()).filter(Boolean);
+  const invalid = agents.filter((a) => !ALLOWED_AGENTS.has(a));
+  if (invalid.length) throw new Error(`unknown PR review agent(s): ${invalid.join(", ")}. Allowed: ${Array.from(ALLOWED_AGENTS).join(", ")}`);
+  return [...new Set(agents)];
+}
+function validateGitRef(ref) {
+  if (!/^[A-Za-z0-9._/\-]+$/.test(ref)) throw new Error(`unsafe ref: ${ref}`);
+}
+function hasGitHubToken() {
+  return !!(process.env.GH_TOKEN || process.env.GITHUB_TOKEN);
+}
+function fetchPrMetadata(pr) {
+  if (!hasGitHubToken()) return null;
+  try {
+    const out = execFileSync("gh", ["pr", "view", String(pr), "--json", "number,url,title,author,state,baseRefName,headRefName"], {
+      encoding: "utf-8",
+      maxBuffer: 5 * 1024 * 1024,
+      timeout: 3e4
+    });
+    const parsed = JSON.parse(out);
+    return {
+      number: parsed.number,
+      url: parsed.url,
+      title: parsed.title,
+      state: parsed.state,
+      baseRefName: parsed.baseRefName,
+      headRefName: parsed.headRefName,
+      author: parsed.author?.login
+    };
+  } catch {
+    return null;
+  }
+}
+function fetchDiffSource(opts) {
+  const target = parsePrTarget(opts.target);
+  if (opts.diffOverride != null) {
+    const isPr = target.pr != null;
+    return {
+      kind: isPr ? "github-pr" : "git",
+      label: isPr ? `PR ${String(target.pr)}` : opts.base ? `${opts.base}...${opts.head || "HEAD"}` : "injected diff",
+      diff: opts.diffOverride,
+      metadata: opts.metadataOverride ?? null
+    };
+  }
+  const pr = target.pr;
+  const base = opts.base ?? target.base ?? "main";
+  const head = opts.head ?? target.head ?? "HEAD";
+  if (pr != null) {
+    try {
+      const diff = execFileSync("gh", ["pr", "diff", String(pr)], {
+        encoding: "utf-8",
+        maxBuffer: 50 * 1024 * 1024,
+        timeout: 6e4
+      });
+      return { kind: "github-pr", label: `PR ${String(pr)}`, diff, metadata: fetchPrMetadata(pr) };
+    } catch (err) {
+      const msg = err.stderr?.toString() ?? err.message;
+      throw new Error(`gh pr diff ${String(pr)} failed: ${msg.trim() || "unknown error"}. For local review without GitHub auth, use --base/--head instead.`);
+    }
+  }
+  validateGitRef(base);
+  validateGitRef(head);
+  try {
+    const diff = execFileSync("git", ["diff", `${base}...${head}`], {
+      encoding: "utf-8",
+      maxBuffer: 50 * 1024 * 1024,
+      timeout: 3e4
+    });
+    return { kind: "git", label: `${base}...${head}`, diff, metadata: null };
+  } catch (err) {
+    throw new Error(`git diff ${base}...${head} failed: ${err.message}`);
+  }
+}
+function metadataMarkdown(meta) {
+  if (!meta) return "";
+  const parts = [
+    meta.number != null ? `#${meta.number}` : void 0,
+    meta.title,
+    meta.state,
+    meta.author ? `by @${meta.author}` : void 0,
+    meta.baseRefName && meta.headRefName ? `${meta.baseRefName} \u2190 ${meta.headRefName}` : void 0,
+    meta.url
+  ].filter(Boolean);
+  return parts.length ? `
+PR metadata: ${parts.join(" \xB7 ")}` : "";
+}
+function buildGitContext(src) {
+  const parts = [`Source: ${src.label}`, `Reviewer: aicli v${VERSION}`];
+  if (src.metadata?.number != null) parts.push(`PR: #${src.metadata.number}`);
+  if (src.metadata?.title) parts.push(`Title: ${src.metadata.title}`);
+  if (src.metadata?.baseRefName && src.metadata?.headRefName) parts.push(`Base/Head: ${src.metadata.baseRefName}...${src.metadata.headRefName}`);
+  return parts.join(" | ");
+}
+function buildSummaryPrompt(diff, gitContext) {
+  return `# Pull Request Summary Request
+
+Summarize this pull request for maintainers. Focus on what changed, why it matters, risk areas, tests implied by the diff, and release-note worthy behavior.
+
+## Git Context
+${gitContext}
+
+## Code Changes (diff)
+\`\`\`diff
+${diff}
+\`\`\`
+
+## Output Format
+1. **Summary**: 2-4 bullets
+2. **Key Files**: important files and why they changed
+3. **Risk / Review Focus**: areas reviewers should inspect
+4. **Testing Notes**: tests present or missing
+5. **Suggested PR Description**: concise markdown description`;
+}
+function buildAgentPrompt(agent, diff, gitContext) {
+  const focus = {
+    security: "security vulnerabilities, secret handling, authz/authn, injection, unsafe network/file behavior",
+    bugs: "logic bugs, edge cases, race conditions, incorrect assumptions, regressions",
+    tests: "missing tests, weak assertions, CI breakage risk, testability gaps",
+    maintainability: "complexity, naming, duplication, API shape, future maintenance risk"
+  };
+  return `# PR Review Agent: ${agent}
+
+Review the diff only from this lens: ${focus[agent]}. Be concise and cite file paths/lines when possible.
+
+## Git Context
+${gitContext}
+
+## Diff
+\`\`\`diff
+${diff}
+\`\`\`
+
+## Output Format
+- Findings first, ordered by severity
+- If no material findings, say so clearly
+- Keep recommendations actionable`;
+}
+async function getProvider(opts) {
+  const config = new ConfigManager();
+  const registry = new ProviderRegistry();
+  await registry.initialize(
+    (id) => config.getApiKey(id),
+    (id) => ({ baseUrl: config.get("customBaseUrls")[id], timeout: config.get("timeouts")[id], proxy: config.get("proxy") }),
+    config.get("customProviders")
+  );
+  const providerId = opts.provider ?? config.getDefaultProvider();
+  if (!registry.has(providerId)) throw new Error(`Provider '${providerId}' is not configured. Set the right AICLI_API_KEY_* env var or pass --provider.`);
+  const info = registry.get(providerId).info;
+  return { registry, providerId, modelId: opts.model ?? config.get("defaultModels")[providerId] ?? info.defaultModel };
+}
+async function runPrompt(registry, providerId, modelId, prompt) {
+  const provider = registry.get(providerId);
+  const resp = await provider.chat({
+    messages: [{ role: "user", content: prompt, timestamp: /* @__PURE__ */ new Date() }],
+    model: modelId,
+    stream: false,
+    temperature: 0.3,
+    maxTokens: 8192
+  });
+  return resp.content?.trim() ?? "";
+}
+async function runPrCli(opts) {
+  let source;
+  try {
+    source = fetchDiffSource(opts);
+  } catch (err) {
+    return { exitCode: 2, markdown: `\u274C ${err.message}`, posted: false, severity: { critical: 0, high: 0, warning: 0, info: 0 } };
+  }
+  const diff = source.diff.trim();
+  if (!diff) return { exitCode: 0, markdown: `\u2705 No changes to review (${source.label}).`, posted: false, severity: { critical: 0, high: 0, warning: 0, info: 0 } };
+  const parsedTarget = parsePrTarget(opts.target);
+  const requestedAgents = parseAgentList(opts.agents);
+  if (opts.post && source.kind !== "github-pr") {
+    return { exitCode: 2, markdown: "\u274C --post requires a GitHub PR number or URL target.", posted: false, severity: { critical: 0, high: 0, warning: 0, info: 0 } };
+  }
+  if (opts.post && opts.action === "summarize") {
+    return { exitCode: 2, markdown: "\u274C --post is currently supported for pr review and pr security-review only.", posted: false, severity: { critical: 0, high: 0, warning: 0, info: 0 } };
+  }
+  if (opts.post && requestedAgents.length > 0) {
+    return { exitCode: 2, markdown: "\u274C --post is currently not supported together with --agents; run without --agents or post the Markdown manually.", posted: false, severity: { critical: 0, high: 0, warning: 0, info: 0 } };
+  }
+  if ((opts.action === "review" || opts.action === "security-review") && requestedAgents.length === 0) {
+    const ci = await runCi({
+      pr: source.kind === "github-pr" ? parsedTarget.pr : void 0,
+      diffOverride: diff,
+      base: source.kind === "git" ? source.label.split("...")[0] : void 0,
+      post: !!opts.post,
+      update: opts.update !== false,
+      skipCode: opts.action === "security-review",
+      skipSecurity: opts.action === "review",
+      detailed: !!opts.detailed,
+      maxDiffChars: opts.maxDiffChars,
+      provider: opts.provider,
+      model: opts.model,
+      dryRun: !!opts.dryRun
+    });
+    return { exitCode: ci.exitCode, markdown: ci.markdown + metadataMarkdown(source.metadata), posted: ci.posted, severity: ci.severity };
+  }
+  const maxDiff = opts.maxDiffChars ?? 3e4;
+  const { diff: trimmedDiff, truncated } = truncateDiff(diff, maxDiff);
+  const gitContext = buildGitContext(source);
+  let providerBundle;
+  try {
+    providerBundle = await getProvider(opts);
+  } catch (err) {
+    return { exitCode: 2, markdown: `\u274C ${err.message}`, posted: false, severity: { critical: 0, high: 0, warning: 0, info: 0 } };
+  }
+  const sections = [`## \u{1F916} aicli pr ${opts.action}`, `*Provider: \`${providerBundle.providerId}\` \xB7 Model: \`${providerBundle.modelId}\` \xB7 Source: \`${source.label}\` \xB7 aicli v${VERSION}*`, ""];
+  if (source.metadata) sections.push(`> ${metadataMarkdown(source.metadata).trim()}`, "");
+  try {
+    const prompt = opts.action === "security-review" ? buildSecurityReviewPrompt(trimmedDiff, gitContext) : opts.action === "summarize" ? buildSummaryPrompt(trimmedDiff, gitContext) : buildReviewPrompt(trimmedDiff, gitContext, !!opts.detailed);
+    sections.push(await runPrompt(providerBundle.registry, providerBundle.providerId, providerBundle.modelId, prompt), "");
+    if (requestedAgents.length) {
+      const outputs = await Promise.all(requestedAgents.map(async (agent) => ({ agent, output: await runPrompt(providerBundle.registry, providerBundle.providerId, providerBundle.modelId, buildAgentPrompt(agent, trimmedDiff, gitContext)) })));
+      sections.push("### Agent Lenses", "");
+      for (const item of outputs) sections.push(`#### ${item.agent}`, item.output, "");
+    }
+  } catch (err) {
+    return { exitCode: 2, markdown: `\u274C PR review call failed: ${err.message}`, posted: false, severity: { critical: 0, high: 0, warning: 0, info: 0 } };
+  }
+  if (truncated) sections.push(`> \u26A0 Diff was truncated to ${maxDiff} chars (original: ${diff.length}). Consider splitting large PRs.`);
+  const markdown = sections.join("\n");
+  return { exitCode: 0, markdown, posted: false, severity: countSeverity(markdown) };
+}
+export {
+  parseAgentList,
+  parsePrTarget,
+  runPrCli
+};

package/dist/{run-tests-J2JQ57R3.js → run-tests-K7QR5QN4.js}

@@ -1,7 +1,7 @@
 import {
   executeTests,
   runTestsTool
-} from "./chunk-RS4WBI73.js";
+} from "./chunk-QAYOI57M.js";
 export {
   executeTests,
   runTestsTool

package/dist/{run-tests-QJD43AML.js → run-tests-ZDSA3QES.js}

@@ -2,8 +2,8 @@
 import {
   executeTests,
   runTestsTool
-} from "./chunk-3TSHNZKI.js";
-import "./chunk-7JES2NWR.js";
+} from "./chunk-VTH7BLXK.js";
+import "./chunk-XJGEQIYS.js";
 export {
   executeTests,
   runTestsTool

package/dist/{server-WDLV3W4F.js → server-22YF3U34.js}

@@ -1,13 +1,13 @@
 #!/usr/bin/env node
 import {
   ToolRegistry
-} from "./chunk-6C3JYNM6.js";
+} from "./chunk-MGBMNCHG.js";
 import "./chunk-T2NL5ZIA.js";
 import "./chunk-BXP6YZ2P.js";
-import "./chunk-3TSHNZKI.js";
+import "./chunk-VTH7BLXK.js";
 import {
   runTool
-} from "./chunk-X6OXS7KU.js";
+} from "./chunk-SNJAOXFT.js";
 import {
   getDangerLevel,
   schemaToJsonSchema
@@ -15,7 +15,8 @@ import {
 import "./chunk-TZQHYZKT.js";
 import {
   VERSION
-} from "./chunk-7JES2NWR.js";
+} from "./chunk-XJGEQIYS.js";
+import "./chunk-HOSJZMQS.js";
 import "./chunk-4BKXL7SM.js";
 import "./chunk-TB4W4Y4T.js";
 import "./chunk-KHYD3WXE.js";