Skip to content

✨ Feature: add skill repository lifecycle and management UI#3393

Open
Summer-Si wants to merge 16 commits into
developfrom
smm/skill_repo
Open

✨ Feature: add skill repository lifecycle and management UI#3393
Summer-Si wants to merge 16 commits into
developfrom
smm/skill_repo

Conversation

@Summer-Si

Copy link
Copy Markdown
Contributor

普通用户视图+仓库页
non_ad_repo

管理员视图+仓库页, admin有审核tab页:
ad_repo

仓库卡片
复制功能:从仓库复制到我的
copy_to_my

详情弹窗:
skill_detail

admin下架skill:
ad_depo

我的tab页:
my_tab

创建skill:
create_skill

编辑skill:
update_skill

申请发布skill:
push_skill_to_repo

admin审核tab页:
review

智能体配置,skill详情弹窗:
config_agent_detail

@Summer-Si Summer-Si requested review from Dallas98 and WMC001 as code owners July 9, 2026 07:08
Comment thread backend/services/skill_service.py Fixed
@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Comment thread backend/services/skill_service.py Fixed
Comment thread backend/services/skill_service.py Fixed
Comment thread backend/services/skill_service.py Fixed
Comment thread backend/services/skill_service.py Fixed
Comment thread backend/services/skill_service.py Fixed
Comment thread backend/services/skill_service.py Fixed
Comment thread backend/services/skill_service.py Fixed
os.makedirs(config_dir, exist_ok=True)
path = _local_skill_config_yaml_path(skill_name, local_skills_dir)
config_dir = os.path.dirname(path)
os.makedirs(config_dir, exist_ok=True)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

该告警属于 CodeQL 未能识别跨函数路径净化逻辑导致的误报。

在执行 os.makedirs() / os.path.exists() 之前,路径已经统一通过
_resolve_local_skill_path() 处理。
image

raise SkillException(f"Skill '{skill_name}' already exists locally")
if resolved:
local_skill_dir = _resolve_local_skill_path(resolved, skill_name)
if os.path.exists(local_skill_dir):

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

该告警属于 CodeQL 未能识别跨函数路径净化逻辑导致的误报。

在执行 os.makedirs() / os.path.exists() 之前,路径已经统一通过
_resolve_local_skill_path() 处理。

image

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