Skip to content

data(fallacies): shorten 13 verbose desc_fr (#191) - #308

Merged
jsboige merged 1 commit into
masterfrom
data/fallacies-desc-fr-shorten
May 18, 2026
Merged

data(fallacies): shorten 13 verbose desc_fr (#191)#308
jsboige merged 1 commit into
masterfrom
data/fallacies-desc-fr-shorten

Conversation

@jsboige

@jsboige jsboige commented May 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Surgical raccourcissement de 13 entrées Fallacies desc_fr >200 chars (verbosité parent/enfant, redondances internes). Préserve sémantique scientifique exacte selon validation po-2023 cycle 42.

Scope (Q3 cycle 42 ack jsboige)

  • PR FR seule : desc_fr uniquement, 13 cellules
  • EN/RU/PT régénérés ultérieurement via DatasetUpdater post-merge (cohérence multilingue maintenue par re-run gpt-5.5)

Doublons (Q1 cycle 42 ack jsboige)

  • PK 460 + PK 915 : "Lance à incendie du mensonge" — cross-link intentionnel, même texte
  • PK 138 + PK 1074 : "Effet du lampadaire" — cross-link intentionnel, même texte
  • Pas de fusion : conservation des 2 copies par paire (cohérence taxonomique)

Statistiques

Métrique Avant Après Réduction
Total chars (13 cells) 2962 1610 -46%
Moyenne chars 228 124 -46%
Max chars 297 (PK 145) 146 (PK 371) —
PK le plus raccourci 145 -63% —
PK le moins raccourci 371 -25% —

Patterns appliqués

  1. Suppression métadiscours ("Vous utilisez une technique de propagande qui consiste à" → "Vous diffusez")
  2. Suppression redondances internes ("rapide, répétitive et continue" → "en masse")
  3. Préservation structure "Vous + verbe + complément" (ton instructif uniforme du jeu)

Implementation

Application via script Python byte-level surgical (cf. feedback_csv_surgical_edits pattern, déjà utilisé PRs #301 et #306). Préserve quoting/encoding/whitespace.

11 textes uniques substitués (2 pour les paires cross-link + 9 unitaires) = 13 cellules au total.

Verification

  • CSV parses correctly : 1408 records, no field count change
  • git diff --stat : 13 insertions / 13 deletions exactement
  • Spot-check 3 cells (PK 138, 460, 917) : nouvelles tailles vérifiées (123, 132, 112 chars)
  • Aucune autre colonne touchée

Test plan

  • CI Debug + Release SUCCESS
  • CsvHelper parse OK (1408 records, no field count change)
  • Visual regen Tarot FR sample 5 cartes (PK 145, 460, 138, 208, 917) — post-merge, cycle 56 strategy
  • Pas d'overflow texte sur cartes raccourcies
  • FR uniquement : EN/RU/PT desc inchangées (régression)

Issue

Closes #191 phase 1 (FR). EN/RU/PT régénération suit (post smoke test gpt-5.5 + cycle 56 stratégie).

🤖 Generated with Claude Code

@jsboige @claude
data(fallacies): shorten 13 verbose desc_fr (#191)
cb7ca73 
Surgical raccourcissement de 13 entrees Fallacies desc_fr >200 chars
(verbosite parent/enfant, redondances internes). Preserve semantique
scientifique exacte selon validation po-2023 cycle 42.

Q1/Q2/Q3 jsboige integres (cycle 42 arbitrages, 2026-05-18):
- Q1: PK 460/915 (Lance incendie) et PK 138/1074 (Lampadaire) sont
  des cross-link intentionnels (PAS de fusion), meme texte applique
  aux 2 PKs de chaque paire.
- Q2: Issue #191 confirmee (verified OPEN).
- Q3: PR FR seule, EN/RU/PT regeneres ulterieurement via DatasetUpdater
  post-merge (coherence multilingue maintenue par re-run gpt-5.5).

Scope:
- Cellules modifiees: 13 (PK 138, 145, 208, 371, 373, 460, 505, 522,
  607, 613, 915, 917, 1074)
- Textes distincts (post-dedup paires): 11
- Reduction totale: 2962 -> 1610 chars (-46%)
- Max: PK 145 (-63%), Min: PK 371 (-25%)

Aucune autre colonne touchee (text_fr, example_fr, desc_en/ru/pt inchanges).

Closes #191 phase 1 (FR). EN/RU/PT regeneration suit.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@jsboige
jsboige merged commit 377613d into master May 18, 2026
3 checks passed
This was referenced May 24, 2026
Merged
Closed
@jsboige
jsboige deleted the data/fallacies-desc-fr-shorten branch June 1, 2026 21:41
@jsboige jsboige mentioned this pull request Jun 9, 2026
Merged
@clusterManager-Myia clusterManager-Myia mentioned this pull request Jun 9, 2026
Merged
jsboige added a commit that referenced this pull request Jun 10, 2026
@jsboige @claude
fix(memo): #443 Mémo Back grid — language-invariant control-break gro…
1fd4aa0 
…uping (#449)

The Mémo Back card collapsed from the spec 7×3×3 taxonomy grid (7 families ×
3 sub-families × 3 sub-sub-families) to an irregular shape. Root cause: the
template grouped the grid via name-coincidence markers
({{#ifCond Famille "==" text_fr}} …) that fired only on rows whose text_fr
happened to equal their own level label. FR rename campaigns (#351 ×27,
#308 ×13) changed text_fr → the coincidences vanished → families collapsed
(Tricherie sub-families = 0, Erreur math/raisonnement = 1, …). The taxonomy
DATA is correct; only the template's grouping was broken.

Fix (lane ai-01 = TEMPLATE, data is fine):
- CardPen main.js: add 3 control-break helpers reading the language-invariant
  FR taxonomy columns (Sous-Famille / Soussousfamille), present in every row
  regardless of render language. They carry NO column token, so the localizer
  (CardSetLocalization.FormatField appends "}}") cannot corrupt the helper call:
    ifFamilyHeader   = Sous-Famille=='' && Soussousfamille==''
    ifSubfamilyHeader= Sous-Famille!=''  && Soussousfamille==''
    ifLeafFirst      = Soussousfamille!='' on first occurrence (control-break)
- Argumentum_Memo_Back_fr.json: swap the 3 name-coincidence conditions for the
  new helpers (display tokens {{Famille}}/{{Sous-Famille}}/{{Soussousfamille}}
  unchanged → localization preserved); fix colorPalette to iterate the rowset
  and emit one swatch per family-root (was fixed indices 0/25/50/…/150, which
  duplicated Influence and omitted Obstruction).

Single shared template → fix propagates to all 8 languages.

Validated three ways on the real injected data (taxonomy WHERE carte∈{1,2},
176 rows): (1) Node + vendored Handlebars → 7 families / 21 sub-families /
63 leaves; (2) OLD-template render proving the regression (10/53, Tricherie=0);
(3) LIVE CardPen engine (Playwright) → 7×3×3 DOM + 7 distinct palette colors
(Obstruction red present, no Influence duplicate) + viewed screenshot.

Refs #443

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
jsboige added a commit that referenced this pull request Jun 10, 2026
@jsboige @claude
fix(rules): index-independent heading CSS + cover titles (#438 follow…
5ea58c8 
…-up) (#450)

* fix(memo): #443 Mémo Back grid — language-invariant control-break grouping

The Mémo Back card collapsed from the spec 7×3×3 taxonomy grid (7 families ×
3 sub-families × 3 sub-sub-families) to an irregular shape. Root cause: the
template grouped the grid via name-coincidence markers
({{#ifCond Famille "==" text_fr}} …) that fired only on rows whose text_fr
happened to equal their own level label. FR rename campaigns (#351 ×27,
#308 ×13) changed text_fr → the coincidences vanished → families collapsed
(Tricherie sub-families = 0, Erreur math/raisonnement = 1, …). The taxonomy
DATA is correct; only the template's grouping was broken.

Fix (lane ai-01 = TEMPLATE, data is fine):
- CardPen main.js: add 3 control-break helpers reading the language-invariant
  FR taxonomy columns (Sous-Famille / Soussousfamille), present in every row
  regardless of render language. They carry NO column token, so the localizer
  (CardSetLocalization.FormatField appends "}}") cannot corrupt the helper call:
    ifFamilyHeader   = Sous-Famille=='' && Soussousfamille==''
    ifSubfamilyHeader= Sous-Famille!=''  && Soussousfamille==''
    ifLeafFirst      = Soussousfamille!='' on first occurrence (control-break)
- Argumentum_Memo_Back_fr.json: swap the 3 name-coincidence conditions for the
  new helpers (display tokens {{Famille}}/{{Sous-Famille}}/{{Soussousfamille}}
  unchanged → localization preserved); fix colorPalette to iterate the rowset
  and emit one swatch per family-root (was fixed indices 0/25/50/…/150, which
  duplicated Influence and omitted Obstruction).

Single shared template → fix propagates to all 8 languages.

Validated three ways on the real injected data (taxonomy WHERE carte∈{1,2},
176 rows): (1) Node + vendored Handlebars → 7 families / 21 sub-families /
63 leaves; (2) OLD-template render proving the regression (10/53, Tricherie=0);
(3) LIVE CardPen engine (Playwright) → 7×3×3 DOM + 7 distinct palette colors
(Obstruction red present, no Influence duplicate) + viewed screenshot.

Refs #443

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* fix(rules): index-independent heading CSS + cover titles (#438 follow-up)

The 24->15 Rules deck restructure (#438) broke the heading CSS, which keyed
banners and number-boxes to card INDEX (nth-of-type + hardcoded content
'01'..'05' + [class~="4"]). After the restructure those indices were wrong:

  1. «Déroulé de la manche» rendered in a colored banner instead of a plain
     big title like «Règles du jeu…».
  2. «Argumentum» rendered big on the sub-game cover cards instead of the rule
     name.
  3. Inconsistent paragraph headers: empty floating number boxes + size
     mismatches on cards 6/10/12/14/15.

Fix (index-independent, semantic headings):
  - h2:first-child = card main title  -> plain, big, no banner
  - other h2       = section sub-head -> plain, thin theme-colored underline
  - h3             = numbered step     -> theme-colored banner; the «1.» number
                     is already in the heading text, so no separate (broken) box
  - removed the leading «# Argumentum» line from the 4 sub-game cover rows
    (Rules_07/09/11/13) across all 8 language columns so «## <rule name>»
    becomes the title; Rules_01 (main cover) keeps «# Argumentum».
  - tightened heading margins/padding so the densest cards do not overflow
    worse than the committed baseline.

Validated live on the real CardPen engine (all 15 cards): zero bannered
titles, covers lead with the rule name, uniform theme-colored step banners;
total text overflow reduced 208px -> 110px vs baseline (cards 11/13 fully
un-clipped, only card 12 +1px = parity).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
@jsboige jsboige mentioned this pull request Jul 3, 2026
Merged
jsboige added a commit that referenced this pull request Jul 3, 2026
@jsboige Your @claude
docs(i18n): Rules 15x8 coverage audit = 120/120 OK + Fallacies EN res…
fd56bba 
…iduals NO-OP proof (#661)

Secondaire dispatch ai-01 (msg kp26j6) — data pendant of visual verdict #140.

Rules audit (Cards/Rules/Argumentum Rules - Cards.csv, read-only):
- 15 records x 8 languages = 120/120 cells OK (populated + correct script).
- 0 empty, 0 contamination residual post-#640 (scanner #647 = 0).
- 2 heuristic flags cleared as false positives: Rules_01 ZH (说谎者学校
  present, latin inflation from brand "Argumentum"); embromador (canonical PT
  game-term for baratineur, 77x in Scenarii PT).

Primaire (Fallacies EN residuals) = NO-OP, proven empirically:
- 0 empty desc_en across 1408 rows; PK 371/607 (cited empty) already populated
  (120/79 chars).
- Scanner #647 TOTAL=0; 0 EN identical-to-FR; 0 EN reading as French.
- The #351/#308 drift (40 PK) was resolved in an earlier cycle.

Reporting the no-op rather than fabricating work / spending gpt-5.5 on nothing
(pattern: verify before concluding, [[matcher-no-match-is-not-content-absent]]).

Read-only, 0 CSV write. Base 9c19e51.

Co-authored-by: Your <your.email@example.com>
Co-authored-by: Claude-Code <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat(content): simplify French source texts for broader accessibility (preserve taxonomy nuances)

1 participant

@jsboige