Skip to content

Update: add local skill validation workflow - #22

Open
dwk-123 wants to merge 3 commits into
Impertio-Studio:mainfrom
dwk-123:main
Open

Update: add local skill validation workflow#22
dwk-123 wants to merge 3 commits into
Impertio-Studio:mainfrom
dwk-123:main

Conversation

@dwk-123

@dwk-123 dwk-123 commented Jun 24, 2026

Copy link
Copy Markdown

Summary

  • Make tools/quick_validate.py stdlib-only and add repo-wide --all validation
  • Fix the two existing structural validation issues without changing website-deploy's v15-v16 compatibility
  • Add local GitHub Actions validation plus source verification docs for official Frappe/Frappe Docker URLs
  • Document the exact splitlines() <= 500 line-count rule in CONTRIBUTING.md

Validation

  • python3 tools/quick_validate.py skills/source/core/frappe-core-database
  • python3 tools/quick_validate.py skills/source --all
  • verified 61 skills discovered
  • verified no SKILL.md exceeds 500 splitlines
  • packaged smoke tests for frappe-ops-deployment and frappe-core-database
  • git diff --check

Notes

  • The validator intentionally checks structure only; source accuracy evidence is recorded in docs/validation/frappe-source-verification-2026-06-24.md
  • No runtime DB/schema/API changes

@dwk-123
dwk-123 requested a review from FreekHeijting as a code owner June 24, 2026 13:29
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.

1 participant