Skip to content

Conversation

Copy link

Copilot AI commented Aug 24, 2025

Plan to update processors module to follow docs/style-guide.md with minimal changes:

  • Explore repository structure and identify processors module
  • Review current code for style guide compliance
  • Identify specific style guide violations
  • Fix field visibility and final modifiers in FilteringLogRecordProcessor
  • Add final modifiers to private fields in Interceptable*Exporter classes (already compliant)
  • Replace manual service registration with @autoservice annotation
  • Fix test compilation issues after making FilteringLogRecordProcessor final
  • Remove unnecessary extra semicolons in test code
  • Run spotlessApply to ensure formatting consistency
  • Verify all changes with tests

Changes Made:

FilteringLogRecordProcessor.java

  • Made class final per style guide for public non-internal classes
  • Changed public fields to private final for proper encapsulation

EventToSpanEventBridgeComponentProvider.java

  • Added @AutoService(ComponentProvider.class) annotation to replace manual service registration
  • Added @SuppressWarnings("rawtypes") to handle generic service provider warning
  • Removed manual service file as @autoservice generates it automatically

build.gradle.kts

  • Added AutoService annotation processor and compile-only dependencies
  • Added explanatory comment about AutoService usage

FilteringLogRecordProcessorTest.java

  • Fixed test compilation by removing empty anonymous class body {}
  • Cleaned up extra semicolons after field declarations

All tests pass and code formatting is applied according to the style guide.

Fixes #276.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: trask <218610+trask@users.noreply.github.com>
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.

Update processors module to follow style guide

2 participants