chore: add executable permission to security check script#1679
Conversation
|
@Asritha11111 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
github-actions
Bot
added
gssoc26
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds a shell script to scan the codebase for accidental client-side exposure of the Supabase service role key.
Changes:
- New
scripts/check-env-security.shscript that greps forNEXT_PUBLIC_SUPABASE_SERVICE_ROLE_KEYand fails if found.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| echo "🔍 Scanning for potential SERVICE_ROLE_KEY exposure in client-side code..." | ||
|
|
||
| # This command searches for any mention of NEXT_PUBLIC_SUPABASE_SERVICE_ROLE_KEY | ||
| if grep -r "NEXT_PUBLIC_SUPABASE_SERVICE_ROLE_KEY" --include="*.{js,jsx,ts,tsx}" .; then |
| @@ -0,0 +1,12 @@ | |||
| #!/bin/bash | |||
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Thanks for your first PR on DevTrack! 🎉
A maintainer will review it within 48 hours. While you wait:
- Make sure CI is passing (type-check + lint)
- Double-check the PR description is filled out and the issue is linked
- Feel free to ask questions in Discussions if you need help
If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!
Priyanshu-byte-coder
added
gssoc:approved
Priyanshu-byte-coder
merged commit b9c5def
into
Priyanshu-byte-coder:main
|
🎉 Merged! Thanks for contributing to DevTrack. If the project has been useful to you, a ⭐ star on the repo is the easiest way to support it — it helps DevTrack get discovered by more developers. Keep an eye on open issues for your next contribution! |
Summary
Enhanced the existing
PULL_REQUEST_TEMPLATE.mdto improve contribution quality, accessibility awareness, and reviewer guidance.Closes #1107
Type of Change
Changes Made
How to Test
Steps for the reviewer to verify this works:
Screenshots (if UI change)
N/A
Checklist
npm run lintpasses locallynpm run type-check)Accessibility Checklist
Additional Notes
This update standardizes pull request submissions and helps maintain consistent review quality across contributions.