Skip to content

fix(rag): fix rag hook not work and add log for bailian rag fails#542

Merged
AlbumenJ merged 2 commits intoagentscope-ai:mainfrom
shiyiyue1102:main-fix-rag-hook
Jan 14, 2026
Merged

fix(rag): fix rag hook not work and add log for bailian rag fails#542
AlbumenJ merged 2 commits intoagentscope-ai:mainfrom
shiyiyue1102:main-fix-rag-hook

Conversation

@shiyiyue1102
Copy link
Contributor

@shiyiyue1102 shiyiyue1102 commented Jan 13, 2026

Change-Id: I5f5709dd126491e2626740dc538f4c915b0c6ba5

AgentScope-Java Version

[The version of AgentScope-Java you are working on, e.g. 1.0.6, check your pom.xml dependency version or run mvn dependency:tree | grep agentscope-parent:pom(only mac/linux)]

Description

  1. fix rag hook not working
  2. remove enableOnlyForUserQueries,it make no sense
  3. optimize logs of bai lian rag client

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has been formatted with mvn spotless:apply
  • All tests are passing (mvn test)
  • Javadoc comments are complete and follow project conventions
  • Related documentation has been updated (e.g. links, examples, etc.)
  • Code is ready for review

Change-Id: I5f5709dd126491e2626740dc538f4c915b0c6ba5
@shiyiyue1102 shiyiyue1102 requested review from a team and Copilot January 13, 2026 09:50
Copy link
Contributor

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.

Change-Id: I8d4430ba9bb0ee670c130f62d20854627394122b
@codecov
Copy link

codecov bot commented Jan 13, 2026

Codecov Report

❌ Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...in/java/io/agentscope/core/rag/GenericRAGHook.java 25.00% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@AlbumenJ AlbumenJ merged commit 840fa76 into agentscope-ai:main Jan 14, 2026
4 checks passed
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