Thomas metrics

[thomasjib]

re-named branch from #100 for docker action to work

[Copilot]

Pull request overview

This PR enables metric tracking for all protocols by refactoring the metrics implementation to use the ConnectionTracker pattern. Instead of manually wrapping connections with metrics at the protocol level, metrics tracking is now registered as a tracker with the router, which applies it automatically to all routed connections.

Changes:

• Removed direct metrics wrapping from the algeneva inbound protocol (metrics.NewConn call)
• Implemented a new MetricsTracker that conforms to the adapter.ConnectionTracker interface
• Added comprehensive test coverage for the tracker functionality
• Updated OpenTelemetry dependencies from v1.38.0 to v1.39.0
• Configured a default noop MeterProvider to disable metrics unless explicitly enabled
• Integrated metrics tracker conditionally in cmd_run.go based on MeterProvider type

Reviewed changes

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

Show a summary per file

File	Description
protocol/algeneva/inbound.go	Removed direct metrics wrapping - metrics now applied via tracker pattern
metrics/tracker.go	New ConnectionTracker implementation for metrics, wrapping connections and packet connections
metrics/tracker_test.go	Comprehensive unit test validating metrics tracking for TCP connections
cmd/main.go	Set default noop MeterProvider and added necessary imports
cmd/cmd_run.go	Added conditional metrics tracker initialization based on MeterProvider type
go.mod	Updated OpenTelemetry dependencies to v1.39.0 and added new indirect dependencies
go.sum	Updated checksums for new dependency versions
.gitignore	Added standard .DS_Store exclusion for macOS

---

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