fix(isolated-declarations): correctly emit string property keys for non-identifiable name

[magic-akari]

• Closes: oxc-transform: invalid syntax output with isolatedDeclaration for object keys that require quotes #11318
• Closes: Escape with Quotes for isolatedDeclarations #11338

[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 addresses issue #11318 by fixing the emission of string property keys for non-identifiable names in isolated declarations.

• Updates snapshot tests and fixture files to include examples of string property keys that are not valid identifiers.
• Conditions in the emitter now check if a string literal or template literal represents a valid identifier before emitting it as a static identifier.

Reviewed Changes

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

File	Description
crates/oxc_isolated_declarations/tests/snapshots/object.snap	Updated snapshot to reflect correct string property keys and new declarations (type family, theme, X).
crates/oxc_isolated_declarations/tests/fixtures/object.ts	Updated fixture test with new syntax for type family, theme, and an object with reserved words as keys.
crates/oxc_isolated_declarations/src/types.rs	Refactored property key emission logic to conditionally emit identifiers only when the literal is a valid identifier.

Comments suppressed due to low confidence (1)

crates/oxc_isolated_declarations/src/types.rs:82

• [nitpick] Consider adding a comment to clarify that only the first quasi of a template literal is used for identifier validation, in case multi-segment templates are supported in the future.

PropertyKey::TemplateLiteral(literal) if is_identifier_name(&literal.quasis[0].value.raw) => {

[codspeed-hq]

CodSpeed Instrumentation Performance Report

Merging #11329 will not alter performance

_{Comparing magic-akari:fix/issue-11318 (0102309) with main (3249ab6)}

Summary

✅ 34 untouched benchmarks
⁉️ 4 dropped benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

	Benchmark	BASE	HEAD	Change
⁉️	semantic[RadixUIAdoptionSection.jsx]	72 µs	N/A	N/A
⁉️	semantic[binder.ts]	3.9 ms	N/A	N/A
⁉️	semantic[cal.com.tsx]	25.4 ms	N/A	N/A
⁉️	semantic[react.development.js]	1.5 ms	N/A	N/A

[Dunqing]

Thanks! Overall are good!

[Dunqing]

Can you pick some tests from #11338?