mat/homeassistant-comwatt
Watch
1
0
Fork
You've already forked homeassistant-comwatt
 0
Code Issues 36 Pull requests 1 Projects Releases Packages Activity Actions

[Mineur] Manifest : integration_type/loggers absents, clés vides du template à supprimer #108

New issue
Open
opened 2026-08-26 05:33:48 +00:00 by mat · 0 comments
mat commented 2026-08-26 05:33:48 +00:00
Owner
Copy link

Problème / Recommandation — manifest.json

  1. "homekit": {}, "ssdp": [], "zeroconf": [] : valeurs par défaut de template → à supprimer (bruit).
  2. integration_type: "hub" absent : le défaut hassfest est hub, correct ici pour un compte cloud multi-appareils, mais à expliciter.
  3. loggers: ["comwatt_client"] absent : permet aux utilisateurs de filtrer les logs de la lib.
  4. iot_class: "cloud_push" : défendable grâce au flux WebSocket push (dominant côté temps réel), mais les sensors site sont du pur polling ~2 min — documenter le choix (note dans docs/) pour couper court aux remises en question.
  5. documentation pointe vers un blob README GitHub : valide hassfest, mais l'usage recommandé est une page dédiée stable.
  6. qualifiers (clé récente, ex. ["cloud", "energy"]) : optionnel.
  7. Penser à bump version vers 1.0.0 au moment du tag.

Constaté lors de la review de code pré-v1.0 (v0.8.1).

## Problème / Recommandation — `manifest.json` 1. `"homekit": {}`, `"ssdp": []`, `"zeroconf": []` : valeurs par défaut de template → à supprimer (bruit). 2. `integration_type: "hub"` absent : le défaut hassfest est `hub`, correct ici pour un compte cloud multi-appareils, mais à expliciter. 3. `loggers: ["comwatt_client"]` absent : permet aux utilisateurs de filtrer les logs de la lib. 4. `iot_class: "cloud_push"` : défendable grâce au flux WebSocket push (dominant côté temps réel), mais les sensors site sont du pur polling ~2 min — documenter le choix (note dans `docs/`) pour couper court aux remises en question. 5. `documentation` pointe vers un blob README GitHub : valide hassfest, mais l'usage recommandé est une page dédiée stable. 6. `qualifiers` (clé récente, ex. `["cloud", "energy"]`) : optionnel. 7. Penser à bump `version` vers `1.0.0` au moment du tag. --- Constaté lors de la review de code pré-v1.0 (v0.8.1).
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix-nan-energy
refactor/config-flow-tidy
refactor/shared-entity-base
refactor/drop-site-tuples
chore/release-0.5.0
docs/api-exploration
fix/46-value-guard
chore/release-0.4.0
7-problème-de-connexion-version-anterieure-a-gen4-2
update-python-comwatt
7-problème-de-connexion-version-anterieure-a-gen4
3-adjust-update-frequency-for-comwattenergysensor
9-enhancement-request-additional-device-control-modes-for-comwatt-integration
fix/power-sensor
fix/device-child-not-containing-id#10
4-optimize-api-client-handling-for-multiple-sensors
use-go.comwatt.com
v0.8.2
v0.8.1
v0.8.0
v0.7.0
v0.6.2
v0.6.1
v0.6.0
v0.5.2
v0.5.1
v0.5.0
v0.4.0
v0.3.2
v0.3.1
v0.3
v0.2.3
v0.2.2
v0.2.1
v0.2.0
v0.1.9
v0.1.8
v0.1.7
v0.1.6
v0.1.5.2
v0.1.5.1
v0.1.5
v0.1.4.1
v0.1.4
v0.1.3
v0.1.2
v0.1.1
v0.1
Labels
Clear labels   
v1.0 · bloquant
À corriger absolument avant la release v1.0

  
v1.0 · majeur
Fortement recommandé avant la v1.0

  
v1.0 · mineur
Polish / robustesse, non bloquant

No labels
v1.0 · bloquant
v1.0 · majeur
v1.0 · mineur
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
mat/homeassistant-comwatt#108
No description provided.