Is there any reason this can't call `getPatternIdentifiers` to reuse that logic? It looks like it's used in essentially the same type of thing, except it excludes `Identifer` nodes and handles the cases recursively for nested patterns and this doesn't.