fix: support request-scoped rollout logging config#449
Conversation
Allow TypeScript rollout loggers to use per-request gateway URLs and API keys so remote rollout servers can log status to the same tracing environment used for model calls. Made-with: Cursor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c4aa43a. Configure here.
| waitUntil: (promise: Promise<any>) => globalWaitUntil?.(promise) | ||
| }) | ||
| ] | ||
| }); |
There was a problem hiding this comment.
Duplicated logger configuration will drift over time
Low Severity
The logger configuration inside createRolloutLogger (level, format, console transport) is a near-exact copy of the module-level logger configuration. If either copy is updated in the future (e.g., log level, format options, additional transports), the other will silently remain stale. Extracting shared config (format, level, console transport factory) into a reusable constant or helper would eliminate this risk.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit c4aa43a. Configure here.


Summary
gatewayBaseUrlandapiKeysupport to the TypeScript Fireworks logging transport.createRolloutLoggerto accept an options object while preserving the existing string-name call shape.Test plan
npx tsc --noEmitfromtypescript/Made with Cursor
Note
Cursor Bugbot is generating a summary for commit c4aa43a. Configure here.