--- name: pharo-tdd description: Drive a Genie change test-first through the genie MCP tools, working entirely from the live image (no files). Use when adding behavior or fixing a bug in the image — anything where a test should lead. --- # Pharo TDD loop (live image, via the genie MCP tools) All work goes through the `genie` MCP tools — there are no files. Prefer the most specific MCP tool available: read with the read tools, change methods with `define_method`, change classes with `define_class`, and use dedicated refactoring/removal tools when they exist. Use `eval` only as a last resort when no specific MCP tool can express the operation, and keep it minimal. Confirm every change by reading it back. Never assume a class/selector exists — look it up first. ## Loop 1. **Locate.** Find the class and its test class: `list_classes` (package) → `list_methods` → `get_method_source`. Find the matching `...Test` (TestCase subclass). If none exists, create it via `define_class` (superclass `TestCase`). 2. **Red.** Write or adjust ONE failing test with `define_method` (protocol `tests`). Run it: `run_test` with `class_name`. **Expect failure.** If it passes immediately, the test isn't exercising the new behavior — fix the test before writing code. 3. **Green.** Implement the smallest method that could pass, via `define_method`. Run `run_test` again → expect green. Read the method back with `get_method_source` to confirm what's installed. 4. **Regression check.** Run `run_test` for the whole `package_name`. A green class test with a red package is not done. 5. **Refactor.** Clean up while green, re-running `run_test` after each step. (For larger cleanups, switch to the `pharo-refactor` skill.) 6. **Done.** Persist with `save_image` gated on tests — pass `test_package` so it runs them and saves the image only if green. Summarize what changed (class >> selector list). ## Debugging a red test Genie returns full stack traces: failing block source, receiver, and its variables. Read them. Reproduce with `run_test`, read/search tools, and the smallest existing MCP call that exercises the behavior. Use `eval` only if the needed probe cannot be expressed through a dedicated MCP tool; if you must use it, make it minimal and non-mutating unless the task explicitly requires a mutation. Fix the smallest thing the trace implicates, then re-run the test. Don't guess. ## Rules - One test, one reason to fail, at a time. - Modern class API only (`ShiftClassInstaller` / `define_class`), never the deprecated `subclass:instanceVariableNames:classVariableNames:package:`. - Tests are methods too — install them with `define_method`, protocol `tests`. - `eval` is always a last resort, never the default path.