Skip to content

feat(dataset-updater): align translation tasks to gpt-5.5 mandate (#192, #299 follow-up) - #467

Merged
jsboige merged 1 commit into
masterfrom
feat/192-gpt55-model-alignment
Jun 13, 2026
Merged

feat(dataset-updater): align translation tasks to gpt-5.5 mandate (#192, #299 follow-up)#467
jsboige merged 1 commit into
masterfrom
feat/192-gpt55-model-alignment

Conversation

@jsboige

@jsboige jsboige commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

What

Enforces the standing gpt-5.5-only translation mandate. The eco-tier gpt-5.4-mini is banned for translations ("plus de fenêtre eco-tier ; pushback si un worker propose 5.4-mini"). This PR flips every gpt-5.4-mini occurrence in DatasetUpdaterRootConfig.cs to gpt-5.5.

Code-only, single file. References the #466 DOC-ONLY investigation docs/investigations/2026-06-14-llm-model-quality-datasetupdater.md, which identified the gpt-5.4-mini residue as drift from the mandate.

Count note: the investigation's pre-edit table stated 10× gpt-5.4-mini. The working-tree file actually carries 12 such occurrences (the doc undercounted — it tabbed the 4 Rules ES/AR/FA/ZH tasks but its "10" summary did not fully reconcile, and Fallacies→EN was still on the eco tier despite the doc's synthesis text). All 12 are translation tasks → all move up, per the "every occurrence" directive.

Exactly which tasks changed (line, name, old → new)

# Line Task name Old → New
1 246 Translate Fallacies to English by branch empty-only 0-shot gpt-5.4-minigpt-5.5
2 309 Translate Fallacies to Russian by chunk empty-only 0-shot gpt-5.4-minigpt-5.5
3 373 Translate Fallacies to Portuguese by chunk empty-only 0-shot gpt-5.4-minigpt-5.5
4 579 Translate Virtues to Russian by chunk empty-only 0-shot gpt-5.4-minigpt-5.5
5 643 Translate Virtues to Portuguese by chunk empty-only 0-shot gpt-5.4-minigpt-5.5
6 692 Generate Virtues Portuguese Wikipedia links empty-only 0-shot gpt-5.4-minigpt-5.5
7 933 Translate Scenarii to Russian by chunk empty-only 0-shot gpt-5.4-minigpt-5.5
8 1001 Translate Scenarii to Portuguese by chunk empty-only 0-shot gpt-5.4-minigpt-5.5
9 1302 Translate Rules to Spanish by chunk 0-shot gpt-5.4-minigpt-5.5
10 1345 Translate Rules to Arabic by chunk 0-shot gpt-5.4-minigpt-5.5
11 1388 Translate Rules to Farsi by chunk 0-shot gpt-5.4-minigpt-5.5
12 1431 Translate Rules to Chinese Simplified by chunk 0-shot gpt-5.4-minigpt-5.5

Count: 12 changed lines (git diff --stat = 12 insertions / 12 deletions; file otherwise byte-identical).

Intentionally left untouched (separate decision for @jsboige)

The 4 gpt-5.4 (non-mini) tasks are FR-native generation / refinement, not translation. The mandate is about translation, so these are left as-is and flagged here as a distinct decision:

Line Task name Model (unchanged) Why kept
61 Update Virtues Taxonomy by chunks 1-shot gpt-5.4 FR taxonomy creation/refinement
117 Update Fallacies French Description by branch 0-shot gpt-5.4 FR desc_fr refinement
182 Update Fallacies French example by branch 0-shot gpt-5.4 FR example_fr creative generation
446 Cleanup Fallacies translations by chunk empty-only 0-shot gpt-5.4 multi-lang cleanup review (quality tier)

If you want these aligned to gpt-5.5 too (or the cleanup-review one in particular), that's a separate call — happy to follow up.

No runtime / cost effect

All 47 tasks remain Enabled = false. There is no execution or API cost until a deliberate run is enabled.

Validation

  • Build + tests: dotnet test Argumentum.AssetConverter.Tests159 pass / 0 fail / 5 skip (skips = GSheet/GDrive + Freeplane GUI, expected).
  • EOL integrity: git ls-files --eoli/lf w/crlf preserved (no line-ending flip).
  • Diff scope: git diff shows exactly the 12 Model = lines; no other byte changed.

Model distribution after: gpt-5.5 31 → 43, gpt-5.4-mini 12 → 0, gpt-5.4 4 (unchanged), total 47.

🤖 Generated with Claude Code

…, #299 follow-up)

Enforce the standing gpt-5.5-only translation mandate: the eco-tier
gpt-5.4-mini is banned for translations. Flips every gpt-5.4-mini
occurrence (12 translation tasks) to gpt-5.5 in DatasetUpdaterRootConfig.cs.

Follows the #466 DOC-ONLY investigation
(docs/investigations/2026-06-14-llm-model-quality-datasetupdater.md),
which flagged the gpt-5.4-mini residue as drift from the mandate (note:
the working-tree file actually carries 12 such tasks, not the 10 stated
in the doc's pre-edit table — the doc undercounted; all are translations
and all move up).

The 4 gpt-5.4 (non-mini) tasks are intentionally left untouched: they are
FR-native generation/refinement (Virtues taxonomy FR, Fallacies FR
description, Fallacies FR example, multi-lang cleanup review), not
translation. The mandate concerns translation only — flagged as a
separate decision for jsboige.

All 47 tasks remain Enabled=false, so there is no runtime or cost effect
until a deliberate run is enabled.

Validation: dotnet test green (159 pass / 0 fail / 5 skip); EOL attribute
preserved (i/lf w/crlf); git diff is exactly 12 Model= line changes,
file otherwise byte-identical.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jsboige
jsboige merged commit d411704 into master Jun 13, 2026
3 checks passed
jsboige added a commit that referenced this pull request Jun 14, 2026
…t 155->159 (#469)

The v0.9.0 validation brief (PR #462) was written from a pre-#460 reading
and flagged doc discrepancies that #460 had already resolved, plus it
under-reported the test count.

Corrections (3 files, docs-only):

1. release-v0.9.0-validation-brief.md - reframe §3.1/§3.2 as RESOLVED by
   PR #460 (36124be): MindMap SVGs correctly state FR/EN/RU/PT committed
   (21) + ES/AR/FA/ZH pending (CHANGELOG L16, RELEASE-NOTES L25/L35/L92);
   Generated Assets counts reconciled to 64 (Tarot 24 + Poker 16 + Web 24).
   Simplifies jsboige's residual decision to: regenerate MindMaps 8-lang
   (Track 1a, option A) vs status quo (option B, already applied by #460).
   Also fixes a structural duplication in the prior commit (stale §3.1 RED
   / §3.2 YELLOW sections that should have been removed) and adds §3.4.

2. §3.4 (new) - test-count drift: CHANGELOG L70 + RELEASE-NOTES L46 said
   155 tests, but dashboard reports 159 after #465/#28. Verified by grep
   [Fact]/[Theory]: Issue28TargetDissociationTests.cs = 4 tests; #466/#467
   /#468 are doc-only / Enabled=false config -> 0 runtime tests. 155+4=159.

3. CHANGELOG.md L70 + RELEASE-NOTES-v0.9.0.md L46 - 155 -> 159 tests
   ("+4 from #465/#28 front/back dissociation").

Only §3.3 trivial date placeholders remain (normal, pre-tag). Gate release
unchanged - no tag/launch/merge.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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