リファクタリング・パフォーマンス改善

[mumeinosato]

リファクタリング（MainForm.csダイエット計画）

• 専用クラスの実装
• 翻訳データのjson管理
• キャラクターデータのjson管理
• 状態等をenumで管理
• UIを手動管理から自動管理へ変更
• UI変更ロジックの修正

パフォーマンス改善
メモリリーク、リソースリークを修正
新たなリソースリークが発生しているかも

[Copilot]

Pull request overview

MainForm の責務を専用クラスへ分割しつつ、翻訳・キャラクターデータを埋め込み JSON リソースへ移行して、メモリ/リソースリーク改善を狙ったリファクタリングPRです。

Changes:

• Tray/Theme/Font/I18n/Update/Process監視/RPC などをクラス分割して MainForm をスリム化
• 翻訳データ（日本語/英語）とキャラクターデータを JSON 化して埋め込みリソースからロード
• UpdateService/ProcessWatcher/NativeMethods 追加によるリソース管理とロジック整理

Reviewed changes

Copilot reviewed 22 out of 26 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
GkmStatus/src/ui/TrayIconManager.cs	トレイアイコン描画/メニュー状態更新/破棄を分離
GkmStatus/src/ui/Translation.cs	I18n を JSON 埋め込みリソースからロードする実装へ移行
GkmStatus/src/ui/ThemeManager.cs	テーマ適用と Win テーマ取得を分離
GkmStatus/src/ui/FontManager.cs	埋め込みフォント読み込みと破棄を分離
GkmStatus/src/ui/CustomRenderer.cs	MenuStrip/ContextMenu のカスタム描画を分離
GkmStatus/src/native/NativeMethods.cs	SendMessage/DestroyIcon/SetForegroundWindow 等の P/Invoke を集約
GkmStatus/src/UpdateService.cs	GitHub release API による更新チェックをサービス化
GkmStatus/src/ProcessWatcher.cs	プロセス監視ロジックを Timer ベースでクラス化
GkmStatus/src/DiscordRpc.cs	Discord RPC をラップするサービス/状態イベントを導入
GkmStatus/src/ConfigManager.cs	config.json のロード/セーブ/デフォルト生成を分離
GkmStatus/src/Characters.cs	キャラクターデータを JSON リソースからロードする仕組みを追加
GkmStatus/src/AppLogic.cs	enum/設定変換/URLバリデーション等のヘルパーを整理
GkmStatus/src/AppConstants.cs	定数・リソースパス・色定義を集約
GkmStatus/Translation.cs	旧 I18n 実装を削除
GkmStatus/Resources/produce_characters.json	キャラクターデータ JSON を追加
GkmStatus/Resources/I18n/japanese.json	日本語翻訳 JSON を追加
GkmStatus/Resources/I18n/english.json	英語翻訳 JSON を追加
GkmStatus/Properties/Resources.resx	リソースファイルを追加/更新
GkmStatus/Properties/Resources.Designer.cs	resx 生成コードを追加/更新
GkmStatus/MainForm.resx	トレイ関連のデザイナメタデータ追加
GkmStatus/MainForm.cs	MainForm の責務を各サービス/マネージャへ移譲し整理
GkmStatus/MainForm.Designer.cs	Designer を使わず動的 UI 構築する実装へ移行
GkmStatus/GkmStatus.csproj	JSON リソースの埋め込み、resx 生成設定を追加
GkmStatus/AboutForm.resx	AboutForm の resx を追加
GkmStatus/AboutForm.cs	AboutForm を partial + ApplyComponent に寄せる変更
GkmStatus/AboutForm.Designer.cs	AboutForm の動的 UI 構築をこちらへ移行

Files not reviewed (3)

• GkmStatus/AboutForm.Designer.cs: Language not supported
• GkmStatus/MainForm.Designer.cs: Language not supported
• GkmStatus/Properties/Resources.Designer.cs: Language not supported

Comments suppressed due to low confidence (1)

GkmStatus/AboutForm.cs:41

• ShowLicense(object, LinkLabelLinkClickedEventArgs) in this file is no longer wired up: the link click handler in AboutForm.Designer.cs calls a different parameterless ShowLicense() method, duplicating the license rendering logic across partials. Consider keeping a single ShowLicense implementation and wiring lnkLicense.LinkClicked to it (or removing the unused overload) to avoid future drift.

        private void ShowLicense(object sender, LinkLabelLinkClickedEventArgs e)
        {
            try
            {
                var assembly = System.Reflection.Assembly.GetExecutingAssembly();
                using Stream? stream = assembly.GetManifestResourceStream("GkmStatus.Resources.fonts.IBM_Plex_Sans_JP.OFL.txt");
                if (stream != null)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The About dialog title is set to App_Name here, which changes the window caption from the localized “About” title (About_Title) used previously. Consider using the About_Title translation key for this.Text so the dialog is labeled correctly in both languages.

Suggested change

	this.Text = I18n.T(I18n.Text_List.App_Name);
	this.Text = I18n.T(I18n.Text_List.About_Title);