Hola,
Tras una solicitud sobre la integración externa, he subido estas 3 nuevas informaciones:
Claude había señalado efectivamente el punto bloqueante:
El punto que bloqueaba
El CLAUDE.md decía que NO₂, O₃ y SO₂ no tienen categoría de concentración en Gladys y que solo su subíndice era publicable.
¿Es posible añadir estas categorías de funcionalidad del dispositivo al mismo nivel que PM10 y PM2.5 por favor?
Aquí están los diferentes umbrales dados por la IA:
Para Gladys, lo haría aún más simple
Si tu objetivo es activar escenarios domóticos, pondría 4 umbrales :
NO₂
< 40 µg/m³40–100100–200> 200
O₃
< 100 µg/m³100–160160–240> 240
SO₂
< 40 µg/m³40–100100–300> 300
Gracias
La PR está aquí:
master ← prohand:claude/air-quality-no2-o3-so2-72g6vy
ouvert 08:49AM - 11 Aug 26 UTC
Air quality devices report gaseous pollutant concentrations next to the particul… ate ones, but Gladys could only model PM2.5 and PM10. Add three sibling categories holding the raw mass concentration of NO2, O3 and SO2, with the same plumbing as the particulate categories:
- constants: the three categories with their scope comment, and the mass concentration units they accept (mg/µg/ng per cubic meter)
- front: icon, English/French/German translations, climate history group, MQTT catalog defaults (µg/m³, 0-1000, read only) and preview values
- dashboard badge: a four-level color helper, so the badge turns green / yellow / orange / red on the guideline thresholds of each gas (NO2 40/100/200, O3 100/160/240, SO2 40/100/300 µg/m³)
- MQTT: the matching Home Assistant discovery device classes
- HomeKit: the NitrogenDioxideDensity, OzoneDensity and SulphurDioxideDensity characteristics, merged into the shared AirQualitySensor service
- MCP: the categories join the sensor and history feature lists
Claude-Session: https://claude.ai/code/session_01Lvz85g8gzN3eBXRYnBuDGD
### Description
## Forum
### Checklist
- [ ] Tests pass: `cd server && npm run coverage` (Codecov requires 100% coverage on changed lines) and Cypress (`npm run cypress:run`) if the UI changed
- [ ] Linter and prettier pass on both front and server (`npm run eslint`, `npm run prettier`)
- [ ] No undocumented breaking change
## Summary by CodeRabbit
* **New Features**
* Added support for NO₂, O₃, and SO₂ air-quality sensors.
* Displayed localized pollutant labels, units, icons, previews, value ranges, and color-coded readings.
* Added sensor history and configuration support.
* Integrated pollutant readings with HomeKit and Home Assistant.
* **Tests**
* Added coverage for unit conversion, sensor characteristics, and air-quality notifications.
Y el feedback de claude al final del desarrollo:
Dos puntos a conocer
Zigbee2MQTT no está cableado: no expone una propiedad estándar para estos gases, no quise inventar un nombre de clave.
Y está probado con dispositivos MQTT
@pierre-gilles
¡Genial!
Dile simplemente a Claude « Hay comentarios sobre la PR » en la conversación donde hiciste el desarrollo, y Claude lee todos los comentarios, responde a los mensajes por sí mismo, asesora sobre los comentarios a hacer, e implementa solo si está de acuerdo con los comentarios (¡no todos los comentarios son relevantes, es un debate!).
@prohand Lo he revisado y está bien para mí, ¡gracias por la PR! Ya está fusionado.
pierre-gilles:
Dile simplemente a Claude « Hay comentarios sobre la PR » en la conversación donde hiciste el desarrollo, y Claude lee todos los comentarios, y por sí mismo responde a los mensajes, asesora sobre los comentarios a hacer, e implementa solo si está de acuerdo con los comentarios (no todos los comentarios son pertinentes, ¡es un debate!)
Sin problema, lo haré la próxima vez
Gracias
Cierro este post porque la funcionalidad ya está disponible en Gladys Assistant 4.86:
Salut à tous !
Super heureux de sortir Gladys Assistant 4.86.
C’est simple, c’est la plus grosse version depuis le début du projet, avec 68 PR fusionnées en une semaine
