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
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
|
"""Triskel-backed layout: SESE decomposition, in character cells.
`triskel <https://github.com/triskellib/triskel>`_ lays a CFG out by splitting it
into Single-Entry Single-Exit regions first, laying each region out on its own,
and pasting the results back as super-nodes. On our corpus that takes functions
that our own layered engine draws with up to 41 edge crossings down to 0 or 1,
and it routes loop edges around the side of the graph the way IDA does instead
of straight back up the middle. ``docs/TRISKEL_EVAL.md`` has the measurements.
This module owns the whole impedance mismatch between a float/pixel layout
engine and a grid of character cells. Three things make that mismatch small:
1. **We work in cells, not pixels.** Our fork exposes ``set_spacing()``, so the
gutters and the edge-lane pitch are set in cells (3 / 1 / 1) and node sizes
are handed over in cells. Upstream's constants are pixels (50 / 40 / 30);
feeding those a 32px-tall cell rounds two adjacent edge lanes onto the same
row, which in a terminal means two differently-coloured edges fighting over
one cell. In cell units the output is integral and lanes never collide.
2. **Triskel's routes are already orthogonal.** Zero diagonal segments out of
2471 on the corpus, so every segment is a run of ``─`` or ``│``.
3. **Ports already land on the box border**, spread along it by degree, which is
exactly what our own ``_ports`` does.
What it does NOT do is trust the library with degenerate input. Self-loops and
disconnected graphs make it throw, an empty graph used to segfault, and a
segfault takes the TUI down with it. Both are handled here, before the call.
"""
from __future__ import annotations
import os
from . import graph as G
# Spacing, in cells. X_GUTTER is the gap between boxes in a layer, Y_GUTTER the
# gap between a box and the first edge lane, EDGE_HEIGHT the pitch between
# stacked horizontal edge runs -- so EDGE_HEIGHT >= 1 is what guarantees two
# lanes never share a row.
HGAP = 3
VGAP = 1
LANE = 1
#: Columns between two weakly-connected components laid out side by side.
COMPONENT_GAP = 4
_mod: object | None = None
_tried = False
def module():
"""The ``pytriskel`` extension, or None. Imported lazily and cached.
``$IDATUI_TRISKEL_PATH`` points at a build tree (our fork's
``build/bindings/python``) for development installs.
"""
global _mod, _tried
if _tried:
return _mod
_tried = True
path = os.environ.get("IDATUI_TRISKEL_PATH")
if path:
import sys
if path not in sys.path:
sys.path.insert(0, path)
try:
import pytriskel # noqa: PLC0415
except ImportError:
return None
# Upstream ships wheels whose get_waypoints() always throws (a missing
# <pybind11/stl.h>), and without waypoints there are no edges to draw. Fail
# the availability check rather than dying mid-layout.
if not hasattr(pytriskel, "set_spacing"):
return None
_mod = pytriskel
return _mod
def available() -> bool:
return module() is not None
def _components(g: G._Graph) -> list[list[int]]:
"""Weakly-connected components, entry's component first.
Triskel raises ``EMPTY BL`` on a disconnected graph (its cycle-equivalence
bracket lists run dry), and IDA flowcharts do contain unreachable blocks --
``tests/test_graph.py:t_unreachable`` is exactly that shape. So we split the
work ourselves and stack the pieces side by side, which is also what the
native engine effectively does.
"""
adj: dict[int, set[int]] = {i: set() for i in g.nodes}
for e in g.edges:
adj[e.src].add(e.dst)
adj[e.dst].add(e.src)
seen: set[int] = set()
comps: list[list[int]] = []
for start in g.nodes:
if start in seen:
continue
stack, comp = [start], []
seen.add(start)
while stack:
i = stack.pop()
comp.append(i)
for j in adj[i]:
if j not in seen:
seen.add(j)
stack.append(j)
comps.append(sorted(comp))
return comps
def _edge_type(pt, kind: str):
if kind == G.E_TRUE:
return pt.EdgeType.T
if kind == G.E_FALSE:
return pt.EdgeType.F
return pt.EdgeType.Default
def _clean(pts: list[tuple[int, int]]) -> list[tuple[int, int]]:
"""Drop duplicate and collinear waypoints.
Triskel emits doubled points where it stitches region layouts together (a
back edge came back with 20 waypoints, 6 of them duplicates). A doubled
point is a zero-length segment, which would make the corner-glyph pass read
a direction of "nowhere".
"""
out: list[tuple[int, int]] = []
for p in pts:
if out and out[-1] == p:
continue
out.append(p)
i = 1
while i < len(out) - 1:
a, b, c = out[i - 1], out[i], out[i + 1]
if (a[0] == b[0] == c[0]) or (a[1] == b[1] == c[1]):
del out[i]
else:
i += 1
return out
def run(g: G._Graph, root: int) -> tuple[list[G.Route], int]:
"""Position every node in ``g`` and return (routes, layer count).
Mirrors the contract of ``graph._native_engine``: nodes come back with
``x``/``y``/``rank``/``order`` set, edges keep their real direction (triskel
handles cycles internally, so nothing is flipped), and routes are cell
polylines.
"""
pt = module()
if pt is None:
raise RuntimeError("pytriskel is not available")
pt.set_spacing(x_gutter=float(HGAP), y_gutter=float(VGAP),
edge_height=float(LANE))
by_src: dict[int, list[G.Edge]] = {}
for e in g.edges:
by_src.setdefault(e.src, []).append(e)
routes: list[G.Route] = []
x_off = 0
for comp in _components(g):
members = set(comp)
edges = [e for i in comp for e in by_src.get(i, []) if e.dst in members]
x_off = _layout_component(pt, g, comp, edges, routes, x_off)
# The native engine learns which edges are back edges from its DFS, because
# it has to reverse them to get a DAG. Triskel handles cycles internally and
# tells us nothing, so recover it from the drawing: an edge that does not
# descend is one control flow comes back along. This is style only (purple,
# and the `loop:` reading in the RPC surface) -- the direction is untouched,
# which is why ``flipped`` stays False on every triskel route.
for e in g.edges:
e.back = g.nodes[e.dst].y <= g.nodes[e.src].y
# Ranks are a layout concept the rest of the app navigates by (`w`/`b`, the
# RPC surface). Triskel doesn't expose them -- it has regions, not layers --
# so recover bands from the y coordinates the boxes actually landed on.
real = [n for n in g.nodes.values() if not n.dummy]
bands = sorted({n.y for n in real})
rank_of = {y: r for r, y in enumerate(bands)}
for n in real:
n.rank = rank_of[n.y]
for y in bands:
row = sorted((n for n in real if n.y == y), key=lambda n: n.x)
for k, n in enumerate(row):
n.order = k
_repair_boxes(g, routes)
_verify(g, routes)
return routes, len(bands)
def _layout_component(pt, g: G._Graph, comp: list[int], edges: list[G.Edge],
routes: list[G.Route], x_off: int) -> int:
"""Lay one component out, shifted right by ``x_off``. Returns the next x."""
builder = pt.make_layout_builder()
tid = {}
for nid in comp:
n = g.nodes[nid]
# NOTE the argument order: make_node(height, width). Upstream's Python
# docstring says "width and height", which is the other way round; our
# fork makes them keyword arguments so it cannot be got wrong silently.
tid[nid] = builder.make_node(height=float(n.h), width=float(n.w))
teid = [(builder.make_edge(tid[e.src], tid[e.dst], _edge_type(pt, e.kind)), e)
for e in edges]
lay = builder.build()
polys: list[tuple[G.Edge, list[tuple[float, float]]]] = []
for eid, e in teid:
polys.append((e, [(p.x, p.y) for p in lay.get_waypoints(eid)]))
# Triskel's origin is not its bounding box: a loop edge routed around the
# side runs to y = -1, above every node. Normalise on everything drawn, not
# just the boxes, or the canvas clips its own edges.
xs = [lay.get_coords(tid[i]).x for i in comp]
ys = [lay.get_coords(tid[i]).y for i in comp]
xs += [x for _, wps in polys for x, _ in wps]
ys += [y for _, wps in polys for _, y in wps]
min_x, min_y = min(xs, default=0.0), min(ys, default=0.0)
def cell(x: float, y: float) -> tuple[int, int]:
return int(round(y - min_y)), int(round(x - min_x)) + x_off
for nid in comp:
n = g.nodes[nid]
p = lay.get_coords(tid[nid])
n.y, n.x = cell(p.x, p.y)
right = max((g.nodes[i].x + g.nodes[i].w for i in comp), default=x_off)
for e, wps in polys:
pts = _clean([cell(x, y) for x, y in wps])
if len(pts) < 2:
continue
_snap_ports(g, e, pts)
pts = _clean(pts)
routes.append(G.Route(edge=e, pts=pts, head=True, tail=True,
flipped=False))
right = max(right, max(c for _, c in pts) + 1)
return right + COMPONENT_GAP
def _box_index(g: G._Graph) -> tuple[dict[int, list[G.Node]], dict[int, list[G.Node]]]:
"""(boxes strictly covering each column, boxes strictly covering each row).
"Strictly" because a cell ON the border is where ports, arrowheads and tees
legitimately live; only the interior is off limits.
"""
by_col: dict[int, list[G.Node]] = {}
by_row: dict[int, list[G.Node]] = {}
for n in g.nodes.values():
if n.dummy:
continue
for c in range(n.x + 1, n.right):
by_col.setdefault(c, []).append(n)
for r in range(n.y + 1, n.bottom):
by_row.setdefault(r, []).append(n)
return by_col, by_row
def _hits(by_col, by_row, p: tuple[int, int], q: tuple[int, int]) -> list[G.Node]:
"""Boxes whose interior a straight segment from ``p`` to ``q`` runs into."""
(r0, c0), (r1, c1) = p, q
if c0 == c1:
lo, hi = (r0, r1) if r0 <= r1 else (r1, r0)
return [n for n in by_col.get(c0, ())
if n.y < hi and lo < n.bottom]
lo, hi = (c0, c1) if c0 <= c1 else (c1, c0)
return [n for n in by_row.get(r0, ())
if n.x < hi and lo < n.right]
def _repair_boxes(g: G._Graph, routes: list[G.Route]) -> int:
"""Detour any segment that runs through a box. Returns the number moved.
Triskel does not actually guarantee this. On the corpus one edge in 128
functions comes back drawn through a block (``sub_69C0``: a vertical at
x=235 crossing a box spanning x=222.5..236.5, in float space -- so it is the
library's own layout, not our rounding). Two cells is nothing in a PNG,
where the box is opaque and painted last. In a terminal the box is mostly
holes: the edge appears *inside* the disassembly text, and ``edge_at``
happily reports an edge under a cell the user reads as code.
``docs/GRAPH_VIEW.md`` states that no edge ever crosses a box and
``tests/test_graph.py`` counts it across the corpus, so rather than weaken
the claim we push the offending run out to the nearest side of the box it
hits. Only interior segments are moved -- the first and last carry the port
and the arrowhead, and those belong on the border.
"""
by_col, by_row = _box_index(g)
moved = 0
for rt in routes:
for i in range(1, len(rt.pts) - 2):
p, q = rt.pts[i], rt.pts[i + 1]
for _ in range(4):
hit = _hits(by_col, by_row, p, q)
if not hit:
break
n = hit[0]
if p[1] == q[1]: # vertical: shift column
col = p[1]
out = n.x - 1 # nearest side of the box
if col - n.x > n.right - col or out < 0:
out = n.right + 1 # never detour off-canvas
p, q = (p[0], out), (q[0], out)
else: # horizontal: shift row
row = p[0]
out = n.y - 1
if row - n.y > n.bottom - row or out < 0:
out = n.bottom + 1
p, q = (out, p[1]), (out, q[1])
rt.pts[i], rt.pts[i + 1] = p, q
moved += 1
return moved
def _verify(g: G._Graph, routes: list[G.Route]) -> None:
"""Raise if any segment still crosses a box, so ``layout()`` falls back.
The invariant is worth more than the engine: a layout with more crossings
beats one that draws edges through the code.
"""
by_col, by_row = _box_index(g)
for rt in routes:
for p, q in zip(rt.pts, rt.pts[1:]):
hit = _hits(by_col, by_row, p, q)
if hit:
raise RuntimeError(
f"edge {rt.edge.src}->{rt.edge.dst} crosses block "
f"{hit[0].id} at {p}-{q} and could not be detoured")
def _snap_ports(g: G._Graph, e: G.Edge, pts: list[tuple[int, int]]) -> None:
"""Pull the polyline's ends onto the box borders, in place.
Triskel leaves a node at ``y + height`` -- the first row *below* the box,
because it thinks in half-open pixel rectangles while our boxes own rows
``y .. y+h-1`` inclusive and draw a border on the last one. Landing the end
points on the border row is what lets the arrowhead and the port tee replace
a border character instead of floating one cell off it.
"""
src, dst = g.nodes[e.src], g.nodes[e.dst]
def clamp(n: G.Node, col: int) -> int:
return max(n.x + 1, min(col, n.x + n.w - 2))
if len(pts) == 2:
# A straight drop between two boxes: one column has to satisfy both, or
# the "line" acquires a kink with no corner glyph to explain it.
col = clamp(dst, clamp(src, pts[0][1]))
down = pts[1][0] >= pts[0][0]
pts[0] = (src.bottom if down else src.y, col)
pts[1] = (dst.y if down else dst.bottom, col)
return
# tail: src's bottom border if the edge leaves downward, its top if not
old_r, old_c = pts[0]
col = clamp(src, old_c)
pts[0] = (src.bottom if pts[1][0] >= old_r else src.y, col)
if pts[1][1] == old_c: # the first segment was vertical: keep it
pts[1] = (pts[1][0], col)
# head: dst's top border if the edge arrives downward, its bottom if not
old_r, old_c = pts[-1]
col = clamp(dst, old_c)
pts[-1] = (dst.y if pts[-2][0] <= old_r else dst.bottom, col)
if pts[-2][1] == old_c:
pts[-2] = (pts[-2][0], col)
|