-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
ci: remove broken links and add ecosystem link validator #6421
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
- Remove 24 plugins with inaccessible GitHub repositories - Update 5 URLs that were redirecting to new locations - Add validation script (scripts/validate-ecosystem-links.js) - Add weekly GitHub Action to detect broken links Removed plugins (404/abandoned): - @fastify-userland/request-id, typeorm-query-runner, workflows - fastify-http-client, fastify-knexjs, fastify-knexjs-mock - fastify-mongo-memory, fastify-objectionjs, fastify-slonik - fastify-autocrud, fastify-file-upload, fastify-firebase - fastify-firebase-auth, fastify-nats, fastify-orientdb - fastify-socket.io, fastify-sse-v2, fastify-tokenize - fastify-twitch-ebs-tools, fastify-vite, typeorm-fastify-plugin Updated redirects: - zeit/next.js -> vercel/next.js - clerkinc/javascript -> clerk/javascript - nucleode/arecibo -> ducktors/arecibo - squaremo/amqp.node -> amqp-node/amqplib - Dev-Manny/fastify-appwrite -> maniecodes/fastify-appwrite
jean-michelet
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GG!
Should we write some unit tests?
gurgunday
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Check out repo | ||
| uses: actions/checkout@v4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both referenced actions are knocking about on old versions, I think the latest is v6?
Co-authored-by: Frazer Smith <[email protected]> Signed-off-by: Matteo Collina <[email protected]>
Co-authored-by: Frazer Smith <[email protected]> Signed-off-by: Matteo Collina <[email protected]>
- Refactor script to use fetch instead of https module for testability - Export functions for testing while maintaining CLI functionality - Add comprehensive tests using undici MockAgent for HTTP mocking - Test extractGitHubLinks with various markdown patterns - Test checkGitHubRepo for 200, 404, and error scenarios 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Summary
scripts/validate-ecosystem-links.js) to check all GitHub links.github/workflows/validate-ecosystem-links.yml) to automatically detect broken linksRemoved plugins
Abandoned accounts (as reported in security advisory):
@fastify-userland/request-id,@fastify-userland/typeorm-query-runner,@fastify-userland/workflowsfastify-http-client(kenuyx)fastify-knexjs,fastify-knexjs-mock,fastify-mongo-memory(chapuletta)fastify-objectionjs(jarcodallo)fastify-slonik(Unbuttun)Additional broken links found by validation:
fastify-autocrud,fastify-file-upload,fastify-firebase,fastify-firebase-authfastify-nats,fastify-orientdb,fastify-socket.io,fastify-sse-v2fastify-tokenize,fastify-twitch-ebs-tools,fastify-vite(community),typeorm-fastify-pluginUpdated redirects
Test plan
node scripts/validate-ecosystem-links.jswithGITHUB_TOKEN- all 320 links pass