Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 16, 2026

Addresses two code quality issues from review feedback on the AsyncLocalStorage-based ramp context implementation.

Changes

  • Remove redundant context wrapper in recovery worker: phaseProcessor.processRamp() already wraps execution with runWithRampContext, making the outer wrapper in the recovery worker unnecessary and causing nested contexts.

  • Fix logger format spacing: Conditionally include label to prevent double spaces when empty:

    // Before: produces double space when label is ""
    return `${timestampPrefix} ${level} ${label} ${rampPrefix}${message}`;
    
    // After: single space when label is empty
    return `${timestampPrefix} ${level}${label ? ` ${label}` : ""} ${rampPrefix}${message}`;
  • Improve traceability: Added explicit state.id to recovery worker log messages for clarity outside the ramp context.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@netlify
Copy link

netlify bot commented Jan 16, 2026

Deploy Preview for vortex-sandbox ready!

Name Link
🔨 Latest commit 1ac8e70
🔍 Latest deploy log https://app.netlify.com/projects/vortex-sandbox/deploys/696a665f69bdd900075c7da2
😎 Deploy Preview https://deploy-preview-1016--vortex-sandbox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Co-authored-by: ebma <6690623+ebma@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve logging on backend Remove nested context wrapper and fix logger spacing Jan 16, 2026
Copilot AI requested a review from ebma January 16, 2026 16:26
@ebma ebma marked this pull request as ready for review January 16, 2026 16:46
@ebma ebma merged commit c6f99ab into improve-logging-on-backend Jan 16, 2026
4 checks passed
@ebma ebma deleted the copilot/sub-pr-1015 branch January 16, 2026 16:46
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.

2 participants