mirror of
https://github.com/anthropics/claude-plugins-official.git
synced 2026-05-11 14:05:52 -03:00
Merge pull request #1621 from anthropics/fix/validate-frontmatter-shell-injection
Harden validate-frontmatter workflow
This commit is contained in:
commit
99832739a1
15
.github/workflows/validate-frontmatter.yml
vendored
15
.github/workflows/validate-frontmatter.yml
vendored
@ -9,6 +9,10 @@ on:
|
|||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
validate:
|
validate:
|
||||||
|
# Fork PRs are auto-closed by close-external-prs.yml, so skip validation
|
||||||
|
# for them entirely. This also prevents untrusted filenames from forks
|
||||||
|
# from ever reaching the shell steps below.
|
||||||
|
if: github.event.pull_request.head.repo.full_name == github.repository
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
@ -20,16 +24,19 @@ jobs:
|
|||||||
|
|
||||||
- name: Get changed frontmatter files
|
- name: Get changed frontmatter files
|
||||||
id: changed
|
id: changed
|
||||||
|
env:
|
||||||
|
GH_TOKEN: ${{ github.token }}
|
||||||
|
PR_NUMBER: ${{ github.event.pull_request.number }}
|
||||||
run: |
|
run: |
|
||||||
# Use diff-filter=AMRC to exclude deleted files (D) - only Added, Modified, Renamed, Copied
|
# Use diff-filter=AMRC to exclude deleted files (D) - only Added, Modified, Renamed, Copied
|
||||||
FILES=$(gh pr diff ${{ github.event.pull_request.number }} --name-only --diff-filter=AMRC | grep -E '(agents/.*\.md|skills/.*/SKILL\.md|commands/.*\.md)$' || true)
|
FILES=$(gh pr diff "$PR_NUMBER" --name-only --diff-filter=AMRC | grep -E '(agents/.*\.md|skills/.*/SKILL\.md|commands/.*\.md)$' || true)
|
||||||
echo "files<<EOF" >> "$GITHUB_OUTPUT"
|
echo "files<<EOF" >> "$GITHUB_OUTPUT"
|
||||||
echo "$FILES" >> "$GITHUB_OUTPUT"
|
echo "$FILES" >> "$GITHUB_OUTPUT"
|
||||||
echo "EOF" >> "$GITHUB_OUTPUT"
|
echo "EOF" >> "$GITHUB_OUTPUT"
|
||||||
env:
|
|
||||||
GH_TOKEN: ${{ github.token }}
|
|
||||||
|
|
||||||
- name: Validate frontmatter
|
- name: Validate frontmatter
|
||||||
if: steps.changed.outputs.files != ''
|
if: steps.changed.outputs.files != ''
|
||||||
|
env:
|
||||||
|
FILES: ${{ steps.changed.outputs.files }}
|
||||||
run: |
|
run: |
|
||||||
echo "${{ steps.changed.outputs.files }}" | xargs bun .github/scripts/validate-frontmatter.ts
|
printf '%s\n' "$FILES" | xargs bun .github/scripts/validate-frontmatter.ts
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user