Skip to content

MATTER-6215 MATTER-5950: Refactor EVSE and Multi Sensor apps#857

Merged
Michael Duggan (miduggan24) merged 14 commits into
release_2.9-1.6from
project_upgrades_evse_app
Jul 9, 2026
Merged

MATTER-6215 MATTER-5950: Refactor EVSE and Multi Sensor apps#857
Michael Duggan (miduggan24) merged 14 commits into
release_2.9-1.6from
project_upgrades_evse_app

Conversation

@miduggan24

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

Copy link
Copy Markdown
Contributor

Issue Link:
MATTER-6215
MATTER-5950

Description of Problem/Feature:
Extension PR to pick up project upgrades refactor for EVSE and Multi Sensor apps

Description of Fix/Solution:

  • Bump submodule to latest matter_sdk release branch
  • Update necessary slcp/slcw files
  • Update README files

Testing Done:
CI (full run with smoke)
Local testing of overrides


Note

Medium Risk
Changes how EVSE apps are built and where application logic is customized (generated AppTask vs customer overrides), which can break existing forks that edited AppTask.cpp directly until they migrate to CustomerAppTask.

Overview
Aligns Matter EVSE Thread and Wi-Fi sample projects with the same CustomerAppTask / CRTP *Impl() extension model used by other Silicon Labs Matter examples.

Build / project wiring: Both .slcp files stop compiling the in-tree AppTask.cpp and including AppTask.h directly. They now pull in CustomerAppTask sources/headers, add EvseConfig.h as a config file, and rely on matter_evse.slcc to emit AppTask.h, AppTaskImpl.h, and AppTask.cpp into autogen/ via new template_file entries. The component SBOM license metadata is updated from MSLA to Apache 2.0.

Documentation: Thread and Wi-Fi READMEs gain an Extending Base App Implementation section (table of contents link included) that explains overriding Silicon Labs behavior through CustomerAppTask, how data-model callbacks moved from DataModelCallbacks.cpp into overridable AppTask hooks, sample header/cpp snippets, and pointers to autogen/AppTaskImpl.h and autogen/AppTask.cpp as the override reference.

Reviewed by Cursor Bugbot for commit 6ecddd3. 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 ef7c95d. Configure here.

Comment thread slc/apps/evse_app/thread/matter_thread_soc_evse_app_freertos.slcp
@miduggan24 Michael Duggan (miduggan24) changed the title MATTER-6215: Refactor EVSE app MATTER-6215 MATTER-5950: Refactor EVSE and Multi Sensor apps Jul 9, 2026
@miduggan24 Michael Duggan (miduggan24) marked this pull request as ready for review July 9, 2026 12:38
@miduggan24 Michael Duggan (miduggan24) requested a review from a team as a code owner July 9, 2026 12:38
@cursor

cursor Bot commented Jul 9, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Example app build wiring and autogen templates change how customers override behavior; Jenkins smoke now runs three named pipelines only on main/release branches instead of always-on single smoke.

Overview
Aligns EVSE and multi-sensor Thread/Wi-Fi SLC projects with the CustomerAppTask customization model: projects no longer compile vendored AppTask.cpp / AppTask.h directly and instead pull CustomerAppTask from the shared Silicon Labs customer layer, while matter_evse and matter_multi_sensor_app components now declare AppTask / AppTaskImpl / AppTask.cpp as template_file entries so defaults land under autogen/. Multi-sensor builds drop explicit SensorManager, DataModelCallbacks, and app AppTask sources from the .slcp and add SensorConfig.h (and EvseConfig.h for EVSE) as config inputs.

Documentation for all four example READMEs gains an “Extending Base App Implementation” section covering CRTP *Impl() overrides, migration of data-model callbacks into AppTask, sample CustomerAppTask code, and (EVSE only) notes that energy-management delegates are still edited outside CRTP.

CI: the Jenkins Trigger SQA Smoke Pipeline stage no longer runs on every branch (removes a debug || true), and production smoke triggers change from a single smoke pipeline to smoke-thread, smoke-wifi, and smoke-cmp. matter_evse SBOM metadata switches license to Apache 2.0.

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

Comment thread jenkins_integration/Jenkinsfile
@miduggan24 Michael Duggan (miduggan24) merged commit 32d8aaf into release_2.9-1.6 Jul 9, 2026
245 checks passed
@miduggan24 Michael Duggan (miduggan24) deleted the project_upgrades_evse_app branch July 9, 2026 19:49
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.

3 participants