Skip to content

fix(config): change default LOG_LEVEL from debug to info#120

Merged
vigneshrajsb merged 1 commit intomainfrom
fix/log-level-default
Mar 2, 2026
Merged

fix(config): change default LOG_LEVEL from debug to info#120
vigneshrajsb merged 1 commit intomainfrom
fix/log-level-default

Conversation

@vigneshrajsb
Copy link
Contributor

Summary

  • The lifecycle-web component has no LOG_LEVEL env var set in its helm chart, so it falls back to the code-level default in src/shared/config.ts.
  • That default was 'debug', causing verbose debug output in the web component in production.
  • The worker component already has LOG_LEVEL=info set explicitly; this change aligns the web component's fallback with that value.

Change

src/shared/config.ts line 91: getServerRuntimeConfig('LOG_LEVEL', 'debug')getServerRuntimeConfig('LOG_LEVEL', 'info')

Test plan

  • Deploy lifecycle-web to a non-production environment without LOG_LEVEL set and confirm logs are at info level, not debug.
  • Confirm the worker component (which sets LOG_LEVEL=info explicitly) is unaffected.
  • No unit tests exist for config.ts; the change is a one-line default value correction.

🤖 Generated with Claude Code

The lifecycle-web component does not set LOG_LEVEL in its helm chart and
relies on the code-level default. That default was incorrectly set to
'debug', causing verbose debug output in production. Changed to 'info'
to match the explicit value already used by the worker component.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner March 2, 2026 00:01
@vigneshrajsb vigneshrajsb merged commit 91822c1 into main Mar 2, 2026
1 check passed
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