Repository navigation
Parsing Broken: Remove smallest pattern with nested expansion is broken #254
Description
The grammar doesn't appear to handle quoted expansion inside of ${} when removing the smallest suffix pattern via %.
This is broken: "${1%"$2"*}", this is not: "${1%$2*}". The broken behavior only appears when followed by a double quoted string.
I've tried to reduce the issue as much as possible to ease debugging. The following demonstrates the issue:
#!/bin/sh
remove_smallest_1(){
echo "${1%$2*}"
}
remove_smallest_1 deleted d
remove_smallest_2(){
echo "${1%"$2"*}"
}
remove_smallest_2 deleted d
echo "broke"
Note: not quoting the expansion within "${}" is warned about by shellcheck SC2295, which is how I discovered this issue.
Using treesitter with neovim:
Activity
holmanb commentedon Mar 12, 2024
@amaanq I'm not familiar with this codebase. If I were to try to start looking for the source of this issue, where would I start?
It makes files that contain this syntax unusable.
holmanb commentedon Apr 24, 2024
@amaanq would you say that this is a duplicate of #214?
This sounds like a similar class of problem. I'm happy to close if you'd rather. I've just silenced shellcheck on this line in the codebase I discovered this in to use the syntax that treesitter understands.
That said, shellcheck actually recommends this syntax, so more reports of this type of issue might crop up.