[NOTICKET][AI] docs: update PROGRESS for tool layer + tracing + gated slow-path wiring

PROGRESS.md

@@ -2,8 +2,71 @@
 
 Persistent tracker mirroring the 42-item ownership table in `REPO_CONTEXT.md` "Team — division of work". Update as PRs land. Future Claude Code sessions read this to know what's already done.
 
-**Last updated**: 2026-06-08 (slow-path agent layer — TaskRunner + Assembler + Coordinator)
-**Current open PR**: `pr/1` — active. Cleanup PR committed and pushed.
+**Last updated**: 2026-06-10 (tool layer complete + Langfuse tracing + gated slow-path wiring)
+**Current open PR**: `pr/2` — active.
+
+---
+
+## What just shipped (2026-06-09/10 — tool layer, tracing, slow-path wiring)
+
+Big stretch since the slow-path workers landed. The tool layer (teammate-owned) is
+now **complete and real**, the slow path is **wired into `ChatHandler` behind a gate**,
+and the whole chat pipeline is **traced**. Fast path still untouched; live behavior
+unchanged (flags default off).
+
+**Tool layer — COMPLETE (teammate, KM-624→630).** `src/tools/` was re-created (the
+2026-05-11 note about deleting it is superseded). Now teammate-owned:
+- `src/tools/analytics/` — the 8 **composite** `analyze_*` computes (descriptive,
+  aggregate, comparison, contribution, profile, correlation, segment, trend) +
+  prompt-style DESCRIPTIONs (KM-624/625).
+- `src/tools/contracts.py` — canonical `ToolSpec`/`ToolRegistry`/`ToolOutput` (KM-627).
+  `agents/planner/contracts.py` now just re-exports them + keeps the `BusinessContext`
+  stub (lead's).
+- `src/tools/registry.py::analytics_registry()` (KM-628); `src/tools/invoker.py` +
+  `src/tools/data_access.py` — `AnalyticsToolInvoker` (KM-629), `DataAccessToolInvoker`
+  + `CompositeToolInvoker` (KM-630). All never-throw. **Pattern A confirmed** (`analyze_*`
+  take a `data` `${t<id>}` placeholder from an upstream `query_structured`).
+- **Verified live E2E (2026-06-09):** real `query_structured` against a user's Neon
+  Postgres → `analyze_trend` → Assembler. `analyze_contribution` surfaced a real tool
+  bug (Decimal vs float in `decomposition.py`) — degrade-and-continue held; teammate's
+  domain to fix. **Directive:** agent side does NOT modify `src/tools/` without confirmation.
+
+**Planner — realigned to the real tools (KM-626).** `registry.py::default_registry()`
+composes the real `analytics_registry()` + a local stub for the 4 data-access tools.
+Few-shots grown to **A–D**: A `analyze_contribution`, B `analyze_trend`, C mixed
+structured+unstructured (`retrieve_documents`, independent branch), D `analyze_aggregate`.
+`parallelizable_with` **removed** from `Task` (schema/validator/examples/prompt) —
+TaskRunner derives parallelism from `depends_on` alone.
+
+**Slow-path wiring — built, GATED OFF (KM-626).** `agents/chat_handler.py` gains a
+`structured→slow` branch behind `ChatHandler(enable_slow_path=False)`: when on it builds
+a per-request `CompositeToolInvoker` (composition root) + `SlowPathCoordinator`, streams
+`chat_answer`, persists the `analysis_record`. Two seams isolate the remaining blockers:
+- `agents/planner/business_context.py::get_business_context(user_id)` — async stub
+  `BusinessContext`; TODO(lead) swap for the real read.
+- `agents/slow_path/store.py` — `AnalysisStore` Protocol + `NullAnalysisStore` (logs
+  only). Real store = `analysis_records` table in the catalog DB (Neon `dataeyond`) —
+  **table not created yet**. `chat_answer` still emitted as one chunk (not token-streamed).
+
+**Observability — Langfuse tracing wired (KM-631).** `src/observability/langfuse/
+tracing.py` — `RequestTracer`/`NullTracer`/`TracingToolInvoker` + `_redact`. One trace
+per request groups Orchestrator.classify, Planner.plan (each retry = its own generation),
+Assembler.assemble, Chatbot.astream + tool spans (latency/metadata only). Gated:
+`ChatHandler(enable_tracing=False)`; `api/v1/chat.py` opts in (`=True`). PII policy:
+Orchestrator+Planner unmasked (question + PII-safe summary); Assembler+Chatbot masked
+(see real rows/chunks); tool spans carry name + arg keys + row count only. Zero added
+LLM tokens; verified live to US Cloud.
+
+**Live evals green (2026-06-09, real Azure 4o):** `RUN_PLANNER_EVAL=1` and
+`RUN_SLOW_PATH_EVAL=1` both pass — Planner emits valid catalog-consistent `QueryIR` and
+wires Pattern A correctly; self-corrects via retry.
+
+**Open follow-ups:** real `BusinessContext` (lead); create `analysis_records` table +
+real `AnalysisStore`; register data-access `ToolSpec`s upstream (`data_access_registry()`)
+or keep the planner stub; 4o → GPT-mini deployment swap; flip `enable_slow_path` on once
+`BusinessContext` is real. NOTE: 3 test files pre-existing broken from rename rot
+(`test_chat_handler.py`, `test_intent_router.py`, `test_answer_agent.py` import the old
+`answer_agent`/`intent_router` module names).
 
 ---