Python cleanup suite

Usage

This recipe has no required configuration options. You’ll need the Moderne CLI configured before running the command below.

mod run . --recipe org.openrewrite.python.cleanup.PythonBestPractices

If the recipe isn’t available locally, install it with:

mod config recipes pip install openrewrite-static-analysis

Definition

This recipe runs the following recipes in order.

• Use {} literal instead of dict() constructororg.openrewrite.python.cleanup.DictLiteral
• Use [] literal instead of list() constructororg.openrewrite.python.cleanup.ListLiteral
• Use () literal instead of tuple() constructororg.openrewrite.python.cleanup.TupleLiteral
• Prefer set literals in in membership testsorg.openrewrite.python.cleanup.CollectionIntoSet
• Flatten redundant collection constructor wrapping a literalorg.openrewrite.python.cleanup.UnwrapIterableConstruction
• Drop unnecessary 0 start argument from range()org.openrewrite.python.cleanup.RemoveZeroFromRange
• Drop unnecessary step 1 argument from range()org.openrewrite.python.cleanup.RemoveUnitStepFromRange
• Compare to None with identity operators (is / is not)org.openrewrite.python.cleanup.NoneCompare
• Fold same-literal ==/!= comparisons to boolean constantsorg.openrewrite.python.cleanup.EqualityIdentity
• Collapse self-cancelling ^ / - with duplicate operands to 0org.openrewrite.python.cleanup.BinOpIdentity
• Rewrite self-multiplication as ** 2org.openrewrite.python.cleanup.SquareIdentity
• Remove explicit True/False comparisonsorg.openrewrite.python.cleanup.SimplifyBooleanComparison
• Collapse boolean ternary to bare conditionorg.openrewrite.python.cleanup.BooleanIfExpIdentity
• Eliminate boolean literal from and/ororg.openrewrite.python.cleanup.RemoveRedundantBoolean
• Replace len() emptiness check with truthinessorg.openrewrite.python.cleanup.SimplifyLenComparison
• Compare string to "" instead of checking len()org.openrewrite.python.cleanup.SimplifyStrLenComparison
• Substitute constant collection condition with booleanorg.openrewrite.python.cleanup.CollectionToBool
• Flatten negated logic via De Morgan's identitiesorg.openrewrite.python.cleanup.DeMorgan
• Reorder comparisons to put literals on the rightorg.openrewrite.python.cleanup.FlipComparison
• Replace .find() check with in / not inorg.openrewrite.python.cleanup.SimplifySubstringSearch
• Consolidate repeated == with or into inorg.openrewrite.python.cleanup.MergeComparisons
• Merge isinstance() callsorg.openrewrite.python.cleanup.MergeIsinstance
• Replace self-referencing ternary with ororg.openrewrite.python.cleanup.OrIfExpIdentity
• Swap ternary branches to drop negated conditionorg.openrewrite.python.cleanup.SwapIfExpression
• Shorten assignment to compound operator formorg.openrewrite.python.cleanup.AugAssign
• Remove redundant ternary conditionorg.openrewrite.python.cleanup.RemoveRedundantCondition
• Collapse nested `if into a single and` conditionorg.openrewrite.python.cleanup.MergeNestedIfs
• Convert `else: if to eliforg.openrewrite.python.cleanup.MergeElseIfIntoElif
• Flip empty `if`-body by negating the conditionorg.openrewrite.python.cleanup.SwapIfElseBranches
• Drop `pass-only elif` by negating its conditionorg.openrewrite.python.cleanup.RemovePassElif
• Simplify negated `elif to elseorg.openrewrite.python.cleanup.RemoveRedundantIf
• Drop `else after early-exit if` branchorg.openrewrite.python.cleanup.RemoveUnnecessaryElse
• Use inline conditional for simple `if/else` assignmentorg.openrewrite.python.cleanup.AssignIfExp
• Remove redundant pass statementsorg.openrewrite.python.RemovePass
• Strip trailing `continue` from loop bodyorg.openrewrite.python.cleanup.RemoveRedundantContinue
• Drop `exists() check before is_dir()/is_file()org.openrewrite.python.cleanup.RemoveRedundantPathExists
• Delete no-op assert True statementsorg.openrewrite.python.cleanup.RemoveAssertTrue
• Strip dead code after terminal statementsorg.openrewrite.python.cleanup.RemoveUnreachableCode
• Delete if blocks whose body is only passorg.openrewrite.python.cleanup.RemoveEmptyNestedBlock
• Unwrap `str() from print()` argumentsorg.openrewrite.python.cleanup.RemoveStrFromPrint
• Remove redundant None default from dict.get()org.openrewrite.python.cleanup.RemoveNoneFromDefaultGet
• Drop `f` prefix from strings without placeholdersorg.openrewrite.python.cleanup.RemoveRedundantFstring
• Strip `str()` from f-string placeholdersorg.openrewrite.python.cleanup.RemoveStrFromFstring
• Prefer `startswith/endswith` over slice comparisonorg.openrewrite.python.cleanup.StrPrefixSuffix
• Use bracket access for `re.Match` groupsorg.openrewrite.python.cleanup.UseGetitemForReMatchGroups
• Fold constants and flatten nested f-stringsorg.openrewrite.python.cleanup.SimplifyFstringFormatting
• Replace string slicing with removeprefix/removesuffixorg.openrewrite.python.cleanup.UseStringRemoveAffix
• Drop redundant .keys() on dict iterationorg.openrewrite.python.cleanup.RemoveDictKeys
• Convert one-item dict.update() to bracket assignmentorg.openrewrite.python.cleanup.SimplifyDictionaryUpdate
• Use dict union operator instead of double-star unpackingorg.openrewrite.python.cleanup.UseDictionaryUnion
• Unwrap unnecessary dict() from union operandsorg.openrewrite.python.cleanup.RemoveRedundantConstructorInDictUnion
• Deduplicate repeated keys in dict literalsorg.openrewrite.python.cleanup.RemoveDuplicateDictKey
• Deduplicate repeated elements in set literalsorg.openrewrite.python.cleanup.RemoveDuplicateSetKey
• Iterate over file objects directly, not via readlines()org.openrewrite.python.cleanup.UseFileIterator
• Flatten for/else when the loop has no breakorg.openrewrite.python.cleanup.UselessElseOnLoop
• Collapse for-yield loop into yield fromorg.openrewrite.python.cleanup.YieldFrom
• Use comprehension syntax instead of list()/set() around generatorsorg.openrewrite.python.cleanup.CollectionBuiltinToComprehension
• Pass iterable directly to any()/all() instead of identity generatororg.openrewrite.python.cleanup.SimplifyGenerator
• Use generator expression instead of list comprehension in iterable-accepting callsorg.openrewrite.python.cleanup.ComprehensionToGenerator
• Simplify identity comprehension to list()/set() callorg.openrewrite.python.cleanup.IdentityComprehension
• Swap not all()/not any() by negating the comparisonorg.openrewrite.python.cleanup.InvertAnyAll
• Apply De Morgan's law to any(not ...)/all(not ...)org.openrewrite.python.cleanup.InvertAnyAllBody
• Rewrite any(v == literal ...) as literal in collectionorg.openrewrite.python.cleanup.ConvertAnyToIn
• Simplify sum(1 for x in items if cond) to sum(bool(cond) for x in items)org.openrewrite.python.cleanup.SimplifyConstantSum
• Narrow bare except: to except Exception:org.openrewrite.python.cleanup.DoNotUseBareExcept
• Unwrap one-element exception tuple in exceptorg.openrewrite.python.cleanup.SimplifySingleExceptionTuple
• Chain exceptions with raise ... from in except blocksorg.openrewrite.python.cleanup.RaiseFromPreviousError
• Replace try/except: pass with contextlib.suppress()org.openrewrite.python.cleanup.UseContextlibSuppress
• Standardize @classmethod first parameter to clsorg.openrewrite.python.cleanup.ClassMethodFirstArgName
• Eliminate inplace=True in favor of reassignmentorg.openrewrite.python.cleanup.PandasAvoidInplace
• Convert apply('name') to a direct method invocationorg.openrewrite.python.cleanup.ReplaceApplyWithMethodCall
• Use .isna() instead of == np.nan comparisonsorg.openrewrite.python.cleanup.UseIsna
• Convert and/or ternary trick to conditional expressionorg.openrewrite.python.cleanup.TernaryToIfExpression
• Guard mutable default arguments with None sentinelorg.openrewrite.python.cleanup.DefaultMutableArg
• Simplify temp-variable swap to tuple unpackingorg.openrewrite.python.cleanup.SwapVariable
• Remove break/continue outside looporg.openrewrite.python.cleanup.BreakOrContinueOutsideLoop
• Remove return/yield outside functionorg.openrewrite.python.cleanup.ReturnOrYieldOutsideFunction
• Use datetime.now() instead of datetime.today()org.openrewrite.python.cleanup.UseDatetimeNowNotToday