oxc-project/oxc - CodSpeed

oxc

Blog Docs Changelog

Performance History

Latest Results

fix(transformer/object-rest-spread): correct scope id when moving bindings (#22419) Previously, given: ``` for (var { x, ...rest } of iterable) {} ``` The destructuring pattern appears in the `for` head, but `var` bindings are not owned by that scope - `x` and `rest` are hoisted into the current hoist scope. After transform: ``` for (var _ref of iterable) { var x = _ref.x; var rest = _objectWithoutProperties(_ref, _excluded); } ``` The bindings moved into the loop scope, but since we were passing the `for` loop's scope, `move_binding_by_symbol_id` was a no-op as the var decls were not defined in the `for` loop's scope. This PR get's the declared scope of the symbol and uses it rather than assuming that the var is scoped to the function

main

2 hours ago

c/05-14-fix_transformer_object-rest-spread_correct_scope_id_when_moving_bindings

2 hours ago

perf(transformer/object-rest-spread): collect `Vec<SymbolId` over `Vec<BindingIdentifier>` (#22418) `SymbolId` is copy, hence cheaper to clone into the `Vec` rather than cloning the whole `BindingIdentifier` (and only using the `symbol_id` property of it.

main

2 hours ago

c/05-14-fix_transformer_object-rest-spread_correct_scope_id_when_moving_bindings

3 hours ago

fix(transformer/object-rest-spread): correct scope id when moving bindings

c/05-14-fix_transformer_object-rest-spread_correct_scope_id_when_moving_bindings

3 hours ago

perf(transformer/object-rest-spread): collect `Vec<SymbolId` over `Vec<Ident>`

c/05-14-perf_transformer_object-rest-spread_collect_vec_symbolid_over_vec_ident_

3 hours ago

Merge branch 'main' into feat/minifier-unused-properties

eryue0220:feat/minifier-unused-properties

4 hours ago

fix(codegen): keep `Object.defineProperty` property name as plain string in minify (#22400) Fix `cjs-module-lexer` failing to detect `Object.defineProperty(exports, "name", ...)` named exports after oxc minify. `calculate_quote_maybe_backtick` favored backtick when all three quote costs tied at zero, so a plain ASCII property name like `"getInclusionReasons"` was emitted as a backtick template. [`cjs-module-lexer`](https://github.com/nodejs/cjs-module-lexer) (used by Node for CJS/ESM interop) only matches plain string literals there, so the affected exports silently dropped out of its export set — downstream ESM consumers then failed with `Named export 'X' not found`. In `CallExpression::gen_expr`, detect `Object.defineProperty` / `Reflect.defineProperty` and print the 2nd argument via `print_string_literal(name, false)` (the existing `allow_backtick: false` path). Other arguments — and nested strings inside the descriptor — still benefit from backtick optimization. Minify-only: outside minify, `print_string_literal` uses `self.quote` (never backtick), and `print_arguments` is needed there to preserve argument comments. ### Before / after ```js // input Object.defineProperty(exports, "getInclusionReasons", { ... }); ``` ```js // before Object.defineProperty(exports,`getInclusionReasons`,{...}); // after Object.defineProperty(exports,"getInclusionReasons",{...}); ``` Closes #22342 AI assisted.

main

4 hours ago

Latest Branches

0%

fix(transformer/object-rest-spread): correct scope id when moving bindings#22419

3 hours ago

f969770

c/05-14-fix_transformer_object-rest-spread_correct_scope_id_when_moving_bindings

0%

perf(transformer/object-rest-spread): collect `Vec<SymbolId` over `Vec<BindingIdentifier>`#22418

3 hours ago

dce1dad

c/05-14-perf_transformer_object-rest-spread_collect_vec_symbolid_over_vec_ident_

-5%

feat(minifier): remove unused object properties#22351

4 hours ago

304edbb

eryue0220:feat/minifier-unused-properties

© 2026 CodSpeed Technology

Home Terms Privacy Docs