From 99a8a8739153a69334943ab17a190f914c18efdf Mon Sep 17 00:00:00 2001 From: Roy Osherove <575051+royosherove@users.noreply.github.com> Date: Sat, 22 Aug 2026 22:47:38 +0000 Subject: [PATCH 1/3] fix(wizard): remove [WebUI]/[Terminal] prefixes from pack chooser MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Roy: 'remove the ugly [WebUI] and [Terminal] stuff from the initial lowkey wizard'. - Drop the in_list detection loop and the branching item construction; the pack list now just shows 'name — description' with the existing '(experimental)' suffix intact. - Simplify the post-choose parse: PACK_NAME='${pack_choice%% —*}' plus a trailing-space trim (no more bracket-strip needed). Diff: +2 / -11. bash -n install.sh: OK. --- install.sh | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/install.sh b/install.sh index 183a740..33422e1 100755 --- a/install.sh +++ b/install.sh @@ -1956,16 +1956,7 @@ choose_pack() { [[ "${PACK_NAMES[$i]}" == "$sp" ]] || continue pname="${PACK_NAMES[$i]}" - # Determine UI type prefix - in_list=false - for wp in "${webui_packs[@]}"; do - [[ "$pname" == "$wp" ]] && in_list=true && break - done - if [[ "$in_list" == true ]]; then - item="[WebUI] ${pname} — ${PACK_DESCS[$i]}" - else - item="[Terminal] ${pname} — ${PACK_DESCS[$i]}" - fi + item="${pname} — ${PACK_DESCS[$i]}" [[ "${PACK_EXPERIMENTAL[$i]}" == "true" ]] && item+=" (experimental)" gum_items+=("$item") @@ -2000,7 +1991,7 @@ choose_pack() { "${gum_items[@]}" \ || { fail "Pack selection is required"; } PACK_NAME="${pack_choice%% —*}" - PACK_NAME="${PACK_NAME##*] }" # Strip [WebUI]/[Terminal] prefix if present + PACK_NAME="${PACK_NAME%" "}" # trim trailing space before the em dash separator, if any for i in "${!PACK_NAMES[@]}"; do if [[ "${PACK_NAMES[$i]}" == "$PACK_NAME" && "${PACK_EXPERIMENTAL[$i]}" == "true" ]]; then warn "${PACK_NAME} is experimental — expect rough edges" From 7857daa7fb19cd1e85718b2574e6323bef22a5cd Mon Sep 17 00:00:00 2001 From: Roy Osherove <575051+royosherove@users.noreply.github.com> Date: Sat, 22 Aug 2026 22:48:25 +0000 Subject: [PATCH 2/3] fix(wizard): default Cognito WebUI prompt to No (kirocrew) Roy: 'also make the cognito login default false for kirocrew pack choice (same PR)'. configure_webui_auth() calls confirm with default_yes for the 'Protect ${pack_name} WebUI with Cognito login?' question. Switch to default_no so the user has to opt in. Cognito adds a companion us-east-1 stack, a Lambda@Edge zip build, and 5-15 min of extra CloudFront propagation; making it opt-in matches the friction it adds. --webui-no-auth flag and unattended (AUTO_YES) skip paths unchanged. --- install.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/install.sh b/install.sh index 33422e1..713ae15 100755 --- a/install.sh +++ b/install.sh @@ -2117,7 +2117,7 @@ configure_webui_auth() { return 0 fi - if [[ "${AUTO_YES:-false}" != true ]] && ! confirm "Protect ${pack_name} WebUI with Cognito login? (enterprise-grade)" "default_yes"; then + if [[ "${AUTO_YES:-false}" != true ]] && ! confirm "Protect ${pack_name} WebUI with Cognito login? (enterprise-grade)" "default_no"; then warn "WebUI authentication disabled; use SSM/VPN-only access." return 0 fi From c9f4af50f556a01601174e8f2e1c9b8c509bd7a1 Mon Sep 17 00:00:00 2001 From: Roy Osherove <575051+royosherove@users.noreply.github.com> Date: Sat, 22 Aug 2026 22:50:02 +0000 Subject: [PATCH 3/3] fix(wizard): drop border from final next-steps box for easier link copy MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Roy: 'in the final output of the install, the result should not have | content | in the middle of the output box in ascii. the output box should just be floor and top, since links are harder to copy between the || lines. (same PR)' The rounded border on the next-steps gum style adds vertical rails (│ ... │) that interfere with triple-click select and copy of the dashboard URL and shell commands inside the box. Drop --border rounded and --border-foreground on the next_block style call. Padding and margin stay, so the block still stands out visually via foreground color and spacing. The info_block above was already borderless; this brings next_block in line. Diff: +3 / -2. bash -n install.sh: OK. --- install.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/install.sh b/install.sh index 713ae15..a0169e6 100755 --- a/install.sh +++ b/install.sh @@ -3036,8 +3036,9 @@ show_complete() { echo "" echo -e "$info_block" | $GUM style --foreground 255 --padding "1 2" --margin "0 2" echo "" - echo -e "$next_block" | $GUM style --border rounded --border-foreground 82 \ - --foreground 117 --bold --padding "1 2" --margin "0 2" + # No border on the next-steps block: URLs and commands need to be easy to + # copy without the box side rails breaking word-wrap or triple-click select. + echo -e "$next_block" | $GUM style --foreground 117 --bold --padding "1 2" --margin "0 2" echo "" # Try to copy connect command to clipboard