bosun 0.40.4 → 0.40.5

package/README.md

@@ -183,8 +183,15 @@ npm run hooks:install
 
 If you find this project useful or would like to stay up to date with new releases, a star is appreciated!
 
-[![Star History Chart](https://api.star-history.com/image?repos=VirtEngine/Bosun&type=date&legend=top-left)](https://www.star-history.com/?repos=VirtEngine%2FBosun&type=date&legend=top-left)
-
+## Star History
+
+<a href="https://www.star-history.com/?repos=VirtEngine%2FBosun&type=date&legend=top-left">
+ <picture>
+   <source media="(prefers-color-scheme: dark)" srcset="https://api.star-history.com/image?repos=VirtEngine/Bosun&type=date&theme=dark&legend=top-left" />
+   <source media="(prefers-color-scheme: light)" srcset="https://api.star-history.com/image?repos=VirtEngine/Bosun&type=date&legend=top-left" />
+   <img alt="Star History Chart" src="https://api.star-history.com/image?repos=VirtEngine/Bosun&type=date&legend=top-left" />
+ </picture>
+</a>
 ---
 
 ## License

package/agent/review-agent.mjs

@@ -122,24 +122,36 @@ function extractRepoSlug(prUrl) {
 
 /**
  * Get the PR diff using `gh pr diff` or `git diff`.
- * @param {{ prUrl?: string, branchName?: string }} opts
+ * @param {{ prUrl?: string, prNumber?: number|string, repoSlug?: string, branchName?: string, cwd?: string|null }} opts
  * @returns {{ diff: string, truncated: boolean }}
  */
-function getPrDiff({ prUrl, branchName }) {
+function getPrDiff({ prUrl, prNumber, repoSlug, branchName, cwd }) {
   let diff = "";
+  const commandOptions = {
+    encoding: "utf8",
+    timeout: 30_000,
+    stdio: ["pipe", "pipe", "pipe"],
+    ...(cwd ? { cwd } : {}),
+  };
 
   // Strategy 1: gh pr diff
-  const prNumber = extractPrNumber(prUrl);
-  const repoSlug = extractRepoSlug(prUrl);
-  if (prNumber && repoSlug) {
+  const resolvedPrNumber = Number.isFinite(Number(prNumber))
+    ? Number(prNumber)
+    : extractPrNumber(prUrl);
+  const resolvedRepoSlug = String(repoSlug || "").trim() || extractRepoSlug(prUrl);
+  if (resolvedPrNumber) {
     try {
-      const result = spawnSync(
-        "gh",
-        ["pr", "diff", String(prNumber), "--repo", repoSlug],
-        { encoding: "utf8", timeout: 30_000, stdio: ["pipe", "pipe", "pipe"] },
-      );
-      if (result.status === 0 && result.stdout?.trim()) {
-        diff = result.stdout;
+      const attempts = [];
+      if (resolvedRepoSlug) {
+        attempts.push(["pr", "diff", String(resolvedPrNumber), "--repo", resolvedRepoSlug]);
+      }
+      attempts.push(["pr", "diff", String(resolvedPrNumber)]);
+      for (const args of attempts) {
+        const result = spawnSync("gh", args, commandOptions);
+        if (result.status === 0 && result.stdout?.trim()) {
+          diff = result.stdout;
+          break;
+        }
       }
     } catch {
       /* fall through to git diff */
@@ -149,13 +161,16 @@ function getPrDiff({ prUrl, branchName }) {
   // Strategy 2: git diff main...<branch>
   if (!diff && branchName) {
     try {
-      const result = spawnSync("git", ["diff", `main...${branchName}`], {
-        encoding: "utf8",
-        timeout: 30_000,
-        stdio: ["pipe", "pipe", "pipe"],
-      });
-      if (result.status === 0 && result.stdout?.trim()) {
-        diff = result.stdout;
+      const attempts = [
+        ["diff", `origin/main...${branchName}`],
+        ["diff", `main...${branchName}`],
+      ];
+      for (const args of attempts) {
+        const result = spawnSync("git", args, commandOptions);
+        if (result.status === 0 && result.stdout?.trim()) {
+          diff = result.stdout;
+          break;
+        }
       }
     } catch {
       /* ignore */
@@ -343,6 +358,17 @@ export class ReviewAgent {
       console.warn(`${TAG} queueReview called without task id — skipping`);
       return;
     }
+    const hasReviewReference = Boolean(
+      String(task.prUrl || "").trim() ||
+        String(task.prNumber || "").trim() ||
+        String(task.branchName || "").trim(),
+    );
+    if (!hasReviewReference) {
+      console.warn(
+        `${TAG} queueReview skipped for ${task.id}: no prUrl, prNumber, or branchName`,
+      );
+      return;
+    }
 
     if (this.#seen.has(task.id)) {
       console.log(`${TAG} task ${task.id} already queued/in-flight — skipping`);
@@ -463,7 +489,10 @@ export class ReviewAgent {
     // 1. Get PR diff
     const { diff, truncated } = getPrDiff({
       prUrl: task.prUrl,
+      prNumber: task.prNumber,
+      repoSlug: task.repoSlug,
       branchName: task.branchName,
+      cwd: task.worktreePath || null,
     });
 
     if (!diff) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bosun",
-  "version": "0.40.4",
+  "version": "0.40.5",
   "description": "Bosun Autonomous Engineering — manages AI agent executors with failover, extremely powerful workflow builder, and a massive amount of included default workflow templates for autonomous engineering, creates PRs via Vibe-Kanban API, and sends Telegram notifications. Supports N executors with weighted distribution, multi-repo projects, and auto-setup.",
   "type": "module",
   "license": "Apache-2.0",
@@ -129,6 +129,7 @@
     "bench:swebench:import": "node bench/swebench/bosun-swebench.mjs import",
     "bench:swebench:export": "node bench/swebench/bosun-swebench.mjs export",
     "bench:swebench:eval": "node bench/swebench/bosun-swebench.mjs eval",
+    "bench:library:resolver": "node bench/library/library-resolver-bench.mjs",
     "mutate": "npx stryker run",
     "mutate:incremental": "npx stryker run --incremental",
     "mutate:report": "node scripts/mutation-report.mjs",