This PR applies targeted simplifications to recently added code, reducing duplication and improving clarity while preserving all functionality.

Files Simplified

• lua/markdown-plus/list/handlers.lua — extracted build_list_prefix helper used in 4 places
• lua/markdown-plus/list/parser.lua — simplified get_next_marker and get_previous_marker
• lua/markdown-plus/utils.lua — renamed ambiguous text variable to text_parts

Improvements Made

1. Extracted build_list_prefix helper (handlers.lua)

The pattern of building a list item prefix from indent + marker + optional checkbox appeared 4 times verbatim (in create_next_list_item, handle_enter, handle_normal_o, handle_normal_O). Extracted into a local helper:

local function build_list_prefix(indent, marker, checkbox)
  local prefix = indent .. marker .. " "
  if checkbox then prefix = prefix .. "[ ] " end
  return prefix
end

2. Simplified get_next_marker (parser.lua)

Reduced 6 branches to 3 by extracting the delimiter directly from the marker string (list_info.marker:match("[%.%)]$")). The ordered and ordered_paren cases share identical logic modulo delimiter, same for each letter pair.

3. Simplified get_previous_marker (parser.lua)

Three near-identical blocks (each doing: "check previous item at same indent, increment, or return initial") collapsed into one shared lookup block. The get_line + parse_list_line call now appears once instead of 3 times.

4. Renamed text → text_parts (utils.lua)

In get_text_in_range, the accumulator was ambiguously named text (same name as the concept being built). Renamed to text_parts for clarity.

Changes Based On

Recent changes from:

• Commit d4d836b — feat(ci): add agentic workflow code-simplifier (feat(ci): add agentic workflow code-simplifier #222) which introduced the full codebase

Testing

• ✅ No functional changes — logic is identical, only structure differs
• ⚠️ Test suite requires plenary.nvim (not installed in CI environment); linting tools (luacheck, stylua) also not available in this runner
• All changes are purely structural refactors with no behavior difference

Review Focus

Please verify:

• build_list_prefix correctly replicates the 4 inline patterns it replaces
• get_next_marker delimiter extraction (marker:match("[%.%)]$")) covers all 6 type variants correctly
• get_previous_marker restructuring preserves the "check prev item → fallback to initial" logic for all 3 ordered/letter types

References:

• §22793324374

Generated by Code Simplifier · ◷

To install this agentic workflow, run

gh aw add github/gh-aw/.github/workflows/code-simplifier.md@852cb06ad52958b402ed982b69957ffc57ca0619

network:
  allowed:
    - defaults
    - "index.crates.io"

• expires on Mar 8, 2026, 6:04 AM UTC