Use anstream to auto-configure stderr in init_tracing

Summary: Now that we depend on `anstream` for color stripping, we no longer need the manual stripping logic implemented in `init_tracing()`.

Reviewed By: rchen152

Differential Revision: D74350739

fbshipit-source-id: 047a79a4e7f194700fadfaf6cb0244b6f296b212

Author: grievejia

pyrefly/lib/commands/run.rs

@@ -51,7 +51,7 @@ fn init_color(color: ColorChoice) {
 
 impl CommonGlobalArgs {
     pub fn init(&self) {
-        init_tracing(self.verbose, false, false);
+        init_tracing(self.verbose, false);
         init_thread_pool(self.threads);
         init_color(self.color);
     }

pyrefly/lib/test/util.rs

@@ -384,7 +384,7 @@ pub fn get_batched_lsp_operations_report_no_cursor(
 }
 
 pub fn init_test() {
-    init_tracing(true, true, true);
+    init_tracing(true, true);
     // Enough threads to see parallelism bugs, but not too many to debug through.
     init_thread_pool(ThreadCount::NumThreads(NonZeroUsize::new(3).unwrap()));
 }

pyrefly/lib/util/trace.rs

@@ -7,10 +7,9 @@
 
 //! Utilities for working with the `tracing` crate.
 
-use std::io::IsTerminal;
-use std::io::stderr;
 use std::sync::Once;
 
+use anstream::stderr;
 use tracing_subscriber::Layer;
 use tracing_subscriber::filter::EnvFilter;
 use tracing_subscriber::filter::LevelFilter;
@@ -23,7 +22,7 @@ static INIT_TRACING_ONCE: Once = Once::new();
 
 /// Set up tracing so it prints to stderr, and can be used for output.
 /// Most things should use `info` and `debug` level for showing messages.
-pub fn init_tracing(verbose: bool, force_ansi: bool, testing: bool) {
+pub fn init_tracing(verbose: bool, testing: bool) {
     INIT_TRACING_ONCE.call_once(|| {
         const ENV_VAR: &str = "PYREFLY_LOG";
         let mut env_filter = EnvFilter::from_env(ENV_VAR);
@@ -41,7 +40,6 @@ pub fn init_tracing(verbose: bool, force_ansi: bool, testing: bool) {
             .with_file(false)
             .without_time()
             .with_writer(stderr)
-            .with_ansi(force_ansi || stderr().is_terminal())
             .with_target(false);
 
         if testing {