How bad would it be to tamper with the “walk transitive dependency graph” function? In particular, I want to add optional prelude and epilogue to the traversal, functions that get called right before expanding children, and right after. Context for this is implementing Dependency Banning, and the propagation of banning constraints through dependencies, and the desire not to make it n^2

Purely from an API standpoint I see no issue. This is a noop for existing consumers of the traversal and sounds otherwise reasonable.

Cool, thanks! I’ll do a separate PR with that.

Yeah, sounds pretty reasonable to me 🙂