Rename fromAbsoluteString to fromAbsolutePathString (#567)

Author: sproshev

rag/rag-base/src/commonMain/kotlin/ai/koog/rag/base/files/FileSystemProvider.kt

@@ -32,7 +32,7 @@ public object FileSystemProvider {
          * @return A path object representing the absolute path.
          * @throws IllegalArgumentException if [path] is not absolute.
          */
-        public fun fromAbsoluteString(path: String): Path
+        public fun fromAbsolutePathString(path: String): Path
 
         /**
          * Resolves a [path] string against a [base] path.

rag/rag-base/src/commonMain/kotlin/ai/koog/rag/base/files/FilteredFileSystemProvider.kt

@@ -82,7 +82,7 @@ internal open class FilteredReadOnly<P>(
 
     override fun toAbsolutePathString(path: P): String = fs.toAbsolutePathString(path)
 
-    override fun fromAbsoluteString(path: String): P = fs.fromAbsoluteString(path)
+    override fun fromAbsolutePathString(path: String): P = fs.fromAbsolutePathString(path)
 
     override fun fromRelativeString(base: P, path: String): P = fs.fromRelativeString(base, path)
 

rag/rag-base/src/jvmMain/kotlin/ai/koog/rag/base/files/JVMFileSystemProvider.kt

@@ -67,7 +67,7 @@ public object JVMFileSystemProvider {
          * @return The normalized Path representation of the given string.
          * @throws IllegalArgumentException if the resolved path is not absolute.
          */
-        override fun fromAbsoluteString(path: String): Path {
+        override fun fromAbsolutePathString(path: String): Path {
             val resolvedPath = Path.of(toSystemDependentName(path)).normalize()
             require(resolvedPath.isAbsolute) { "Resolved path must be absolute" }
             return resolvedPath

rag/rag-base/src/jvmTest/kotlin/ai/koog/rag/base/files/JVMFileSystemProviderTest.kt

@@ -39,9 +39,9 @@ class JVMFileSystemProviderTest : KoogTestBase() {
     private val readWrite = JVMFileSystemProvider.ReadWrite
 
     @Test
-    fun `test fromAbsoluteString with non-existent path`() {
+    fun `test fromAbsolutePathString with non-existent path`() {
         val nonExistentPath = file1.absolute().pathString + "non_existent"
-        val result = readOnly.fromAbsoluteString(nonExistentPath).toString()
+        val result = readOnly.fromAbsolutePathString(nonExistentPath).toString()
         assertEquals(nonExistentPath, result)
     }
 
@@ -147,9 +147,9 @@ class JVMFileSystemProviderTest : KoogTestBase() {
     }
 
     @Test
-    fun `test ReadOnly fromAbsoluteString`() {
+    fun `test ReadOnly fromAbsolutePathString`() {
         val filePathString = file1.absolute().pathString
-        val fileFromPath = readOnly.fromAbsoluteString(filePathString)
+        val fileFromPath = readOnly.fromAbsolutePathString(filePathString)
         assertEquals(file1, fileFromPath)
     }
 
@@ -260,14 +260,14 @@ class JVMFileSystemProviderTest : KoogTestBase() {
     @Test
     fun `test ReadOnly path windows delimiters`() {
         val filePathString = file1.absolute().pathString.replace("/", "\\")
-        val fileFromString = readOnly.fromAbsoluteString(filePathString)
+        val fileFromString = readOnly.fromAbsolutePathString(filePathString)
         assertEquals(file1, fileFromString)
     }
 
     @Test
     fun `test ReadOnly path unix delimiters`() {
         val filePathString = file1.absolute().pathString.replace("\\", "/")
-        val fileFromString = readOnly.fromAbsoluteString(filePathString)
+        val fileFromString = readOnly.fromAbsolutePathString(filePathString)
         assertEquals(file1, fileFromString)
     }
 
@@ -279,10 +279,10 @@ class JVMFileSystemProviderTest : KoogTestBase() {
     }
 
     @Test
-    fun `test ReadOnly fromAbsoluteString throws exception when resolved path is not absolute`() {
+    fun `test ReadOnly fromAbsolutePathString throws exception when resolved path is not absolute`() {
         val relativePathString = "relative/test/path"
         assertThrows(IllegalArgumentException::class.java) {
-            readOnly.fromAbsoluteString(relativePathString)
+            readOnly.fromAbsolutePathString(relativePathString)
         }
     }
 
@@ -303,7 +303,7 @@ class JVMFileSystemProviderTest : KoogTestBase() {
     @Test
     fun `test ReadOnly path starting with slash windows`() {
         val filePathString = "\\" + file1.absolute().pathString.replace("/", "\\")
-        val fileFromString = readOnly.fromAbsoluteString(filePathString)
+        val fileFromString = readOnly.fromAbsolutePathString(filePathString)
         assertEquals(file1, fileFromString)
     }
 
@@ -401,9 +401,9 @@ class JVMFileSystemProviderTest : KoogTestBase() {
     }
 
     @Test
-    fun `test ReadWrite fromAbsoluteString`() {
+    fun `test ReadWrite fromAbsolutePathString`() {
         val filePathString = file1.absolute().pathString
-        val fileFromPath = readWrite.fromAbsoluteString(filePathString)
+        val fileFromPath = readWrite.fromAbsolutePathString(filePathString)
         assertEquals(file1, fileFromPath)
     }
 

rag/rag-base/src/jvmTest/kotlin/ai/koog/rag/base/files/JVMFilteredFileSystemProviderTest.kt

@@ -91,7 +91,7 @@ class JVMFilteredFileSystemProviderTest : KoogTestBase() {
     fun `test select does not throw`() {
         assertNotNull(fsReadOnly.parent(src2))
         assertNotNull(fsReadOnly.fromRelativeString(src2, assertNotNull(fsReadOnly.relativize(src2, file2))))
-        assertNotNull(fsReadOnly.fromAbsoluteString(assertNotNull(fsReadOnly.toAbsolutePathString(file2))))
+        assertNotNull(fsReadOnly.fromAbsolutePathString(assertNotNull(fsReadOnly.toAbsolutePathString(file2))))
         assertNotNull(fsReadOnly.name(src2))
         assertNotNull(fsReadOnly.extension(file2))
     }

rag/vector-storage/src/commonTest/kotlin/ai/koog/rag/vector/mocks/MockDocumentProviders.kt

@@ -45,7 +45,7 @@ class MockDocumentProvider(val mockFileSystem: MockFileSystem) : DocumentProvide
 class MockFileSystemProvider(val mockFileSystem: MockFileSystem) : FileSystemProvider.ReadWrite<String> {
     override fun toAbsolutePathString(path: String): String = path
 
-    override fun fromAbsoluteString(path: String): String = path
+    override fun fromAbsolutePathString(path: String): String = path
 
     override fun fromRelativeString(base: String, path: String): String = "$base/$path"