Skip to content

fix(wizard): remove [WebUI]/[Terminal] prefixes from pack chooser (do not merge yet) #93

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
royosherove merged 3 commits into from
Aug 22, 2026
+6 −14
Merged

fix(wizard): remove [WebUI]/[Terminal] prefixes from pack chooser (do not merge yet) #93

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
99a8a87
fix(wizard): remove [WebUI]/[Terminal] prefixes from pack chooser
royosherove Aug 22, 2026
7857daa
fix(wizard): default Cognito WebUI prompt to No (kirocrew)
royosherove Aug 22, 2026
c9f4af5
fix(wizard): drop border from final next-steps box for easier link copy
royosherove Aug 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 6 additions & 14 deletions install.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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")
Expand Down Expand Up @@ -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"
Expand Down Expand Up @@ -2126,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
Expand Down Expand Up @@ -3045,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
Expand Down