The planner-run, review-loop and migration-43 tests leave test/crew-parks.test.ts for their own file #3602

For: the second half of #3495's re-plan. It brings test/crew-parks.test.ts under 10 s by moving tests, not rewriting them.

Done:

• [ ] These describe blocks move to a new test/crew-parks-planner.test.ts:

- "the planner's run, with a stand-in Claude Code" - "a pull whose review loop is spent is the planner's (#1655)" - "migration 43"

• [ ] No file under src/ changes, so the type check cannot fail on this pull. Every test and assertion moves unchanged, and the shared helpers at the top of test/crew-parks.test.ts (lines 1–158 on main 8885c0b3) are copied into the new file, or moved into a test/helpers/ module that both files import. scripts/gate.ts finds test files by reading test/, so the new file joins the gate by itself.
• [ ] The three files pass with 32 tests between them. Each file runs under 10 s, with the times named in the pull.

Files: test/crew-parks.test.ts, test/crew-parks-planner.test.ts. Risk: the same as for the first move.