Amin Ghadersohi 62f4df3ec0 fix(mcp): address self-review findings — comments, dedup, modern types ...

- schema_validator.py: add circular-import comment to both local registry
  imports (H1); extract valid_types before the conditional so all_types()
  is called once instead of in each error branch (N1)
- plugin.py: expand BaseChartPlugin docstring to list all default method
  behaviours including schema_error_hint (N3); add comment warning that
  native_viz_types is a class-level shared dict — subclasses must override
  as a class attribute, not mutate in place (M1)
- registry.py: expand _reset_for_testing() docstring with explicit warning
  that direct global assignment is not reverted by pytest monkeypatch —
  callers must restore state in teardown (M2)
- mcp_config.py: replace Dict/Optional from typing with dict/X|None modern
  syntax; remove now-unused Optional and Dict imports (N2)
- initialization/__init__.py: add docstring to configure_mcp_chart_registry()
  explaining the known two-call pattern in MCP-standalone startup and why
  the stale-config window between the two calls is benign in practice (H2)

2026-06-10 23:06:04 +00:00

..

__init__.py

fix(mcp): address self-review findings — comments, dedup, modern types

2026-06-10 23:06:04 +00:00