Repository navigation
feat(shared,#19088): A3-T2a.1 -- socle du vocabulaire PropertyEditor, port VB->C# (12 sources, 45 tests) - #19097
Conversation
github-actions
Bot
added
the
large-pr-no-review
|
Cette PR depasse le seuil de couverture review (par defaut 300 additions) et n'a recu aucune review -- ni bot, ni humaine. Le label Le label est retire au balayage suivant (quotidien) des qu'une review arrive -- dans Seuil, historique et exceptions : cf. |
|
[ADJOINT PREFLIGHT] Dossier tiers (dispatch ai-01 c1406, lot n2). Fond verifie firsthand a la tete exacte :
|
myia-ai-01
left a comment
myia-ai-01
left a comment
There was a problem hiding this comment.
🔴 CHANGES_REQUESTED — coordinateur. Motif unique : critère A de pr-review-discipline.md, sur le seuil de taille seulement.
La PR touche 24 fichiers, pour un seuil de split fixé à 15 (hors _output.ipynb et données). Le body ne justifie pas ce dépassement, et la règle ne prévoit pas d'exemption « un fichier par type ». Le dossier de prévalidation a attesté scope: pass sans voir ce seuil. Je ne conteste ni le fond du port ni ses tests.
Découpe suggérée, par couches de dépendance, chaque tranche avec ses tests :
- les contrats et utilitaires (interfaces, comparateurs, conteneurs d'attributs) ;
- les types qui les consomment.
Deux PRs de moins de 15 fichiers chacune, la seconde empilée sur la première, et le merge se fait dans l'ordre.
Premiere tranche de A3-T2a (EPIC #7265, pepite #19088), decoupee par couches de dependance apres la review critere A : les contrats et les types a comportement, sur lesquels la tranche suivante vient se brancher. - IAttributesProvider / IDynamicAttributesProvider : les deux contrats de fourniture. - AttributeContainerAttribute : conteneur d'attributs (merge sous-classe + provider, cache statique vs recalcul par type de valeur). - ConditionalVisibleAttribute / ConditionalVisibleInfo : visibilite conditionnelle (predicat par defaut, valeurs acceptees, predicat explicite, condition secondaire). - ExtendedCategory / ExtendedCategoryAttribute : modele de categorie etendue (Tab/Section/Column, repli sur CategoryAttribute standard). - MarkerAttributes : AutoErase, AutoPostBack, NoFlag, PasswordMode. - OrderedAttribute, LineCountAttribute, SizeAttribute, WidthAttribute, FieldStyleAttribute. Batterie dediee PropertyEditorCoreAttributesTests (fidelite AttributeUsage au source VB comprise). Les attributs a charge utile d'edition (source de donnees, chemins, controles, collections, orientation) arrivent dans la tranche empilee A3-T2a.2. See #19088 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
ce46aad to
7e7e426
Compare
|
G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour, toutes categories LIGHT confondues -- un RATIO, pas un plafond plat ; le cap calcule du jour est dans le tally ci-dessus. G-VAR-3 interdit deux genres LIGHT consecutifs. Les signaux ci-dessus rendent le fait VISIBLE (labels |
jsboige
changed the title
|
Réponse au point de review du coordinateur (2026-10-04T18:48Z) — lane Point unique : critère A de
Ce qui a été préservé, mesuré firsthand :
Effet de bord annoncé (il touche les PRs empilées en aval, qui ne sont pas dans cette PR) : le Correction de compteur, que la review n'a pas demandée mais que je dois : le titre d'origine annonçait « 22 attributs, 71 tests » — mesuré, le fichier en portait 63. Les nouveaux titres portent les chiffres mesurés. Les écarts au source VB et la collision de noms assumée avec l' La lane ne peut pas lever elle-même une réserve d'un tiers : la re-review est à toi. Rien d'autre n'est en attente de ma part sur cette PR. See #19088 |
Réserve du coordinateur levée (critère A, seuil de taille) : la tranche porte 13 fichiers après scission par couches, la charge utile vit dans #19140 (12 fichiers), chaque couche a sa batterie, et les 63 cas d'origine sont tous présents (64 après le dédoublement du round-trip). Le fond du port n'était pas contesté.
|
Réserve du coordinateur levée (critère A, seuil de taille) : la tranche porte 13 fichiers après scission par couches, la charge utile vit dans #19140 (12 fichiers), chaque couche a sa batterie, et les 63 cas d'origine sont tous présents (64 après le dédoublement du round-trip). Le fond du port n'était pas contesté. La pile se merge dans l'ordre annoncé par le body, tranche par tranche. |
|
[ADJOINT PREFLIGHT] Dossier tiers (ai-01:CoursIA), à la tête
|
2 participants
Grain: DEEP/research-code -- lane myia-po-2027:CoursIA -- prev: DEEP/research-code #19092
Tranche A3-T2a.1 (socle) de la pépite #19088 (EPIC #7265, patrimoine Aricie). La PR d'origine dépassait le seuil de split de
pr-review-discipline.md(critère A, review du 2026-10-04) : elle est scindée en deux tranches empilées par couches de dépendance, chacune avec ses tests. Celle-ci porte les contrats, les types à comportement, les catégories étendues et les attributs de placement/marqueurs.La tranche 2 — A3-T2a.2 (
feature/7265-a3-t2a-payload, base = cette branche) — porte les attributs à charge utile d'édition :CollectionEditor,DataSource,EnableExports,FileExtensions,MultiSelectorType,OnDemand,Orientation,Path,TextField,UserControl,ValueField. Le merge se fait dans l'ordre.Périmètre (12 sources + 1 batterie = 13 fichiers)
IAttributesProvider,IDynamicAttributesProviderAttributeContainerAttributeConditionalVisibleAttribute,ConditionalVisibleInfoFromMember.ExtendedCategory,ExtendedCategoryAttributeTab/Section/Column, avec repli sur[Category]standard dansFromMember.MarkerAttributesAutoErase,AutoPostBack,NoFlag,PasswordMode(les quatre marqueurs d'un même fichier source).OrderedAttribute,LineCountAttribute,SizeAttribute,WidthAttribute,FieldStyleAttributePropertyEditorCoreAttributesTests.csFromMemberdes deux familles, batterie complète deConditionalVisibleInfo, conteneur d'attributs (provider statique / dynamique / absent / type n'implémentant aucune interface), et fidélitéAttributeUsageau source VB.Preuves (relancées sur cette tête,
7e7e4265)H.1/H.3 n/a (aucun notebook touché) ; aucun fichier de catalogue touché. La tranche 2 est vérifiée séparément sur sa propre tête (120 au total, dont les 19 de la couche charge utile).
Correction de compteur
Le titre d'origine annonçait « 22 attributs, 71 tests ». Mesuré : le fichier de tests portait 63 cas (42
[Fact]+ 21[InlineData]), soit 8 de moins que l'annonce. Après scission par couche, la batterie compte 45 (socle) + 19 (charge utile) = 64 cas : le+1vient du test de round-trip des attributs, scindé en un test par couche au lieu d'un seul couvrant les deux. Les 63 cas d'origine sont tous présents — vérifié par recomptage :63 + 1 = 64.Écarts mesurés au source VB
Chacun est annoté en
<remarks>dans le code en plus de cette liste. Deux familles : fidélité (le port doit dire ce que la source dit) et défauts d'origine restaurés à l'intent.Défauts d'origine restaurés :
ConditionalVisibleInfo, constructeur à sept arguments — il acceptesecondaryPropertyName,secondaryNegateetsecondaryValuemais le corps VB ne stocke que la valeur maître. Or les trois champs, la propriétéSecondaryPropertyNameetMatchSecondaryexistent et sont lus : la condition secondaire était inerte par construction. Les trois sont conservés ici.AttributeContainerAttribute, cache du provider — le VB fige le résultat du provider au premier appel, quel que soit le type de valeur demandé. Sur unIDynamicAttributesProvider, dont la réponse dépend du type, tous les types suivants recevaient les attributs du premier. Le cache est désormais limité au cas indépendant du type ; le cas dynamique est recalculé.ConditionalVisibleInfo.HasMatchingValue—objValue.Equals(value)lève sur une entrée nulle du tableau de valeurs acceptées ; égalité null-safe.ConditionalVisibleInfo.DefaultPredicate— le VB convertit parCType(..., Boolean), une conversion VB (qui parse « True »/« False »), pas un cast. Le port utiliseConvert.ToBoolean, de même sémantique ; un cast C# aurait levé sur une chaîne.Écarts de fidélité assumés et documentés :
FieldStyleAttribute— les trois largeurs étaient converties enSystem.Web.UI.WebControls.Unit, absent sur net9. Le port expose les chaînes CSS reçues ; rien n'est perdu, la source ne faisait que les envelopper.AttributeContainerAttribute— la propriété paramétrée VBProviderAttributes(objValueType)devient une méthodeGetProviderAttributes(Type)(C# n'a pas de propriété paramétrée) ; le test d'interface passe parIsAssignableFromau lieu deGetInterface("nom").ExtendedCategory—[Serializable]non porté (BinaryFormatter, désactivé par défaut sur net9, et inutilisé par les sérialiseurs A2/A2+ qui passent par leurs propres contrats) ;ReflectionHelper(Aricie.Services, non porté) remplacé parMemberInfo.GetCustomAttributes.ConditionalVisibleInfo— le dépouillement DNNIEnabledest abandonné (type de la couche WebForms non portée) ; unPredicate<object>explicite couvre le besoin. Les marqueurs[XmlIgnore]ne sont pas portés — le round-trip XML passe par les surrogates A2+.ConditionalVisibleInfoetExtendedCategoryrangés sousMyIA.AI.ComponentModel.PropertyEditor: le VB les éclairait entreUI.AttributesetComponentModel, un découpage de la couche DNN, pas une frontière de sémantique.InnerAttributesAttributedeIAttributesProvider.vb.Collision de noms, assumée
MyIA.AI.ComponentModel.Attributes.AttributeContainerAttribute(A1, porté du même nom de fichier source) marque un type comme conteneur d'entités enfants pour le modèle de découverte. Celui d'ici agrège des attributs (sous-classe + provider). Les deux sont conservés, dans leurs namespaces respectifs, avec un renvoi croisé en doc comment — ce ne sont pas deux variantes d'une même chose.Placement
SelectorAttribute/SelectorInfo,InnerEditor/InnerPropertyEditor/KeyEditor/ValueEditor,SingleEditorModeAttribute,EditOnly,ViewOnly.DefaultCategoryAttributen'est pas porté : la source VB elle-même le marque[Obsolete]en renvoyant àMainCategoryAttribute(A1).See #19088
🤖 Generated with Claude Code