Repository navigation
CLI gaps for the Windows Companion (VibeTV #437) - #11
Conversation
`config providers` had only the text form (`codex: enabled default
(Codex)`); `--json` was rejected, although docs/CONFIGURATION.md already
names it. It now prints an array of `{provider, displayName, enabled,
defaultEnabled}`, the shape of CodexBar's own CLI (0.63.0). `--format
text|json` and `--pretty` are accepted as there. The text form is
unchanged and comes from the same rows.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
CodexBar's `serve` takes `--request-timeout`; here the option was rejected
and the server exited, so a caller had to leave it out on Windows.
The option now means what it means upstream: a request still unanswered
after that many seconds gets `504 Gateway Timeout` with
`{"error":"request timed out"}`. Its work is not stopped: a slow dashboard
snapshot finishes and is cached, so the next request is answered from it.
`0` waits without a deadline. Fractions are allowed, the cap is one day, a
negative value is an error.
One deliberate difference: upstream's default is 30 seconds, the default
here is 0. Without the option the server keeps waiting for slow providers
as it did before, so callers that do not pass it see no change.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Windows-Test (Beelink) als Ersatz für die CIDie Fork-CI läuft für diesen Branch nicht (CircleCI meldet keine Check-Runs, auch #10 nicht). Stattdessen wurde der Test auf einem echten Windows-Rechner gefahren. Build: Kein nativer Windows-Build. Die Binaries wurden auf dem Mac mit
Lib-Tests (Windows):
CLI (Windows, mit eigener
Die echten Settings des Beelink blieben unverändert (Länge und Zeitstempel). Keine Installation, keine Anmeldungen angefasst. Testordner danach gelöscht. Offen: der native Windows-Lauf für die zwei Fixture-Tests. |
|
@marcus7989 Question from the VibeTV side (issue DreamyTalesPAN/CodexBar-Display#437): is this PR ready to merge? If yes, could you merge it together with #9 and tag a new release (e.g. |
# Conflicts: # rust/src/cli/config.rs
578d5c8 to
eafc532
Compare
|
Paul, heads-up for review: this branch is now based on vibetv, and it includes your #9 head 370647c as a merge. The only conflict was in rust/src/cli/config.rs, resolved as in our local test merge; the Rust tree is identical to what we tested on Windows. The previous head was 578d5c8 (the branch was rebased, so the force push was lease-protected). |
Adds four CLI pieces the VibeTV Windows Companion needs (DreamyTalesPAN/CodexBar-Display#437). Each change is additive, so existing calls keep their output. Based on
main, independent of #9.CODEXBAR_CONFIGsets the settings file path, same contract as macOS.usage --provider enabledruns every provider switched on in settings in one call. Bareusagestays Claude-only.config claude-consent on|offsetsclaude_allow_reading_claude_code_credentials.cost --refreshis accepted.costalready rescans on every run, so the flag changes nothing.Not in this PR:
serve --request-timeoutandconfig providers --json. #9 covers both.Validation:
cargo test -p codexbar --libpasses except 4 Windows path tests (pi_family x2, codex_sessions, core::hooks). They already fail onmainon macOS and count only on Windows CI.cargo fmt --checkis clean.