perf: Improve --profile instrumentation coverage to account for full wall-clock time

The CPU profile previously captured only 204ms of a 1051ms run because
chrome tracing was enabled too late (after config resolution) and many
key functions lacked instrumentation. This moves tracing enablement
earlier and adds #[tracing::instrument] to the uninstrumented gaps.

Author: anthonyshew

crates/turborepo-lib/src/cli/mod.rs

@@ -1292,6 +1292,7 @@ fn get_command(cli_args: &mut Args) -> Result<Command, Error> {
 ///
 /// returns: Result<Payload, Error>
 #[tokio::main]
+#[tracing::instrument(skip_all)]
 pub async fn run(
     repo_state: Option<RepoState>,
     #[allow(unused_variables)] logger: &TurboSubscriber,
@@ -1586,6 +1587,13 @@ pub async fn run(
             let event = CommandEventBuilder::new("run").with_parent(&root_telemetry);
             event.track_call();
 
+            // Enable chrome tracing before any real work so that config
+            // resolution and CommandBase construction are captured.
+            let profile_file = run_args.profile_file_and_include_args();
+            if let Some((ref file_path, include_args)) = profile_file {
+                let _ = logger.enable_chrome_tracing(file_path, include_args);
+            }
+
             let base = CommandBase::new(cli_args.clone(), repo_root, version, color_config)?;
             event.track_ui_mode(base.opts.run_opts.ui_mode);
 
@@ -1594,12 +1602,6 @@ pub async fn run(
                 return Ok(1);
             }
 
-            let profile_file = run_args.profile_file_and_include_args();
-            if let Some((ref file_path, include_args)) = profile_file {
-                // TODO: Do we want to handle the result / error?
-                let _ = logger.enable_chrome_tracing(file_path, include_args);
-            }
-
             run_args.track(&event);
             let exit_code = run::run(base, event).await.inspect(|code| {
                 if *code != 0 {

crates/turborepo-lib/src/commands/mod.rs

@@ -45,6 +45,7 @@ pub struct CommandBase {
 }
 
 impl CommandBase {
+    #[tracing::instrument(skip_all)]
     pub fn new(
         args: Args,
         repo_root: AbsoluteSystemPathBuf,
@@ -76,6 +77,7 @@ impl CommandBase {
         }
     }
 
+    #[tracing::instrument(skip_all)]
     pub fn load_config(
         repo_root: &AbsoluteSystemPath,
         args: &Args,

crates/turborepo-lib/src/commands/run.rs

@@ -7,6 +7,7 @@ use turborepo_ui::sender::UISender;
 
 use crate::{commands::CommandBase, run, run::builder::RunBuilder};
 
+#[tracing::instrument(skip_all)]
 pub async fn run(base: CommandBase, telemetry: CommandEventBuilder) -> Result<i32, run::Error> {
     let signal = get_signal()?;
     let handler = SignalHandler::new(signal);

crates/turborepo-lib/src/microfrontends.rs

@@ -30,6 +30,7 @@ struct ConfigInfo {
 
 impl MicrofrontendsConfigs {
     /// Constructs a collection of configurations from disk
+    #[tracing::instrument(skip_all)]
     pub fn from_disk(
         repo_root: &AbsoluteSystemPath,
         package_graph: &PackageGraph,

crates/turborepo-lib/src/run/builder.rs

@@ -63,6 +63,7 @@ pub struct RunBuilder {
 }
 
 impl RunBuilder {
+    #[tracing::instrument(skip_all)]
     pub fn new(base: CommandBase) -> Result<Self, Error> {
         let api_client = base.api_client()?;
 
@@ -405,6 +406,7 @@ impl RunBuilder {
         })
     }
 
+    #[tracing::instrument(skip_all)]
     fn build_engine<'a>(
         &self,
         pkg_dep_graph: &PackageGraph,

crates/turborepo-lib/src/run/mod.rs

@@ -573,6 +573,7 @@ impl Run {
         info!("Proxy shutdown complete, proceeding with visitor cleanup");
     }
 
+    #[instrument(skip_all)]
     async fn execute_visitor(
         &self,
         ui_sender: Option<UISender>,
@@ -735,6 +736,7 @@ impl Run {
         Ok(exit_code)
     }
 
+    #[instrument(skip_all)]
     pub async fn run(&self, ui_sender: Option<UISender>, is_watch: bool) -> Result<i32, Error> {
         let proxy_shutdown = self.start_proxy_if_needed().await?;
         self.setup_cache_shutdown_handler();

crates/turborepo-repository/src/inference.rs

@@ -73,6 +73,7 @@ impl RepoState {
     /// * `reference_dir`: Turbo's invocation directory
     ///
     /// returns: Result<RepoState, Error>
+    #[tracing::instrument(skip_all)]
     pub fn infer(reference_dir: &AbsoluteSystemPath) -> Result<Self, Error> {
         reference_dir
             .ancestors()

crates/turborepo-run-summary/src/tracker.rs

@@ -343,6 +343,7 @@ impl<'a> RunSummary<'a> {
         }
     }
 
+    #[tracing::instrument(skip_all)]
     fn close_dry_run(
         &mut self,
         pkg_dep_graph: &PackageGraph,
@@ -358,6 +359,7 @@ impl<'a> RunSummary<'a> {
         self.format_and_print_text(pkg_dep_graph, ui)
     }
 
+    #[tracing::instrument(skip_all)]
     fn format_and_print_text(
         &mut self,
         pkg_dep_graph: &PackageGraph,

crates/turborepo-task-executor/src/exec.rs

@@ -603,6 +603,7 @@ pub struct DryRunExecutor<H> {
 }
 
 impl<H: HashTrackerProvider> DryRunExecutor<H> {
+    #[tracing::instrument(skip_all, fields(task = %self.task_id))]
     pub async fn execute_dry_run(&self, tracker: TaskTracker<()>) -> Result<(), InternalError> {
         if let Ok(Some(status)) = self.task_cache.exists().await {
             self.hash_tracker