@m1212e/rumble 0.23.18 → 0.23.19

package/out/index.cjs

@@ -211,7 +211,7 @@ function mapNullFieldsToUndefined(obj) {
 }
 //#endregion
 //#region package.json
-var version = "0.23.18";
+var version = "0.23.19";
 //#endregion
 //#region lib/helpers/errorLogging.ts
 function hasMessage(error) {
@@ -592,6 +592,24 @@ function recordSpanErrors(span, errors) {
 function recordSpanError(span, error) {
 	recordSpanErrors(span, [error]);
 }
+/**
+* Errors the operation wrapper has already reported.
+*
+* A transport can see the same failure twice: sofa, for instance, hands
+* `result.errors` to its error handler *after* execute has returned, so without
+* this the very same error would produce two log lines and two exception events,
+* which quietly doubles error counts in dashboards. A WeakSet keeps this free of
+* bookkeeping — entries disappear with the errors themselves.
+*/
+const reportedErrors = /* @__PURE__ */ new WeakSet();
+function markErrorsReported(errors) {
+	for (const error of errors) if (typeof error === "object" && error !== null) reportedErrors.add(error);
+}
+/** True when every error was already reported by the operation wrapper. */
+function errorsAlreadyReported(errors) {
+	if (errors.length === 0) return false;
+	return errors.every((error) => typeof error === "object" && error !== null && reportedErrors.has(error));
+}
 function safeJsonStringify(value) {
 	try {
 		return JSON.stringify(value, (_key, inner) => typeof inner === "bigint" ? inner.toString() : inner) ?? "null";
@@ -1632,6 +1650,7 @@ function buildTracedExecute(executeFn, rumbleInput, transport) {
 						...errorsLogField(result.errors)
 					}, "graphql execute completed with errors");
 					recordSpanErrors(span, result.errors);
+					markErrorsReported(result.errors);
 				} else log?.info({
 					...telemetry.logFields,
 					[FIELD_DURATION_MS]: durationMs(start)
@@ -1644,6 +1663,7 @@ function buildTracedExecute(executeFn, rumbleInput, transport) {
 					...errorLogField(error)
 				}, "graphql execute threw");
 				recordSpanError(span, error);
+				markErrorsReported([error]);
 				throw error;
 			}
 		};
@@ -1685,6 +1705,7 @@ function buildTracedSubscribe(subscribeFn, rumbleInput, transport) {
 							...errorsLogField(execResult.errors)
 						}, "graphql subscribe completed with errors");
 						recordSpanErrors(span, execResult.errors);
+						markErrorsReported(execResult.errors);
 					}
 					return execResult;
 				}
@@ -1706,6 +1727,7 @@ function buildTracedSubscribe(subscribeFn, rumbleInput, transport) {
 					...errorLogField(error)
 				}, "graphql subscribe threw");
 				recordSpanError(span, error);
+				markErrorsReported([error]);
 				throw error;
 			}
 		};
@@ -2870,12 +2892,14 @@ export const r = rumble({
 			execute: buildTracedExecute(args.execute ?? graphql.execute, rumbleInput, "rest"),
 			subscribe: buildTracedSubscribe(args.subscribe ?? graphql.subscribe, rumbleInput, "rest"),
 			errorHandler(errors) {
-				recordSpanErrors(_opentelemetry_api.trace.getActiveSpan(), errors);
-				telemetryLogger(rumbleInput)?.error({
-					[ATTR_TRANSPORT]: "rest",
-					...traceCorrelationFields(rumbleInput),
-					...errorsLogField(errors)
-				}, "rest request failed");
+				if (!errorsAlreadyReported(errors)) {
+					recordSpanErrors(_opentelemetry_api.trace.getActiveSpan(), errors);
+					telemetryLogger(rumbleInput)?.error({
+						[ATTR_TRANSPORT]: "rest",
+						...traceCorrelationFields(rumbleInput),
+						...errorsLogField(errors)
+					}, "rest request failed");
+				}
 				return userErrorHandler?.(errors) ?? sofaErrorResponse(errors);
 			}
 		});