neighbors*

Do you explicitly want to specify edges=*? (because it is already the default value)

What about rev:rev in the edges parameter? Or are we only interested in the "outer" revisions?

I don't think we need this part in the doc, the task on the forge is already enough (and I already claimed it :p).

Again, what about rev:rev?

Same comment as above on the TODOs.

yeah, it was on purpose to make it explicit here (and in case the default ever changes)

Oh, that's a good point. I wouldn't block on this one to merge, because we can actually do both, but I punt to @grouss about what we actually want to benchmark here.

Fair enough on the pertinence of having this here (hence I've removed it).

However, this assumes that I've actually written down the implementation correctly and that it shouldn't be pivot around. (I'm assuming you'll let me know if changes are needed.)

Same answer: invocation of @grouss :)

after discussing with @grouss: this is fine as is (i.e., recursion on the dir/cnt layer, but no recursion on the rev layer)

after discussion with @grouss this one, OTOH, does need a rev:rev addition, I'll update the diff accordingly