Skip to content

Refactor skill lifecycle projection and docs#78

Merged
yun-zhi-ztl merged 3 commits intomainfrom
analysis/skill-lifecycle-audit
Mar 18, 2026
Merged

Refactor skill lifecycle projection and docs#78
yun-zhi-ztl merged 3 commits intomainfrom
analysis/skill-lifecycle-audit

Conversation

@vsxd
Copy link
Collaborator

@vsxd vsxd commented Mar 18, 2026

Summary

  • unify skill lifecycle reads around a single backend projection for detail, my-skills, stars, search, and compat mapping
  • normalize lifecycle semantics by making withdraw-review revert pending versions to draft and recomputing latest published pointer on yank
  • migrate frontend pages and docs to the new lifecycle model and remove old compatibility fields/logic

Testing

  • cd server && ./mvnw -pl skillhub-domain,skillhub-app -am test -Dtest=SkillQueryServiceTest,SkillSearchAppServiceTest,ReviewServiceTest,SkillGovernanceServiceTest,MySkillAppServiceTest,SkillLifecycleControllerTest,SkillControllerTest,MeControllerTest,ClawHubCompatControllerTest -Dsurefire.failIfNoSpecifiedTests=false
  • cd web && pnpm install
  • cd web && pnpm typecheck
  • cd web && pnpm test

@yun-zhi-ztl yun-zhi-ztl merged commit f3dbbf8 into main Mar 18, 2026
2 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