fix(codegen): remove unnecessary double parentheses from satisfies expressions

[Boshen]

Summary

• Fixed TSSatisfiesExpression to only add parentheses when precedence requires it
• Before: ((x) satisfies y), After: x satisfies y
• Aligned implementation with TSAsExpression pattern
• Added idempotency tests to ensure semantic correctness

Test plan

• Added ts_satisfies_expression test with 9 idempotency test cases
• All codegen tests pass
• Clippy passes

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR fixes unnecessary double parentheses in the code generation for TypeScript satisfies expressions. The implementation previously always wrapped satisfies expressions with double parentheses (e.g., ((x) satisfies y)), which was incorrect and inconsistent with how as expressions are handled.

Changes:

• Refactored TSSatisfiesExpression code generation to only add parentheses when precedence requires it
• Aligned implementation with the existing TSAsExpression pattern
• Added comprehensive idempotency tests covering 9 different scenarios

Reviewed changes

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

Show a summary per file

File	Description
crates/oxc_codegen/src/gen.rs	Refactored TSSatisfiesExpression::gen_expr to use precedence-based wrapping logic matching TSAsExpression, removing unconditional double parentheses
crates/oxc_codegen/tests/integration/ts.rs	Added 9 idempotency test cases covering various precedence scenarios (member access, calls, logical operators, equality)
crates/oxc_codegen/tests/integration/snapshots/ts.snap	Updated snapshot showing removal of unnecessary parentheses from ((x) satisfies y) to x satisfies y
crates/oxc_codegen/tests/integration/snapshots/pure_comments.snap	Updated snapshots showing correct handling of satisfies expressions with pure comments
crates/oxc_codegen/tests/integration/snapshots/minify.snap	Updated minified output snapshot reflecting the parentheses fix

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Boshen]

Merge activity

• Jan 14, 8:57 AM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Jan 14, 8:57 AM UTC: Boshen added this pull request to the Graphite merge queue.
• Jan 14, 9:03 AM UTC: Merged by the Graphite merge queue.

[codspeed-hq]

CodSpeed Performance Report

Merging this PR will not alter performance

_{Comparing fix/codegen-satisfies-parens (1e24a21) with main (1874765)}

Summary

✅ 42 untouched benchmarks
⏩ 3 skipped benchmarks¹

Footnotes

1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩