@tarcisiopgs/lisa 1.36.0 → 1.37.1

package/README.md

@@ -53,7 +53,7 @@ If something fails — pre-push hooks, quota limits, stuck processes — Lisa ha
 - **Smart activity detection** — reads agent session logs to prevent false stuck kills during analysis phases
 - **Progress comments** — posts real-time status updates on issues as Lisa works through stages
 - **Context enrichment** — greps for issue-related files and surfaces them in the agent prompt
-- **PR reviewers & assignees** — auto-request reviews and assign PRs via config; `self` keyword resolves to the authenticated user
+- **PR reviewers & assignees** — auto-request reviews and assign PRs via config; `self` keyword resolves to the authenticated user. Manage reviewers interactively from the TUI detail view (`r`)
 - **Self-healing** — orphan recovery on startup, push failure retry, stuck process detection
 - **Guardrails** — past failures are injected into future prompts to avoid repeating mistakes
 - **Lineage context** — plan-decomposed issues get sibling task awareness, preventing duplicate work in concurrent mode
@@ -360,6 +360,7 @@ The real-time Kanban board shows issue progress, streams provider output, and de
 |-----|--------|
 | `↑` `↓` | Scroll output log |
 | `o` | Open PR in browser |
+| `r` | Toggle reviewer picker (add/remove reviewers on the PR) |
 | `m` | Merge PR (warns if CI not passed) |
 | `Esc` | Back to board |
 

package/dist/chunk-4MZ2565Y.js

@@ -0,0 +1,32 @@
+#!/usr/bin/env node
+
+// src/errors.ts
+function formatError(err) {
+  if (err instanceof Error) {
+    const cause = err.cause ? ` (caused by: ${formatError(err.cause)})` : "";
+    return `${err.message}${cause}`;
+  }
+  return String(err);
+}
+var LisaError = class extends Error {
+  constructor(message, options) {
+    super(message, options);
+    this.name = "LisaError";
+  }
+};
+var SourceError = class extends LisaError {
+  constructor(message, source, statusCode, options) {
+    super(message, options);
+    this.source = source;
+    this.statusCode = statusCode;
+    this.name = "SourceError";
+  }
+  source;
+  statusCode;
+};
+
+export {
+  formatError,
+  LisaError,
+  SourceError
+};

package/dist/chunk-HPWL5JRW.js

@@ -0,0 +1,113 @@
+#!/usr/bin/env node
+
+// src/output/logger.ts
+import { appendFileSync, existsSync, mkdirSync, writeFileSync } from "fs";
+import { dirname } from "path";
+import pc from "picocolors";
+var logFilePath = null;
+var outputMode = "default";
+var logLevel = "default";
+function setOutputMode(mode) {
+  outputMode = mode;
+}
+function getOutputMode() {
+  return outputMode;
+}
+function setLogLevel(level) {
+  logLevel = level;
+}
+function shouldPrintToConsole() {
+  return outputMode !== "tui" && logLevel !== "quiet";
+}
+function initLogFile(path) {
+  const dir = dirname(path);
+  if (!existsSync(dir)) {
+    mkdirSync(dir, { recursive: true });
+  }
+  writeFileSync(path, `[${timestamp()}] Log started
+`);
+  logFilePath = path;
+}
+function timestamp() {
+  return (/* @__PURE__ */ new Date()).toLocaleTimeString("en-US", { hour12: false });
+}
+function writeToFile(level, message) {
+  if (logFilePath) {
+    appendFileSync(logFilePath, `[${timestamp()}] [${level}] ${message}
+`);
+  }
+}
+function log(message) {
+  if (shouldPrintToConsole()) {
+    console.error(`${pc.cyan("[lisa]")} ${pc.dim(timestamp())} ${message}`);
+  }
+  writeToFile("info", message);
+}
+function warn(message) {
+  if (shouldPrintToConsole()) {
+    console.error(`${pc.yellow("[lisa]")} ${pc.dim(timestamp())} ${message}`);
+  }
+  writeToFile("warn", message);
+}
+function error(message) {
+  if (shouldPrintToConsole()) {
+    console.error(`${pc.red("[lisa]")} ${pc.dim(timestamp())} ${message}`);
+  }
+  writeToFile("error", message);
+}
+function ok(message) {
+  if (shouldPrintToConsole()) {
+    console.error(`${pc.green("[lisa]")} ${pc.dim(timestamp())} ${message}`);
+  }
+  writeToFile("ok", message);
+}
+function verbose(message) {
+  if (logLevel !== "verbose") return;
+  if (shouldPrintToConsole()) {
+    console.error(`${pc.dim("[lisa]")} ${pc.dim(timestamp())} ${pc.dim(message)}`);
+  }
+  writeToFile("verbose", message);
+}
+function divider(session) {
+  log(`${"\u2501".repeat(3)} Session ${session} ${"\u2501".repeat(3)}`);
+}
+function banner() {
+  if (outputMode !== "default" || logLevel === "quiet") return;
+  const title = " lisa \u266A  autonomous issue resolver ";
+  const border = "\u2500".repeat(title.length);
+  console.error(pc.yellow(`
+  \u250C${border}\u2510`));
+  console.error(pc.yellow(`  \u2502`) + pc.bold(pc.white(title)) + pc.yellow("\u2502"));
+  console.error(pc.yellow(`  \u2514${border}\u2518
+`));
+}
+function updateNotice(update) {
+  if (outputMode !== "default" || logLevel === "quiet") return;
+  const msg = `Update available ${pc.dim(update.currentVersion)} \u2192 ${pc.green(pc.bold(update.latestVersion))}`;
+  const cmd = `Run ${pc.cyan("npm i -g @tarcisiopgs/lisa")} to update`;
+  const lines = [msg, cmd];
+  const strip = (s) => s.replace(/\x1b\[[0-9;]*m/g, "");
+  const maxLen = Math.max(...lines.map((l) => strip(l).length));
+  const pad = (s) => s + " ".repeat(maxLen - strip(s).length);
+  console.error(pc.yellow(`  \u250C${"\u2500".repeat(maxLen + 2)}\u2510`));
+  for (const line of lines) {
+    console.error(pc.yellow("  \u2502 ") + pad(line) + pc.yellow(" \u2502"));
+  }
+  console.error(pc.yellow(`  \u2514${"\u2500".repeat(maxLen + 2)}\u2518
+`));
+}
+
+export {
+  setOutputMode,
+  getOutputMode,
+  setLogLevel,
+  initLogFile,
+  log,
+  warn,
+  error,
+  ok,
+  verbose,
+  divider,
+  banner,
+  updateNotice
+};

package/dist/{chunk-AFKXWCAM.js → chunk-I5HJELVZ.js}

@@ -5,17 +5,22 @@ import {
   resolveModels,
   runWithFallback,
   saveLineage
-} from "./chunk-6VIN5PMW.js";
+} from "./chunk-YRKJONH5.js";
+import {
+  normalizeLabels
+} from "./chunk-LR2GREZS.js";
 import {
   error,
   log,
-  normalizeLabels,
   ok,
   warn
-} from "./chunk-V44FTYWZ.js";
+} from "./chunk-HPWL5JRW.js";
+import {
+  LisaError
+} from "./chunk-4MZ2565Y.js";
 
 // src/cli/error.ts
-var CliError = class extends Error {
+var CliError = class extends LisaError {
   exitCode;
   constructor(message, exitCode = 1) {
     super(message);

package/dist/{chunk-V44FTYWZ.js → chunk-LR2GREZS.js}

@@ -2,96 +2,12 @@
 import {
   notify
 } from "./chunk-72CYGBT4.js";
-
-// src/output/logger.ts
-import { appendFileSync, existsSync, mkdirSync, writeFileSync } from "fs";
-import { dirname } from "path";
-import pc from "picocolors";
-var logFilePath = null;
-var outputMode = "default";
-var logLevel = "default";
-function setOutputMode(mode) {
-  outputMode = mode;
-}
-function getOutputMode() {
-  return outputMode;
-}
-function setLogLevel(level) {
-  logLevel = level;
-}
-function shouldPrintToConsole() {
-  return outputMode !== "tui" && logLevel !== "quiet";
-}
-function initLogFile(path) {
-  const dir = dirname(path);
-  if (!existsSync(dir)) {
-    mkdirSync(dir, { recursive: true });
-  }
-  writeFileSync(path, `[${timestamp()}] Log started
-`);
-  logFilePath = path;
-}
-function timestamp() {
-  return (/* @__PURE__ */ new Date()).toLocaleTimeString("en-US", { hour12: false });
-}
-function writeToFile(level, message) {
-  if (logFilePath) {
-    appendFileSync(logFilePath, `[${timestamp()}] [${level}] ${message}
-`);
-  }
-}
-function log(message) {
-  if (shouldPrintToConsole()) {
-    console.error(`${pc.cyan("[lisa]")} ${pc.dim(timestamp())} ${message}`);
-  }
-  writeToFile("info", message);
-}
-function warn(message) {
-  if (shouldPrintToConsole()) {
-    console.error(`${pc.yellow("[lisa]")} ${pc.dim(timestamp())} ${message}`);
-  }
-  writeToFile("warn", message);
-}
-function error(message) {
-  if (shouldPrintToConsole()) {
-    console.error(`${pc.red("[lisa]")} ${pc.dim(timestamp())} ${message}`);
-  }
-  writeToFile("error", message);
-}
-function ok(message) {
-  if (shouldPrintToConsole()) {
-    console.error(`${pc.green("[lisa]")} ${pc.dim(timestamp())} ${message}`);
-  }
-  writeToFile("ok", message);
-}
-function divider(session) {
-  log(`${"\u2501".repeat(3)} Session ${session} ${"\u2501".repeat(3)}`);
-}
-function banner() {
-  if (outputMode !== "default" || logLevel === "quiet") return;
-  const title = " lisa \u266A  autonomous issue resolver ";
-  const border = "\u2500".repeat(title.length);
-  console.error(pc.yellow(`
-  \u250C${border}\u2510`));
-  console.error(pc.yellow(`  \u2502`) + pc.bold(pc.white(title)) + pc.yellow("\u2502"));
-  console.error(pc.yellow(`  \u2514${border}\u2518
-`));
-}
-function updateNotice(update) {
-  if (outputMode !== "default" || logLevel === "quiet") return;
-  const msg = `Update available ${pc.dim(update.currentVersion)} \u2192 ${pc.green(pc.bold(update.latestVersion))}`;
-  const cmd = `Run ${pc.cyan("npm i -g @tarcisiopgs/lisa")} to update`;
-  const lines = [msg, cmd];
-  const strip = (s) => s.replace(/\x1b\[[0-9;]*m/g, "");
-  const maxLen = Math.max(...lines.map((l) => strip(l).length));
-  const pad = (s) => s + " ".repeat(maxLen - strip(s).length);
-  console.error(pc.yellow(`  \u250C${"\u2500".repeat(maxLen + 2)}\u2510`));
-  for (const line of lines) {
-    console.error(pc.yellow("  \u2502 ") + pad(line) + pc.yellow(" \u2502"));
-  }
-  console.error(pc.yellow(`  \u2514${"\u2500".repeat(maxLen + 2)}\u2518
-`));
-}
+import {
+  warn
+} from "./chunk-HPWL5JRW.js";
+import {
+  SourceError
+} from "./chunk-4MZ2565Y.js";
 
 // src/ui/state.ts
 import { EventEmitter } from "events";
@@ -102,51 +18,101 @@ import { execa } from "execa";
 
 // src/sources/base.ts
 var REQUEST_TIMEOUT_MS = 3e4;
+var MAX_ATTEMPTS = 3;
+var BASE_BACKOFF_MS = 1e3;
 function normalizeLabels(config) {
   return Array.isArray(config.label) ? config.label : config.label ? [config.label] : [];
 }
 function createApiClient(baseUrl, getHeaders, name) {
-  async function request(method, path, body) {
-    const url = `${baseUrl}${path}`;
-    const headers = {
-      ...await getHeaders(),
-      "Content-Type": "application/json"
-    };
-    const res = await fetch(url, {
-      method,
-      headers,
-      body: body !== void 0 ? JSON.stringify(body) : void 0,
-      signal: AbortSignal.timeout(REQUEST_TIMEOUT_MS)
-    });
-    if (!res.ok) {
-      const text = await res.text();
-      throw new Error(`${name} API error (${res.status}): ${text}`);
+  function isRetryable(error) {
+    if (error instanceof Error) {
+      if (error.name === "AbortError") return true;
+      if (error instanceof TypeError) return true;
+      if (error instanceof SourceError && error.statusCode) {
+        return error.statusCode >= 500;
+      }
+      const match = error.message.match(/API error \((\d+)\)/);
+      if (match?.[1]) {
+        const status = Number.parseInt(match[1], 10);
+        return status >= 500;
+      }
     }
-    if (method === "DELETE" || res.status === 204) return void 0;
-    return await res.json();
+    return false;
   }
-  return {
-    get: (path) => request("GET", path),
-    post: (path, body) => request("POST", path, body),
-    put: (path, body) => request("PUT", path, body),
-    patch: (path, body) => request("PATCH", path, body),
-    delete: (path) => request("DELETE", path),
-    /** Raw request for non-JSON bodies (e.g. Trello form-encoded). */
-    raw: async (method, path, init) => {
+  async function retryableRequest(operation) {
+    let lastError;
+    for (let attempt = 0; attempt < MAX_ATTEMPTS; attempt++) {
+      try {
+        return await operation();
+      } catch (error) {
+        lastError = error;
+        if (!isRetryable(error) || attempt === MAX_ATTEMPTS - 1) {
+          throw error;
+        }
+        const delay = BASE_BACKOFF_MS * 2 ** attempt;
+        warn(
+          `${name} API request failed (attempt ${attempt + 1}/${MAX_ATTEMPTS}), retrying in ${delay}ms...`
+        );
+        await new Promise((resolve) => setTimeout(resolve, delay));
+      }
+    }
+    throw lastError;
+  }
+  async function request(method, path, body, schema) {
+    return retryableRequest(async () => {
       const url = `${baseUrl}${path}`;
-      const headers = await getHeaders();
+      const headers = {
+        ...await getHeaders(),
+        "Content-Type": "application/json"
+      };
       const res = await fetch(url, {
         method,
-        headers: { ...headers, ...init?.headers },
-        body: init?.body,
+        headers,
+        body: body !== void 0 ? JSON.stringify(body) : void 0,
         signal: AbortSignal.timeout(REQUEST_TIMEOUT_MS)
       });
       if (!res.ok) {
         const text = await res.text();
-        throw new Error(`${name} API error (${res.status}): ${text}`);
+        throw new SourceError(`${name} API error (${res.status}): ${text}`, name, res.status);
       }
       if (method === "DELETE" || res.status === 204) return void 0;
-      return await res.json();
+      const data = await res.json();
+      if (schema) {
+        const result = schema.safeParse(data);
+        if (!result.success) {
+          throw new Error(
+            `${name} API response validation failed for ${method} ${path}: ${result.error.message}`
+          );
+        }
+        return result.data;
+      }
+      return data;
+    });
+  }
+  return {
+    get: (path, schema) => request("GET", path, void 0, schema),
+    post: (path, body, schema) => request("POST", path, body, schema),
+    put: (path, body, schema) => request("PUT", path, body, schema),
+    patch: (path, body, schema) => request("PATCH", path, body, schema),
+    delete: (path) => request("DELETE", path),
+    /** Raw request for non-JSON bodies (e.g. Trello form-encoded). */
+    raw: async (method, path, init) => {
+      return retryableRequest(async () => {
+        const url = `${baseUrl}${path}`;
+        const headers = await getHeaders();
+        const res = await fetch(url, {
+          method,
+          headers: { ...headers, ...init?.headers },
+          body: init?.body,
+          signal: AbortSignal.timeout(REQUEST_TIMEOUT_MS)
+        });
+        if (!res.ok) {
+          const text = await res.text();
+          throw new SourceError(`${name} API error (${res.status}): ${text}`, name, res.status);
+        }
+        if (method === "DELETE" || res.status === 204) return void 0;
+        return await res.json();
+      });
     }
   };
 }
@@ -161,7 +127,10 @@ async function getToken() {
     if (stdout.trim()) return stdout.trim();
   } catch {
   }
-  throw new Error("GitHub authentication required: set GITHUB_TOKEN or run `gh auth login`");
+  throw new SourceError(
+    "GitHub authentication required: set GITHUB_TOKEN or run `gh auth login`",
+    "github-issues"
+  );
 }
 async function getAuthHeaders() {
   const token = await getToken();
@@ -247,8 +216,15 @@ var GitHubIssuesSource = class {
     const isOrphanDetection = !!config.pick_from && !validStates.includes(config.pick_from);
     const filterLabels = isOrphanDetection ? [config.pick_from] : normalizeLabels(config);
     const label = filterLabels.map((l) => encodeURIComponent(l)).join(",");
-    const path = `/repos/${owner}/${repo}/issues?labels=${label}&state=open&sort=created&direction=asc&per_page=100`;
-    const issues = (await api().get(path)).filter((i) => !i.pull_request);
+    const issues = [];
+    let page = 1;
+    while (true) {
+      const path = `/repos/${owner}/${repo}/issues?labels=${label}&state=open&sort=created&direction=asc&per_page=100&page=${page}`;
+      const batch = (await api().get(path)).filter((i) => !i.pull_request);
+      issues.push(...batch);
+      if (batch.length < 100) break;
+      page++;
+    }
     if (issues.length === 0) return null;
     const unblocked = [];
     const blocked = [];
@@ -388,9 +364,16 @@ var GitHubIssuesSource = class {
     const { owner, repo } = parseOwnerRepo(config.scope);
     const labels = normalizeLabels(config);
     const label = labels.map((l) => encodeURIComponent(l)).join(",");
-    const path = `/repos/${owner}/${repo}/issues?labels=${label}&state=open&sort=created&direction=asc&per_page=100`;
-    const issues = (await api().get(path)).filter((i) => !i.pull_request);
-    return issues.map((issue) => ({
+    const allIssues = [];
+    let page = 1;
+    while (true) {
+      const path = `/repos/${owner}/${repo}/issues?labels=${label}&state=open&sort=created&direction=asc&per_page=100&page=${page}`;
+      const batch = (await api().get(path)).filter((i) => !i.pull_request);
+      allIssues.push(...batch);
+      if (batch.length < 100) break;
+      page++;
+    }
+    return allIssues.map((issue) => ({
       id: makeIssueId(owner, repo, issue.number),
       title: issue.title,
       description: issue.body ?? "",
@@ -457,7 +440,7 @@ function getBaseUrl() {
 }
 function getAuthHeaders2() {
   const token = process.env.GITLAB_TOKEN;
-  if (!token) throw new Error("GITLAB_TOKEN must be set");
+  if (!token) throw new SourceError("GITLAB_TOKEN must be set", "gitlab-issues");
   return { "PRIVATE-TOKEN": token };
 }
 var _api2;
@@ -512,8 +495,15 @@ var GitLabIssuesSource = class {
     const isOrphanDetection = !!config.pick_from && !validStates.includes(config.pick_from);
     const filterLabels = isOrphanDetection ? [config.pick_from] : normalizeLabels(config);
     const label = filterLabels.map((l) => encodeURIComponent(l)).join(",");
-    const path = `/projects/${project}/issues?labels=${label}&state=opened&per_page=100`;
-    const issues = await api2().get(path);
+    const issues = [];
+    let page = 1;
+    while (true) {
+      const path = `/projects/${project}/issues?labels=${label}&state=opened&per_page=100&page=${page}`;
+      const batch = await api2().get(path);
+      issues.push(...batch);
+      if (batch.length < 100) break;
+      page++;
+    }
     if (issues.length === 0) return null;
     const unblocked = [];
     const blocked = [];
@@ -626,9 +616,16 @@ var GitLabIssuesSource = class {
     const project = parseGitLabProject(config.scope);
     const labelsArr = normalizeLabels(config);
     const label = labelsArr.map((l) => encodeURIComponent(l)).join(",");
-    const path = `/projects/${project}/issues?labels=${label}&state=opened&per_page=100`;
-    const issues = await api2().get(path);
-    return issues.map((issue) => ({
+    const allIssues = [];
+    let page = 1;
+    while (true) {
+      const path = `/projects/${project}/issues?labels=${label}&state=opened&per_page=100&page=${page}`;
+      const batch = await api2().get(path);
+      allIssues.push(...batch);
+      if (batch.length < 100) break;
+      page++;
+    }
+    return allIssues.map((issue) => ({
       id: makeIssueId2(config.scope, issue.iid),
       title: issue.title,
       description: issue.description ?? "",
@@ -739,17 +736,18 @@ function stopMergePolling(key) {
   }
 }
 function startMergePolling(issueId, prUrl) {
-  if (activePolls.has(prUrl)) return;
+  const key = `${issueId}:${prUrl}`;
+  if (activePolls.has(key)) return;
   const intervalId = setInterval(() => {
     checkPrMergedByUrl(prUrl).then((merged) => {
       if (merged) {
-        stopMergePolling(prUrl);
+        stopMergePolling(key);
         kanbanEmitter.emit("issue:merged", issueId);
       }
     }).catch(() => {
     });
   }, MERGE_POLL_INTERVAL_MS);
-  activePolls.set(prUrl, intervalId);
+  activePolls.set(key, intervalId);
 }
 var KanbanEmitter = class extends EventEmitter {
 };
@@ -893,6 +891,14 @@ function useKanbanState(bellEnabled, initialCards = []) {
     const onSubstatus = (issueId, substatus) => {
       setCards((prev) => prev.map((c) => c.id === issueId ? { ...c, substatus } : c));
     };
+    const onReviewersUpdated = (issueId, reviewers) => {
+      setCards((prev) => prev.map((c) => c.id === issueId ? { ...c, reviewers } : c));
+    };
+    const onAvailableReviewers = (issueId, available) => {
+      setCards(
+        (prev) => prev.map((c) => c.id === issueId ? { ...c, availableReviewers: available } : c)
+      );
+    };
     const MAX_OUTPUT_SIZE = 2e5;
     const outputBuffer = /* @__PURE__ */ new Map();
     let flushTimer = null;
@@ -941,6 +947,8 @@ function useKanbanState(bellEnabled, initialCards = []) {
     kanbanEmitter.on("provider:resumed", onProviderResumed);
     kanbanEmitter.on("issue:log-file", onLogFile);
     kanbanEmitter.on("issue:substatus", onSubstatus);
+    kanbanEmitter.on("issue:reviewers-updated", onReviewersUpdated);
+    kanbanEmitter.on("issue:available-reviewers", onAvailableReviewers);
     kanbanEmitter.on("issue:output", onOutput);
     const onModelChanged = (model) => setModelInUse(model);
     kanbanEmitter.on("provider:model-changed", onModelChanged);
@@ -987,6 +995,8 @@ function useKanbanState(bellEnabled, initialCards = []) {
       kanbanEmitter.off("provider:resumed", onProviderResumed);
       kanbanEmitter.off("issue:log-file", onLogFile);
       kanbanEmitter.off("issue:substatus", onSubstatus);
+      kanbanEmitter.off("issue:reviewers-updated", onReviewersUpdated);
+      kanbanEmitter.off("issue:available-reviewers", onAvailableReviewers);
       kanbanEmitter.off("issue:output", onOutput);
       kanbanEmitter.off("provider:model-changed", onModelChanged);
       kanbanEmitter.off("work:empty", onEmpty);
@@ -1015,17 +1025,6 @@ function useKanbanState(bellEnabled, initialCards = []) {
 }
 
 export {
-  setOutputMode,
-  getOutputMode,
-  setLogLevel,
-  initLogFile,
-  log,
-  warn,
-  error,
-  ok,
-  divider,
-  banner,
-  updateNotice,
   REQUEST_TIMEOUT_MS,
   normalizeLabels,
   createApiClient,