fix(tsinjector): pass lnum to line_parser

[phanen]

It's non trivial to determine where the "prompt" line is.
When we have reverse/default layout, info/header/footer border

This is a workaround for user find the prompt hl annoying:

opts.command_history._treesitter = function(line, lnum)
  if lnum == 1 then return end
  return line:match("(.-):?(%d+)[: ](.+)$")
end

Summary by CodeRabbit

• Enhancements
  • Treesitter callbacks now receive line number information as an additional parameter, enabling custom parsers to perform more contextual text processing.

[coderabbitai]

📝 Walkthrough

Walkthrough

The changes extend the _treesitter callback function signature to accept an additional line number argument. The type annotation in types.lua is updated to reflect this new parameter, and the corresponding implementation in tsinjector.lua is modified to pass the current line index when invoking the parser.

Changes

Cohort / File(s)	Summary
Type annotation update lua/fzf-lua/types.lua	Updated _treesitter callback type to include lnum: integer parameter: (fun(line: string, lnum: integer): ...) instead of (fun(line: string): ...)
Parser invocation lua/fzf-lua/win/tsinjector.lua	Modified default parser signature to accept the line number argument and updated the invocation to pass the current line index i when calling line_parser(line:sub(min_col), i)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A line number hops into view,
The parser now knows what to do!
Each line gets its place,
In the treesitter's embrace—
Row by row, making syntax shine through! 🌳✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: passing the line number parameter to the line_parser function in the tsinjector.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ts-lnum

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

lua/fzf-lua/win/tsinjector.lua (1)

130-131: ⚠️ Potential issue | 🟡 Minor

Update parameter type annotation for line_parser.

The line_parser parameter type annotation at line 130 also needs updating to include the lnum parameter. This will resolve the remaining [redundant-parameter] warning at line 174.

🔧 Proposed fix

----@param line_parser (fun(line: string):string?,string?,string?,string?)|boolean?
+---@param line_parser (fun(line: string, lnum: integer):string?,string?,string?,string?)|boolean?
 ---@return function detach

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@lua/fzf-lua/win/tsinjector.lua` around lines 130 - 131, Update the type
annotation for the parameter named line_parser in tsinjector.lua to include the
lnum argument (e.g., change (fun(line:
string):string?,string?,string?,string?)|boolean? to a signature that accepts
(line: string, lnum: number) and returns the four optional strings or nil);
locate the annotation above the function that declares/detects line_parser and
add the lnum parameter to match the runtime usage (this will remove the
[redundant-parameter] warning at the call site around line 174).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@lua/fzf-lua/win/tsinjector.lua`:
- Around line 135-136: The type annotation for default_line_parser is out of
sync with its implementation and types.lua: update the local annotation for
default_line_parser to include the second parameter (lnum) so the signature
matches the actual function (e.g., fun(line: string, lnum: number):
string?,string?,string?,string?) and remove the redundant-parameter warning;
target the annotation immediately above the default_line_parser function to
correct the mismatch.
- Line 174: Update the _treesitter parser functions to accept the second ignored
parameter by changing their signatures from function(line) to function(line, _);
specifically modify the _treesitter implementations named git_log,
command_history, and search_history (the functions currently declared with a
single parameter) so they match the default tsinjector.lua signature
function(line, _) and thereby explicitly accept the lnum/second argument passed
by line_parser(line:sub(min_col), i).

---

Outside diff comments:
In `@lua/fzf-lua/win/tsinjector.lua`:
- Around line 130-131: Update the type annotation for the parameter named
line_parser in tsinjector.lua to include the lnum argument (e.g., change
(fun(line: string):string?,string?,string?,string?)|boolean? to a signature that
accepts (line: string, lnum: number) and returns the four optional strings or
nil); locate the annotation above the function that declares/detects line_parser
and add the lnum parameter to match the runtime usage (this will remove the
[redundant-parameter] warning at the call site around line 174).

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 36a25265-8532-4dd9-bb2f-538977ff4db4

📥 Commits

Reviewing files that changed from the base of the PR and between bde73a6 and f70e87f.

📒 Files selected for processing (2)

• lua/fzf-lua/types.lua
• lua/fzf-lua/win/tsinjector.lua

[phanen]

-> Fix 2633-1 -> #2632

This is a hack but not sure if there better idea
Related: #1731 (comment)