fix(NODE-5056): EJSON.parse date handling when useBigInt64=true

[W-A-James]

Description

EJSON.parse fails to correctly deserialize cEJSON strings which contain date fields.

What is changing?

• EJSON.parse now correctly deserializes cEJSON dates
• Added new BSONError subclass, BSONRuntimeError to cover internal failures

Is there new documentation needed for these changes?

No

Double check the following

• Ran npm run lint script
• Self-review completed using the steps outlined here
• PR title follows the correct format: type(NODE-xxxx)[!]: description
  • Example: feat(NODE-1234)!: rewriting everything in coffeescript
• Changes are covered by tests
• New TODOs have a related JIRA ticket

[nbbeeken]

sorry jumping on this early just noticed some things

[W-A-James]

Should we change the tsdoc comment for the BSONError class? At the moment it reads:

BSONError objects are throw when runtime errors occur.

Having a subclass by the name of BSONRuntimeError seems confusing if that's what we have for the description of its parent class even if we specify that BSONRuntimeError specifically occurs when we have invalid internal state.

[nbbeeken]

Sounds like a good improvement although there's still plenty of usage of it, someday we'll have descriptive errors as needed. We can make it more generic and omit the "runtime" bit. "thrown when BSON encounters an error" and then maybe an additional note about how it is the parent of all other errors we intentionally throw.