Added skill validation

Author: robert7

CHANGELOG.md

@@ -8,6 +8,8 @@ All notable changes to this project will be documented in this file.
 
 - Renamed the local OpenClaw skill package directory to `skills/` and updated live repository references, including
   the ClawHub upload script path.
+- Extended `./code-quality.sh` to validate and package every committed skill under `skills/` using the local
+  `skill-creator` tooling before running the Node test suite.
 
 ## [0.8.0] - 2026-03-04
 

code-quality.sh

@@ -15,7 +15,7 @@ echo "Running Code Quality Checks"
 echo "========================================="
 echo ""
 
-echo -e "${YELLOW}[1/5] Running TypeScript type check...${NC}"
+echo -e "${YELLOW}[1/6] Running TypeScript type check...${NC}"
 if npm run typecheck; then
   echo -e "${GREEN}✓ Type check passed${NC}"
 else
@@ -24,7 +24,7 @@ else
 fi
 echo ""
 
-echo -e "${YELLOW}[2/5] Running ESLint...${NC}"
+echo -e "${YELLOW}[2/6] Running ESLint...${NC}"
 if npm run lint; then
   echo -e "${GREEN}✓ Lint check passed${NC}"
 else
@@ -33,7 +33,7 @@ else
 fi
 echo ""
 
-echo -e "${YELLOW}[3/5] Running Prettier format check...${NC}"
+echo -e "${YELLOW}[3/6] Running Prettier format check...${NC}"
 if npm run format:check; then
   echo -e "${GREEN}✓ Format check passed${NC}"
 else
@@ -43,7 +43,16 @@ else
 fi
 echo ""
 
-echo -e "${YELLOW}[4/5] Running tests...${NC}"
+echo -e "${YELLOW}[4/6] Validating repository skills...${NC}"
+if python3 -B "$(dirname "$0")/scripts/test_repo_skills.py"; then
+  echo -e "${GREEN}✓ Skill validation passed${NC}"
+else
+  echo -e "${RED}✗ Skill validation failed${NC}"
+  FAILURES=$((FAILURES + 1))
+fi
+echo ""
+
+echo -e "${YELLOW}[5/6] Running tests...${NC}"
 if npm test; then
   echo -e "${GREEN}✓ Tests passed${NC}"
 else
@@ -52,7 +61,7 @@ else
 fi
 echo ""
 
-echo -e "${YELLOW}[5/5] Running coverage check...${NC}"
+echo -e "${YELLOW}[6/6] Running coverage check...${NC}"
 if npm run test:coverage; then
   echo -e "${GREEN}✓ Coverage check passed${NC}"
 else

scripts/test_repo_skills.py

@@ -0,0 +1,57 @@
+#!/usr/bin/env python3
+"""
+Regression checks for skills committed in this repository.
+"""
+
+from __future__ import annotations
+
+import sys
+import tempfile
+from pathlib import Path
+from unittest import TestCase, TextTestRunner, defaultTestLoader
+
+SCRIPT_DIR = Path(__file__).resolve().parent
+REPO_ROOT = SCRIPT_DIR.parent
+SKILLS_ROOT = REPO_ROOT / "skills"
+SKILL_CREATOR_SCRIPTS = SKILLS_ROOT / "skill-creator" / "scripts"
+SKILL_DOC_FILENAME = "SKILL.md"
+
+if str(SKILL_CREATOR_SCRIPTS) not in sys.path:
+    sys.path.insert(0, str(SKILL_CREATOR_SCRIPTS))
+
+from package_skill import package_skill
+from quick_validate import validate_skill
+
+
+def iter_skill_dirs() -> list[Path]:
+    return sorted(skill_md.parent for skill_md in SKILLS_ROOT.glob(f"*/{SKILL_DOC_FILENAME}"))
+
+
+class TestRepositorySkills(TestCase):
+    def test_repository_contains_skills(self) -> None:
+        self.assertTrue(iter_skill_dirs(), f"No skills found under {SKILLS_ROOT}")
+
+    def test_all_repository_skills_validate(self) -> None:
+        for skill_dir in iter_skill_dirs():
+            with self.subTest(skill=skill_dir.name):
+                valid, message = validate_skill(skill_dir)
+                self.assertTrue(valid, f"{skill_dir.name}: {message}")
+
+    def test_all_repository_skills_package(self) -> None:
+        with tempfile.TemporaryDirectory(prefix="test_repo_skills_") as temp_dir:
+            output_dir = Path(temp_dir)
+            for skill_dir in iter_skill_dirs():
+                with self.subTest(skill=skill_dir.name):
+                    result = package_skill(skill_dir, output_dir)
+                    self.assertIsNotNone(result, f"{skill_dir.name}: packaging failed")
+                    self.assertTrue(result.exists(), f"{skill_dir.name}: archive missing")
+
+
+def main() -> int:
+    suite = defaultTestLoader.loadTestsFromTestCase(TestRepositorySkills)
+    result = TextTestRunner(verbosity=2).run(suite)
+    return 0 if result.wasSuccessful() else 1
+
+
+if __name__ == "__main__":
+    sys.exit(main())