mother: preflight numpy/Pillow imports, not just python3 #179

Merged
clawdie merged 1 commit from mother-geodesic-deps into main 2026-06-24 19:31:28 +02:00
Owner

Corrects an inaccuracy from #178: geodesic-dome-mcp is not stdlib-only — it imports numpy and PIL (Pillow) at module load (lines 12-13, unguarded).

Why this matters

The #178 preflight only ran command -v python3. That passes on a host that has python3 but lacks numpy/Pillow — and geodesic-dome-mcp then fails only when the MCP host first invokes it. That's the exact late-failure the preflight was meant to prevent, so the check was effectively false confidence.

Changes

  • setup-mother.sh — after the python3-exists check, add python3 -c "import numpy, PIL"; abort with a pkg install py311-numpy py311-pillow hint if either is missing. Now the preflight verifies the real dependency.
  • MOTHER-SETUP.md — correct the prereq: python3 + numpy + Pillow, install via pkg install python3 py311-numpy py311-pillow.

Scope note: this hardens mother-on-osa setup (G4), separate from the ISO build lane. sh -n clean.

🤖 Generated with Claude Code

Corrects an inaccuracy from #178: `geodesic-dome-mcp` is **not** stdlib-only — it imports `numpy` and `PIL` (Pillow) at module load (lines 12-13, unguarded). ### Why this matters The #178 preflight only ran `command -v python3`. That passes on a host that has python3 but lacks numpy/Pillow — and `geodesic-dome-mcp` then fails *only when the MCP host first invokes it*. That's the exact late-failure the preflight was meant to prevent, so the check was effectively false confidence. ### Changes - **`setup-mother.sh`** — after the python3-exists check, add `python3 -c "import numpy, PIL"`; abort with a `pkg install py311-numpy py311-pillow` hint if either is missing. Now the preflight verifies the *real* dependency. - **`MOTHER-SETUP.md`** — correct the prereq: `python3` + `numpy` + `Pillow`, install via `pkg install python3 py311-numpy py311-pillow`. Scope note: this hardens **mother-on-osa setup (G4)**, separate from the ISO build lane. `sh -n` clean. 🤖 Generated with Claude Code
clawdie added 1 commit 2026-06-24 19:30:45 +02:00
mother: preflight numpy/Pillow imports, not just python3
Some checks failed
CI / markdown (pull_request) Has been cancelled
CI / port (pull_request) Has been cancelled
CI / agent-jail-pkgs (pull_request) Has been cancelled
CI / rust (pull_request) Has been cancelled
0021199077
geodesic-dome-mcp imports numpy + PIL at module load (not stdlib-only, as
#178 incorrectly stated). A present python3 therefore proves nothing — the
preflight would pass on a host missing numpy/Pillow and the tool would fail
only when the MCP host first invokes it.

- setup-mother.sh: add a 'python3 -c "import numpy, PIL"' check after the
  python3-exists check, with a pkg install py311-numpy py311-pillow hint.
- MOTHER-SETUP.md: correct the prereq from 'stdlib only, no pip' to
  'python3 + numpy + Pillow'.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
clawdie merged commit 4c3cca5936 into main 2026-06-24 19:31:28 +02:00
clawdie deleted branch mother-geodesic-deps 2026-06-24 19:31:33 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: clawdie/colibri#179
No description provided.