Skip to content

Conversation

@harutiro
Copy link
Member

@harutiro harutiro commented Sep 23, 2025

概要

Issue #7に対応し、プロジェクト内で参照されていない未使用のUIコンポーネントを削除しました。

削除したファイル

以下の3つのファイルを削除しました:

  1. ImagePickerView.swift

    • パス: UWBViewerSystem/Presentation/Components/ImagePickerView.swift
    • 理由: プロジェクト内で一切参照されておらず、画像選択機能も現在未使用
  2. InformationSection.swift

    • パス: UWBViewerSystem/Presentation/Components/InformationSection.swift
    • 理由: プロジェクト内で一切参照されておらず、情報表示セクション機能も現在未使用
  3. PrimaryButton.swift

    • パス: UWBViewerSystem/Presentation/Components/PrimaryButton.swift
    • 理由: プロジェクト内で一切参照されておらず、プライマリボタン機能も現在未使用

削除理由

  • コードベースの検索により、これらのファイルがプロジェクト内のどこからも参照されていないことを確認
  • 未使用コードを削除することで、保守性とコードの可読性を向上
  • ビルドサイズの削減とコンパイル時間の短縮に寄与

確認事項

ビルド確認: エラーなくビルドが完了することを確認済み
テスト実行: すべてのSwiftTestが正常に通ることを確認済み
コードフォーマット: SwiftFormatによるフォーマットが適用済み
参照チェック: 削除対象ファイルがプロジェクト内で参照されていないことを確認済み

その他

  • 削除したコンポーネントは将来的に必要になった場合、Gitの履歴から復元可能
  • プロジェクトの機能には一切影響なし

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 新機能
    • なし
  • リファクタリング
    • 画像選択用のUIシートを削除し、関連するUIを整理。
    • 情報表示セクションと手順表示コンポーネントを削除して画面構成を簡素化。
    • カスタムボタン群(主要/セカンダリ/破壊的)を削除し、ボタン表現を統一。
  • 雑務
    • 旧コンポーネントとプレビューを削除し、不要な資産を削減。

@coderabbitai
Copy link

coderabbitai bot commented Sep 23, 2025

Walkthrough

本PRはSwiftUIコンポーネントを削除し、画像ピッカー、情報/手順表示、各種ボタン(Primary/Secondary/Destructive)のビュー、関連API、およびプレビューをプロジェクトから取り除きます。

Changes

Cohort / File(s) Change Summary
Image Picker コンポーネント削除
UWBViewerSystem/Presentation/Components/ImagePickerView.swift
ImagePickerView: UIViewControllerRepresentableCoordinatormakeUIViewController/updateUIViewController等ライフサイクル、imagePickerSheet View拡張を全面削除
情報・手順UI削除
UWBViewerSystem/Presentation/Components/InformationSection.swift
InformationSectionStepInstructionView(ネスト型 Step 含む)、初期化子・レイアウト・プレビューを全面削除
ボタンUI削除
UWBViewerSystem/Presentation/Components/PrimaryButton.swift
PrimaryButtonSecondaryButtonDestructiveButton のビュー実装、初期化子、スタイル、プレビューを全面削除

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

  • 未使用コンポーネントの削除 #7 — 削除対象コンポーネント(ImagePickerView、InformationSection/StepInstructionView、Primary/Secondary/DestructiveButton)が本PRと一致するため関連が高い

Poem

ぴょんと跳ねて削ったコード
余白の原に風が通る
ボタンも手順も今日はお休み
画像ピッカーは旅に出た
うさぎは耳で微笑む 🐰✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed タイトル「Issue #7: 未使用コンポーネントの削除」はプルリクの主目的である参照されていないUIコンポーネントの削除を簡潔かつ明確に表しており、曖昧な表現や不要なファイル一覧を含まず変更内容と整合しています。
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch issue-7-remove-unused-components

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aeacb72 and 2b7090f.

📒 Files selected for processing (3)
  • UWBViewerSystem/Presentation/Components/ImagePickerView.swift (0 hunks)
  • UWBViewerSystem/Presentation/Components/InformationSection.swift (0 hunks)
  • UWBViewerSystem/Presentation/Components/PrimaryButton.swift (0 hunks)
💤 Files with no reviewable changes (3)
  • UWBViewerSystem/Presentation/Components/ImagePickerView.swift
  • UWBViewerSystem/Presentation/Components/InformationSection.swift
  • UWBViewerSystem/Presentation/Components/PrimaryButton.swift

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@harutiro harutiro force-pushed the issue-7-remove-unused-components branch from aeacb72 to 2b7090f Compare September 23, 2025 13:14
@harutiro harutiro merged commit 4da6625 into main Sep 23, 2025
2 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.

2 participants