MATTER-5951: Refrigerator app refactor#856
Draft
Michael Duggan (miduggan24) wants to merge 8 commits into
Draft
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
Bugbot Autofix is ON. A cloud agent has been kicked off to fix the reported issue. You can view the agent here.
Reviewed by Cursor Bugbot for commit 823e8b0. Configure here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Issue Link:
MATTER-5951
Description of Problem/Feature:
Extension PR to pick up project upgrades refactor for refrigerator app
Description of Fix/Solution:
Testing Done:
CI (full run with smoke)
Local testing of overrides
Note
Medium Risk
Restructures which sources and generated files compile into the refrigerator firmware; regressions would show up as build or runtime behavior changes until the paired matter_sdk bump lands.
Overview
Aligns Thread and Wi‑Fi refrigerator SLC projects with the post–project-upgrade app structure: app logic is no longer wired in via direct
AppTask.cpp,RefrigeratorManager.cpp,DataModelCallbacks.cpp, and related headers in the.slcpfiles. Builds now pullCustomerAppTaskfrom the shared Silicon Labs customer platform layer and registerRefrigeratorConfig.has a config file.The
matter_refrigeratorcomponent now templatesAppTask.h,AppTaskImpl.h, andAppTask.cppintoautogen/, so defaults and overridable*Impl()APIs are generated at build/upgrade time rather than living only in the example tree.Documentation: both refrigerator READMEs gain an Extending Base App Implementation section (TOC link included) that explains CRTP overrides in
CustomerAppTask, where attribute callbacks moved after removingDataModelCallbacks.cpp, and sampleAppInitImpl/ButtonEventHandlerImplcode plus pointers toautogen/AppTaskImpl.handautogen/AppTask.cpp.Reviewed by Cursor Bugbot for commit 823e8b0. Bugbot is set up for automated code reviews on this repo. Configure here.