chore: Hint user to run force build when contract format changes (#1903)

getdozer · Aug 24, 2023 · def4272 · def4272
1 parent ea5678e
commit def4272
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/dozer-cli/src/errors.rs b/dozer-cli/src/errors.rs
@@ -204,8 +204,12 @@ pub enum BuildError {
     FieldNotFound(String),
     #[error("File system error {0:?}: {1}")]
     FileSystem(PathBuf, std::io::Error),
+    #[error(
+        "Failed to load existing contract: {0}. You have to run a force build: `dozer build --force`."
+    )]
+    FailedToLoadExistingContract(#[source] serde_json::Error),
     #[error("Serde json error: {0}")]
-    SerdeJson(#[from] serde_json::Error),
+    SerdeJson(#[source] serde_json::Error),
     #[error("Failed to generate proto files: {0:?}")]
     FailedToGenerateProtoFiles(#[from] GenerationError),
     #[error("Storage error: {0}")]

diff --git a/dozer-cli/src/simple/build/contract/mod.rs b/dozer-cli/src/simple/build/contract/mod.rs
@@ -205,8 +205,7 @@ fn serde_json_to_path(path: impl AsRef<Path>, value: &impl Serialize) -> Result<
         .write(true)
         .open(path.as_ref())
         .map_err(|e| BuildError::FileSystem(path.as_ref().into(), e))?;
-    serde_json::to_writer_pretty(file, value)?;
-    Ok(())
+    serde_json::to_writer_pretty(file, value).map_err(BuildError::SerdeJson)
 }
 
 fn serde_json_from_path<T>(path: impl AsRef<Path>) -> Result<T, BuildError>
@@ -217,7 +216,7 @@ where
         .read(true)
         .open(path.as_ref())
         .map_err(|e| BuildError::FileSystem(path.as_ref().into(), e))?;
-    Ok(serde_json::from_reader(file)?)
+    serde_json::from_reader(file).map_err(BuildError::FailedToLoadExistingContract)
 }
 
 mod modify_schema;