feat(cargo-shuttle): beta deploy.deny_dirty config, allow dirty deplo…

…ys by default (#1894)

* feat(cargo-shuttle): beta deploy.deny_dirty config, allow dirty deploys by default

* i got booled

cargo-shuttle/src/config.rs

@@ -154,6 +154,14 @@ pub struct ProjectConfig {
     // unused in cargo-shuttle, used in new platform builder.
     // is used here to validate the type if used.
     pub build_assets: Option<Vec<String>>,
+
+    pub deploy: Option<ProjectDeployConfig>,
+}
+/// Deployment command config
+#[derive(Deserialize, Serialize, Default)]
+pub struct ProjectDeployConfig {
+    /// set to true to deny deployments with uncommited changes. (can use `--allow-dirty`)
+    pub deny_dirty: Option<bool>,
 }
 
 /// .shuttle/config.toml schema (internal project-local config)
@@ -487,6 +495,19 @@ impl RequestContext {
             .as_ref()
     }
 
+    /// # Panics
+    /// Panics if the project configuration has not been loaded.
+    pub fn deny_dirty(&self) -> Option<bool> {
+        self.project
+            .as_ref()
+            .unwrap()
+            .as_ref()
+            .unwrap()
+            .deploy
+            .as_ref()
+            .and_then(|d| d.deny_dirty)
+    }
+
     /// Check if the current project id has been loaded.
     pub fn project_id_found(&self) -> bool {
         self.project_internal

cargo-shuttle/src/lib.rs

@@ -2442,10 +2442,12 @@ impl Shuttle {
             trace!(?repo_path, "found git repository");
 
             let dirty = is_dirty(&repo);
-            if !args.allow_dirty && dirty.is_err() {
+            deployment_req.git_dirty = Some(dirty.is_err());
+
+            let check_dirty = !self.beta || self.ctx.deny_dirty().is_some_and(|d| d);
+            if check_dirty && !args.allow_dirty && dirty.is_err() {
                 bail!(dirty.unwrap_err());
             }
-            deployment_req.git_dirty = Some(dirty.is_err());
 
             if let Ok(head) = repo.head() {
                 // This is typically the name of the current branch
@@ -3219,7 +3221,7 @@ fn is_dirty(repo: &Repository) -> Result<()> {
         }
 
         writeln!(error).expect("to append error");
-        writeln!(error, "To proceed despite this and include the uncommitted changes, pass the `--allow-dirty` or `--ad` flag").expect("to append error");
+        writeln!(error, "To proceed despite this and include the uncommitted changes, pass the `--allow-dirty` flag (alias `--ad`)").expect("to append error");
 
         bail!(error);
     }