Skip to content

MATTER-5951: Refrigerator app refactor#856

Draft
Michael Duggan (miduggan24) wants to merge 8 commits into
release_2.9-1.6from
project_upgrades_refrigerator_app
Draft

MATTER-5951: Refrigerator app refactor#856
Michael Duggan (miduggan24) wants to merge 8 commits into
release_2.9-1.6from
project_upgrades_refrigerator_app

Conversation

@miduggan24

@miduggan24 Michael Duggan (miduggan24) commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

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 .slcp files. Builds now pull CustomerAppTask from the shared Silicon Labs customer platform layer and register RefrigeratorConfig.h as a config file.

The matter_refrigerator component now templates AppTask.h, AppTaskImpl.h, and AppTask.cpp into autogen/, 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 removing DataModelCallbacks.cpp, and sample AppInitImpl / ButtonEventHandlerImpl code plus pointers to autogen/AppTaskImpl.h and autogen/AppTask.cpp.

Reviewed by Cursor Bugbot for commit 823e8b0. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

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.

Comment thread slc/component/matter-examples/matter_refrigerator.slcc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant