Fix Named export 'Integer' not found error in es module node project

[dconeybe]

The Firestore Node bundles now all have a dependency on @firebase/webchannel-wrapper (previously, only the "browser" bundles had this dependency). If a project consuming the Firebase SDK does not define "type": "module" in its package.json, or defines "type": "commonjs" then the Firestore bundle index.node.cjs.js is used. This bundle has the line

var webchannelWrapper = require('@firebase/webchannel-wrapper');

which works.

However, if a project consuming the Firebase SDK does defines "type": "module" then the Firestore bundle index.node.mjs is used. This bundle has the line

import { Integer, Md5 } from '@firebase/webchannel-wrapper';

which fails at runtime with this error:

file:///home/dconeybe/tmp/bftest/mjs/node_modules/@firebase/firestore/dist/index.node.mjs:7
import { Integer, Md5 } from '@firebase/webchannel-wrapper';
         ^^^^^^^
SyntaxError: Named export 'Integer' not found. The requested module '@firebase/webchannel-wrapper' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@firebase/webchannel-wrapper';
const { Integer, Md5 } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:190:5)

Node.js v18.14.2

Process finished with exit code 1

This seems to be because the @webchannel-wrapper bundle index.esm2017.js is being incorrectly loaded as a commonjs module, when, in fact, it is an es module.

This PR fixes the problem by simply changing the .js extension to .mjs to indicate that it is an es module and not a commonjs module.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 25bbd56

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[google-oss-bot]

Size Report ¹

Affected Products

• @firebase/firestore

  Type	Base (ecb4454)	Merge (7f1b85f)	Diff
  browser	278 kB	282 kB	+3.46 kB (+1.2%)
  esm5	346 kB	350 kB	+3.77 kB (+1.1%)
  main	554 kB	561 kB	+6.63 kB (+1.2%)
  module	278 kB	282 kB	+3.46 kB (+1.2%)
  react-native	279 kB	282 kB	+3.41 kB (+1.2%)

• bundle

  Type	Base (ecb4454)	Merge (7f1b85f)	Diff
  firestore (Persistence)	284 kB	298 kB	+13.9 kB (+4.9%)
  firestore (Query Cursors)	222 kB	236 kB	+13.9 kB (+6.3%)
  firestore (Query)	220 kB	234 kB	+13.9 kB (+6.3%)
  firestore (Read data once)	207 kB	221 kB	+13.9 kB (+6.7%)
  firestore (Realtime updates)	209 kB	223 kB	+13.9 kB (+6.7%)
  firestore (Transaction)	190 kB	200 kB	+10.6 kB (+5.6%)
  firestore (Write data)	190 kB	200 kB	+10.6 kB (+5.6%)

• firebase

  Type	Base (ecb4454)	Merge (7f1b85f)	Diff
  firebase-compat.js	757 kB	771 kB	+13.7 kB (+1.8%)
  firebase-firestore-compat.js	324 kB	337 kB	+13.7 kB (+4.2%)
  firebase-firestore.js	327 kB	341 kB	+13.8 kB (+4.2%)

Test Logs

• Base (ecb4454): https://github.com/firebase/firebase-js-sdk/actions/runs/4701339012
• Merge (7f1b85f): https://github.com/firebase/firebase-js-sdk/actions/runs/4702215138

1. https://storage.googleapis.com/firebase-sdk-metric-reports/ORogQOQxJA.html

[dconeybe]

Note: This PR is being superseded by #7228.