Feat(matter) adds matter water freeze detector endpoint #12101
Conversation
SuGlider
requested review from
lucasssvaz,
me-no-dev and
pedrominatel
as code owners
👋 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 27m 58s ⏱️ Results for commit c893b68. ♻️ 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 Freeze Detector devices, enabling ESP32-based smart home devices to detect and report water freeze conditions through the Matter protocol. The implementation follows the established patterns used by similar sensor endpoints like MatterContactSensor.
Key Changes:
- New MatterWaterFreezeDetector endpoint class with boolean state management (detected/not detected)
- Complete working example with WiFi/BLE commissioning, button control, LED indication, and automatic state simulation
- Comprehensive documentation including API reference, usage examples, and integration guides
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| libraries/Matter/src/MatterEndpoints/MatterWaterFreezeDetector.h | Header file defining the MatterWaterFreezeDetector class with freeze state management API |
| libraries/Matter/src/MatterEndpoints/MatterWaterFreezeDetector.cpp | Implementation of water freeze detector endpoint using BooleanState cluster |
| libraries/Matter/src/Matter.h | Added include and friend class declaration for new endpoint |
| libraries/Matter/keywords.txt | Added Arduino IDE syntax highlighting keywords for the new class and methods |
| libraries/Matter/examples/MatterWaterFreezeDetector/MatterWaterFreezeDetector.ino | Complete working example demonstrating commissioning, manual control, and automatic simulation |
| libraries/Matter/examples/MatterWaterFreezeDetector/README.md | Comprehensive documentation covering hardware requirements, setup, usage, and troubleshooting |
| libraries/Matter/examples/MatterWaterFreezeDetector/ci.yml | CI configuration for testing the example |
| docs/en/matter/matter.rst | Updated endpoint list to include water freeze detector |
| docs/en/matter/ep_water_freeze_detector.rst | Complete API reference documentation with code examples |
| CMakeLists.txt | Added new source file to build system |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
P-R-O-C-H-Y
added
Status: Pending Merge
github-project-automation
bot
moved this from In Review
to Done
in Arduino ESP32 Core Project Roadmap
Description of Change
Adds new Matter Endpoint for Water Freeze Detector endpoint, related example and documentation.
Test Scenarios
ESP32-S3 with posted example
Related links
None