refactor(transformer/class-properties): do not pass `ScopeId` into `insert_instance_inits` #8001

overlookmotel · 2024-12-18T16:51:29Z

No need to pass this var through various functions, as they already have access to self to get it itself.

overlookmotel · 2024-12-18T16:51:55Z

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

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

_{An organization admin has enabled the Graphite Merge Queue in this repository.} _{Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.}

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

codspeed-hq · 2024-12-18T16:59:53Z

CodSpeed Performance Report

Merging #8001 will not alter performance

_{Comparing 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ (059a5dd) with main (f1adf9f)}

Summary

✅ 29 untouched benchmarks

Dunqing · 2024-12-20T03:09:43Z

Merge activity

Dec 19, 10:09 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
Dec 19, 10:09 PM EST: A user added this pull request to the Graphite merge queue.
Dec 19, 10:41 PM EST: A user merged this pull request with the Graphite merge queue.

…nsert_instance_inits` (#8001) No need to pass this var through various functions, as they already have access to `self` to get it itself.

## [0.43.0] - 2024-12-21 - de4c772 traverse: [**BREAKING**] Rename `Ancestor::is_via_*` methods to `is_parent_of_*` (#8031) (overlookmotel) - ed75e42 semantic: [**BREAKING**] Make SymbolTable fields `pub(crate)` instead of `pub` (#7999) (Boshen) ### Features - 75b775c allocator: `Vec<u8>::into_string` (#8017) (overlookmotel) - 8547e02 ast: Implement `allocator_api2` for `Allocator` (#8043) (Boshen) - 63a95e4 ast: Add `AstBulder::move_property_key` (#7998) (overlookmotel) - 897a1a8 transformer/class-properties: Exit faster from super replacement visitor (#8028) (overlookmotel) - 3ea4109 transformer/class-properties: Transform super update expressions within static prop initializer (#7997) (Dunqing) - cc57db3 transformer/class-properties: Transform super assignment expressions within static prop initializer (#7991) (Dunqing) - 6b6444b traverse: Record current block scope (#8007) (overlookmotel) ### Bug Fixes - 043252d transformer/class-properties: Replace `this` and class name in static blocks (#8035) (overlookmotel) - 273795d transformer/class-properties: Run other transforms on static properties, static blocks, and computed keys (#7982) (overlookmotel) ### Performance - c0dd3f8 ast: `move_expression` and `move_statement` produce dummy with no span (#7995) (overlookmotel) - 862838f codegen: Remove useless to_owned (#8014) (Dunqing) - 2736657 semantic: Allocate `UnresolvedReferences` in allocator (#8046) (Boshen) - 2e8872c semantic: Allocate child scope in allocator (#8045) (Boshen) - 414e828 semantic: Allocate symbol data in Allocator (#8012) (Boshen) - 7aebed0 semantic: Allocate `Bindings` in allocator (#8021) (Boshen) - 0f9308f transformer/react-refresh: Reduce allocations (#8018) (overlookmotel) - 0deb9e6 transformer/react-refresh: Reserve capacity in hook key string (#8016) (overlookmotel) - 7b70347 transformer/react-refresh: Avoid allocating string in each hook call (#8013) (Dunqing) ### Documentation - df5c341 ast: Improve docs for `AstBuilder::move_*` methods (#7994) (overlookmotel) ### Refactor - f1adf9f semantic: `ScopeTree::rename_binding` remove old binding first (#8020) (overlookmotel) - 02f968d semantic: Change `Bindings` to a plain `FxHashMap` (#8019) (Boshen) - e7476a1 semantic: Remove `serialize` (#8015) (Boshen) - 1cf7b83 semantic: Simplify handling namespace stack (#7987) (Dunqing) - 48cb52b semantic: Remove resetting `current_reference_flags` in visit functions (#7986) (Dunqing) - 3250a47 semantic: Remove unused current_symbol_flags (#7985) (Dunqing) - efe96ec semantic: Use `Stack` for function stack node ids (#7984) (Dunqing) - ac097e9 transformer/class-properties: Rename file (#8036) (overlookmotel) - 059a5dd transformer/class-properties: Do not pass `ScopeId` into `insert_instance_inits` (#8001) (overlookmotel) - 0a38eea transformer/class-properties: Use `temp_var_name_base` to generate temp var names for `super` transform (#8004) (overlookmotel) - d1b7181 transformer/class-properties: Rename var (#8006) (overlookmotel) - 5a23d72 transformer/class-properties: Remove outdated comment (#8000) (overlookmotel) - b3a5f3e transformer/class-properties: Mark `transform_assignment_expression_if_super_member_assignment_target` as inline (#7993) (Dunqing) ### Testing - bcb33c0 semantic: Add a test for catch parameters reference (#7988) (Dunqing) Co-authored-by: Boshen <1430279+Boshen@users.noreply.github.com>

overlookmotel mentioned this pull request Dec 18, 2024

feat(ast): add AstBulder::move_property_key #7998

Merged

github-actions bot added the A-transformer Area - Transformer / Transpiler label Dec 18, 2024

overlookmotel mentioned this pull request Dec 18, 2024

fix(transformer/class-properties): run other transforms on static properties, static blocks, and computed keys #7982

Merged

github-actions bot added the C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior label Dec 18, 2024

This was referenced Dec 18, 2024

docs(ast): improve docs for AstBuilder::move_* methods #7994

Merged

perf(ast): move_expression and move_statement produce dummy with no span #7995

Merged

overlookmotel marked this pull request as ready for review December 18, 2024 16:52

overlookmotel requested a review from Dunqing as a code owner December 18, 2024 16:52

overlookmotel marked this pull request as draft December 18, 2024 16:52

overlookmotel force-pushed the 12-18-feat_ast_add_astbulder_move_property_key_ branch from 0ed4a9f to 977f5cc Compare December 18, 2024 16:59

overlookmotel force-pushed the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch from 766179d to 51061df Compare December 18, 2024 16:59

overlookmotel marked this pull request as ready for review December 18, 2024 17:00

overlookmotel force-pushed the 12-18-feat_ast_add_astbulder_move_property_key_ branch from 977f5cc to 6046763 Compare December 18, 2024 23:08

overlookmotel force-pushed the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch from 51061df to 2e88a2e Compare December 18, 2024 23:08

overlookmotel force-pushed the 12-18-feat_ast_add_astbulder_move_property_key_ branch from 6046763 to ed9895d Compare December 18, 2024 23:11

overlookmotel force-pushed the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch from 2e88a2e to 7c732bd Compare December 18, 2024 23:11

overlookmotel changed the base branch from 12-18-feat_ast_add_astbulder_move_property_key_ to 12-18-perf_ast_move_expression_and_move_statement_produce_dummy_with_no_span December 18, 2024 23:11

overlookmotel force-pushed the 12-18-perf_ast_move_expression_and_move_statement_produce_dummy_with_no_span branch 2 times, most recently from 3df6665 to abcdf6b Compare December 18, 2024 23:23

overlookmotel force-pushed the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch 2 times, most recently from 972fdb1 to 57c6149 Compare December 19, 2024 01:34

This was referenced Dec 19, 2024

feat(traverse): record current block scope #8007

Merged

chore(transformer): enable class-properties plugin #7750

Merged

overlookmotel force-pushed the 12-18-perf_ast_move_expression_and_move_statement_produce_dummy_with_no_span branch from abcdf6b to d0a2649 Compare December 19, 2024 20:50

overlookmotel force-pushed the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch from 57c6149 to 393fdd1 Compare December 19, 2024 20:50

overlookmotel mentioned this pull request Dec 19, 2024

feat(transformer/class-properties): exit faster from super replacement visitor #8028

Merged

overlookmotel force-pushed the 12-18-perf_ast_move_expression_and_move_statement_produce_dummy_with_no_span branch from d0a2649 to deb6c60 Compare December 19, 2024 22:23

overlookmotel force-pushed the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch from 393fdd1 to a7c1544 Compare December 19, 2024 22:23

This was referenced Dec 19, 2024

refactor(traverse)!: rename Ancestor::is_via_* methods to is_parent_of_* #8031

Merged

fix(transformer/class-properties): replace this and class name in static blocks #8035

Merged

Dunqing added the 0-merge Merge with Graphite Merge Queue label Dec 20, 2024

refactor(transformer/class-properties): do not pass ScopeId into `i…

059a5dd

…nsert_instance_inits` (#8001) No need to pass this var through various functions, as they already have access to `self` to get it itself.

Dunqing force-pushed the 12-18-perf_ast_move_expression_and_move_statement_produce_dummy_with_no_span branch from deb6c60 to c0dd3f8 Compare December 20, 2024 03:11

Dunqing force-pushed the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch from a7c1544 to 059a5dd Compare December 20, 2024 03:12

overlookmotel mentioned this pull request Dec 20, 2024

refactor(transformer/class-properties): rename file #8036

Merged

Base automatically changed from 12-18-perf_ast_move_expression_and_move_statement_produce_dummy_with_no_span to main December 20, 2024 03:37

graphite-app bot merged commit 059a5dd into main Dec 20, 2024
28 checks passed

graphite-app bot deleted the 12-18-refactor_transformer_class-properties_do_not_pass_scopeid_into_insert_instance_inits_ branch December 20, 2024 03:41

oxc-bot mentioned this pull request Dec 21, 2024

release(crates): v0.43.0 #8054

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor(transformer/class-properties): do not pass `ScopeId` into `insert_instance_inits` #8001

refactor(transformer/class-properties): do not pass `ScopeId` into `insert_instance_inits` #8001

overlookmotel commented Dec 18, 2024 •

edited

Loading

overlookmotel commented Dec 18, 2024 •

edited

Loading

codspeed-hq bot commented Dec 18, 2024 •

edited

Loading

Dunqing commented Dec 20, 2024 •

edited by graphite-app bot

Loading

refactor(transformer/class-properties): do not pass ScopeId into insert_instance_inits #8001

refactor(transformer/class-properties): do not pass ScopeId into insert_instance_inits #8001

Conversation

overlookmotel commented Dec 18, 2024 • edited Loading

overlookmotel commented Dec 18, 2024 • edited Loading

How to use the Graphite Merge Queue

codspeed-hq bot commented Dec 18, 2024 • edited Loading

CodSpeed Performance Report

Merging #8001 will not alter performance

Summary

Dunqing commented Dec 20, 2024 • edited by graphite-app bot Loading

Merge activity

refactor(transformer/class-properties): do not pass `ScopeId` into `insert_instance_inits` #8001

refactor(transformer/class-properties): do not pass `ScopeId` into `insert_instance_inits` #8001

overlookmotel commented Dec 18, 2024 •

edited

Loading

overlookmotel commented Dec 18, 2024 •

edited

Loading

codspeed-hq bot commented Dec 18, 2024 •

edited

Loading

Dunqing commented Dec 20, 2024 •

edited by graphite-app bot

Loading