forked from gforcada/flake8-builtins
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathflake8_builtins.py
246 lines (204 loc) · 8.56 KB
/
flake8_builtins.py
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
from flake8 import utils as stdin_utils
import ast
import builtins
import inspect
class BuiltinsChecker:
name = 'flake8_builtins'
version = '1.5.2'
assign_msg = 'A001 variable "{0}" is shadowing a Python builtin'
argument_msg = 'A002 argument "{0}" is shadowing a Python builtin'
class_attribute_msg = 'A003 class attribute "{0}" is shadowing a Python builtin'
import_msg = 'A004 import statement "{0}" is shadowing a Python builtin'
names = []
ignore_list = {
'__name__',
'__doc__',
'credits',
'_',
}
def __init__(self, tree, filename):
self.tree = tree
self.filename = filename
@classmethod
def add_options(cls, option_manager):
option_manager.add_option(
'--builtins-ignorelist',
metavar='builtins',
parse_from_config=True,
comma_separated_list=True,
help='A comma separated list of builtins to skip checking',
)
@classmethod
def parse_options(cls, options):
if options.builtins_ignorelist is not None:
cls.ignore_list.update(options.builtins_ignorelist)
cls.names = {
a[0] for a in inspect.getmembers(builtins) if a[0] not in cls.ignore_list
}
flake8_builtins = getattr(options, 'builtins', None)
if flake8_builtins:
cls.names.update(flake8_builtins)
def run(self):
tree = self.tree
if self.filename == 'stdin':
lines = stdin_utils.stdin_get_value()
tree = ast.parse(lines)
for statement in ast.walk(tree):
for child in ast.iter_child_nodes(statement):
child.__flake8_builtins_parent = statement
function_nodes = [ast.FunctionDef]
if getattr(ast, 'AsyncFunctionDef', None):
function_nodes.append(ast.AsyncFunctionDef)
function_nodes = tuple(function_nodes)
for_nodes = [ast.For]
if getattr(ast, 'AsyncFor', None):
for_nodes.append(ast.AsyncFor)
for_nodes = tuple(for_nodes)
with_nodes = [ast.With]
if getattr(ast, 'AsyncWith', None):
with_nodes.append(ast.AsyncWith)
with_nodes = tuple(with_nodes)
comprehension_nodes = (
ast.ListComp,
ast.SetComp,
ast.DictComp,
ast.GeneratorExp,
)
value = None
for statement in ast.walk(tree):
if isinstance(statement, (ast.Assign, ast.AnnAssign, ast.NamedExpr)):
value = self.check_assignment(statement)
elif isinstance(statement, function_nodes):
value = self.check_function_definition(statement)
elif isinstance(statement, for_nodes):
value = self.check_for_loop(statement)
elif isinstance(statement, with_nodes):
value = self.check_with(statement)
elif isinstance(statement, ast.excepthandler):
value = self.check_exception(statement)
elif isinstance(statement, comprehension_nodes):
value = self.check_comprehension(statement)
elif isinstance(statement, (ast.Import, ast.ImportFrom)):
value = self.check_import(statement)
elif isinstance(statement, ast.ClassDef):
value = self.check_class(statement)
if value:
yield from value
def check_assignment(self, statement):
msg = self.assign_msg
if type(statement.__flake8_builtins_parent) is ast.ClassDef:
msg = self.class_attribute_msg
if isinstance(statement, ast.Assign):
stack = list(statement.targets)
else: # This is `ast.AnnAssign` or `ast.NamedExpr`:
stack = [statement.target]
while stack:
item = stack.pop()
if isinstance(item, (ast.Tuple, ast.List)):
stack.extend(list(item.elts))
elif isinstance(item, ast.Name) and item.id in self.names:
yield self.error(item, message=msg, variable=item.id)
elif isinstance(item, ast.Starred):
if hasattr(item.value, 'id') and item.value.id in self.names:
yield self.error(
statement,
message=msg,
variable=item.value.id,
)
elif hasattr(item.value, 'elts'):
stack.extend(list(item.value.elts))
def check_function_definition(self, statement):
if statement.name in self.names:
msg = self.assign_msg
if type(statement.__flake8_builtins_parent) is ast.ClassDef:
msg = self.class_attribute_msg
yield self.error(statement, message=msg, variable=statement.name)
all_arguments = []
all_arguments.extend(statement.args.args)
all_arguments.extend(getattr(statement.args, 'kwonlyargs', []))
all_arguments.extend(getattr(statement.args, 'posonlyargs', []))
for arg in all_arguments:
if isinstance(arg, ast.arg) and arg.arg in self.names:
yield self.error(
arg,
message=self.argument_msg,
variable=arg.arg,
)
def check_for_loop(self, statement):
stack = [statement.target]
while stack:
item = stack.pop()
if isinstance(item, (ast.Tuple, ast.List)):
stack.extend(list(item.elts))
elif isinstance(item, ast.Name) and item.id in self.names:
yield self.error(statement, variable=item.id)
elif isinstance(item, ast.Starred):
if hasattr(item.value, 'id') and item.value.id in self.names:
yield self.error(
statement,
variable=item.value.id,
)
elif hasattr(item.value, 'elts'):
stack.extend(list(item.value.elts))
def check_with(self, statement):
for item in statement.items:
var = item.optional_vars
if isinstance(var, (ast.Tuple, ast.List)):
for element in var.elts:
if isinstance(element, ast.Name) and element.id in self.names:
yield self.error(statement, variable=element.id)
elif (
isinstance(element, ast.Starred)
and element.value.id in self.names
):
yield self.error(
element,
variable=element.value.id,
)
elif isinstance(var, ast.Name) and var.id in self.names:
yield self.error(statement, variable=var.id)
def check_exception(self, statement):
exception_name = statement.name
if exception_name is None:
return
if exception_name in self.names:
yield self.error(statement, variable=exception_name)
def check_comprehension(self, statement):
for generator in statement.generators:
if (
isinstance(generator.target, ast.Name)
and generator.target.id in self.names
):
yield self.error(statement, variable=generator.target.id)
elif isinstance(generator.target, (ast.Tuple, ast.List)):
for tuple_element in generator.target.elts:
if (
isinstance(tuple_element, ast.Name)
and tuple_element.id in self.names
):
yield self.error(statement, variable=tuple_element.id)
def check_import(self, statement):
for name in statement.names:
collision = None
if name.name in self.names and name.asname is None:
collision = name.name
elif name.asname in self.names:
collision = name.asname
if collision:
yield self.error(
statement,
message=self.import_msg,
variable=collision,
)
def check_class(self, statement):
if statement.name in self.names:
yield self.error(statement, variable=statement.name)
def error(self, statement, variable, message=None):
if not message:
message = self.assign_msg
return (
statement.lineno,
statement.col_offset,
message.format(variable),
type(self),
)