plan_driven 0.1.0 → 0.2.0

data/app/controllers/plan_driven/wizard/configuration_controller.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+module PlanDriven
+  module Wizard
+    # The interview's questions and the service connections. Like every other page, it only
+    # reads; each form runs one `plan-driven` command.
+    class ConfigurationController < ApplicationController
+      def show
+        config = PlanDriven.configuration
+        @template = config.template
+        @base = config.base_template
+        @needed = Connections.needed(config).map(&:name)
+      end
+
+      def question
+        job = start(Commands.argv("question", question_fields))
+        redirect_to configuration_path(job: job.id, anchor: "questions")
+      rescue ArgumentError => e
+        redirect_to configuration_path(anchor: "questions"), alert: e.message
+      end
+
+      def connect
+        service = Connections.find(params[:service])
+        key = params[:api_key].to_s.strip
+        raise ArgumentError, "Paste the #{service.title} key first." if key.empty?
+
+        job = start(Commands.argv("connect", service: service.name), stdin: "#{key}\n")
+        redirect_to configuration_path(job: job.id, anchor: "connections")
+      rescue ArgumentError => e
+        redirect_to configuration_path(anchor: "connections"), alert: e.message
+      end
+
+      def check
+        job = start(Commands.argv(params[:do] == "questions" ? "questions" : "doctor", {}))
+        redirect_to configuration_path(job: job.id, anchor: params[:do] == "questions" ? "questions" : "connections")
+      end
+
+      private
+
+      # Only what differs from the question as it is now, so an unchanged field stays the default.
+      def question_fields
+        key = params[:key].to_s.strip
+        return { "key" => key, "remove" => "1" } if params[:remove].present?
+
+        current = PlanDriven.configuration.template[key]
+        fields = changed_text(current).merge("key" => key)
+        required = params[:required] == "1"
+        fields["required"] = required ? "1" : "0" if current.nil? || required != current.required
+        fields
+      end
+
+      def changed_text(current)
+        %w[title question group].each_with_object({}) do |name, fields|
+          value = params[name].to_s.strip
+          next if value.empty? || (current && value == current.public_send(name).to_s)
+
+          fields[name] = value
+        end
+      end
+    end
+  end
+end

data/app/controllers/plan_driven/wizard/jobs_controller.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module PlanDriven
+  module Wizard
+    # Polled by the terminal panel while a command runs.
+    class JobsController < ApplicationController
+      def show
+        data = Job.find(params[:id]).to_h
+        data["plan"] = data["output"][/✓ (PD-\d+) drafted/, 1]
+        render json: data
+      rescue ArgumentError => e
+        render json: { error: e.message }, status: :not_found
+      end
+    end
+  end
+end

data/app/controllers/plan_driven/wizard/plans_controller.rb

@@ -0,0 +1,102 @@
+# frozen_string_literal: true
+
+require "tempfile"
+
+module PlanDriven
+  module Wizard
+    # Every form posts here, and every post becomes one `plan-driven` command. The pages only read.
+    class PlansController < ApplicationController
+      before_action :find_plan, only: %i[show run file]
+
+      def index
+        @plans = Plan.order(id: :desc)
+      end
+
+      def new
+        @template = PlanDriven.configuration.template
+      end
+
+      def create
+        @template = PlanDriven.configuration.template
+        answers = interview_answers
+        missing = @template.asked.select { |section| section.required && answers[section.key].to_s.strip.empty? }
+        return missing_answers(missing) if missing.any?
+
+        job = start(Commands.argv("new", title: params[:title]), stdin: interview_input(answers))
+        redirect_to root_path(job: job.id)
+      rescue ArgumentError => e
+        redirect_to new_plan_path, alert: e.message
+      end
+
+      def show
+        @step = params[:step].presence || Steps.current(@plan)
+        @template = PlanDriven.configuration.template
+      end
+
+      def run
+        job = start(Commands.argv(params[:do], command_fields))
+        redirect_to plan_path(@plan.key, params[:step].presence, job: job.id, open: params[:open].presence,
+                                                                 anchor: params[:open].presence)
+      rescue ArgumentError => e
+        redirect_to plan_path(@plan.key, params[:step].presence), alert: e.message
+      end
+
+      def run_global
+        job = start(Commands.argv(params[:do], {}))
+        redirect_to root_path(job: job.id)
+      rescue ArgumentError => e
+        redirect_to root_path, alert: e.message
+      end
+
+      def actor
+        session[:plan_driven_actor] = params[:actor].to_s.strip.presence
+        redirect_back fallback_location: root_path
+      end
+
+      def file
+        path = Renderer.directory(@plan).join(params[:name])
+        return head(:not_found) unless path.file?
+
+        send_file path, disposition: "inline"
+      end
+
+      private
+
+      def find_plan
+        @plan = Plan.find_by_reference!(params[:key])
+      rescue ActiveRecord::RecordNotFound => e
+        redirect_to root_path, alert: e.message
+      end
+
+      def command_fields
+        fields = params.permit(:section, :instruction, :role, :note, :ticket, :feedback, :text, tickets: []).to_h
+        fields[:plan] = @plan.key
+        text = fields.delete("text")
+        fields[:file] = section_file(text) if params[:do] == "edit"
+        fields
+      end
+
+      def interview_answers
+        params.fetch(:answers, {}).permit(*@template.asked.map(&:key)).to_h
+      end
+
+      def missing_answers(missing)
+        flash.now[:alert] = "#{missing.map(&:title).join(", ")} #{missing.one? ? "is" : "are"} required."
+        render(:new, status: :unprocessable_entity)
+      end
+
+      def interview_input(answers)
+        Commands.interview_input(@template, answers)
+      end
+
+      # `edit --from` reads the new text from a file; it stays in tmp/ so the command can be rerun.
+      def section_file(text)
+        dir = Job.dir.join("sections")
+        FileUtils.mkdir_p(dir)
+        path = dir.join("#{@plan.key}-#{params[:section]}-#{SecureRandom.hex(4)}.md")
+        File.write(path, text.to_s.gsub("\r\n", "\n"))
+        path.relative_path_from(PlanDriven.configuration.root_path).to_s
+      end
+    end
+  end
+end

data/app/views/layouts/plan_driven/wizard/application.html.erb

@@ -0,0 +1,139 @@
+<!doctype html>
+<html lang="en">
+<head>
+  <meta charset="utf-8">
+  <meta name="viewport" content="width=device-width, initial-scale=1">
+  <title><%= content_for?(:title) ? yield(:title) : "plan_driven" %></title>
+  <%= csrf_meta_tags %>
+  <style>
+    :root { --ink:#1f2330; --muted:#6b7280; --line:#e5e7eb; --bg:#f7f7f9; --accent:#cc342d; --ok:#15803d; --warn:#b45309; --bad:#b91c1c; }
+    * { box-sizing: border-box; }
+    [id] { scroll-margin-top: 64px; }
+    body { margin:0; font:15px/1.5 -apple-system, BlinkMacSystemFont, "Segoe UI", Inter, sans-serif; color:var(--ink); background:var(--bg); }
+    a { color:var(--accent); }
+    header.top { display:flex; align-items:center; gap:16px; padding:12px 24px; height:53px; background:#fff; border-bottom:1px solid var(--line); position:sticky; top:0; z-index:10; }
+    header.top .brand { font-weight:700; text-decoration:none; color:var(--ink); }
+    header.top .brand span { color:var(--accent); }
+    header.top form { margin-left:auto; display:flex; gap:6px; align-items:center; color:var(--muted); font-size:13px; }
+    .layout { display:grid; grid-template-columns:minmax(0, 1fr) minmax(360px, 42%); min-height:calc(100vh - 53px); }
+    main { padding:24px 32px 48px; }
+    aside.terminal { background:#0f1117; color:#d6d9e0; display:flex; flex-direction:column; position:sticky; top:53px; height:calc(100vh - 53px); }
+    .terminal .bar { display:flex; align-items:center; gap:8px; padding:10px 14px; border-bottom:1px solid #232734; font-size:12px; color:#8b93a7; }
+    .terminal .bar .state { margin-left:auto; padding:2px 8px; border-radius:10px; background:#232734; }
+    .terminal .bar .state.running { background:#1e3a8a; color:#dbeafe; }
+    .terminal .bar .state.succeeded { background:#14532d; color:#dcfce7; }
+    .terminal .bar .state.failed { background:#7f1d1d; color:#fee2e2; }
+    .terminal pre { margin:0; padding:14px; overflow:auto; flex:1; font:13px/1.45 ui-monospace, SFMono-Regular, Menlo, monospace; white-space:pre-wrap; word-break:break-word; }
+    .terminal .cmd { color:#fbbf24; }
+    .terminal .ok { color:#4ade80; } .terminal .warn { color:#fbbf24; } .terminal .bad { color:#f87171; }
+    .terminal .empty { color:#6b7280; padding:14px; font-size:13px; }
+    h1 { font-size:24px; margin:0 0 4px; } h2 { font-size:18px; margin:28px 0 8px; } h3 { font-size:15px; margin:18px 0 6px; }
+    .muted { color:var(--muted); }
+    .card { background:#fff; border:1px solid var(--line); border-radius:10px; padding:16px 18px; margin:12px 0; }
+    .steps { display:flex; gap:4px; margin:16px 0 20px; padding:0; list-style:none; }
+    .steps li { flex:1; }
+    .steps a, .steps span { display:block; padding:8px 10px; border-radius:8px; font-size:13px; text-decoration:none; color:var(--muted); background:#eef0f3; }
+    .steps a { color:var(--ink); background:#fff; border:1px solid var(--line); }
+    .steps .here { background:var(--accent); color:#fff; border-color:var(--accent); }
+    .steps b { display:block; font-size:14px; }
+    .nav { display:flex; justify-content:space-between; margin-top:28px; }
+    button, .button { font:inherit; font-size:14px; padding:7px 14px; border-radius:8px; border:1px solid var(--line); background:#fff; color:var(--ink); cursor:pointer; text-decoration:none; display:inline-block; }
+    button.primary, .button.primary { background:var(--accent); color:#fff; border-color:var(--accent); }
+    button:disabled, .button.disabled { opacity:.45; cursor:not-allowed; pointer-events:none; }
+    body.busy form button { opacity:.45; pointer-events:none; }
+    input[type=text], textarea { font:inherit; width:100%; padding:8px 10px; border:1px solid var(--line); border-radius:8px; background:#fff; }
+    textarea { min-height:90px; font:13px/1.45 ui-monospace, SFMono-Regular, Menlo, monospace; }
+    header.top input[type=text] { width:260px; padding:4px 8px; font-size:13px; }
+    label { display:block; font-weight:600; margin:14px 0 4px; }
+    label small { font-weight:400; color:var(--muted); }
+    .row { display:flex; gap:8px; align-items:center; flex-wrap:wrap; }
+    .row input[type=text] { flex:1; min-width:180px; }
+    table { border-collapse:collapse; width:100%; font-size:14px; }
+    th, td { text-align:left; padding:7px 8px; border-bottom:1px solid var(--line); vertical-align:top; }
+    .pill { display:inline-block; font-size:12px; padding:1px 8px; border-radius:10px; background:#eef0f3; }
+    .pill.good { background:#dcfce7; color:var(--ok); } .pill.bad { background:#fee2e2; color:var(--bad); } .pill.wait { background:#fef3c7; color:var(--warn); }
+    .errors li { color:var(--bad); } .warnings li { color:var(--warn); }
+    details.section { background:#fff; border:1px solid var(--line); border-radius:10px; margin:8px 0; }
+    details.section > summary { padding:10px 14px; cursor:pointer; font-weight:600; }
+    details.section > div { padding:0 16px 14px; }
+    .doc table { font-size:13px; } .doc pre { background:#f3f4f6; padding:10px; border-radius:6px; overflow:auto; font-size:12px; }
+    header.top nav { display:flex; gap:14px; margin-left:12px; font-size:14px; }
+    header.top nav a { color:var(--muted); text-decoration:none; } header.top nav a:hover { color:var(--ink); }
+    input[type=password], select { font:inherit; padding:8px 10px; border:1px solid var(--line); border-radius:8px; background:#fff; }
+    .row input[type=password] { flex:1; min-width:180px; }
+    .grid2 { display:grid; grid-template-columns:minmax(0, 1fr) minmax(0, 2fr); gap:12px; }
+    label.check { display:inline-flex; gap:6px; align-items:center; font-weight:400; margin:0; }
+    .card.unused { opacity:.75; }
+    .flash { padding:10px 14px; border-radius:8px; margin-bottom:12px; background:#fee2e2; color:var(--bad); }
+  </style>
+</head>
+<body>
+  <header class="top">
+    <a class="brand" href="<%= root_path %>">plan<span>_</span>driven <span class="muted" style="font-weight:400">wizard</span></a>
+    <nav><%= link_to "Plans", root_path %> <%= link_to "Configuration", configuration_path %></nav>
+    <%= form_with url: actor_path, method: :post do |f| %>
+      acting as <%= f.text_field :actor, value: acting_as %> <button type="submit">Set</button>
+    <% end %>
+  </header>
+  <div class="layout">
+    <main>
+      <% if flash[:alert] %><div class="flash"><%= flash[:alert] %></div><% end %>
+      <%= yield %>
+    </main>
+    <aside class="terminal" data-job="<%= current_job && job_path(current_job.id) %>"
+           data-plan-url="<%= plan_path("PLAN_KEY") %>" data-running="<%= current_job&.state&.dig("status") == "running" %>">
+      <div class="bar">
+        <span>●</span><span>terminal · what the wizard runs</span>
+        <span class="state <%= current_job&.state&.dig("status") %>"><%= current_job ? current_job.state["status"] : "idle" %></span>
+      </div>
+      <% if current_job %>
+        <pre id="term"><span class="cmd">$ <%= current_job.state["command"] %></span>
+<%= current_job.output %></pre>
+      <% else %>
+        <div class="empty">Every button on the left runs one plan-driven command. It shows up here, with its output, exactly as it would in your terminal.</div>
+      <% end %>
+    </aside>
+  </div>
+  <script>
+    (function () {
+      var aside = document.querySelector("aside.terminal");
+      var url = aside.dataset.job;
+      if (!url) return;
+      var pre = document.getElementById("term");
+      var badge = aside.querySelector(".state");
+      var wasRunning = aside.dataset.running === "true";
+      function escape(text) { return text.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;"); }
+      function paint(text) {
+        return escape(text).split("\n").map(function (line) {
+          if (/^\s*✓/.test(line)) return '<span class="ok">' + line + "</span>";
+          if (/^\s*!/.test(line)) return '<span class="warn">' + line + "</span>";
+          if (/^\s*✗/.test(line)) return '<span class="bad">' + line + "</span>";
+          return line;
+        }).join("\n");
+      }
+      function render(job) {
+        pre.innerHTML = '<span class="cmd">$ ' + escape(job.command) + "</span>\n" + paint(job.output);
+        pre.scrollTop = pre.scrollHeight;
+        badge.textContent = job.status;
+        badge.className = "state " + job.status;
+      }
+      function poll() {
+        fetch(url, { headers: { Accept: "application/json" } }).then(function (r) { return r.json(); }).then(function (job) {
+          render(job);
+          if (job.status === "running") { document.body.classList.add("busy"); return setTimeout(poll, 600); }
+          document.body.classList.remove("busy");
+          if (!wasRunning) return;
+          setTimeout(function () {
+            if (job.plan && !/\/plans\/PD-/.test(location.pathname)) {
+              location.href = aside.dataset.planUrl.replace("PLAN_KEY", job.plan) + location.search;
+            } else {
+              location.reload();
+            }
+          }, 900);
+        });
+      }
+      poll();
+    })();
+  </script>
+</body>
+</html>

data/app/views/plan_driven/wizard/configuration/show.html.erb

@@ -0,0 +1,104 @@
+<% content_for :title, "Configuration" %>
+<% config = PlanDriven.configuration %>
+<h1>Configuration</h1>
+<p class="muted">Connect the services this app uses, and change the interview. Every button runs one
+  <code>plan-driven</code> command; the panel on the right shows it.</p>
+
+<h2 id="connections">Connections</h2>
+<div class="card">
+  <p style="margin-top:0">
+    Plans and tickets are drafted with <strong><%= config.llm_provider %> · <%= config.llm_model %></strong>.
+    Tickets go to
+    <strong><%= config.agent_provider.to_sym == :local ? "local agents (#{config.agent_command.presence || "no agent_command set"})" : "Cursor cloud agents" %></strong>.
+  </p>
+  <p class="muted" style="margin-bottom:0">To use another model or agent, change it in
+    <code>config/initializers/plan_driven.rb</code>, for example <code>config.llm_provider = :anthropic</code>, or
+    <code>config.agent_provider = :local</code> with <code>config.agent_command = "claude -p"</code>, then restart the server.</p>
+</div>
+
+<% PlanDriven::Connections::SERVICES.sort_by { |s| @needed.include?(s.name) ? 0 : 1 }.each do |service| %>
+  <% source = PlanDriven::Credentials.source(service.key) %>
+  <% used = @needed.include?(service.name) %>
+  <div class="card service <%= "unused" unless used %>" id="service_<%= service.name %>">
+    <div class="row" style="justify-content:space-between">
+      <div>
+        <strong><%= service.title %></strong>
+        <% if source %>
+          <span class="pill good">key set · <%= source.sub(Dir.home, "~") %></span>
+        <% elsif used %>
+          <span class="pill bad">not connected</span>
+        <% else %>
+          <span class="pill">not used by this configuration</span>
+        <% end %>
+        <div class="muted" style="font-size:13px"><%= service.purpose %>.
+          <a href="<%= service.url %>" target="_blank" rel="noopener">Get a key ↗</a></div>
+      </div>
+    </div>
+    <%= form_with url: configuration_connect_path, method: :post, class: "row connect", style: "margin-top:10px" do %>
+      <%= hidden_field_tag :service, service.name %>
+      <%= password_field_tag :api_key, nil, id: "api_key_#{service.name}", autocomplete: "off",
+                             placeholder: "Paste the #{service.title} key", "aria-label": "#{service.title} key" %>
+      <button type="submit" class="<%= "primary" if used && !source %>"><%= source ? "Replace" : "Connect" %></button>
+    <% end %>
+  </div>
+<% end %>
+<p class="muted" style="font-size:13px">A key is checked with the service first, then stored in
+  <code>~/.plan_driven/config</code> (0600), never in the app. It goes to the command on stdin, so it isn't in
+  the panel or the logs. A key in the environment always wins.</p>
+<%= button_to "Check every connection", configuration_check_path(do: "doctor"), form: { style: "display:inline" } %>
+
+<h2 id="questions">Interview questions</h2>
+<p class="muted">What <code>plan-driven new</code> and the New plan form ask. Changes are written to
+  <code><%= PlanDriven::Interview::PATH %></code>; commit it so the whole team gets the same interview. A new required question
+  also applies to plans still in draft: they need an answer before they can be submitted.</p>
+
+<% @template.asked.each do |section| %>
+  <% origin = PlanDriven::Interview.origin(section.key, template: @base) %>
+  <div class="card question" id="question_<%= section.key %>">
+    <%= form_with url: configuration_question_path, method: :post do %>
+      <%= hidden_field_tag :key, section.key %>
+      <div class="row" style="justify-content:space-between">
+        <span><code><%= section.key %></code>
+          <span class="pill <%= origin == "default" ? "" : "wait" %>"><%= origin %></span>
+          <span class="muted" style="font-size:13px"><%= section.group %></span></span>
+        <label class="check"><%= hidden_field_tag :required, "0", id: nil %>
+          <%= check_box_tag :required, "1", section.required, id: "required_#{section.key}" %> required</label>
+      </div>
+      <div class="grid2">
+        <div><label for="title_<%= section.key %>">Title</label>
+          <%= text_field_tag :title, section.title, id: "title_#{section.key}" %></div>
+        <div><label for="question_text_<%= section.key %>">Question</label>
+          <%= text_field_tag :question, section.question, id: "question_text_#{section.key}" %></div>
+      </div>
+      <div class="row" style="margin-top:10px">
+        <button type="submit">Save</button>
+        <% unless origin == "default" %>
+          <button type="submit" name="remove" value="1"><%= origin == "added" ? "Remove" : "Back to the default" %></button>
+        <% end %>
+      </div>
+    <% end %>
+  </div>
+<% end %>
+
+<div class="card" id="new_question">
+  <h3 style="margin-top:0">Add a question</h3>
+  <%= form_with url: configuration_question_path, method: :post do %>
+    <div class="grid2">
+      <div><label for="new_key">Key <small>lower case and _, like success_metric</small></label>
+        <%= text_field_tag :key, nil, id: "new_key", pattern: "[a-z][a-z0-9_]{1,39}", required: true %></div>
+      <div><label for="new_title">Title</label>
+        <%= text_field_tag :title, nil, id: "new_title", required: true %></div>
+    </div>
+    <label for="new_question_text">Question</label>
+    <%= text_field_tag :question, nil, id: "new_question_text", required: true,
+                       placeholder: "How will we know it worked?" %>
+    <div class="row" style="margin-top:10px">
+      <label for="new_group" style="margin:0">Plan section group</label>
+      <%= select_tag :group, options_for_select(PlanDriven::Template::GROUPS, "Overview"), id: "new_group" %>
+      <label class="check"><%= hidden_field_tag :required, "0", id: nil %>
+        <%= check_box_tag :required, "1", false, id: "new_required" %> required</label>
+      <button type="submit" class="primary" style="margin-left:auto">Add question</button>
+    </div>
+  <% end %>
+</div>
+<%= button_to "List the questions", configuration_check_path(do: "questions"), form: { style: "display:inline" } %>

data/app/views/plan_driven/wizard/plans/_agents.html.erb

@@ -0,0 +1,44 @@
+<div class="card row">
+  <span class="muted">Each ready ticket goes to its own coding agent, which opens one pull request.
+    Tickets wait for the ones they depend on.</span>
+  <span style="margin-left:auto"></span>
+  <%= run_button "Refresh agents & PRs", "status" %>
+  <%= run_button "Start ready tickets", "develop", {}, primary: true,
+                 disabled: !%w[tickets_approved in_development].include?(@plan.status) %>
+</div>
+
+<div class="card">
+  <table>
+    <thead><tr><th>Ticket</th><th>Status</th><th>Agent / PR</th><th></th></tr></thead>
+    <tbody>
+      <% @plan.tickets.each do |ticket| %>
+        <tr>
+          <td><strong><%= ticket.key %></strong> <%= ticket.title %>
+            <% if ticket.status == "approved" && ticket.blocked_by.any? %><div class="muted">waits for <%= ticket.blocked_by.join(", ") %></div><% end %></td>
+          <td><span class="pill <%= { "merged" => "good", "failed" => "bad", "changes_requested" => "bad" }.fetch(ticket.status, "wait") %>"><%= ticket.status.tr("_", " ") %></span></td>
+          <td>
+            <% if ticket.agent_url.present? %><%= link_to "agent", ticket.agent_url, target: "_blank", rel: "noopener" %><% end %>
+            <% if ticket.pr_url.present? %><%= link_to "PR ##{ticket.pr_number}", ticket.pr_url, target: "_blank", rel: "noopener" %><% end %>
+          </td>
+          <td>
+            <% if %w[pr_open pr_approved changes_requested].include?(ticket.status) %>
+              <div class="row">
+                <%= run_button "Review", "review", { ticket: ticket.key } %>
+                <%= run_button "Approve PR", "approve-pr", { ticket: ticket.key }, disabled: ticket.status != "pr_open" %>
+                <%= run_button "Merge", "merge", { ticket: ticket.key }, primary: true, disabled: ticket.status != "pr_approved" %>
+              </div>
+              <%= form_with url: run_plan_path(@plan.key), method: :post, class: "row", html: { style: "margin-top:6px" } do %>
+                <%= hidden_field_tag :do, "feedback", id: nil %><%= hidden_field_tag :step, "agents", id: nil %>
+                <%= hidden_field_tag :ticket, ticket.key, id: nil %>
+                <%= text_field_tag :feedback, nil, id: "feedback_#{ticket.key}", placeholder: "ask the agent for a change" %>
+                <button type="submit">Send</button>
+              <% end %>
+            <% elsif ticket.status == "failed" || ticket.ready? %>
+              <%= run_button "Start #{ticket.key}", "develop", { tickets: [ticket.key] } %>
+            <% end %>
+          </td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+</div>

data/app/views/plan_driven/wizard/plans/_approve.html.erb

@@ -0,0 +1,40 @@
+<% roles = PlanDriven.configuration.plan_approvals.map(&:to_s) %>
+<p class="muted">Approvals belong to a revision: editing the plan after this sends it back for approval.
+  You're signing as <strong><%= acting_as %></strong>.</p>
+
+<div class="card">
+  <table>
+    <thead><tr><th>Role</th><th>Revision <%= @plan.revision %></th><th></th></tr></thead>
+    <tbody>
+      <% roles.each do |role| %>
+        <% approval = @plan.approvals_for_revision.where(role: role).order(:created_at).last %>
+        <tr>
+          <td><strong><%= role %></strong></td>
+          <td>
+            <% if approval&.decision == "approved" %>
+              <span class="pill good">approved</span> <span class="muted">by <%= approval.actor %></span>
+            <% elsif approval %>
+              <span class="pill bad"><%= approval.decision %></span> <span class="muted"><%= approval.note %></span>
+            <% else %>
+              <span class="pill wait">waiting</span>
+            <% end %>
+          </td>
+          <td>
+            <% if @plan.status == "in_review" && approval&.decision != "approved" %>
+              <%= form_with url: run_plan_path(@plan.key), method: :post, class: "row" do %>
+                <%= hidden_field_tag :step, "approve", id: nil %><%= hidden_field_tag :role, role, id: nil %>
+                <%= text_field_tag :note, nil, id: "note_#{role}", placeholder: "note (needed to send it back)" %>
+                <button type="submit" name="do" value="approve" class="primary">Approve</button>
+                <button type="submit" name="do" value="reject">Send back</button>
+              <% end %>
+            <% end %>
+          </td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+</div>
+
+<% if @plan.status == "draft" %>
+  <p class="muted">The plan is a draft; submit it on the Plan step first.</p>
+<% end %>

data/app/views/plan_driven/wizard/plans/_finish.html.erb

@@ -0,0 +1,24 @@
+<div class="card row">
+  <span class="muted">Run the plan's Cucumber scenarios on the merged code, then write the report that ties
+    every acceptance criterion to its proof.</span>
+  <span style="margin-left:auto"></span>
+  <%= run_button "Run the evidence", "evidence" %>
+  <%= run_button "Tokens & cost", "usage" %>
+  <%= run_button "Write the delivery report", "report", {}, primary: true %>
+</div>
+
+<% if (run = @plan.evidence_runs.last) %>
+  <div class="card">
+    <strong>Last evidence run</strong>
+    <span class="pill <%= run.passed? ? "good" : "bad" %>"><%= run.status %></span>
+    <span class="muted"><%= run.scenarios.size %> scenarios<%= " at #{run.commit_sha.to_s[0, 7]}" if run.commit_sha %></span>
+  </div>
+<% end %>
+
+<% files = %w[delivery-report.html delivery-report.pdf].select { |name| PlanDriven::Renderer.directory(@plan).join(name).file? } %>
+<% if files.any? %>
+  <div class="card">
+    <strong>Delivery report</strong>
+    <% files.each do |name| %> · <%= link_to name, plan_file_path(@plan.key, name), target: "_blank", rel: "noopener" %><% end %>
+  </div>
+<% end %>

data/app/views/plan_driven/wizard/plans/_plan.html.erb

@@ -0,0 +1,52 @@
+<% report = PlanDriven::Guards::Report.from_h(@plan.guard_report) %>
+<div class="card">
+  <div class="row">
+    <strong>Guards</strong>
+    <span class="pill <%= report.ok? ? "good" : "bad" %>"><%= report.ok? ? "all checks passed" : "#{report.errors.size} to fix" %></span>
+    <span style="margin-left:auto"></span>
+    <%= run_button "Run the guards", "check" %>
+    <%= run_button "Write HTML & PDF", "pdf" %>
+    <%= run_button "Submit for review", "submit", primary: true, disabled: !(@plan.status == "draft" && report.ok?) %>
+  </div>
+  <%= guard_list(report) %>
+  <p class="muted" style="margin:6px 0 0">
+    <% %w[plan.html plan.pdf].each do |name| %>
+      <% if PlanDriven::Renderer.directory(@plan).join(name).file? %>
+        <%= link_to name, plan_file_path(@plan.key, name), target: "_blank", rel: "noopener" %>
+      <% end %>
+    <% end %>
+  </p>
+</div>
+
+<p class="muted">Every section can be changed: edit the text yourself, or tell the model what to change.
+  Either one is a new revision, checked by the guards again<%= ", and sends an approved plan back to draft" if @plan.status != "draft" %>.</p>
+
+<% @template.grouped.each do |group, sections| %>
+  <% next if sections.empty? %>
+  <h2><%= group %></h2>
+  <% sections.each do |section| %>
+    <details class="section" id="<%= section.key %>" <%= "open" if params[:open] == section.key %>>
+      <summary><%= section.title %>
+        <% if @plan.section(section.key).strip.empty? %><span class="pill wait">empty</span><% end %></summary>
+      <div>
+        <div class="doc"><%= markdown(@plan.section(section.key)) %></div>
+        <% if @plan.editable? %>
+          <% fields = { step: "plan", section: section.key, open: section.key } %>
+          <h3>Edit the text</h3>
+          <%= form_with url: run_plan_path(@plan.key), method: :post do %>
+            <% fields.merge(do: "edit").each do |name, value| %><%= hidden_field_tag name, value, id: nil %><% end %>
+            <%= text_area_tag :text, @plan.section(section.key), rows: 10, id: "text_#{section.key}" %>
+            <p><button type="submit">Save <%= section.title %></button></p>
+          <% end %>
+          <h3>Or ask for a redraft</h3>
+          <%= form_with url: run_plan_path(@plan.key), method: :post, class: "row" do %>
+            <% fields.merge(do: "redraft").each do |name, value| %><%= hidden_field_tag name, value, id: nil %><% end %>
+            <%= text_field_tag :instruction, nil, id: "instruction_#{section.key}",
+                                                  placeholder: "e.g. add a colour column to categories" %>
+            <button type="submit">Redraft</button>
+          <% end %>
+        <% end %>
+      </div>
+    </details>
+  <% end %>
+<% end %>

data/app/views/plan_driven/wizard/plans/_tickets.html.erb

@@ -0,0 +1,42 @@
+<% can_draft = %w[approved ticketed].include?(@plan.status) %>
+<div class="card">
+  <%= form_with url: run_plan_path(@plan.key), method: :post, class: "row" do %>
+    <%= hidden_field_tag :do, "tickets" %><%= hidden_field_tag :step, "tickets" %>
+    <%= text_field_tag :instruction, nil, placeholder: @plan.tickets.any? ? "what to change, e.g. split T2 into model and UI" : "optional guidance for the tickets",
+                       disabled: !can_draft %>
+    <button type="submit" class="<%= "primary" if @plan.tickets.empty? %>" <%= "disabled" unless can_draft %>>
+      <%= @plan.tickets.any? ? "Redraft tickets" : "Draft tickets" %></button>
+  <% end %>
+</div>
+
+<% if @plan.tickets.any? %>
+  <div class="card">
+    <table>
+      <thead><tr><th>Ticket</th><th>Type</th><th>Est.</th><th>Depends on</th><th>Status</th></tr></thead>
+      <tbody>
+        <% @plan.tickets.each do |ticket| %>
+          <tr>
+            <td><strong><%= ticket.key %></strong> <%= ticket.title %>
+              <details><summary class="muted">acceptance criteria</summary>
+                <ul><% ticket.criteria.each do |criterion| %><li><%= criterion.is_a?(Hash) ? criterion.values.join(" ") : criterion %></li><% end %></ul>
+              </details></td>
+            <td><%= ticket.ticket_type %></td>
+            <td><%= ticket.estimate %></td>
+            <td><%= ticket.dependencies.join(", ").presence || "—" %></td>
+            <td><span class="pill"><%= ticket.status.tr("_", " ") %></span></td>
+          </tr>
+        <% end %>
+      </tbody>
+    </table>
+  </div>
+
+  <% if @plan.status == "ticketed" %>
+    <div class="card row">
+      <span>Approvals needed: <strong><%= @plan.missing_ticket_approvals.join(", ").presence || "none" %></strong></span>
+      <span style="margin-left:auto"></span>
+      <% @plan.missing_ticket_approvals.each do |role| %>
+        <%= run_button "Approve tickets as #{role}", "approve-tickets", { role: role }, primary: true %>
+      <% end %>
+    </div>
+  <% end %>
+<% end %>