Skip to content

fix(csv): correct translation micro-issues across 4 datasets (#202) - #203

Merged
jsboige merged 1 commit into
masterfrom
fix/epic-202-csv-text-fixes
Apr 22, 2026
Merged

fix(csv): correct translation micro-issues across 4 datasets (#202)#203
jsboige merged 1 commit into
masterfrom
fix/epic-202-csv-text-fixes

Conversation

@jsboige

@jsboige jsboige commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Phase 1 micro-fixes from Epic #202 — targeted corrections to translation quality issues identified during the publication audit. All changes are surgical (21 insertions, 22 deletions across 4 CSV files), verified by CSV parser assertions, and backed by build+tests.

Fallacies Taxonomy (9 fixes)

  • PK 927: link_en pointed to fr.wikipedia instead of en.wikipedia (Creepypasta)
  • PK 142: typo "susequent" → "subsequent" + expanded desc_en to match FR richness (58 → 136 chars)
  • PK 225: expanded desc_en for Paradoxe du menteur (38 → 104 chars)
  • PK 326: expanded desc_en for Appel à la honte + removed trailing newline artifact (59 → 163 chars)
  • PK 327: expanded desc_en for Appel au dégoût (57 → 170 chars)
  • PK 150: expanded desc_ru for Archaïsme — was severely truncated (21 → 166 chars)
  • PK 549: filled empty desc_ru for Salut en budo (0 → 95 chars)
  • PK 856: expanded desc_en (29 → 96 chars) and desc_pt (30 → 92 chars) for Expression vague

Rules (2 files)

  • Fixed PT apostrophe: "Liars 'School" → "Liars' School" (both Cards and Print&Play CSVs)

Scenarii (11 fixes)

  • Replaced untranslated French "baratineur" → "smooth talker" in all 11 EN context/issue cells

Not in scope (flagged for Phase 2/3)

Test plan

  • Python CSV parser verification: 1408 rows parse correctly with all 9 Fallacies fixes
  • Uniqueness assertions on all replacement strings before applying
  • dotnet build — 0 errors, 19 warnings (all pre-existing)
  • dotnet test — 43 passed, 1 skipped (Freeplane GUI), 0 failed
  • git diff --stat confirms surgical changes only (4 files, 21+/22-)

🤖 Generated with Claude Code

…cenarii (#202)

Fallacies Taxonomy (9 fixes):
- PK 927: link_en pointed to fr.wikipedia instead of en.wikipedia (Creepypasta)
- PK 142: fix typo "susequent" → "subsequent" + expand desc_en to match FR richness
- PK 225: expand desc_en for Paradoxe du menteur (38 → 104 chars)
- PK 326: expand desc_en for Appel à la honte + remove trailing newline
- PK 327: expand desc_en for Appel au dégoût (57 → 170 chars)
- PK 150: expand desc_ru for Archaïsme (21 → 166 chars)
- PK 549: fill empty desc_ru for Salut en budo
- PK 856: expand desc_en + desc_pt for Expression vague

Rules (2 files):
- Fix PT apostrophe: "Liars 'School" → "Liars' School"

Scenarii (11 fixes):
- Replace untranslated "baratineur" → "smooth talker" in EN columns

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jsboige

jsboige commented Apr 19, 2026

Copy link
Copy Markdown
Contributor Author

Review — CSV Translation Micro-Fixes

Verdict: LGTM, ready to merge

All 22 corrections verified in the diff:

Fallacies (9 fixes) — ✅

  • PK 927: fr.wikipediaen.wikipedia (Creepypasta) — correct locale fix
  • PK 142: typo susequentsubsequent + desc_en expanded 58→136 chars
  • PK 225, 326, 327: desc_en expanded to match FR richness
  • PK 150: desc_ru completed (21→166 chars, was truncated)
  • PK 549: desc_ru filled (was empty)
  • PK 856: desc_en + desc_pt expanded

Rules PT (2 fixes) — ✅

  • Liars 'SchoolLiars' School apostrophe fix (both CSVs)

Scenarii (11 fixes) — ✅

  • baratineursmooth talker in all EN context/issue cells

Surgical changes only (21+/22-), no FR source touched, no quoting damage. Flagged for Phase 2: PT Scenarii still has 64 untranslated "baratineur" + PT Rules needs comprehensive MT review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant