@tarcisiopgs/lisa 1.33.3 → 1.35.0

package/README.md

@@ -28,7 +28,7 @@ lisa         # start the agent loop
 ## How It Works
 
 ```
-  Plan → Create issues → Fetch → Implement → Push → Open PR → Update board → Next
+  Plan → Create issues → Fetch → Implement → Push → Open PR → CI Monitor → Review Monitor → Update board → Next
 ```
 
 Lisa starts and shows a Kanban board. If the queue is empty, press `n` to plan — describe a goal and the AI brainstorms with you (asking clarifying questions), presents its understanding for your confirmation, then decomposes the goal into atomic issues created directly in your tracker. You can review, edit, reorder, delete, or regenerate the plan with feedback before approving. Press `r` to start processing. Lisa picks the highest-priority labeled issue, moves it to "In Progress", sends a structured prompt to the AI agent, and monitors execution. The agent works in an isolated git worktree, implements the change, runs tests, and commits. Lisa pushes, opens a PR, moves the ticket to "In Review", and picks up the next one.
@@ -47,11 +47,16 @@ If something fails — pre-push hooks, quota limits, stuck processes — Lisa ha
 - **Model fallback** — chain models; transient errors (429, quota, timeout) auto-switch to the next
 - **Real-time TUI** — Kanban board with live provider output, plan mode, merge PRs with `m`
 - **CI monitoring** — polls CI after PR creation, re-invokes the agent to fix failures automatically
+- **Review monitoring** — polls PR reviews after CI, auto-addresses reviewer feedback (GitHub)
+- **Reaction engine** — configurable actions for CI failures, review changes, stuck agents
+- **Session state tracking** — real-time visibility into agent pipeline phase (implementing → validating → CI → review)
+- **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
 - **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
 - **Project context** — auto-generates `.lisa/context.md` with your stack, conventions, and constraints
 
 ## Providers
@@ -109,6 +114,7 @@ lisa config --set loop.cooldown=5  # set nested config values
 lisa doctor                 # diagnose setup issues (config, provider, env, git)
 lisa context refresh        # regenerate project context
 lisa feedback --pr URL      # inject PR review feedback into guardrails
+lisa sessions               # list active session states (supports --json)
 ```
 
 Append `--json` to any command for machine-readable output. Use `--verbose` / `--quiet` to control log verbosity.
@@ -252,6 +258,31 @@ ci_monitor:
   poll_timeout: 600          # max seconds to wait for CI
   block_on_failure: false    # revert issue if CI never passes
 
+# Post-PR review monitoring (GitHub only)
+review_monitor:
+  enabled: true
+  max_retries: 2             # fix attempts on review changes (default: 2)
+  poll_interval: 60          # seconds between review checks (default: 60)
+  poll_timeout: 3600         # max seconds to wait for review (default: 3600)
+
+# Configurable reactions (override defaults)
+reactions:
+  ci_failed:
+    action: reinvoke         # reinvoke | notify | skip
+    max_retries: 3
+    escalate_after: 30m
+  changes_requested:
+    action: reinvoke
+    max_retries: 2
+    escalate_after: 1h
+  approved:
+    action: notify
+  agent_stuck:
+    action: notify
+  validation_failed:
+    action: reinvoke
+    max_retries: 2
+
 progress_comments:
   enabled: true              # post real-time status on issues
 
@@ -275,7 +306,7 @@ hooks:
 After the agent implements an issue, Lisa runs a multi-stage validation pipeline before creating a PR:
 
 ```
-Agent implements → Proof of Work (lint/test/typecheck) → Spec Compliance → PR
+Agent implements → Proof of Work (lint/test/typecheck) → Spec Compliance → PR → CI Monitor → Review Monitor
 ```
 
 **Proof of Work** runs configured shell commands (lint, typecheck, test). If any fail, the agent is re-invoked with the error output to fix the issue.
@@ -318,9 +349,10 @@ The real-time Kanban board shows issue progress, streams provider output, and de
 | Key | Action | Key | Action |
 |-----|--------|-----|--------|
 | `←` `→` | Switch columns | `k` | Kill current issue |
-| `↑` `↓` | Navigate cards | `n` | Open plan mode |
-| `↵` | Open detail view | `r` | Run (from idle) |
-| `p` | Pause / resume | `q` | Quit |
+| `↑` `↓` | Navigate cards | `s` | Skip current issue |
+| `↵` | Open detail view | `n` | Open plan mode |
+| `p` | Pause / resume | `r` | Run (from idle) |
+| `m` | Merge PR (opens detail + triggers merge) | `q` | Quit |
 
 **Detail view**
 

package/dist/{chunk-UDQRHMVH.js → chunk-6SNT7TND.js}

@@ -3,15 +3,16 @@ import {
   buildContextMdBlock,
   readContext,
   resolveModels,
-  runWithFallback
-} from "./chunk-5724FKM6.js";
+  runWithFallback,
+  saveLineage
+} from "./chunk-6VIN5PMW.js";
 import {
   error,
   log,
   normalizeLabels,
   ok,
   warn
-} from "./chunk-PPRXJHPW.js";
+} from "./chunk-V44FTYWZ.js";
 
 // src/cli/error.ts
 var CliError = class extends Error {
@@ -163,7 +164,7 @@ function ensureAcceptanceCriteria(description, criteria) {
 
 ${checklist}`;
 }
-async function createPlanIssues(source, config, plan) {
+async function createPlanIssues(source, config, plan, workspace) {
   if (!source.createIssue) {
     throw new Error(`Source "${source.name}" does not support createIssue`);
   }
@@ -219,6 +220,21 @@ _Depends on: ${depRefs}_`;
       );
     }
   }
+  if (createdIds.length > 1 && workspace) {
+    const lineage = {
+      planId: plan.createdAt,
+      goal: plan.goal,
+      issues: sorted.map((issue, idx) => ({
+        id: createdIds[idx] ?? `unknown-${idx}`,
+        title: issue.title,
+        order: issue.order
+      }))
+    };
+    try {
+      saveLineage(workspace, lineage);
+    } catch {
+    }
+  }
   return createdIds;
 }
 

package/dist/{chunk-5724FKM6.js → chunk-6VIN5PMW.js}

@@ -11,7 +11,7 @@ import {
   normalizeLabels,
   ok,
   warn
-} from "./chunk-PPRXJHPW.js";
+} from "./chunk-V44FTYWZ.js";
 import {
   appendEntry,
   buildGuardrailsSection,
@@ -1180,7 +1180,8 @@ function buildPrompt(opts) {
     step,
     previousResults = [],
     isLastStep = false,
-    relevantFiles
+    relevantFiles,
+    lineageBlock
   } = opts;
   let manifestPath = opts.manifestPath;
   if (!manifestPath && variant === "branch" && config) {
@@ -1194,6 +1195,7 @@ function buildPrompt(opts) {
   const depBlock = issue.dependency ? buildDependencyContext(issue.dependency) : "";
   const specWarningBlock = buildSpecWarningBlock(issue.specWarning);
   const contextMdBlock = buildContextMdBlock(repoContextMd ?? null);
+  const dodBlock = buildDefinitionOfDone(issue.description ?? "");
   const relevantFilesBlock = relevantFiles ?? "";
   const prBase = issue.dependency ? issue.dependency.branch : baseBranch;
   const prCreateBlock = buildPrCreateInstruction(platform2, prBase);
@@ -1266,10 +1268,7 @@ ${repoEntries}
    - Verify each acceptance criteria (if present)
    - Respect any stack or technical constraints (if present)
 ${testBlock}${hookBlock}
-4. **Validate**: Run the project's linter/typecheck/tests if available:
-   - Check \`package.json\` (or equivalent) for lint, typecheck, check, or test scripts.
-   - Run whichever validation scripts exist (e.g., \`npm run lint\`, \`npm run typecheck\`).
-   - Fix any errors before proceeding.
+4. ${buildValidateStep(testRunner ?? null, pm)}
 ${readmeBlock}
 **CRITICAL \u2014 Do NOT stop here. The following steps (commit, push, PR, manifest) are MANDATORY. Skipping them means the task has FAILED.**
 
@@ -1305,10 +1304,7 @@ ${readmeBlock}
    - Follow the implementation instructions exactly
    - Verify each acceptance criteria relevant to your scope
 ${testBlock}${hookBlock}
-2. **Validate**: Run the project's linter/typecheck/tests if available:
-   - Check \`package.json\` (or equivalent) for lint, typecheck, check, or test scripts.
-   - Run whichever validation scripts exist (e.g., \`npm run lint\`, \`npm run typecheck\`).
-   - Fix any errors before proceeding.
+2. ${buildValidateStep(testRunner ?? null, pm)}
 ${readmeBlock}
 **CRITICAL \u2014 Do NOT stop here. The following steps (commit, push, PR, manifest) are MANDATORY. Skipping them means the task has FAILED.**
 
@@ -1341,10 +1337,7 @@ ${trackerStep}
    - Verify each acceptance criteria (if present)
    - Respect any stack or technical constraints (if present)
 ${testBlock}${hookBlock}
-2. **Validate**: Run the project's linter/typecheck/tests if available:
-   - Check \`package.json\` (or equivalent) for lint, typecheck, check, or test scripts.
-   - Run whichever validation scripts exist (e.g., \`npm run lint\`, \`npm run typecheck\`).
-   - Fix any errors before proceeding.
+2. ${buildValidateStep(testRunner ?? null, pm)}
 ${readmeBlock}
 **CRITICAL \u2014 Do NOT stop here. The following steps (commit, push, PR, manifest) are MANDATORY. Skipping them means the task has FAILED.**
 
@@ -1384,6 +1377,7 @@ ${branchRenameInstruction}
     rulesSection = buildRulesSection(projectContext?.environment);
   }
   const taskHint = buildTaskTypeHint(issue.title);
+  const lineageSection = lineageBlock ?? "";
   return `${preamble}${taskHint}
 ${workContext}${contextBlock ? `
 ${contextBlock}
@@ -1391,6 +1385,8 @@ ${contextBlock}
 ${relevantFilesBlock}
 ` : ""}${depBlock ? `
 ${depBlock}
+` : ""}${lineageSection ? `
+${lineageSection}
 ` : ""}
 ## Issue
 
@@ -1401,7 +1397,7 @@ ${depBlock}
 ### Description
 
 ${issue.description}
-${specWarningBlock}${scopeSection}${GUARDRAILS_PLACEHOLDER}
+${specWarningBlock}${dodBlock}${scopeSection}${GUARDRAILS_PLACEHOLDER}
 ${instructions}
 
 ${rulesSection}
@@ -1415,7 +1411,7 @@ Before finishing, verify ALL of the following are true:
 - [ ] Manifest file is written with \`prUrl\` field
 If ANY item is unchecked, go back and complete it. Do NOT finish with incomplete steps.`;
 }
-function buildImplementPrompt(issue, config, testRunner, pm, projectContext, cwd, manifestPath, repoContextMd, relevantFiles) {
+function buildImplementPrompt(issue, config, testRunner, pm, projectContext, cwd, manifestPath, repoContextMd, relevantFiles, lineageBlock) {
   const workspace = resolve(config.workspace);
   const resolvedManifestPath = manifestPath ?? getManifestPath(workspace);
   if (config.workflow === "worktree") {
@@ -1430,7 +1426,8 @@ function buildImplementPrompt(issue, config, testRunner, pm, projectContext, cwd
       cwd,
       platform: config.platform,
       repoContextMd,
-      relevantFiles
+      relevantFiles,
+      lineageBlock
     });
   }
   return buildPrompt({
@@ -1445,7 +1442,8 @@ function buildImplementPrompt(issue, config, testRunner, pm, projectContext, cwd
     platform: config.platform,
     repoContextMd,
     config,
-    relevantFiles
+    relevantFiles,
+    lineageBlock
   });
 }
 function buildNativeWorktreePrompt(issue, repoPath, testRunner, pm, baseBranch, projectContext, manifestPath, platform2 = "cli", repoContextMd, relevantFiles) {
@@ -1484,12 +1482,26 @@ function buildTestInstructions(testRunner, pm = "npm") {
   if (!testRunner) return "";
   const testCmd = pm === "bun" ? "bun run test" : `${pm} run test`;
   return `
-**MANDATORY \u2014 Unit Tests:**
-This project uses **${testRunner}** as its test runner.
-- You MUST write unit tests (\`*.test.ts\`) for every new file or module you create.
-- Tests should cover the main functionality, edge cases, and error scenarios.
-- Run \`${testCmd}\` and ensure ALL tests pass before committing.
-- Do NOT skip writing tests \u2014 the PR will be blocked if tests are missing or failing.
+**MANDATORY \u2014 Test-Driven Development (TDD):**
+This project uses **${testRunner}**. Follow the RED \u2192 GREEN \u2192 REFACTOR cycle strictly:
+1. **RED**: Write the failing tests first \u2014 before writing any implementation code.
+   Run \`${testCmd}\` and confirm the new tests fail. If they pass immediately, the tests are wrong.
+2. **GREEN**: Write the minimum implementation to make the tests pass.
+   Run \`${testCmd}\` \u2014 all tests must pass before continuing.
+3. **REFACTOR**: Clean up the code without breaking tests. Run \`${testCmd}\` one final time to confirm.
+- Cover the main functionality, edge cases, and error scenarios.
+- Do NOT write implementation before tests. The PR will be blocked if tests are missing or written after the fact.
+`;
+}
+function buildDefinitionOfDone(description) {
+  const criteria = description.split("\n").map((l) => l.trim()).filter((l) => /^- \[ \]/.test(l));
+  if (criteria.length === 0) return "";
+  return `
+## Definition of Done
+
+Verify each item before finishing:
+
+${criteria.join("\n")}
 `;
 }
 function buildSpecWarningBlock(warning) {
@@ -1525,6 +1537,14 @@ function buildEnvironmentDependencyRule(env) {
   }
   return "";
 }
+function buildValidateStep(testRunner, pm = "npm") {
+  const testCmd = pm === "bun" ? "bun run test" : `${pm} run test`;
+  const testLine = testRunner ? `   - Run \`${testCmd}\` \u2014 ALL tests must pass (final gate after the TDD cycle).
+` : "";
+  return `**Validate**: Confirm all quality gates before committing:
+${testLine}   - Run lint/typecheck scripts if available (e.g., \`npm run lint\`, \`npm run typecheck\`).
+   - Fix every error. Do NOT commit with failing tests or lint errors.`;
+}
 function buildPreCommitHookInstructions() {
   return `
 **Pre-commit hooks:**
@@ -1731,7 +1751,7 @@ If ANY item is unchecked, go back and complete it. Do NOT finish with incomplete
 // src/providers/aider.ts
 import { mkdtempSync as mkdtempSync2, rmSync as rmSync2, writeFileSync as writeFileSync2 } from "fs";
 import { tmpdir as tmpdir2 } from "os";
-import { join as join4 } from "path";
+import { join as join5 } from "path";
 
 // src/providers/output-buffer.ts
 import { appendFileSync } from "fs";
@@ -1791,7 +1811,7 @@ var OutputBuffer = class {
 import { execFile as execFile2, spawn as spawn3 } from "child_process";
 import { mkdtempSync, rmSync, writeFileSync } from "fs";
 import { tmpdir } from "os";
-import { join as join3 } from "path";
+import { join as join4 } from "path";
 import { promisify as promisify2 } from "util";
 
 // src/session/overseer.ts
@@ -1833,6 +1853,13 @@ function createSessionTimeout(proc, timeoutSeconds) {
   };
 }
 
+// src/session/activity.ts
+import { closeSync, openSync, readdirSync as readdirSync2, readSync, statSync as statSync2 } from "fs";
+import { homedir } from "os";
+import { join as join3 } from "path";
+var TAIL_BYTES = 128 * 1024;
+var DEFAULT_IDLE_THRESHOLD_MS = 5 * 60 * 1e3;
+
 // src/session/overseer.ts
 var execFileAsync = promisify(execFile);
 var STUCK_MESSAGE = "\n[lisa-overseer] Provider killed: no git changes detected within the stuck threshold. Eligible for fallback.\n";
@@ -2052,8 +2079,8 @@ async function isCommandAvailable(command, args = ["--version"]) {
 }
 async function runProviderProcess(config, prompt, opts) {
   const start = Date.now();
-  const tmpDir = mkdtempSync(join3(tmpdir(), "lisa-"));
-  const promptFile = join3(tmpDir, "prompt.md");
+  const tmpDir = mkdtempSync(join4(tmpdir(), "lisa-"));
+  const promptFile = join4(tmpDir, "prompt.md");
   writeFileSync(promptFile, prompt, { encoding: "utf-8", mode: 384 });
   try {
     const promptCatExpr = `"$(cat '${escapeShellPath(promptFile)}')"`;
@@ -2180,8 +2207,8 @@ var AiderProvider = class {
     try {
       if (opts.model) validateShellArg(opts.model, "model");
       const modelFlag = opts.model ? `--model ${opts.model}` : "";
-      const aiderTmpDir = mkdtempSync2(join4(tmpdir2(), "lisa-aider-"));
-      const aiderPromptFile = join4(aiderTmpDir, "prompt.md");
+      const aiderTmpDir = mkdtempSync2(join5(tmpdir2(), "lisa-aider-"));
+      const aiderPromptFile = join5(aiderTmpDir, "prompt.md");
       writeFileSync2(aiderPromptFile, prompt, { encoding: "utf-8", mode: 384 });
       const config = {
         name: "aider",
@@ -4310,9 +4337,9 @@ function resolveModels(config) {
 
 // src/session/context-manager.ts
 import { existsSync as existsSync3, mkdirSync, readFileSync as readFileSync3, writeFileSync as writeFileSync3 } from "fs";
-import { join as join5 } from "path";
+import { join as join6 } from "path";
 function getContextPath(dir) {
-  return join5(dir, ".lisa", "context.md");
+  return join6(dir, ".lisa", "context.md");
 }
 function contextExists(dir) {
   return existsSync3(getContextPath(dir));
@@ -4327,6 +4354,70 @@ function readContext(dir) {
   }
 }
 
+// src/plan/lineage.ts
+import { existsSync as existsSync4, mkdirSync as mkdirSync2, readdirSync as readdirSync3, readFileSync as readFileSync4, writeFileSync as writeFileSync4 } from "fs";
+import { join as join7 } from "path";
+function lineageDir(workspace) {
+  return join7(workspace, ".lisa", "lineage");
+}
+function sanitizePlanId(planId) {
+  return planId.replace(/[^a-zA-Z0-9_\-.]/g, "_");
+}
+function lineagePath(workspace, planId) {
+  return join7(lineageDir(workspace), `${sanitizePlanId(planId)}.json`);
+}
+function saveLineage(workspace, lineage) {
+  const dir = lineageDir(workspace);
+  if (!existsSync4(dir)) {
+    mkdirSync2(dir, { recursive: true });
+  }
+  const path = lineagePath(workspace, lineage.planId);
+  writeFileSync4(path, JSON.stringify(lineage, null, "	"), "utf-8");
+}
+function loadLineageForIssue(workspace, issueId) {
+  const dir = lineageDir(workspace);
+  if (!existsSync4(dir)) return null;
+  let files;
+  try {
+    files = readdirSync3(dir).filter((f) => f.endsWith(".json"));
+  } catch {
+    return null;
+  }
+  for (const file of files) {
+    try {
+      const raw = readFileSync4(join7(dir, file), "utf-8");
+      const lineage = JSON.parse(raw);
+      if (lineage.issues.some((issue) => issue.id === issueId)) {
+        return lineage;
+      }
+    } catch {
+    }
+  }
+  return null;
+}
+function buildLineagePromptBlock(lineage, currentIssueId) {
+  if (lineage.issues.length <= 1) return "";
+  const sorted = [...lineage.issues].sort((a, b) => a.order - b.order);
+  const taskList = sorted.map((issue, idx) => {
+    const marker = issue.id === currentIssueId ? " <-- (this task)" : "";
+    return `  ${idx + 1}. [${issue.id}] ${issue.title}${marker}`;
+  }).join("\n");
+  const siblings = sorted.filter((issue) => issue.id !== currentIssueId).map((issue) => `- [${issue.id}] ${issue.title}`).join("\n");
+  return `## Task Hierarchy
+
+**Goal:** ${lineage.goal}
+
+This task is part of a decomposed plan with ${lineage.issues.length} subtasks:
+
+${taskList}
+
+## Parallel Work
+
+The following sibling tasks may be running concurrently. Do NOT duplicate their work:
+
+${siblings}`;
+}
+
 export {
   analyzeProject,
   runValidationCommands,
@@ -4359,5 +4450,8 @@ export {
   contextExists,
   readContext,
   WATCH_POLL_INTERVAL_MS,
-  resolveModels
+  resolveModels,
+  saveLineage,
+  loadLineageForIssue,
+  buildLineagePromptBlock
 };