[pylint] Preserve original value format (PLR6104)

[InSyncWithFoo]

Summary

Resolves #11672.

Test Plan

cargo nextest run and cargo insta test.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

[InSyncWithFoo]

The existing implementation is surprisingly complex. On top of that, it uses the generator to create the fix, but that's not strictly necessary. String slicing is much simpler and only requires an extra hardcoded if for the Expr::Named case.

[MichaReiser]

Thanks. Could you revert your changes to AugmentedOperator (and, if there are any, other refactors that aren't strictly necessary for the fix). I'd prefer to keep the changes minimal to the fix or have separate PRs for refactor and fix.

[InSyncWithFoo]

I'd like to believe it isn't my fault that the mdformat check failed. A simple uvx pre-commit run --all-files commit should fix it, but I won't submit a PR as I'm not interested in finding the cause.

[AlexWaygood]

I'd like to believe it isn't my fault that the mdformat check failed. A simple uvx pre-commit run --all-files commit should fix it, but I won't submit a PR as I'm not interested in finding the cause.

I believe it's due to one of the mdformat plugins we use having released a new version. We don't have these pinned in our pre-commit config file; we probably should.

Two of the plugins we use cut new releases a couple of hours ago, one of them a major version bump:

• KyleKing/mdformat-mkdocs@ce67108
• KyleKing/mdformat-admon@f6f2c23

[AlexWaygood]

I filed #14992 to fix the pre-commit issue.

[MichaReiser]

Thanks