FE-879: Lazy per-slice cook worktrees and shared node_modules for brownfield

[kostandinang]

Stack Context

Stacks on FE-864 (#212). Refines the brownfield per-slice worktree provisioning FE-755 introduced -- makes it lazy and shares node_modules -- without changing what cook produces. (brunch serve / FE-878 is what surfaced the startup cost.)

What?

• Lazy provisioning -- slice-worktree creation moves out of the eager wireHandlers loop into resolveSliceCwd, via a new idempotent ensureSliceWorktree. A slice's worktree is materialized on first fire, so a run touching 2 of 8 slices pays for 2 worktrees, not 8. Rework re-fires are no-ops; provisioning is synchronous (execFileSync), so concurrent fires of distinct slices serialize on the JS thread -- no two git worktree add calls overlap under the parallel policy.
• Shared node_modules -- each slice symlinks node_modules/ to the parent worktree's single copy instead of CoW-copying it per slice (SHAREABLE_TOP_LEVEL_ENTRIES). walkFiles already skips symlinks, so the shared tree is never re-walked during dependency seeding, merge, or promotion. Other gitignored dirs (dist/) still copy per slice.

Why?

In codebase mode the slice layout is always per-slice, and every slice's worktree was provisioned up front -- N git worktree add + N recursive node_modules CoW copies, paid synchronously before any slice fired (9 of each for an 8-slice plan). The per-slice node_modules copy dominates wall-clock.

Behavior / risk

Correctness-neutral: same worktrees on the same branches, just lazy; deps resolve through the symlink. Trade-off: build caches under node_modules (.cache, .vite) become shared across parallel slices -- fine for cook's transient runs; documented at the call site.

Tests

npm run verify green. New unit tests: slice node_modules is a symlink (not a copy), other gitignored content still copies, ensureSliceWorktree idempotent. Brownfield integration smoke unchanged.

[kostandinang]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• FE-879: Lazy per-slice cook worktrees and shared node_modules for brownfield #223 👈 (View in Graphite)
• FE-864: Orchestrator improvements umbrella — brownfield feature delivery from spec #212 : 1 other dependent PR (#213 )
• FE-843: Expand cook toolchain profiles, selected at plan time #198
• FE-841: Embed pi as an in-process SDK to drop the external pi CLI dependency #194 : 1 other dependent PR (#211 )
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

PR Summary

Low Risk
Performance-focused brownfield cook hardening with tests and documented trade-off (shared tool caches under node_modules during parallel slices); same branches/worktrees, just lazier provisioning.

Overview
Lazy per-slice worktrees: Codebase mode no longer provisions every slice’s git worktree in wireHandlers. Provisioning moves to resolveSliceCwd via new ensureSliceWorktree (idempotent on rework; rejects slice ids that collide with existing parent paths that aren’t real cook worktrees).

Shared node_modules: After slice checkout, copyMissingTopLevelEntries can symlink named top-level dirs (today node_modules via SHAREABLE_TOP_LEVEL_ENTRIES) instead of CoW-copying; other gitignored dirs like dist/ still copy per slice.

Docs: memory/PLAN.md records cook-worktree-laziness (FE-879) as done and marks the prior over-copy / eager-seeding follow-on resolved.

Tests: Coverage for the node_modules symlink, non-node_modules CoW copy, and ensureSliceWorktree idempotency + collision errors.

^{Reviewed by Cursor Bugbot for commit 8c8b78b. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit e35fa92. Configure here.}