You missed the requirement of having the file name derived from the names of ancestors. You'd need to add someting like:

$file = join '_', map { $_->first_child_text('name') } $node->ancestor
+s( 'node' );    # still need to add name from root/name
[download]

(Which I know you know, but for the OP's benefit.)

qq, a Twig fan