Skip to content

🐛 Bugfix: SkillManager lifecycle bug causing local_skill_path constructed with wrong tenant_id#3465

Merged
WMC001 merged 9 commits into
developfrom
xyc/skill_manager_bugfix
Jul 23, 2026
Merged

🐛 Bugfix: SkillManager lifecycle bug causing local_skill_path constructed with wrong tenant_id#3465
WMC001 merged 9 commits into
developfrom
xyc/skill_manager_bugfix

Conversation

@Jasonxia007

@Jasonxia007 Jasonxia007 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

🐛 Bugfix: SkillManager lifecycle bug causing local_skill_path constructed with wrong tenant_id

image

@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.58537% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/services/skill_service.py 87.17% 5 Missing ⚠️
sdk/nexent/core/tools/write_skill_file_tool.py 94.44% 1 Missing ⚠️
sdk/nexent/skills/skill_manager.py 98.82% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@Jasonxia007 Jasonxia007 self-assigned this Jul 21, 2026
# Conflicts:
#	sdk/nexent/core/tools/__init__.py
#	sdk/nexent/core/tools/read_skill_md_tool.py
#	sdk/nexent/core/tools/run_skill_script_tool.py
#	test/sdk/core/tools/test_read_skill_md_tool.py
#	test/sdk/core/tools/test_run_skill_script_tool.py
@WMC001
WMC001 merged commit f0a4165 into develop Jul 23, 2026
12 of 13 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