In config.py, get_allow_recursion() has a comment that contradicts its behavior:
if val is None:
return True # Default to False for safety
The comment says "Default to False for safety" but the function returns True when no config value is set.
Two possible fixes depending on intent:
- If
True is correct: update the comment to match
- If
False was intended: update the return value to match
Flagging for clarification before touching either.
In
config.py,get_allow_recursion()has a comment that contradicts its behavior:The comment says "Default to False for safety" but the function returns
Truewhen no config value is set.Two possible fixes depending on intent:
Trueis correct: update the comment to matchFalsewas intended: update the return value to matchFlagging for clarification before touching either.