claude-teammate 0.1.255 → 0.1.256

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-teammate",
-  "version": "0.1.255",
+  "version": "0.1.256",
   "description": "CLI bootstrapper for Claude Teammate.",
   "license": "MIT",
   "type": "module",

package/src/memory.js

@@ -458,7 +458,10 @@ function uniqueGitHubIssues(githubIssues) {
       continue;
     }
 
-    const key = githubIssue.repo_url || githubIssue.url;
+    const key =
+      githubIssue.repo_url && githubIssue.number
+        ? `${githubIssue.repo_url}#${githubIssue.number}`
+        : githubIssue.url || githubIssue.repo_url;
     if (!key) {
       continue;
     }

package/src/worker/jira-issue-workflow.js

@@ -325,23 +325,38 @@ export async function processJiraIssue({
     }
 
     let repoCheckoutUpdated = false;
-    for (const repo of liveRepos) {
-      let validatedLocalPath;
-      try {
-        validatedLocalPath = await services.validateOrEnsureLocalRepo(repo.url, runtimePaths.reposDir, repo.local_path);
-      } catch (error) {
+    const repoValidationOutcomes = await Promise.allSettled(
+      liveRepos.map((repo) =>
+        services.validateOrEnsureLocalRepo(repo.url, runtimePaths.reposDir, repo.local_path).then(
+          (validatedLocalPath) => ({ repo, validatedLocalPath }),
+          (error) => {
+            error._repo = repo;
+            throw error;
+          }
+        )
+      )
+    );
+
+    for (const outcome of repoValidationOutcomes) {
+      if (outcome.status === "rejected") {
+        const error = outcome.reason;
+        const failedRepo = error._repo ?? liveRepos[0];
         if (isRepoCheckoutNotReadyError(error)) {
-          issueMemory.last_error = `Repository checkout is not ready yet for ${repo.url}. The worker will retry automatically.`;
+          issueMemory.last_error = `Repository checkout is not ready yet for ${failedRepo.url}. The worker will retry automatically.`;
           issueMemory = await services.saveIssueMemory(issueMemoryRecord.filePath, detail, issueMemory);
           await logger.info("Skipping Jira issue until checkout is ready", {
             issue: detail.key,
-            repo: repo.url,
-            localPath: error.localPath || repo.local_path
+            repo: failedRepo.url,
+            localPath: error.localPath || failedRepo.local_path
           });
           return buildIssueState(detail, { repos: liveRepos }, issueMemory, null, issueMemory.last_error);
         }
         throw error;
       }
+    }
+
+    for (const outcome of repoValidationOutcomes) {
+      const { repo, validatedLocalPath } = outcome.value;
       if (validatedLocalPath !== repo.local_path) {
         repo.local_path = validatedLocalPath;
         repoCheckoutUpdated = true;

package/src/worker/pollers/review-pull-request-discussions.js

@@ -1,6 +1,7 @@
 import { getForgeBotUserForRepo } from "../forge-sync.js";
 import { processReviewSuggestionFeedback as defaultProcessReviewSuggestionFeedback } from "../review-suggestion.js";
 import { pollStuckTasks } from "../stuck-tasks.js";
+import { markWorkQueueItemRunning, queueWorkQueueItem, removeWorkQueueItem } from "../worker-state.js";
 import { loadKnownRepos } from "./helpers.js";
 
 export function createReviewPullRequestDiscussionPoller({
@@ -20,6 +21,17 @@ export function createReviewPullRequestDiscussionPoller({
     return `review-discussion:${repoUrl}!${prNumber}`;
   }
 
+  function buildWorkQueueItem(pr) {
+    return {
+      id: buildTaskId(pr.repoUrl, pr.number),
+      kind: "review_discussion",
+      label: `${String(pr?.repoUrl || "").trim()}!${String(pr?.number || "").trim()}`,
+      title: String(pr?.title || "").trim(),
+      repoUrl: String(pr?.repoUrl || "").trim(),
+      pullRequestNumber: String(pr?.number || "").trim()
+    };
+  }
+
   return async function runReviewPullRequestDiscussionPoll() {
     const repos = await loadKnownRepos(projectRoot, forgeRegistry);
     const repoUrls = repos.map((repo) => repo.url);
@@ -63,9 +75,14 @@ export function createReviewPullRequestDiscussionPoller({
         continue;
       }
       inFlight.add(taskId);
+      const workQueueItem = buildWorkQueueItem(pr);
+      queueWorkQueueItem(state, workQueueItem);
+      await persistState();
 
       void (async () => {
         const release = await pool.acquire();
+        markWorkQueueItemRunning(state, workQueueItem);
+        await persistState();
         try {
           const repo = repos.find((r) => r.url === pr.repoUrl);
           await processReviewSuggestionFeedback({
@@ -88,6 +105,8 @@ export function createReviewPullRequestDiscussionPoller({
           });
         } finally {
           inFlight.delete(taskId);
+          removeWorkQueueItem(state, workQueueItem.id);
+          await persistState();
           release();
         }
       })();

package/src/worker/pollers/tracked-issues.js

@@ -1,10 +1,5 @@
 import { listIssueMemoryRecords } from "../../memory.js";
-import {
-  findIssueMemoryByGitHubIssue,
-  getForgeBotUserForRepo,
-  getLatestGitHubComment,
-  isForgeBotAuthor
-} from "../forge-sync.js";
+import { getForgeBotUserForRepo, getLatestGitHubComment, isForgeBotAuthor } from "../forge-sync.js";
 import { checkCommentAuthorIsMember, hasEyesReaction, hasPlusOneReaction, isApprovalComment } from "../pull-request.js";
 import { normalizeRepoIdentity } from "../repo-utils.js";
 import { buildIssueQueueId, buildQueuedTrackedIssueState, reconcileQueueEntry } from "../state-builders.js";
@@ -60,9 +55,9 @@ export function createTrackedIssuePoller({
   }
 
   async function getDispatchDecision({ issue, repo, provider, botUser, issueMemoryIndex }) {
-    const issueMemoryRecord =
-      issueMemoryIndex.get(`${normalizeRepoIdentity(repo.url || "")}#${String(issue.number || "").trim()}`) ||
-      (await findIssueMemoryByGitHubIssue(projectRoot, repo.url, issue.number));
+    const issueMemoryRecord = issueMemoryIndex.get(
+      `${normalizeRepoIdentity(repo.url || "")}#${String(issue.number || "").trim()}`
+    );
     if (!issueMemoryRecord) {
       await logger.info("GitHub issue has no linked Jira memory, not queueing for work", {
         issue: issue.number,

package/src/worker/review-suggestion.js

@@ -1,17 +1,17 @@
-import { readFileSync } from "node:fs";
+import { readFile } from "node:fs/promises";
 import path from "node:path";
 import { isClaudeCliError, runClaudeRevisionPrompt } from "../claude.js";
 import { toErrorMessage } from "./utils.js";
 
 const FILE_CONTEXT_LINES = 20;
 
-function readFileContext(repoPath, filePath, targetLine) {
+async function readFileContext(repoPath, filePath, targetLine) {
   if (!repoPath || !filePath) {
     return "";
   }
   try {
     const absPath = path.join(repoPath, filePath);
-    const content = readFileSync(absPath, "utf8");
+    const content = await readFile(absPath, "utf8");
     const lines = content.split("\n");
     const start = Math.max(0, targetLine - FILE_CONTEXT_LINES - 1);
     const end = Math.min(lines.length, targetLine + FILE_CONTEXT_LINES);
@@ -60,7 +60,7 @@ export async function processReviewSuggestionFeedback({
   const taskId = `suggestionRevision:${repoUrl}!${prNumber}`;
 
   for (const thread of threads) {
-    const fileContext = readFileContext(repoPath, thread.file, thread.line);
+    const fileContext = await readFileContext(repoPath, thread.file, thread.line);
     let result;
     try {
       result = await runClaudeRevisionPromptFn({