benbot/nvim-treesitter
1
0
Fork 0
mirror of https://github.com/nvim-treesitter/nvim-treesitter.git synced 2026-07-20 20:30:05 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(python): improve f-string injections (#8644)
Some checks failed
Lint / Lint Lua files (push) Has been cancelled
Details
Lint / Lint query files (push) Has been cancelled
Details
Lint / Lint docs (push) Has been cancelled
Details
Tests / Build (push) Has been cancelled
Details

Browse source 
Problem: Neovim removed match limits, which can lead to catastrophic performance with injection queries.

Solution: Tighten up f-string injections.
This commit is contained in:
Christian Clason 2026-07-20 00:29:57 +02:00 • committed by GitHub
parent 2497714755
commit 7248feaca4
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 23 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

19
runtime/queries/python/injections.scm
View file

@ -3,7 +3,12 @@
object: (identifier) @_re)
arguments: (argument_list
(string
(string_content) @injection.content))
(string_start)
[
(string_content) @injection.content
(interpolation)
]+
(string_end)))
(#eq? @_re "re")
(#set! injection.language "regex"))
@ -12,11 +17,13 @@
object: (identifier) @_re)
arguments: (argument_list
(concatenated_string
[
(string
(string_content) @injection.content)
(comment)
]+))
(string
(string_start)
[
(string_content) @injection.content
(interpolation)
]+
(string_end))))
(#eq? @_re "re")
(#set! injection.language "regex"))

10
tests/query/injections/python/test.py
View file

@ -14,5 +14,15 @@ re_test = re.compile(
# comment
# ^ @comment
)
a.a(
f"<td class=\"a{a}\">{a}</td>"
f"<td class=\"a{a}\">{a}</td>"
f"<td class=\"a{a}\">{a}</td>"
f"<td class=\"a{a}\">{a}</td>"
f"<td class=\"a{a}\">{a}</td>"
f"<td class=\"a{a}\">{a}</td>"
f"<td class=\"a{a}\">{a}</td>"
f"<td class=\"a{a}\">{a}</td>"
)
print("foo %s bar %d" % ("arg1", 2))
# ^ @printf