Skip to content

feat: GitHub Action, webhook signature verification, and repo restructure#9

Merged
gubbysbyte merged 4 commits into
mainfrom
dev-kailash
May 7, 2026
Merged

feat: GitHub Action, webhook signature verification, and repo restructure#9
gubbysbyte merged 4 commits into
mainfrom
dev-kailash

Conversation

@gubbysbyte

@gubbysbyte gubbysbyte commented May 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add GitHub Action (action.yml + action/index.js + dist/) for zero-infrastructure Discord notifications — no server needed, just add a workflow YAML and DISCORD_WEBHOOK_URL secret
  • Add webhook signature verification middleware (X-Hub-Signature-256 HMAC) to secure the webhook server against forged requests
  • Upgrade action runtime from node20 to node24

Restructure

  • Moved src/action/action/ to separate server code from Action source
  • Moved all docs (CONTRIBUTING, CHANGELOG, SECURITY, CODE_OF_CONDUCT, WORKFLOW) into docs/
  • Added docs/explanation.md covering both modes, architecture, and env vars
  • Added CLAUDE.md for Claude Code context
  • Switched from npm to pnpm, removed package-lock.json
  • Added act test event fixtures in tests/events/

Test plan

  • Verify Discord notification appears on push event via GitHub Action
  • Verify webhook server rejects requests with invalid/missing X-Hub-Signature-256
  • Run act push -e tests/events/push.json --secret-file .secrets locally

🤖 Generated with Claude Code

kailash1602-exe and others added 4 commits March 20, 2026 18:31
…tructure

- Add GitHub Action (action/index.js + action.yml + dist/) for zero-infrastructure Discord notifications
- Add webhook signature verification middleware (X-Hub-Signature-256 HMAC validation)
- Restructure: move src/action/ → action/, move docs to docs/ folder
- Add act test event fixtures (tests/events/)
- Switch from npm to pnpm, remove package-lock.json
- Add CLAUDE.md and docs/explanation.md

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@gubbysbyte

Copy link
Copy Markdown
Owner Author

This seems okay, now i am going to merge it in my main branch, thank for the security updates and the github action file.

@gubbysbyte gubbysbyte closed this May 7, 2026
@gubbysbyte gubbysbyte reopened this May 7, 2026
@gubbysbyte
gubbysbyte merged commit a387bd2 into main May 7, 2026
4 of 6 checks passed
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.

2 participants