Skip to content

Refactor/clean env logs 819#1799

Open
warisu wants to merge 6 commits into
EarnQuestOne:mainfrom
warisu:refactor/clean-env-logs-819
Open

Refactor/clean env logs 819#1799
warisu wants to merge 6 commits into
EarnQuestOne:mainfrom
warisu:refactor/clean-env-logs-819

Conversation

@warisu

@warisu warisu commented Jun 26, 2026

Copy link
Copy Markdown

📝 Pull Request Summary

Description

This PR addresses Issue #819 (FE-030) by introducing an internal logging abstraction (writeLogDiagnostic) inside the environment pipeline. It cleanly strips verbose tracing metadata from client view streams when executing within active production environments.

Key Modifications

  • Production Stream Isolation: Enforces unconditional log suppression inside writeLogDiagnostic when NODE_ENV === 'production'.
  • Configurable Telemetry Overrides: Replaces hardcoded standard stream dumps with a toggle variable (ENABLE_DEBUG_LOGS) for staging and dev sandboxes.
  • Harness Diagnostics Added (>90%): Deployed dedicated spy assertions ensuring zero console noise escapes from the utility module.

Closes #819
Closes #815
Closes #812

@warisu warisu requested a review from RUKAYAT-CODER as a code owner June 26, 2026 16:04
@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@warisu 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

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Great job so far

There’s just one blocker — the workflow is failing. Could you take a look and fix it so all checks pass?
You could pull from the main to get the changes before pushing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants