Skip to content

fix: add missing Guava version and jspecify dependency#31

Open
limingjing6666 wants to merge 1 commit into
nageoffer:mainfrom
limingjing6666:fix/compile-deps
Open

fix: add missing Guava version and jspecify dependency#31
limingjing6666 wants to merge 1 commit into
nageoffer:mainfrom
limingjing6666:fix/compile-deps

Conversation

@limingjing6666
Copy link
Copy Markdown

@

Summary

  • Fix two compilation-blocking issues in framework/pom.xml that prevent mvn clean install on any machine

Changes

  1. Add Guava version (33.4.0-jre) — dependency was declared without <version>, Maven refuses to resolve it
  2. Add jspecify dependency (1.0.0) — ApplicationContextHolder.java imports @NonNull from org.jspecify.annotations, but this optional transitive dep of Guava is not resolved automatically

Verification

[INFO] Reactor Summary for ragent-ai 0.0.1-SNAPSHOT:
[INFO] ragent-ai .......................................... SUCCESS
[INFO] framework .......................................... SUCCESS
[INFO] infra-ai ........................................... SUCCESS
[INFO] bootstrap .......................................... SUCCESS
[INFO] mcp-server ......................................... SUCCESS
[INFO] BUILD SUCCESS
  • Java 17, Windows 11
  • All 5 modules compile successfully after fix

Before / After

Before After
mvn compile fails: dependencies.dependency.version for guava is missing Passes
mvn compile fails: 找不到org.jspecify.annotations包 Passes

🤖 Generated with Claude Code
@

@
fix: add missing Guava version and jspecify dependency

- Guava dependency lacked <version> tag, causing Maven compilation failure
- jspecify is an optional transitive dependency of Guava not resolved automatically
- ApplicationContextHolder uses @nonnull from jspecify, causing "symbol not found"

Both issues block compilation with "mvn clean install" on any machine.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@
Copilot AI review requested due to automatic review settings May 9, 2026 15:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants