From 9695ae5fc796e8833e18b331bc062bfbe68cadd2 Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Wed, 29 Apr 2026 17:17:07 +0100
Subject: [PATCH] Add setup completion polish guardrails

---
 .../__tests__/SetupCompletionPanel.guardrails.test.ts | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/frontend-modern/src/components/SetupWizard/__tests__/SetupCompletionPanel.guardrails.test.ts b/frontend-modern/src/components/SetupWizard/__tests__/SetupCompletionPanel.guardrails.test.ts
index 31f1e2160..2a2c328d7 100644
--- a/frontend-modern/src/components/SetupWizard/__tests__/SetupCompletionPanel.guardrails.test.ts
+++ b/frontend-modern/src/components/SetupWizard/__tests__/SetupCompletionPanel.guardrails.test.ts
@@ -37,6 +37,7 @@ describe('SetupCompletionPanel guardrails', () => {
   it('describes setup completion through one compact source-choice next-step surface', () => {
     expect(setupCompletionPanelSource).toContain('SOURCE_STRATEGY_OPTIONS');
     expect(setupCompletionPanelSource).toContain('Source choices');
+    expect(setupCompletionPanelSource).toContain('<ul class="mt-2 space-y-1.5 text-left">');
     expect(setupCompletionPanelSource).toContain("title: 'Platform API'");
     expect(setupCompletionPanelSource).toContain("title: 'Pulse Agent'");
     expect(setupCompletionPanelSource).toContain("title: 'Use both'");
@@ -59,6 +60,16 @@ describe('SetupCompletionPanel guardrails', () => {
     expect(setupCompletionPanelSource).not.toContain('ProxmoxIcon');
   });
 
+  it('does not present setup guidance as extra credentials or clickable-looking source cards', () => {
+    expect(setupCompletionPanelSource).not.toMatch(
+      /<code[^>]*>\s*\{ADD_INFRASTRUCTURE_PATH\}\s*<\/code>/,
+    );
+    expect(setupCompletionPanelSource).not.toContain('grid gap-2 sm:grid-cols-3');
+    expect(setupCompletionPanelSource).not.toContain(
+      'rounded-md border border-border bg-surface px-3 py-2.5',
+    );
+  });
+
   it('keeps connected infrastructure classification on the canonical setup model', () => {
     expect(setupCompletionPanelSource).toContain('buildSetupCompletionConnectedSystems');
     expect(setupCompletionPanelSource).toContain('buildSetupCompletionViewModel');