Skip to content

fix(runtime): release session lock during system tool execution#23

Merged
Yumiue merged 1 commit intofeat-memo-scope-layeringfrom
fork-pr-354-1776499938
Apr 18, 2026
Merged

fix(runtime): release session lock during system tool execution#23
Yumiue merged 1 commit intofeat-memo-scope-layeringfrom
fork-pr-354-1776499938

Conversation

@fennoai
Copy link
Copy Markdown

@fennoai fennoai Bot commented Apr 18, 2026

Requested by @Yumiue

Summary

  • Narrowed session lock scope in ExecuteSystemTool to only session snapshot loading
  • Added regression test to ensure lock is released before long-running tool execution
  • Simplified system tool ID generation helpers and duplicated test assertions

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: Yumiue <188874804+Yumiue@users.noreply.github.com>
@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Yumiue Yumiue merged commit e8feab3 into feat-memo-scope-layering Apr 18, 2026
1 check passed
@fennoai fennoai Bot deleted the fork-pr-354-1776499938 branch April 18, 2026 08:54
Yumiue pushed a commit that referenced this pull request Apr 19, 2026
fix(subagent): align permission rejection and policy limits semantics
Yumiue pushed a commit that referenced this pull request Apr 24, 2026
…lution

Fix/tui markdown render pollution
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.

3 participants