Marcos Paulo
48c31dc1b3
Ten git worktrees each carried their own 225 MB node_modules (1.1 GB across five) and paid 11s per `npm ci`. pnpm hardlinks from a shared store: the same five worktrees cost ~250 MB total, and a fresh install is 4s. What changed beyond the mechanical rename: - `overrides` moved to `pnpm-workspace.yaml`. pnpm 11 does not read the `pnpm` field in package.json *or* npm's top-level `overrides`, and it fails silently — the vite/defu/language-server pins would have quietly stopped applying. - Build scripts are blocked by default in pnpm; esbuild and sharp are allowed explicitly via `allowBuilds` (renamed from `onlyBuiltDependencies` in 11). - `packageManager` + `engines` pin the toolchain. - gate.sh rejects a package-lock.json/yarn.lock/bun.lock outright, so an agent running `npm install` out of habit fails loudly instead of building a second, divergent dependency tree. - CI bootstraps pnpm with `npm install --global pnpm@11.25.0` rather than corepack (unbundled as of Node 25) or pnpm/action-setup (this self-hosted act-runner has never run a job; fetching a third-party action is not something to discover on the first one). Two pre-existing CI bugs fixed while in the file: - the gate installed with `npm install --package-lock=false`, which discarded the lockfile the previous session had just fixed. - the visual-regression step imported `playwright`, which is not a dependency, and `visual-regression.mjs` has no compare mode anyway — in CI it overwrote its own baselines and passed unconditionally. Removed with a comment; it comes back when it can diff. The `publish` job is now manual (`workflow_dispatch`). During the migration dist/ holds three HTML files against the live pages branch's ten, so publishing on every push to main would take the site down to a stub. Restore at task 20. HANDOVER.md's incident log still says npm where it describes what happened at the time; that is history, not a missed rename. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
51 lines
1.7 KiB
Bash
Executable File
51 lines
1.7 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
# Spin up (or tear down) an isolated worktree for one refactor task.
|
|
#
|
|
# .agents/scripts/worktree.sh start 07 route-cards
|
|
# .agents/scripts/worktree.sh finish 07 route-cards
|
|
#
|
|
# One task, one worktree, one agent. See .agents/rules/git-worktrees.md
|
|
set -euo pipefail
|
|
|
|
action=${1:?usage: worktree.sh <start|finish> <task-number> <slug>}
|
|
number=${2:?task number, e.g. 07}
|
|
slug=${3:?slug, e.g. route-cards}
|
|
|
|
dir="../af-task-${number}"
|
|
branch="refactor/task-${number}-${slug}"
|
|
plan="plans/astro-refactor/task-${number}-${slug}.md"
|
|
|
|
case "$action" in
|
|
start)
|
|
[ -f "$plan" ] || echo "warning: no plan at $plan — check the task number" >&2
|
|
git fetch origin
|
|
git worktree add "$dir" -b "$branch" origin/main
|
|
|
|
# Not optional. `.husky/_` is generated by install and is NOT committed, so
|
|
# a fresh worktree has hooks configured but absent — every commit would pass
|
|
# unchecked. --prefer-offline keeps ten parallel spin-ups off the registry.
|
|
( cd "$dir" && pnpm install --frozen-lockfile && .agents/scripts/verify-hooks.sh )
|
|
|
|
echo
|
|
echo "worktree : $dir"
|
|
echo "branch : $branch"
|
|
echo "brief : $plan"
|
|
echo
|
|
echo "next: cd $dir && read the brief end to end before writing anything"
|
|
;;
|
|
finish)
|
|
# Never remove a worktree with uncommitted work in it.
|
|
if [ -n "$(git -C "$dir" status --porcelain)" ]; then
|
|
echo "refusing: $dir has uncommitted changes" >&2
|
|
git -C "$dir" status --short >&2
|
|
exit 1
|
|
fi
|
|
git worktree remove "$dir"
|
|
echo "removed $dir — branch $branch kept; delete it after the merge lands"
|
|
;;
|
|
*)
|
|
echo "unknown action: $action (expected start|finish)" >&2
|
|
exit 2
|
|
;;
|
|
esac
|