[WWSTCERT-13446, WWSTCERT-13450, WWSTCERT-13454, WWSTCERT-13458] SONOFF/SWV1C by yanggx24 · Pull Request #3157 · SmartThingsCommunity/SmartThingsEdgeDrivers

yanggx24 · 2026-07-31T05:46:07Z

Check all that apply

Type of Change

WWST Certification Request
- If this is your first time contributing code:
  - I have reviewed the README.md file
  - I have reviewed the CODE_OF_CONDUCT.md file
  - I have signed the CLA
- I plan on entering a WWST Certification Request or have entered a request through the WWST Certification console at developer.smartthings.com
Bug fix
New feature
Refactor

Checklist

I have performed a self-review of my code
I have commented my code in hard-to-understand areas
I have verified my changes by testing with a device or have communicated a plan for testing
I am adding new behavior, such as adding a sub-driver, and have added and run new unit tests to cover the new behavior

Description of Change

Summary of Completed Tests

KKlimczukS · 2026-07-31T09:41:20Z

+    [capabilities.valve.ID] = {
+      [capabilities.valve.commands.open.NAME] = valve_open_handler,
+      [capabilities.valve.commands.close.NAME] = valve_close_handler,
+    }
+  },


Did You check if Your device works with the zigbe-valve driver default handling ? (without this custom handling)

KKlimczukS · 2026-07-31T09:54:58Z

+      [OnOff.ID] = {
+        [OnOff.attributes.OnOff.ID] = onoff_attr_handler
+      },


Did You check if Your device works with the zigbe-valve driver default handling ? (without this custom handling)

github-actions · 2026-07-31T09:55:40Z

Duplicate profile check: Passed - no duplicate profiles detected.

github-actions · 2026-07-31T09:55:40Z

Profile category check: ✅ Passed - all profiles have a category defined.

github-actions · 2026-07-31T09:56:19Z

Invitation URL:
https://bestow-regional.api.smartthings.com/invite/gV2qEO1Qp929

github-actions · 2026-07-31T09:57:14Z

zigbee-valve_coverage.xml

File	Coverage
All files	`99%`	✅
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-valve/src/sonoff/init.lua	`97%`	✅
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-valve/src/sinope/init.lua	`97%`	✅

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 30cf526

github-actions · 2026-07-31T09:57:20Z

Test Results

73 files 543 suites 0s ⏱️
3 263 tests 2 819 ✅ 0 💤 0 ❌ 444 🔥
5 256 runs 4 668 ✅ 0 💤 0 ❌ 588 🔥

For more details on these errors, see this check.

Results for commit 30cf526.

♻️ This comment has been updated with latest results.

yanggx24 · 2026-08-01T03:02:58Z

Thanks for the review.

Yes, I checked this behavior with the SONOFF SWV1C device. The device uses the Zigbee OnOff cluster for valve control and reports the valve state through the OnOff attribute.

The custom handlers were added to explicitly map:

valve.open / valve.close commands to Zigbee OnOff On / Off
OnOff attribute reports back to SmartThings valve.open / valve.closed events

This is also needed because the device reports the OnOff value as ZCL_DATA_TYPE_UINT8. In Lua, numeric 0 is truthy, so the handler explicitly checks for false and 0 to avoid treating an OFF report as open.

I can also re-test without the custom handlers and confirm whether the default zigbee-valve handling covers this device correctly.

ldeora · 2026-08-03T08:16:04Z

I can also re-test without the custom handlers and confirm whether the default zigbee-valve handling covers this device correctly.

Has this implementation been tested with the valve left open for longer than ten minutes?

Several Hydro ONE users have reported that the valve closes itself after approximately ten minutes because of its device-side manual watering duration. From the current PR, it appears that Open only sends the standard Zigbee On command and does not read or update the SONOFF 0xFC11 attribute 0x501D (manualDefaultSettings).

Could you confirm whether the tested devices remained physically open beyond ten minutes, and which model and firmware version were used? It would also be useful to know whether newer firmware has removed the default limit or whether the WWST integration is expected to configure 0x501D.

yanggx24 · 2026-08-03T10:22:53Z

I can also re-test without the custom handlers and confirm whether the default zigbee-valve handling covers this device correctly.

Has this implementation been tested with the valve left open for longer than ten minutes?

Several Hydro ONE users have reported that the valve closes itself after approximately ten minutes because of its device-side manual watering duration. From the current PR, it appears that Open only sends the standard Zigbee On command and does not read or update the SONOFF 0xFC11 attribute 0x501D (manualDefaultSettings).

Could you confirm whether the tested devices remained physically open beyond ten minutes, and which model and firmware version were used? It would also be useful to know whether newer firmware has removed the default limit or whether the WWST integration is expected to configure 0x501D.

Just now, I tested a 10-minute manual valve opening action, and after the timeout, the device reported an off status, and the UI on the app also updated accordingly.
The firmware version of the tested device is v 1.1.0.
This smartthings driver will not affect the automatic shutdown after 10 minutes. For this device, our plan is to first complete the WWST certification, and then provide more support for private attributes.

ldeora · 2026-08-03T10:40:23Z

I can also re-test without the custom handlers and confirm whether the default zigbee-valve handling covers this device correctly.

Has this implementation been tested with the valve left open for longer than ten minutes?
Several Hydro ONE users have reported that the valve closes itself after approximately ten minutes because of its device-side manual watering duration. From the current PR, it appears that Open only sends the standard Zigbee On command and does not read or update the SONOFF 0xFC11 attribute 0x501D (manualDefaultSettings).
Could you confirm whether the tested devices remained physically open beyond ten minutes, and which model and firmware version were used? It would also be useful to know whether newer firmware has removed the default limit or whether the WWST integration is expected to configure 0x501D.

Just now, I tested a 10-minute manual valve opening action, and after the timeout, the device reported an off status, and the UI on the app also updated accordingly.

To make it clear: the valve won't stay open for longer than 10 minutes, even when it was opened "manually" (Zigbee On command)?

yanggx24 · 2026-08-04T01:40:57Z

To make it clear: the valve won't stay open for longer than 10 minutes, even when it was opened "manually" (Zigbee On command)?

Yes.

aleclorimer · 2026-08-04T17:46:49Z

-  sub_drivers = require("sub_drivers"),
-  health_check = false,
-  shared_device_thread_enabled = true,


Add the new sonoff subdriver to the sub_drivers.lua file in this driver and use lazy loading.

aleclorimer · 2026-08-04T17:47:25Z

-  health_check = false,
-  shared_device_thread_enabled = true,


These should be kept

KKlimczukS · 2026-08-10T10:02:52Z

@yanggx24 Please update Your branch.

yanggx24 · 2026-08-11T03:05:24Z

@yanggx24 Please update Your branch.

I have merged the latest code, is there anything else missing?

yanggx24 · 2026-08-24T02:25:22Z

@KKlimczukS Hello,what else do I need to do？

yanggx24 · 2026-08-25T08:59:23Z

Hi @KKlimczukS @aleclorimer,

I have updated the branch with the Sonoff sub-driver lazy-loading changes, synchronized it with the latest main, and added the coverage test fix.

The device was tested with SONOFF SWV1C firmware v1.1.0. The 10-minute automatic shutoff is device-side behavior, and the private 0xFC11/0x501D support is outside this WWST certification PR.

Could you please re-review the latest commit and let me know if any blocking changes remain?

KKlimczukS

@yanggx24 Should this PR contain also changes from #3171 ?

aleclorimer · 2026-08-27T19:41:09Z

This smartthings driver will not affect the automatic shutdown after 10 minutes. For this device, our plan is to first complete the WWST certification, and then provide more support for private attributes.

I think this PR should include the changes required to configure private attributes if that's the plan for later.

yanggx24 · 2026-08-28T02:43:05Z

Thanks for the feedback.

I’d prefer to keep this PR limited to SWV1C. SWV2C is a different hardware product, maintained by a different developer, and its development timeline is separate from SWV1C. There is already a dedicated PR for SWV2C, so I don’t think it would be appropriate to merge the two devices into this submission.

The current SWV1C PR also does not include any private-attribute or custom-capability support. I would like to keep it that way. Private-attribute support is not part of this WWST submission.

This is also consistent with the SmartThings PR guidance, which does not allow custom Capabilities for WWST submissions and may reject drivers using custom Zigbee, Z-Wave, or Matter clusters:

For these reasons, I suggest keeping this PR focused on the standard SWV1C implementation, while handling SWV2C and any future private-attribute support separately.

feat:sonoff swv valve

d3ca759

KKlimczukS requested changes Jul 31, 2026

View reviewed changes

KKlimczukS reviewed Jul 31, 2026

View reviewed changes

KKlimczukS requested a review from cbaumler July 31, 2026 09:59

Zigbee Valve:modify comment language and file name

244e0ee

yanggx24 force-pushed the main branch from 53138de to 244e0ee Compare August 1, 2026 02:44

KKlimczukS mentioned this pull request Aug 3, 2026

[WWSTCERT-12824, WWSTCERT-12828, WWSTCERT-12832, WWSTCERT-12836] fingerprints for Sonoff valves #3027

Closed

13 tasks

cbaumler requested review from aleclorimer, cjswedes and nand-nor August 4, 2026 17:01

aleclorimer reviewed Aug 4, 2026

View reviewed changes

Zigbee Valve:Add Sonoff valve subdriver lazy loading

539f602

KKlimczukS changed the title ~~SONOFF/SWV1C~~ [WWSTCERT-13085, WWSTCERT-13089, WWSTCERT-13093, WWSTCERT-13097] SONOFF/SWV1C Aug 5, 2026

KKlimczukS requested a review from aleclorimer August 10, 2026 10:02

Merge branch 'SmartThingsCommunity:main' into main

caee9a2

Merge branch 'main' into main

b938a79

yanggx24 requested a review from KKlimczukS August 19, 2026 05:55

yanggx24 added 2 commits August 19, 2026 14:36

test: fix zigbee valve lazy loading coverage

3270a93

Merge branch 'main' into main

30cf526

KKlimczukS changed the title ~~[WWSTCERT-13085, WWSTCERT-13089, WWSTCERT-13093, WWSTCERT-13097] SONOFF/SWV1C~~ [WWSTCERT-13446, WWSTCERT-13450, WWSTCERT-13454, WWSTCERT-13458, WWSTCERT-13462, WWSTCERT-13465] SONOFF/SWV1C Aug 27, 2026

KKlimczukS changed the title ~~[WWSTCERT-13446, WWSTCERT-13450, WWSTCERT-13454, WWSTCERT-13458, WWSTCERT-13462, WWSTCERT-13465] SONOFF/SWV1C~~ [WWSTCERT-13446, WWSTCERT-13450, WWSTCERT-13454, WWSTCERT-13458] SONOFF/SWV1C Aug 27, 2026

KKlimczukS mentioned this pull request Aug 27, 2026

[WWSTCERT-13462, WWSTCERT-13465] Sonoff SWV2C valve support #3171

Open

12 tasks

KKlimczukS reviewed Aug 27, 2026

View reviewed changes

Uh oh!

Conversation

yanggx24 commented Jul 31, 2026

Type of Change

Checklist

Description of Change

Summary of Completed Tests

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KKlimczukS Jul 31, 2026

Choose a reason for hiding this comment

Uh oh!

KKlimczukS Jul 31, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jul 31, 2026

Uh oh!

github-actions Bot commented Jul 31, 2026

Uh oh!

github-actions Bot commented Jul 31, 2026

Uh oh!

github-actions Bot commented Jul 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Jul 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

yanggx24 commented Aug 1, 2026

Uh oh!

ldeora commented Aug 3, 2026

Uh oh!

yanggx24 commented Aug 3, 2026

Uh oh!

ldeora commented Aug 3, 2026

Uh oh!

yanggx24 commented Aug 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

aleclorimer Aug 4, 2026

Choose a reason for hiding this comment

Uh oh!

aleclorimer Aug 4, 2026

Choose a reason for hiding this comment

Uh oh!

KKlimczukS commented Aug 10, 2026

Uh oh!

yanggx24 commented Aug 11, 2026

Uh oh!

yanggx24 commented Aug 24, 2026

Uh oh!

yanggx24 commented Aug 25, 2026

Uh oh!

KKlimczukS left a comment

Choose a reason for hiding this comment

Uh oh!

aleclorimer commented Aug 27, 2026

Uh oh!

yanggx24 commented Aug 28, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

github-actions Bot commented Jul 31, 2026 •

edited

Loading

github-actions Bot commented Jul 31, 2026 •

edited

Loading

yanggx24 commented Aug 4, 2026 •

edited

Loading