Skip to content

test - #751

Open
Suimrah wants to merge 30 commits into
bootdotdev:mainfrom
Suimrah:main
Open

test#751
Suimrah wants to merge 30 commits into
bootdotdev:mainfrom
Suimrah:main

Conversation

@Suimrah

@Suimrah Suimrah commented Aug 14, 2026

Copy link
Copy Markdown

No description provided.

Suimrah and others added 30 commits August 14, 2026 12:59
npm 10 (Node 22) installs vite's optional peer dependency esbuild@0.28.2,
but the lockfile was generated by npm 11 (Node 26), which omits optional
peers. That mismatch made `npm ci` fail with EUSAGE on every run.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The style job used a bare `run:` key, which is only valid on a step, so it
had neither a runner nor any steps and failed workflow validation. Give it
a real job body that checks out, installs, and runs format:check.

Also fix package.json: a missing comma after the dev script and a trailing
comma after format:check made it invalid JSON.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ci: add style job, fix package.json syntax
The Deploy job pinned node-version 22, which bundles npm 10.9.8. npm 10
tries to satisfy vite 8's *optional* esbuild peer dependency
(^0.27.0 || ^0.28.0) instead of skipping it, so `npm ci` demanded
esbuild@0.28.2 and its 23 platform binaries -- none of which are in
package-lock.json, since the lockfile was generated by npm 11.

ci.yml already uses Node 26 and has been green throughout. Aligning cd.yml
on 26 fixes the install and lets `npm run build` actually run.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ci: use Node 26 in cd.yml to match ci.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant