Uh oh!
There was an error while loading. Please reload this page.
fix: use 'environment' key for MCP env vars in skill example - #38969
fix: use 'environment' key for MCP env vars in skill example#38969definity wants to merge 1 commit into
Conversation
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
The following comment was made by an LLM, it may be inaccurate: Potential Duplicate Found:
This PR appears to be addressing the same issue—fixing the MCP environment variable key from |
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
definity
commented
Jul 26, 2026
Perhaps PR #35867 fixes this more completely, but I will leave it to the maintainers to decide! |
Issue for this PR
Fixes (partially) #35860
Type of change
What does this PR do?
The built-in skill example uses
envfor local MCP servers. The schema and runtime expectenvironment. This causes env vars to silently drop when users or models follow the example provided by OpenCode.How did you verify your code works?
Tested before/after making this one change. I was working through this using a local model and it even said it followed the skill example verbatim.
Checklist