Revert format change

Author: nojaf

rewatch/src/format.rs

@@ -98,11 +98,6 @@ fn format_files(bsc_exe: &Path, files: Vec<String>, check: bool) -> Result<()> {
 
             if output.status.success() {
                 let original_content = fs::read_to_string(file)?;
-                // On Windows, bsc outputs CRLF to stdout (text mode), but files have LF.
-                // Normalize by removing CR before comparison.
-                #[cfg(windows)]
-                let formatted_content = String::from_utf8_lossy(&output.stdout).replace("\r\n", "\n");
-                #[cfg(not(windows))]
                 let formatted_content = String::from_utf8_lossy(&output.stdout);
                 if original_content != formatted_content {
                     if check {

rewatch/src/watcher.rs

@@ -502,8 +502,8 @@ async fn async_watch(
                         cmd::run(a)
                     }
                     let timing_total_elapsed = timing_total.elapsed();
-                    let compilation_type = if initial_build { "initial" } else { "incremental" };
                     if show_progress {
+                        let compilation_type = if initial_build { "initial" } else { "incremental" };
                         if plain_output {
                             println!("Finished {compilation_type} compilation")
                         } else {
@@ -515,13 +515,6 @@ async fn async_watch(
                                 timing_total_elapsed.as_secs_f64()
                             );
                         }
-                    } else {
-                        // Always log completion for verbose modes
-                        log::info!(
-                            "Finished {} compilation in {:.2}s",
-                            compilation_type,
-                            timing_total_elapsed.as_secs_f64()
-                        );
                     }
 
                     // Populate mtime cache after build completes
@@ -574,12 +567,6 @@ async fn async_watch(
                             timing_total_elapsed.as_secs_f64()
                         );
                     }
-                } else {
-                    // Always log completion for verbose modes
-                    log::info!(
-                        "Finished compilation in {:.2}s",
-                        timing_total_elapsed.as_secs_f64()
-                    );
                 }
 
                 // Populate mtime cache after build completes

tests/rewatch_tests/fixture/.gitattributes

@@ -1,13 +1,5 @@
 import { execSync } from "node:child_process";
-import {
-  cp,
-  mkdtemp,
-  readdir,
-  readFile,
-  realpath,
-  rm,
-  writeFile,
-} from "node:fs/promises";
+import { cp, mkdtemp, readdir, realpath, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import path, { join } from "node:path";
 import { fileURLToPath } from "node:url";
@@ -51,29 +43,6 @@ async function assertFixtureClean() {
   }
 }
 
-/**
- * Normalize CRLF to LF in .res and .resi files.
- * This is needed on Windows where git checkout may convert line endings
- * despite .gitattributes settings, causing the formatter to detect changes.
- */
-async function normalizeLineEndings(dir) {
-  const entries = await readdir(dir, { withFileTypes: true });
-  for (const entry of entries) {
-    const fullPath = join(dir, entry.name);
-    if (entry.isDirectory()) {
-      if (entry.name !== "node_modules" && entry.name !== ".yarn") {
-        await normalizeLineEndings(fullPath);
-      }
-    } else if (entry.name.endsWith(".res") || entry.name.endsWith(".resi")) {
-      const content = await readFile(fullPath, "utf8");
-      const normalized = content.replace(/\r\n/g, "\n");
-      if (normalized !== content) {
-        await writeFile(fullPath, normalized);
-      }
-    }
-  }
-}
-
 export async function createSandbox() {
   // Ensure fixture is clean before copying
   await assertFixtureClean();
@@ -85,12 +54,6 @@ export async function createSandbox() {
   // Copy the fixture directory to the sandbox
   await cp(fixtureDir, realDir, { recursive: true });
 
-  // On Windows, git checkout may use CRLF line endings despite .gitattributes.
-  // Normalize to LF so the formatter doesn't detect spurious changes.
-  if (process.platform === "win32") {
-    await normalizeLineEndings(realDir);
-  }
-
   // Run yarn install to create proper node_modules workspace symlinks.
   // rewatch resolves dependencies via node_modules/<pkgName> and checks if
   // the canonical path is inside the workspace root (no node_modules segment)