feat: 画面共有とカメラ共有をマルチモーダルモデルに関係なく利用可能に #416
Conversation
- 画面共有・カメラ共有ボタンをデフォルトで表示 - AI設定に「マルチモーダルモデルで画像を自動送信」トグルを追加 - 画像アップロードボタンはマルチモーダルモデル時のみ表示（既存仕様維持） - 自動画像送信はトグル設定とマルチモーダルモデル判定に基づいて制御 Fixes #340 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: tegnike <tegnike@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Walkthroughマルチモーダルモデル利用時の画像自動送信に関する新しい設定が追加されました。設定ストア、UI（設定画面）、フォーム処理、メニューのボタン表示ロジック、ローカライズファイル（英語・日本語）に変更が加えられています。 Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant SettingsPanel
participant SettingsStore
participant Form
participant AIModel
User->>SettingsPanel: 「画像自動送信」トグルを操作
SettingsPanel->>SettingsStore: autoSendImagesInMultiModal を更新
User->>Form: メッセージ送信
Form->>SettingsStore: autoSendImagesInMultiModal, AIモデル設定を参照
alt autoSendImagesInMultiModal有効 & マルチモーダル & 画像共有中
Form->>Form: 画像キャプチャ
Form->>AIModel: 画像＋テキスト送信
else
Form->>AIModel: テキストのみ送信
end
Assessment against linked issues
Assessment against linked issues: Out-of-scope changes（該当なし） Possibly related PRs
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
npm error Exit handler never called! 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@claude
|
|
Claude finished @tegnike's task in 5m 25s —— View job 完了 ✅マルチモーダルモデル設定の追加改善を実装しました。 タスクリスト:
実装内容:
対象AI サービス:
特殊対応:
これで、マルチモーダルモデルを選択していない時は「マルチモーダルモデルで画像を自動送信」トグルがdisabledになり、非マルチモーダルモデルを選択した際は自動的にオフになります。 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
locales/en/translation.json(1 hunks)locales/ja/translation.json(1 hunks)src/components/form.tsx(3 hunks)src/components/menu.tsx(2 hunks)src/components/settings/modelProvider.tsx(2 hunks)src/features/stores/settings.ts(3 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (3)
src/components/settings/modelProvider.tsx (1)
src/components/textButton.tsx (1)
TextButton(4-13)
src/components/menu.tsx (3)
src/components/iconButton.tsx (1)
IconButton(14-51)src/features/constants/aiModels.ts (1)
isMultiModalModel(264-266)src/features/constants/settings.ts (1)
AIService(63-63)
src/components/form.tsx (2)
src/features/constants/aiModels.ts (1)
isMultiModalModel(264-266)src/features/constants/settings.ts (1)
AIService(63-63)
🪛 GitHub Actions: Lint and Format
src/components/settings/modelProvider.tsx
[error] 98-98: Prettier formatting error: Replace (s) => s.autoSendImagesInMultiModal with a multiline format. (prettier/prettier)
[error] 1351-1351: Prettier formatting error: Replace {t('AutoSendImagesInMultiModalDescription')} with a multiline format. (prettier/prettier)
src/components/form.tsx
[error] 21-21: Prettier formatting error: Replace (s) => s.autoSendImagesInMultiModal with a multiline format. (prettier/prettier)
[error] 38-38: Prettier formatting error: Replace autoSendImagesInMultiModal && with a multiline format. (prettier/prettier)
[error] 40-40: Prettier formatting error: Delete extra indentation. (prettier/prettier)
[error] 54-54: Prettier formatting error: Replace multiple props list with multiline format. (prettier/prettier)
🪛 GitHub Actions: Run Tests
src/components/settings/modelProvider.tsx
[error] 98-98: Prettier formatting error: Replace (s) => s.autoSendImagesInMultiModal with a multiline format.
[error] 1351-1351: Prettier formatting error: Replace {t('AutoSendImagesInMultiModalDescription')} with a multiline format.
src/components/form.tsx
[error] 21-21: Prettier formatting error: Replace (s) => s.autoSendImagesInMultiModal with a multiline format.
[error] 38-38: Prettier formatting error: Replace autoSendImagesInMultiModal && with a multiline format.
[error] 40-40: Prettier formatting error: Delete extra indentation.
[error] 54-54: Prettier formatting error: Replace long parameter list with multiline format.
🔇 Additional comments (9)
src/features/stores/settings.ts (3)
191-191: LGTM! 新しい設定プロパティが適切に追加されています。プロパティ名が機能を明確に表現しており、型定義も適切です。
442-443: LGTM! 環境変数からのデフォルト値設定が適切です。環境変数のロジックが他のboolean設定と一貫性があり、デフォルトでtrueになることでユーザビリティが向上します。
623-623: LGTM! 永続化設定が適切に追加されています。ユーザーの設定選択が適切に保存され、ブラウザのリロード後も維持されます。
locales/en/translation.json (1)
225-226: LGTM! 英語の翻訳が適切に追加されています。キー名が明確で、翻訳文も機能を正確に説明しています。
locales/ja/translation.json (1)
227-228: LGTM! 日本語の翻訳が適切に追加されています。自然な日本語で機能が正確に説明されており、英語版と意味が一致しています。
src/components/menu.tsx (1)
232-278: 条件分岐ロジックの改善が適切に実装されています画面共有とWebカメラボタンをマルチモーダルモデルから分離する変更が正しく実装されています。これにより、YouTubeモード以外では常にこれらの機能が利用可能になり、画像アップロードボタンのみがマルチモーダルモデル専用として残されています。PR目標に完全に合致した実装です。
src/components/form.tsx (3)
10-11: 新しいインポートが適切に追加されていますマルチモーダルモデル判定とAIサービス型定義のインポートが正しく追加され、新機能の実装に必要な依存関係が適切に管理されています。
21-23: 設定ストアからの状態取得ロジックが正しく実装されています自動画像送信設定とAIサービス・モデル選択の状態取得が適切に実装されています。これにより条件分岐での適切な判定が可能になります。
🧰 Tools
🪛 GitHub Actions: Lint and Format
[error] 21-21: Prettier formatting error: Replace
(s) => s.autoSendImagesInMultiModalwith a multiline format. (prettier/prettier)🪛 GitHub Actions: Run Tests
[error] 21-21: Prettier formatting error: Replace
(s) => s.autoSendImagesInMultiModalwith a multiline format.
37-52: 条件付き画像送信ロジックが正しく実装されていますマルチモーダルモデルと自動送信設定に基づく条件分岐ロジックが適切に実装されています。画像キャプチャが必要な場合のみ遅延送信を行い、それ以外は即座に送信する仕組みが正しく動作します。
🧰 Tools
🪛 GitHub Actions: Lint and Format
[error] 38-38: Prettier formatting error: Replace
autoSendImagesInMultiModal &&with a multiline format. (prettier/prettier)
[error] 40-40: Prettier formatting error: Delete extra indentation. (prettier/prettier)
🪛 GitHub Actions: Run Tests
[error] 38-38: Prettier formatting error: Replace
autoSendImagesInMultiModal &&with a multiline format.
[error] 40-40: Prettier formatting error: Delete extra indentation.
- Disable toggle when non-multimodal model is selected - Auto-turn off toggle when switching from multimodal to non-multimodal model - Apply to all AI service model selection handlers - Skip auto-disable for LMStudio/Ollama/OpenRouter (dynamic model input) Co-authored-by: tegnike <tegnike@users.noreply.github.com>
Co-Authored-By: nikechan <o3nike.teg.14@gmail.com>
…_142811 feat: 画面共有とカメラ共有をマルチモーダルモデルに関係なく利用可能に
概要
マルチモーダルモデルに関係なく画面共有とカメラ共有を利用できるようにする改善。
変更内容
Fixes #340
🤖 Generated with Claude Code
Summary by CodeRabbit
新機能
UIの改善
翻訳