Skip to content

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

Merged
wynxing merged 1 commit into
feat-memo-scope-layeringfrom
fork-pr-354-1776499938
Apr 18, 2026
Merged

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

Conversation

@fennoai

@fennoai fennoai Bot commented Apr 18, 2026

Copy link
Copy Markdown

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!

@wynxing wynxing 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
wynxing pushed a commit that referenced this pull request Apr 19, 2026
fix(subagent): align permission rejection and policy limits semantics
wynxing 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