refactor(minifier): inline statement methods

[Boshen]

No description provided.

[graphite-app]

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.

[Copilot]

Pull Request Overview

This PR refactors the minifier's peephole optimization code by inlining statement methods for better performance and code organization. The refactoring changes method signatures from returning Option<Statement> to directly mutating the statement in place.

• Consolidates dead code removal logic into individual statement type methods
• Eliminates the central remove_dead_code_exit_statement dispatcher method
• Changes method signatures to take &mut Statement and mutate in place rather than returning new statements

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
remove_unused_declaration.rs	Updates function and class declaration removal methods to mutate statements in place
remove_dead_code.rs	Refactors all statement optimization methods to use direct mutation pattern
mod.rs	Replaces centralized dispatcher with explicit match statements in exit_statement methods
minimize_expression_in_boolean_context.rs	Removes the try_fold_stmt_in_boolean_context helper method
minifier.rs	Uncomments a debug print statement for testing

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[codspeed-hq]

CodSpeed Instrumentation Performance Report

Merging #13044 will not alter performance

_{Comparing 08-13-refactor_minifier_inline_statement_methods (7381231) with main (53c51f9)¹}

Summary

✅ 34 untouched benchmarks

Footnotes

1. No successful run was found on main (e1d23e4) during the generation of this report, so 53c51f9 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩