dcdpr / jp / 28932561173

69%

Build Type

push

github

Committed by web-flow

Commit Message

fix(llm, cli, task): Fail fast on missing provider credentials (#855)

`jp query` no longer spends time spawning title generation, waiting for
MCP servers, or loading attachments when the assistant provider is
misconfigured (e.g. a missing API key environment variable). The
missing-credential error now surfaces immediately, before any of that
side-effectful work starts, instead of only after the request attempt
fails deep inside the turn.

`jp_llm::provider` gains a `preflight` function that validates a
provider's credentials and configuration synchronously, with no I/O.
`query` calls it before scheduling background work, and
`TitleGeneratorTask` calls it before spawning, so a misconfigured title
model is skipped with a warning instead of failing the whole query after
holding teardown open.

When a command run is aborted and its state discarded, `jp` now cancels
the task handler's soft-cancellation token immediately, so background
tasks tied to the discarded run skip the soft wait and move straight to
the 2s grace period instead of blocking teardown for up to 10s.

---------

Signed-off-by: Jean Mertz <git@jeanmertz.com>

Coverage Stats

38 of 63 new or added lines in 6 files covered. (60.32%)

90 existing lines in 6 files now uncovered.

39022 of 57195 relevant lines covered (68.23%)

672.93 hits per line