fix: use bunyan stdSerializers for error fields

[skwowet]

Summary

The Node 24 logging fix used Axios toJSON() so bunyan would not dump TLS sockets. That stopped the nango-worker crash, but toJSON() still includes the full request config, so prod logs for merge-suggestions 409s contained Authorization: Bearer ….

Bunyan 1.8.12 already allowlists error fields (message, name, stack, code, signal) for this reason. This PR switches to that serializer instead of a custom Axios shape.

Changes

• Remove the homemade toJSON() error serializer in @crowd/logging
• Use Bunyan.stdSerializers for err
• Map Temporal’s top-level error field to the same bunyan serializer (log.error(err) already uses err; Temporal logs { error })

[github-actions]

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

• feat: add user authentication (CM-123)
• feat: add user authentication (IN-123)

Projects:

• CM: Community Data Platform
• IN: Insights

Please add a Jira issue key to your PR title.

[Copilot]

Pull request overview

Updates shared logging to prevent sensitive Axios configuration from appearing in logs.

Changes:

• Uses Bunyan’s standard error serializers.
• Maps Temporal’s error field to the standard err serializer.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.