Refactor extraction functions to resolve confusion around adding/returning extracted AST

[anderslanglands]

Currently some of the extraction functions return an AST object, eg ClassDecl, while others insert what they’re extracting directly into the AST. To make it more confusing, some functions update already_visited themselves, while others expect the caller to do it.

We need to figure out if we can make them all work the same way and do so.