[DNM] Gree Integration rewrite by p-monteiro · Pull Request #373 · RobHofmann/HomeAssistant-GreeClimateComponent

p-monteiro · 2025-09-23T20:20:16Z

Caution

This PR changes the domain of the integration, so current users will lose their device configs!
The config schema also changed, so verify your YAML if you use it to configure the devices.
This no longer overwrites the official integration.

This PR introduces a rewrite of this integration. The main goal of this was to better align the integration with the Home Assistant (HA) development workflow with two main changes:

Completely separate device API code from HA code
Align with code style guidelines (mainly Ruff code formatting and full typing)

Breaking changes

Major version bump
New domain for the integration
Configs have a different schema, devices need to be re-added
Translations need to be redone (can use the old ones, but need to be migrated to the new template)

Features

Device features are now user-configurable (automatic detection removed due to unreliable Gree API)
Async device communication via DataUpdateCoordinator
GreeDevice abstraction for device logic
Auto Encryption support
Expose all device sensors as entities
Add device fault detection support
Add diagnostics download for config entries and devices
Add option to set device temperature units (device display)
Add RestoreEntity support in ClimateEntity
Auto recovery of device IP

Improvements

Improve config flow with better validation and error handling
Allow reconfiguration of config entries (e.g., device IP changes)
Allow individual device removal
Make entity restore behavior configurable
Refactor Gree API to be fully declarative (remove string guessing)
Improve API structure and reuse of logic
Improve entity creation and organization

Fixes

Improve error handling across config flow and entity setup

Known problems:

Broken VRF Device Status (needs debugging, could be fixed later)

As you can see, there are a lot of changes, and I understand if this does not go through as is.

- Change config flow to: be multi step and allow setting the device features; allow to reconfigure an entry - Separate HA and Device API logic: device API is now declarative and handles device behaviour. HA entities expose the device - Implement a coordinator - Async device communication - More error handling

- Move consts around - Proper error report on config_flow - Proper translations on config_flow - Configuration of timeout - Fetch device info during binding

p-monteiro · 2025-09-24T21:05:16Z

@RobHofmann I was adding the temperature step back, but I'm not sure of its utility.
Is there any device that accepts non-integer ºC/ºF?

… and device comunication state

domialex · 2025-09-26T18:24:44Z

@p-monteiro
The temperature step should not be in the configuration entities anyways, it should be in the climate options. That was a mistake on my part.

p-monteiro · 2025-09-26T18:27:52Z

Hi @domialex, that's fine. I'm more interested in the real use case for it. Should it be locked to integers? I don't think any device supports half degrees or even 0.1 steps, as is the lower limit for the current slider...

p-monteiro · 2026-03-18T21:57:06Z

@ChillingSilence, packet capture would be the best way if you actually know how to do it. I'm not good at it.

ChillingSilence · 2026-03-18T22:13:40Z

It looks like it's still doing stuff on UDP port 7000? All the five packets are pretty much the same

The actual commands afterwards from "the cloud" seem to happen on TCP port 1982.

Is this helpful as a starting point thought @p-monteiro ?
Grateful for your assistance, thanks again 🙏

p-monteiro · 2026-03-18T22:19:35Z

@ChillingSilence It's performing the normal {"t":"scan"} broadcast. What is the device sending back after those?
src: 192.168.22.120 dst: 192.168.22.86

Also, I would prefer not to pollute this PR with this since it is unrelated to it. Open a new issue, and we can continue discussing there.

Adds a binary sensor that shows if there is a fault reported by the device

p-monteiro · 2026-03-26T16:46:33Z

@RobHofmann I'm down to get this merged into a different branch and published as an alpha/beta version. Fixes can then be made against that. I'll let you decide whatever you deem fit for the project. Remember, this is a breaking update.
The only thing I prefer to do extra is to drop yaml config as per HA guidelines (https://github.com/home-assistant/architecture/blob/master/adr/0010-integration-configuration.md#decision), but I understand if you are against it.

Refer to the original PR message (#373 (comment))

This workflow uses the version from the manifest to automatically create a tag and a release. Also prevents two consecutive releases with the same version

Use proper types V2 decrypt and verify combined Debug trimmed garbage (may be useful)

p-monteiro added 23 commits September 23, 2025 20:24

Add support for external temperature and humidity sensors

187ef3c

Make manifest compliant with HA requirements

1188118

Cleanup old code in preparation for backporting features

c71392f

Use encryption version as enum and cleanup

73ebfe2

Multiple config_flow enhancements:

e0d9886

- Move consts around - Proper error report on config_flow - Proper translations on config_flow - Configuration of timeout - Fetch device info during binding

Listen to external sensor states if using them

74809e9

Try to fix external entities not being unselectable

f088882

Reimplement Auto Light and X-Fan

9a78e8f

Remove timeout from the coordinator

29cbd31

Feature: Restore entity states configurable by user

97201de

Change set_fan debug

03b9597

Fix auto x-fan logic

98ce923

Improve Temperature Units select and restore state

5425b80

Realign with main branch for easier migrations

8686f44

Add API support for device discovery

4f0876b

Gree API: Reuse pack decryption code

ce79013

Fix translations

3adef52

Fix imports

f779529

Bring back add device by discovery

3a9f0b2

Fix Temperature Unit selector

034c93d

Fix current temperature units in Climate Entity

6ff856d

Request a update when system units are changed

4622516

p-monteiro added 3 commits September 24, 2025 23:41

Implement RestoreEntity in GreeClimate

fd5a17e

Use _attr_ instead of the cached property

e013a19

Implement availability check based on coordinator data update success…

58856aa

… and device comunication state

Improve README

6181526

p-monteiro added 4 commits March 24, 2026 14:51

Revert fetch_device_info tests and prevent device key in logs

6ef40a1

Add initial support for Fault Detection feature

6951d75

Adds a binary sensor that shows if there is a fault reported by the device

Code cleanup

21f74e1

Update readme on UDP protocol requirement

fb7d5a1

p-monteiro added 22 commits March 26, 2026 23:35

Fix config timeout and use full mac

d52318e

Updated PT translation

54ea633

Change mac address naming to clarify functions

fd95bb5

Fix mac split variable

9e79d2a

Improve device removal and use only device mac for coordinator

cde3856

Fix entity creation, availability and restore configs

28ee5eb

Add device polling rate to the config

a5722f6

Align with recommended HACS actions

bd11d65

Add hacs.json and required manifest fields

dc3f7c3

Only validate master branch

0f37e8f

Update Readme

d66e08d

Add a release workflow

f527344

This workflow uses the version from the manifest to automatically create a tag and a release. Also prevents two consecutive releases with the same version

Readme typo

51b221e

Improve issue templates

39de394

Improve cipher code

73c86fc

Use proper types V2 decrypt and verify combined Debug trimmed garbage (may be useful)

More cipher cleanups

be338c8

Correct info

7f8a562

Proxy update_state through coordinator

10cf27b

Add capability of self correcting IP

fdcd2da

Bump version

657882a

Fix auto IP recovery config save and lower timeouts

90cbb0a

Add clamping to temperature helpers

0840276

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[DNM] Gree Integration rewrite#373

[DNM] Gree Integration rewrite#373
p-monteiro wants to merge 110 commits intoRobHofmann:masterfrom
p-monteiro:gree-rewrite

p-monteiro commented Sep 23, 2025 •

edited

Loading

Uh oh!

p-monteiro commented Sep 24, 2025

Uh oh!

domialex commented Sep 26, 2025

Uh oh!

p-monteiro commented Sep 26, 2025

Uh oh!

p-monteiro commented Mar 18, 2026

Uh oh!

ChillingSilence commented Mar 18, 2026

Uh oh!

p-monteiro commented Mar 18, 2026

Uh oh!

p-monteiro commented Mar 26, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Conversation

p-monteiro commented Sep 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Breaking changes

Features

Improvements

Fixes

Known problems:

Uh oh!

p-monteiro commented Sep 24, 2025

Uh oh!

domialex commented Sep 26, 2025

Uh oh!

p-monteiro commented Sep 26, 2025

Uh oh!

p-monteiro commented Mar 18, 2026

Uh oh!

ChillingSilence commented Mar 18, 2026

Uh oh!

p-monteiro commented Mar 18, 2026

Uh oh!

p-monteiro commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

p-monteiro commented Sep 23, 2025 •

edited

Loading

p-monteiro commented Mar 26, 2026 •

edited

Loading