fix(dashboard): cosmetic — version double-v, provider label, CLI blurb - #90
Merged
Conversation
Three UI fixes reported from the Home page: - Version rendered "vv0.11.0…": the server version string already carries its leading "v" when built from a release tag (server/vX.Y.Z), but the UI also hard-prefixed "v". Add lib/version.ts → formatVersion() which normalizes to exactly one leading "v" (robust for both tagged builds and the "0.0.0-dev" placeholder), and route all four display sites through it: Footer (now "cix (v0.11.0)" — version parenthesized), HomePage status stat, and the UpdateBanner's available/running lines. - HomePage status panel: rename the "Sidecar" stat to "Provider" — the embedding provider isn't necessarily an in-process sidecar (can be a remote ollama/voyage/openai backend), so "Provider" is accurate either way. - HomePage CLI blurb claimed `cix --help` "does everything the dashboard does, plus reindex…". Untrue — the CLI does code search/navigation + local indexing but not server administration (users, API keys, runtime config). Reword to state that split accurately. Source-only; dist is rebuilt at image build. Dashboard builds clean (tsc + vite). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Three small Home-page fixes you flagged.
1. Version shows
vv0.11.0…The server version string already carries its leading
vwhen built from a release tag (server/vX.Y.Z), but the UI also hard-prefixedv→ doublev.lib/version.ts→formatVersion()normalizes to exactly one leadingv(robust for both tagged builds and the0.0.0-devplaceholder, which has nov).cix (v0.11.0)— version parenthesized, per your suggestion), HomePage status stat, and the UpdateBanner's "available" / "you're running" lines.2. "Sidecar" status label → "Provider"
The embedding provider isn't necessarily an in-process sidecar — it can be a remote
ollama/voyage/openaibackend.Provideris accurate in all cases. (The Footer already labels by provider kind, so this aligns them.)3. CLI blurb was inaccurate
The CLI is a code-search/indexing client — it does not administer the server (users, API keys, runtime config). Reworded to state the split honestly:
Notes
dist/is gitignored and rebuilt at image build. Dashboard builds clean (tsc -b && vite build).cix (v0.11.0)), where it reads as tool-name + version. In the labeled status stat and the update banner sentence, barev0.11.0reads better. Easy to change if you'd prefer parens everywhere.🤖 Generated with Claude Code