Fix for 238 and doc update for 264

[bdeitte]

Fix for #238 and update docs after understanding better #264

[Copilot]

Pull request overview

This PR fixes issue #238 by adding sanitization of protocol-breaking characters in metric names and tags, and updates documentation to reflect OpenTelemetry Collector compatibility (issue #264).

Changes:

• Adds sanitization logic to prevent protocol-breaking characters (:, |, \n, #, ,) in metric names and tags from creating malformed StatsD packets
• Introduces separate sanitization functions for metric names, tag keys, and tag values with appropriate character restrictions for each
• Updates documentation to highlight OpenTelemetry Collector compatibility and reorder backend references

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
lib/helpers.js	Adds sanitizeMetricName and sanitizeTagValue functions, updates sanitizeTags to handle newlines and hash characters, modifies formatTags to sanitize tag keys and values separately
lib/statsd.js	Applies metric name sanitization before constructing StatsD messages
test/helpers.js	Adds comprehensive test coverage for the new sanitization functions and updates existing tests to reflect sanitization behavior
test/statsFunctions.js	Adds integration tests verifying sanitization works correctly across different metric types and scenarios
test/childClient.js	Updates test assertion to expect sanitized tag format
README.md	Updates documentation to list OpenTelemetry as a supported backend and adds compatibility matrix
CHANGES.md	Documents the bug fix and documentation updates

---

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