Skip to content

ci: add CI gating merges to main - #1

Merged
aimsise merged 1 commit into
mainfrom
amp/ci
Jun 8, 2026
Merged

ci: add CI gating merges to main#1
aimsise merged 1 commit into
mainfrom
amp/ci

Conversation

@aimsise

@aimsise aimsise commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Adds GitHub Actions CI that gates merges to main. Gating job "ci" runs three dependency-free Node validators on every PR/push: PII/secret guard, repo structure + content validation (SKILL.md frontmatter, testset.json, skill structure, in-repo markdown links), and eval-harness ESM syntax. A separate non-blocking job checks external links (lychee, retries+cache, fail=false). Security: pull_request only (never pull_request_target), permissions contents:read, actions SHA-pinned, no secrets. Verified locally: passes clean repo, catches injected PII, harness ESM check passes. After merge, the "ci" status check will be wired as required in the main ruleset.

Adds .github/workflows/ci.yml plus three dependency-free Node validators run on every PR and push to main (gating job "ci"): a PII/secret guard (generic home-path/email/secret patterns, no owner identity hardcoded, no self-match on a clean tree), repo structure + content validation (SKILL.md frontmatter, testset.json, skill structure, in-repo markdown links), and an ES-module syntax check for the eval harness. A separate non-blocking job checks external links via lychee (retries+cache, fail=false) so network flakiness never gates a merge.

Security: triggers on pull_request (never pull_request_target), top-level permissions contents read, every third-party action pinned to a commit SHA, no secrets. Verified locally: passes the clean repo (14 tracked files, zero false positives incl the guard scanning itself), catches injected home-path/email violations, harness ESM check passes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@aimsise
aimsise merged commit aec440c into main Jun 8, 2026
2 checks passed
@aimsise
aimsise deleted the amp/ci branch June 8, 2026 13:40
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