feat(acp): answer bubble shows what was chosen

The collapsed ask_user answer bubble only said "Answered N questions".
It now lists one line per question under that header, e.g.
"Q2 Controls → radio, textarea · changed", with long answers cut;
clicking still expands the full text. Confirm answers are written as
"value (label)" like choices so the overview shows the label.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: nullmastermind

src/AskUserForm.cpp

@@ -564,7 +564,8 @@ QString confirmText(const Question &q, bool yes)
     const QString label = yes ? q.yesLabel : q.noLabel;
     if (label.isEmpty() || label.compare(plain, Qt::CaseInsensitive) == 0)
         return plain;
-    return QStringLiteral("%1 (%2)").arg(label, plain);
+    // "value (label)", like choices: the overview shows what is in brackets.
+    return QStringLiteral("%1 (%2)").arg(plain, label);
 }
 
 QString quoted(const QString &text)
@@ -1040,6 +1041,75 @@ QJsonObject elicitationContent(const Spec &spec, const QVector<Answer> &answers)
     return content;
 }
 
+namespace {
+
+// "value (label)" -> "label"; quoted free text -> unquoted.
+QString displayItem(QString item)
+{
+    item = item.trimmed();
+    if (item.startsWith(QLatin1String("other: ")))
+        return QStringLiteral("other: ") + displayItem(item.mid(7));
+    if (item.size() >= 2 && item.startsWith(QLatin1Char('"')) && item.endsWith(QLatin1Char('"')))
+        return item.mid(1, item.size() - 2);
+    if (item.endsWith(QLatin1Char(')'))) {
+        const qsizetype open = item.indexOf(QLatin1String(" ("));
+        if (open > 0)
+            return item.mid(open + 2, item.size() - open - 3);
+    }
+    return item;
+}
+
+} // namespace
+
+QStringList answerOverview(const QString &text, int maxAnswerChars)
+{
+    QStringList out;
+    if (!text.startsWith(QLatin1String(kAnswersHeader)))
+        return out;
+    static const QLatin1String kRecommended(" (recommended)");
+    static const QString kChanged = QStringLiteral(" — changed (recommended: ");
+    const QStringList lines = text.split(QLatin1Char('\n'));
+    for (int i = 1; i < lines.size(); ++i) {
+        const QString &line = lines.at(i);
+        if (line.isEmpty() || line.at(0).isSpace() || line.startsWith(QLatin1String("Form: ")))
+            continue;
+        const qsizetype space = line.indexOf(QLatin1Char(' '));
+        const qsizetype titleEnd = line.indexOf(QLatin1String("\": "));
+        if (space <= 0 || titleEnd < space || line.at(space + 1) != QLatin1Char('"'))
+            continue;
+        const QString id = line.left(space);
+        const QString title = line.mid(space + 2, titleEnd - space - 2);
+        QString answer = line.mid(titleEnd + 3);
+        bool changed = false;
+        if (answer.endsWith(kRecommended)) {
+            answer.chop(kRecommended.size());
+        } else if (const qsizetype c = answer.indexOf(kChanged); c >= 0) {
+            answer.truncate(c);
+            changed = true;
+        }
+        if (answer.endsWith(QLatin1Char('|'))) {
+            // Multi-line text: its first line follows, indented.
+            const QString next = i + 1 < lines.size() ? lines.at(i + 1).trimmed() : QString();
+            answer.chop(1);
+            answer += next;
+        }
+        QString shown;
+        if (answer.startsWith(QLatin1Char('"')) || answer.startsWith(QLatin1String("change requested: "))) {
+            shown = answer.startsWith(QLatin1Char('"')) ? displayItem(answer) : answer;
+        } else {
+            QStringList items = answer.split(QStringLiteral(", "));
+            for (QString &item : items)
+                item = displayItem(item);
+            shown = items.join(QStringLiteral(", "));
+        }
+        if (maxAnswerChars > 1 && shown.size() > maxAnswerChars)
+            shown = shown.left(maxAnswerChars - 1) + QStringLiteral("…");
+        out.append(QStringLiteral("%1 %2 → %3").arg(id, title, shown)
+                   + (changed ? QStringLiteral(" · changed") : QString()));
+    }
+    return out;
+}
+
 QString dismissPrompt(const QString &note)
 {
     QString out = QLatin1String(kAnswersHeader) + QLatin1Char('\n') + QLatin1String(kDismissedLine);

src/AskUserForm.h

@@ -15,6 +15,7 @@
 #include <QJsonObject>
 #include <QJsonValue>
 #include <QString>
+#include <QStringList>
 #include <QVector>
 
 #include <cstdint>
@@ -149,6 +150,9 @@ QString formatAnswers(const Spec &spec, const QVector<Answer> &answers);
 QString answerSummary(const Spec &spec, const QVector<Answer> &answers);
 // Same summary, recomputed from the prompt text (restored history).
 QString summaryFromAnswersText(const QString &text);
+// One line per question from the prompt text: "Q1 Title → answer[ · changed]",
+// answers cut to `maxAnswerChars`. Empty for dismiss / clarify prompts.
+QStringList answerOverview(const QString &text, int maxAnswerChars = 90);
 
 QString dismissPrompt(const QString &note);
 QString clarifyPrompt(const Question &q);

src/widgets/AcpMessageWidget.cpp

@@ -351,7 +351,7 @@ void AcpMessageWidget::setContent(const QVector<AcpProtocol::AcpContentBlock> &c
     rerender();
 }
 
-void AcpMessageWidget::setCollapsedSummary(const QString &summary)
+void AcpMessageWidget::setCollapsedSummary(const QString &summary, const QStringList &overview)
 {
     if (m_role != QLatin1String("user") || !m_layout)
         return;
@@ -366,14 +366,25 @@ void AcpMessageWidget::setCollapsedSummary(const QString &summary)
         if (m_chatFontSet)
             m_summaryHeader->setFont(m_chatFont);
         m_layout->insertWidget(0, m_summaryHeader);
+        m_summaryOverview = new QLabel(this);
+        m_summaryOverview->setTextFormat(Qt::PlainText);
+        m_summaryOverview->setWordWrap(true);
+        m_summaryOverview->setTextInteractionFlags(Qt::TextSelectableByMouse);
+        if (m_chatFontSet)
+            m_summaryOverview->setFont(m_chatFont);
+        m_layout->insertWidget(1, m_summaryOverview);
         connect(m_summaryHeader, &QToolButton::toggled, this, [this](bool expanded) {
             for (QWidget *w : std::as_const(m_userBlocks))
                 w->setVisible(expanded);
+            // Expanded shows the full text; the overview would repeat it.
+            m_summaryOverview->setVisible(!expanded && !m_summaryOverview->text().isEmpty());
             m_summaryHeader->setText(m_summary + (expanded ? QStringLiteral("  ▾") : QStringLiteral("  ▸")));
             updateGeometry();
         });
     }
     m_summary = QStringLiteral("↳ ") + summary;
+    m_summaryOverview->setText(overview.join(QLatin1Char('\n')));
+    m_summaryOverview->setVisible(!overview.isEmpty());
     m_summaryHeader->setChecked(false);
     m_summaryHeader->setText(m_summary + QStringLiteral("  ▸"));
     for (QWidget *w : std::as_const(m_userBlocks))
@@ -457,6 +468,12 @@ QSize AcpMessageWidget::sizeHint() const
     int contentW = 0;
     if (m_summaryHeader)
         contentW = m_summaryHeader->sizeHint().width();
+    if (m_summaryOverview && !m_summaryOverview->isHidden()) {
+        const QFontMetrics fm = m_summaryOverview->fontMetrics();
+        const QStringList lines = m_summaryOverview->text().split(QLatin1Char('\n'));
+        for (const QString &line : lines)
+            contentW = qMax(contentW, fm.horizontalAdvance(line));
+    }
     for (const QWidget *w : m_userBlocks) {
         const auto *lbl = qobject_cast<const QLabel *>(w);
         if (!lbl || lbl->isHidden())
@@ -694,6 +711,8 @@ void AcpMessageWidget::setChatFont(const QFont &font)
     }
     if (m_summaryHeader)
         m_summaryHeader->setFont(font);
+    if (m_summaryOverview)
+        m_summaryOverview->setFont(font);
     // User-role bodies are QLabels built lazily in setContent(); stamp the ones
     // that exist now (newly-created ones are stamped in setContent()).
     for (QWidget *w : m_userBlocks) {

src/widgets/AcpMessageWidget.h

@@ -82,10 +82,10 @@ class AcpMessageWidget : public QFrame
     // conversation" buttons at the bottom of the bubble, always visible.
     void setResponseActions(bool enabled);
 
-    // User role: show one clickable `summary` line instead of the content (set
-    // after setContent); clicking toggles the full text. Used for ask_user
-    // answer prompts.
-    void setCollapsedSummary(const QString &summary);
+    // User role: show a clickable `summary` line plus one `overview` line per
+    // answer instead of the content (set after setContent); clicking toggles
+    // the full text. Used for ask_user answer prompts.
+    void setCollapsedSummary(const QString &summary, const QStringList &overview = {});
 
     bool isCollapsed() const { return m_collapsed; }
     QString role() const { return m_role; }
@@ -145,6 +145,7 @@ class AcpMessageWidget : public QFrame
     MarkdownView *m_markdown = nullptr;    // == m_browser for the assistant role
     QToolButton *m_thoughtHeader = nullptr; // thought role
     QToolButton *m_summaryHeader = nullptr; // user role, collapsed summary
+    QLabel *m_summaryOverview = nullptr;    // one line per answer while collapsed
     QString m_summary;
     QLabel *m_goalBadge = nullptr;         // Goal badge on result rows
     QVBoxLayout *m_layout = nullptr;

src/widgets/AcpSessionView.cpp

@@ -2333,7 +2333,8 @@ void AcpSessionView::applyMessageMarker(AcpMessageWidget *w, int messageIndex)
     if (!m_model || messageIndex < 0 || messageIndex >= m_model->messages().size())
         return;
     if (m_model->messages().at(messageIndex).marker == QLatin1String(AskUserForm::kMarkerAnswers))
-        w->setCollapsedSummary(AskUserForm::summaryFromAnswersText(w->plainText()));
+        w->setCollapsedSummary(AskUserForm::summaryFromAnswersText(w->plainText()),
+                               AskUserForm::answerOverview(w->plainText()));
 }
 
 void AcpSessionView::restoreExpiredForm()

tests/test_ask_user_form.cpp

@@ -44,6 +44,7 @@ private slots:
     void unwrapsNestedArguments();
     void recognisesOmpDeviceWrite();
     void infersRecommendedFromRecommendation();
+    void overviewListsEachAnswer();
 };
 
 void TestAskUserForm::parsesGrillRound()
@@ -179,7 +180,7 @@ void TestAskUserForm::formatsAnswersForTheAgent()
         "Q2 \"Controls\": radio, textarea, other: \"slider\" — changed (recommended: radio, checkbox)\n"
         "  note: keep it small\n"
         "Q3 \"Module layout\": change requested: \"move core to lib/\" — changed (recommended: accept)\n"
-        "Q4 \"Scan inside?\": Skip (no) (recommended)\n"
+        "Q4 \"Scan inside?\": no (Skip) (recommended)\n"
         "Q5 \"Describe it\": no answer\n"
         "Q6 \"Days\": 45 — changed (recommended: 30)\n");
     QCOMPARE(formatAnswers(spec, answers), expected);
@@ -411,5 +412,32 @@ void TestAskUserForm::infersRecommendedFromRecommendation()
     QVERIFY(spec->questions.at(0).recommended.isNull());
 }
 
+void TestAskUserForm::overviewListsEachAnswer()
+{
+    const Spec spec = grillSpec();
+    const QVector<Answer> answers{
+        answer("Q1", QStringLiteral("b2")),
+        answer("Q2", QJsonArray{QStringLiteral("radio"), QStringLiteral("textarea")}, "", "keep it small"),
+        answer("Q3", QStringLiteral("accept")),
+        answer("Q4", false),
+        answer("Q5", QStringLiteral("line one\nline two")),
+        answer("Q6", QJsonValue()),
+    };
+    const QStringList lines = answerOverview(formatAnswers(spec, answers), 40);
+    QCOMPARE(lines, QStringList({
+        QStringLiteral("Q1 Architecture → No-op tool"),
+        QStringLiteral("Q2 Controls → radio, textarea · changed"),
+        QStringLiteral("Q3 Module layout → accepted"),
+        QStringLiteral("Q4 Scan inside? → Skip"),
+        QStringLiteral("Q5 Describe it → line one"),
+        QStringLiteral("Q6 Days → no answer"),
+    }));
+
+    // Long answers are cut; dismiss/clarify prompts have no overview.
+    const QStringList cut = answerOverview(formatAnswers(spec, {answer("Q5", QString(60, QLatin1Char('x')))}), 20);
+    QCOMPARE(cut.at(4), QStringLiteral("Q5 Describe it → ") + QString(19, QLatin1Char('x')) + QStringLiteral("…"));
+    QVERIFY(answerOverview(dismissPrompt(QString())).isEmpty());
+}
+
 QTEST_GUILESS_MAIN(TestAskUserForm)
 #include "test_ask_user_form.moc"