Cks-mcp

Added
- **Regression tests for version reconstruction errors** – `compare_versions` and `explain_diff` now have explicit coverage proving that reconstruction failures (e.g. `Object '...' already exists` during patch replay) surface as structured `{"error": ...}` responses rather than unhandled exceptions.

Notes
- No production code changes; test coverage only.

---

Fixed
- **Google Gemini unsupported schema keywords** – `additionalProperties`, `$schema`, `$ref`, `anyOf`, `oneOf`, `allOf`, `default`, `examples`, `const`, `readOnly`, `writeOnly`, and `deprecated` are now stripped during Google provider schema normalization.
- **Google provider schema normalization** – `_normalize_schema_for_google()` now recursively removes unsupported keywords and collapses `anyOf`/`oneOf`/`allOf` to an empty object schema before building Gemini `functionDeclarations`. Source `inputSchema` objects remain unchanged.

Added
- Regression tests verifying every registered tool schema normalizes to a Google-compatible shape and that source schemas remain untouched.

Changed
- No changes to public MCP tool schemas or parameter semantics.

---

Fixed
- **Google `functionResponse.name`** – `ai_chat` now stamps `_google_tool_name` on `tool_result` blocks, and `_to_google_contents()` falls back to resolving the tool name from prior `tool_use` blocks by `tool_use_id`. This prevents Google Gemini from returning HTTP 400 `Name cannot be empty`.
- **Tool schema audit** – all registered tool schemas now pass strict JSON Schema validation: arrays declare `items` with `type`, simple object properties declare `type`, and zero-argument tools explicitly set `additionalProperties: false`.

Added
- `tests/test_tool_schema_compliance.py` walks every registered tool schema and fails on missing `items`, missing property `type`, or ambiguous empty-object schemas.
- Regression tests for `functionResponse.name` resolution in Google tool calling.

Changed
- No public API or parameter semantics changed.

---

Fixed
- **Google Gemini tool-calling schema validation** – `ai_chat` and `evolve_knowledge` schemas no longer send array parameters with missing/empty `items` to Google. Gemini requires `items.type`; the schemas now define it explicitly.
- **Google provider schema normalization** – added `_normalize_schema_for_google()` to recursively repair array schemas before building Gemini function declarations, preventing future schema regressions from breaking tool calling.

Tests
- Added regression coverage for Gemini array-schema normalization and all registered tool schemas.
- Full suite green.

---

Fixed
- **Imports after cks-runtime restructure** – `server.py` and the standalone agents (`critic_agent`, `enrichment_agent`, `fork_resolution_agent`, `pipeline_agent`) now import `CksCoreAdapter` from `cks_runtime.adapters.cks_core` instead of the removed `cks_runtime_plugins.cks_core` path.
- **`clone_graph` hidden dependency** – `from cks.evolution import CanonicalRelation` is no longer relied upon. The handler now imports `CanonicalRelation` from its canonical home `cks.core`.
- Updated test imports and mock paths to match the new module layout of `cks-runtime` and `cks-core`.

Tests
- 1121 passed, 6 skipped.

---

Changed
- **Internal package restructuring** – LLM providers moved into `cks_mcp.llm.providers` (one module per provider), HTTP transport modules into `cks_mcp.transport`, telemetry/observability into `cks_mcp.observability`, and standalone agents into `cks_mcp.agents`.
- `middleware.py` left in place because it is transport-agnostic and used by both stdio and HTTP paths.
- Updated console script entry points in `pyproject.toml` to the new module paths.
- Made provider imports in `LLMClient` lazy, resolving a latent circular import after the providers package was introduced.

Fixed
- All internal imports and test mock paths updated to the new module layout.
- `ruff` and full test suite remain green.

Tests
- 1121 passed, 6 skipped.

---