Install
openclaw skills install python-mutable-default-argsA Python function uses a mutable object (list, dict, set) as a default argument, sharing state across calls in a way that produces silent bugs.
openclaw skills install python-mutable-default-argsPython evaluates default argument values once at function definition time, not on each call. If the default is a mutable object — a list, dict, or set — that object is shared across every call that uses the default. Mutating it inside the function modifies the default for all future calls. The bug is invisible until the second or later call and produces state-dependent failures that are hard to trace.
def add_item(item, items=[]): # ← shared list
items.append(item)
return items
add_item("a") # ["a"]
add_item("b") # ["a", "b"] ← unexpected; second caller sees first caller's data
Use None as the default and initialize the mutable inside the function body:
def add_item(item, items=None):
if items is None:
items = []
items.append(item)
return items
This is the canonical Python idiom. Every call that omits items gets a fresh list.
Applies equally to dict, set, and any other mutable type. The rule is: never use a mutable as a default argument.
When reviewing code, scan every function signature for =[], ={}, =set() as a heuristic for this pattern.
Linters (pylint rule W0102, ruff rule B006) will flag this automatically — enable them if the codebase doesn't already.