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

stacklok / toolhive / 18312986310
51%

Build:
DEFAULT BRANCH: main
Ran 07 Oct 2025 12:46PM UTC
Jobs 1
Files 280
Run time 1min
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

07 Oct 2025 12:41PM UTC coverage: 45.916% (+0.09%) from 45.822%
18312986310

push

github

web-flow
Add secrets management support to ToolHive MCP server (#2006)

* Add secrets management support to ToolHive MCP server

Implement comprehensive secrets management functionality for the ToolHive MCP server:

- Add list_secrets tool to list available secrets from ToolHive secrets store
- Add set_secret tool to set secrets from file paths (file-based input only)
- Enhance run_server tool with secrets parameter support
- Add SecretMapping struct for structured secret name/target specification
- Include comprehensive test coverage for all new functionality
- Integrate with existing ToolHive secrets providers (encrypted, 1Password, etc.)

The run_server tool now accepts a secrets array parameter allowing users to
pass secrets to MCP servers when running them, matching the CLI --secret flag
functionality but through the MCP interface.

* Add documentation for secrets management design decisions

Clarify the following aspects of the secrets management implementation:
- Description field in SecretInfo is populated by providers that support
  it (e.g., 1Password provides "Vault :: Item :: Field" format) and
  remains empty for providers without description support
- SecretMapping intentionally excludes Description field as it's only
  relevant for listing/discovery, not for runtime secret mapping
- ListSecrets request parameter follows MCP tool handler interface
  requirements but is unused since list_secrets takes no arguments

Addresses review feedback from @yrobla on PR #2006.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>

107 of 163 new or added lines in 5 files covered. (65.64%)

16860 of 36719 relevant lines covered (45.92%)

16.33 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
2
57.52
3.99% pkg/mcp/server/run_server.go
25
26.47
pkg/mcp/server/list_secrets.go
29
55.38
pkg/mcp/server/set_secret.go
Jobs
ID Job ID Ran Files Coverage
1 18312986310.1 07 Oct 2025 12:46PM UTC 280
45.92
GitHub Action Run
Source Files on build 18312986310
  • Tree
  • List 280
  • Changed 4
  • Source Changed 3
  • Coverage Changed 4
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #18312986310
  • 718e2cb7 on github
  • Prev Build on main (#18312970313)
  • Next Build on main (#18317216753)
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

© 2025 Coveralls, Inc