scriptoriumadmin/OpenSpace
1
0
Fork 0
mirror of https://github.com/HKUDS/OpenSpace.git synced 2026-09-10 22:41:02 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #30 from voidborne-d/fix/fix-skill-already-registered

Browse source 
fix: register_skill_dir returns existing SkillMeta for already-registered skills
This commit is contained in:
Dennis-yxchen 2026-03-31 15:01:26 +08:00
commit ccda314d34
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
openspace/skill_engine/registry.py
View file

@ -299,8 +299,8 @@ class SkillRegistry:
skill_dir: Path to a directory containing ``SKILL.md``.
Returns:
:class:`SkillMeta` if newly registered, ``None`` if already
present, the directory is invalid, or the skill fails safety checks.
:class:`SkillMeta` if newly registered or already present,
``None`` if the directory is invalid or the skill fails safety checks.
"""
skill_file = skill_dir / "SKILL.md"
if not skill_file.exists():
@ -321,7 +321,7 @@ class SkillRegistry:
meta = self._parse_skill(skill_dir.name, skill_dir, skill_file, content)
if meta.skill_id in self._skills:
logger.debug(f"register_skill_dir: {meta.skill_id} already exists")
return None
return self._skills[meta.skill_id]
self._skills[meta.skill_id] = meta
self._content_cache[meta.skill_id] = content
logger.info(f"Hot-registered skill: {meta.skill_id}")