• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

stacklok / toolhive / 25062537185
66%

Build:
DEFAULT BRANCH: main
Ran 28 Apr 2026 03:44PM UTC
Jobs 1
Files 705
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

28 Apr 2026 03:38PM UTC coverage: 63.946% (+0.03%) from 63.917%
25062537185

push

github

web-flow
thv llm: complete setup/teardown orchestration (#5068)

* thv llm: complete setup/teardown orchestration

Add the missing pieces that turn the adapter scaffolding into a working
end-to-end flow:

- thv llm setup now accepts inline --gateway-url / --issuer / --client-id
  (and the other config flags) so "config set" and "setup" can be combined
  in a single command. Flags are merged into the persisted config before
  the rest of setup runs.

- setup triggers an interactive OIDC browser login after configuring all
  detected tools, so the refresh token is stored before any tool or proxy
  tries to use it.

- setup starts the localhost proxy as a background process when any
  proxy-mode tool (Cursor, VS Code, Xcode) was configured, and saves the
  PID to config so teardown can stop it.

- teardown sends SIGINT to the stored proxy PID (if any) before clearing
  the PID from config. The PID is always zeroed in the final UpdateConfig
  call regardless of whether the signal succeeded.

- pkg/llm.ProxyConfig gains a PID field (yaml/json: "pid") to hold the
  background proxy process ID across CLI invocations.

E2E tests in cli_llm_setup_test.go cover: inline-flag config + tool
patching, full teardown revert, single-tool targeted teardown, no-config
error, and --purge-tokens token cleanup.

Fixes #5031

* thv llm: complete setup/teardown orchestration

- setup accepts inline --gateway-url / --issuer / --client-id (and the
  other config flags), merged into persisted config before running, so
  config-set and setup can be combined in a single invocation
- OIDC browser login runs after tool config files are patched so the
  refresh token is cached before tools use it; proxy-mode tools receive
  a hint to run "thv llm proxy start" (foreground command, no daemon)
- teardown reverts all (or a single named) tool config patches;
  --purge-tokens deletes cached OIDC tokens from the secrets provider
- loginFunc injected into runLLMSetup for keyring-free unit tests
- pu... (continued)

24 of 79 new or added lines in 1 file covered. (30.38%)

4 existing lines in 2 files now uncovered.

60462 of 94551 relevant lines covered (63.95%)

57.23 hits per line

Uncovered Changes

Lines Coverage ∆ File
55
29.81
1.47% cmd/thv/app/llm.go

Coverage Regressions

Lines Coverage ∆ File
2
29.81
1.47% cmd/thv/app/llm.go
2
73.63
-0.64% pkg/runner/config.go
Jobs
ID Job ID Ran Files Coverage
1 25062537185.1 28 Apr 2026 03:44PM UTC 705
63.95
GitHub Action Run
Source Files on build 25062537185
  • Tree
  • List 705
  • Changed 9
  • Source Changed 1
  • Coverage Changed 9
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #25062537185
  • ae1b32ff on github
  • Prev Build on main (#25059386146)
  • Next Build on main (#25073322202)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc