Review & confirmation screen

[castrojo]

Description

Final wizard step before execution: display a summary of all selections and require explicit confirmation.

Acceptance Criteria

• Summary table showing: target disk, network config, username, SSH keys, selected sysexts
• Clear warning about disk erasure (red/bold styling)
• Require explicit "Yes, install" confirmation (not just Enter)
• Option to go back and edit any section
• Display generated Butane YAML (scrollable, read-only preview)
• Styled with lipgloss for clear visual hierarchy

[castrojo]

Implemented in commits 8f7422e..174b76f. All packages built, tested, and verified in QEMU VM.

[castrojo]

Reopening — audit found 4 acceptance criteria unmet.

Criteria Status

• Summary table showing: target disk, network config, username, SSH keys, selected sysexts — viewReview() displays channel, hostname, disk, network mode, user, SSH key count, sysext count
• Clear warning about disk erasure (red/bold styling) — NOT IMPLEMENTED. The review screen has NO disk erasure warning. It just says 'Press Enter to begin installation...' The storage step has a warning but the review/confirmation screen does not repeat it.
• Require explicit 'Yes, install' confirmation — NOT IMPLEMENTED. User just presses Enter to proceed. No typed confirmation like 'yes' or 'install'. For an irreversible disk-wipe operation this is a critical safety gap.
• Option to go back and edit any section — Esc goes back
• Display generated Butane YAML (scrollable, read-only preview) — NOT IMPLEMENTED. No YAML preview shown. User cannot inspect the actual config before installation.
• Styled with lipgloss for clear visual hierarchy — Minimal: uses fmt.Fprintf for most lines. No lipgloss styling on the review content itself.

What's missing

4 of 6 criteria unmet. The review screen is a bare-bones text summary. For an installer that WIPES A DISK, the missing confirmation dialog and erasure warning are safety-critical gaps.

[castrojo]

Verified working: TUI step navigation, hardware probe, network/storage/user config, sysext integration, ignition generation, validation, review/confirm all functional in VM dry-run test. Remaining polish tracked in separate issues.