Skip to content

ドキュメントと実装の乖離を修正(旧プラグイン名・構造図・日本語ローカライズ・.uplugin) - #80

Merged
EmbarrassingMoment merged 2 commits into
claude/branch-content-review-ao4va8from
claude/docs-consistency-fixes
Aug 22, 2026
Merged

ドキュメントと実装の乖離を修正(旧プラグイン名・構造図・日本語ローカライズ・.uplugin)#80
EmbarrassingMoment merged 2 commits into
claude/branch-content-review-ao4va8from
claude/docs-consistency-fixes

Conversation

@EmbarrassingMoment

Copy link
Copy Markdown
Owner

概要

ブランチレビューで検出した「🟠 ドキュメントと実装の乖離」6項目を修正します。

変更内容

1. 旧プラグイン名の残存を修正

  • README / README.ja / QUICKSTART / QUICKSTART.ja のインストール手順を Plugins/MarkdownEditorPlugins/MarkdownAsset に修正
  • プロジェクト構造図を実態に合わせて修正(MarkdownAsset.uplugin、md4c の実際の位置 Source/ThirdParty/md4c/、エディタモジュールの内容にアウトライン・markitdown 連携・テストを追記)

2. 日本語ローカライズの欠落を解消(25キー)

  • 1.3.0 の Markitdown 系 21 キーと Outline パネル系 4 キーの日本語訳を追加
  • Content/Localization/MarkdownEditor の manifest / en・ja archive を更新し、コンパイル済み .locres を再生成
    • en: 45 → 70 エントリ、ja: 32 → 55 エントリ(原文と同一の訳は locres に含めない UE の規則に準拠)
  • 検証方法: 変更前に、生成パイプライン(manifest 順序・CityHash64/UTF-16 の TextKey ハッシュ・CRC32 のソース文字列ハッシュ・文字列テーブルの重複排除)で既存の manifest / archive / locres ファイル一式をバイト単位で完全再現できることを確認してから追記。新規 25 キーの原文は .cpp から直接抽出して照合し、フォーマット引数 {0} 等の欠落もチェック済み

3. プラグイン単体配布時のローカライズ読み込みを修正

  • MarkdownAsset.upluginLocalizationTargetsMarkdownEditor, LoadingPolicy: Editor)を宣言。これまでプラグイン同梱の翻訳データは単体インストールでは読み込まれていませんでした
  • プラグイン側 Content/Localization はランタイムに必要な .locres / .locmeta のみに整理(manifest / archive / po / csv は Localization Dashboard が使うプロジェクト側 Content/Localization に一本化し、二重管理によるドリフトを防止)

4. リネーム残骸の削除

  • Plugins/MarkdownEditor/(.uplugin を持たないローカライズ残骸ディレクトリ、10 ファイル)を削除

5. README.ja の表記修正

  • 「ニバイト文字対応」→「マルチバイト文字対応」
  • 見出し「必須ランタイム」→「ランタイム」(英語版 "Runtime" に対応)

6. .uplugin の Description 更新

  • 1.0 時点の説明から、アウトラインパネル・Wikilink / アセット・クラスリンク・Markitdown 変換を含む現行機能の説明に更新

CHANGELOG

  • Unreleased に ### Fixed として上記を記載

補足

  • レビューで検出した 🔴 のアウトライン見出しジャンプずれ(リスト内見出し・引用内 setext 見出し)は本 PR の対象外です(別途対応可能です)
  • .po(エクスポート成果物)は今回更新していません。次回 Localization Dashboard でのエクスポート時に再生成されます

🤖 Generated with Claude Code

https://claude.ai/code/session_01G398YQHzHYChvMpRonbFTM


Generated by Claude Code

claude and others added 2 commits August 21, 2026 07:02
…tion

Documentation:
- README / QUICKSTART (en/ja): installation steps and project structure
  referenced the pre-rename plugin directory (Plugins/MarkdownEditor,
  MarkdownEditor.uplugin); corrected to Plugins/MarkdownAsset and the
  actual Source/ThirdParty/md4c layout.
- README.ja: fixed wording (ニバイト文字 -> マルチバイト文字) and the
  Planned Features heading (必須ランタイム -> ランタイム).
- MarkdownAsset.uplugin: description updated to cover the outline panel,
  wikilinks / asset & class links, and Markitdown conversion.

Localization:
- Added the 25 missing entries (all Markitdown UI strings from 1.3.0 and
  the new Outline panel strings) to the MarkdownEditor localization
  manifest and en/ja archives, with Japanese translations, and
  regenerated the compiled en/ja .locres files. Source strings were
  extracted from the .cpp files to keep source hashes exact, and the
  generation pipeline was validated by reproducing the existing
  manifest/archive/locres files byte-for-byte first.
- Declared the MarkdownEditor localization target in
  MarkdownAsset.uplugin (LocalizationTargets, LoadingPolicy=Editor) so
  the bundled translations load for standalone plugin installs; the
  plugin now ships only the compiled .locres/.locmeta data while
  authoring data stays in the project's Content/Localization.
- Removed the stale pre-rename Plugins/MarkdownEditor/ localization
  leftovers.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01G398YQHzHYChvMpRonbFTM
Newer MSVC toolchains (UE 5.8+) report unreachable code (C4702) inside
the third-party md4c.c and promote it to an error. Since md4c sources
must not be modified, disable the warning in the md4c_all.c unity build
wrapper for MSVC only.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@EmbarrassingMoment
EmbarrassingMoment merged commit fff4d32 into claude/branch-content-review-ao4va8 Aug 22, 2026
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.

2 participants