project-chip/connectedhomeip

Reduce/remove usage of "all ids/structs/enums" and include exact cluster dependencies

Open

#38 667 ouverte le 29 avr. 2025

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (1 659 forks)batch import
good first issuestale

Métriques du dépôt

Stars
 (6 410 stars)
Métriques de merge PR
 (Merge moyen 8j 5h) (375 PRs mergées en 30 j)

Description

We have split cluster specific data into separate files, however we still have usages of global things assumed within various locations. part of it is src/app/clusters which may need treating one by one, however others are the rest of the SDK code.

We should replace https://github.com/project-chip/connectedhomeip/blob/master/zzz_generated/app-common/app-common/zap-generated/cluster-objects.h usage and https://github.com/project-chip/connectedhomeip/blob/master/zzz_generated/app-common/app-common/zap-generated/cluster-enums.h usage with actual cluster/.../....h includes instead

and ensure references for ids include the correct header.

Practical example, WriteClient needs AccessControl::Id https://github.com/project-chip/connectedhomeip/blob/master/src/app/WriteClient.cpp#L273 and it should not assume that this is somehow available through transitive dependencies (it seems to do that now).

Need to figure out how to audit usages of these IDs and clean up references.

Guide contributeur

Direction de recherche
Examiner les inclusions actuelles dans src/app/WriteClient.cpp et les fichiers similaires pour identifier où les en têtes globaux comme cluster objects.h et cluster enums.h sont utilisés. Ensuite, mapper chaque ID, structure ou énumération utilisée à son en tête de cluster spécifique et remplacer l'inclusion globale par l'inclusion spécifique. Utiliser des outils comme grep ou des scripts pour auditer toutes les utilisations dans le dépôt.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50