28457005695
68%

Ran 30 Jun 2026 03:49PM UTC

Jobs 1

Files 774

Run time 1min

Badge

Embed ▾

pending completion

Build # 28457005695

Build Type

push

github

Committed by

web-flow

Commit Message

Fix DetachedEnvVarValidator rejecting optional secret env vars (#5689)

The else-if chain in DetachedEnvVarValidator.Validate fired on any
unset secret env var regardless of its Required flag, returning
"missing required secret environment variable" even for secrets
marked required=false (e.g. OCI_TOKEN, CONTEXT7_API_KEY).

Remove the erroneous else-if branch so optional secrets with no
supplied value and no default are silently skipped, matching the
behaviour of CLIEnvVarValidator.

Fixes #5688

Co-authored-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

70995 of 105141 relevant lines covered (67.52%)

64.2 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	28457005695.1	30 Jun 2026 03:49PM UTC	774	67.52	GitHub Action Run

stacklok / toolhive / 28457005695
68%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 28457005695

stacklok / toolhive / 28457005695 68%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 28457005695

stacklok / toolhive / 28457005695
68%

README BADGES
x