Fix joining of paths

Author: InAnYan

module/move/assistant/Cargo.toml

@@ -57,7 +57,6 @@ serde_with = "3.11.0"
 error_tools = "0.17.0"
 derive_tools = { version = "0.32.0", features = ["full"] }
 regex = { version = "1.10.3" }
-itertools = "0.13.0"
 serde_yaml = "0.9"
 
 [dev-dependencies]

module/move/assistant/src/agents/path.rs

@@ -12,7 +12,6 @@ mod private
     sync::LazyLock,
   };
 
-  use itertools::Itertools;
   use regex::Regex;
 
   /// Path separator string.
@@ -99,23 +98,24 @@ mod private
 
     /// Creates an owned `Path` by joining a given path to `self`.
     ///
-    /// Returns `Err(io::Error)` is the `path` is an absolute path.
+    /// If path is joined with an absolute path, then this absolute
+    /// path will be returned.
     #[ inline ]
-    pub fn join( &self, path : &Path ) -> Result< Self, io::Error >
+    pub fn join( &self, path : &Path ) ->  Self
     {
       if path.is_absolute()
       {
-        Err( io::Error::from( io::ErrorKind::InvalidData ) )
+        path.clone()
       }
       else
       {
         if self.0.ends_with( PATH_SEPARATOR )
         {
-          Ok( Self( format!( "{}{}", self.0, path.0 ) ) )
+          Self( format!( "{}{}", self.0, path.0 ) )
         }
         else
         {
-          Ok( Self( format!( "{}::{}", self.0, path.0 ) ) )
+          Self( format!( "{}::{}", self.0, path.0 ) )
         }
       }
     }

module/move/assistant/tests/inc/agents_tests/path_test.rs

@@ -135,8 +135,7 @@ fn path_join_relative()
 
   let combined = orig_path.join( &append );
 
-  assert!( combined.is_ok() );
-  assert_eq!( combined.unwrap().inner(), "agent::completion" );
+  assert_eq!( combined.inner(), "agent::completion" );
 }
 
 #[ test ]
@@ -147,7 +146,7 @@ fn path_join_absolute()
 
   let combined = orig_path.join( &append );
 
-  assert!( combined.is_err() );
+  assert_eq!( combined.inner(), "::completion" );
 }
 
 #[ test ]
@@ -158,8 +157,7 @@ fn path_join_root()
 
   let combined = orig_path.join( &append );
 
-  assert!( combined.is_ok() );
-  assert_eq!( combined.unwrap().inner(), "::agent" );
+  assert_eq!( combined.inner(), "::agent" );
 }
 
 #[ test ]
@@ -170,8 +168,7 @@ fn path_join_trailing()
 
   let combined = orig_path.join( &append );
 
-  assert!( combined.is_ok() );
-  assert_eq!( combined.unwrap().inner(), "agents::completion" );
+  assert_eq!( combined.inner(), "agents::completion" );
 }
 
 #[ test ]