blob: 7f732a823b954f261a56a9576e5409c6f0d23536 [file] [log] [blame] [edit]
#!/usr/bin/env python3
# Copyright (C) 2023 The Android Open Source Project
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
"""
Enforce import rules for https://ui.perfetto.dev.
Directory structure encodes ideas about the expected dependency graph
of the code in those directories. Both in a fuzzy sense: we expect code
withing a directory to have high cohesion within the directory and low
coupling (aka fewer imports) outside of the directory - but also
concrete rules:
- "base should not depend on the fronted"
- "plugins should only directly depend on the public API"
- "we should not have circular dependencies"
Without enforcement exceptions to this rule quickly slip in. This
script allows such rules to be enforced at presubmit time.
"""
import sys
import os
import re
import collections
import argparse
ROOT_DIR = os.path.dirname(
os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
UI_SRC_DIR = os.path.join(ROOT_DIR, 'ui', 'src')
# Current plan for the dependency tree of the UI code (2023-09-21)
# black = current
# red = planning to remove
# green = planning to add
PLAN_DOT = """
digraph g {
mithril [shape=rectangle, label="mithril"];
protos [shape=rectangle, label="//protos/perfetto"];
_gen [shape=ellipse, label="/gen"];
_base [shape=ellipse, label="/base"];
_core [shape=ellipse, label="/core"];
_engine [shape=ellipse, label="/engine"];
_frontend [shape=ellipse, label="/frontend" color=red];
_common [shape=ellipse, label="/common" color=red];
_controller [shape=ellipse, label="/controller" color=red];
_tracks [shape=ellipse, label="/tracks" color=red];
_widgets [shape=ellipse, label="/widgets"];
_public [shape=ellipse, label="/public"];
_plugins [shape=ellipse, label="/plugins"];
_chrome_extension [shape=ellipse, label="/chrome_extension"];
_trace_processor [shape=ellipse, label="/trace_processor" color="green"];
_protos [shape=ellipse, label="/protos"];
engine_worker_bundle [shape=cds, label="Engine worker bundle"];
frontend_bundle [shape=cds, label="Frontend bundle"];
engine_worker_bundle -> _engine;
frontend_bundle -> _core [color=green];
frontend_bundle -> _frontend [color=red];
_core -> _public;
_plugins -> _public;
_widgets -> _base;
_core -> _base;
_core -> _widgets;
_widgets -> mithril;
_plugins -> mithril;
_core -> mithril
_plugins -> _widgets;
_core -> _chrome_extension;
_frontend -> _widgets [color=red];
_common -> _core [color=red];
_frontend -> _core [color=red];
_controller -> _core [color=red];
_frontend -> _controller [color=red];
_frontend -> _common [color=red];
_controller -> _frontend [color=red];
_controller -> _common [color=red];
_common -> _controller [color=red];
_common -> _frontend [color=red];
_tracks -> _frontend [color=red];
_tracks -> _controller [color=red];
_common -> _chrome_extension [color=red];
_core -> _trace_processor [color=green];
_engine -> _trace_processor [color=green];
_engine -> _common [color=red];
_engine -> _base;
_gen -> protos;
_core -> _gen [color=red];
_core -> _protos;
_protos -> _gen;
_trace_processor -> _protos [color=green];
_trace_processor -> _public [color=green];
npm_trace_processor [shape=cds, label="npm trace_processor" color="green"];
npm_trace_processor -> engine_worker_bundle [color="green"];
npm_trace_processor -> _trace_processor [color="green"];
}
"""
class Failure(object):
def __init__(self, path, rule):
self.path = path
self.rule = rule
def __str__(self):
nice_path = ["ui/src" + name + ".ts" for name in self.path]
return ''.join([
'Forbidden dependency path:\n\n ',
'\n -> '.join(nice_path),
'\n',
'\n',
str(self.rule),
'\n',
])
class AllowList(object):
def __init__(self, allowed, dst, reasoning):
self.allowed = allowed
self.dst = dst
self.reasoning = reasoning
def check(self, graph):
for node, edges in graph.items():
for edge in edges:
if re.match(self.dst, edge):
if not any(re.match(a, node) for a in self.allowed):
yield Failure([node, edge], self)
def __str__(self):
return f'Only items in the allowlist ({self.allowed}) may directly depend on "{self.dst}" ' + self.reasoning
class NoDirectDep(object):
def __init__(self, src, dst, reasoning):
self.src = src
self.dst = dst
self.reasoning = reasoning
def check(self, graph):
for node, edges in graph.items():
if re.match(self.src, node):
for edge in edges:
if re.match(self.dst, edge):
yield Failure([node, edge], self)
def __str__(self):
return f'"{self.src}" may not directly depend on "{self.dst}" ' + self.reasoning
class NoDep(object):
def __init__(self, src, dst, reasoning):
self.src = src
self.dst = dst
self.reasoning = reasoning
def check(self, graph):
for node in graph:
if re.match(self.src, node):
for connected, path in bfs(graph, node):
if re.match(self.dst, connected):
yield Failure(path, self)
def __str__(self):
return f'"{self.src}" may not depend on "{self.dst}" ' + self.reasoning
class NoCircularDeps(object):
def __init__(self):
pass
def check(self, graph):
for node in graph:
for child in graph[node]:
for reached, path in dfs(graph, child):
if reached == node:
yield Failure([node] + path, self)
def __str__(self):
return f'circular dependencies can cause complex issues'
# We have three kinds of rules:
# NoDirectDep(a, b) = files matching regex 'a' cannot *directly* import
# files matching regex 'b' - but they may indirectly depend on them.
# NoDep(a, b) = as above but 'a' may not even transitively import 'b'.
# NoCircularDeps = forbid introduction of circular dependencies
RULES = [
AllowList(
['/protos/index'],
r'/gen/protos',
'protos should be re-exported from /protos/index without the nesting.',
),
NoDirectDep(
r'/plugins/.*',
r'/core/.*',
'instead plugins should depend on the API exposed at ui/src/public.',
),
NoDirectDep(
r"/frontend/.*",
r"/core_plugins/.*",
"core code should not depend on plugins.",
),
NoDirectDep(
r"/core/.*",
r"/core_plugins/.*",
"core code should not depend on plugins.",
),
NoDirectDep(
r"/base/.*",
r"/core_plugins/.*",
"core code should not depend on plugins.",
),
#NoDirectDep(
# r'/tracks/.*',
# r'/core/.*',
# 'instead tracks should depend on the API exposed at ui/src/public.',
#),
NoDep(
r'/core/.*',
r'/plugins/.*',
'otherwise the plugins are no longer optional.',
),
NoDep(
r'/core/.*',
r'/frontend/.*',
'trying to reduce the dependency mess as we refactor into core',
),
NoDep(
r'/core/.*',
r'/common/.*',
'trying to reduce the dependency mess as we refactor into core',
),
NoDep(
r'/core/.*',
r'/controller/.*',
'trying to reduce the dependency mess as we refactor into core',
),
NoDep(
r'/base/.*',
r'/core/.*',
'core should depend on base not the other way round',
),
NoDep(
r'/base/.*',
r'/common/.*',
'common should depend on base not the other way round',
),
NoDep(
r'/common/.*',
r'/chrome_extension/.*',
'chrome_extension must be a leaf',
),
# Widgets
NoDep(
r'/widgets/.*',
r'/frontend/.*',
'widgets should only depend on base',
),
NoDep(
r'/widgets/.*',
r'/core/.*',
'widgets should only depend on base',
),
NoDep(
r'/widgets/.*',
r'/plugins/.*',
'widgets should only depend on base',
),
NoDep(
r'/widgets/.*',
r'/common/.*',
'widgets should only depend on base',
),
# Bigtrace
NoDep(
r'/bigtrace/.*',
r'/frontend/.*',
'bigtrace should not depend on frontend',
),
NoDep(
r'/bigtrace/.*',
r'/common/.*',
'bigtrace should not depend on common',
),
NoDep(
r'/bigtrace/.*',
r'/engine/.*',
'bigtrace should not depend on engine',
),
NoDep(
r'/bigtrace/.*',
r'/trace_processor/.*',
'bigtrace should not depend on trace_processor',
),
NoDep(
r'/bigtrace/.*',
r'/traceconv/.*',
'bigtrace should not depend on traceconv',
),
NoDep(
r'/bigtrace/.*',
r'/tracks/.*',
'bigtrace should not depend on tracks',
),
NoDep(
r'/bigtrace/.*',
r'/controller/.*',
'bigtrace should not depend on controller',
),
# Fails at the moment as we have several circular dependencies. One
# example:
# ui/src/frontend/cookie_consent.ts
# -> ui/src/frontend/globals.ts
# -> ui/src/frontend/router.ts
# -> ui/src/frontend/pages.ts
# -> ui/src/frontend/cookie_consent.ts
#NoCircularDeps(),
]
def all_source_files():
for root, dirs, files in os.walk(UI_SRC_DIR, followlinks=False):
for name in files:
if name.endswith('.ts') and not name.endswith('.d.ts'):
yield os.path.join(root, name)
def is_dir(path, cache={}):
try:
return cache[path]
except KeyError:
result = cache[path] = os.path.isdir(path)
return result
def remove_prefix(s, prefix):
return s[len(prefix):] if s.startswith(prefix) else s
def remove_suffix(s, suffix):
return s[:-len(suffix)] if s.endswith(suffix) else s
def find_imports(path):
src = path
src = remove_prefix(src, UI_SRC_DIR)
src = remove_suffix(src, '.ts')
directory, _ = os.path.split(src)
with open(path) as f:
s = f.read()
for m in re.finditer("^import[^']*'([^']*)';", s, flags=re.MULTILINE):
raw_target = m[1]
if raw_target.startswith('.'):
target = os.path.normpath(os.path.join(directory, raw_target))
if is_dir(UI_SRC_DIR + target):
target = os.path.join(target, 'index')
else:
target = raw_target
yield (src, target)
def path_to_id(path):
path = path.replace('/', '_')
path = path.replace('-', '_')
path = path.replace('@', '_at_')
path = path.replace('.', '_')
return path
def is_external_dep(path):
return not path.startswith('/')
def bfs(graph, src):
seen = set()
queue = [(src, [])]
while queue:
node, path = queue.pop(0)
if node in seen:
continue
seen.add(node)
path = path[:]
path.append(node)
yield node, path
queue.extend([(child, path) for child in graph[node]])
def dfs(graph, src):
seen = set()
queue = [(src, [])]
while queue:
node, path = queue.pop()
if node in seen:
continue
seen.add(node)
path = path[:]
path.append(node)
yield node, path
queue.extend([(child, path) for child in graph[node]])
def write_dot(graph, f):
print('digraph g {', file=f)
for node, edges in graph.items():
node_id = path_to_id(node)
shape = 'rectangle' if is_external_dep(node) else 'ellipse'
print(f'{node_id} [shape={shape}, label="{node}"];', file=f)
for edge in edges:
edge_id = path_to_id(edge)
print(f'{node_id} -> {edge_id};', file=f)
print('}', file=f)
def do_check(options, graph):
for rule in RULES:
for failure in rule.check(graph):
print(failure)
return 1
return 0
def do_desc(options, graph):
print('Rules:')
for rule in RULES:
print(" - ", end='')
print(rule)
def do_print(options, graph):
for node, edges in graph.items():
for edge in edges:
print("{}\t{}".format(node, edge))
def do_dot(options, graph):
def simplify(path):
if is_external_dep(path):
return path
return os.path.dirname(path)
if options.simplify:
new_graph = collections.defaultdict(set)
for node, edges in graph.items():
for edge in edges:
new_graph[simplify(edge)]
new_graph[simplify(node)].add(simplify(edge))
graph = new_graph
if options.ignore_external:
new_graph = collections.defaultdict(set)
for node, edges in graph.items():
if is_external_dep(node):
continue
for edge in edges:
if is_external_dep(edge):
continue
new_graph[edge]
new_graph[node].add(edge)
graph = new_graph
write_dot(graph, sys.stdout)
return 0
def do_plan_dot(options, _):
print(PLAN_DOT, file=sys.stdout)
return 0
def main():
parser = argparse.ArgumentParser(description=__doc__)
parser.set_defaults(func=do_check)
subparsers = parser.add_subparsers()
check_command = subparsers.add_parser(
'check', help='Check the rules (default)')
check_command.set_defaults(func=do_check)
desc_command = subparsers.add_parser('desc', help='Print the rules')
desc_command.set_defaults(func=do_desc)
print_command = subparsers.add_parser('print', help='Print all imports')
print_command.set_defaults(func=do_print)
dot_command = subparsers.add_parser(
'dot',
help='Output dependency graph in dot format suitble for use in graphviz (e.g. ./tools/check_imports dot | dot -Tpng -ograph.png)'
)
dot_command.set_defaults(func=do_dot)
dot_command.add_argument(
'--simplify',
action='store_true',
help='Show directories rather than files',
)
dot_command.add_argument(
'--ignore-external',
action='store_true',
help='Don\'t show external dependencies',
)
plan_dot_command = subparsers.add_parser(
'plan-dot',
help='Output planned dependency graph in dot format suitble for use in graphviz (e.g. ./tools/check_imports plan-dot | dot -Tpng -ograph.png)'
)
plan_dot_command.set_defaults(func=do_plan_dot)
graph = collections.defaultdict(set)
for path in all_source_files():
for src, target in find_imports(path):
graph[src].add(target)
graph[target]
options = parser.parse_args()
return options.func(options, graph)
if __name__ == '__main__':
sys.exit(main())