fix: Fix project graph cache invalidation between moon releases. (#664)

* Include version in cache.

* Update changelog.

Author: milesj

crates/core/project-graph/src/graph_hasher.rs

@@ -1,19 +1,34 @@
 use moon_config::{ProjectsAliasesMap, ProjectsSourcesMap};
-use moon_hasher::{hash_btree, Hasher, Sha256};
+use moon_hasher::{hash_btree, Digest, Hasher, Sha256};
 use serde::{Deserialize, Serialize};
-use std::collections::BTreeMap;
+use std::{collections::BTreeMap, env};
 
-#[derive(Default, Deserialize, Serialize)]
+#[derive(Deserialize, Serialize)]
 #[serde(rename_all = "camelCase")]
 pub struct GraphHasher {
     aliases: BTreeMap<String, String>,
 
     configs: BTreeMap<String, String>,
 
     sources: BTreeMap<String, String>,
+
+    // Version of the moon CLI. We need to include this so that the graph
+    // cache is invalidated between each release, otherwise internal Rust
+    // changes (in project or task crates) are not reflected until the cache
+    // is invalidated, which puts the program in a weird state.
+    version: String,
 }
 
 impl GraphHasher {
+    pub fn new() -> Self {
+        GraphHasher {
+            aliases: BTreeMap::default(),
+            configs: BTreeMap::default(),
+            sources: BTreeMap::default(),
+            version: env::var("MOON_VERSION").unwrap_or_default(),
+        }
+    }
+
     pub fn hash_aliases(&mut self, aliases: &ProjectsAliasesMap) {
         self.aliases.extend(aliases.to_owned());
     }
@@ -32,6 +47,7 @@ impl Hasher for GraphHasher {
         hash_btree(&self.aliases, sha);
         hash_btree(&self.configs, sha);
         hash_btree(&self.sources, sha);
+        sha.update(self.version.as_bytes());
     }
 
     fn serialize(&self) -> serde_json::Value {

crates/core/project-graph/src/project_builder.rs

@@ -556,7 +556,7 @@ impl<'ws> ProjectGraphBuilder<'ws> {
             return Ok(String::new());
         }
 
-        let mut hasher = GraphHasher::default();
+        let mut hasher = GraphHasher::new();
 
         // Hash aliases and sources as-is as they're very explicit
         hasher.hash_aliases(aliases);
@@ -574,6 +574,7 @@ impl<'ws> ProjectGraphBuilder<'ws> {
             &self.workspace.root,
         )?;
 
+        // Hash project-level config (moon.yml)
         let config_hashes = self
             .workspace
             .vcs
@@ -583,6 +584,7 @@ impl<'ws> ProjectGraphBuilder<'ws> {
 
         hasher.hash_configs(&config_hashes);
 
+        // Hash workspace-level configs (entire .moon folder)
         let config_hashes = self
             .workspace
             .vcs

packages/cli/CHANGELOG.md

@@ -7,6 +7,8 @@
 - Fixed an issue where hashing would attempt to hash a directory and crash.
 - Fixed an issue where attempting to hash a large number of files (think 10,000) would hang.
 - Fixed an issue where offline checks would take longer than expected.
+- Fixed an issue where the project graph cache would not invalidate when Rust internals have
+  changed.
 
 ## 0.25.1