Skip to content

Improve openwebui detection and confirmation prompts#61

Merged
BillJr99 merged 1 commit into
masterfrom
claude/focused-albattani-iHSHn
May 24, 2026
Merged

Improve openwebui detection and confirmation prompts#61
BillJr99 merged 1 commit into
masterfrom
claude/focused-albattani-iHSHn

Conversation

@BillJr99
Copy link
Copy Markdown
Owner

Summary

This PR enhances the install tool script to better handle OpenWebUI detection and improves the user experience for confirmation prompts.

Key Changes

  • Enhanced _it_confirm() function:

    • Now displays context-aware hints (e.g., "Y/n" vs "y/N") based on the default value
    • Refactored to use direct TTY I/O operations (_it_open_tty, file descriptors 3 and 4) instead of delegating to _it_read()
    • Properly handles case-insensitive default values
  • Added OpenWebUI endpoint configuration:

    • When OpenWebUI is not reachable at the default URL, the script now prompts users to provide a custom endpoint (useful for Docker deployments on non-standard ports)
    • Stores the custom endpoint in CLK_OPENWEBUI_ENDPOINT environment variable
    • Optionally prompts for API key configuration via CLK_OPENWEBUI_API_KEY for authenticated instances
    • Re-checks tool availability after configuration before proceeding with installation
    • Provides clear feedback if the service remains unreachable after configuration

Implementation Details

  • The confirmation prompt now uses a case statement to determine the appropriate hint text based on the default value
  • OpenWebUI configuration is only triggered when the tool name is "openwebui" and initial detection fails
  • Configuration values are persisted to the environment file for future use
  • The flow gracefully handles both successful re-detection and continued unavailability scenarios

https://claude.ai/code/session_01HbzTVqzDcadZrQhRQM2mrF

…tall

openwebui ships as a Docker service rather than a CLI, and the installer's
check_tool only looked at the default localhost:8080. Users running the
official image on a different port were told it wasn't installed and
prompted to install it again. Now, when the default probe misses, the
wizard first asks for the endpoint (and optional API key), persists them
to .env, and re-checks before falling through to the install suggestion.

Also fixes the y/n prompt that rendered as "[Y] [Y]:" because both
_it_confirm and _it_read appended the default — _it_confirm now prints
the hint directly as "[Y/n]" or "[y/N]".
@BillJr99 BillJr99 merged commit b46c6b2 into master May 24, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants