chore: address PR review feedback

- runAttemptSystem.ts: remove unreachable `if (packetError)` block.
  After `if (!packet) { return; }`, packet is truthy, which means
  packetError is null per tryCatch's contract. Pre-existing dead code.
- platform.v3.server.ts: fix stale comment that referenced logger.warn —
  the helper only counts a metric (no per-call logging by design).

Author: ericallam

apps/webapp/app/services/platform.v3.server.ts

@@ -44,9 +44,10 @@ function initializeClient() {
 }
 
 const client = singleton("billingClient", initializeClient);
-// Failures from @trigger.dev/platform billing client calls. Logged at warn
-// (so they stay in stdout) and counted as a metric — Sentry was the wrong
-// tool for this signal because every task invocation hits these paths.
+// Failures from @trigger.dev/platform billing client calls are tracked via
+// this metric (with low-cardinality {function, kind} labels) rather than
+// logged. Every task invocation hits these paths, so per-call logs were too
+// noisy; dashboard the counter for visibility instead.
 const platformClientMeter = metrics.getMeter("trigger.dev/platform-client");
 const platformClientFailuresCounter = platformClientMeter.createCounter(
   "platform_client.failures_total",

internal-packages/run-engine/src/engine/systems/runAttemptSystem.ts

@@ -1980,24 +1980,12 @@ export class RunAttemptSystem {
     }
 
     if (completion.flushedMetadata) {
-      const [packetError, packet] = await tryCatch(parsePacket(completion.flushedMetadata));
+      const [, packet] = await tryCatch(parsePacket(completion.flushedMetadata));
 
       if (!packet) {
         return;
       }
 
-      if (packetError) {
-        // Customer's flushedMetadata packet failed to parse — system handles
-        // it (we return), so log at warn rather than error.
-        this.$.logger.warn("RunEngine.completeRunAttempt(): failed to parse flushed metadata", {
-          runId,
-          flushedMetadata: completion.flushedMetadata,
-          error: packetError,
-        });
-
-        return;
-      }
-
       const metadata = FlushedRunMetadata.safeParse(packet);
 
       if (!metadata.success) {