Skip to content

fix(extensions): prevent AutoContextMemory compression from hanging#1075

Open
Xiaozhiyao wants to merge 2 commits intoagentscope-ai:mainfrom
Xiaozhiyao:fix_1028_autocontext_timeout
Open

fix(extensions): prevent AutoContextMemory compression from hanging#1075
Xiaozhiyao wants to merge 2 commits intoagentscope-ai:mainfrom
Xiaozhiyao:fix_1028_autocontext_timeout

Conversation

@Xiaozhiyao
Copy link
Copy Markdown

AgentScope-Java Version

1.0.11

Description

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

Testing

mvn -pl agentscope-extensions/agentscope-extensions-autocontext-memory spotless:apply
mvn -pl agentscope-extensions/agentscope-extensions-autocontext-memory -am "-Dtest=AutoContextMemoryTest,AutoContextHookTest" test

@Xiaozhiyao Xiaozhiyao requested a review from a team March 30, 2026 04:00
@cla-assistant
Copy link
Copy Markdown

cla-assistant bot commented Mar 30, 2026

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link
Copy Markdown

cla-assistant bot commented Mar 30, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

❌ Patch coverage is 87.30159% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ope/core/memory/autocontext/AutoContextMemory.java 85.96% 0 Missing and 8 partials ⚠️

📢 Thoughts on this report? Let us know!

@Xiaozhiyao
Copy link
Copy Markdown
Author

Added follow-up timeout regression tests and pushed an update. Relevant tests pass locally.

@LearningGp
Copy link
Copy Markdown
Collaborator

@shiyiyue1102 PTAL

@Xiaozhiyao
Copy link
Copy Markdown
Author

Hi @shiyiyue1102 , sorry to ping again. Could you please take a look when you have a moment?

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.

[Bug]:AutoContextMemory卡死造成agent响应卡住,在AutoContextMemory.java:753

2 participants