Skip to content

デッドコード削除とparserリファクタリングによるコードベース簡素化#35

Merged
harakeishi merged 4 commits intomainfrom
test/add-comprehensive-test-coverage
Feb 3, 2026
Merged

デッドコード削除とparserリファクタリングによるコードベース簡素化#35
harakeishi merged 4 commits intomainfrom
test/add-comprehensive-test-coverage

Conversation

@harakeishi
Copy link
Owner

概要

未使用の resolver パッケージ(約1,336行)と generator パッケージの未使用インターフェース・型(86行)を削除し、parser/yaml.goparsePortString
を単一責務の小さな関数に分割した。公開インターフェースの変更はなし。

変更内容

  • internal/resolver/ パッケージ全体を削除(conflict.go, conflict_test.go, interfaces.go の計約1,336行)
  • internal/generator/interfaces.go から未使用のインターフェース・型11個を削除(86行削減、UnifiedOverrideGenerator のみ残存)
  • parsePortStringextractProtocol + buildPortMappingFromRegexMatches に分割(SRP)
  • ポート解析用の正規表現 portStringPattern をパッケージレベルにプリコンパイル
  • リファクタリング計画ドキュメント docs/plans/2026-02-03-simplify-gopose.md を追加

期待すること

  • 約1,400行のデッドコード削除による保守性の向上
  • メソッド分割による parser パッケージの可読性・テスタビリティ向上
  • 正規表現プリコンパイルによるパース処理の微小なパフォーマンス改善
  • 不要な依存関係の削減によるビルド・テスト速度の改善

動作確認

項目 証憑(スクショなど) 備考
go build ./... ビルド成功 CLI出力 コンパイルエラーなし
go test ./... 全パッケージ PASS CLI出力 全7パッケージでテスト通過
resolver 削除後のビルド・テスト CLI出力 他パッケージから未参照であることを確認済み
interfaces.go 削除後のビルド・テスト CLI出力 UnifiedOverrideGenerator のみが使用されていることを確認済み
parsePortString リファクタリング後のパーサーテスト CLI出力 既存テストケース8件すべて通過(標準形式、IP指定、UDP、環境変数展開等)
エッジケースファイルの解析テスト CLI出力 testdata/parser/edge_cases.yml による統合テスト通過

harakeishi and others added 4 commits February 3, 2026 10:10
@harakeishi harakeishi merged commit 0654ccb into main Feb 3, 2026
4 checks passed
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