Skip to content

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Aug 25, 2025

Update opamp-client module to follow style guide

Analysis completed and identified the following style guide violations to fix:

  • Remove final keyword from internal classes (21 files with public final class)
  • Add static imports for commonly used utilities like Objects.requireNonNull
  • Verify field declarations are marked final where possible
  • Check for method parameter/local variable final usage that should be removed
  • Verify class organization follows the preferred order
  • Apply appropriate visibility modifiers

Files to update identified:

  • OpampClientImpl.java, OkHttpSender.java, RetryAfterParser.java, SystemTime.java
  • OpampClientState.java, AgentToServerAppenders.java
  • Various appender classes (8 files)
  • RecipeManager.java, RequestRecipe.java
  • HttpRequestService.java, WebSocketRequestService.java
  • Delay classes and OpampClientBuilder.java

Fixes #403.


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

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 opamp-client module to follow style guide
2 participants