From 2b47cac3ed516437507ced08c1ce7b4326abe6a1 Mon Sep 17 00:00:00 2001 From: Armando Fernandez Date: Sat, 5 Sep 2026 19:54:56 -0700 Subject: [PATCH] Adopt straightforward planning with visible acceptance checks and final testing --- MandoCode | 2 +- src/MandoCode.Desktop/Services/PlanCardHtml.cs | 11 ++++++++++- .../ViewModels/PlanInstructionEditor.cs | 2 ++ 3 files changed, 13 insertions(+), 2 deletions(-) diff --git a/MandoCode b/MandoCode index 1f7a7ec..05ad42c 160000 --- a/MandoCode +++ b/MandoCode @@ -1 +1 @@ -Subproject commit 1f7a7ec89aa8a12f276e906e330ec398332146dc +Subproject commit 05ad42ca40eedd7b3556d81aaa6393ab19b94baa diff --git a/src/MandoCode.Desktop/Services/PlanCardHtml.cs b/src/MandoCode.Desktop/Services/PlanCardHtml.cs index a6e8898..0dd8b22 100644 --- a/src/MandoCode.Desktop/Services/PlanCardHtml.cs +++ b/src/MandoCode.Desktop/Services/PlanCardHtml.cs @@ -15,8 +15,17 @@ public static string Build(TaskPlan plan) sb.Append("
Proposed plan
"); sb.Append(""); foreach (var step in plan.Steps) + { sb.Append($"" + - $""); + $""); + } sb.Append("
StepDescriptionWhat it will do
{step.StepNumber}{Escape(step.Description)}{Escape(step.Instruction)}
{Escape(step.Instruction)}"); + if (step.AcceptanceCriteria.Count > 0) + { + sb.Append("

Acceptance checks

    "); + foreach (var criterion in step.AcceptanceCriteria) sb.Append($"
  1. {Escape(criterion)}
  2. "); + sb.Append("
"); + } + sb.Append("
"); return sb.ToString(); } diff --git a/src/MandoCode.Desktop/ViewModels/PlanInstructionEditor.cs b/src/MandoCode.Desktop/ViewModels/PlanInstructionEditor.cs index 8ea58ac..b20009c 100644 --- a/src/MandoCode.Desktop/ViewModels/PlanInstructionEditor.cs +++ b/src/MandoCode.Desktop/ViewModels/PlanInstructionEditor.cs @@ -12,6 +12,8 @@ public static bool Apply(TaskStep step, string? revisedInstruction) var revised = revisedInstruction.Trim(); step.Instruction = revised; + step.AcceptanceCriteria = [revised]; + step.EvidenceFollowupUsed = false; // The description is a UI label, but a stale label makes the review card misleading. // Always derive it from the instruction the user actually approved. step.Description = revised.Length > 60 ? revised[..57] + "..." : revised;