Skip to content

v0.1.7 — doctor refinement: own-stack port detection + drop RAM check

Choose a tag to compare

View all tags
@Rishabhmannu Rishabhmannu released this 30 May 14:31
v0.1.7
8d50097

Tightens `financebench doctor` to be less noisy when your stack is already running, and drops the RAM check that was generating false positives on macOS.

Fixes

  • Port check: detect our own running stack instead of flagging it. Pre-0.1.7, when your financebench stack was up, lsof saw ports 8000/6333/5432/6380 held by Docker Desktop's backend PID. Doctor flagged them as conflicts and recommended `kill ` — which would kill Docker Desktop itself. Now we `docker ps` first, match container names ending in `-api-1` / `-qdrant-1` / `-postgres-1` / `-redis-1`, and report PASS with "in use by (your running stack)" when the port belongs to us. The wizard's `up -d --build` is idempotent, so this is correctly not a blocking condition. (cli/doctor/checks.py)

  • RAM check removed. The previous `psutil.virtual_memory().available` reading was too pessimistic on macOS — the OS aggressively caches in RAM, reports `available` low even when memory pressure is fine, and pages to SSD-backed swap under pressure. On 16 GB Apple Silicon, "3.2 GB available" triggered a WARN that was almost always false. Removed cleanly. If real OOMs ever surface, a cross-platform RAM probe via `vm_stat` (Darwin) and `/proc/meminfo` (Linux) can come back as a proper signal.

  • psutil dropped from main dependencies (was added in 0.1.6 only for the now-removed RAM check). CLI bundle: 37 MB → 36 MB.

What the doctor output looks like now when your stack is up

Ports
  [✓] Port 8000 (api)          in use by repo-api-1 (your running stack)
  [✓] Port 6333 (qdrant)       in use by repo-qdrant-1 (your running stack)
  [✓] Port 5432 (postgres)     in use by repo-postgres-1 (your running stack)
  [✓] Port 6380 (redis)        in use by repo-redis-1 (your running stack)

Pre-0.1.7, those four lines were a blocking failure plus three warnings with bogus "kill 8360" fix recipes pointing at Docker Desktop's PID.

Upgrade

pip install --upgrade financebench-rag-agent
cd ~/.financebench/repo
git pull
docker compose -f compose.minimal.yml up -d --build

Verification

  • 23 doctor tests pass (was 19; +4 new for own-stack scenarios)
  • `twine check` PASSED on wheel + sdist
  • Fresh-venv smoke test PASSED (36 MB site-packages)
  • Live `financebench doctor` on dev machine correctly reports running postgres container as PASS, not WARN

Next

0.2.0 — pre-built GHCR image with multi-arch CI. Cuts M1 install from ~10 min build to ~90s pull, eliminates the per-version dep redownload entirely.

Assets 4
Loading