Implemented goto_next & goto_previous

Author: Tobias Laross

README.md

@@ -20,6 +20,7 @@ Vim's built-in marks are great, but they're global and get messy fast. Marksman
 - **Persistent storage** - Your marks survive Neovim restarts with automatic backup
 - **Smart naming** - Context-aware auto-generation using Treesitter and pattern matching
 - **Quick access** - Jump to marks with single keys or interactive UI
+- **Sequential navigation** — Jump to next/previous marks with wrap-around behavior
 - **Enhanced search** - Find marks by name, file path, or content with real-time filtering
 - **Mark reordering** - Move marks up/down to organize them as needed
 - **Multiple integrations** - Works with Telescope, Snacks.nvim, and more
@@ -238,6 +239,8 @@ local marksman = require("marksman")
 -- Basic operations return { success, message, ... }
 local result = marksman.add_mark("my_mark")
 local result = marksman.goto_mark("my_mark")  -- or goto_mark(1) for index
+local result = marksman.goto_next()
+local result = marksman.goto_previous()
 local result = marksman.delete_mark("my_mark")
 local result = marksman.rename_mark("old", "new")
 

lua/marksman/init.lua

@@ -289,6 +289,77 @@ function M.goto_mark(name_or_index)
 	end
 end
 
+local function get_current_mark_index(storage_module)
+	local mark_names = storage_module.get_mark_names()
+	local count = #mark_names
+
+	if count == 0 then
+		return nil, "No marks available"
+	end
+	if count == 1 then
+		return nil, "Not enough marks to move"
+	end
+
+	local marks = storage_module.get_marks()
+	local current_file = vim.fn.expand("%:p")
+	local current_line = vim.fn.line(".")
+
+	local best_index = nil
+	local best_distance = nil
+
+	for index, mark_name in ipairs(mark_names) do
+		local mark = marks[mark_name]
+		if mark.file == current_file then
+			if mark.line == current_line then
+				return index, nil
+			end
+			local distance = math.abs(mark.line - current_line)
+			if not best_distance or distance < best_distance then
+				best_index = index
+				best_distance = distance
+			end
+		end
+	end
+
+	return best_index or 1, nil
+end
+
+---Jump to next mark
+---@return table result Result with success and message
+function M.goto_next()
+	local storage_module = get_storage()
+	if not storage_module then
+		return { success = false, message = "Failed to load storage module" }
+	end
+
+	local current_index, err = get_current_mark_index(storage_module)
+	if not current_index then
+		return { success = false, message = err }
+	end
+
+	local mark_names = storage_module.get_mark_names()
+	local next_index = (current_index % #mark_names) + 1
+	return M.goto_mark(next_index)
+end
+
+---Jump to previous mark
+---@return table result Result with success and message
+function M.goto_previous()
+	local storage_module = get_storage()
+	if not storage_module then
+		return { success = false, message = "Failed to load storage module" }
+	end
+
+	local current_index, err = get_current_mark_index(storage_module)
+	if not current_index then
+		return { success = false, message = err }
+	end
+
+	local mark_names = storage_module.get_mark_names()
+	local previous_index = ((current_index - 2) % #mark_names) + 1
+	return M.goto_mark(previous_index)
+end
+
 ---Delete a mark by name
 ---@param name string Mark name to delete
 ---@return table result Result with success and message

tests/marksman_spec.lua

@@ -125,6 +125,65 @@ describe("marksman.nvim", function()
 			assert.equals(3, vim.fn.col("."))
 		end)
 
+		it("jumps to next mark with wrap-around", function()
+			-- open the test file and place marks on lines 1, 2, and 3
+			vim.cmd("edit " .. test_file)
+
+			vim.fn.cursor(1, 1)
+			marksman.add_mark("m1")
+
+			vim.fn.cursor(2, 1)
+			marksman.add_mark("m2")
+
+			vim.fn.cursor(3, 1)
+			marksman.add_mark("m3")
+
+			-- start at m1
+			vim.fn.cursor(1, 1)
+			local result = marksman.goto_next()
+			assert.is_true(result.success)
+			assert.equals(2, vim.fn.line("."), "Should jump from m1 to m2")
+
+			-- now at m2 -> next should be m3
+			result = marksman.goto_next()
+			assert.is_true(result.success)
+			assert.equals(3, vim.fn.line("."), "Should jump from m2 to m3")
+
+			-- now at m3 -> next should wrap back to m1
+			result = marksman.goto_next()
+			assert.is_true(result.success)
+			assert.equals(1, vim.fn.line("."), "Should wrap from m3 to m1")
+		end)
+
+		it("jumps to previous mark with wrap-around", function()
+			vim.cmd("edit " .. test_file)
+
+			vim.fn.cursor(1, 1)
+			marksman.add_mark("m1")
+
+			vim.fn.cursor(2, 1)
+			marksman.add_mark("m2")
+
+			vim.fn.cursor(3, 1)
+			marksman.add_mark("m3")
+
+			-- start at m1 -> previous should wrap to m3
+			vim.fn.cursor(1, 1)
+			local result = marksman.goto_previous()
+			assert.is_true(result.success)
+			assert.equals(3, vim.fn.line("."), "Should wrap from m1 to m3")
+
+			-- now at m3 -> previous should be m2
+			result = marksman.goto_previous()
+			assert.is_true(result.success)
+			assert.equals(2, vim.fn.line("."), "Should jump from m3 to m2")
+
+			-- now at m2 -> previous should be m1
+			result = marksman.goto_previous()
+			assert.is_true(result.success)
+			assert.equals(1, vim.fn.line("."), "Should jump from m2 to m1")
+		end)
+
 		it("deletes marks", function()
 			vim.cmd("edit " .. test_file)
 			marksman.add_mark("delete_me")