Skip to content

Fix stale/misleading startup banner (dynamic models + accurate direct/proxy)#25

Merged
0xadvait merged 1 commit into
mainfrom
fix-stale-banner
Jun 24, 2026
Merged

Fix stale/misleading startup banner (dynamic models + accurate direct/proxy)#25
0xadvait merged 1 commit into
mainfrom
fix-stale-banner

Conversation

@0xadvait

Copy link
Copy Markdown
Collaborator

Two stale bits in the startup banner:

  • Hardcoded model names ('… DeepSeek V4 Pro …', 'hosted 70B fallback') go stale when the default changes. Now read the default + fallback model from config (bare names), so the banner is always accurate.
  • Bogus 'IP-masked via ' in direct mode: the banner probed the rotating proxy and claimed IP-masking whenever the proxy process answered a ping -- but in direct mode (default) og-veil's egress doesn't use it. Gate the claim on a real .proxy marker (written only by GHOST_PROXY=1 installs); otherwise show direct. install.sh writes/removes the marker.

Banner now reads e.g. 👻 ghost · deepseek-v4-pro via TEE gateway (OHTTP/og-veil) · direct · ✅ connected · fallback: hermes-4-70b. (Also unloaded a leftover proxy LaunchAgent on the dev box that was triggering the false IP-masked claim.) bash -n clean.

- Read the default + fallback model names from config instead of hardcoding them, so
  the banner never goes stale when the default changes (was still saying a fixed
  model + 'hosted 70B fallback').
- Stop claiming 'IP-masked via <ip>' in direct mode. The banner probed the rotating
  proxy and claimed IP-masking whenever the proxy process answered -- even though, in
  direct mode (the default), og-veil's egress does NOT go through it. Gate the claim
  on a real .proxy marker (written only by GHOST_PROXY=1 installs); otherwise show
  'direct'. install.sh writes/removes the marker accordingly.
@0xadvait
0xadvait merged commit 3ce5cfd into main Jun 24, 2026
3 checks passed
@0xadvait
0xadvait deleted the fix-stale-banner branch June 24, 2026 16:55
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