fix(create): inject default fmt: {} config and rebrand oxfmt messages

crates/vite_migration/src/vite_config.rs

@@ -1012,6 +1012,27 @@ export default defineConfig({
         );
     }
 
+    #[test]
+    fn test_merge_json_config_content_empty_object() {
+        let vite_config = r#"import { defineConfig } from 'vite-plus';
+
+export default defineConfig({
+  lint: { options: { typeAware: true, typeCheck: true } },
+});"#;
+
+        let result = merge_json_config_content(vite_config, "{}", "fmt").unwrap();
+        assert!(result.updated);
+        assert_eq!(
+            result.content,
+            r#"import { defineConfig } from 'vite-plus';
+
+export default defineConfig({
+  fmt: {},
+  lint: { options: { typeAware: true, typeCheck: true } },
+});"#
+        );
+    }
+
     #[test]
     fn test_merge_tsdown_config_content_simple() {
         let vite_config = r#"import { defineConfig } from 'vite-plus';

packages/cli/binding/Cargo.toml

@@ -10,6 +10,7 @@ rolldown = ["dep:rolldown_binding"]
 anyhow = { workspace = true }
 async-trait = { workspace = true }
 clap = { workspace = true, features = ["derive"] }
+cow-utils = { workspace = true }
 fspy = { workspace = true }
 rustc-hash = { workspace = true }
 napi = { workspace = true }

packages/cli/binding/src/cli.rs

@@ -12,6 +12,7 @@ use clap::{
     Parser, Subcommand,
     error::{ContextKind, ContextValue, ErrorKind},
 };
+use cow_utils::CowUtils;
 use owo_colors::OwoColorize;
 use rustc_hash::FxHashMap;
 use serde::{Deserialize, Serialize};
@@ -743,6 +744,33 @@ async fn resolve_and_execute_with_stdout_filter(
     Ok(ExitStatus(output.status.code().unwrap_or(1) as u8))
 }
 
+/// Like `resolve_and_execute`, but captures stderr, applies a text filter,
+/// and writes the result to real stderr. Stdout remains inherited (streaming).
+async fn resolve_and_execute_with_stderr_filter(
+    resolver: &SubcommandResolver,
+    subcommand: SynthesizableSubcommand,
+    resolved_vite_config: Option<&ResolvedUniversalViteConfig>,
+    envs: &Arc<FxHashMap<Arc<OsStr>, Arc<OsStr>>>,
+    cwd: &AbsolutePathBuf,
+    cwd_arc: &Arc<AbsolutePath>,
+    filter: impl Fn(&str) -> Cow<'_, str>,
+) -> Result<ExitStatus, Error> {
+    let mut cmd =
+        resolve_and_build_command(resolver, subcommand, resolved_vite_config, envs, cwd, cwd_arc)
+            .await?;
+    cmd.stderr(Stdio::piped());
+
+    let child = cmd.spawn().map_err(|e| Error::Anyhow(e.into()))?;
+    let output = child.wait_with_output().await.map_err(|e| Error::Anyhow(e.into()))?;
+
+    use std::io::Write;
+    let stderr = String::from_utf8_lossy(&output.stderr);
+    let filtered = filter(&stderr);
+    let _ = std::io::stderr().lock().write_all(filtered.as_bytes());
+
+    Ok(ExitStatus(output.status.code().unwrap_or(1) as u8))
+}
+
 struct CapturedCommandOutput {
     status: ExitStatus,
     stdout: String,
@@ -1258,6 +1286,17 @@ async fn execute_direct_subcommand(
                     |_| Cow::Borrowed(""),
                 )
                 .await?
+            } else if matches!(&other, SynthesizableSubcommand::Fmt { .. }) {
+                resolve_and_execute_with_stderr_filter(
+                    &resolver,
+                    other,
+                    None,
+                    &envs,
+                    cwd,
+                    &cwd_arc,
+                    |s| s.cow_replace("oxfmt --init", "vp fmt --init"),
+                )
+                .await?
             } else {
                 resolve_and_execute(&resolver, other, None, &envs, cwd, &cwd_arc).await?
             }

packages/cli/snap-tests-global/create-missing-typecheck/snap.txt

@@ -6,6 +6,7 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix",
   },
+  fmt: {},
   lint: { options: { typeAware: true, typeCheck: true } },
 });
 
@@ -17,6 +18,7 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix",
   },
+  fmt: {},
   lint: { options: { typeAware: true, typeCheck: true } },
 });
 

packages/cli/snap-tests-global/migration-baseurl-tsconfig/snap.txt

@@ -15,6 +15,7 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
+  fmt: {},
   lint: {
     "rules": {
       "no-unused-vars": "error"

packages/cli/snap-tests-global/migration-chained-lint-staged-pre-commit/snap.txt

@@ -30,6 +30,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"

packages/cli/snap-tests-global/migration-env-prefix-lint-staged/snap.txt

@@ -30,6 +30,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"

packages/cli/snap-tests-global/migration-eslint-lint-staged/snap.txt

@@ -30,6 +30,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {
     "plugins": [
       "oxc",

packages/cli/snap-tests-global/migration-eslint-lintstagedrc/snap.txt

@@ -31,6 +31,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {
     "plugins": [
       "oxc",

packages/cli/snap-tests-global/migration-eslint/snap.txt

@@ -37,6 +37,7 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
+  fmt: {},
   lint: {
     "plugins": [
       "oxc",

packages/cli/snap-tests-global/migration-existing-husky-lint-staged/snap.txt

@@ -30,6 +30,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"

packages/cli/snap-tests-global/migration-existing-lint-staged-config/snap.txt

@@ -31,6 +31,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.ts": "vp lint --fix"

packages/cli/snap-tests-global/migration-existing-pnpm-exec-lint-staged/snap.txt

@@ -30,6 +30,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"

packages/cli/snap-tests-global/migration-from-tsdown-json-config/snap.txt

@@ -22,6 +22,7 @@ export default defineConfig({
       "cwd": "./src"
     }
   },
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   server: {
     port: 3000,
@@ -72,6 +73,7 @@ export default defineConfig({
       "cwd": "./src"
     }
   },
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   server: {
     port: 3000,

packages/cli/snap-tests-global/migration-from-tsdown/snap.txt

@@ -27,6 +27,7 @@ export default defineConfig({
     "*": "vp check --fix"
   },
   pack: tsdownConfig,
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
 });
 
@@ -78,6 +79,7 @@ export default defineConfig({
     "*": "vp check --fix"
   },
   pack: tsdownConfig,
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
 });
 

packages/cli/snap-tests-global/migration-lint-staged-in-scripts/snap.txt

@@ -31,6 +31,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.js": "vp lint --fix"

packages/cli/snap-tests-global/migration-lintstagedrc-json/snap.txt

@@ -104,6 +104,7 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     "*.@(js|ts|tsx|yml|yaml|md|json|html|toml)": [

packages/cli/snap-tests-global/migration-lintstagedrc-staged-exists/snap.txt

@@ -34,6 +34,7 @@ lintstagedrc.json still exists
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   staged: {
     '*.js': 'vp check --fix',

packages/cli/snap-tests-global/migration-merge-vite-config-js/snap.txt

@@ -12,6 +12,7 @@ export default {
   staged: {
     "*": "vp check --fix"
   },
+  fmt: {},
   lint: {
     "rules": {
       "no-unused-vars": "error"

packages/cli/snap-tests-global/migration-monorepo-pnpm-overrides-dependency-selector/snap.txt

@@ -13,6 +13,7 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
   plugins: [react()],
 });

packages/cli/snap-tests-global/migration-monorepo-yarn4/snap.txt

@@ -15,6 +15,7 @@ export default defineConfig({
   staged: {
     "*": "vp check --fix"
   },
+  fmt: {},
   lint: {
     "rules": {
       "no-unused-vars": "error"

packages/cli/snap-tests-global/migration-subpath/snap.txt

@@ -36,8 +36,8 @@ VITE+ - The Unified Toolchain for the Web
 import { defineConfig } from 'vite-plus';
 
 export default defineConfig({
+  fmt: {},
   lint: {"options":{"typeAware":true,"typeCheck":true}},
-
 });
 
 > git config --local core.hooksPath || echo 'core.hooksPath is not set' # should NOT be set

packages/cli/snap-tests/fmt-no-config-message/package.json

@@ -0,0 +1,6 @@
+{
+  "name": "@test/fmt-no-config-message",
+  "version": "1.0.0",
+  "private": true,
+  "type": "module"
+}

packages/cli/snap-tests/fmt-no-config-message/snap.txt

@@ -0,0 +1,16 @@
+> vp fmt 2>&1 # should show 'vp fmt --init' instead of 'oxfmt --init'
+Finished in <variable>ms on 4 files using <variable> threads.
+No config found, using defaults. Please add a config file or try `vp fmt --init` if needed.
+
+> vp fmt --init
+Added 'fmt' to 'vite.config.ts'.
+
+> cat vite.config.ts # should have fmt config
+export default {
+  fmt: {
+    ignorePatterns: [],
+  },
+};
+
+> vp fmt 2>&1 # should no longer show 'No config found' message
+Finished in <variable>ms on 4 files using <variable> threads.

packages/cli/snap-tests/fmt-no-config-message/src/index.js

@@ -0,0 +1,5 @@
+function hello() {
+  return 'world';
+}
+
+export { hello };

packages/cli/snap-tests/fmt-no-config-message/steps.json

@@ -0,0 +1,8 @@
+{
+  "commands": [
+    "vp fmt 2>&1 # should show 'vp fmt --init' instead of 'oxfmt --init'",
+    "vp fmt --init",
+    "cat vite.config.ts # should have fmt config",
+    "vp fmt 2>&1 # should no longer show 'No config found' message"
+  ]
+}

packages/cli/snap-tests/fmt-no-config-message/vite.config.ts

@@ -0,0 +1 @@
+export default {};

packages/cli/src/migration/migrator.ts

@@ -726,6 +726,7 @@ export function rewriteStandaloneProject(
   }
   mergeViteConfigFiles(projectPath, silent, report);
   injectLintTypeCheckDefaults(projectPath, silent, report);
+  injectFmtDefaults(projectPath, silent, report);
   mergeTsdownConfigFile(projectPath, silent, report);
   // rewrite imports in all TypeScript/JavaScript files
   rewriteAllImports(projectPath, silent, report);
@@ -771,6 +772,7 @@ export function rewriteMonorepo(
   }
   mergeViteConfigFiles(workspaceInfo.rootDir, silent, report);
   injectLintTypeCheckDefaults(workspaceInfo.rootDir, silent, report);
+  injectFmtDefaults(workspaceInfo.rootDir, silent, report);
   mergeTsdownConfigFile(workspaceInfo.rootDir, silent, report);
   // rewrite imports in all TypeScript/JavaScript files
   rewriteAllImports(workspaceInfo.rootDir, silent, report);
@@ -1340,6 +1342,21 @@ export function injectLintTypeCheckDefaults(
   );
 }
 
+export function injectFmtDefaults(
+  projectPath: string,
+  silent = false,
+  report?: MigrationReport,
+): void {
+  injectConfigDefaults(
+    projectPath,
+    'fmt',
+    '.vite-plus-fmt-init.oxfmtrc.json',
+    JSON.stringify({}),
+    silent,
+    report,
+  );
+}
+
 function injectConfigDefaults(
   projectPath: string,
   configKey: string,