refactor(ast): change plain_function to accept FunctionBody as a required parameter

[Dunqing]

The plain_function method is used to construct js-only AST, and the FunctionBody is never empty in js

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[Dunqing]

• refactor(ast): change plain_function to accept FunctionBody as a required parameter #6709 👈
• refactor(traverse): change generate_uid_in_based_on_node to accept a generic type parameter as node type #6708
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #6709 will not alter performance

_{Comparing 10-21-refactor_ast_change_plain_function_to_accept_functionbody_as_a_required_parameter (0e9b695) with main (d6609e9)}

Summary

✅ 30 untouched benchmarks

[Boshen]

Merge activity

• Oct 20, 10:55 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Oct 20, 10:55 PM EDT: A user added this pull request to the Graphite merge queue.
• Oct 20, 11:16 PM EDT: A user merged this pull request with the Graphite merge queue.