Chris Farhood
983498765e
Branch protection on main requires three status checks: - CI / lint (pull_request) [was already satisfied] - CI / ci (pull_request) [new: validates JSON files] - Promotion Gate / Promotion Gate (pull_request) [new: validates skills structure] Adding the ci job and Promotion Gate workflow so all required checks can pass on PRs, unblocking future merges to main. Co-Authored-By: Paperclip <noreply@paperclip.ing>
57 lines
1.4 KiB
YAML
57 lines
1.4 KiB
YAML
name: CI
|
|
|
|
on:
|
|
push:
|
|
branches: [main]
|
|
pull_request:
|
|
branches: [main]
|
|
|
|
jobs:
|
|
lint:
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- uses: actions/checkout@v4
|
|
|
|
- name: Install linters
|
|
run: |
|
|
sudo apt-get update
|
|
sudo apt-get install -y --no-install-recommends shellcheck yamllint
|
|
|
|
- name: Lint Markdown
|
|
uses: DavidAnson/markdownlint-cli2-action@v19
|
|
with:
|
|
globs: "**/*.md"
|
|
|
|
- name: Lint YAML
|
|
run: yamllint .
|
|
|
|
- name: Shellcheck
|
|
run: shellcheck scripts/*.sh
|
|
|
|
- name: Validate skill frontmatter
|
|
run: |
|
|
set -e
|
|
fail=0
|
|
for f in skills/*/SKILL.md; do
|
|
fm=$(awk 'BEGIN{c=0} /^---$/{c++; next} c==1{print} c>=2{exit}' "$f")
|
|
for key in name description; do
|
|
if ! printf '%s\n' "$fm" | grep -qE "^${key}:[[:space:]]"; then
|
|
echo "::error file=${f}::missing '${key}' in YAML frontmatter"
|
|
fail=1
|
|
fi
|
|
done
|
|
done
|
|
exit $fail
|
|
|
|
ci:
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- uses: actions/checkout@v4
|
|
|
|
- name: Validate JSON files
|
|
run: |
|
|
find . -name "*.json" -not -path "./.git/*" | while read -r f; do
|
|
python3 -m json.tool "$f" > /dev/null || { echo "::error file=$f::Invalid JSON"; exit 1; }
|
|
done
|
|
echo "All JSON files valid"
|