From 610d5141267ada335aff86dcf19dbf21d5bf485a Mon Sep 17 00:00:00 2001
From: Lucas Franceschino <lucas@cryspen.com>
Date: Tue, 13 Aug 2024 17:32:26 +0200
Subject: [PATCH] fix(cli): when we generate a file that exists and has the
 same content, skip writing

---
 cli/subcommands/src/cargo_hax.rs     | 22 ++++++++++++++++++----
 hax-types/src/diagnostics/message.rs |  3 ++-
 2 files changed, 20 insertions(+), 5 deletions(-)

diff --git a/cli/subcommands/src/cargo_hax.rs b/cli/subcommands/src/cargo_hax.rs
index 6c3b52a6c..ffcabb968 100644
--- a/cli/subcommands/src/cargo_hax.rs
+++ b/cli/subcommands/src/cargo_hax.rs
@@ -161,8 +161,18 @@ impl HaxMessage {
                 let message = Level::Error.title(&message);
                 eprintln!("{}", renderer.render(message))
             }
-            Self::WroteFile { path } => {
-                let title = format!("hax: wrote file {}", path.display());
+            Self::ProducedFile { mut path, wrote } => {
+                // Make path relative if possible
+                if let Ok(current_dir) = std::env::current_dir() {
+                    if let Ok(relative) = path.strip_prefix(current_dir) {
+                        path = PathBuf::from(".").join(relative).to_path_buf();
+                    }
+                }
+                let title = if wrote {
+                    format!("hax: wrote file {}", path.display())
+                } else {
+                    format!("hax: unchanged file {}", path.display())
+                };
                 eprintln!("{}", renderer.render(Level::Info.title(&title)))
             }
             Self::HaxEngineFailure { exit_code } => {
@@ -279,8 +289,12 @@ fn run_engine(
                     } else {
                         let path = out_dir.join(&file.path);
                         std::fs::create_dir_all(&path.parent().unwrap()).unwrap();
-                        std::fs::write(&path, file.contents).unwrap();
-                        HaxMessage::WroteFile { path }.report(message_format, None)
+                        let mut wrote = false;
+                        if fs::read_to_string(&path).as_ref().ok() != Some(&file.contents) {
+                            std::fs::write(&path, file.contents).unwrap();
+                            wrote = true;
+                        }
+                        HaxMessage::ProducedFile { path, wrote }.report(message_format, None)
                     }
                 }
                 FromEngine::DebugString(debug) => {
diff --git a/hax-types/src/diagnostics/message.rs b/hax-types/src/diagnostics/message.rs
index 47ad79a43..0aff7720c 100644
--- a/hax-types/src/diagnostics/message.rs
+++ b/hax-types/src/diagnostics/message.rs
@@ -12,8 +12,9 @@ pub enum HaxMessage {
     EngineNotFound {
         is_opam_setup_correctly: bool,
     } = 0,
-    WroteFile {
+    ProducedFile {
         path: PathBuf,
+        wrote: bool,
     } = 1,
     HaxEngineFailure {
         exit_code: i32,