fix(refile): Remove leading slash when refiling to destination

Author: kristijanhusak

lua/orgmode/utils/fs.lua

@@ -75,10 +75,12 @@ function M.trim_common_root(paths)
   local common_root = get_common_root()
 
   if #common_root == 0 then
-    return paths
+    return vim.tbl_map(function(path)
+      return path:gsub('^/', '')
+    end, paths)
   end
 
-  local root = table.concat(common_root, '/') .. '/'
+  local root = '/' .. table.concat(common_root, '/') .. '/'
   local result = {}
   for _, path in ipairs(paths) do
     local relative_path = path:sub(#root + 1)

tests/plenary/utils/fs_spec.lua

@@ -90,9 +90,9 @@ end)
 describe('trim_common_root', function()
   it('trims the common root when all files share the same root with the shortest one', function()
     local result = fs_utils.trim_common_root({
-      'foo/bar/baz.org',
-      'foo/bar/baz/bar.org',
-      'foo/bar/baz3.org',
+      '/foo/bar/baz.org',
+      '/foo/bar/baz/bar.org',
+      '/foo/bar/baz3.org',
     })
     assert.are.same({
       'baz.org',
@@ -103,11 +103,11 @@ describe('trim_common_root', function()
 
   it('trims the common root when there are multiple different roots', function()
     local result = fs_utils.trim_common_root({
-      'foo/bar/tea/notes.org',
-      'foo/bar/tea/todos.org',
-      'foo/bar/baz/work.org',
-      'foo/bar/baz/personal.org',
-      'foo/bar/baz/project.org',
+      '/foo/bar/tea/notes.org',
+      '/foo/bar/tea/todos.org',
+      '/foo/bar/baz/work.org',
+      '/foo/bar/baz/personal.org',
+      '/foo/bar/baz/project.org',
     })
 
     assert.are.same({
@@ -121,11 +121,11 @@ describe('trim_common_root', function()
 
   it('returns paths as they are if they do not share the common root', function()
     local result = fs_utils.trim_common_root({
-      'foo/bar/tea/notes.org',
-      'foo/bar/tea/todos.org',
-      'foo/bar/baz/work.org',
-      'foo/bar/baz/personal.org',
-      'other/bar/baz/project.org',
+      '/foo/bar/tea/notes.org',
+      '/foo/bar/tea/todos.org',
+      '/foo/bar/baz/work.org',
+      '/foo/bar/baz/personal.org',
+      '/other/bar/baz/project.org',
     })
 
     assert.are.same({