Improve GitHub visibility and contributor onboarding - #1
Conversation
Greptile SummaryThe PR substantially improves repository discovery and contributor onboarding.
Confidence Score: 3/5The PR should not merge until the validation workflow pins both executable actions to immutable commit SHAs. The workflow still resolves Files Needing Attention: .github/workflows/validate.yml
|
| Filename | Overview |
|---|---|
| .github/workflows/validate.yml | Adds lightweight catalog validation, but its executable actions remain referenced through mutable major-version tags. |
| scripts/validate-catalog.mjs | Adds dependency-free validation for manifests, package scripts, README links, and local social-card signatures. |
| scripts/play.mjs | Adds catalog listing, interactive selection, and pnpm-based development-server launching. |
| README.md | Reorganizes the landing page around featured experiences, categories, reusable implementation patterns, and local setup. |
| CONTRIBUTING.md | Documents contribution scope, local setup, validation expectations, rights checks, and pull-request practices. |
Reviews (2): Last reviewed commit: "Store all experience cards in the reposi..." | Re-trigger Greptile
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-node@v4 |
There was a problem hiding this comment.
Mutable workflow action references
The workflow resolves both executable actions through mutable v4 tags, allowing upstream tag movement to change the code run in pull-request and main-branch validation without a repository change. Pin both actions to full commit SHAs to make these CI dependencies immutable.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Summary
pnpm playlauncher and catalog validator covering all 39 experience manifests, READMEs, package scripts, source links, and live-demo links.Verification
npx -y node@20 scripts/validate-catalog.mjsnpx -y node@20 scripts/play.mjs --listpnpm install --filter ./experiences/vanguard-protocol --frozen-lockfilepnpm --filter ./experiences/vanguard-protocol buildpnpm play vanguard-protocol(Vite launched successfully at127.0.0.1:5188)git diff --checkGitHub admin follow-ups after merge
These settings cannot be applied by a pull request and are recorded in
docs/maintainer-playbook.md:.github/assets/social-preview.jpgunder Settings → Social preview.new-experience,dependencies, andskip-changeloglabels.