From fc6c7bce225be790cf83aeb881fad74278b8a809 Mon Sep 17 00:00:00 2001 From: Bilal Syed Date: Sun, 31 May 2026 03:00:49 +0500 Subject: [PATCH] docs: fix stale test and type-check paths in CONTRIBUTING The documented contributor commands point at directories that do not exist on a fresh clone: - `mypy --strict src/cortex/` -> there is no `src/cortex/`; pyproject sets `packages = ["mcp_server"]`, so the path is `mcp_server/`. - `pytest tests/{unit,integration,benchmark}` -> `testpaths = ["tests_py"]` and the suites live under `tests_py/` (`tests_py/integration`, `tests_py/benchmarks`, ...). Docs-only: corrected the paths to match pyproject.toml and the tree, and realigned the comment column. --- CONTRIBUTING.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 10faa7b6..16079a74 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -107,7 +107,7 @@ project-specific rules: - **No mutable default arguments.** No globals except for read-once configuration objects. - **No bare `except:`.** Catch the specific exception you mean. -- **Type-checked at `--strict`.** `mypy --strict src/cortex/` must pass. +- **Type-checked at `--strict`.** `mypy --strict mcp_server/` must pass. - **§4.1 File ≤500 lines, §4.2 function ≤50 lines.** The full standard lives in @@ -118,11 +118,11 @@ The full standard lives in ## Testing ```bash -pytest # full suite (~2500 tests) -pytest tests/unit # unit only -pytest tests/integration # PostgreSQL-backed integration -pytest tests/benchmark -k locomo # subset -pytest -x --ff # stop on first fail, run failures first +pytest # full suite (~2500 tests) +pytest tests_py/core # one subsystem (unit-level) +pytest tests_py/integration # PostgreSQL-backed integration +pytest tests_py/benchmarks -k locomo # subset +pytest -x --ff # stop on first fail, run failures first ``` Tests run against a local PostgreSQL instance. CI provisions a fresh DB