Skip to content

fix(studio): tolerate SkillSpace skill metadata - #765

Merged
yaozheng-fang merged 5 commits into
volcengine:mainfrom
richarddancin:codex/skillspace-skill-validation-pr
Jul 28, 2026
Merged

fix(studio): tolerate SkillSpace skill metadata#765
yaozheng-fang merged 5 commits into
volcengine:mainfrom
richarddancin:codex/skillspace-skill-validation-pr

Conversation

@richarddancin

Copy link
Copy Markdown
Contributor

Summary

  • fetch SKILL.md from SkillSpace package metadata when the detail response omits inline content
  • normalize legacy SkillSpace SKILL.md frontmatter so YAML-sensitive descriptions do not crash debug generation
  • relax local skill upload validation to the minimum needed for packaging: require SKILL.md and safe paths, but do not reject name/description formatting locally

Tests

  • npm test -- markdownPromptEditor.test.mjs
  • ./.venv/bin/python -m py_compile veadk/cli/generated_agent_skills.py tests/cli/test_generated_agent_backend_codegen.py tests/cli/test_generated_agent_backend_codegen_extended.py tests/cli/test_frontend_skill_spaces.py veadk/cli/cli_frontend.py
  • direct Python metadata repro for legacy SkillSpace frontmatter and relaxed local skill materialization
  • npm run build

Note: pytest and ruff are not installed in the local venv, so I could not run those exact local commands before opening the PR.

@richarddancin
richarddancin force-pushed the codex/skillspace-skill-validation-pr branch from 6f12a60 to c663662 Compare July 28, 2026 10:02
@yaozheng-fang
yaozheng-fang merged commit 3684bd9 into volcengine:main Jul 28, 2026
7 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