⚙️ FEATURE-#36: Share MCP servers with pycodeloop's native saved: registry

[FernandoCelmer]

Description

• src/services/mcpRegistry.service.ts — New service that reads/writes ~/.pycodeloop/config.json's mcp_servers section directly (same shape as pycodeloop's MCPServerRegistry), preserving every other top-level key. Exports splitCommand() (tokenizes a raw launch command into command/args, mirroring shlex.split() on the CLI side), saveMcpServer(), listSavedMcpServers(), and deleteSavedMcpServer().
• src/features/settings/settings.controller.ts — manageMcpServers()'s add flow now calls maybeSaveToRegistry() before writing to VS Code settings. If the user opts in, the server is written to the shared pycodeloop registry and pycodeloop.mcpServers stores saved:<name> instead of the raw command string. Declining the prompt keeps the existing raw-command behavior unchanged.
• test/mcpRegistry.service.test.ts — Unit tests covering splitCommand tokenizing: bare command, positional args, single/double-quoted segments, extra whitespace.

Motivation and Context

pycodeloop/mcp.py's MCPServerRegistry persists named MCP server configs so the CLI can reference one as --mcp saved:<name>, meant to be shared between the interactive CLI and serve. The plugin reimplemented its own MCP list entirely inside VS Code settings, never touching that registry — a user running both the extension and pycodeloop chat/run in a terminal had to configure the same servers twice. This is an additive change; declining the prompt keeps prior behavior exactly.

Closes #36.

Types of changes

• New feature (change which adds functionality)
• Bug fix
• Documentation

Checklist

• I have performed a self-review of my own code
• I have added tests that prove my feature works
• I have updated the CHANGELOG
• I have updated the documentation accordingly

[FernandoCelmer]

Code Review

Code issues found: 4

#	Severity	Issue
1	[Blocking]	Sync I/O blocks extension host
2	[Blocking]	writeConfig — silent data loss on write failure
3	[Blocking]	saveMcpServer silently overwrites existing entries
4	[Blocking]	Registry name not validated — colon/space breaks saved: token

[FernandoCelmer]

Code Review Summary

Code issues found: 4

#	Severity	Comment
1	[Blocking]	Sync I/O blocks extension host
2	[Blocking]	writeConfig — silent data loss on write failure
3	[Blocking]	saveMcpServer silently overwrites existing entries
4	[Blocking]	Registry name not validated — colon/space breaks saved: token

[FernandoCelmer]

Automated review — 3 blocking issues found (scores 75–90). See inline comments.