Add support for errorprone 2.24.1 static analysis tool and Github Action to check for issues reported

[Pearl1594]

Description

This PR adds the ErrorProne java static analysis tool that inspects the code's abstract syntax tree at compile time and report common bug patterns and API misuses.

Issues reported by this tool have been addressed in a separate PR: #12419

NOTE: Temporarily added branch: add-errorprone and 4.20 to push and pull request branches respectively to see this github action run.

Types of changes

• Breaking change (fix or feature that would cause existing functionality to change)
• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)
• Build/CI
• Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Bug Severity

• BLOCKER
• Critical
• Major
• Minor
• Trivial

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 4.15%. Comparing base (45c3819) to head (0ea7f6a).

Additional details and impacted files

@@ Coverage Diff @@## ghi11438-errorprone-fixes #12425 +/- ##
==========================================================
Coverage 4.15% 4.15% ==========================================================
Files 404 404 Lines 32966 32966 Branches 5893 5893 ==========================================================
Hits 1370 1370 Misses 31420 31420 Partials 176 176 

Flag	Coverage Δ
uitests	4.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

This PR integrates ErrorProne 2.24.1, a Java static analysis tool that inspects code's abstract syntax tree at compile time to detect common bug patterns and API misuses. The PR configures ErrorProne in the Maven build process and adds a GitHub Actions workflow to validate the codebase against ErrorProne checks.

Changes:

• Added ErrorProne 2.24.1 as a compiler plugin with warnings mode
• Created a GitHub Actions workflow to run ErrorProne analysis in strict mode
• Updated codespell dictionary to exclude "errorprone" as a typo

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
pom.xml	Added ErrorProne version property and configured maven-compiler-plugin with ErrorProne integration
.github/workflows/errorprone.yml	New GitHub Actions workflow that runs ErrorProne static analysis in strict mode and fails on detected issues
.github/linters/codespell.txt	Added "errorprone" to the codespell ignore list to prevent false positives

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The workflow includes temporary test branches ('add-errorprone' and '4.20') that should be removed before merging to main. According to the PR description, these were added temporarily to test the workflow.

Suggested change

	branches: [ main, add-errorprone ]
	pull_request:
	branches: [ main, '4.20' ]
	branches: [ main ]
	pull_request:
	branches: [ main ]

[Copilot]

The encoding configuration was added explicitly, which is good practice. However, this duplicates the project-level property 'project.build.sourceEncoding' already set to UTF-8 at line 50. While not harmful, it's redundant since Maven uses the project property by default.

Suggested change

<encoding>UTF-8</encoding>

[vishesh92]

clgtm

[DaanHoogland]

looks good, but won’t we get a 0% pass-rate this way?

[DaanHoogland]

@Pearl1594 , should we try to go for jdk17 instead of 11?

[github-actions]

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

[DaanHoogland]

@Pearl1594 should we add this to the fixes?

[Pearl1594]

I'd probably merge the fixes first, and then merge this.

[DaanHoogland]

I'd probably merge the fixes first, and then merge this.

/me back to pretending to be patient