Development CLI: Remove targets record file on Clean.

cli/development-cli/CHANGELOG.md

@@ -1,3 +1,9 @@
+# 0.4.1
+
+## Changes:
+
+* Clean command removes checksum records entirely, making it more reliable.
+
 # 0.4.0
 
 ## Changes:

cli/development-cli/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "cargo-chipmunk"
-version = "0.4.0"
+version = "0.4.1"
 authors = ["Ammar Abou Zor <[email protected]>"]
 edition = "2021"
 description = "CLI Tool for chipmunk application development"

cli/development-cli/src/build_state_records.rs

@@ -259,15 +259,6 @@ impl BuildStateRecords {
             })
     }
 
-    /// Remove the target from the states records
-    pub fn remove_state_if_exist(&mut self, target: Target) -> anyhow::Result<()> {
-        self.involved_targets.insert(target);
-
-        self.states.remove(&target);
-
-        Ok(())
-    }
-
     /// Clears the states from previous build then calculates the states for the involved targets.
     fn update_records(&mut self) -> anyhow::Result<()> {
         self.states.clear();

cli/development-cli/src/main.rs

@@ -152,6 +152,8 @@ async fn main_process(command: Command) -> Result<(), Error> {
             JobsState::init(JobsConfig::new(false));
             init_tracker(ui_mode);
             validate_dev_tools()?;
+
+            BuildStateRecords::remove_records_file()?;
             let targets = get_targets_or_all(target);
             let results = jobs_runner::run(&targets, JobType::Clean).await?;
             (JobType::Clean, results)

cli/development-cli/src/target/mod.rs

@@ -9,7 +9,6 @@ use std::{borrow::Cow, collections::BTreeSet, fmt::Display, iter, path::PathBuf,
 use tokio::fs;
 
 use crate::{
-    build_state_records::BuildStateRecords,
     dev_tools::DevTool,
     fstools,
     job_type::JobType,
@@ -515,15 +514,6 @@ impl Target {
     pub async fn reset(&self) -> anyhow::Result<SpawnResult> {
         let job_def = JobDefinition::new(*self, JobType::Clean);
 
-        {
-            // Clean doesn't differentiate between development and production, and both of them will be
-            // cleaned from the files when the data are persisted.
-            let mut checksum = BuildStateRecords::get(false)?.lock().map_err(|err| {
-                anyhow::anyhow!("Error while acquiring items jobs mutex: Error {err}")
-            })?;
-            checksum.remove_state_if_exist(*self)?;
-        }
-
         let mut paths_to_remove = vec![self.cwd().join("dist")];
         let path = match self.kind() {
             TargetKind::Ts => self.cwd().join("node_modules"),