dcdpr / jp / 16470166745

58%

Build Type

push

github

Committed by web-flow

Commit Message

feat(config, mcp)!: Implement nested inheritance for MCP config (#218)

The previous MCP configuration resolution was based on a simple,
two-level inheritance model that was evaluated at runtime using getter
methods. This approach was brittle and led to complex consumer code.

This commit refactors the configuration loading to use a more robust,
multi-level inheritance model implemented within the `with_fallback`
logic of the partial configuration structs. This centralizes resolution
logic, making it more predictable and simplifying consuming code, as the
final `Config` struct is always fully resolved.

`Tool` configurations now inherit settings through multiple layers of
specificity, from a specific tool on a specific server to global tool
settings on a global server.

BREAKING CHANGE: The inheritance and resolution logic for MCP server and
tool configurations has been completely overhauled.

The previous model used a simple fallback from a specific server/tool
configuration to a global (*) one. The new model implements a more
complex, multi-level inheritance system that correctly merges
configurations from different sources (e.g., user config and default
config).

Users should review their configurations to ensure they are compatible
with the new, more predictable inheritance rules. This may allow for
simplification of existing configurations.

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

Run Details

492 of 533 new or added lines in 8 files covered. (92.31%)

35 existing lines in 3 files now uncovered.

4949 of 12509 relevant lines covered (39.56%)

5.5 hits per line