Skip to content

Commit bd77147

Browse files
docs: Improve docstrings and comments in VSCode manager
- Update _load_config docstring to reflect full return structure - Capitalize MCP consistently in test comments Co-authored-by: JoJoJoJoJoJoJo <23650418+JoJoJoJoJoJoJo@users.noreply.github.com>
1 parent b9f0ceb commit bd77147

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/mcpm/clients/managers/vscode.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ def _load_config(self) -> Dict[str, Any]:
4646
}
4747
4848
Returns:
49-
Dict containing the client configuration with at least {"servers": {}}
49+
Dict containing the client configuration with at least {"servers": {}, "inputs": []}
5050
"""
5151
# Create empty config with the correct structure
5252
empty_config = {self.configure_key_name: {}, "inputs": []}

tests/test_clients/test_vscode.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ def test_load_config_structure(self, vscode_manager):
109109
assert "inputs" in config
110110
assert isinstance(config["inputs"], list)
111111

112-
# Should NOT have mcp wrapper
112+
# Should NOT have MCP wrapper
113113
assert "mcp" not in config
114114

115115
def test_load_config_preserves_servers(self, vscode_manager):
@@ -147,7 +147,7 @@ def test_save_config_structure(self, vscode_manager):
147147
# Should have inputs array
148148
assert "inputs" in saved_config
149149

150-
# Should NOT have mcp wrapper
150+
# Should NOT have MCP wrapper
151151
assert "mcp" not in saved_config
152152

153153
def test_save_config_preserves_inputs(self, vscode_manager):

0 commit comments

Comments
 (0)