Skip to content

install: only show the API-key hint when no key is configured#2

Merged
gvonnessi merged 1 commit into
mainfrom
installer-key-hint
Jun 18, 2026
Merged

install: only show the API-key hint when no key is configured#2
gvonnessi merged 1 commit into
mainfrom
installer-key-hint

Conversation

@gvonnessi

Copy link
Copy Markdown
Contributor

The install.sh/install.ps1 post-install message printed the "next: add a read-only API key" hint unconditionally — even when vastline install had just reported an already-configured key (as happens on a machine that already used the tool or the vast CLI).

Gate the hint on vastline key path, which exits 0 when a key resolves from any source ($VAST_API_KEY, ~/.config/vastline/vast_api_key, or the vast CLI's key). Verified: hint suppressed when a key is present, shown when none.

The installers printed the "next: add a read-only API key" nudge
unconditionally, even when `vastline install` had just reported an
existing key. Gate it on `vastline key path` (exit 0 = a key resolves
from env, vastline's config, or the vast CLI) so it only appears when
setup is actually needed.
@gvonnessi gvonnessi merged commit be27ea8 into main Jun 18, 2026
3 checks passed
@gvonnessi gvonnessi deleted the installer-key-hint branch June 18, 2026 12:54
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.

1 participant