chore: fix doc paths and drift, drop unused mypy overrides, ignore tool caches - #40
Conversation
…re tool caches - FRAGMENTS.md pointed at layers/<x>.py, which does not exist from the repo root; the modules live under convergence/layers/. README got the same fix earlier. - mypy reported the pexpect.*/wexpect.* override entries as unused (nothing imports either); remove them so the note goes away. - Ignore .mypy_cache/ and .ruff_cache/, both created by the CI checks. Verified: ruff check ., mypy convergence web, pytest (272 passed, 84.92% coverage), and the web/site data drift check all pass. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U142aAXSzf4pUt89HSvXTH
|
CI status on 586b8c3: the That failure is not this PR's. The job reaches the Claude Code SDK and gets back Nothing in this diff ( Generated by Claude Code |
- README counted "fourteen data files"; data/ also carries the adversarial, holdout, and regression sets the tests use. Say so in the intro and the layout block. - Add the --db-table, --db-limit, --summary, --sender, and --seq flags from demo.py to the Run examples (each example was run to confirm). - .env.example: add CONVERGENCE_MAX_BODY_BYTES and CONVERGENCE_LLM_TIMEOUT_S, both read by web/server.py, with their defaults. - ENGINEERING.md said the SDK is touched "in exactly one file (adapters/)"; adapters/ is a directory with one module per backend. ruff, mypy, pytest (272 passed), and the web data drift check pass. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01U142aAXSzf4pUt89HSvXTH
What changed
layers/<module>.pynow readconvergence/layers/<module>.py. The bare paths do not exist from the repo root; README received the same fix in a1719e6.pexpect.*andwexpect.*from the mypyignore_missing_importsoverride list. Nothing imports either module, and mypy itself reported the entries as unused..mypy_cache/and.ruff_cache/, both created by the CI-mandated tools.--db-table,--db-limit,--summary,--senderand--seqflags fromdemo.py(each example was run).CONVERGENCE_MAX_BODY_BYTESandCONVERGENCE_LLM_TIMEOUT_S, both read byweb/server.py, with their defaults.adapters/)" now says vendor SDKs are imported only insideconvergence/adapters/, one module per backend (verified by grep).Verification
Same commands as
.github/workflows/ci.yml, run on the final head:The
claude-reviewcheck fails for a reason outside this diff; see the PR comment (repository OAuth token).Left for a maintainer
web/VERIFICATION.mdtest counts are stale versus the current suite (it cites 173 deterministic tests; there are 272). It reads as a point-in-time evidence record, so it was not rewritten.llmextra in pyproject listsanthropicandopenaionly;requirements.txtalso listsgoogle-generativeaiandpywinpty. Whether the extra should match is a packaging decision.docs/plans/*anddocs/specs/*are not linked from anywhere outsidedocs/.🤖 Generated with Claude Code
https://claude.ai/code/session_01U142aAXSzf4pUt89HSvXTH