project-chip/connectedhomeip

Support "specification name" across more items within the SDK

Ouverte

#38 891 ouverte le 12 mai 2025

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C++ (1 659 forks)batch import
good first issuestale

Métriques du dépôt

Stars
 (6 410 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

The ZAP templates vary usage of asUpperCamelCase and asUpperCamelCase preserveAcronyms=true and code generation is inconsistent for acronym handling (e.g. C++ and darwin differ).

When preserveAcronyms makes a difference, we should preserve the XML name (with spaces and special characters and all) so that code generation can make a decision on how to encode things.

https://github.com/project-chip/connectedhomeip/pull/38888 adds support for constant names. We should go beyond that:

  • determine a good syntax throughout (for enums/bitmaps/struct/command names, structure member names, cluster names)
  • apply this syntax to update clusters. It is likely constants are the main ones affected though. May need to manually validate (e.g. using the Unit testing cluster)

Guide contributeur

Direction de recherche
Examinez l'utilisation actuelle de asUpperCamelCase et preserveAcronyms dans les modèles ZAP, puis proposez une syntaxe cohérente pour les noms de spécification dans les énumérations, bitmaps, structures, noms de commandes, noms de membres de structure et noms de clusters. Commencez par examiner la PR #38888 et le cluster de tests unitaires.
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.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Plus d'une semaine
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
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20