fix(init): lint, early-return guard, and test coverage for feature blurbs

- Fix Biome import order in logging-ui.ts (renderTextTable after markdown)
- Fix Biome format: collapse single-item success array, split long JSX condition
- Fix LoggingUI.summary() early-return guard to account for featureBlurbs
- Fix buildSummary null guard to include featureBlurbs in the emptiness check
- Add test coverage: formatters (blurb population, suppressed Features row,
  positional label matching, display-order sort), ink-report (blurbs section),
  logging-ui (blurbs table, no section when absent)

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

Author: betegon

src/lib/init/feedback.ts

@@ -7,9 +7,7 @@ const FEEDBACK_COMMANDS: Record<InitFeedbackOutcome, string> = {
 };
 
 const FEEDBACK_COPY: Record<InitFeedbackOutcome, string[]> = {
-  success: [
-    "Tell us what felt great or rough:",
-  ],
+  success: ["Tell us what felt great or rough:"],
   cancelled: [
     "Sad to see setup stop. Was something going sideways?",
     "Tell us so we can fix it:",

src/lib/init/formatters.ts

@@ -75,7 +75,7 @@ function buildSummary(output: WizardOutput): WizardSummary | null {
     })
     .filter((b): b is { label: string; blurb: string } => b !== null);
 
-  if (fields.length === 0 && changedFiles.length === 0) {
+  if (fields.length === 0 && changedFiles.length === 0 && featureBlurbs.length === 0) {
     return null;
   }
 

src/lib/init/ui/ink-app.tsx

@@ -1176,7 +1176,8 @@ function SummaryPanel({
       {summary.changedFiles !== undefined && summary.changedFiles.length > 0 ? (
         <ChangedFilesTree files={summary.changedFiles} />
       ) : null}
-      {summary.featureBlurbs !== undefined && summary.featureBlurbs.length > 0 ? (
+      {summary.featureBlurbs !== undefined &&
+      summary.featureBlurbs.length > 0 ? (
         <Box flexDirection="column" flexShrink={0} marginTop={1}>
           <Text bold color={MUTED}>
             Here&apos;s what we set up

src/lib/init/ui/logging-ui.ts

@@ -18,11 +18,11 @@
  * logs deterministic and free of carriage returns.
  */
 
-import { renderTextTable } from "../../formatters/text-table.js";
 import {
   renderInlineMarkdown,
   renderMarkdown,
 } from "../../formatters/markdown.js";
+import { renderTextTable } from "../../formatters/text-table.js";
 import { formatFeedbackHint, type InitFeedbackOutcome } from "../feedback.js";
 import { buildFileTree, flattenTree } from "./file-tree.js";
 import type {
@@ -95,7 +95,11 @@ export class LoggingUI implements WizardUI {
   }
 
   summary(summary: WizardSummary): void {
-    if (summary.fields.length === 0 && !summary.changedFiles?.length) {
+    if (
+      summary.fields.length === 0 &&
+      !summary.changedFiles?.length &&
+      !summary.featureBlurbs?.length
+    ) {
       return;
     }
     // Compact two-column key/value listing — one line per field. The

test/lib/init/formatters.test.ts

@@ -184,6 +184,120 @@ describe("formatResult", () => {
   });
 });
 
+describe("formatResult with featureBlurbs", () => {
+  test("populates featureBlurbs from output.featureBlurbs paired positionally with output.features", () => {
+    const { ui, calls } = createMockUI();
+    formatResult(
+      {
+        status: "success",
+        result: {
+          platform: "Next.js",
+          projectDir: "/app",
+          features: ["errorMonitoring", "performanceMonitoring"],
+          featureBlurbs: [
+            { feature: "errorMonitoring", blurb: "Captures exceptions." },
+            { feature: "performanceMonitoring", blurb: "Traces requests." },
+          ],
+        },
+      },
+      ui
+    );
+
+    const summary = summaryCall(calls);
+    expect(summary?.featureBlurbs).toEqual([
+      { label: "Error Monitoring", blurb: "Captures exceptions." },
+      { label: "Tracing", blurb: "Traces requests." },
+    ]);
+  });
+
+  test("suppresses the Features row when featureBlurbs are present", () => {
+    const { ui, calls } = createMockUI();
+    formatResult(
+      {
+        status: "success",
+        result: {
+          platform: "Next.js",
+          features: ["errorMonitoring"],
+          featureBlurbs: [
+            { feature: "errorMonitoring", blurb: "Captures exceptions." },
+          ],
+        },
+      },
+      ui
+    );
+
+    const summary = summaryCall(calls);
+    expect(summary?.fields.some((f) => f.label === "Features")).toBe(false);
+  });
+
+  test("shows the Features row when featureBlurbs are absent", () => {
+    const { ui, calls } = createMockUI();
+    formatResult(
+      {
+        status: "success",
+        result: {
+          platform: "Next.js",
+          features: ["errorMonitoring", "sessionReplay"],
+        },
+      },
+      ui
+    );
+
+    const summary = summaryCall(calls);
+    expect(summary?.fields.some((f) => f.label === "Features")).toBe(true);
+  });
+
+  test("uses output.features for labels regardless of what the agent echoed in feature field", () => {
+    const { ui, calls } = createMockUI();
+    formatResult(
+      {
+        status: "success",
+        result: {
+          platform: "Next.js",
+          features: ["errorMonitoring", "sessionReplay"],
+          // Agent echoed back wrong IDs
+          featureBlurbs: [
+            { feature: "error_monitoring", blurb: "Blurb A." },
+            { feature: "session-replay", blurb: "Blurb B." },
+          ],
+        },
+      },
+      ui
+    );
+
+    const summary = summaryCall(calls);
+    // Labels come from output.features positionally, not blurb.feature
+    expect(summary?.featureBlurbs?.[0]?.label).toBe("Error Monitoring");
+    expect(summary?.featureBlurbs?.[1]?.label).toBe("Session Replay");
+  });
+
+  test("sorts featureBlurbs by canonical display order", () => {
+    const { ui, calls } = createMockUI();
+    formatResult(
+      {
+        status: "success",
+        result: {
+          platform: "Next.js",
+          // Server returned performanceMonitoring before errorMonitoring
+          features: ["performanceMonitoring", "errorMonitoring"],
+          featureBlurbs: [
+            { feature: "performanceMonitoring", blurb: "Traces." },
+            { feature: "errorMonitoring", blurb: "Captures." },
+          ],
+        },
+      },
+      ui
+    );
+
+    const summary = summaryCall(calls);
+    // errorMonitoring comes before performanceMonitoring in FEATURE_DISPLAY_ORDER
+    expect(summary?.featureBlurbs?.map((b) => b.label)).toEqual([
+      "Error Monitoring",
+      "Tracing",
+    ]);
+  });
+});
+
 describe("formatError", () => {
   test("logs the error message", () => {
     const { ui, calls } = createMockUI();

test/lib/init/ui/ink-report.test.ts

@@ -49,6 +49,37 @@ describe("formatSuccessReport with summary fields", () => {
   });
 });
 
+describe("formatSuccessReport with featureBlurbs", () => {
+  test("renders Here's what we set up heading and blurb content", () => {
+    const output = stripAnsi(
+      formatSuccessReport("Done!", {
+        fields: [],
+        featureBlurbs: [
+          { label: "Error Monitoring", blurb: "Captures exceptions." },
+          { label: "Tracing", blurb: "Traces requests end-to-end." },
+        ],
+      })
+    );
+    expect(output).toContain("Here's what we set up");
+    expect(output).toContain("Error Monitoring");
+    expect(output).toContain("Captures exceptions.");
+    expect(output).toContain("Tracing");
+    expect(output).toContain("Traces requests end-to-end.");
+  });
+
+  test("no blurbs section when featureBlurbs is absent", () => {
+    const output = stripAnsi(formatSuccessReport("Done!", { fields: [] }));
+    expect(output).not.toContain("Here's what we set up");
+  });
+
+  test("no blurbs section when featureBlurbs is empty", () => {
+    const output = stripAnsi(
+      formatSuccessReport("Done!", { fields: [], featureBlurbs: [] })
+    );
+    expect(output).not.toContain("Here's what we set up");
+  });
+});
+
 describe("formatSuccessReport with changedFiles", () => {
   test("shows Changed files heading and file paths", () => {
     const output = stripAnsi(

test/lib/init/ui/logging-ui.test.ts

@@ -307,4 +307,27 @@ describe("LoggingUI summary", () => {
     expect(lines.some((l) => l.includes("−") && l.includes("b.ts"))).toBe(true);
     expect(lines.some((l) => l.includes("~") && l.includes("c.ts"))).toBe(true);
   });
+
+  test("featureBlurbs renders Here's what we set up table with label and blurb", () => {
+    const { ui, stdout } = createUI();
+    ui.summary({
+      fields: [],
+      featureBlurbs: [
+        { label: "Error Monitoring", blurb: "Captures exceptions." },
+        { label: "Tracing", blurb: "Traces requests." },
+      ],
+    });
+    const out = stdout();
+    expect(out).toContain("Here's what we set up");
+    expect(out).toContain("Error Monitoring");
+    expect(out).toContain("Captures exceptions.");
+    expect(out).toContain("Tracing");
+    expect(out).toContain("Traces requests.");
+  });
+
+  test("no featureBlurbs section when featureBlurbs is absent", () => {
+    const { ui, stdout } = createUI();
+    ui.summary({ fields: [{ label: "Platform", value: "Next.js" }] });
+    expect(stdout()).not.toContain("Here's what we set up");
+  });
 });