Files

T

Marcos Paulo 48c31dc1b3 build: migrate from npm to pnpm

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>

2026-09-05 04:29:42 +00:00

3.0 KiB

Raw Blame History

Task 15 — Full guide

Agent: page-migrator · Model: Codex — hardest task in the plan Depends on: 05, 10, 13 · Parallel with: 16 Worktree: .agents/scripts/worktree.sh start 15 page-full-guide

Goal

/full-guide/ → Astro. 22 KB of HTML, a 50 KB script, 30 KB of CSS, twelve render functions, bilingual throughout.

What `app.js` actually is

Not application code — a bilingual content database (task 05 already moved it) plus ~12 render* functions that swap innerHTML on tab clicks. Once content is a collection, the remaining JS is small: tab state and a language toggle.

Islands

Only these hydrate. Everything else is server-rendered.

Island	Directive	Why
Phase tabs	`client:visible`	click-driven panel swap
Tree / worker / route / model / effort / skill selectors	`client:visible`	same pattern; consider one generic selector island
Language toggle	`client:idle`	page-wide, not urgent
Copy-prompt buttons	`client:visible`	clipboard

If you end up with 12 separate islands you have missed the pattern — they are one selector component with different data.

Asserted by verify.mjs — all must survive

const phases, const handsOnPrompts, const modelGuide, const skillSources, const skillInstallPrompts, render('plan'), renderTree, renderWorker, renderRoute, renderModelProvider, renderEffort, renderSkillFile, renderSkillWorkflow, renderCommonSkill, renderHandsOn, copyPrompt.

These are implementation-detail assertions — they look deletable and are not. Each pins a feature. Coordinate with task 19 to replace each with an output-level assertion of the same behaviour. Never delete one yourself.

Also: data-copy-target="prompt-install-skills|prompt-basic|prompt-skills", hands-on/starter/, additional-reading.md, role="tablist", <table>.

Watch for

copyPrompt uses navigator.clipboard with a document.execCommand fallback. Keep both — the fallback exists for non-secure contexts.
The hands-on prompt strings are copy-pasted by attendees into an agent. Exact whitespace and line breaks matter.
responsive.css (30 KB) mostly serves this page. Port what is needed, prove the rest dead, delete it. Screenshots are the proof.

Done when

Snapshot diff empty
Every interaction works: all tabs, both languages, all copy buttons
Keyboard: arrow keys move between tabs; focus visible throughout
JS payload smaller than today's 50 KB (content is now static)
Screenshots match at four widths; checklist complete; pnpm run gate green

3.0 KiB Raw Blame History