Suggest incorrectly substitutes variables that have more than one character

[sebastien-rosset]

With the rule below:

	m.Match(`$x.$f($fmt, $e)`).
		Suggest(`$x.$f($fmt, zap.Error($e))`)

If the code is:

logger.Sugar().Errorf("error: %v", err)

Then the suggestion will be slightly incorrect:

suggestion: logger.Sugar().Errorf(Errorfmt, zap.Error(err))

The rule works if the $fmt variable is replaced with a variable that has one character, such as $y:

	m.Match(`$x.$f($y, $e)`).
		Suggest(`$x.$f($y, zap.Error($e))`)

In that case the suggestion will be correct:

suggestion: logger.Sugar().Errorf("error: %v", zap.Error(err))

[quasilyte]

Interesting! I though that I fixed this in #123.
Have you tried the latest version?
I probably need to make a new release and attach compiled binaries there...

[sebastien-rosset]

Ah yes, it's fixed in the latest version of ruleguard. I think it's because I was using golangci, which hadn't bumped the version to the latest. I am closing this issue.