@tarcisiopgs/lisa 1.34.0 → 1.35.0

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

@@ -5,7 +5,7 @@ import {
   resolveModels,
   runWithFallback,
   saveLineage
-} from "./chunk-2DONWCCQ.js";
+} from "./chunk-6VIN5PMW.js";
 import {
   error,
   log,

package/dist/{chunk-2DONWCCQ.js → chunk-6VIN5PMW.js}

@@ -1195,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);
@@ -1267,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.**
 
@@ -1306,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.**
 
@@ -1342,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.**
 
@@ -1405,7 +1397,7 @@ ${lineageSection}
 ### Description
 
 ${issue.description}
-${specWarningBlock}${scopeSection}${GUARDRAILS_PLACEHOLDER}
+${specWarningBlock}${dodBlock}${scopeSection}${GUARDRAILS_PLACEHOLDER}
 ${instructions}
 
 ${rulesSection}
@@ -1490,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) {
@@ -1531,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:**

package/dist/{chunk-5FODVMWF.js → chunk-BWND35E5.js}

@@ -32,7 +32,7 @@ import {
   resolveModels,
   runValidationCommands,
   runWithFallback
-} from "./chunk-2DONWCCQ.js";
+} from "./chunk-6VIN5PMW.js";
 import {
   divider,
   error,

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   parseStructuredOutput,
   runPlanWizard,
   savePlan
-} from "./chunk-S2PRXET6.js";
+} from "./chunk-6SNT7TND.js";
 import {
   detectDefaultBranch,
   detectGitRepos,
@@ -37,7 +37,7 @@ import {
   runLoop,
   saveConfig,
   validateConfig
-} from "./chunk-5FODVMWF.js";
+} from "./chunk-BWND35E5.js";
 import {
   buildContextMdBlock,
   createProvider,
@@ -49,7 +49,7 @@ import {
   readContext,
   resolveModels,
   runWithFallback
-} from "./chunk-2DONWCCQ.js";
+} from "./chunk-6VIN5PMW.js";
 import {
   banner,
   error,
@@ -1555,7 +1555,7 @@ async function reviewAndCreate(plan2, planPath, opts) {
     log("Run `lisa run` when ready.");
     return;
   }
-  const { runLoop: runLoop2 } = await import("./loop-D4AKZUXG.js");
+  const { runLoop: runLoop2 } = await import("./loop-N5D27JQX.js");
   await runLoop2(config2, {
     once: false,
     watch: false,
@@ -2023,7 +2023,7 @@ Add them to your ${shell} and run: source ${shell}`));
     const initialCards = persistence.load();
     persistedCards = initialCards;
     persistence.start();
-    const { registerPlanBridge } = await import("./tui-bridge-6C4IRKUQ.js");
+    const { registerPlanBridge } = await import("./tui-bridge-LQDYRWHY.js");
     const cleanupPlan = registerPlanBridge(merged);
     onBeforeExit = () => {
       persistence.stop();

package/dist/{loop-D4AKZUXG.js → loop-N5D27JQX.js}

@@ -3,10 +3,10 @@ import {
   checkoutBaseBranches,
   runDemoLoop,
   runLoop
-} from "./chunk-5FODVMWF.js";
+} from "./chunk-BWND35E5.js";
 import {
   WATCH_POLL_INTERVAL_MS
-} from "./chunk-2DONWCCQ.js";
+} from "./chunk-6VIN5PMW.js";
 import "./chunk-V44FTYWZ.js";
 import "./chunk-3EOEDL3T.js";
 import "./chunk-7OCDGYDM.js";

package/dist/{tui-bridge-6C4IRKUQ.js → tui-bridge-LQDYRWHY.js}

@@ -6,12 +6,12 @@ import {
   markdownToIssue,
   parseStructuredOutput,
   savePlan
-} from "./chunk-S2PRXET6.js";
+} from "./chunk-6SNT7TND.js";
 import {
   createSource,
   resolveModels,
   runWithFallback
-} from "./chunk-2DONWCCQ.js";
+} from "./chunk-6VIN5PMW.js";
 import {
   error,
   kanbanEmitter,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tarcisiopgs/lisa",
-  "version": "1.34.0",
+  "version": "1.35.0",
   "description": "Autonomous issue resolver",
   "keywords": [
     "loop",