Skip to content

Conversation

@jkebinger
Copy link
Collaborator

Summary

  • Reduced SSE event logging from INFO to DEBUG level in SseConfigStreamingSubscriber.java:112,145,156 to minimize log noise
  • Removed redundant error logging that was duplicating exception information
  • Changed if/if to if/else if for cleaner event handling
  • Bumped version from 1.0.2 to 1.0.3 across all modules

Details

The SSE subscriber was logging routine operational events (comment events, stream completions) at INFO level, which created unnecessary noise in production logs. These events are normal and expected, so DEBUG level is more appropriate.

Test plan

  • Verify reduced log output in production
  • Confirm SSE connection still works as expected
  • Validate version 1.0.3 builds successfully

🤖 Generated with Claude Code

jkebinger and others added 2 commits October 16, 2025 11:05
Changed SSE event logging from INFO to DEBUG level to reduce log noise:
- Comment events now logged at DEBUG instead of INFO
- Removed redundant "Unexpected error encountered" INFO log in onError
- Changed "Unexpected stream completion" from INFO to DEBUG in onComplete
- Changed if/if to if/else if for cleaner event handling

These are normal operational events that don't need INFO level logging.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Updated version from 1.0.2 to 1.0.3 across all modules.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Contributor

@jdwyah jdwyah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@jkebinger jkebinger merged commit afd3525 into main Oct 16, 2025
1 check passed
@jkebinger jkebinger deleted the reduce-sse-logging branch October 16, 2025 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants