refactor: rename BuiltInCommand to Command and update references

Author: branchseer

crates/vite_task/src/cli/mod.rs

@@ -1,14 +1,14 @@
 use std::sync::Arc;
 
-use clap::Subcommand;
+use clap::Parser;
 use vite_path::AbsolutePath;
 use vite_str::Str;
 use vite_task_graph::{TaskSpecifier, query::TaskQueryKind};
 use vite_task_plan::plan_request::{PlanOptions, PlanRequest, QueryPlanRequest};
 
 /// vite task CLI subcommands
-#[derive(Debug, Subcommand)]
-pub enum BuiltInCommand {
+#[derive(Debug, Parser)]
+pub enum Command {
     /// Run tasks
     Run {
         /// `packageName#taskName` or `taskName`.
@@ -41,7 +41,7 @@ pub enum CLITaskQueryError {
     PackageNameSpecifiedWithRecursive { package_name: Str, task_name: Str },
 }
 
-impl BuiltInCommand {
+impl Command {
     /// Convert to `PlanRequest`, or return an error if invalid.
     pub fn into_plan_request(
         self,

crates/vite_task/src/lib.rs

@@ -4,7 +4,7 @@ mod maybe_str;
 pub mod session;
 
 // Public exports for vite_task_bin
-pub use cli::BuiltInCommand;
+pub use cli::Command;
 pub use session::{LabeledReporter, Reporter, Session, SessionCallbacks, TaskSynthesizer};
 pub use vite_task_graph::{
     config::{

crates/vite_task/src/session/mod.rs

@@ -22,7 +22,7 @@ use vite_task_plan::{
 };
 use vite_workspace::{WorkspaceRoot, find_workspace_root};
 
-use crate::{cli::BuiltInCommand, collections::HashMap};
+use crate::{cli::Command, collections::HashMap};
 
 #[derive(Debug)]
 enum LazyTaskGraph<'a> {
@@ -100,14 +100,9 @@ impl vite_task_plan::PlanRequestParser for PlanRequestParser<'_> {
         }
 
         // Try built-in "run" command (handles "vite run build" in scripts)
-        #[derive(Parser)]
-        enum BuiltInParser {
-            #[clap(flatten)]
-            Command(BuiltInCommand),
-        }
-        if let Ok(BuiltInParser::Command(built_in)) = BuiltInParser::try_parse_from(
-            std::iter::once(program).chain(args.iter().map(Str::as_str)),
-        ) {
+        if let Ok(built_in) =
+            Command::try_parse_from(std::iter::once(program).chain(args.iter().map(Str::as_str)))
+        {
             return Ok(Some(built_in.into_plan_request(cwd)?));
         }
 
@@ -223,7 +218,7 @@ impl<'a> Session<'a> {
     pub async fn plan_from_cli(
         &mut self,
         cwd: Arc<AbsolutePath>,
-        command: BuiltInCommand,
+        command: Command,
     ) -> Result<ExecutionPlan, vite_task_plan::Error> {
         let plan_request = command.into_plan_request(&cwd).map_err(|error| {
             TaskPlanErrorKind::ParsePlanRequestError {

crates/vite_task_bin/src/main.rs

@@ -3,7 +3,7 @@ use std::{process::ExitCode, sync::Arc};
 use clap::Parser;
 use vite_path::{AbsolutePath, current_dir};
 use vite_task::{
-    BuiltInCommand, Session,
+    Command, Session,
     session::reporter::{ExitStatus, LabeledReporter},
 };
 use vite_task_bin::OwnedSessionCallbacks;
@@ -12,7 +12,7 @@ use vite_task_bin::OwnedSessionCallbacks;
 #[command(name = "vite", version)]
 struct Cli {
     #[command(subcommand)]
-    command: BuiltInCommand,
+    command: Command,
 }
 
 #[tokio::main]

crates/vite_task_plan/tests/plan_snapshots/main.rs

@@ -8,15 +8,15 @@ use redact::redact_snapshot;
 use tokio::runtime::Runtime;
 use vite_path::{AbsolutePath, AbsolutePathBuf, RelativePathBuf};
 use vite_str::Str;
-use vite_task::{BuiltInCommand, Session};
+use vite_task::{Command, Session};
 use vite_workspace::find_workspace_root;
 
 /// Local parser wrapper for BuiltInCommand
 #[derive(Parser)]
 #[command(name = "vite")]
 enum Cli {
     #[clap(flatten)]
-    Command(BuiltInCommand),
+    Command(Command),
 }
 
 #[derive(serde::Deserialize, Debug)]