fix(node): ensure fireAndForget option is respected (#1004)

| 🚥 Resolves ISSUE_ID |
| :------------------- |

## 🧰 Changes

[ReadMe Community Slack
Thread](https://readmecommunity.slack.com/archives/C066P4UL27N/p1717105349072129)!

Fixing an issue noted by the folks at Mothership where our
`fireAndForget` option wasn't being respected in the `node` package.
This is largely due to the option not being passed in correctly to the
function argument. Taking this fix a step further and exposing the
entire user-provided `options` object to this function.

## 🧬 QA & Testing

Provide as much information as you can on how to test what you've done.

Author: gratcliff

packages/node/src/lib/log.ts

@@ -25,7 +25,7 @@ function doSend(readmeApiKey: string, options: Options) {
   queue = [];
 
   // Make the log call
-  metricsAPICall(readmeApiKey, json).catch(e => {
+  metricsAPICall(readmeApiKey, json, options).catch(e => {
     // Silently discard errors and timeouts.
     if (options.development) throw e;
   });

packages/node/src/lib/metrics-log.ts

@@ -1,3 +1,4 @@
+import type { Options } from './log';
 import type { UUID } from 'crypto';
 import type { Har } from 'har-format';
 import type { Response } from 'node-fetch';
@@ -88,11 +89,7 @@ function getLogIds(body: OutgoingLogBody | OutgoingLogBody[]): LogId {
   return body._id;
 }
 
-export function metricsAPICall(
-  readmeAPIKey: string,
-  body: OutgoingLogBody[],
-  fireAndForget = false,
-): Promise<LogResponse> {
+export function metricsAPICall(readmeAPIKey: string, body: OutgoingLogBody[], options: Options): Promise<LogResponse> {
   const signal = timeoutSignal(config.timeout);
   const encodedKey = Buffer.from(`${readmeAPIKey}:`).toString('base64');
 
@@ -131,7 +128,7 @@ export function metricsAPICall(
       });
   };
 
-  if (fireAndForget) {
+  if (options.fireAndForget) {
     makeRequest();
     return Promise.resolve<LogResponse>({
       ids: getLogIds(body),