Skip to content

fix: replace console.* calls with structured logging service#836

Open
boalambo wants to merge 3 commits into
rinafcode:mainfrom
boalambo:fix/sync-engine-conflict3
Open

fix: replace console.* calls with structured logging service#836
boalambo wants to merge 3 commits into
rinafcode:mainfrom
boalambo:fix/sync-engine-conflict3

Conversation

@boalambo

Copy link
Copy Markdown

Closes #753

Changes

  • Replaced all console.log/error/warn calls in 25 production files with structured logger using createLogger() from src/lib/logging/index.ts
  • Added ESLint rule no-console: error to .eslintrc.json to prevent regression
  • Updated .eslintignore to allow console.* in test and script files
  • Resolved merge conflicts with upstream/main

Acceptance Criteria Met

  • Zero console.* calls remain in src/ production code
  • ESLint rule prevents future regressions

@drips-wave

drips-wave Bot commented Jun 29, 2026

Copy link
Copy Markdown

@boalambo Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

[Tech-debt] 13 source files use console.log/error/warn instead of the structured logging service

1 participant