fix(commonjs): handle global shorthand property correctly

When `global` is used as a shorthand property in an object literal (e.g.,
`{global}`), the plugin was replacing `global` with
`commonjsHelpers.commonjsGlobal`, resulting in invalid syntax
`{commonjsHelpers.commonjsGlobal}`.

This fix follows the same pattern used for the `require` shorthand case:
when detecting a shorthand property, prepend `global: ` to convert it to a
full property before replacing the value.

Fixes rollup/rollup#6242

Author: Kai Gritun

packages/commonjs/src/transform-commonjs.js

@@ -322,6 +322,12 @@ export default async function transformCommonjs(
             case 'global':
               uses.global = true;
               if (!ignoreGlobal) {
+                if (isShorthandProperty(parent)) {
+                  // as key and value are the same object, isReference regards
+                  // both as references, so we need to skip now
+                  skippedNodes.add(parent.value);
+                  magicString.prependRight(node.start, 'global: ');
+                }
                 replacedGlobal.push(node);
               }
               return;

packages/commonjs/test/fixtures/function/shorthand-global/_config.js

@@ -0,0 +1,3 @@
+module.exports = {
+  description: 'correctly replaces shorthand `global` property in object'
+};

packages/commonjs/test/fixtures/function/shorthand-global/main.js

@@ -0,0 +1,7 @@
+const HOST = {
+  global
+};
+
+module.exports = {
+  HOST
+};