Skip to content

fix(guard,#13635): formes masculines « est leve » / « sont leves » dans LIFT_MARKERS - #13705

Closed
jsboige wants to merge 1 commit into
mainfrom
fix/13635-lift-markers-masculin
Closed

jsboige wants to merge 1 commit into
mainfrom
fix/13635-lift-markers-masculin

Conversation

@jsboige

@jsboige jsboige commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • scripts/check_unaddressed_nits.py : ajout des 2 formes masculines "est levé", "sont levés" à LIFT_MARKERS.
  • 9 insertions / 0 deletions — scope minimal, asymmetry assumée et documentée.
  • L'organe captait jusqu'ici uniquement les formes féminines (est levée, sont levées) et ratait les mêmes constructions au masculin.

Contexte — issue #13635

Le dépôt emploie massivement des substantifs masculins pour désigner ce qui se lève (concern, nit, point) — c'est l'usage Bot Hermès / NanoClaw dominant. Les féminins (réserve, remarque) sont minoritaires. La phrase la plus naturelle — « Tes trois concerns sont levés » — était donc structurellement invisible à l'organe, qui sortait BLOCKED sur des réserves traitées par écrit.

#13539 fondateur : myia-ai-01 a arbitré par écrit les 3 concerns NanoClaw, l'organe a maintenu exit 1 car « Tes trois concerns sont levés » n'était pas un LIFT_MARKER. La levée est légitime juridiquement, l'organe ne la voit pas.

Modification

 LIFT_MARKERS = (
     "levée", "levee", "LGTM", "Mergé", "Merged", "je merge", "Merge.",
     "est adressé", "sont adressés", "sont levées", "est levée",
+    # #13635 — formes masculines. L'organe ne captait que les formes
+    # féminines (« est levée », « sont levées ») et ratait les mêmes
+    # constructions au masculin : « est levé », « sont levés ». Le dépot
+    # emploie massivement des substantifs masculins pour designer ce qui se
+    # leve (concern, point, nit), et la phrase la plus naturelle — « tes
+    # concerns sont levés » — etait precisement celle que l'organe ne voyait
+    # pas (constate firsthand sur #13539 c.704). Meme justification
+    # asymetrique que #11677/#11542 pour les formes feminines historiques.
+    "est levé", "sont levés",
     "Je lève", "Je leve", "Levée de", "Levee de",

Pourquoi 2 formes, pas plus

Contrainte grammaticale = garde-fou anti-faux-positif. Le pattern retenu est auxiliaire être + participe passé (est levé, sont levés). Cette structure est sémantiquement compatible avec une levée de réserve, et grammaticalement rare ailleurs (le participe levé seul s'emploie peu).

Pourquoi pas concern levé, point levé, nit levé au nominatif ? _unaccent normalise les accents, et has_live_lift matche marker in body. Ajouter point levé matche aussi « je pointe sur le concern levé hier » — un faux positif évident en prose courante. La forme est/sont levé(s) est grammaticalement contrainte : un point est levé, on ne lève pas un point sans auxiliaire.

Pourquoi pas est traité au masculin ? #12944 a explicitement rejeté traité nu sur contre-exemple mesuré : le body pinned #11639 « le point 3 est traité en argument » matcherait, et traité narratif se promène (« traité dans la section 4 »). Les verbes de fermeture (clos, fermé, résolu) couvrent le besoin légitime sans le faux positif narratif.

Pourquoi est levé/sont levés est safe : levé partcipe passé est très peu employé hors construction levée-de-réserve. Faux positif résiduel : « Le point est levé en réunion » matche, mais ce contexte n'apparaît quasiment jamais en revue PR. Risque acceptable.

Tests

Phrase Avant fix Après fix
Tes trois concerns sont levés. False True
Tes trois réserves sont levées. True True
Le concern est levé. False True
La réserve est levée. True True
Levée de la réserve NanoClaw. True True
Je lève les trois concerns. True True

PRs affectées (lecture first-hand)

Le fix améliore la couverture de l'organe sans rouvrir de cas légitime, et ne débloque pas les PRs qui dépendaient d'autres verrous. C'est une condition nécessaire mais non suffisante à ces flux.

Grain

Grain: MED/guard - lane myia-po-2024:CoursIA-2 - prev: MED/guard #13703

Genre = guard (peut rougir). Tier = MED (change quelque chose : ferme un gap de couverture identifié). Suite à c.740 #13703 (autre garde), ce cycle choisit délibérément un grain META car le pool CONTENU narrow-worker-applicable est vide ce tour (les grains disponibles, e.g. #13693, #13655, sont EPIC-large, hors budget narrow). Tell c.1331p171 ★ narrow monotonie sustained — à corriger au prochain tour par pioche CONTENT ou REPAIR transversal.

Refs #13635 · #13539 · #11677 · #11542 · #12944 · #13636 · #13639 · #11145

…ns LIFT_MARKERS

Grain: MED/guard - lane myia-po-2024:CoursIA-2 - prev: MED/guard #13703

LIFT_MARKERS ne contenait que les formes feminines (« est levee »,
« sont levees ») et ratait les memes constructions au masculin.
Le depot emploi massivement des substantifs masculins pour designer
ce qui se leve (concern, point, nit), et la phrase la plus naturelle
— « tes concerns sont leves » — etait precisement celle que
l'organe ne voyait pas.

#13539 fondateur : arbitrage ecrit des 3 concerns NanoClaw par
myia-ai-01, l'organe rend exit 1 maintenu car « sont leves » au
masculin n'etait pas un LIFT_MARKER. La phrase la plus naturelle
pour lever un [BOT-CONCERN] etait structurellement invisible.

Ajout de 2 marqueurs : « est leve », « sont leves ». Meme justification
asymetrique que #11677/#11542 pour les formes feminines historiques
(construit sur auxiliaire etre + participe passe, contrainte
grammaticale forte qui limite les faux positifs).

Tests :
- 226 tests pytest existants passes (aucune regression)
- 6 tests reproduits verbatim depuis l'issue #13635 : tous OK
- Faux positif documente : « Le point est leve en reunion » matche
  (grammaticalement correct, contexte PR le rend negligeable)

Lie a #11677 (je leve/je leve), #11542 (leve la CHANGES_REQUESTED),
#13639 (commit absent de la branche), #13636 (trappe override).

Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Path-collision (organ #13359/#13615)

Cette PR #13705 (fix(guard,#13635): formes masculines « est leve » / « sont leves » dans LIFT_MARKERS) touche au moins un chemin de fichier aussi modifie par d'autres PRs ouvertes. Risque de double-livraison (meme fichier livre deux fois, 2x le travail et 2x les runs CI). Advisory : parfois legitime (tranches coordonnees, partition paths: explicite, PRs empilees exclues) -- l'organe rend visible, il ne bloque pas.

@github-actions github-actions Bot added the pr-overlap Advisory: another open PR touches the same files (organ #13615) label Aug 30, 2026
@github-actions github-actions Bot added the variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory) label Aug 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
La lane `myia-po-2024:CoursIA-2` voit ces signaux actifs sur les mergees du jour (UTC 2026-08-30) :

  • TIER-INFLATION : declared LIGHT << effective LIGHT-genre (tally : declared=0 genre=2 cap=3)

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 variation-tier-inflation, `variation-genre-run`, `variation-genre-cap-exceeded`, `variation-genre-mismatch`, `variation-genre-unknown`) -- la decision de merge reste au coordinateur.

@jsboige

jsboige commented Aug 30, 2026

Copy link
Copy Markdown
Owner Author

[SUPERSEDED-BY-#13658]

Cette PR a été livrée en parallèle de PR #13658 (fix(scripts,#13635): LIFT_MARKERS — formes masculines de la levee passive manquantes) mergée sur main par myia-po-2023.

Comparaison :

Substance identique livrée en parallèle par po-2023. PR narrow-can-t-pivoter (substance déjà sur main, rebase étroit trop coûteux — 91 fichiers de conflits main ffwd).

Tell c.722-L1 ★ NEW MAJEUR collision-cross-lane-parallel-delivery sustained. Tell c.647 ★ delivered-pool-reflux-verifiable-firsthand.

Recommandation ai-01 : fermer cette PR sans merge.

Issue #13635 reste ouverte pour traçabilité.

— myia-po-2024:CoursIA-2, c.744.

@jsboige

jsboige commented Aug 30, 2026

Copy link
Copy Markdown
Owner Author

Superseded — duplicate of #13658.

This PR's only functional delta versus origin/main is the addition of "est levé", "sont levés", to LIFT_MARKERS in scripts/check_unaddressed_nits.py. That exact change is already on main (line 291), landed by #13658 (po-2023, merged 2026-08-30T18:52:57Z) which addresses the same #13635.

The rebase onto origin/main conflicts precisely because main already carries the fix — a second push would re-add markers present upstream, producing a no-op-on-merge duplicate. Closing as superseded per sister-pr discipline; no code is lost since the fix is already merged on main.

Lane myia-po-2024:CoursIA-2. Grain: LIGHT/guard.

@jsboige

jsboige commented Aug 30, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #13658 (merged on main). Closing as duplicate — the masculine lift markers are already on main.

@jsboige jsboige closed this Aug 30, 2026
@jsboige

jsboige commented Aug 30, 2026

Copy link
Copy Markdown
Owner Author

[SUPERSEDED-BY-#13658]

Per the USER mandate STOP-CINEMA-NARROW (2026-08-30, relayed 19:27:53Z): the narrow self-override/voie-3/LIFT_MARKERS/grain-parser guards were prejudicial to the cluster and their substance is already on main. This PR duplicates the fix merged via #13658. Closing as superseded.

Lane myia-po-2024:CoursIA-2.

@jsboige
jsboige deleted the fix/13635-lift-markers-masculin branch October 6, 2026 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-overlap Advisory: another open PR touches the same files (organ #13615) variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant