feat: add player hearbeats

[lusu007]

Pull Request

Description

Type of Change

• 🐛 Bug fix
• ✨ New feature
• 💥 Breaking change
• ♻️ Refactoring
• 📚 Documentation
• 🔧 Chore

Related Issues

None

Testing

• Unit tests pass
• Manual testing completed
• New tests added for new functionality

Checklist

• I have performed a self-review of my own code
• Tests have been added/updated and pass (if needed)
• Documentation has been updated (if needed)

[Copilot]

Pull request overview

This pull request adds player heartbeat functionality to maintain active player session state in the presence service. The heartbeat mechanism periodically sends batch updates for all connected players to ensure the backend service knows which players are still online.

Note: The PR title contains a spelling error: "hearbeats" should be "heartbeats".

Changes:

• Added a scheduled task that sends periodic heartbeat batches for all connected players
• Extended the gRPC client to support heartbeat batch operations
• Integrated heartbeat scheduler lifecycle management into the plugin initialization and shutdown

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
velocity/src/main/kotlin/gg/grounds/presence/PlayerPresenceService.kt	Added heartbeatBatch method to send player heartbeats via gRPC client
velocity/src/main/kotlin/gg/grounds/presence/PlayerHeartbeatScheduler.kt	New scheduler class that periodically sends heartbeats for all connected players
velocity/src/main/kotlin/gg/grounds/GroundsPluginPlayer.kt	Integrated heartbeat scheduler with plugin lifecycle (start/stop)
common/src/main/kotlin/gg/grounds/player/presence/GrpcPlayerPresenceClient.kt	Added heartbeatBatch gRPC method with error handling
common/build.gradle.kts	Changed dependency to local-SNAPSHOT version and added local repositories

Comments suppressed due to low confidence (2)

common/build.gradle.kts:15

• The dependency version has been changed to 'local-SNAPSHOT' which is typically used only for local development. This should not be merged to a production branch. Ensure this is changed back to a proper release version (e.g., '0.1.0' or an appropriate version) before merging.
  common/build.gradle.kts:5
• Adding mavenLocal() and mavenCentral() repositories appears to be related to the local-SNAPSHOT dependency. These repository additions should be removed along with reverting the dependency version to a proper release version before merging to production.

    maven {
        url = uri("https://maven.pkg.github.com/groundsgg/*")

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[lusu007]

Merge groundsgg/library-grpc-contracts#33 and groundsgg/service-player#15 before this one.