refactor: move typeAware/typeCheck from CLI flags to vite.config.ts (#739)

oxlint natively reads `options.typeAware` and `options.typeCheck` from
its config file, so there's no need for `vp check` to inject these as
CLI flags when calling `vp lint`.

- Remove `--no-type-aware` / `--no-type-check` flags from `vp check`
- Add `options: { typeAware: true, typeCheck: true }` defaults in
  `vp lint --init`, `vp migrate`, and project root vite.config.ts
- Strip `--type-aware` / `--type-check` from migrated scripts
- Remove `oxc.typeAware` from VS Code settings (extension reads from
  config automatically)
- Add `OxlintOptions` type with `typeAware` and `typeCheck` fields

Author: fengmk2

Cargo.lock

@@ -722,8 +722,6 @@ fn delegated_help_doc(command: &str) -> Option<HelpDoc> {
                         row("--fix", "Auto-fix format and lint issues"),
                         row("--no-fmt", "Skip format check"),
                         row("--no-lint", "Skip lint check"),
-                        row("--no-type-aware", "Disable type-aware linting"),
-                        row("--no-type-check", "Disable TypeScript type checking"),
                         row("-h, --help", "Print help"),
                     ],
                 ),

crates/vite_global_cli/src/help.rs

@@ -26,7 +26,13 @@ fn rewrite_script(script: &str, rules: &[RuleConfig<SupportLang>]) -> String {
     let result = ast_grep::apply_loaded_rules(&preprocessed, rules);
 
     // Step 3: Replace cross-env marker back with "cross-env " (only if we replaced it)
-    if has_cross_env { result.replace(CROSS_ENV_MARKER, CROSS_ENV_REPLACEMENT) } else { result }
+    let result = if has_cross_env {
+        result.replace(CROSS_ENV_MARKER, CROSS_ENV_REPLACEMENT)
+    } else {
+        result
+    };
+
+    result
 }
 
 /// Transform all script strings in a JSON object using the provided function.

crates/vite_migration/src/package.rs

@@ -1,7 +1,8 @@
-use std::path::Path;
+use std::{borrow::Cow, path::Path, sync::LazyLock};
 
 use ast_grep_config::{GlobalRules, RuleConfig, from_yaml_string};
 use ast_grep_language::{LanguageExt, SupportLang};
+use regex::Regex;
 use vite_error::Error;
 
 use crate::ast_grep;
@@ -100,15 +101,33 @@ fn merge_json_config_content(
     // Check if the config uses a function callback (for informational purposes)
     let uses_function_callback = check_function_callback(vite_config_content)?;
 
+    // Strip "$schema" property — it's a JSON Schema annotation not valid in OxlintConfig
+    let ts_config = strip_schema_property(ts_config);
+
     // Generate the ast-grep rules with the actual config
-    let rule_yaml = generate_merge_rule(ts_config, config_key);
+    let rule_yaml = generate_merge_rule(&ts_config, config_key);
 
     // Apply the transformation
     let (content, updated) = ast_grep::apply_rules(vite_config_content, &rule_yaml)?;
 
     Ok(MergeResult { content, updated, uses_function_callback })
 }
 
+/// Regex to match `"$schema": "..."` lines (with optional trailing comma).
+static RE_SCHEMA: LazyLock<Regex> =
+    LazyLock::new(|| Regex::new(r#"(?m)^\s*"\$schema"\s*:\s*"[^"]*"\s*,?\s*\n"#).unwrap());
+
+/// Strip the `"$schema"` property from a JSON/JSONC config string.
+///
+/// JSON config files (`.oxlintrc.json`, `.oxfmtrc.json`) often contain a
+/// `"$schema"` annotation that is meaningful only for editor validation.
+/// When the JSON content is embedded into `vite.config.ts`, the `$schema`
+/// property causes a TypeScript type error because it is not part of
+/// `OxlintConfig` / `OxfmtConfig`.
+fn strip_schema_property(config: &str) -> Cow<'_, str> {
+    RE_SCHEMA.replace_all(config, "")
+}
+
 /// Check if the vite config uses a function callback pattern
 fn check_function_callback(vite_config_content: &str) -> Result<bool, Error> {
     // Match both sync and async arrow functions
@@ -870,6 +889,52 @@ export default defineConfig({
         );
     }
 
+    #[test]
+    fn test_strip_schema_property() {
+        // With trailing comma
+        let input = r#"{
+  "$schema": "https://raw.githubusercontent.com/nicolo-ribaudo/tc39-proposal-json-schema/refs/heads/main/schema.json",
+  "rules": {
+    "no-console": "warn"
+  }
+}"#;
+        let result = strip_schema_property(input);
+        assert!(!result.contains("$schema"));
+        assert!(result.contains(r#""no-console": "warn""#));
+
+        // Without trailing comma
+        let input = r#"{
+  "$schema": "https://example.com/schema.json"
+}"#;
+        let result = strip_schema_property(input);
+        assert!(!result.contains("$schema"));
+
+        // No $schema - unchanged
+        let input = r#"{
+  "rules": {}
+}"#;
+        assert_eq!(strip_schema_property(input), input);
+    }
+
+    #[test]
+    fn test_merge_json_config_content_strips_schema() {
+        let vite_config = r#"import { defineConfig } from 'vite';
+
+export default defineConfig({});"#;
+
+        let oxlint_config = r#"{
+  "$schema": "https://raw.githubusercontent.com/nicolo-ribaudo/tc39-proposal-json-schema/refs/heads/main/schema.json",
+  "rules": {
+    "no-console": "warn"
+  }
+}"#;
+
+        let result = merge_json_config_content(vite_config, oxlint_config, "lint").unwrap();
+        assert!(result.updated);
+        assert!(!result.content.contains("$schema"));
+        assert!(result.content.contains(r#""no-console": "warn""#));
+    }
+
     #[test]
     fn test_indent_multiline() {
         // Single line - no change

crates/vite_migration/src/vite_config.rs

@@ -28,7 +28,7 @@
   "vue-mini": {
     "repository": "https://github.com/vue-mini/vue-mini.git",
     "branch": "master",
-    "hash": "c51332662993dde44f665822bdea94cd0abf368b"
+    "hash": "23df6ba49e29d3ea909ef55874f59b973916d177"
   },
   "vite-plugin-react": {
     "repository": "https://github.com/vitejs/vite-plugin-react.git",

ecosystem-ci/repo.json

@@ -115,12 +115,6 @@ pub enum SynthesizableSubcommand {
         /// Skip lint check
         #[arg(long = "no-lint")]
         no_lint: bool,
-        /// Disable type-aware linting
-        #[arg(long = "no-type-aware")]
-        no_type_aware: bool,
-        /// Disable TypeScript type checking
-        #[arg(long = "no-type-check")]
-        no_type_check: bool,
         /// File paths to check (passed through to fmt and lint)
         #[arg(trailing_var_arg = true)]
         paths: Vec<String>,
@@ -1040,14 +1034,7 @@ async fn execute_direct_subcommand(
     let cwd_arc: Arc<AbsolutePath> = cwd.clone().into();
 
     let status = match subcommand {
-        SynthesizableSubcommand::Check {
-            fix,
-            no_fmt,
-            no_lint,
-            no_type_aware,
-            no_type_check,
-            paths,
-        } => {
+        SynthesizableSubcommand::Check { fix, no_fmt, no_lint, paths } => {
             if no_fmt && no_lint {
                 output::error("No checks enabled");
                 print_summary_line(
@@ -1140,13 +1127,6 @@ async fn execute_direct_subcommand(
                 if fix {
                     args.push("--fix".to_string());
                 }
-                if !no_type_aware {
-                    args.push("--type-aware".to_string());
-                    // --type-check requires --type-aware as prerequisite
-                    if !no_type_check {
-                        args.push("--type-check".to_string());
-                    }
-                }
                 if has_paths {
                     args.extend(paths.iter().cloned());
                 }
@@ -1171,18 +1151,7 @@ async fn execute_direct_subcommand(
 
                 match analyze_lint_output(&combined_output) {
                     Some(Ok(success)) => {
-                        let type_checks_enabled = !no_type_aware && !no_type_check;
-                        let issue_label = if type_checks_enabled {
-                            if fix {
-                                "Found no warnings, lint errors, or type errors"
-                            } else {
-                                "Found no warnings, lint errors, or type errors"
-                            }
-                        } else if fix {
-                            "Found no warnings or lint errors"
-                        } else {
-                            "Found no warnings or lint errors"
-                        };
+                        let issue_label = "Found no warnings, lint errors, or type errors";
 
                         let message = format!(
                             "{issue_label} in {}",

packages/cli/binding/src/cli.rs

@@ -1,3 +1,4 @@
 {
+  "ignoredPlatforms": ["win32"],
   "commands": ["vp dlx -s cowsay hello # should work without package.json"]
 }

packages/cli/snap-tests-global/command-dlx-no-package-json/steps.json

@@ -1,4 +1,5 @@
 {
+  "ignoredPlatforms": ["win32"],
   "commands": [
     {
       "command": "cd scripts && vp create --no-interactive vite:application # from non-monorepo subdir",

packages/cli/snap-tests-global/create-from-nonworkspace-subdir/steps.json

@@ -37,6 +37,10 @@ export default defineConfig({
   lint: {
     "rules": {
       "no-unused-vars": "error"
+    },
+    "options": {
+      "typeAware": true,
+      "typeCheck": true
     }
   },
 });

packages/cli/snap-tests-global/migration-auto-create-vite-config/snap.txt

@@ -0,0 +1,5 @@
+{
+  "rules": {
+    "no-unused-vars": "error"
+  }
+}