Classes with decorators were being removed as dead code even when unused. Decorators may have side effects (registration, global state modification, etc.), so these classes must be preserved. **Changes** - Added check in `remove_unused_class()` to return early when `!c.decorators.is_empty()` - Applies to both class declarations and class expressions - Added test coverage for both cases **Example** ```js // Before: removed entirely @test class Test {} // After: preserved @test class Test {} ``` This aligns with existing behavior for class members with decorators (line 697).  <details> <summary>Original prompt</summary> > > ---- > > *This section details on the original issue you should resolve* > > <issue_title>minifier: classes with decorators should not be removed</issue_title> > <issue_description>```js > @test > class Test {} > ``` > This should not be removed even if it's not used because `@test` may have a sideeffect. > </issue_description> > > ## Comments on the Issue (you are @copilot in this section) > > <comments> > </comments> > </details>  - Fixes oxc-project#16877  --- 💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more [Copilot coding agent tips](https://gh.io/copilot-coding-agent-tips) in the docs. --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: sapphi-red <49056869+sapphi-red@users.noreply.github.com>

Uh oh!

minifier: classes with decorators should not be removed #16877

Description

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions