Inital example of a bare skill that gives contributors their installed mcp servers but only the skills that have been vetted#13376
Conversation
…d mcp servers but only the skills that have been vetted
There was a problem hiding this comment.
Code Review
This pull request introduces a new 'bare-agent' configuration, including its metadata in agent.json and settings in config.yaml. The feedback identifies a discrepancy between the agent's description and its configuration: while the description states the agent should inherit MCP servers, the configuration lacks the necessary settings to enable this. A suggestion was provided to explicitly enable MCP server inheritance to align the implementation with the intended design.
| skills: | ||
| inherit_user: false | ||
| skills_paths: [] |
There was a problem hiding this comment.
The PR description and the agent's description in agent.json state that this agent should inherit the user's installed MCP servers. However, the current configuration only addresses skills and explicitly disables their inheritance. To ensure MCP servers are inherited as intended, you should explicitly add the mcp_servers configuration with inherit_user: true within the customization_discovery_config block.
skills:
inherit_user: false
skills_paths: []
mcp_servers:
inherit_user: true|
Visit the preview URL for this PR (updated for commit 868e5ad): https://flutter-docs-prod--pr13376-r-bare-agent-to-avoid-most-bbwu209f.web.app |
gaaclarke
left a comment
There was a problem hiding this comment.
Interesting approach, let's give it a shot. It will be useful for the sort of problem I was running into.
@gaaclarke was frustrated by a skill that was triggered when he didnt want or expect. Full context in go/flutter-website-skills-frustration-chat

We could fix that specific skill but that does not remove the class of issue, what to do when skills are triggering and making you work more complicated. Until we have resolver style tests that can regression test false positive activations then a custom agent can be a work around in the moment and as a pattern.
Presubmit checklist
of 80 characters or fewer.