.github/workflows/check.yml triggers on pull_request: alone, so a direct or fast-forward push to main runs no CI.
Concrete failure: the packages/skills workspace (added in 494f778) landed via the prototype-driven-migration fast-forward with package-lock.json left out of sync, breaking npm ci on every subsequent PR. It stayed invisible because pushes to main aren't checked — only surfaced when #53 became the first PR after the migration. (Fixed in #53 via lockfile sync.)
Do: add a push: branches: [main] trigger to check.yml, and/or branch protection requiring PRs into main. Recorded as an Open tension in INVARIANTS.md.
Found during /close after #53.
.github/workflows/check.ymltriggers onpull_request:alone, so a direct or fast-forward push tomainruns no CI.Concrete failure: the
packages/skillsworkspace (added in494f778) landed via the prototype-driven-migration fast-forward withpackage-lock.jsonleft out of sync, breakingnpm cion every subsequent PR. It stayed invisible because pushes to main aren't checked — only surfaced when #53 became the first PR after the migration. (Fixed in #53 via lockfile sync.)Do: add a
push: branches: [main]trigger tocheck.yml, and/or branch protection requiring PRs into main. Recorded as an Open tension in INVARIANTS.md.Found during /close after #53.