The catch on the throw-vs-retry behavior was exactly the right instinct — that's the kind of silent-behavior-change that would've been nasty to debug months from now. Good that it flagged it instead of just letting the default happen.
On the log gap: I'd leave it as-is. You've got no evidence it's ever fired, it's a genuinely obscure edge case (200 OK, well-formed envelope, no errors, no products), and the logLabel prefix means if it ever does happen you'll still see something in the logs pointing at this job — just not the exact original message. Not worth adding complexity to the helper to preserve a log line for a branch that's never fired in production. If it becomes relevant later, it's a one-line fix then.