fix(es/minifier): Hoist props when only props are used (#10891)

**Description:**

We should not break the obj declaration into var declarations of its
props if the obj var is directly used somewhere.

Not sure if the condition `accessed_props_count < usage.ref_count` is
too strong or too weak.

**Related issue:**

 - Closes #10849

Author: CPunisher

.changeset/selfish-colts-smell.md

@@ -0,0 +1,6 @@
+---
+swc_core: patch
+swc_ecma_minifier: patch
+---
+
+fix(es/minifier): hoist props when only props are usd

crates/swc/tests/tsc-references/nullishCoalescingOperator12.2.minified.js

@@ -1,3 +1,6 @@
 //// [nullishCoalescingOperator12.ts]
 var _obj_arr;
-for (const i of null != (_obj_arr = null == obj ? void 0 : []) ? _obj_arr : []);
+const obj = {
+    arr: []
+};
+for (const i of null != (_obj_arr = null == obj ? void 0 : obj.arr) ? _obj_arr : []);

crates/swc_ecma_minifier/src/compress/optimize/props.rs

@@ -60,6 +60,15 @@ impl Optimizer<'_> {
                 return None;
             }
 
+            let accessed_props_count: u32 = usage.accessed_props.values().sum();
+            if accessed_props_count < usage.ref_count {
+                log_abort!(
+                    "hoist_props: Variable `{}` is directly used without accessing its properties",
+                    name.id
+                );
+                return None;
+            }
+
             // We should abort if unknown property is used.
             let mut unknown_used_props = self
                 .data

crates/swc_ecma_minifier/tests/fixture/issues/10849/input.js

@@ -0,0 +1,4 @@
+(function () {
+    const obj = { value: 42 };
+    console.log(obj === null || obj === void 0 ? void 0 : obj.value);
+})();

crates/swc_ecma_minifier/tests/fixture/issues/10849/output.js

@@ -0,0 +1,4 @@
+const obj = {
+    value: 42
+};
+console.log(null == obj ? void 0 : obj.value);