@@ -193,10 +193,14 @@ pub(crate) fn update_agent_core(input: UpdateAgentInput) -> Result<AgentsSetting
193193 if let Some ( old_relative_path) = managed_relative_path_from_config ( old_value) {
194194 let new_relative_path = managed_relative_config_for_name ( & name) ;
195195 if old_relative_path != new_relative_path {
196- let old_abs_path =
197- resolve_safe_managed_abs_path_for_read ( & codex_home, & old_relative_path) ?;
198- let new_abs_path =
199- resolve_safe_managed_abs_path_for_write ( & codex_home, & new_relative_path) ?;
196+ let old_abs_path = resolve_safe_managed_abs_path_for_read (
197+ & codex_home,
198+ & old_relative_path,
199+ ) ?;
200+ let new_abs_path = resolve_safe_managed_abs_path_for_write (
201+ & codex_home,
202+ & new_relative_path,
203+ ) ?;
200204 if new_abs_path. exists ( ) {
201205 return Err ( format ! (
202206 "target config file already exists: {}" ,
@@ -316,8 +320,9 @@ pub(crate) fn delete_agent_core(input: DeleteAgentInput) -> Result<AgentsSetting
316320 if let Some ( relative_path) = managed_relative_path_from_config ( config_file. as_str ( ) ) {
317321 let target = resolve_safe_managed_abs_path_for_read ( & codex_home, & relative_path) ?;
318322 if target. exists ( ) {
319- let backup = std:: fs:: read ( & target)
320- . map_err ( |err| format ! ( "Failed to read agent config file before delete: {err}" ) ) ?;
323+ let backup = std:: fs:: read ( & target) . map_err ( |err| {
324+ format ! ( "Failed to read agent config file before delete: {err}" )
325+ } ) ?;
321326 std:: fs:: remove_file ( & target)
322327 . map_err ( |err| format ! ( "Failed to delete agent config file: {err}" ) ) ?;
323328 deleted_config_backup = Some ( ( target, backup) ) ;
@@ -326,7 +331,9 @@ pub(crate) fn delete_agent_core(input: DeleteAgentInput) -> Result<AgentsSetting
326331 }
327332 }
328333
329- if let Err ( persist_error) = config_toml_core:: persist_global_config_document ( & codex_home, & document) {
334+ if let Err ( persist_error) =
335+ config_toml_core:: persist_global_config_document ( & codex_home, & document)
336+ {
330337 if let Some ( ( path, backup) ) = deleted_config_backup {
331338 if let Err ( restore_error) = std:: fs:: write ( & path, backup) {
332339 return Err ( format ! (
@@ -600,7 +607,9 @@ fn managed_relative_path_from_config(raw_path: &str) -> Option<PathBuf> {
600607 }
601608}
602609
603- fn resolve_managed_agent_config_relative_path ( agent_name : & str ) -> Result < ( PathBuf , PathBuf ) , String > {
610+ fn resolve_managed_agent_config_relative_path (
611+ agent_name : & str ,
612+ ) -> Result < ( PathBuf , PathBuf ) , String > {
604613 let name = normalize_agent_lookup_name ( agent_name) ?;
605614 let codex_home = resolve_codex_home ( ) ?;
606615 let ( _, document) = config_toml_core:: load_global_config_document ( & codex_home) ?;
@@ -803,7 +812,10 @@ mod tests {
803812 fn normalize_agent_name_accepts_expected_shape ( ) {
804813 assert_eq ! ( normalize_agent_name( "explorer" ) . expect( "valid" ) , "explorer" ) ;
805814 assert_eq ! ( normalize_agent_name( "a-1_b" ) . expect( "valid" ) , "a-1_b" ) ;
806- assert_eq ! ( normalize_agent_name( " Explorer " ) . expect( "valid" ) , "explorer" ) ;
815+ assert_eq ! (
816+ normalize_agent_name( " Explorer " ) . expect( "valid" ) ,
817+ "explorer"
818+ ) ;
807819 assert_eq ! ( normalize_agent_name( "A-1_B" ) . expect( "valid" ) , "a-1_b" ) ;
808820 assert_eq ! (
809821 normalize_agent_name( "Hello world" ) . expect( "valid" ) ,
@@ -825,13 +837,11 @@ mod tests {
825837 #[ test]
826838 fn managed_path_detection_accepts_agents_prefix ( ) {
827839 assert_eq ! (
828- managed_relative_path_from_config( "./agents/researcher.toml" )
829- . expect( "managed path" ) ,
840+ managed_relative_path_from_config( "./agents/researcher.toml" ) . expect( "managed path" ) ,
830841 PathBuf :: from( "agents/researcher.toml" )
831842 ) ;
832843 assert_eq ! (
833- managed_relative_path_from_config( "agents/researcher.toml" )
834- . expect( "managed path" ) ,
844+ managed_relative_path_from_config( "agents/researcher.toml" ) . expect( "managed path" ) ,
835845 PathBuf :: from( "agents/researcher.toml" )
836846 ) ;
837847 }
@@ -857,10 +867,7 @@ mod tests {
857867 let mut role = clone_role_table ( role_item) . expect ( "clone role" ) ;
858868 role[ "description" ] = value ( "New" ) ;
859869
860- assert_eq ! (
861- role. get( "custom_key" ) . and_then( Item :: as_str) ,
862- Some ( "keep" )
863- ) ;
870+ assert_eq ! ( role. get( "custom_key" ) . and_then( Item :: as_str) , Some ( "keep" ) ) ;
864871 assert_eq ! ( role. get( "description" ) . and_then( Item :: as_str) , Some ( "New" ) ) ;
865872 }
866873
@@ -910,8 +917,7 @@ mod tests {
910917
911918 #[ test]
912919 fn build_template_content_uses_provided_model_and_reasoning ( ) {
913- let content =
914- build_template_content ( Some ( "blank" ) , Some ( "gpt-5.1" ) , Some ( "high" ) , None ) ;
920+ let content = build_template_content ( Some ( "blank" ) , Some ( "gpt-5.1" ) , Some ( "high" ) , None ) ;
915921 assert ! ( content. contains( "model = \" gpt-5.1\" " ) ) ;
916922 assert ! ( content. contains( "model_reasoning_effort = \" high\" " ) ) ;
917923 }
0 commit comments