LowFile-scopeC# Plug & play

Centralize string constants

Define shared string literals (route names, keys, claim types) as constants/enums in a single place.

Why this matters

Prevents drift and typos across the codebase.

Side-by-side examples engineers can pattern-match during review.

Avoid

if (role == "AdminRoleV2") { /* ... */ }

Prefer

public static class Roles { public const string Admin = "AdminRoleV2"; }
if (role == Roles.Admin) { /* ... */ }

Bad

if(type=="A"){}

Good

if(type==Types.Admin){}

From the same buckets as this rule.

Critical
Avoid Duplicating String Literals
Store repeated string literals in constants or variables to improve maintainability and reduce redundancy.
duplication-complexitymaintainability
Critical
Avoid Redundant `None` Comparisons
Identify cases where `x is None` or `x is not None` is used redundantly when `x` is always or never `None`. This may indicate a deeper logic issue.
duplication-complexityreadability-refactor+1
High
Avoid Functions That Always Return the Same Value
Identify functions that return the same value regardless of input. This may indicate poor design or a logic error.
duplication-complexityreadability-refactor+1