Add Log & Errors Review
to your pull requests.

P1OAuth token refresh verifies statusText rather than response.ok · Log & Errors Review

When Salesforce returns HTTP 200 with an empty statusText (common in HTTP/2) or non-OK text, getClient throws HttpError(400) and aborts valid token refresh.

+     if (!response.ok) throw new HttpError({ statusCode: 400, message: response.statusText });

P2Inconsistent metric tag name 'shards' instead of 'shard' · Log & Errors Review

The metric timer uses tag key 'shards' while sibling metrics 'spans.buffer.flusher.produce' and 'spans.buffer.segment_size_bytes' use 'shard', preventing unified aggregation across flusher produce metrics.

+                 with metrics.timer("spans.buffer.flusher.wait_produce", tags={"shard": shard_tag}):

P1Plugin request completion log drops traceID field after refactor · Log & Errors Review

When a plugin request completes, the log emitted by LoggerMiddleware omits traceID because it was removed from logParams and is not set in instrumentContext, breaking distributed trace correlation for operators.

P1Migration crashes when embed_category is not configured · Log & Errors Review

When embed_category has no matching category (the default), execute(...)[0] is nil, raising NoMethodError on ['id'] and halting migrations during deployment.