feat(matter): adds water leak detector ep#12102
Conversation
👋 Hello SuGlider, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Test Results 83 files 83 suites 25m 11s ⏱️ Results for commit 1272d1c. ♻️ This comment has been updated with latest results. |
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Pull request overview
This PR adds a new Matter endpoint for Water Leak Detector devices to the ESP32 Arduino Matter library. The implementation follows the established patterns used for similar sensor endpoints like ContactSensor, providing a boolean state sensor that can report water leak detection status (detected/not detected).
Key Changes:
- New
MatterWaterLeakDetectorendpoint class with get/set methods and operator overloads - Complete example sketch demonstrating commissioning, button control, LED indication, and simulated sensor behavior
- Comprehensive documentation including API reference and user guide
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
libraries/Matter/src/MatterEndpoints/MatterWaterLeakDetector.h |
Header file defining the MatterWaterLeakDetector class with boolean state interface |
libraries/Matter/src/MatterEndpoints/MatterWaterLeakDetector.cpp |
Implementation of water leak detector endpoint using BooleanState cluster |
libraries/Matter/src/Matter.h |
Added include and friend declaration for MatterWaterLeakDetector |
libraries/Matter/keywords.txt |
Added syntax highlighting keywords for the new class and methods |
libraries/Matter/examples/MatterWaterLeakDetector/MatterWaterLeakDetector.ino |
Example sketch demonstrating water leak detector with button control and simulation |
libraries/Matter/examples/MatterWaterLeakDetector/README.md |
Comprehensive user guide covering setup, usage, and troubleshooting |
libraries/Matter/examples/MatterWaterLeakDetector/ci.yml |
CI configuration for the example |
docs/en/matter/matter.rst |
Updated sensor endpoints list to include MatterWaterLeakDetector |
docs/en/matter/ep_water_leak_detector.rst |
Complete API reference documentation for the endpoint |
CMakeLists.txt |
Added MatterWaterLeakDetector.cpp to build configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
libraries/Matter/examples/MatterWaterLeakDetector/MatterWaterLeakDetector.ino
Outdated
Show resolved
Hide resolved
libraries/Matter/examples/MatterWaterLeakDetector/MatterWaterLeakDetector.ino
Outdated
Show resolved
Hide resolved
lucasssvaz
left a comment
There was a problem hiding this comment.
LGTM. Please fix the spelling issues reported by copilot
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Description of Change
Adds a new Matter Endpoint: Water Leak Detector, with related example, readme and documentation.
Test Scenarios
ESP32-S3 with the provided example. CI and execution Logs.
Related links
None