Skip to content

feat: UDP messaging client #37#46

Open
dkstlzk wants to merge 16 commits intoAISocietyIITJ:tasksfrom
dkstlzk:tasks
Open

feat: UDP messaging client #37#46
dkstlzk wants to merge 16 commits intoAISocietyIITJ:tasksfrom
dkstlzk:tasks

Conversation

@dkstlzk
Copy link
Copy Markdown
Collaborator

@dkstlzk dkstlzk commented Jun 30, 2025

📝 What does this Pull Request do?

Adds a basic UDP messaging client in Go with a CLI.
This client allows sending and receiving text messages over a local network using static UDP ports. The user can specify the local and target ports via CLI flags.

🔧 Type of Change

  • 🐛 Bug fix (fixes something that wasn't working)
  • ✨ New feature (adds new functionality)
  • 💅 Style change (colors, fonts, layout - no functionality change)
  • 📚 Documentation update (README, guides, comments)
  • 🧪 Tests (adding or fixing tests)
  • 🔧 Refactoring (improving code without changing functionality)

🎯 Which part of the project does this affect?

  • 🧠 AI/ML Models (AI components in ai/)
  • 🔧 Backend (Server logic in backend/)
  • 🌐 Frontend (CLI & React.js in frontend/)
  • 🌐 Network Layer (P2P networking in network/)
  • � Shared Utilities (Common code in shared/)
  • 📚 Documentation
  • 🚀 CI/CD (GitHub workflows)

🧪 How did you test this?

Tested by running two instances locally in separate terminals, exchanging messages between them.
Verified:

  • Messages are received correctly.
  • exit command cleanly terminates both instances.
  • No errors or nil address logs on exit.
  • Default port values work as expected when no flags are passed.

📸 Screenshots (if you changed the UI)

Screenshot From 2025-06-30 21-03-22
Screenshot From 2025-06-30 21-03-43

✅ Checklist

  • 🔍 I tested my changes and they work
  • 💬 My commit messages follow the format (like feat: add login button)
  • 🌿 My branch has a descriptive name (like add-login-button or fix-spelling)
  • 📝 I have correctly updated the workflows for any new tech stack that I used
  • 📖 I read the Contributing Guide if this is my first PR
  • 🤔 I would be comfortable explaining my changes to someone else

🤝 Additional Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants