fix: the slots accept any Mapping at runtime; eight docstrings stop calling cloud tool scoping server-side

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP

[rejojer]

Follow-ups to #424 / #428, from the round-10 review of #427.

• index= / chat= were widened to Mapping[str, Any] in 4e9c56c so the exported TypedDicts pass a checker, but _resolve_index_slot / _resolve_chat_slot still dispatched on isinstance(..., dict) — a MappingProxyType or ChainMap was pyright-clean and raised "must be a string or a dict" at construction. They now narrow on Mapping; the existing comprehension already copies, so a read-only proxy proves the caller's mapping is never mutated (test_slots_take_any_mapping, red on the previous head).
• The eight remaining "scoping is server-side" docstrings (four as_*, three *_agent_config, _tool_specs) said the same untrue thing about doc_id on cloud that 4e9c56c corrected at three sites — cloud tools carry no allowlist, targeting is prompt-level. Deleted rather than reworded; the runtime error already explains it.
• local_chat.py's module docstring predates own-model chat over the cloud bridge; storage_path's prose now names the os.PathLike that 5e2dc9b typed.

435 tests green (434 + 1); pyright on the three touched files unchanged; the without-frameworks leg simulated locally on the new test.

https://claude.ai/code/session_01VQ6mruXZBgw9Hjii8KPbQP