Skip to content

fix(cli) use vp fmt instead of bin in create#972

Open
Salnika wants to merge 1 commit intovoidzero-dev:mainfrom
Salnika:fix-cli-create-fmt-call
Open

fix(cli) use vp fmt instead of bin in create#972
Salnika wants to merge 1 commit intovoidzero-dev:mainfrom
Salnika:fix-cli-create-fmt-call

Conversation

@Salnika
Copy link
Contributor

@Salnika Salnika commented Mar 16, 2026

vp create was formatting newly scaffolded projects by invoking oxfmt directly instead of going through vp fmt. I
n Yarn PnP projects, that bypassed Vite+'s normal config resolution path, so oxfmt failed to load the generated vite.config.ts

The fix changes the formatting step to call vp fmt --write instead of executing oxfmt directly.

$> vp create --verbose
...

Wrote editor config to .vscode/settings.json

Wrote editor config to .vscode/extensions.json

Rewrote imports in one file

  synth-tools/vite.config.ts

✔ Merged staged config into synth-tools/vite.config.ts

✔ Git hooks configured
◇ Dependencies installed

◇ Format failed




Failed to load configuration file.
/Users/USER/workspace/perso/projects/web/synth-tools/vite.config.ts
Ensure the file has a valid default export of a JSON-serializable configuration object.


You may need to run "vp fmt --write" manually in /Users/User/workspace/perso/projects/web/synth-tools
◇ Scaffolded synth-tools with Vite+ monorepo
• Node 24.14.0  yarn 4.12.0
✓ Dependencies installed in 2.7s
→ Next: cd synth-tools && vp dev apps/website

@netlify
Copy link

netlify bot commented Mar 16, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit ff8b8be
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69b82e1a58f5be0008c04b33

1 similar comment
@netlify
Copy link

netlify bot commented Mar 16, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit ff8b8be
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69b82e1a58f5be0008c04b33

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