Skip to content

ensure logs are json serializable#399

Merged
xzrderek merged 1 commit intomainfrom
derekx/ensure-logs-are-json
Jan 7, 2026
Merged

ensure logs are json serializable#399
xzrderek merged 1 commit intomainfrom
derekx/ensure-logs-are-json

Conversation

@xzrderek
Copy link
Contributor

@xzrderek xzrderek commented Jan 7, 2026

Note

Improves Fireworks logging robustness by pre-serializing the request payload and handling JSON serialization errors before network calls.

  • Pre-serialize payload to payloadJson with try/catch; on failure, emit error and skip send
  • Reuse payloadJson for POST body and debug size logging; include serialized payload in debug error logs
  • Minor: bump typescript/package.json version to 0.1.12

Written by Cursor Bugbot for commit 8d08bfe. This will update automatically on new commits. Configure here.

@xzrderek xzrderek merged commit f62f1fc into main Jan 7, 2026
3 checks passed
@xzrderek xzrderek deleted the derekx/ensure-logs-are-json branch January 7, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant