Skip to content

Add roborock action Set Vacuum Zoned Cleaning#45686

Draft
Thomas55555 wants to merge 1 commit into
home-assistant:nextfrom
Thomas55555:roborock_zoned_cleaning
Draft

Add roborock action Set Vacuum Zoned Cleaning#45686
Thomas55555 wants to merge 1 commit into
home-assistant:nextfrom
Thomas55555:roborock_zoned_cleaning

Conversation

@Thomas55555
Copy link
Copy Markdown
Contributor

Proposed change

Add description for Roborock zoned cleaning action

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Copilot AI review requested due to automatic review settings June 1, 2026 19:23
@Thomas55555 Thomas55555 requested a review from allenporter as a code owner June 1, 2026 19:23
@home-assistant home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Jun 1, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 1, 2026

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit d8bc727
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6a1ddc2c50190000083486d7
😎 Deploy Preview https://deploy-preview-45686--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds documentation for starting a Roborock zoned-clean action in the integration docs.

Changes:

  • Documented the new roborock.set_vacuum_zoned_cleaning action and its required parameters.
  • Added a YAML example demonstrating how to call the action with coordinates and repeat count.

Comment on lines +321 to +323
- **Data attribute**: `repeats`
- **Description**: Number of times to clean the zone.
- **Optional**: No.

##### Action Set Vacuum Zoned Cleaning

The `roborock.set_vacuum_zoned_cleaning` action will start cleaning a
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please remove the line length limit and apply the flowing text style.

Comment on lines +321 to +323
- **Data attribute**: `repeats`
- **Description**: Number of times to clean the zone.
- **Optional**: No.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- **Data attribute**: `repeats`
- **Description**: Number of times to clean the zone.
- **Optional**: No.
- **Data attribute**: `repeats`
- **Description**: Number of additional times the zone cleaning is repeated. `0` cleans the zone once. Allowed values are `0`, `1`, or `2`.
- **Optional**: No.

I think Copilot has a point there. Maybe something like this? Please edit if I got it wrong.

Comment on lines +303 to +304
You can use the `roborock.get_vacuum_current_position` action to help determine
the coordinates for the zone.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
You can use the `roborock.get_vacuum_current_position` action to help determine
the coordinates for the zone.
You can use the `roborock.get_vacuum_current_position` action to help determine
the coordinates for the zone. The coordinates are in the vacuum's internal map coordinate system (the same one used by `roborock.get_vacuum_current_position` and `roborock.set_vacuum_goto_position`). The dock is located near `(25500, 25500)`. `(x1, y1)` and `(x2, y2)` are two opposite corners of the rectangular zone.

Would it be worth adding some context here about the coordinate system?

y1: 21363
x2: 27425
y2: 22816
repeats: 0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
repeats: 0
# Clean the zone once, then repeat it one more time (two passes total).
repeats: 1

Maybe change it to 1, with an explainer?

@home-assistant home-assistant Bot marked this pull request as draft June 2, 2026 04:14
@home-assistant
Copy link
Copy Markdown

home-assistant Bot commented Jun 2, 2026

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants