Fix/market data review

[stevehome]

No description provided.

[stevehome]

extra tests and fixes
ok to merge

[MarouaneBD]

Code review

Found 2 issues:

1. README.md was updated to require CEREBRAS_API_KEY, but planning/PLAN.md (the authoritative project spec embedded in CLAUDE.md) still specifies OPENROUTER_API_KEY as the required environment variable and describes the LLM integration as "LiteLLM → OpenRouter". The two documents are now inconsistent — PLAN.md section 5 and 9 should be updated to reflect the switch to direct Cerebras API.

finally/README.md

Lines 42 to 46 in 639ee2e

	| Variable | Required | Description |
	|----------|----------|-------------|
	| `CEREBRAS_API_KEY` | Yes | LLM inference via Cerebras |
	| `MASSIVE_API_KEY` | No | Real market data via Massive/Polygon. Uses simulator if absent. |
	| `LLM_MOCK` | No | Set `true` for deterministic mock LLM responses (testing/CI) |

2. The integration example in planning/MARKET_API.md shows app.include_router(router, prefix="/api"), but create_stream_router() returns a router that already stores routes under /stream/prices (with the router itself carrying the /api/stream prefix). Following the example literally would register the endpoint at /api/api/stream/prices instead of /api/stream/prices.

finally/planning/MARKET_API.md

Lines 184 to 188 in 639ee2e

	router = create_stream_router(price_cache)
	app.include_router(router, prefix="/api")
	# Registers: GET /api/stream/prices (text/event-stream)
	```

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}