Skip to content

Conversation

@SuGlider
Copy link
Collaborator

@SuGlider SuGlider commented Dec 4, 2025

Description of Change

Adds a new Matter endpoint for Temperature Controlled Cabinet with related documentation and examples.

Test Scenarios

CI
Provided examples

Related links

none

@SuGlider SuGlider added this to the 3.3.0 milestone Dec 4, 2025
@SuGlider SuGlider self-assigned this Dec 4, 2025
@SuGlider SuGlider added the Area: Matter Issues and Feature Request about Matter Protocol label Dec 4, 2025
@SuGlider SuGlider requested a review from lucasssvaz as a code owner December 4, 2025 18:52
@SuGlider SuGlider moved this from Todo to In Progress in Arduino ESP32 Core Project Roadmap Dec 4, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

Warnings
⚠️

Some issues found for the commit messages in this PR:

  • the commit message "fix(matter): bad content in source file":
    • body's lines must not be longer than 100 characters
  • the commit message "fix(matter): typo":
    • summary looks too short
  • the commit message "fix(matter): typo":
    • summary looks too short
  • the commit message "fix(matter): typo":
    • summary looks too short
  • the commit message "fix(matter): typo":
    • summary looks too short
  • the commit message "fix(matter): typo":
    • summary looks too short
  • the commit message "fix(matter): typo":
    • summary looks too short

Please fix these commit messages - here are some basic tips:

  • follow Conventional Commits style
  • correct format of commit message should be: <type/action>(<scope/component>): <summary>, for example fix(esp32): Fixed startup timeout issue
  • allowed types are: change,ci,docs,feat,fix,refactor,remove,revert,test
  • sufficiently descriptive message summary should be between 10 to 72 characters and start with upper case letter
  • avoid Jira references in commit messages (unavailable/irrelevant for our customers)

TIP: Install pre-commit hooks and run this check when committing (uses the Conventional Precommit Linter).

Messages
📖 This PR seems to be quite large (total lines of code: 2045), you might consider splitting it into smaller PRs

👋 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 ...


This automated output is generated by the PR linter DangerJS, which checks if your Pull Request meets the project's requirements and helps you fix potential issues.

DangerJS is triggered with each push event to a Pull Request and modify the contents of this comment.

Please consider the following:
- Danger mainly focuses on the PR structure and formatting and can't understand the meaning behind your code or changes.
- Danger is not a substitute for human code reviews; it's still important to request a code review from your colleagues.
- Resolve all warnings (⚠️ ) before requesting a review from human reviewers - they will appreciate it.
- Addressing info messages (📖) is strongly recommended; they're less critical but valuable.
- To manually retry these Danger checks, please navigate to the Actions tab and re-run last Danger workflow.

Review and merge process you can expect ...


We do welcome contributions in the form of bug reports, feature requests and pull requests.

1. An internal issue has been created for the PR, we assign it to the relevant engineer.
2. They review the PR and either approve it or ask you for changes or clarifications.
3. Once the GitHub PR is approved we do the final review, collect approvals from core owners and make sure all the automated tests are passing.
- At this point we may do some adjustments to the proposed change, or extend it by adding tests or documentation.
4. If the change is approved and passes the tests it is merged into the default branch.

Generated by 🚫 dangerJS against 9fae46d

Copilot finished reviewing on behalf of SuGlider December 4, 2025 18:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new Matter endpoint MatterTemperatureControlledCabinet that implements the Matter temperature control standard for managing temperature-controlled appliances like refrigerators, freezers, and wine coolers. The implementation supports two mutually exclusive operating modes: temperature_number (precise setpoint control with min/max/step) and temperature_level (preset-based control with supported levels array).

Key changes:

  • New endpoint class with dual-mode initialization supporting both temperature setpoint and temperature level control
  • Comprehensive examples demonstrating both operating modes with dynamic temperature updates
  • Complete API documentation including usage examples and integration guides

Reviewed changes

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

Show a summary per file
File Description
MatterTemperatureControlledCabinet.h Header file defining the endpoint class with dual-mode API (temperature_number and temperature_level)
MatterTemperatureControlledCabinet.cpp Implementation with attribute management, validation, and custom endpoint for temperature_level mode
Matter.h Added include and friend class declaration for the new endpoint
keywords.txt Added new API keywords and fixed duplicate onChangeMode entry
CMakeLists.txt Added source file to build configuration
MatterTemperatureControlledCabinet example Demonstrates temperature_number mode with dynamic setpoint cycling
MatterTemperatureControlledCabinetLevels example Demonstrates temperature_level mode with dynamic level cycling
README.md files Comprehensive setup and usage documentation for both examples
ep_temperature_controlled_cabinet.rst Complete API reference documentation with examples
matter.rst Added endpoint to documentation index

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

SuGlider and others added 2 commits December 4, 2025 16:05
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@SuGlider SuGlider marked this pull request as draft December 4, 2025 19:06
SuGlider and others added 5 commits December 4, 2025 16:15
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

Test Results

 83 files   83 suites   25m 56s ⏱️
 55 tests  55 ✅ 0 💤 0 ❌
598 runs  598 ✅ 0 💤 0 ❌

Results for commit 9fae46d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

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.

MemoryFLASH [bytes]FLASH [%]RAM [bytes]RAM [%]
TargetDECINCDECINCDECINCDECINC
ESP32C5000.000.00000.000.00
ESP32S30⚠️ +6720.00⚠️ +0.04000.000.00
ESP32S20⚠️ +6720.00⚠️ +0.04000.000.00
ESP32C3000.000.00000.000.00
ESP32C6000.000.00000.000.00
ESP32H2000.000.00000.000.00
ESP320⚠️ +6720.00⚠️ +0.04000.000.00
Click to expand the detailed deltas report [usage change in BYTES]
TargetESP32C5ESP32S3ESP32S2ESP32C3ESP32C6ESP32H2ESP32
ExampleFLASHRAMFLASHRAMFLASHRAMFLASHRAMFLASHRAMFLASHRAMFLASHRAM
libraries/Matter/examples/MatterColorLight00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterCommissionTest00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterComposedLights00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterContactSensor00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterDimmableLight00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterEnhancedColorLight00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterEvents00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterFan00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterHumiditySensor00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterLambdaSingleCallbackManyEPs00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterMinimum00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterOccupancySensor00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterOnIdentify00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterOnOffLight00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterOnOffPlugin00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterPressureSensor00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterRainSensor00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterSmartButton00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterTemperatureControlledCabinet--------------
libraries/Matter/examples/MatterTemperatureControlledCabinetLevels--------------
libraries/Matter/examples/MatterTemperatureLight00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterTemperatureSensor00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterThermostat00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterWaterFreezeDetector00⚠️ +6720⚠️ +6720000000⚠️ +6720
libraries/Matter/examples/MatterWaterLeakDetector00⚠️ +6720⚠️ +6720000000⚠️ +6720

Copilot finished reviewing on behalf of SuGlider December 4, 2025 19:48
@SuGlider SuGlider added the Status: Review needed Issue or PR is awaiting review label Dec 4, 2025
@SuGlider SuGlider marked this pull request as ready for review December 4, 2025 19:51
@SuGlider SuGlider changed the title Feat/matter temperature controlled cabinet ep feat(matter): adds temperature controlled cabinet ep Dec 4, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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


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

SuGlider and others added 6 commits December 4, 2025 17:28
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@SuGlider SuGlider moved this from In Progress to In Review in Arduino ESP32 Core Project Roadmap Dec 4, 2025
Reinstate important note about mutually exclusive features for temperature control modes.
@SuGlider SuGlider added Status: Pending Merge Pull Request is ready to be merged and removed Status: Review needed Issue or PR is awaiting review labels Dec 6, 2025
pre-commit-ci-lite bot and others added 4 commits December 6, 2025 18:32
Refactor MatterTemperatureControlledCabinet to support both temperature_number and temperature_level features. Introduce new methods for handling temperature levels and update existing methods to accommodate the new structure.
@me-no-dev me-no-dev merged commit 792297f into master Dec 8, 2025
80 checks passed
@me-no-dev me-no-dev deleted the feat/matter_temperature_controlled_cabinet_ep branch December 8, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Matter Issues and Feature Request about Matter Protocol Status: Pending Merge Pull Request is ready to be merged

Projects

Development

Successfully merging this pull request may close these issues.

5 participants