docs(release): align v0.9.0 notes with master HEAD reality (#456 follow-up) - #460
Merged
Conversation
…ow-up) Three discrepancies surfaced during the #456 sur-pièce validation review are corrected so the merged docs are truthful as of master HEAD: - MindMap SVGs: docs claimed "8 languages / ~40 SVGs". Ground truth = FR/EN/RU/PT only (21 SVGs committed under Cards/Fallacies/Mindmaps/). ES/AR/FA/ZH pipeline is configured (StaticConversions + MindMapLocalization) but SVG regen is pending an attended FreePlane GUI run (#458 Track 1a). Fixes the internal contradiction between CHANGELOG L16 (claimed 8) and L47 (said 4), and RELEASE-NOTES L92 which omitted ES. - Generated-assets table: per-type PDF counts did not reconcile. Replaced with the verified breakdown that sums to 64 (8 doc types × 8 languages): Tarot 24 + Poker 16 + Fallacies Web 24 = 64. - Prose: removed "MindMap SVGs ... for each language" overstatement. Verified against Release Target (64 PDFs across ar/en/es/fa/fr/pt/ru/zh) and Cards/Fallacies/Mindmaps (fr=6, en/ru/pt=5 → 21). No release action taken — gate held pending jsboige sur-pièce validation. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
clusterManager-Myia
left a comment
clusterManager-Myia
left a comment
Collaborator
There was a problem hiding this comment.
[NanoClaw]
Docs-only follow-up to #456, correcting three discrepancies between release notes and the actual state of master HEAD.
CHANGELOG.md (2 lines)
- MindMap SVG bullet: correctly narrows "all 8 languages" to FR/EN/RU/PT (21 SVGs), with ES/AR/FA/ZH flagged as pending via StaticConversions + MindMapLocalization and tracked as #458 Track 1a.
- SVG generation bullet: count fixed from "20 SVGs, 4 languages" to "21 SVGs across FR/EN/RU/PT".
RELEASE-NOTES-v0.9.0.md (7 lines)
- Asset table: rows restructured from vague per-type estimates (~64/~8/~16/~16/~40) to precise per-type counts (24 Tarot / 16 Poker / 24 FallaciesWeb = 64 total PDFs) plus a separate MindMap row showing 4 langs / 21 SVGs. This is a clear improvement in accuracy and readability.
- Intro paragraph: "all 8 languages" claim narrowed for MindMap SVGs; the Known Limitations section expanded with the technical reason (FreePlane SendKeys.SendWait desktop automation) and explicit tracker reference (#458 Track 1a).
Verdict: LGTM -- clean, well-grounded corrections. No secrets, no code changes, no risk of regression.
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Corrects 3 discrepancies in merged CHANGELOG.md + RELEASE-NOTES-v0.9.0.md so docs are truthful as of master HEAD
7b57251e.Changes (ground-truth verified):
Prep for jsboige Sunday validation. Option B (docs align to current reality). If Option A (Track 1a regen) chosen, trivial bump post-regen.
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com