Sync csa branch with main#851
Merged
sarthak shaha (Sarthak-Shaha) merged 125 commits intomainfrom Feb 13, 2026
Merged
Conversation
* [Darwin] extract inline partials zap is not a fan * share command and attribute declarations template
* Use 31 bits to encapsulate platform error values * Add tests for platform encapsulation * Fix casting * Implement copilot suggestions and add more tests * Change casting from unsigned to signed on Darwin * Keep kPlatform for backward-compatibility * Drop deprecated annotation * Update src/lib/core/CHIPError.h Co-authored-by: Karsten Sperling <113487422+ksperling-apple@users.noreply.github.com> * Fix bit counting which should be 0-based * Rename const values to be more meaningful * Check min/max value for given type when encapsulating * Address review comments and fix issues after merge with master * Update silabs code * Address issues reported by copilot * Restyled by clang-format * Simplify IsRange() implementation --------- Co-authored-by: Karsten Sperling <113487422+ksperling-apple@users.noreply.github.com> Co-authored-by: Restyled.io <commits@restyled.io>
* fix: Add cross-compilation support for libdatachannel build - Pass target_cc/target_cxx from args.gn to CMake for cross-compilation - Support generic CPU type detection (arm, arm64, x64, x86) - Disable examples/tests during cross-compilation to avoid host library linking issues - Maintain backward compatibility with existing arm64 sysroot-based compilation * build_libdatachannel.py: Improve handling of --target_c and --target_cc arguments Signed-off-by: Maciej Grela <m.grela@samsung.com> * Restyled by isort * build_libdatachannel.py: Use click exceptions for more useful error messages Signed-off-by: Maciej Grela <m.grela@samsung.com> * build_libdatachannel.py: --target-cc and --target-cxx options need to be used together Signed-off-by: Maciej Grela <m.grela@samsung.com> --------- Signed-off-by: Maciej Grela <m.grela@samsung.com> Co-authored-by: Maciej Grela <m.grela@samsung.com> Co-authored-by: Restyled.io <commits@restyled.io>
* Fix compilation error Signed-off-by: Doru-Cristian Gucea <doru-cristian.gucea@nxp.com> * Restyled by clang-format --------- Signed-off-by: Doru-Cristian Gucea <doru-cristian.gucea@nxp.com> Co-authored-by: Restyled.io <commits@restyled.io>
Signed-off-by: Martin Girardot <martin.girardot@nxp.com>
* created guide * fixinf comments * Restyled by prettier-markdown * fix typos * move to docs/guides --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Decouple illuminance measurement cluster * Restyled by clang-format * Restyled by gn * Code update * Address review comment * Address review comments * Addressing review comments * Restyled by clang-format * Add README.md and address review comment * Restyled by prettier-markdown * Update unit tests --------- Co-authored-by: Restyled.io <commits@restyled.io>
… global values (#42929) * Decouple `NetworkCommissioningCluster` * Fix `Server` not defined in `CodegenInstance.h` * Restyled by clang-format * Remove unused headers * Fix `WifiRootNodeDevice` * Add needed context for `WifiRootNodeDevice` And rename `platformMgr` to `platformManager` * Make context in `RootNodeDevice` protected * Move `BreadcrumbTracker` into `Context` * Restyled by clang-format * Fix --------- Co-authored-by: Restyled.io <commits@restyled.io>
…n and Aggregate Type (#42732) * Add 'WirelessSignalType' wireless signal strength assessment or measurement type. * Add 'WirelessSignal' wireless signal type and strength assessment. * Replace 'int8_t rssi' with 'WirelessSignal signal'. This allows encoding not only the current type of quantitative dBm-style wireless signal measurements but also a qualitative, 0-100 wireless signal assessment. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the test RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type. * Update RSSI assignment to reflect 'WirelessSignal'. This updates the RSSI assignment to reflect the newly-introduced 'WirelessSignal' wireless signal type and strength assessment type.
Contributor
|
The check for |
* consolidate cleanup sites to use the shared CleanupSession(sessionId) * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Add missing logic --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* esp32: Add Ethernet support for ESP32-P4 * Use TEMPORARY_RETURN_IGNORED * add esp32p4 board in esp32 builder * update all_targets_linux_x64.txt * Update examples/all-clusters-app/esp32/sdkconfig.eth.esp32p4 Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update examples/all-clusters-app/esp32/sdkconfig.eth.esp32p4 Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * build ethernet * Update .github/workflows/examples-esp32.yaml Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update examples/all-clusters-app/esp32/sdkconfig.eth.esp32p4 Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Add port retry * Address CI * Address comments * clang * Add close to BLE * add unit tests * address CI * address comments * clang * clang * clang
…te (#42960) * Darwin: Avoid enum-compare-switch warning / error in extension template * zap regen
* Initial commit with `GetTestEventTriggerDelegate` removing * Restyled by clang-format * Update ՝RootNodeDevice՝ to pass ՝GetTestEventTriggerDelegate՝ to ՝GeneralDiagnosticsCluster՝ * Restyled by clang-format * Add `GeneralDiagnostics` cluster dependency injection to `Server` * Reduce dependency on cluster-specific headers in Server.cpp * Inject initialization timestamp through constructor instead of using `Server::GetInstance().TimeSinceInit()`. - Add GetInitTimestamp() to Server - Add initTimestamp parameter to GeneralDiagnosticsCluster - Add local TimeSinceInit() method using injected timestamp - Update all call sites and tests * Restyled by clang-format * Move initialization timestamp dependency injection from Server to GeneralDiagnosticsCluster * Restyled by clang-format * Add dummy `TestEventTriggerDelegate` for testing `GeneralDiagnosticsCluster` * Extract IntegrationDelegate creation into separate function MakeIntegrationDelegate and simplify test event trigger delegate * Restyled by clang-format * Make IntegrationDelegate static in MatterGeneralDiagnosticsClusterInitCallback and MatterGeneralDiagnosticsClusterShutdownCallback * Restyled by clang-format * Rename optionalAttributeSet to allOptionalAttributesSet in TestGeneralDiagnosticsCluster * Initialize Server::mInitTimestamp in constructor Ensure GetInitTimestamp() is valid before Server::Init() is called, fixing initialization order issue with RootNodeDevice. * Restyled by clang-format * Clear GeneralDiagnostics cluster state before shutting down clusters to prevent dangling pointers * Remove `Server::GetInitTimestamp()` and keep `Server()` constructor empty * Refactor GeneralDiagnosticsCluster to use node startup timestamp instead of init timestamp naming * Restyled by clang-format * Use TimeSinceNodeStartup instead of TimeSinceInit in EVSEManufacturer and update RootNodeDevice to include GeneralDiagnosticsServer CodegenIntegration * Restyled by clang-format * Add CodegenIntegration.h to general-diagnostics-server sources * Refactor code to improve formatting consistency and move global variables * Restyled by clang-format * Refactor CodegenIntegration.cpp to remove static variables * Restyled by clang-format * Update GeneralDiagnosticsCluster to use the current TestEventTriggerDelegate * Add CodegenIntegration to GeneralDiagnosticsCluster * Move GeneralDiagnostics cluster dependency injection setup to before data model provider initialization * Pass TimeSinceNodeStartup to HandleTimeSnapshotWithPosixTime instead of retrieving from cluster * Remove DummyTestEventTriggerDelegate and pass nullptr instead * Remove testEventTriggerDelegate parameter from PopulateCodeDrivenDataModelProvider * Restyle * Refactor GeneralDiagnosticsCluster to remove global variables * Refactor GeneralDiagnosticsCluster to pass testEventTriggerDelegate to Create * Remove CodegenIntegration.h from general-diagnostics-server sources * Pass TimeSinceNodeStartup to HandleTimeSnapshot instead of retrieving from cluster * Remove unused functions and includes from CodegenIntegration.h * Refactor CodegenIntegration to remove MakeIntegrationDelegate function * Restyled by whitespace * Restyled by clang-format * Simplify IntegrationDelegate initialization and configuration * Restyled by clang-format * Remove unused include * Simplify IntegrationDelegate initialization and configuration by removing unused members and using Server instance for required data * Restyled by clang-format * Replace local Server instance with Server::GetInstance() * Remove unused includes from CodegenIntegration.cpp * Restyled by clang-format * Remove unused include and simplify GeneralDiagnosticsCluster creation * Restyled by clang-format * Fix GeneralDiagnosticsCluster creation in RootNodeDevice * Restyled by clang-format * Decouple `GeneralDiagnosticsCluster` from global `GetDiagnosticDataProvider()` Use dependency injection for DiagnosticDataProvider instead of calling the global GetDiagnosticDataProvider() function. - Add DiagnosticDataProvider* parameter to constructor - Replace GetDiagnosticDataProvider() calls with member variable - Update all cluster creation sites to pass DiagnosticDataProvider * Restyled by clang-format * Refactor GeneralDiagnosticsCluster creation to simplify code and remove unnecessary casts * Reformat GeneralDiagnosticsCluster code for better readability and organization * Restyled by clang-format * Revert to TimeSinceInit() in EVSEManufacturerImpl * Revert Server to use mInitTimestamp instead of mNodeStartupTimestamp * Restyle * Restyle * Added null checking for mDiagnosticDataProvider and remove unused method * Initialize mTestEventTriggerDelegate member in constructor Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Added GetNodeStartupTimestamp() method documentation Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Verify mDiagnosticDataProvider is not null before use in GeneralDiagnosticsCluster * Spelling mistake Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Rename gServer and remove unused method * Restyled by clang-format * Typo correction Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Refactor event system to use monotonic startup time from Server * Restyled by clang-format * Update GeneralDiagnosticsCluster initialization * Fix startup timestamp fallback when event system is uninitialized * Remove unused include Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Remove `GetNodeStartupTimestamp` method * Remove unused include Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Refactor `GeneralDiagnosticsCluster` to use a `Context` struct for dependencies * Restyled by clang-format * Remove unused include Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Refactor GeneralDiagnosticsCluster initialization in Infineon platform * Remove unused include Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Remove unused include Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Remove ambiguous zero check for startup timestamp * Remove SetTestEventTriggerDelegate * Add nullptr check for InteractionModelEngine * Restyled by clang-format * Remove repeating dnssdServer from DataModelProvider configuration * Remove mNodeStartupTimestamp member, use event system timestamp directly * Update examples/all-devices-app/posix/main.cpp Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Make TimeSinceNodeStartup and GetTestEventTriggerDelegate protected * Restyled by clang-format * Remove unused node startup timestamp retrieval * Fix testEventTriggerDelegate initialization timing in all-devices-app Use initParams.testEventTriggerDelegate directly instead of calling Server::GetInstance().GetTestEventTriggerDelegate() before Server::Init(). This ensures the delegate is properly passed to GeneralDiagnosticsCluster when it's set in initParams before cluster creation. * Restyled by clang-format * Use Milliseconds64 for event timing precision Change GetMonotonicStartupTime() and TimeSinceNodeStartup() to return Milliseconds64 instead of Microseconds64, since microsecond precision is not available. Also move TimeSinceNodeStartup() implementation to .cpp file. * Restyled by clang-format * Fix TimeSinceNodeStartup calculation * Handle invalid startup time in GeneralDiagnosticsCluster * Update src/app/clusters/general-diagnostics-server/GeneralDiagnosticsCluster.cpp Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Andrei Litvin <andy314@gmail.com>
* changed expected values from 5c and 6c to 0 * Set to 1 the following PICS: DRLK.S.A0004=1 DRLK.S.A0005=1 DRLK.S.A0006=1 * Set maxValue to 255 in steps 11a, 12a and 13a * updated ci-pics-values, corrected step 31d, added missing step 32d * updated values for 40b, 42b, and updated PICs for steps 42a/b/c, 43a/b/c and 44a/b/c * Finally enabled the following PICs in both ZAP tool for lock-app and ci-pics-values DRLK.S.A0004=1 DRLK.S.A0005=1 DRLK.S.A0006=1 DRLK.S.A0027=1 DRLK.S.A002a=1 DRLK.S.A002c=1 and left the following PICs disabled DRLK.S.A0022=0 DRLK.S.A0028=0 DRLK.S.A0032=0 DRLK.S.A0035=0 DRLK.S.M.LocalProgrammingFeaturesAttributeWritable=0 * Gated steps 5a, 5b, 6a and 6b for DoorOpenEvents and DoorClosedEvents * Update src/app/tests/suites/certification/Test_TC_DRLK_2_1.yaml Co-authored-by: C Freeman <cecille@google.com> * automated steps 5c and 6c * added steps 5d, 5e, 6d and 6e * Fix review comments: revert maxValue to 253 and fix PICS logic - Steps 11a, 12a, 13a: Revert maxValue from 255 to 253 (0xFD) as per spec - Step 30g: Negate LocalProgrammingFeaturesAttributeWritable PICS so UNSUPPORTED_WRITE is only expected when attribute is NOT writable * [lock-app] Remove DPS attributes without DPS feature support Disabled DoorOpenEvents, DoorClosedEvents, and OpenPeriod attributes as lock-app does not implement DPS feature. Updated test and PICS accordingly. * Fixed PICS conditions for DPS attributes in steps 5x, 6x, 7x - Added DRLK.S.F05 feature check to all DoorOpenEvents, DoorClosedEvents and OpenPeriod test cases - Added negative test cases for unsupported attributes * Fixed steps 5x, 6x and 7x * applying suggestions * quotes --------- Co-authored-by: C Freeman <cecille@google.com> Co-authored-by: Andrey Khodyrev <khodyrev@google.com>
…arwin-specific arguments. (#42953) * Add ability to retry zap-cli and separate out darwin arguments. * A bit better logging. * Add break.
…er (#42963) * Update build version in cross compile docker to include gstreamer1.0-plugins-ugly. Signed-off-by: Raveendra Karu <r.karu@samsung.com> * Update cross compile docker version to 183. Signed-off-by: Raveendra Karu <r.karu@samsung.com> --------- Signed-off-by: Raveendra Karu <r.karu@samsung.com>
* Add a connection timeout timer to clean up the stale session. * Use CancelConnectionTimer instead of checking state * Address review comment * Schedule pending timer cleanup
Co-authored-by: Andrei Litvin <andy314@gmail.com>
* cleanup fabric table when deleting cached VideoPlayer * clang * add unit tests * clang * clang * clang * clang * fix CI * CI * CI
* Fixes camera clusters for tc-idm tests * Restyled by clang-format * Address comments --------- Co-authored-by: Restyled.io <commits@restyled.io>
… classes from global dependencies (#42979) * Decouple `Binding::Manager` and `Binding::PendingNotificationMap` * Restyled by clang-format * Fix namespace issue * Restyled by clang-format * Fix namespace issue 2 * make the instance of `BindingTable` to be member of the `BindingManager` * Restyle * Restyled by clang-format * Fix test * Fix * Comple error fix * Fix test 2 * Address comments * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: AniDashyan <ani.dashyan77@gmail.com>
…070) Bumps [cryptography](https://github.com/pyca/cryptography) from 44.0.1 to 46.0.5. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@44.0.1...46.0.5) --- updated-dependencies: - dependency-name: cryptography dependency-version: 46.0.5 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…on attempt (#42826) * wpa_supplicant: Retry when WLAN_STATUS_AUTH_TIMEOUT seen on association attempt Signed-off-by: Maciej Grela <m.grela@samsung.com> * Restyled by clang-format * Apply LLM suggestions, a bit more detailed debug, minor improvements Signed-off-by: Maciej Grela <m.grela@samsung.com> * wpa_supplicant: Initialize mAssociationRetriesLeft outside of the interface properties handler Signed-off-by: Maciej Grela <m.grela@samsung.com> * wpa_supplicant: Move mutex, drop std::atomic Signed-off-by: Maciej Grela <m.grela@samsung.com> --------- Signed-off-by: Maciej Grela <m.grela@samsung.com> Co-authored-by: Restyled.io <commits@restyled.io>
After moving the UserLabel and FixedLabel initialization to code-driven, we must change the initialization order for SetDeviceInfoProvider to ensure that it is intialized before Matter server. Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
* Remove duplicate runs ... there is no point to run both with and without debug. * Fix arguments.
483a5ca to
05d46e6
Compare
* Generate FixedLabel cluster with Alchemy * Add UserLabel
* Thread: disable security when Thread not started * Update GenericThreadStackManagerImpl_OpenThread_LwIP.cpp
…e VS doc (#43100) * Updating the test steps as per the VS doc * fix typo errors * Fixing small typo * Update Test_TC_DD_1_11.yaml * Restyled by whitespace * Restyled by prettier-yaml --------- Co-authored-by: Restyled.io <commits@restyled.io>
* created tc_bool_2_2 script with steps 1, 2a, 2b, 3a and 3b * created AllClustersAppCommandHandler::OnBooleanStateChangeHandler * added logs to OnBooleanStateChangeHandler and added naming condition to HandleCommand * Updated steps 2a, 2b, 3a and 3b. Communication through named pipe is ok * updated logs for boolean state change and updated manual steps 2a and 3a * completed steps 4a, 4b, 4c, 4d and 4e * refactored script, cleaned the code, created set_dut_state_value() and read_state_value_from_dut() * removed commented code for running manual, tested automatic and manual, ran isort * removed Test_TC_BOOL_2_2.yaml * Restyled by clang-format * added validations for JSON commands for SetBooleanState in HandleCommand * Removed harcoded path for the app pipe * Added recommended comments for is_pics_sdk_ci_only * Added PICS for CI * Update src/python_testing/TC_BOOL_2_2.py removing default=1 from self.get_endpoint() Co-authored-by: C Freeman <cecille@google.com> * Updated pics_TC_BOOLCFG_2_1(self) to pics_TC_BOOL_2_2(self) * updated pics BOOLCFG.S to BOOL.S * refactored script to remove duplicated steps * Fix RET504 ruff errors - return directly without unnecessary assignments * Remove TC_BOOL_2_2 from manualTests.json - test is now automated * Fix KeyError: Keep BooleanState category with empty array instead of removing it * Remove orphaned OccupancyPresentTimerHandler declaration This function declaration was left orphaned after the Occupancy Sensing Cluster was re-implemented as code-driven in commit fdc5697. The implementation was properly removed but the header declaration remained due to a merge conflict resolution issue. * Changed the decorator from has_attribute to has_cluster to better match the PICS. * fix imports * Restyled by isort * Add PICS gating for StateChange event in TC-BOOL-2.2 test * Refactored event support check into is_state_change_event_supported() method to allow future feature map validation. * Updated TC-BOOL-2.2 test steps: added attribute subscription and reorganized script into logical sections * Implemented attribute report verification in TC-BOOL-2.2 StateValue subscription * Update test to match test specs * Fix linter * Fix wait reports events * Add run all devices in CI * Adding CHGEVENT validation * Adding fifo file to args in CI arguments * Adding TODO comment * Resolve comments --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: C Freeman <cecille@google.com> Co-authored-by: Andrey Khodyrev <khodyrev@google.com> Co-authored-by: Ramiro Recchia <rrecchia@google.com>
…cripts for JFDS-2.3 and JFDS-2.4 (#42958)
* Decouple switch cluster * Restyled by clang-format * Code update * Restyled by clang-format * Restyled by prettier-json * Code update * Restyled by clang-format * Addressing review comments * Restyled by clang-format * Code update * Code update * Restyled by clang-format * Code update * Adding validations * Restyled by clang-format * Code update * Restyled by clang-format * Addressing review comments * Code update --------- Co-authored-by: Restyled.io <commits@restyled.io>
* esp32: Add all-devices-app to esp32 CI. * Remove duplicate entry Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
… that do not natively support u64 formating (#43121)
* Initial commit * Restyled by clang-format * Refactor ChimeCluster test setup * Reset branch to master state * Migrate ChimeCluster to use context.attributeStorage for attribute persistence * Fix ChimeCluster attribute loading * Refactor ChimeCluster attribute loading * Added comment * Update ChimeCluster attribute loading with error handling * Update ChimeCluster to use LogErrorOnFailure for attribute storage writes * Enhance ChimeCluster attribute loading with error logging for persistence failures * Update ChimeCluster logging to use detail level for persistence failure messages * Update ChimeCluster to return failure status for context validation in attribute setting * Update ChimeCluster logging to use error level for persistence failure messages * Update ChimeCluster to consistently validate context before attribute setting * Restore project state * Fix ChimeServer SEGV by using lazy cluster initialization Use LazyRegisteredServerCluster to defer ChimeCluster construction until Init() when SafeAttributePersistenceProvider is available. Prevents crash when ChimeServer is constructed before Server::Init(). * Restyled by clang-format * Add SafeAttributePersistenceProvider to main application files This update includes the addition of SafeAttributePersistenceProvider in both the ESP32 and POSIX main application files. * Restyled by clang-format * Refactor TestChimeCluster setup and teardown methods Removed the setting of SafeAttributePersistenceProvider in the TestChimeCluster's SetUp and TearDown methods to streamline the test initialization process. * Update TestChimeCluster to use ServerClusterContext for persistence initialization Modified the SetUp method in TestChimeCluster to initialize the SafeAttributePersistenceProvider using the ServerClusterContext instead of the TestContext, ensuring proper context usage during tests. * Simplify ChimeServer::GetEndpointId() to use stored mEndpointId * Add SafeAttributePersistenceProvider header to RootNodeDevice * Restyled by clang-format * Refactor ChimeCluster to use Context for dependency injection * Restyled by clang-format * Remove unused include Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Refactor ChimeCluster constructor to remove delegate parameter * Restyled by clang-format * Add null check for SafeAttributePersistenceProvider in ChimeDevice --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ion; fix discriminator subtype lookup (#42933) * Adds a TCP enabled version of the ci-pics-values file, added file: `ci-pics-values-tcp` * Fixes missing TXT record T key advertising for commissionable nodes when TCP is enabled * * Updates discriminator subtype presence verification to require an exact match, avoiding false positives when multiple discriminators or DUTs are present on the network * Updates TCP bit (0x04) verification to only assert when TCP is enabled * Updates CI test scenarios to enable TCP in the PICS (TCP is enabled in the app by default) * adds clarifying comment * removes dedundant T key verification * removes ci-pics-values-tcp, adds MCORE.SC.TCP=1 to ci-pics-values * removes ci-pics-values-tcp reference * adds MCORE.SC.TCP to PICS.yaml * removes unnecessary variable * updates T key verification logic * removes redundant T key bit 0 check * * absorb extra t key verifications into assert_valid_t_key * adds is_dut_tcp_supported to mdns network utils * updates assert_valid_t_key unit tests * log=false on txt record fetch * re-number steps * Restyled by ruff * updates TC-SC-4.3 as per T key verification fix * restores Dnssd. cpp/h * updates mdns class readme * Restyled by prettier-markdown * adds/refactors tcp support checks * re-order steps * adds dut tcp check for t key verifications * Restyled by autopep8 --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Adding new test TC_DGGEN_2_5 * Update Imports * Restyle * Update src/python_testing/TC_DGGEN_2_5.py Co-authored-by: C Freeman <cecille@google.com> * Update src/python_testing/TC_DGGEN_2_5.py Co-authored-by: C Freeman <cecille@google.com> * Update src/python_testing/TC_DGGEN_2_5.py Co-authored-by: C Freeman <cecille@google.com> * Update src/python_testing/TC_DGGEN_2_5.py Co-authored-by: C Freeman <cecille@google.com> * Update src/python_testing/TC_DGGEN_2_5.py Co-authored-by: C Freeman <cecille@google.com> * Update src/python_testing/TC_DGGEN_2_5.py Co-authored-by: C Freeman <cecille@google.com> * Update src/python_testing/TC_DGGEN_2_5.py Co-authored-by: C Freeman <cecille@google.com> * Remove else * Remove attribute guard step 4 and add step 9 * Updating fabric index * Typo th1 * Regenerate zap * Rephrase step 3 --------- Co-authored-by: C Freeman <cecille@google.com>
66492fc to
07f6efb
Compare
chirag-silabs
approved these changes
Feb 13, 2026
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.
This PR syncs the csa branch with the main branch.
PR MUST BE MERGED WITH MERGE COMMIT - ADMIN MUST ENABLE THE OPTION