Multiple conflicting refactoring regressions?

[diceroll123]

Issue description or question

I'll let the code and comments do the talking!

def blah():
    num = "1"
    name = "hello"
    thestring = str(num) + "_" + name
    # sourcery wants to change this to
    # thestring = f"{num}_{name}"
    # this is correct!
    # (I expect this exact output from the below methods as well)
    print(thestring)

def blah1():
    num = 1
    name = "hello"
    thestring = str(num) + "_" + name
    # sourcery wants to change this to
    # thestring = f"{1}_" + "hello"
    print(thestring)

def blah2():
    num = random.randint(0, 100)
    name = "hello"
    thestring = str(num) + "_" + name
    # sourcery wants to change this to
    # thestring = f"{str(num)}_{name}"
    print(thestring)

Thanks for readin'.

Sourcery Version

v0.11.5

Code editor or IDE name and version

VSCode Version: 1.67.2 (user setup) Commit: c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5 Date: 2022-05-17T18:15:52.058Z Electron: 17.4.1 Chromium: 98.0.4758.141 Node.js: 16.13.0 V8: 9.8.177.13-electron.0 OS: Windows_NT x64 10.0.25120

[ruancomelli]

Hello again, @diceroll123!

You are correct, Sourcery should suggest the same thing in blah1 and blah2.

Thanks for raising this, we'll look into it!

[brendanator]

I've identified a fix which will be included in the next release

[Hellebore]

Resolved with release of 0.12.1