only fix

Author: SupperZum

Cargo.toml

@@ -478,7 +478,7 @@ path = "module/move/wcensor"
 ## willbe
 
 [workspace.dependencies.willbe]
-version = "~0.20.0"
+version = "~0.21.0"
 path = "module/move/willbe"
 
 

module/move/willbe/Cargo.toml

@@ -1,7 +1,7 @@
 # module/move/willbe/Cargo.toml
 [package]
 name = "willbe"
-version = "0.20.0"
+version = "0.21.0"
 edition = "2021"
 authors = [
   "Kostiantyn Wandalen <[email protected]>",

module/move/willbe/src/action/cicd_renew.rs

@@ -144,7 +144,7 @@ mod private
       let content = handlebars.render( "module_push", &data )?;
       file_write( &workflow_file_name, &content )?;
 
-      println!( "file_write : {}", &workflow_file_name.display() );
+      println!( "file_write : {:?}", &workflow_file_name );
     }
 
     dbg!( &workflow_root );

module/move/willbe/src/action/crate_doc.rs

@@ -104,7 +104,7 @@ mod private
   /// 
   /// # Errors
   /// Returns an error if the command arguments are invalid, the workspace cannot be loaded
-  #[ allow( clippy::too_many_lines, clippy::result_large_err ) ]
+  #[allow(clippy::too_many_lines)]
   pub fn doc
   (
     workspace : &Workspace,
@@ -123,17 +123,17 @@ mod private
     // --- Get crate name early for --package argument and file naming ---
     let manifest_path_for_name = crate_dir.as_ref().join( "Cargo.toml" );
     let manifest_content_for_name = fs::read_to_string( &manifest_path_for_name )
-    .map_err( CrateDocError::Io )
-    .context( format!( "Failed to read Cargo.toml at {}", manifest_path_for_name.display() ) )
-    .err_with_report( &report )?;
+      .map_err( CrateDocError::Io )
+      .context( format!( "Failed to read Cargo.toml at {}", manifest_path_for_name.display() ) )
+      .err_with_report( &report )?;
     let manifest_toml_for_name = manifest_content_for_name.parse::< Document >()
-    .map_err( CrateDocError::Toml )
-    .context( format!( "Failed to parse Cargo.toml at {}", manifest_path_for_name.display() ) )
-    .err_with_report( &report )?;
+      .map_err( CrateDocError::Toml )
+      .context( format!( "Failed to parse Cargo.toml at {}", manifest_path_for_name.display() ) )
+      .err_with_report( &report )?;
     let crate_name = manifest_toml_for_name[ "package" ][ "name" ]
-    .as_str()
-    .ok_or_else( || CrateDocError::MissingPackageName( manifest_path_for_name.clone() ) )
-    .err_with_report( &report )?;
+      .as_str()
+      .ok_or_else( || CrateDocError::MissingPackageName( manifest_path_for_name.clone() ) )
+      .err_with_report( &report )?;
     // --- End get crate name early ---
 
     // Define the arguments for `cargo doc`

module/move/willbe/src/action/list.rs

@@ -324,10 +324,10 @@ mod private
   }
 
   #[ allow( clippy::trivially_copy_pass_by_ref, clippy::needless_lifetimes ) ]
-  fn process_package_dependency
+  fn process_package_dependency< 'a >
   (
     workspace : &Workspace,
-    package : &WorkspacePackageRef< '_ >,
+    package : &WorkspacePackageRef< 'a >,
     args : &ListOptions,
     dep_rep : &mut tool::ListNodeReport,
     visited : &mut collection::HashSet< DependencyId >

module/move/willbe/src/action/publish.rs

@@ -110,6 +110,7 @@ mod private
   ///
   /// # Arguments
   /// * `patterns` - A vector of patterns specifying the folders to search for packages.
+  /// * `exclude_dev_dependencies` - A boolean value indicating whether to exclude dev dependencies from manifest before publish.
   /// * `dry` - A boolean value indicating whether to perform a dry run.
   /// * `temp` - A boolean value indicating whether to use a temporary directory.
   ///
@@ -120,6 +121,8 @@ mod private
   (
     patterns : Vec< String >,
     channel : channel::Channel,
+    exclude_dev_dependencies : bool,
+    commit_changes : bool,
     dry : bool,
     temp : bool
   )
@@ -216,6 +219,7 @@ mod private
       &tmp,
       &packages_to_publish,
       dir.clone(),
+      exclude_dev_dependencies
     )?;
     let subgraph = subgraph
     .map( | _, n | n, | _, e | e );
@@ -235,6 +239,8 @@ mod private
     .channel( channel )
     .workspace_dir( CrateDir::try_from( workspace_root_dir ).unwrap() )
     .option_base_temp_dir( dir.clone() )
+    .exclude_dev_dependencies( exclude_dev_dependencies )
+    .commit_changes( commit_changes )
     .dry( dry )
     .roots( roots )
     .packages( queue )

module/move/willbe/src/action/publish_diff.rs

@@ -22,6 +22,7 @@ mod private
   pub struct PublishDiffOptions
   {
     path : PathBuf,
+    exclude_dev_dependencies : bool,
     keep_archive : Option< PathBuf >,
   }
 
@@ -168,7 +169,7 @@ mod private
           std::fs::write( local_path, content )?;
         }
       }
-      diffs.insert( path, crate_diff( &l, &r ).exclude( diff::PUBLISH_IGNORE_LIST ) );
+      diffs.insert( path, crate_diff( &l, &r, o.exclude_dev_dependencies ).exclude( diff::PUBLISH_IGNORE_LIST ) );
       let report = tasks[ current_idx ].info.normal_dependencies.clone();
       let printer : Vec< TreePrinter > = report
       .iter()

module/move/willbe/src/command/mod.rs

@@ -27,6 +27,16 @@ mod private
         .kind( Type::String )
         .optional( true )
         .end()
+      .property( "exclude_dev_dependencies" )
+        .hint( "Setting this option to true will temporarily remove development dependencies before executing the command, then restore them afterward. Default is `true`." )
+        .kind( Type::Bool )
+        .optional( true )
+        .end()
+      .property( "commit_changes" )
+        .hint( "Indicates whether changes should be committed. Default is `false`." )
+        .kind( Type::Bool )
+        .optional( true )
+        .end()
       .property( "dry" )
         .hint( "Enables 'dry run'. Does not publish, only simulates. Default is `true`." )
         .kind( Type::Bool )
@@ -49,6 +59,11 @@ mod private
         .kind( Type::Path )
         .optional( true )
         .end()
+      .property( "exclude_dev_dependencies" )
+        .hint( "Setting this option to true will temporarily remove development dependencies before executing the command, then restore them afterward. Default is `true`." )
+        .kind( Type::Bool )
+        .optional( true )
+        .end()
       .property( "keep_archive" )
         .hint( "Save remote package version to the specified path" )
         .kind( Type::Path )

module/move/willbe/src/command/publish.rs

@@ -17,6 +17,10 @@ mod private
   {
     #[ former( default = Channel::Stable ) ]
     channel : Channel,
+    #[ former( default = false ) ]
+    exclude_dev_dependencies : bool,
+    #[ former( default = false ) ]
+    commit_changes : bool,
     #[ former( default = true ) ]
     dry : bool,
     #[ former( default = true ) ]
@@ -52,10 +56,12 @@ mod private
     let PublishProperties
     {
       channel,
+      exclude_dev_dependencies,
+      commit_changes,
       dry,
       temp
     } = o.props.try_into()?;
-    let plan = action::publish_plan( patterns, channel, dry, temp )
+    let plan = action::publish_plan( patterns, channel, exclude_dev_dependencies, commit_changes, dry, temp )
     .context( "Failed to plan the publication process" )?;
 
     let mut formatted_plan = String::new();
@@ -110,6 +116,10 @@ mod private
       else
       { this };
 
+      this = if let Some( v ) = value
+      .get_owned( "exclude_dev_dependencies" ) { this.exclude_dev_dependencies::< bool >( v ) } else { this };
+      this = if let Some( v ) = value
+      .get_owned( "commit_changes" ) { this.commit_changes::< bool >( v ) } else { this };
       this = if let Some( v ) = value
       .get_owned( "dry" ) { this.dry::< bool >( v ) } else { this };
       this = if let Some( v ) = value

module/move/willbe/src/command/publish_diff.rs

@@ -14,6 +14,8 @@ mod private
   #[ derive( former::Former ) ]
   struct PublishDiffProperties
   {
+    #[ former( default = false ) ]
+    exclude_dev_dependencies : bool,
     keep_archive : Option< PathBuf >,
   }
 
@@ -36,10 +38,11 @@ mod private
   pub fn publish_diff( o : VerifiedCommand ) -> error::untyped::Result< () > // qqq : use typed error
   {
     let path : PathBuf = o.args.get_owned( 0 ).unwrap_or( std::env::current_dir()? );
-    let PublishDiffProperties { keep_archive } = o.props.try_into()?;
+    let PublishDiffProperties { keep_archive, exclude_dev_dependencies } = o.props.try_into()?;
 
     let mut o = action::PublishDiffOptions::former()
-    .path( path );
+    .path( path )
+    .exclude_dev_dependencies( exclude_dev_dependencies );
     if let Some( k ) = keep_archive.clone() { o = o.keep_archive( k ); }
     let o = o.form();
 
@@ -61,6 +64,12 @@ mod private
     {
       let mut this = Self::former();
 
+      this = if let Some( v ) = value
+      .get_owned( "exclude_dev_dependencies" )
+      { this.exclude_dev_dependencies::< bool >( v ) }
+      else
+      { this };
+
       this = if let Some( v ) = value
       .get_owned( "keep_archive" )
       { this.keep_archive::< PathBuf >( v ) }