Skip to content

Conversation

Copy link

Copilot AI commented Jul 19, 2025

Identified and removed unused dependencies to reduce maintenance burden and improve build clarity.

Changes Made

Dependencies Removed:

  1. io.github.hakky54:logcaptor - Test logging capture library that was declared in both dependencyManagement/build.gradle.kts and agent/agent-tooling/build.gradle.kts but never actually used in the codebase

  2. com.uber.nullaway:nullaway - Static analysis tool that was redundantly declared in dependency constraints, as it's already properly configured as a plugin in buildSrc/build.gradle.kts

Analysis Approach:

  • Added com.autonomousapps.dependency-analysis plugin to systematically identify unused dependencies
  • Performed comprehensive codebase analysis to verify actual usage patterns
  • Examined all dependencies in the central dependency management configuration
  • Validated that remaining dependencies are actively utilized across the project

Verification:

  • ✅ Agent tooling compilation successful
  • ✅ Agent tooling tests pass (259 tests completed)
  • ✅ Complete project assembly successful (563 tasks)
  • ✅ Agent jar builds successfully (~40MB final size)
  • ✅ All static analysis checks pass (checkstyle, spotbugs)

Dependencies Retained:

All other dependencies in dependencyManagement/build.gradle.kts were confirmed to be actively used:

  • BOMs (Jackson, OpenTelemetry, Azure SDK, Netty, JUnit, Testcontainers) provide essential version management
  • Individual dependencies like commons-codec, commons-text, gson, etc. have verified usage patterns
  • Existing exclusions for jackson-dataformat-xml and woodstox-core (~1.8MB) remain appropriate

The changes maintain full backward compatibility while reducing unnecessary dependencies and improving build maintainability.

Fixes #76.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Remove unused dependencies Remove unused dependencies: logcaptor and nullaway Jul 19, 2025
Copilot AI requested a review from trask July 19, 2025 16:07
Copilot finished work on behalf of trask July 19, 2025 16:07
Copilot AI requested a review from trask July 19, 2025 16:21
Copilot finished work on behalf of trask July 19, 2025 16:21
Copilot finished work on behalf of trask July 19, 2025 16:25
@trask trask marked this pull request as ready for review July 19, 2025 16:39
@trask trask merged commit d57cda8 into main Jul 19, 2025
40 of 140 checks passed
trask added a commit that referenced this pull request Jul 21, 2025
* Initial plan

* Add dependency analysis plugin to identify unused dependencies

Co-authored-by: trask <[email protected]>

* Remove unused dependencies: logcaptor and nullaway

Co-authored-by: trask <[email protected]>

* Revert dependency-analysis plugin addition from settings.gradle.kts and build.gradle.kts

Co-authored-by: trask <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: trask <[email protected]>
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.

Remove unused dependencies

2 participants