Develop => main #489
Open
Develop => main #489
+25,315
−4,095
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This change refactors the i18n logic to support separate Simplified and Traditional Chinese language variants. Previously, a single 'zh' locale was used for both. - Adds new 'zh-CN' and 'zh-TW' language codes. - Updates all relevant components and data sources to use the new codes. - Ensures proper display of Simplified and Traditional Chinese based on user's browser settings.
Major version upgrades: - Node.js: ^20.0.0 → ^25.2.1 - AI SDK (ai): 4.1 → 6.0.6 - ESLint: 8.x → 9.x with flat config - Next.js: 14.x → 15.x - @charcoal-ui/icons: 2.6.0 → 5.1.0 Breaking changes addressed: - ESLint: Migrate from .eslintrc.json to eslint.config.mjs (flat config) - AI SDK v6: CoreMessage → ModelMessage, maxTokens → maxOutputTokens - AI SDK v6: toDataStreamResponse → createTextStreamResponse - AI SDK v6: Remove deprecated mode parameter from generateObject - Next.js 15: Move API utility files from /api/services/ to /lib/api-services/ - @charcoal-ui/icons v5: Update type assertions for pixiv-icon Other changes: - Remove unintended claude-dispatch.yml workflow file - Add React Compiler rules as warnings for existing code compatibility 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
package.json をなるべく新しいもので動かしたい
Split 1 of the realtime-api branch split strategy. This commit adds: - Kiro SDD (Spec-Driven Development) configuration files - Claude Code agent, command, and skill definitions - Updated .gitignore with new exclusion patterns - Updated .env.example with new environment variables - Enhanced tailwind.config.js - Removed deprecated claude-dispatch workflow Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Refactor useVoiceRecognition, useBrowserSpeechRecognition, useWhisperRecognition hooks - Improve useRealtimeVoiceAPI for better real-time voice API handling - Enhance useAudioProcessing and useSilenceDetection utilities - Update aiModels.ts with OpenAI realtime model configurations - Add comprehensive unit tests for all voice recognition hooks Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
feat: Split 1 - Add development infrastructure (Kiro SDD, Claude Code)
- Fix render-time ref access in useRealtimeVoiceAPI.ts and useVoiceRecognition.ts (use useLayoutEffect)
- Fix defaultModels.openaiRealtime to match openAIRealtimeModels ('gpt-realtime')
- Add afterAll cleanup for global variables in test files
- Fix conditional assertions in useAudioProcessing.test.ts
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
## 変更内容 ### Jest設定の改善 - next/jestを使わない設定に変更（canvasモック問題回避） - jest.setup.jsをCommonJS形式に変換 - jest.resolver.jsを追加してホームディレクトリのnode_modulesを無視 - moduleNameMapperを追加してReactコンポーネントをモック ### GitHub Actions - cairoライブラリのインストールステップを追加 - Ubuntu環境でcanvasが正常に動作するように修正 ### テストの修正 - vercelAi.test.ts: Vercel AI APIの変更に対応（toUIMessageStreamResponse、maxOutputTokens） - useRealtimeVoiceAPI.test.ts: SpeechRecognitionのプロパティ再定義エラー修正 - handlers.test.ts: slidesコンポーネントのモックを追加 - useVoiceRecognition.test.ts: ワーカープロセスタイムアウトのためスキップ ### 常時音声入力モードの改善 - recognitionActiveRefを追加してSpeechRecognition状態を正確に追跡 - isListening React stateと実際のブラウザAPI状態のミスマッチを修正 - 時間ベースのチェックからイベントベースのチェックに変更 ## テスト結果 - 126テスト合格、33テストスキップ - Lint: warning のみ（既存コードの警告） - Build: 成功 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- トリビアルなexpect(true).toBe(true)を実際の検証に置換 - 未使用のstopListeningSpyとcallOrderを削除 - 実際の動作を検証するアサーションに修正 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- expect(true).toBe(true)をmockSpeechRecognition.start/stopの呼び出し確認に変更 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- publicRuntimeConfigを環境変数(NEXT_PUBLIC_BASE_PATH)に移行 - 認識されないoptimizeFontsオプションを削除 - outputFileTracingRootを設定して複数lockfile警告を解消 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- useIsomorphicLayoutEffectを作成してSSR対応 - SpeechRecognition開始時の排他制御を追加 - 複数箇所からの同時呼び出しによるエラーを防止 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Vercel AI SDKの仕様変更により、ストリームプロトコルが 旧形式（0:, 3:, 9: プレフィックス）からSSE JSON形式に変更された。 クライアント側のパーサーを新形式に対応させ、text-delta、 tool-input-start、errorイベントを処理するよう修正。 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
recognition.stop()後のendイベントが発火しない場合に 無限待機になる問題を修正。500msのタイムアウトを設定し、 タイムアウト時は強制的にresolveするよう変更。 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
refactor: Split 2 - Voice recognition hooks and audio processing
OpenAI Embeddingsを使用した過去会話の記憶・検索機能を追加: - メモリ設定UI (ON/OFF、類似度閾値、検索上限) - IndexedDBを使用したローカルメモリストア - 過去会話の自動保存と類似検索 - メモリコンテキストのプロンプト追加機能 - ログファイルからの記憶復元機能 主な変更: - src/features/memory/ - メモリ機能のコアロジック - src/components/settings/memorySettings.tsx - 設定UI - src/pages/api/embedding.ts - Embedding API - Message型にembeddingプロパティを追加 - jest.config.js - JSX変換の設定修正 パッケージ追加: - idb (IndexedDB wrapper) - fake-indexeddb (テスト用) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Updated translation strings for memory settings and conversation history. - Enhanced MemorySettings component to include vectorization options during restore. - Removed log settings from the main settings menu. - Improved API key input handling and visibility based on memory settings. - Added functionality to handle file restoration with vectorization and target log file management. - Updated save-chat-log API to support overwriting existing logs and specifying target filenames. - Cleaned up test cases to reflect changes in memory settings UI and behavior.
Feature/add movie pngtuber
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Claude finished @tegnike's task —— View job PR Review - Comprehensive AnalysisI've conducted a thorough review of this substantial develop → main PR with 100 changed files. Here's my comprehensive feedback: Review Checklist
🎯 Overall Assessment: Strong Implementation with Minor ConcernsThis is an impressive feature-rich release that adds significant value to AITuberKit. The codebase shows excellent engineering practices, comprehensive testing, and thoughtful architecture. ✅ Strengths1. Memory System (RAG-based) - Excellent Implementation
2. PNGTuber Engine - Production-Ready
3. Test Coverage - Exceptional
4. Configuration & Tooling
|
WalkthroughPNGTuberアバター機能、埋め込みベースの長期メモリ（IndexedDB）サブシステムとAPI/UI、音声認識フック群の制御フロー変更、多言語（zh-CN/zh-TW）対応、大量のテスト追加、ESLint/Jest/依存更新、および多数のドキュメント（.claude/.kiro）追加を行いました。 Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant UI as MemorySettings
participant Store as Settings Store
participant Sync as MemoryStoreSync
participant Service as MemoryService
participant IDB as IndexedDB
participant API as /api/embedding
User->>UI: メモリ有効化 / メッセージ送信
UI->>Store: setMemoryEnabled / trigger saveMessageToMemory
Store->>Sync: saveMessageToMemory(message)
Sync->>API: POST /api/embedding (text)
API-->>Sync: embedding vector
Sync->>Service: save(record with embedding)
Service->>IDB: put(record)
User->>UI: コンテキスト要求 (query)
UI->>Store: searchMemoryContext(query)
Store->>API: POST /api/embedding (query)
API-->>Store: query vector
Store->>Service: search(query vector)
Service->>IDB: getAll()/filter by similarity
Service-->>Store: topResults
Store-->>UI: buildContext(topResults)
sequenceDiagram
participant Browser
participant Comp as PNGTuberComponent
participant Engine as PNGTuberEngine
participant Canvas as Canvas2D
participant Audio as AudioContext
Browser->>Comp: mount
Comp->>Engine: new PNGTuberEngine(video, canvases)
Comp->>Engine: loadAsset(path)
Engine->>Canvas: init canvases
Engine->>Audio: initAudioContext()
Browser->>Comp: play audio buffer
Comp->>Engine: playAudioFromBuffer(buffer)
Engine->>Audio: decode & play
Engine->>Engine: lip-sync analysis -> select mouth sprite
Engine->>Canvas: draw mouth sprite (apply chroma-key if enabled)
Engine-->>Comp: play finished
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
🧹 Recent nitpick comments
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 19
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (6)
src/pages/api/convertSlide.ts (2)
212-246: 非Anthropicプロバイダーでスキーマ検証が欠落しています。
output: 'no-schema'を使用しているため、OpenAIやGoogleからのレスポンスはSlideLineResponseインターフェースに対して検証されません。AIが予期しない形式のJSONを返した場合、lineやnotesプロパティにアクセスする際にランタイムエラーが発生する可能性があります。また、246行目の二重型アサーション (
as unknown as) はTypeScriptの型安全性をバイパスしています。🔧 提案: 全プロバイダーでスキーマを使用するか、ランタイム検証を追加
オプション1: 全プロバイダーでスキーマを使用
} else { response = await generateObject({ model: instance(model), messages: [ // ...messages ], - output: 'no-schema', + schema: schema, }) }オプション2: ランタイム検証を追加
if (!response || !response.object) { throw new Error('Invalid response from AI service') } + const obj = response.object as Record<string, unknown> + if (typeof obj.line !== 'string' || typeof obj.notes !== 'string') { + throw new Error('Invalid response structure: missing line or notes') + } - return response.object as unknown as SlideLineResponse + return { line: obj.line, notes: obj.notes } as SlideLineResponse
277-282:folderNameのパストラバーサル脆弱性を確認してください。
folderNameはユーザー入力から直接取得され、path.joinに渡されています。../を含む値が渡された場合、意図しないディレクトリにファイルが書き込まれる可能性があります。🛡️ 入力サニタイズの提案
+ // フォルダ名のサニタイズ + const sanitizedFolderName = (folderName || 'defaultFolder').replace(/[^a-zA-Z0-9_-]/g, '_') const slideDir = path.join( process.cwd(), 'public', 'slides', - folderName || 'defaultFolder' + sanitizedFolderName ).github/workflows/test.yml (1)
19-23: node-versionがREADME.mdと不整合README.mdでは
Node.js: ^25.2.1を指定していますが、ここではnode-version: '20'を使用しています。ドキュメントとCIで整合性を取る必要があります。README.mdのバージョンを下げるか、CIのバージョンを上げるか、プロジェクトの要件に合わせて統一してください。
📝 README.mdに合わせる場合の修正案
- name: Setup Node.js uses: actions/setup-node@v3 with: - node-version: '20' + node-version: '25' cache: 'npm'.claude/commands/kiro/spec-design.md (1)
197-197: プレースホルダーテキストを削除してください。"think hard" はデバッグ用のプレースホルダーまたは開発中の残留テキストのようです。正式なドキュメントとしては不適切なので、削除するか、意図された内容に置き換えてください。
🧹 修正案
- -think hardsrc/features/constants/aiModels.ts (1)
19-122: 複数のモデルIDに実在性の問題が見られます。即座の修正が必要です。以下のモデルIDは検証の結果、問題が確認されました:
- OpenAI: gpt-5.* シリーズ (gpt-5.2-pro、gpt-5.1-codex など) の正確な API 名が公式ドキュメントで明確に確認できません。OpenAI 公式 API リファレンスで実際のモデル名を明示的に確認してください。
- Cohere:
command-lightは 2025 年 9 月の廃止予告リストに含まれており、現在は利用できない可能性があります。- Perplexity:
sonar-deep-researchは公式ドキュメント内で言及されておらず、実在しないモデルである可能性が高いです。sonar、sonar-proのみ確認されています。- Google/Mistral: gemini-2.5-pro-preview-05-06、pixtral-large-latest など、内部に日付を含むバージョン文字列の正確な API 名を各プロバイダの公式リファレンスで確認してください。
推奨事項:
- OpenAI、Anthropic、Google、Mistral などのプロバイダについては、公式 API ドキュメントおよび最新のモデルリストを参照して、使用する各モデルの正確な API 識別子を確認してください。
- Groq、Cohere、Perplexity などが提供する
/modelsエンドポイントを呼び出して、実際に利用可能なモデルを取得し、コード内のリストと照合してください。- 廃止予告されたモデル (特に Cohere の command-light) は代替モデルに置き換えてください。
src/components/settings/index.tsx (1)
147-175: 旧タブ値のフォールバックを入れてください。
永続化されたactiveSettingsTabが旧値（例:log）のままだと、タブ表示が空になり得ます。未定義キーは既定タブへフォールバック or マイグレーションが必要です。🛠️ 例: 未知キーを既定タブへ戻す
+ useEffect(() => { + if (!tabs.some((tab) => tab.key === activeTab)) { + setActiveTab('description') + } + }, [activeTab])
🤖 Fix all issues with AI agents
In @.claude/commands/kiro/spec-status.md:
- Line 97: ドキュメント末尾に残っている文脈のない単語 "think" を削除してください — ファイル内の孤立したトークン "think"
をそのまま取り除き、余分な空白や改行が生じる場合は整えて文書末尾が自然に終わるようにしてください。
In @.claude/commands/kiro/spec-tasks.md:
- Line 153: Remove the stray, context-less word "think" from the end of the
document in .claude/commands/kiro/spec-tasks.md (it’s the leftover fragment
shown in the diff); also scan .claude/commands/kiro/spec-status.md for the same
orphaned "think" occurrence mentioned in the review and remove it so both files
no longer contain the dangling token.
In @.claude/skills/openai-voice-agents/docs/build.md:
- Around line 170-171: The sentence "Additionally to the arguments that the
agent called a particular tool with, you can also access a snapshot of the
current conversation history..." is awkward and should be rephrased; change it
to "In addition to the arguments with which the agent called a particular tool,
you can access a snapshot of the current conversation history tracked by the
Realtime Session." Also review the other occurrences noted (the similar phrasing
at the later paragraphs and the block in lines 400-403) and correct any
typos/ungrammatical constructions to match this clearer wording and ensure
consistent terminology ("Realtime Session", "conversation history", "tools for
delegation").
- Around line 586-592: Typo in the Zod schema: change the outputType object's
key 'reasong' to 'reason' in the schema definition (z.object({...})) and update
any occurrences/usages of the 'reasong' property referenced elsewhere to
'reason' (search for 'reasong' and rename in functions or examples that consume
this schema such as the outputType declaration and any example responses).
In @.claude/skills/openai-voice-agents/docs/quickstart.md:
- Around line 57-58: Fix the minor typos and inconsistent escaping in
quickstart.md: change "a the top level" to "at the top level" on the sentence
that mentions the response containing a "value" string, and normalize the
ephemeral key prefix formatting across all occurrences (Lines referencing the
“ek\_” prefix) to a consistent, readable form such as `ek_` (or plain ek_) and
ensure the same wording is used at the three locations (the sentence near the
"value" string and the two other places referenced) for consistent phrasing.
In @.env.example:
- Around line 491-496: The .env.example currently sets
NEXT_PUBLIC_DEMO_MODE="true", which disables filesystem and local-server
features by default and can confuse developers; change the default to false in
the example file by setting NEXT_PUBLIC_DEMO_MODE="false" so local development
behaves normally and only enable demo mode explicitly when needed (update the
NEXT_PUBLIC_DEMO_MODE entry and any surrounding comment to reflect the new
default).
In `@README.md`:
- Around line 121-122: README.md currently pins Node.js ^25.2.1 and npm ^11.6.2
which are non-LTS/known-broken; update the Node.js entry to an LTS major (e.g.,
^20.x or ^24.x) and set npm to a stable compatible release (e.g., ^10.x or the
npm bundled with the chosen Node LTS), and ensure the README's Node/npm versions
match the Node version used in .github/workflows/test.yml by aligning the
workflow's Node setup (refer to the Node.js version line in the workflow) so
documentation and CI use the same LTS versions.
In `@src/__tests__/features/memory/memoryContextBuilder.test.ts`:
- Around line 244-262: The test constructs timestamps with `2025-01-${String(i +
1).padStart(2, '0')}T12:00:00Z` which produces invalid dates when i+1 > 31;
update the loop that calls `createMemoryRecord` so the timestamp is generated
from a real Date (e.g., create a base Date and add i days, or use the Date
constructor like new Date(2025, 0, 1 + i).toISOString()) before passing it to
`createMemoryRecord` used by `builder.buildContext` and
`builder.estimateTokens`, ensuring all produced timestamps are valid ISO
strings.
In `@src/__tests__/hooks/useRealtimeVoiceAPI.test.ts`:
- Around line 65-108: The afterAll restoration leaves mocks in place when
originalSpeechRecognition/originalWebkitSpeechRecognition are undefined and may
fail to delete because the mock was defined without configurable: true; update
the setup to define the mocks with configurable: true (where
MockSpeechRecognitionClass is assigned to window.SpeechRecognition and
window.webkitSpeechRecognition) and change the afterAll in this test to always
remove the mocked properties (delete (window as any).SpeechRecognition and
delete (window as any).webkitSpeechRecognition) and then, if originals exist,
re-define them with configurable: true and writable: true using
Object.defineProperty; ensure restoration runs unconditionally so mocks cannot
leak to other suites (refer to originalSpeechRecognition,
originalWebkitSpeechRecognition, MockSpeechRecognitionClass, and afterAll).
In `@src/components/pngTuberViewer.tsx`:
- Around line 23-25: The container div uses a non-existent Tailwind class "z-5"
which will not apply the intended stacking; update the class on the element that
renders PNGTuberComponent (the <div className="fixed inset-0 w-screen h-screen
z-5"> in src/components/pngTuberViewer.tsx) to a valid z-index utility such as
"z-10" (or another existing value like "z-0"/"z-20"), or alternatively add "z-5"
to your Tailwind config's theme.extend.zIndex if you truly need that specific
value—pick one approach and make the corresponding change to the class or
Tailwind config.
In `@src/components/settings/memorySettings.tsx`:
- Around line 274-279: The handleChangeChatLog implementation replaces each
message with a new object containing only role and content, dropping other
properties (id, timestamp, embedding, etc.); fix it in the handleChangeChatLog
function by mapping chatLog and returning i === targetIndex ? { ...m, content:
text } : m so you spread the original message (preserving
id/timestamp/embedding) and only overwrite content, then call
homeStore.setState({ chatLog: newChatLog }) as before and ensure
types/interfaces accept the preserved fields.
- Around line 246-258: The POST to '/api/save-chat-log' in memorySettings.tsx
currently ignores HTTP errors; wrap the fetch in a try/catch and check the
Response object (e.g., const res = await fetch(...); if (!res.ok) throw new
Error(`Save failed: ${res.status} ${res.statusText}`)); on error, log or surface
it and avoid calling fetchMemoryCount() when the save failed. Update the code
paths around restoredMessages, selectedFile.name and fetchMemoryCount() to only
proceed after a successful response and ensure any thrown errors are caught and
handled appropriately.
In `@src/features/chat/vercelAIChat.ts`:
- Around line 312-315: The SSE handler currently only flushes the remaining
buffer when isPlainTextStream is true, so incomplete final lines in SSE mode get
dropped; update the logic where buffer is checked before stream close (the block
around isPlainTextStream and controller.enqueue(buffer)) to always enqueue any
non-empty buffer for SSE streams as well (i.e., ensure
controller.enqueue(buffer) runs when buffer is non-empty regardless of
isPlainTextStream), and then clear/reset buffer appropriately so the final
partial data is emitted before the stream closes.
In `@src/features/memory/memoryContextBuilder.ts`:
- Around line 31-44: formatTimestamp currently constructs a Date from
isoTimestamp without validating it, which can produce NaN components; add a
guard in formatTimestamp that checks for an invalid Date (e.g.,
isNaN(date.getTime()) or null/undefined isoTimestamp) and short-circuits with a
clear fallback (for example return "[Invalid Date]" or return the original
isoTimestamp wrapped) before performing the JST conversion and UTC getters so
the rest of the function (jstDate calc and year/month/day/hours/minutes
extraction) never operates on an invalid date.
In `@src/features/memory/memoryStoreSync.ts`:
- Around line 50-72: The searchMemoryContext function should skip searching for
empty or whitespace-only queries to avoid returning irrelevant memories; add an
early return at the start of searchMemoryContext (before calling
settingsStore.getState or before getMemoryService) that trims the query and
returns an empty string when query.trim() === ''. Ensure the change references
the existing function searchMemoryContext and does not alter
MemoryContextBuilder usage or the existing error handling.
- Around line 203-271: The restoreMemoryFromFile function currently aborts the
whole operation if any single memoryService.restoreMemory call throws, causing
inconsistent partial restores; change it so each message restore is wrapped in
its own try/catch inside the loop (refer to restoreMemoryFromFile and
memoryService.restoreMemory), increment actualRestoredCount only on successful
restores, track a failedCount (or collect failed IDs) and log per-item errors
(console.error) without rethrowing, and after the loop return a result object
that accurately reports restoredCount and embeddingCount and sets success based
on whether any items were restored (e.g., success = actualRestoredCount > 0)
instead of letting one failure trigger the outer catch.
In `@src/hooks/useVoiceRecognition.ts`:
- Around line 80-107: The restart logic currently calls
currentHookRef.current.startListening() unconditionally inside setTimeout,
causing possible double-starts; update both the stop handler block and
handleSpeakCompletion to, inside the setTimeout callback, re-read the latest
flags (settingsStore.getState().continuousMicListeningMode,
settingsStore.getState().speechRecognitionMode === 'browser',
homeStore.getState().chatProcessing) and check the current hook's listening
state (e.g., currentHookRef.current?.isListening() or equivalent) before calling
currentHookRef.current.startListening(); only call startListening() when
continuousMicListeningMode is still true, speechRecognitionMode is 'browser',
chatProcessing is false, and the hook is not already listening.
In `@src/pages/api/embedding.ts`:
- Around line 82-92: In the catch block in src/pages/api/embedding.ts replace
the direct check of error.status with an explicit OpenAI APIError check: first
test if error instanceof OpenAI.APIError (from the OpenAI SDK), then read
error.status to handle the 429 rate-limit branch and other HTTP error statuses;
for non-APIError errors, handle them separately (log and return a generic 500
response). Update the catch around the embedding request (the catch that
currently logs "Embedding API error:") to use this instanceof pattern so only
APIError instances have their .status accessed and other errors are safely
handled.
In `@src/pages/api/memory-restore.ts`:
- Around line 38-41: The current filename safety check only looks for '..' and
'/' and misses Windows backslashes; replace the ad-hoc check by using
path.basename to canonicalize the name: require/import Node's path, and in the
handler verify that filename === path.basename(filename) (and optionally still
reject filenames that include '..' for extra safety) before proceeding—if it
doesn't match, return res.status(400).json({ message: 'Invalid filename' });
reference the filename variable and the response object res in
src/pages/api/memory-restore.ts when implementing this change.
♻️ Duplicate comments (1)
.claude/commands/kiro/spec-requirements.md (1)
106-106: "think" ディレクティブについて。
spec-impl.mdと同様に、ファイル末尾にthinkディレクティブがあります。すべてのKiroドキュメントで一貫して使用されている場合は、その目的を共通ドキュメントに記載することを推奨します。
🧹 Nitpick comments (69)
src/components/iconButton.tsx (1)
32-32:as anyの使用は型安全性を損なうため、より具体的な型キャストを検討してください。
String()からas anyへの変更は、TypeScriptの型チェックを完全にバイパスします。pixiv-iconのnameプロパティが期待する具体的な型（例:string）へのキャストを使用することで、型安全性を維持しつつエラーを解消できます。♻️ 提案する修正
- <pixiv-icon name={(isProcessingIcon || '24/Dot') as any} scale="1" /> + <pixiv-icon name={(isProcessingIcon || '24/Dot') as string} scale="1" />- <pixiv-icon name={iconName as any} scale="1" /> + <pixiv-icon name={iconName as string} scale="1" />Also applies to: 43-43
src/utils/buildUrl.ts (1)
6-15: スラッシュ正規化で//を防止
NEXT_PUBLIC_BASE_PATHが末尾/付きで、pathが先頭/の場合に//が生成される可能性があります。rootが空のときは相対パスのままにしつつ、結合時のみ正規化するのが安全です。♻️ 修正案
- const root = process.env.NEXT_PUBLIC_BASE_PATH || '' + const root = (process.env.NEXT_PUBLIC_BASE_PATH || '').replace(/\/+$/, '') // 空白などの特殊文字を含むパスを適切にエンコード // ただし、パス区切り文字（/）はエンコードしない const encodedPath = path .split('/') .map((segment) => encodeURIComponent(segment)) .join('/') - return root + encodedPath + if (!root) return encodedPath + const needsSlash = !encodedPath.startsWith('/') + return root + (needsSlash ? '/' : '') + encodedPathnext.config.js (1)
7-10: BASE_PATH を正規化して一貫適用したいです。
BASE_PATHに末尾スラッシュや先頭なしが入るとbasePath/assetPrefixが無効化される恐れがあります。正規化して 1 か所で使い回すのを推奨します。♻️ 変更案
-const nextConfig = { - reactStrictMode: true, - assetPrefix: process.env.BASE_PATH || '', - basePath: process.env.BASE_PATH || '', - trailingSlash: true, - outputFileTracingRoot: __dirname, - env: { - NEXT_PUBLIC_BASE_PATH: process.env.BASE_PATH || '', - }, -} +const rawBasePath = process.env.BASE_PATH || '' +const normalizedBasePath = rawBasePath + ? `/${rawBasePath.replace(/^\/|\/$/g, '')}` + : '' + +const nextConfig = { + reactStrictMode: true, + assetPrefix: normalizedBasePath, + basePath: normalizedBasePath, + trailingSlash: true, + outputFileTracingRoot: __dirname, + env: { + NEXT_PUBLIC_BASE_PATH: normalizedBasePath, + }, +}.kiro/settings/templates/steering/structure.md (1)
7-19: 任意: 追加パターンセクションの検討現在の構造は適切ですが、以下のパターンセクションを追加することで、テンプレートの有用性がさらに向上する可能性があります：
- テストファイルの配置: ユニットテスト、インテグレーションテストなどの配置パターン
- ビルド/出力ディレクトリ: 生成されるファイルの配置場所
- 静的アセット: 画像、フォント、スタイルシートなどの整理方法
ただし、現在の柔軟な構造でも十分に実用的であり、プロジェクトの必要に応じて追加できます。
src/__tests__/pages/api/convertSlide.test.ts (2)
58-60: 未使用のモック:mockCreateGoogleにはテストがありません。
mockCreateGoogleが宣言されていますが、対応するテストケースがありません。Googleプロバイダーのパスがテストされていないことを意味します。🧪 Googleプロバイダーのテストを追加
it('invokes Google models and returns parsed object', async () => { const googleFactory = jest.fn().mockReturnValue('google-model') mockCreateGoogle.mockReturnValue(googleFactory) mockGenerateObject.mockResolvedValue({ object: { line: 'google-line', notes: 'google-notes' }, } as any) const result = await createSlideLine( baseImage, 'google-key', 'google', 'gemini-pro-vision', 'Japanese', null ) expect(mockCreateGoogle).toHaveBeenCalledWith({ apiKey: 'google-key' }) expect(googleFactory).toHaveBeenCalledWith('gemini-pro-vision') expect(result).toEqual({ line: 'google-line', notes: 'google-notes' }) })
66-133: テストスイートは主要なパスをカバーしていますが、エッジケースの追加を検討してください。現在のテストは OpenAI、Anthropic、およびマルチモーダル非対応のケースをカバーしています。以下のケースも追加すると、より堅牢なテストになります：
- サポートされていないAIサービスに対するエラー
- APIエラーのハンドリング
- 不正なレスポンス形式のハンドリング
🧪 追加テストケースの例
it('throws when AI service is not supported', async () => { await expect( createSlideLine(baseImage, 'key', 'unsupported', 'model', 'English', null) ).rejects.toThrow('is not supported for slide conversion') }) it('throws when generateObject returns invalid response', async () => { const modelFactory = jest.fn().mockReturnValue('openai-model') mockCreateOpenAI.mockReturnValue(modelFactory) mockGenerateObject.mockResolvedValue({ object: null } as any) await expect( createSlideLine(baseImage, 'key', 'openai', 'gpt-4o', 'English', null) ).rejects.toThrow('Invalid response from AI service') })src/__tests__/lib/api-services/vercelAi.test.ts (1)
51-54: 未使用のモック変数があります。
mockCreateTextStreamResponseは宣言されていますが、テスト内で使用されていません。不要であれば削除してください。♻️ 提案される修正
const mockStreamText = streamText as jest.MockedFunction<typeof streamText> const mockGenerateText = generateText as jest.MockedFunction< typeof generateText > -const mockCreateTextStreamResponse = - createTextStreamResponse as jest.MockedFunction< - typeof createTextStreamResponse - >また、インポートからも削除:
-import { streamText, generateText, createTextStreamResponse } from 'ai' +import { streamText, generateText } from 'ai'モック定義からも削除:
jest.mock('ai', () => { const actual = jest.requireActual('ai') return { ...actual, streamText: jest.fn(), generateText: jest.fn(), - createTextStreamResponse: jest.fn(), } }).claude/skills/codex-reviewer/SKILL.md (3)
10-21: 前提条件とセットアップ手順の追加を推奨します。コマンドとパラメータは明確に文書化されていますが、以下の情報が不足しています：
- インストール方法:
codexコマンドのインストール手順- 前提条件: 必要な依存関係や認証情報
- 環境設定: 初回セットアップや設定ファイルの要否
これらの情報を追加することで、このスキルを使用する際のオンボーディングが容易になります。
📝 セットアップセクションの追加例
## 実行コマンド +## 前提条件 + +- `codex` CLI ツールのインストール（インストール方法: [リンク]） +- 適切な認証情報の設定（必要に応じて） +- 対象プロジェクトへの読み取りアクセス権限 + codex exec --full-auto --sandbox read-only --cd <project_directory> "<request>"
23-31: 使用例にサンプル出力を追加すると更に有用です（オプショナル）。使用例は明確で実用的ですが、期待される出力のサンプルを追加すると、ユーザーがこのスキルの動作をより具体的にイメージできます。
💡 出力例の追加案
### コードレビュー codex exec --full-auto --sandbox read-only --cd /path/to/project "このプロジェクトのコードをレビューして、改善点を指摘してください" + +**出力例:** +``` +## コードレビュー結果 + +### 改善点 +1. src/auth.ts: 認証トークンの検証が不足しています +2. src/api/handler.ts: エラーハンドリングの改善が必要です +... +``` ### バグ調査
1-4: セットアップドキュメントの追加を検討してください。OpenAI Codex CLI は現在利用可能です（2025年5月リリース）。スキル定義の参照は正確です。
ただし、ユーザーが本スキルを使用するためには、以下の情報があると便利です：
- インストール手順:
npm i -g@openai/codex`` コマンドでのインストール方法の記載- 前提条件: Node.js/npm のバージョン要件
- プラットフォーム対応: macOS/Linux 完全対応、Windows は実験的サポート
- 認証: OpenAI API キーやログイン設定が必要な場合その説明
これらはオプショナルな改善ですが、ユーザー体験の向上につながります。
.kiro/settings/templates/steering-custom/error-handling.md (1)
20-29: エラーシェイプの拡張を検討してください。基本的なエラー形式は適切ですが、以下のフィールドの追加を検討することをお勧めします：
detailsまたはcontext: 追加の診断情報用（特にバリデーションエラーの場合）pathまたはfield: APIバリデーションエラーでどのフィールドが問題かを示すため💡 拡張されたエラー形式の例
{ "error": { "code": "ERROR_CODE", "message": "Human-readable message", "requestId": "trace-id", "timestamp": "ISO-8601", "details": { "field": "email", "reason": "invalid format" } } }.kiro/settings/rules/ears-format.md (1)
7-9: ローカリゼーションガイダンスに例を追加することを検討してください。英語のトリガーキーワードを保持し、可変部分のみをローカライズするという原則は明確ですが、正しい使用例と誤った使用例を追加すると、より理解しやすくなります。
💡 例の追加案
**正しい例（日本語の場合）:** - When ユーザーがチェックアウトボタンをクリックしたとき, the Checkout Service shall カート内容を検証する **誤った例:** - ユーザーがチェックアウトボタンをクリックしたとき, システムは shall カート内容を検証する.kiro/settings/templates/specs/init.json (1)
5-5: 言語設定のパラメータ化を検討してください。言語フィールドが"ja"にハードコードされています。複数言語対応が必要な場合は、
{{LANGUAGE}}のようなプレースホルダーに変更することを検討してください。♻️ 提案される変更
- "language": "ja", + "language": "{{LANGUAGE}}",.kiro/settings/templates/steering-custom/security.md (1)
1-67: セキュリティ標準ドキュメントの構成は適切です。このドキュメントは、セキュリティのベストプラクティスを網羅的にカバーしており、防御の多層化、最小権限、入力検証、認証・認可、シークレット管理、セッションセキュリティ、セキュアロギング、トランスポートセキュリティ、脆弱性管理などの重要なトピックを含んでいます。TypeScriptのパターン例も実用的で分かりやすいです。
軽微なフォーマット改善
ファイルの最後に改行を追加することをお勧めします（POSIX標準に準拠）。
_Focus on patterns and principles. Link concrete configs to ops docs._ +.kiro/settings/templates/steering-custom/api-standards.md (1)
1-86: API標準ドキュメントは明確で実用的です。このドキュメントは、RESTful APIの設計パターンを適切にカバーしており、エンドポイント命名規則、HTTPメソッドのセマンティクス、リクエスト/レスポンス形式、ステータスコード、認証、バージョニング、ページネーションなど、重要な要素を含んでいます。JSONの例も分かりやすく、実装の参考になります。
軽微なフォーマット改善
ファイルの最後に改行を追加することをお勧めします。
_Focus on patterns and decisions, not endpoint catalogs._ +.kiro/settings/rules/design-review.md (1)
1-127: デザインレビュープロセスのドキュメントは非常に充実しています。このドキュメントは、技術設計文書の品質保証プロセスを明確に定義しており、以下の優れた特徴があります：
- インタラクティブな対話を重視したレビュー哲学
- 重要な問題を3つまでに限定する焦点を絞ったアプローチ
- 要件へのトレーサビリティと証拠の要求
- 長所と短所のバランスの取れた評価
- 明確なGO/NO-GO判定基準
レビュー基準（既存アーキテクチャとの整合性、設計の一貫性、拡張性、型安全性）も包括的で実用的です。
軽微なフォーマット改善
ファイルの最後に改行を追加することをお勧めします。
- **Decision**: GO/NO-GO with clear rationale and next steps +.kiro/settings/rules/steering-principles.md (2)
1-99: ステアリング原則ドキュメントは明確で実用的です。「プロジェクトメモリ」と「網羅的な仕様書」の区別を明確にしており、ゴールデンルール「新しいコードが既存のパターンに従う場合、ステアリングの更新は不要」は優れたガイダンスです。良い例・悪い例の比較も非常に分かりやすいです。
軽微なフォーマット改善
ファイルの最後に改行を追加することをお勧めします。
- **Custom files**: Specialized patterns (API, testing, security, etc.) +
88-88:.kiro/ディレクトリの参照に関するガイダンスの明確化を検討してください。88行目では「
.kiro/specs/と.kiro/steering/への軽い参照は許容される；他の.kiro/ディレクトリは避ける」と記載されていますが、このファイル自体が.kiro/settings/rules/に配置されています。.kiro/settings/への参照が許容されるかどうかを明確にすることで、ガイダンスの一貫性が向上します。.kiro/settings/templates/steering-custom/database.md (1)
1-56: データベース標準ドキュメントは包括的で実用的です。このドキュメントは、データベース設計の重要な側面を適切にカバーしており、以下の優れた内容を含んでいます：
- ドメインファーストモデリングの哲学
- 明確な命名規則（テーブル、カラム、外部キー）
- リレーションシップパターン（1:N、N:N、1:1）
- イミュータブルなマイグレーション戦略とロールバック
- N+1問題を避けるクエリパターン
- トランザクション管理とコネクションプーリング
- データ整合性のための制約（NOT NULL、UNIQUE、CHECK、FK）
- バックアップとリカバリの考慮事項
実装に役立つ実践的なガイダンスです。
軽微なフォーマット改善
ファイルの最後に改行を追加することをお勧めします。
_Focus on patterns and decisions. No environment-specific settings._ +.claude/skills/openai-voice-agents/docs/index.md (1)
17-32: 主要機能リストは包括的ですが、軽微な改善を推奨します。機能リストは明確ですが、以下の点を検討してください：
- Lines 19-20のHTMLアンカーリンクは、Markdownドキュメントでは不要な可能性があります
- Line 32の画像URL（diagram-chained-agent.png）の有効性を確認してください
♻️ HTMLアンカーリンクを削除する提案
### Key features -[Section titled "Key features"](https://openai.github.io/openai-agents-js/guides/voice-agents/index.html#key-features) - - Connect over WebSocket or WebRTC.claude/agents/playwright-reporter.md (1)
64-74: 実行前チェックとツールリストが適切ですが、軽微な表記の確認が必要です。ディレクトリ作成の事前チェックとツールの責務マッピングは明確ですが、Line 73の
mcp**chrome-devtools**\*の表記を確認してください。Markdown のエスケープが意図通りかどうかを検証することを推奨します。♻️ ツール名の表記を修正する提案
もし
mcp__chrome-devtools__*が正しい表記であれば、以下のように修正することを検討してください：-- **mcp**chrome-devtools**\***: ブラウザ操作（スクリーンショット、ページ操作など） +- **mcp__chrome-devtools__\***: ブラウザ操作（スクリーンショット、ページ操作など）.kiro/settings/rules/tasks-generation.md (2)
74-78: オプショナルマーカーの適用基準に矛盾があります。Line 76で「acceptance criteriaを直接参照するsub-taskにのみオプショナルマーカーを適用する」と述べていますが、Line 77-78で「実装作業や統合に不可欠な検証は決してオプショナルとしてマークしない」と述べています。acceptance criteriaを参照するタスクは通常、実装に不可欠なため、この2つの指示は矛盾しているように見えます。
基準を明確化することをお勧めします。例えば：「テストカバレッジのみを目的とし、MVP機能には影響しない追加的な検証タスクのみをオプショナルとしてマークする」
♻️ 提案する修正
### Optional Test Coverage Tasks -- When the design already guarantees functional coverage and rapid MVP delivery is prioritized, mark purely test-oriented follow-up work (e.g., baseline rendering/unit tests) as **optional** using the `- [ ]*` checkbox form. -- Only apply the optional marker when the sub-task directly references acceptance criteria from requirements.md in its detail bullets. -- Never mark implementation work or integration-critical verification as optional—reserve `*` for auxiliary/deferrable test coverage that can be revisited post-MVP. +- When the design already guarantees functional coverage and rapid MVP delivery is prioritized, mark purely test-oriented follow-up work (e.g., baseline rendering/unit tests) as **optional** using the `- [ ]*` checkbox form. +- Only apply the optional marker to auxiliary test tasks that verify already-implemented functionality, not tasks that implement acceptance criteria for the first time. +- Never mark implementation work or integration-critical verification as optional—reserve `*` for deferrable test coverage (e.g., edge case tests, additional unit test suites) that can be revisited post-MVP.
113-131: チェックボックス形式の例にオプショナルマーカーが含まれていません。Lines 74-78でオプショナルタスクのマーカー
- [ ]*について説明していますが、このセクションのチェックボックス形式の例には含まれていません。一貫性を保つため、オプショナルマーカーの使用例を追加することを推奨します。♻️ 提案する追加例
- [ ] 1.3 Sub-task description - Detail items... - _Requirements: Z.Z, W.W_ +- [ ]* 1.4 Optional sub-task description (e.g., additional test coverage) + - Detail items... + - _Requirements: Z.Z_ + - [ ] 2. Next major task (NOT 1 again!) - [ ] 2.1 Sub-task....kiro/settings/templates/specs/research.md (1)
41-43: ステアリングプリンシパルへの参照が不明確です。Line 43の例で「Aligns with existing steering principle X」と記載されていますが、「steering principles」が何を指しているのか、どこで定義されているのかが不明です。テンプレート利用者のために、このコンセプトの簡単な説明または参照先を追加することを推奨します。
.kiro/settings/rules/design-discovery-full.md (1)
29-42: WebSearchとWebFetchツールの定義が必要です。このセクションで「Use WebSearch」(line 32)と「Use WebFetch」(line 38)について言及していますが、これらのツールの説明や使用方法、アクセス方法に関する情報が提供されていません。
リーダーまたは別のドキュメントへの参照を追加して、これらのツールが何であり、どのように使用するかを明確にすることを推奨します。
♻️ 提案する改善
以下のような説明を追加することを検討してください：
### 3. Technology Research **Investigate Best Practices and Solutions**: +> Note: WebSearch and WebFetch are discovery tools available in the development environment. See [tool-reference.md] for usage details. + - **Use WebSearch** to find:.claude/skills/openai-voice-agents/docs/transport.md (1)
56-86: WebSocketの例でaudioバッファ変数が未使用です。Line 67で
myRecordedArrayBufferが宣言されていますが、Line 85でsendAudio(myRecordedArrayBuffer)として使用されています。しかし、Line 67では空のArrayBuffer(0)として初期化されているため、この例では実際の音声データが送信されません。これはドキュメントの例示コードなので重大な問題ではありませんが、コメントを追加して実際の使用時には適切な音声データで初期化する必要があることを明示すると良いでしょう。
♻️ 提案する改善
-const myRecordedArrayBuffer = new ArrayBuffer(0); +// In production, replace with actual recorded audio data +const myRecordedArrayBuffer = new ArrayBuffer(0);.kiro/settings/templates/specs/requirements.md (1)
11-11: 強制コメントの文言に軽微な曖昧性があります。コメントには「leading numeric ID only」と記載されていますが、実際のテンプレート（lines 9, 23）では「Requirement 1:」という形式を使用しており、これには「Requirement」という単語が含まれています。
厳密には「numeric ID only」ではなく「numeric ID（数値の後に任意のテキスト説明）」が正しい説明です。誤解を避けるため、コメントを以下のように明確化することを推奨します：
♻️ 提案する修正
-<!-- Requirement headings MUST include a leading numeric ID only (for example: "Requirement 1: ...", "1. Overview", "2 Feature: ..."). Alphabetic IDs like "Requirement A" are not allowed. --> +<!-- Requirement headings MUST start with a numeric ID (for example: "Requirement 1: ...", "1. Overview", "2 Feature: ..."). Alphabetic IDs like "Requirement A" are not allowed. -->.claude/commands/kiro/validate-design.md (1)
17-17: XMLタグのインデントが不揃いです。Line 17 の
</background_information>タグに余分なスペースインデントがあります。Line 54 の</instructions>タグも同様です。一貫性のためにインデントを揃えることを推奨します。📝 提案される修正
- </background_information> +</background_information>Line 54:
- </instructions> +</instructions>.claude/commands/kiro/validate-impl.md (1)
18-18: 閉じタグのインデントが不一致です。
</background_information>タグ（18行目）と</instructions>タグ（107行目）のインデントが開始タグと一致していません。ツールによるパース処理に影響を与える可能性があります。🔧 修正案
- </background_information> +</background_information>107行目も同様に:
- </instructions> +</instructions>.claude/skills/openai-voice-agents/scripts/search_docs.py (4)
82-83: 冗長な条件チェックがあります。78-79行目で
if not match_indices: return []をチェックしているため、83行目のif match_indices:は常に真となり冗長です。🔧 修正案
# Group nearby matches to avoid overlapping contexts groups = [] - if match_indices: - current_group = [match_indices[0]] - for i in range(1, len(match_indices)): + current_group = [match_indices[0]] + for i in range(1, len(match_indices)):
182-185: コンテキスト表示後の "..." が常に出力されます。各コンテキストスニペットの後に必ず "..." が表示されますが、コンテキストが3つ未満の場合やすべて表示済みの場合は不要かもしれません。
🔧 修正案（省略記号を条件付きで表示）
- for ctx in res['contexts'][:3]: # Show max 3 contexts per file - print(ctx) - print(" ...") + contexts_to_show = res['contexts'][:3] + for i, ctx in enumerate(contexts_to_show): + print(ctx) + if i < len(contexts_to_show) - 1 or len(res['contexts']) > 3: + print(" ...")
16-16: 未使用のインポートがあります。
datetimeモジュールがインポートされていますが、コード内で使用されていません。🔧 修正案
-from datetime import datetime
191-212: エラー時の終了コードが設定されていません。
docs/ディレクトリが存在しない場合、エラーメッセージは出力されますが、終了コードは0のままです。CI/CDやスクリプトでの使用時に問題を検出しにくくなる可能性があります。🔧 修正案
def main(): parser = argparse.ArgumentParser(description="Search Claude Skill documentation.") parser.add_argument("query", help="Search query") parser.add_argument("--max-results", "-n", type=int, default=10, help="Maximum number of results") parser.add_argument("--json", action="store_true", help="Output as JSON") # Default: script's parent directory (scripts/../ = skill root) default_skill_dir = Path(__file__).resolve().parent.parent parser.add_argument("--skill-dir", default=str(default_skill_dir), help="Skill directory (default: auto-detected from script location)") args = parser.parse_args() skill_path = Path(args.skill_dir).resolve() results = search_docs(skill_path, args.query, args.max_results) + if results is None: # docs directory not found + sys.exit(1) + if args.json: format_json(results) else: format_results(results, args.query)また、
search_docs関数でディレクトリが見つからない場合はNoneを返すように変更することを検討してください。.claude/commands/kiro/spec-impl.md (1)
17-17: 閉じタグのインデントが不一致です。
</background_information>タグ（17行目）と</instructions>タグ（78行目）の余分なインデントを削除してください。🔧 修正案
- </background_information> +</background_information>.claude/commands/kiro/spec-requirements.md (1)
17-17: 閉じタグのインデントが不一致です。
</background_information>タグ（17行目）と</instructions>タグ（55行目）のインデントを修正してください。他のKiroドキュメントと同様の問題です。🔧 修正案
- </background_information> +</background_information>55行目も同様に修正してください。
src/pages/api/memory-restore.ts (1)
51-53: 同期的なファイル読み込みはイベントループをブロックする可能性があります。大きなファイルの場合、
readFileSyncはパフォーマンスに影響を与える可能性があります。fs.promises.readFileの使用を検討してください。♻️ 修正案
// ファイルの読み込み - const content = fs.readFileSync(filePath, 'utf-8') + const content = await fs.promises.readFile(filePath, 'utf-8') const messages = JSON.parse(content)src/pages/api/get-pngtuber-list.ts (2)
19-22: HTTPメソッドの検証がありません。このエンドポイントはGETリクエスト専用と思われますが、すべてのHTTPメソッドを受け入れています。他のAPIエンドポイント（memory-restore.ts）と一貫性を持たせるため、メソッド検証を追加することを推奨します。
♻️ 修正案
export default async function handler( req: NextApiRequest, res: NextApiResponse ) { + if (req.method !== 'GET') { + return res.status(405).json({ error: 'Method not allowed' }) + } + const pngtuberDir = path.join(process.cwd(), 'public/pngtuber')
61-65: 同期・非同期のファイル操作が混在しています。
fs.existsSyncは同期的ですが、他の箇所ではfs.promisesを使用しています。一貫性のため、fs.promises.accessまたはfs.promises.statの使用を検討してください。♻️ 修正案
// mouth/ ディレクトリを確認 const mouthDir = path.join(folderPath, 'mouth') - if (!fs.existsSync(mouthDir)) continue + try { + await fs.promises.access(mouthDir) + } catch { + continue + } const mouthFiles = await fs.promises.readdir(mouthDir)src/pages/api/embedding.ts (1)
35-40: 405エラーに対してINVALID_INPUTコードは不適切です。405 Method Not Allowedに対して
INVALID_INPUTコードを使用していますが、これは意味的に不正確です。専用のエラーコード（例:METHOD_NOT_ALLOWED）を追加するか、より汎用的なコードを使用することを検討してください。♻️ 提案
interface EmbeddingError { error: string - code: 'INVALID_INPUT' | 'API_KEY_MISSING' | 'RATE_LIMITED' | 'API_ERROR' + code: 'INVALID_INPUT' | 'API_KEY_MISSING' | 'RATE_LIMITED' | 'API_ERROR' | 'METHOD_NOT_ALLOWED' }if (req.method !== 'POST') { return res.status(405).json({ error: 'Method not allowed', - code: 'INVALID_INPUT', + code: 'METHOD_NOT_ALLOWED', }) }src/hooks/useWhisperRecognition.ts (1)
43-51: MIMEタイプの判定をより正確に行うことを検討してください。
includes('mp3')はaudio/mpegタイプ（MP3の標準MIMEタイプ）にマッチしません。より堅牢なMIMEタイプマッピングを検討してください。♻️ 提案
- // MIMEタイプに基づいて拡張子を設定 - if (mimeType.includes('mp3')) { - fileExtension = 'mp3' - } else if (mimeType.includes('ogg')) { - fileExtension = 'ogg' - } else if (mimeType.includes('wav')) { - fileExtension = 'wav' - } else if (mimeType.includes('mp4')) { - fileExtension = 'mp4' - } + // MIMEタイプに基づいて拡張子を設定 + if (mimeType.includes('mp3') || mimeType.includes('mpeg')) { + fileExtension = 'mp3' + } else if (mimeType.includes('ogg')) { + fileExtension = 'ogg' + } else if (mimeType.includes('wav')) { + fileExtension = 'wav' + } else if (mimeType.includes('mp4') || mimeType.includes('m4a')) { + fileExtension = 'mp4' + }src/__tests__/hooks/useWhisperRecognition.test.ts (1)
132-145: このテストは Requirement 7.1 のテストと重複しています。
stopListeningの参照安定性テスト（Line 80-98）と同じロジックを繰り返しています。このテストケースは削除するか、実際に依存配列の変更をトリガーする異なるシナリオ（例:selectLanguageの変更時の挙動）をテストすることを検討してください。src/features/memory/memoryTypes.ts (1)
16-29:roleフィールドに'system'を含めることを検討してください。現在
roleは'user' | 'assistant'に制限されていますが、システムプロンプトやメタデータをメモリに保存する必要がある場合、'system'ロールが必要になる可能性があります。現時点で必要ない場合は、将来の拡張時に検討してください。src/hooks/useRealtimeVoiceAPI.ts (1)
52-88: WebSocket送信時のエラーハンドリングが不完全
sendTextToWebSocket関数でWebSocketが開いていない場合に早期リターンしていますが、呼び出し元（handleTextDetected）にエラー状態を通知していません。これにより、onChatProcessStartが呼び出されても実際にはメッセージが送信されていない状況が発生する可能性があります。🔧 エラー状態を返す提案
- const sendTextToWebSocket = useCallback((text: string) => { + const sendTextToWebSocket = useCallback((text: string): boolean => { const wsManager = webSocketStore.getState().wsManager const ss = settingsStore.getState() if (wsManager?.websocket?.readyState !== WebSocket.OPEN) { console.error('WebSocket is not open') - return + return false } // ... rest of the function + return true }, [])src/hooks/useAudioProcessing.ts (2)
43-58:checkMicrophonePermissionをuseCallbackでラップすることを推奨
checkMicrophonePermissionはuseCallbackでラップされておらず、レンダリングごとに新しい関数が作成されます。startRecordingの依存配列には含まれていないため、現時点では動作しますが、将来的にこの関数を他の場所で使用する場合に問題が発生する可能性があります。🔧 useCallbackでラップする提案
- const checkMicrophonePermission = async (): Promise<boolean> => { + const checkMicrophonePermission = useCallback(async (): Promise<boolean> => { try { const stream = await navigator.mediaDevices.getUserMedia({ audio: true }) stream.getTracks().forEach((track) => track.stop()) return true } catch (error) { console.error('Microphone permission error:', error) toastStore.getState().addToast({ message: t('Toasts.MicrophonePermissionDenied'), type: 'error', tag: 'microphone-permission-error', }) return false } - } + }, [t])
164-228: stopRecordingのPromiseベースの実装は堅牢MediaRecorderの停止処理を
Promiseでラップし、onstopイベントを待ってからデータを返すアプローチは適切です。トラックの停止順序（MediaRecorder停止後にトラックを停止）も正しいです。ただし、
onstopが呼ばれない場合のタイムアウト処理がないため、Promiseが永久に解決されない可能性があります。🔧 タイムアウト追加の提案
return new Promise<Blob | null>((resolve) => { + const timeout = setTimeout(() => { + console.warn('🔴 MediaRecorder.onstop タイムアウト') + audioChunksRef.current = [] + resolve(null) + }, 5000) + // ... existing code ... mediaRecorder.onstop = () => { + clearTimeout(timeout) // ... existing handler code ... }src/__tests__/hooks/useAudioProcessing.test.ts (1)
45-59: MediaRecorderのモック設定モックは基本的な構造をカバーしていますが、
ondataavailableとonstopのハンドラテストが含まれていません。録音の完全なフローをテストする場合は、これらのイベントをシミュレートするテストを追加することを検討してください。src/features/memory/memoryStore.ts (1)
136-151:getRecentMessagesのパフォーマンス最適化を検討現在の実装は全レコードをメモリに取得してからソート・スライスしています。レコード数が増加すると、メモリ使用量とパフォーマンスに影響が出る可能性があります。
timestampインデックスを使用したカーソルベースの取得を検討してください。🔧 カーソルベースの実装案
async getRecentMessages(limit: number): Promise<MemoryRecord[]> { this.ensureOpen() - // 全レコードを取得してタイムスタンプでソート - const allRecords = await this.db!.getAll(STORE_NAME) - - // タイムスタンプ降順（新しい順）でソート - allRecords.sort((a, b) => { - const timeA = new Date(a.timestamp).getTime() - const timeB = new Date(b.timestamp).getTime() - return timeB - timeA - }) - - // 上限まで返却 - return allRecords.slice(0, limit) + const results: MemoryRecord[] = [] + const tx = this.db!.transaction(STORE_NAME, 'readonly') + const index = tx.store.index('timestamp') + + // 降順でカーソルを開く + let cursor = await index.openCursor(null, 'prev') + + while (cursor && results.length < limit) { + results.push(cursor.value) + cursor = await cursor.continue() + } + + return results }src/features/memory/memoryContextBuilder.ts (1)
154-187: 最新優先のトリミング方針を明確化したいLine 170-180: 最新メモリが収まらない場合に古いメモリが採用され得るため、「古い記憶から削除」の意図を厳密に守るなら、超過時点で
breakするか最新メモリをトリムする方針が無難です。♻️ 例: 最新優先で打ち切る場合
for (let i = formattedMemories.length - 1; i >= 0; i--) { const memory = formattedMemories[i] const memoryTokens = this.estimateTokens(memory + '\n') - if (currentTokens + memoryTokens <= availableTokens) { - selectedMemories.unshift(memory) // 先頭に追加して順序を維持 - currentTokens += memoryTokens - } + if (currentTokens + memoryTokens > availableTokens) { + break + } + selectedMemories.unshift(memory) // 先頭に追加して順序を維持 + currentTokens += memoryTokens }src/__tests__/pages/api/memory-files.test.ts (1)
40-59: テストカバレッジの拡充を検討してください。基本的なテストケースは網羅されていますが、以下のエッジケースのテストを追加すると堅牢性が向上します：
- JSON パースエラーが発生した場合の動作
- ファイル読み取りエラーが発生した場合の動作
📝 追加テストケースの例
it('JSONパースエラー時もエラーをスローしない', async () => { mockFs.existsSync.mockReturnValue(true) mockFs.readdirSync.mockReturnValue(['log_2024-01-01T12-00-00.json'] as any) mockFs.readFileSync.mockReturnValue('invalid json') const { req, res } = createMocks<NextApiRequest, NextApiResponse>({ method: 'GET', }) await handler(req, res) expect(res._getStatusCode()).toBe(200) const data = JSON.parse(res._getData()) expect(data.files).toHaveLength(0) // エラーファイルはスキップされる })src/__tests__/pages/api/memory-restore.test.ts (1)
66-77: セキュリティテストが適切に実装されています。パストラバーサル攻撃の防止テストが含まれている点は評価できます。他のパストラバーサルパターン（例：
..%2F、....//、絶対パス）のテストも追加するとより堅牢になります。src/pages/api/memory-files.ts (1)
56-62: 日付フォーマットの改善を検討してください。現在の実装では
2024-01-01T12:00:00のような形式でcreatedAtが返されますが、タイムゾーン情報が含まれていません。クライアント側で一貫した処理を行うために、ISO 8601 形式（Zサフィックスまたはオフセット付き）に統一することを推奨します。📝 修正案
const createdAt = match - ? match[1].replace(/-(\d{2})-(\d{2})-(\d{2})$/, ':$1:$2:$3') + ? match[1].replace(/-(\d{2})-(\d{2})-(\d{2})$/, ':$1:$2:$3') + 'Z' : new Date().toISOString()src/__tests__/features/memory/memoryContextBuilder.test.ts (1)
23-33:createMemoryRecordヘルパー関数が重複しています同じヘルパー関数が2つの
describeブロック内で定義されています（Lines 23-33 と 184-194）。ファイルスコープまたは親のdescribeブロックスコープに1つ抽出することで、コードの重複を削減できます。♻️ リファクタリング提案
describe('MemoryContextBuilder', () => { let builder: MemoryContextBuilder + const createMemoryRecord = ( + overrides: Partial<MemoryRecord> = {} + ): MemoryRecord => ({ + id: 'test-id', + role: 'user', + content: 'テストメッセージ', + embedding: [0.1, 0.2, 0.3], + timestamp: '2025-01-15T14:30:00Z', + sessionId: 'session-1', + ...overrides, + }) + beforeEach(() => { builder = new MemoryContextBuilder() }) describe('buildContext', () => { - const createMemoryRecord = ( - overrides: Partial<MemoryRecord> = {} - ): MemoryRecord => ({ - id: 'test-id', - role: 'user', - content: 'テストメッセージ', - embedding: [0.1, 0.2, 0.3], - timestamp: '2025-01-15T14:30:00Z', - sessionId: 'session-1', - ...overrides, - })Also applies to: 184-194
src/__tests__/features/memory/memoryStore.test.ts (1)
8-13:structuredCloneポリフィルの制限事項
JSON.parse(JSON.stringify(obj))によるポリフィルは、Dateオブジェクト、undefined値、Map/Set、循環参照などを正しく処理できません。fake-indexeddbのテスト環境では問題ないかもしれませんが、この制限を認識しておくことが重要です。テスト目的では許容範囲ですが、必要に応じてより堅牢な実装（例：
lodash.cloneDeep）を検討してください。src/features/pngTuber/pngTuberHandler.ts (1)
58-63:resetToIdleがasyncだがawaitを使用していません現在の実装では非同期操作がないため、
asyncキーワードは不要です。インターフェース互換性や将来の拡張のために意図的であれば問題ありませんが、不要であれば削除を検討してください。♻️ リファクタリング提案（オプション）
- static async resetToIdle(): Promise<void> { + static resetToIdle(): void { const hs = homeStore.getState() const pngTuberViewer = hs.pngTuberViewer if (!pngTuberViewer) return pngTuberViewer.resetMouth() }src/components/PNGTuberComponent.tsx (1)
122-138:FileReaderのエラーハンドリングを強化することを検討してください。現在は
console.errorのみですが、ユーザーに対してエラーを通知するとより親切です。♻️ 提案: ユーザーへのエラー通知を追加
reader.onerror = () => { console.error('Failed to read image file') + // オプション: ユーザーにエラーを通知 + // setError('画像ファイルの読み込みに失敗しました') }src/__tests__/features/memory/memoryService.test.ts (2)
454-479: パフォーマンステストのコメントと実際の閾値が異なります。コメントでは「100ms以内」と記載されていますが、実際のアサーションは500msになっています。テスト環境のバッファを考慮しているとのことですが、コメントを実際の値に合わせるか、より正確な説明に更新することを検討してください。
♻️ 提案: コメントと実際の閾値を一致させる
- // Search should complete within 100ms (excluding API call time) - // Note: In real tests, we might mock the entire API call time - expect(endTime - startTime).toBeLessThan(500) // Allow some buffer for test environment + // Search should complete within 500ms in test environment + // (Production requirement is 100ms, but test environment overhead is expected) + expect(endTime - startTime).toBeLessThan(500)
483-486:MemorySearchResultインターフェースはテスト内で活用されていません。このインターフェースを定義していますが、テスト内の
results変数の型として使用されていません。型安全性を高めるために、検索結果の型として明示的に使用することを検討してください。src/features/stores/home.ts (1)
240-250: 非同期保存処理でのエラーハンドリングを強化することを検討してください。
void fetch(...)で結果を無視していますが、.catchで既にエラーハンドリングされています。ただし、messagesWithEmbeddingの取得後にfetchが失敗した場合、ユーザーへの通知がコンソールログのみとなっています。重要なデータの保存失敗をユーザーに通知する仕組みがあると、より堅牢になります。
src/components/settings/character.tsx (1)
713-720: カラーコードのテキスト入力にバリデーションがありません。ユーザーが無効なカラーコードを入力した場合、予期しない動作を引き起こす可能性があります。入力値のバリデーションを追加することを検討してください。
♻️ 提案: カラーコードのバリデーションを追加
<input type="text" value={pngTuberChromaKeyColor} onChange={(e) => + { + const value = e.target.value + // 入力中は任意の値を許可し、#で始まる有効なHexカラーのみ適用 + if (/^#[0-9A-Fa-f]{0,6}$/.test(value)) { settingsStore.setState({ - pngTuberChromaKeyColor: e.target.value, + pngTuberChromaKeyColor: value, }) + } + } } className="px-2 py-1 w-24 bg-white rounded-lg border" placeholder="#00FF00" />src/features/memory/memoryService.ts (3)
160-197: 大規模データセットでのパフォーマンス懸念
searchMemoriesは全メモリを取得してからメモリ内で類似度計算を行っています。メモリ数が増加すると、パフォーマンスに影響が出る可能性があります。現時点では問題ないかもしれませんが、将来的にはIndexedDBのインデックスを活用した絞り込みや、ページネーション、あるいはベクトルデータベースへの移行を検討することをお勧めします。
299-306: Embedding次元数の不一致時の処理を強化すべき次元数が期待値と異なる場合、警告ログのみで処理を継続しています。異なる次元数のEmbedding同士で類似度計算を行うと、
cosineSimilarityでエラーが発生するか、不正確な結果になる可能性があります。🔧 修正案
// Embedding次元数の検証 if (data.embedding.length !== EMBEDDING_DIMENSION) { console.warn( `MemoryService: Unexpected embedding dimension: ${data.embedding.length}` ) + return null } return data.embedding
233-251: 一括復元の並列化検討（オプション）
restoreMemoriesは各レコードを順次処理しています。大量のレコードを復元する場合、Promise.allSettledを使用した並列処理で高速化できる可能性があります。ただし、IndexedDBのトランザクション制限を考慮する必要があるため、現在の実装も妥当です。src/components/settings/memorySettings.tsx (2)
491-497: 成功/エラー判定がロケール依存で脆弱
restoreMessage.includes('成功') || restoreMessage.includes('Success')による判定は、翻訳文字列に依存しており、言語追加や翻訳変更で壊れる可能性があります。🔧 修正案: 別のstate変数で成功/エラーを管理
+ const [restoreStatus, setRestoreStatus] = useState<'success' | 'error' | null>(null) const [restoreMessage, setRestoreMessage] = useState<string>('') // ... 復元処理内で - setRestoreMessage(t('MemoryRestoreSuccess')) + setRestoreMessage(t('MemoryRestoreSuccess')) + setRestoreStatus('success') // ... エラー時 - setRestoreMessage(t('MemoryRestoreError')) + setRestoreMessage(t('MemoryRestoreError')) + setRestoreStatus('error') // ... レンダリング部分 {restoreMessage && ( <span - className={`text-sm ${restoreMessage.includes('成功') || restoreMessage.includes('Success') ? 'text-green-600' : 'text-red-600'}`} + className={`text-sm ${restoreStatus === 'success' ? 'text-green-600' : 'text-red-600'}`} > {restoreMessage} </span> )}
566-571: リストのkeyにindexを使用している
chatLog.mapでindexをkeyとして使用しています。メッセージの削除や編集が可能なリストでは、indexをkeyに使うとReactの再レンダリングで予期せぬ動作が発生する可能性があります。メッセージにidプロパティがあれば、それを使用することを推奨します。🔧 修正案
- {chatLog.map((value, index) => { + {chatLog.map((value, index) => { + const key = value.id || `chat-${index}` return ( value.content && ( <div - key={index} + key={key} className="my-2 grid grid-flow-col grid-cols-[100px_1fr_auto] gap-x-fixed" >src/features/pngTuber/pngTuberEngine.ts (3)
80-89: アセット取得の効率化を検討
loadAssetで全アセットリストを取得してから単一のアセットを検索しています。アセット数が多い場合、不要なデータ転送が発生します。将来的には、パスをパラメータとして受け取る専用APIエンドポイント（例：
/api/get-pngtuber-asset?path=${assetPath}）の導入を検討してください。
314-337: 空のcatchブロックに最低限のログ追加を検討
stopAudio内の空のcatchブロックは意図的かもしれませんが、デバッグ時に問題追跡が困難になる可能性があります。少なくともdebugレベルのログ出力を検討してください。🔧 修正案
if (this.currentSource) { try { this.currentSource.stop() this.currentSource.disconnect() } catch { - // ignore + // Already stopped or disconnected - expected in some cases } this.currentSource = null } if (this.workletNode) { try { this.workletNode.port.onmessage = null this.workletNode.disconnect() } catch { - // ignore + // Already disconnected - expected in some cases } this.workletNode = null }
114-124: 動画サイズ取得失敗時の警告追加を検討
videoWidth/videoHeightが取得できない場合のフォールバック値1は、予期せぬキャンバスサイズにつながる可能性があります。この状況が発生した場合は警告ログを出力することを推奨します。🔧 修正案
// キャンバスサイズを動画に合わせる const videoWidth = this.video.videoWidth || 1 const videoHeight = this.video.videoHeight || 1 + + if (!this.video.videoWidth || !this.video.videoHeight) { + console.warn('PNGTuberEngine: Video dimensions not available, using fallback') + } this.mainCanvas.width = videoWidthsrc/features/memory/memoryStoreSync.ts (2)
131-151: Embedding取得前にサービス可用性を確認したいです。
未初期化時の例外/警告を避けるため、isAvailable()で早期リターンする方が安定します。♻️ 修正案
try { const memoryService = getMemoryService() + if (!memoryService.isAvailable()) return message const embedding = await memoryService.fetchEmbedding(content) if (embedding) { return { ...message, embedding } }
160-166: 大量メッセージ時の並列数を抑えたいです。
Promise.allは一括発行になるため、埋め込みAPIのレート制限や負荷に弱い可能性があります。シンプルに逐次処理へ落とすか、将来的にバッチ化を検討すると安全です。♻️ 修正案（逐次処理）
export async function addEmbeddingsToMessages( messages: Message[] ): Promise<Message[]> { const { memoryEnabled } = settingsStore.getState() if (!memoryEnabled) return messages - return Promise.all(messages.map(addEmbeddingToMessage)) + const result: Message[] = [] + for (const message of messages) { + result.push(await addEmbeddingToMessage(message)) + } + return result }
| - Run with no argument or use wildcard | ||
| - Shows all specs in `.kiro/specs/` with their status | ||
|
|
||
| think |
There was a problem hiding this comment.
不明な単語 "think" を削除してください。
ドキュメントの最後に文脈のない "think" という単語が残っています。これは編集の残骸かプレースホルダーの可能性があります。
🧹 修正案
-think📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| think |
🤖 Prompt for AI Agents
In @.claude/commands/kiro/spec-status.md at line 97, ドキュメント末尾に残っている文脈のない単語
"think" を削除してください — ファイル内の孤立したトークン "think"
をそのまま取り除き、余分な空白や改行が生じる場合は整えて文書末尾が自然に終わるようにしてください。
|
|
||
| **Note**: The implementation phase will guide you through executing tasks with appropriate context and validation. | ||
|
|
||
| think |
There was a problem hiding this comment.
不明な単語 "think" を削除してください。
ドキュメントの最後に文脈のない "think" という単語が残っています。これは .claude/commands/kiro/spec-status.md の97行目と同じ問題です。編集の残骸と思われます。
🧹 修正案
-think📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| think |
🤖 Prompt for AI Agents
In @.claude/commands/kiro/spec-tasks.md at line 153, Remove the stray,
context-less word "think" from the end of the document in
.claude/commands/kiro/spec-tasks.md (it’s the leftover fragment shown in the
diff); also scan .claude/commands/kiro/spec-status.md for the same orphaned
"think" occurrence mentioned in the review and remove it so both files no longer
contain the dangling token.
Comment on lines
+170
to
+171
| Additionally to the arguments that the agent called a particular tool with, you can also access a snapshot of the current conversation history that is tracked by the Realtime Session. This can be useful if you need to perform a more complex action based on the current state of the conversation or are planning to use [tools for delegation](https://openai.github.io/openai-agents-js/guides/voice-agents/build.html#delegation-through-tools). | ||
|
|
There was a problem hiding this comment.
英文表現の軽微な修正を推奨します。
Line 170、Line 251、Line 400-403 に不自然な表現・誤字があります。
📝 修正案（文言のみ）
- Additionally to the arguments that the agent called a particular tool with, you can also access a snapshot of the current conversation history ...
+ In addition to the arguments that the agent called a particular tool with, you can also access a snapshot of the current conversation history ...
- ... cut off the response based a predefined classification trigger ...
+ ... cut off the response based on a predefined classification trigger ...
- In either way, the Realtime Session will handle both interrupting the generation of the agent...
- ... you will need to handle this yourself by stopping audio playack of whatever has been queued up to be played.
+ Either way, the Realtime Session will handle both interrupting the generation of the agent...
+ ... you will need to handle this yourself by stopping audio playback of whatever has been queued up to be played.Also applies to: 251-252, 400-403
🤖 Prompt for AI Agents
In @.claude/skills/openai-voice-agents/docs/build.md around lines 170 - 171, The
sentence "Additionally to the arguments that the agent called a particular tool
with, you can also access a snapshot of the current conversation history..." is
awkward and should be rephrased; change it to "In addition to the arguments with
which the agent called a particular tool, you can access a snapshot of the
current conversation history tracked by the Realtime Session." Also review the
other occurrences noted (the similar phrasing at the later paragraphs and the
block in lines 400-403) and correct any typos/ungrammatical constructions to
match this clearer wording and ensure consistent terminology ("Realtime
Session", "conversation history", "tools for delegation").
Comment on lines
57
to
58
| The response will contain a “value” string a the top level, which starts with “ek\_” prefix. You can use this ephemeral key to establish a WebRTC connection later on. Note that this key is only valid for a short period of time and will need to be regenerated. | ||
|
|
There was a problem hiding this comment.
軽微な誤字・表現の修正を推奨します。
Line 57 と Line 110、Line 149 に表記揺れ/誤字があります。
📝 修正案（文言のみ）
- The response will contain a “value” string a the top level, which starts with “ek_” prefix.
+ The response will contain a “value” string at the top level, which starts with “ek_” prefix.
- // for quickly start, you can append the following code to the auto-generated TS code
+ // to get started quickly, you can append the following code to the auto-generated TS code
- Start up your webserver and navigate to the page that includes your new Realtime Agent code.
+ Start up your web server and navigate to the page that includes your new Realtime Agent code.Also applies to: 110-112, 149-149
🤖 Prompt for AI Agents
In @.claude/skills/openai-voice-agents/docs/quickstart.md around lines 57 - 58,
Fix the minor typos and inconsistent escaping in quickstart.md: change "a the
top level" to "at the top level" on the sentence that mentions the response
containing a "value" string, and normalize the ephemeral key prefix formatting
across all occurrences (Lines referencing the “ek\_” prefix) to a consistent,
readable form such as `ek_` (or plain ek_) and ensure the same wording is used
at the three locations (the sentence near the "value" string and the two other
places referenced) for consistent phrasing.
Comment on lines
+203
to
+271
| /** | ||
| * ローカルファイルからメモリを復元する | ||
| * | ||
| * @param filename - 復元するファイル名 | ||
| * @returns 復元結果 | ||
| */ | ||
| export async function restoreMemoryFromFile(filename: string): Promise<{ | ||
| success: boolean | ||
| restoredCount: number | ||
| embeddingCount: number | ||
| }> { | ||
| const { memoryEnabled } = settingsStore.getState() | ||
| if (!memoryEnabled) return RESTORE_FAILURE | ||
|
|
||
| try { | ||
| const response = await fetch('/api/memory-restore', { | ||
| method: 'POST', | ||
| headers: { 'Content-Type': 'application/json' }, | ||
| body: JSON.stringify({ filename }), | ||
| }) | ||
|
|
||
| if (!response.ok) { | ||
| console.error('Failed to restore memory:', response.statusText) | ||
| return RESTORE_FAILURE | ||
| } | ||
|
|
||
| const data = (await response.json()) as { | ||
| messages: Message[] | ||
| restoredCount: number | ||
| embeddingCount: number | ||
| } | ||
|
|
||
| const memoryService = getMemoryService() | ||
| if (!memoryService.isAvailable()) { | ||
| await memoryService.initialize() | ||
| } | ||
|
|
||
| let actualRestoredCount = 0 | ||
|
|
||
| for (const message of data.messages) { | ||
| if (message.role !== 'user' && message.role !== 'assistant') continue | ||
|
|
||
| const content = extractTextContent(message.content) | ||
| if (!content) continue | ||
|
|
||
| await memoryService.restoreMemory({ | ||
| id: `restored-${Date.now()}-${Math.random().toString(36).slice(2, 9)}`, | ||
| sessionId: 'restored', | ||
| role: message.role as 'user' | 'assistant', | ||
| content, | ||
| timestamp: message.timestamp || new Date().toISOString(), | ||
| embedding: message.embedding || null, | ||
| }) | ||
| actualRestoredCount++ | ||
| } | ||
|
|
||
| console.log( | ||
| `MemoryStoreSync: Restored ${actualRestoredCount} memories from ${filename}` | ||
| ) | ||
|
|
||
| return { | ||
| success: true, | ||
| restoredCount: actualRestoredCount, | ||
| embeddingCount: data.embeddingCount, | ||
| } | ||
| } catch (error) { | ||
| console.error('Error restoring memory from file:', error) | ||
| return RESTORE_FAILURE | ||
| } |
There was a problem hiding this comment.
復元途中の失敗で部分復元＋失敗返却になる可能性があります。
現在は1件でも restoreMemory が失敗すると catch に落ちて失敗返却になりますが、既に一部が復元されている可能性があり、結果が不整合になります（再実行時の重複も起きやすい）。1件ごとに失敗を握り、カウントを正しく返す方が安全です。
🛠 修正案
- let actualRestoredCount = 0
+ let actualRestoredCount = 0
+ let failedCount = 0
for (const message of data.messages) {
if (message.role !== 'user' && message.role !== 'assistant') continue
const content = extractTextContent(message.content)
if (!content) continue
- await memoryService.restoreMemory({
- id: `restored-${Date.now()}-${Math.random().toString(36).slice(2, 9)}`,
- sessionId: 'restored',
- role: message.role as 'user' | 'assistant',
- content,
- timestamp: message.timestamp || new Date().toISOString(),
- embedding: message.embedding || null,
- })
- actualRestoredCount++
+ try {
+ await memoryService.restoreMemory({
+ id: `restored-${Date.now()}-${Math.random().toString(36).slice(2, 9)}`,
+ sessionId: 'restored',
+ role: message.role as 'user' | 'assistant',
+ content,
+ timestamp: message.timestamp || new Date().toISOString(),
+ embedding: message.embedding || null,
+ })
+ actualRestoredCount++
+ } catch (error) {
+ failedCount++
+ console.warn('MemoryStoreSync: Failed to restore a message', error)
+ }
}
console.log(
`MemoryStoreSync: Restored ${actualRestoredCount} memories from ${filename}`
)
return {
- success: true,
+ success: failedCount === 0,
restoredCount: actualRestoredCount,
embeddingCount: data.embeddingCount,
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| /** | |
| * ローカルファイルからメモリを復元する | |
| * | |
| * @param filename - 復元するファイル名 | |
| * @returns 復元結果 | |
| */ | |
| export async function restoreMemoryFromFile(filename: string): Promise<{ | |
| success: boolean | |
| restoredCount: number | |
| embeddingCount: number | |
| }> { | |
| const { memoryEnabled } = settingsStore.getState() | |
| if (!memoryEnabled) return RESTORE_FAILURE | |
| try { | |
| const response = await fetch('/api/memory-restore', { | |
| method: 'POST', | |
| headers: { 'Content-Type': 'application/json' }, | |
| body: JSON.stringify({ filename }), | |
| }) | |
| if (!response.ok) { | |
| console.error('Failed to restore memory:', response.statusText) | |
| return RESTORE_FAILURE | |
| } | |
| const data = (await response.json()) as { | |
| messages: Message[] | |
| restoredCount: number | |
| embeddingCount: number | |
| } | |
| const memoryService = getMemoryService() | |
| if (!memoryService.isAvailable()) { | |
| await memoryService.initialize() | |
| } | |
| let actualRestoredCount = 0 | |
| for (const message of data.messages) { | |
| if (message.role !== 'user' && message.role !== 'assistant') continue | |
| const content = extractTextContent(message.content) | |
| if (!content) continue | |
| await memoryService.restoreMemory({ | |
| id: `restored-${Date.now()}-${Math.random().toString(36).slice(2, 9)}`, | |
| sessionId: 'restored', | |
| role: message.role as 'user' | 'assistant', | |
| content, | |
| timestamp: message.timestamp || new Date().toISOString(), | |
| embedding: message.embedding || null, | |
| }) | |
| actualRestoredCount++ | |
| } | |
| console.log( | |
| `MemoryStoreSync: Restored ${actualRestoredCount} memories from ${filename}` | |
| ) | |
| return { | |
| success: true, | |
| restoredCount: actualRestoredCount, | |
| embeddingCount: data.embeddingCount, | |
| } | |
| } catch (error) { | |
| console.error('Error restoring memory from file:', error) | |
| return RESTORE_FAILURE | |
| } | |
| /** | |
| * ローカルファイルからメモリを復元する | |
| * | |
| * `@param` filename - 復元するファイル名 | |
| * `@returns` 復元結果 | |
| */ | |
| export async function restoreMemoryFromFile(filename: string): Promise<{ | |
| success: boolean | |
| restoredCount: number | |
| embeddingCount: number | |
| }> { | |
| const { memoryEnabled } = settingsStore.getState() | |
| if (!memoryEnabled) return RESTORE_FAILURE | |
| try { | |
| const response = await fetch('/api/memory-restore', { | |
| method: 'POST', | |
| headers: { 'Content-Type': 'application/json' }, | |
| body: JSON.stringify({ filename }), | |
| }) | |
| if (!response.ok) { | |
| console.error('Failed to restore memory:', response.statusText) | |
| return RESTORE_FAILURE | |
| } | |
| const data = (await response.json()) as { | |
| messages: Message[] | |
| restoredCount: number | |
| embeddingCount: number | |
| } | |
| const memoryService = getMemoryService() | |
| if (!memoryService.isAvailable()) { | |
| await memoryService.initialize() | |
| } | |
| let actualRestoredCount = 0 | |
| let failedCount = 0 | |
| for (const message of data.messages) { | |
| if (message.role !== 'user' && message.role !== 'assistant') continue | |
| const content = extractTextContent(message.content) | |
| if (!content) continue | |
| try { | |
| await memoryService.restoreMemory({ | |
| id: `restored-${Date.now()}-${Math.random().toString(36).slice(2, 9)}`, | |
| sessionId: 'restored', | |
| role: message.role as 'user' | 'assistant', | |
| content, | |
| timestamp: message.timestamp || new Date().toISOString(), | |
| embedding: message.embedding || null, | |
| }) | |
| actualRestoredCount++ | |
| } catch (error) { | |
| failedCount++ | |
| console.warn('MemoryStoreSync: Failed to restore a message', error) | |
| } | |
| } | |
| console.log( | |
| `MemoryStoreSync: Restored ${actualRestoredCount} memories from ${filename}` | |
| ) | |
| return { | |
| success: failedCount === 0, | |
| restoredCount: actualRestoredCount, | |
| embeddingCount: data.embeddingCount, | |
| } | |
| } catch (error) { | |
| console.error('Error restoring memory from file:', error) | |
| return RESTORE_FAILURE | |
| } | |
| } |
🤖 Prompt for AI Agents
In `@src/features/memory/memoryStoreSync.ts` around lines 203 - 271, The
restoreMemoryFromFile function currently aborts the whole operation if any
single memoryService.restoreMemory call throws, causing inconsistent partial
restores; change it so each message restore is wrapped in its own try/catch
inside the loop (refer to restoreMemoryFromFile and
memoryService.restoreMemory), increment actualRestoredCount only on successful
restores, track a failedCount (or collect failed IDs) and log per-item errors
(console.error) without rethrowing, and after the loop return a result object
that accurately reports restoredCount and embeddingCount and sets success based
on whether any items were restored (e.g., success = actualRestoredCount > 0)
instead of letting one failure trigger the outer catch.
Comment on lines
+80
to
+107
| // 常時マイク入力モードの場合、ストップ後にマイクを再開 | ||
| // （stopAllではコールバックが呼ばれないため、ここで再開処理を行う） | ||
| if ( | ||
| settingsStore.getState().continuousMicListeningMode && | ||
| settingsStore.getState().speechRecognitionMode === 'browser' && | ||
| !homeStore.getState().chatProcessing | ||
| ) { | ||
| console.log('🔄 ストップボタンが押されました。音声認識を再開します。') | ||
| setTimeout(() => { | ||
| currentHookRef.current.startListening() | ||
| }, 300) | ||
| } | ||
| }, []) | ||
|
|
||
| // AIの発話完了後に音声認識を自動的に再開する処理 | ||
| const handleSpeakCompletion = useCallback(() => { | ||
| // 常時マイク入力モードがONで、現在マイク入力が行われていない場合のみ実行 | ||
| if ( | ||
| continuousMicListeningMode && | ||
| // !currentHook.isListening && | ||
| speechRecognitionMode === 'browser' && | ||
| !homeStore.getState().chatProcessing | ||
| ) { | ||
| console.log('🔄 AIの発話が完了しました。音声認識を自動的に再開します。') | ||
| setTimeout(() => { | ||
| currentHook.startListening() | ||
| currentHookRef.current.startListening() | ||
| }, 300) // マイク起動までに少し遅延を入れる | ||
| } | ||
| }, [continuousMicListeningMode, speechRecognitionMode, currentHook]) | ||
| }, [continuousMicListeningMode, speechRecognitionMode]) |
There was a problem hiding this comment.
再開処理は isListening と最新設定を再確認してから実行を
Line 80-105: setTimeout 内で無条件に startListening() されるため、既にリスニング中や設定切替直後に二重起動が起き得ます。タイマー内で最新状態を再チェックする方が安全です。
🐛 提案修正（handleStopSpeaking 例。handleSpeakCompletion も同様に）
console.log('🔄 ストップボタンが押されました。音声認識を再開します。')
setTimeout(() => {
- currentHookRef.current.startListening()
+ const { continuousMicListeningMode, speechRecognitionMode } =
+ settingsStore.getState()
+ const { chatProcessing } = homeStore.getState()
+ if (
+ continuousMicListeningMode &&
+ speechRecognitionMode === 'browser' &&
+ !chatProcessing &&
+ !currentHookRef.current.isListening
+ ) {
+ void currentHookRef.current.startListening()
+ }
}, 300)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| // 常時マイク入力モードの場合、ストップ後にマイクを再開 | |
| // （stopAllではコールバックが呼ばれないため、ここで再開処理を行う） | |
| if ( | |
| settingsStore.getState().continuousMicListeningMode && | |
| settingsStore.getState().speechRecognitionMode === 'browser' && | |
| !homeStore.getState().chatProcessing | |
| ) { | |
| console.log('🔄 ストップボタンが押されました。音声認識を再開します。') | |
| setTimeout(() => { | |
| currentHookRef.current.startListening() | |
| }, 300) | |
| } | |
| }, []) | |
| // AIの発話完了後に音声認識を自動的に再開する処理 | |
| const handleSpeakCompletion = useCallback(() => { | |
| // 常時マイク入力モードがONで、現在マイク入力が行われていない場合のみ実行 | |
| if ( | |
| continuousMicListeningMode && | |
| // !currentHook.isListening && | |
| speechRecognitionMode === 'browser' && | |
| !homeStore.getState().chatProcessing | |
| ) { | |
| console.log('🔄 AIの発話が完了しました。音声認識を自動的に再開します。') | |
| setTimeout(() => { | |
| currentHook.startListening() | |
| currentHookRef.current.startListening() | |
| }, 300) // マイク起動までに少し遅延を入れる | |
| } | |
| }, [continuousMicListeningMode, speechRecognitionMode, currentHook]) | |
| }, [continuousMicListeningMode, speechRecognitionMode]) | |
| // 常時マイク入力モードの場合、ストップ後にマイクを再開 | |
| // （stopAllではコールバックが呼ばれないため、ここで再開処理を行う） | |
| if ( | |
| settingsStore.getState().continuousMicListeningMode && | |
| settingsStore.getState().speechRecognitionMode === 'browser' && | |
| !homeStore.getState().chatProcessing | |
| ) { | |
| console.log('🔄 ストップボタンが押されました。音声認識を再開します。') | |
| setTimeout(() => { | |
| const { continuousMicListeningMode, speechRecognitionMode } = | |
| settingsStore.getState() | |
| const { chatProcessing } = homeStore.getState() | |
| if ( | |
| continuousMicListeningMode && | |
| speechRecognitionMode === 'browser' && | |
| !chatProcessing && | |
| !currentHookRef.current.isListening | |
| ) { | |
| void currentHookRef.current.startListening() | |
| } | |
| }, 300) | |
| } | |
| }, []) | |
| // AIの発話完了後に音声認識を自動的に再開する処理 | |
| const handleSpeakCompletion = useCallback(() => { | |
| // 常時マイク入力モードがONで、現在マイク入力が行われていない場合のみ実行 | |
| if ( | |
| continuousMicListeningMode && | |
| speechRecognitionMode === 'browser' && | |
| !homeStore.getState().chatProcessing | |
| ) { | |
| console.log('🔄 AIの発話が完了しました。音声認識を自動的に再開します。') | |
| setTimeout(() => { | |
| const { continuousMicListeningMode, speechRecognitionMode } = | |
| settingsStore.getState() | |
| const { chatProcessing } = homeStore.getState() | |
| if ( | |
| continuousMicListeningMode && | |
| speechRecognitionMode === 'browser' && | |
| !chatProcessing && | |
| !currentHookRef.current.isListening | |
| ) { | |
| void currentHookRef.current.startListening() | |
| } | |
| }, 300) // マイク起動までに少し遅延を入れる | |
| } | |
| }, [continuousMicListeningMode, speechRecognitionMode]) |
🤖 Prompt for AI Agents
In `@src/hooks/useVoiceRecognition.ts` around lines 80 - 107, The restart logic
currently calls currentHookRef.current.startListening() unconditionally inside
setTimeout, causing possible double-starts; update both the stop handler block
and handleSpeakCompletion to, inside the setTimeout callback, re-read the latest
flags (settingsStore.getState().continuousMicListeningMode,
settingsStore.getState().speechRecognitionMode === 'browser',
homeStore.getState().chatProcessing) and check the current hook's listening
state (e.g., currentHookRef.current?.isListening() or equivalent) before calling
currentHookRef.current.startListening(); only call startListening() when
continuousMicListeningMode is still true, speechRecognitionMode is 'browser',
chatProcessing is false, and the hook is not already listening.
Comment on lines
+82
to
+92
| } catch (error: any) { | ||
| // エラーログを出力 | ||
| console.error('Embedding API error:', error) | ||
|
|
||
| // レート制限エラー | ||
| if (error.status === 429) { | ||
| return res.status(429).json({ | ||
| error: 'Rate limit exceeded. Please try again later.', | ||
| code: 'RATE_LIMITED', | ||
| }) | ||
| } |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, let's examine the full context of the file
wc -l src/pages/api/embedding.tsRepository: tegnike/aituber-kit
Length of output: 255
🏁 Script executed:
# Read the error handling section and surrounding context
sed -n '75,100p' src/pages/api/embedding.tsRepository: tegnike/aituber-kit
Length of output: 255
🌐 Web query:
OpenAI Node.js SDK error handling APIError status code
💡 Result:
Short answer — the OpenAI Node.js SDK throws subclasses of APIError for non-2xx responses. Check err instanceof OpenAI.APIError, then read err.status (HTTP code), err.name (mapped error class like BadRequestError, RateLimitError), err.message and err.headers to handle logic (retry on 429/5xx, surface 4xx to callers, etc.). Example:
- behavior and mapping: 400→BadRequestError, 401→AuthenticationError, 403→PermissionDeniedError, 404→NotFoundError, 422→UnprocessableEntityError, 429→RateLimitError, >=500→InternalServerError. [1][2]
Minimal Node.js pattern:
const client = new OpenAI({ apiKey: process.env.OPENAI_API_KEY });
try {
const res = await client.chat.completions.create({ model: 'gpt-4o', messages: [...] });
} catch (err) {
if (err instanceof OpenAI.APIError) {
// inspect HTTP status and headers
console.log('status', err.status);
console.log('name', err.name);
console.log('request id', err.request_id || err.headers?.['x-request-id']);
if (err.status === 429) { /* backoff & retry / }
else if (err.status >= 500) { / retry or degrade / }
else { / handle/report client errors (4xx) */ }
} else {
throw err;
}
}
Sources: official openai-node error handling docs/README and npm package notes. [1][2]
References
[1] openai/openai-node — Error handling and status→class mapping.
[2] openai package (npm) — error handling examples and properties.
OpenAI SDK のエラーハンドリングに instanceof チェックを追加してください。
OpenAI SDK では、APIError インスタンスは error.status プロパティを持ちます。ただし、現在のコードは error.status === 429 を直接チェックしており、ネットワークエラーなど他の種類のエラーでは status プロパティが存在しないため、エラーハンドリングが不完全です。
以下のパターンで改善してください：
- 最初に
error instanceof OpenAI.APIErrorで確認 - その後、
error.statusにアクセスしてステータスコードを判定 - APIError 以外のエラーは別途処理
🤖 Prompt for AI Agents
In `@src/pages/api/embedding.ts` around lines 82 - 92, In the catch block in
src/pages/api/embedding.ts replace the direct check of error.status with an
explicit OpenAI APIError check: first test if error instanceof OpenAI.APIError
(from the OpenAI SDK), then read error.status to handle the 429 rate-limit
branch and other HTTP error statuses; for non-APIError errors, handle them
separately (log and return a generic 500 response). Update the catch around the
embedding request (the catch that currently logs "Embedding API error:") to use
this instanceof pattern so only APIError instances have their .status accessed
and other errors are safely handled.
Comment on lines
+38
to
+41
| // ファイル名の安全性チェック（パストラバーサル対策） | ||
| if (filename.includes('..') || filename.includes('/')) { | ||
| return res.status(400).json({ message: 'Invalid filename' }) | ||
| } |
There was a problem hiding this comment.
パストラバーサル対策がWindows環境で不完全です。
/ のみをチェックしていますが、Windows環境では \ もパス区切り文字として使用されます。また、path.basename() を使用する方がより堅牢です。
🔒 修正案
// ファイル名の安全性チェック（パストラバーサル対策）
- if (filename.includes('..') || filename.includes('/')) {
+ if (filename.includes('..') || filename.includes('/') || filename.includes('\\') || path.basename(filename) !== filename) {
return res.status(400).json({ message: 'Invalid filename' })
}🤖 Prompt for AI Agents
In `@src/pages/api/memory-restore.ts` around lines 38 - 41, The current filename
safety check only looks for '..' and '/' and misses Windows backslashes; replace
the ad-hoc check by using path.basename to canonicalize the name: require/import
Node's path, and in the handler verify that filename === path.basename(filename)
(and optionally still reject filenames that include '..' for extra safety)
before proceeding—if it doesn't match, return res.status(400).json({ message:
'Invalid filename' }); reference the filename variable and the response object
res in src/pages/api/memory-restore.ts when implementing this change.
- タイポ修正: reasong → reason, a the → at the - 無効なTailwindクラス修正: z-5 → z-10 - テストの日付生成を修正: (i % 28) + 1 で無効な日付を回避 - searchMemoryContext に空クエリの早期リターン追加 - formatTimestamp に無効日付のフォールバック追加 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Contributor
|
Claude finished @tegnike's task —— View job PR Review - Comprehensive Analysis
|
- Introduced translations for PNGTuber asset management and settings in Spanish, French, Hindi, Italian, Korean, Polish, Portuguese, Russian, Thai, Vietnamese, and Chinese. - Added memory management settings including enabling/disabling memory, memory thresholds, search limits, and restoration options across all supported languages.
feat: Split zh locale into zh-CN and zh-TW
Contributor
|
Claude finished @tegnike's task —— View job PR Review - Comprehensive Analysis
|
There was a problem hiding this comment.
Actionable comments posted: 1
Note
Due to the large number of review comments, Critical severity comments were prioritized as inline comments.
🤖 Fix all issues with AI agents
In `@docs/README_zh-TW.md`:
- Around line 122-123: The README_zh-TW.md specifies Node.js: ^20.0.0 and npm:
^10.0.0 which conflicts with other READMEs that list Node.js: ^25.2.1 and npm:
^11.6.2; update README_zh-TW.md to match the chosen project-wide standard
(either change the "Node.js: ^20.0.0" and "npm: ^10.0.0" lines to "Node.js:
^25.2.1" and "npm: ^11.6.2" if aligning to existing files, or preferably update
all README files to a single LTS version such as Node.js 20.x and a compatible
npm version), ensuring the same version strings are used across README.md,
README_ko.md, README_zh-CN.md, and README_zh-TW.md.
♻️ Duplicate comments (3)
locales/fr/translation.json (1)
457-480: 上記と同様のガイドライン違反です。MemorySettings セクションも手動で追加されています。日本語（ja）ロケール以外のファイルは手動で更新しないというコーディングガイドラインに従ってください。
docs/README_ko.md (1)
122-123: Node.jsバージョンの懸念事項（既存レビューと重複）Node.js ^25.2.1とnpm ^11.6.2が指定されていますが、README.mdの既存レビューコメントで指摘されている通り、以下の懸念があります：
- Node.js 25.xは非LTSリリースで、2026年6月頃までのサポートのみ
- npm 11.6.2には既知のCI破損問題とLinux環境でのランタイム失敗が報告されている
本番環境や長期プロジェクトではLTSバージョン（20.x、22.x、24.x）の使用を推奨します。
docs/README_zh-CN.md (1)
122-123: Node.jsバージョンの懸念事項（既存レビューと重複）Node.js ^25.2.1とnpm ^11.6.2が指定されていますが、README.mdの既存レビューコメントで指摘されている通り、以下の懸念があります：
- Node.js 25.xは非LTSリリースで、短期間のサポートのみ
- npm 11.6.2には既知のCI破損問題とLinux環境での実行時エラーが報告されている
安定性のためにLTSバージョンへの変更を検討してください。
🟠 Major comments (27)
locales/it/translation.json-293-305 (1)
293-305: 非jaロケールの手動更新は禁止されていますこの変更は
locales/!(ja)/**/*.jsonのルールに抵触します。イタリア語ファイルの手動更新は避け、所定のローカライズ手順に従ってください。As per coding guidelines, ...locales/it/translation.json-457-480 (1)
457-480: 非jaロケールの手動更新は禁止されていますこの変更もガイドライン違反です。イタリア語ファイルの更新は行わず、正規の翻訳フローで反映してください。As per coding guidelines, ...
locales/ar/translation.json-293-305 (1)
293-305: 非日本語ロケールの直接更新はガイドライン違反です。
locales/!(ja)/**/*.jsonは手動更新禁止のため、この変更はガイドラインに反しています。翻訳フロー/専用手順で更新するか、該当差分の取り扱いを再確認してください。Also applies to: 457-480
locales/fr/translation.json-293-305 (1)
293-305: コーディングガイドラインに違反しています。コーディングガイドラインによると、
locales/!(ja)/**/*.jsonパターンに一致するファイル（日本語以外の言語ファイル）は手動で更新しないことになっています。このフランス語翻訳ファイルは手動で更新されているようです。翻訳は自動化されたシステムや翻訳管理ツールを通じて追加されるべきです。手動での更新は、自動同期時に上書きされる可能性があります。
locales/ru/translation.json-293-305 (1)
293-305: 非jaロケールの手動更新はガイドライン違反です。
locales/ru/translation.jsonは手動更新禁止対象なので、今回の追加分（PNGTuber / MemorySettings など）は翻訳フローに従って反映してください。As per coding guidelines, ...Also applies to: 457-480
docs/logo_licence_zh-TW.md-24-26 (1)
24-26: 法律用語の修正が必要です。26行目の「現狀有姿」は中国語の標準的な法律用語ではありません。英語の "AS IS" に対応する正しい中国語訳は「按現狀」、「依現狀」、または「以現狀」です。法的な正確性のため、標準的な用語に修正することを強く推奨します。
🔧 提案された修正
## 免責聲明 -此商標以"現狀有姿"提供，不保證適用於特定目的或不侵權。對於使用商標造成的任何損害，儲存庫的所有者不承擔任何責任。 +此商標以"按現狀"提供，不保證適用於特定目的或不侵權。對於使用商標造成的任何損害，儲存庫的所有者不承擔任何責任。.claude/commands/merge-pr.md-78-78 (1)
78-78: package-lock.jsonの競合解決ガイダンスを修正してくださいpackage-lock.jsonの競合を手動でマージすることは推奨されません。lock fileは自動生成されるべきファイルであり、手動でのマージはエラーを引き起こす可能性が高いです。
✏️ 推奨される修正
-- **package.json / package-lock.json**: 両方の依存関係を保持し、`npm install` で整合性を確認 +- **package.json**: 両方の依存関係を保持 +- **package-lock.json**: どちらか一方（通常はdevelop側）を採用し、マージ後に `npm install` を実行して再生成補足説明を追加することも推奨します：
> package-lock.jsonは手動でマージせず、必ず `npm install` で再生成してください。手動マージはlock fileの整合性を損なう可能性があります。.claude/commands/merge-pr.md-32-36 (1)
32-36: developブランチの更新ステップが不足しています
git merge developを実行する前に、developブランチを最新の状態に更新するステップが必要です。現在のドキュメントでは、ローカルのdevelopブランチが古い可能性があり、最新の変更がマージされない可能性があります。🔄 推奨される修正
### 3. developブランチをマージ +まず、developブランチを最新の状態に更新します： + +```bash +git fetch origin develop +git checkout develop +git pull origin develop +git checkout - # PRブランチに戻る +``` + +次に、developブランチをマージします： + ```bash git merge developまたは、より簡潔に： ```diff ### 3. developブランチをマージ +```bash +git fetch origin +git merge origin/develop +``` -```bash -git merge develop -```locales/zh-CN/translation.json-293-305 (1)
293-305: コーディングガイドライン違反: 非日本語ロケールファイルの手動更新コーディングガイドラインによると、
locales/!(ja)/**/*.jsonは手動で更新すべきではありません。日本語以外の言語ファイル（zh-CN等）は自動翻訳スクリプト（scripts/auto_translate.py）によって生成されるべきです。このPNGTuberセクションが
locales/ja/translation.jsonに追加されていれば、自動翻訳ワークフローで中国語版が生成されます。locales/zh-CN/translation.json-457-480 (1)
457-480: コーディングガイドライン違反: 非日本語ロケールファイルの手動更新同様に、このメモリ関連の翻訳キーも手動で追加すべきではありません。
推奨対応:
locales/ja/translation.jsonに日本語の翻訳キーを追加- 自動翻訳ワークフローを実行して他言語ファイルを生成
コーディングガイドラインに基づき、日本語ファイルのみを手動更新し、他の言語は自動翻訳に任せてください。
locales/es/translation.json-293-305 (1)
293-305: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/es/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/vi/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/ko/translation.json-293-305 (1)
293-305: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/th/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/vi/translation.json-287-299 (1)
287-299: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/ko/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/hi/translation.json-287-299 (1)
287-299: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/hi/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/th/translation.json-287-299 (1)
287-299: 非日本語ロケールの手動更新は禁止されています。
この追加はガイドライン違反のため差し戻し、翻訳の反映は所定の自動翻訳／同期フローで行ってください。コーディングガイドラインに基づき、locales/pl/translation.json-287-299 (1)
287-299: 非日本語ロケールの手動更新は避けてくださいこの変更は
locales/plの手動更新に該当します。ガイドライン上、更新はlocales/ja/のみで行い、他言語は自動同期のフローに任せてください。As per coding guidelines, non-ja locale files must not be edited manually.locales/de/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は避けてくださいこの変更は
locales/deの手動更新に該当します。ガイドライン上、更新はlocales/ja/のみで行い、他言語は自動同期のフローに任せてください。As per coding guidelines, non-ja locale files must not be edited manually.locales/pl/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は避けてくださいこの変更は
locales/plの手動更新に該当します。ガイドライン上、更新はlocales/ja/のみで行い、他言語は自動同期のフローに任せてください。As per coding guidelines, non-ja locale files must not be edited manually.locales/en/translation.json-326-338 (1)
326-338: 非日本語ロケールの手動更新は避けてくださいこの変更は
locales/enの手動更新に該当します。ガイドライン上、更新はlocales/ja/のみで行い、他言語は自動同期のフローに任せてください。As per coding guidelines, non-ja locale files must not be edited manually.locales/en/translation.json-457-480 (1)
457-480: 非日本語ロケールの手動更新は避けてくださいこの変更は
locales/enの手動更新に該当します。ガイドライン上、更新はlocales/ja/のみで行い、他言語は自動同期のフローに任せてください。As per coding guidelines, non-ja locale files must not be edited manually.locales/de/translation.json-293-305 (1)
293-305: 非日本語ロケールの手動更新は避けてくださいこの変更は
locales/deの手動更新に該当します。ガイドライン上、更新はlocales/ja/のみで行い、他言語は自動同期のフローに任せてください。As per coding guidelines, non-ja locale files must not be edited manually.locales/en/translation.json-371-372 (1)
371-372: 非日本語ロケールの手動更新は避けてくださいこの変更は
locales/enの手動更新に該当します。ガイドライン上、更新はlocales/ja/のみで行い、他言語は自動同期のフローに任せてください。As per coding guidelines, non-ja locale files must not be edited manually.
🟡 Minor comments (2)
.env.example-47-61 (1)
47-61: PNGTuber設定ブロックの構成と命名規則は適切です。新しいPNGTuber機能の環境変数が適切に追加され、命名規則、コメント形式、セクション配置がすべて一貫しており、コーディングガイドラインに準拠しています。
ただし、1つの改善が必要です：
許容値の範囲を明記してください: Line 56の
NEXT_PUBLIC_PNGTUBER_CHROMA_KEY_TOLERANCEのコメントに範囲を追加すると、ユーザーにとってより分かりやすくなります。実装で範囲は 0-255 に制限されているため、この範囲を明記してください。推奨される修正
# PNGTuberクロマキー設定 / PNGTuber chroma key settings NEXT_PUBLIC_PNGTUBER_CHROMA_KEY_ENABLED="false" NEXT_PUBLIC_PNGTUBER_CHROMA_KEY_COLOR="#00FF00" +# PNGTuberクロマキー許容値（0-255） / PNGTuber chroma key tolerance (0-255) -NEXT_PUBLIC_PNGTUBER_CHROMA_KEY_TOLERANCE="50" +NEXT_PUBLIC_PNGTUBER_CHROMA_KEY_TOLERANCE="50"docs/README_pl.md-122-123 (1)
122-123: npm 版数の表記を修正してくださいドキュメント（docs/README_pl.md）に記載されている npm 版数
^11.6.2は、package.json の volta セクションでの指定11.6.2（キャレット記号なし）と一致していません。Volta はツールバージョンの正確なピン留めを管理するため、npm 版数もキャレット記号を削除して11.6.2に修正してください。
🧹 Nitpick comments (5)
.claude/skills/sync-translations/SKILL.md (2)
39-41: コードブロックに言語指定子を追加することを推奨します。マークダウンのベストプラクティスとして、フェンスドコードブロックには言語指定子を付けることが推奨されます。
♻️ 提案する修正
-``` +```text locales/ja/translation.json</details> --- `78-80`: **コードブロックに言語指定子を追加することを推奨します。** コマンド例には言語指定子を付けることで、シンタックスハイライトと可読性が向上します。 <details> <summary>♻️ 提案する修正</summary> ```diff -``` +```bash /sync-translations</details> </blockquote></details> <details> <summary>docs/logo_licence_zh-TW.md (1)</summary><blockquote> `7-9`: **著作権者の明確化を推奨します。** 著作権の帰属が曖昧です。文書の最後に記載されているロゴ作成者（Ruka Design）への具体的な参照を含めることで、より明確になります。 <details> <summary>♻️ 提案された改善</summary> ```diff ## 版權 -此商標的版權屬於標誌的創作者。 +此商標的版權屬於標誌的創作者（Ruka Design）。.claude/commands/merge-pr.md (2)
64-67: コミットメッセージのプレースホルダーに説明を追加コミットメッセージのテンプレートで
<PR branch name>というプレースホルダーを使用していますが、このブランチ名は手順1で取得したheadRefNameから参照できることを明記すると、より分かりやすくなります。📝 提案される改善
4. **解決後の処理** ```bash git add <解決したファイル> - git commit -m "Merge branch 'develop' into <PR branch name>" + git commit -m "Merge branch 'develop' into <PRブランチ名>" ``` + + ※PRブランチ名は手順1で取得した `headRefName` の値、または `git branch --show-current` で確認できます。
69-73: マージ後の動作確認ステップの追加を推奨現在のワークフローでは、最終確認として
git statusとgit logのみを実行していますが、マージが正常に機能するかを確認するステップがありません。特に競合解決後は、ビルドやテストが通ることを確認することが重要です。🧪 提案される改善
5. **最終確認** ```bash git status git log --oneline -3 ``` + +6. **動作確認（推奨）** + ```bash + # 依存関係の競合があった場合 + npm install + + # ビルドが通ることを確認 + npm run build + + # テストが通ることを確認（オプション） + npm test + ``` + + マージ後にビルドエラーやテスト失敗がないことを確認してください。
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (40)
.claude/commands/merge-pr.md.claude/skills/sync-translations/SKILL.md.env.exampleCLAUDE.mdREADME.mddocs/README_en.mddocs/README_ko.mddocs/README_pl.mddocs/README_zh-CN.mddocs/README_zh-TW.mddocs/auto_translate.mddocs/character_model_licence_zh-CN.mddocs/character_model_licence_zh-TW.mddocs/logo_licence_zh-CN.mddocs/logo_licence_zh-TW.mdlocales/ar/translation.jsonlocales/de/translation.jsonlocales/en/translation.jsonlocales/es/translation.jsonlocales/fr/translation.jsonlocales/hi/translation.jsonlocales/it/translation.jsonlocales/ja/translation.jsonlocales/ko/translation.jsonlocales/pl/translation.jsonlocales/pt/translation.jsonlocales/ru/translation.jsonlocales/th/translation.jsonlocales/vi/translation.jsonlocales/zh-CN/translation.jsonlocales/zh-TW/translation.jsonscripts/auto_translate.pysrc/components/settings/based.tsxsrc/features/constants/settings.tssrc/features/messages/synthesizeVoiceGoogle.tssrc/lib/i18n.jssrc/pages/_app.tsxsrc/pages/api/convertSlide.tssrc/pages/api/stylebertvits2.tssrc/utils/voiceLanguage.ts
✅ Files skipped from review due to trivial changes (1)
- locales/zh-TW/translation.json
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/README_en.md
🧰 Additional context used
📓 Path-based instructions (3)
locales/!(ja)/**/*.json
📄 CodeRabbit inference engine (CLAUDE.md)
他の言語ファイル（en、ko、zh等）は手動で更新しない
Files:
locales/es/translation.jsonlocales/zh-CN/translation.jsonlocales/en/translation.jsonlocales/fr/translation.jsonlocales/th/translation.jsonlocales/pl/translation.jsonlocales/pt/translation.jsonlocales/ru/translation.jsonlocales/it/translation.jsonlocales/de/translation.jsonlocales/hi/translation.jsonlocales/ar/translation.jsonlocales/ko/translation.jsonlocales/vi/translation.json
locales/ja/**/*.json
📄 CodeRabbit inference engine (CLAUDE.md)
言語ファイルの更新は日本語（
/locales/ja/）のみ行う
Files:
locales/ja/translation.json
.env.example
📄 CodeRabbit inference engine (CLAUDE.md)
設定画面の項目を追加・更新した場合は、必要に応じて新しい環境変数を
.env.exampleの適切な項目に追加してください。
Files:
.env.example
🧠 Learnings (5)
📚 Learning: 2025-08-05T17:30:02.776Z
Learnt from: CR
Repo: tegnike/aituber-kit PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-08-05T17:30:02.776Z
Learning: Applies to locales/!(ja)/**/*.json : 他の言語ファイル（en、ko、zh等）は手動で更新しない
Applied to files:
CLAUDE.md.claude/skills/sync-translations/SKILL.mddocs/auto_translate.mdlocales/ja/translation.json.env.example
📚 Learning: 2025-08-05T17:30:02.776Z
Learnt from: CR
Repo: tegnike/aituber-kit PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-08-05T17:30:02.776Z
Learning: Applies to locales/ja/**/*.json : 言語ファイルの更新は日本語（`/locales/ja/`）のみ行う
Applied to files:
CLAUDE.md.claude/skills/sync-translations/SKILL.mddocs/auto_translate.mdlocales/ja/translation.json.env.example
📚 Learning: 2025-08-05T17:30:02.776Z
Learnt from: CR
Repo: tegnike/aituber-kit PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-08-05T17:30:02.776Z
Learning: v2.0.0以降は独自ライセンス、非商用利用は無料、商用利用には別途ライセンスが必要、キャラクターモデルの利用には個別のライセンスが必要
Applied to files:
docs/README_ko.mddocs/README_zh-CN.mdREADME.mddocs/character_model_licence_zh-TW.mddocs/README_pl.mddocs/README_zh-TW.md
📚 Learning: 2025-08-05T17:30:02.776Z
Learnt from: CR
Repo: tegnike/aituber-kit PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-08-05T17:30:02.776Z
Learning: Node.js環境用にcanvasをモック化済み
Applied to files:
README.md
📚 Learning: 2025-08-05T17:30:02.776Z
Learnt from: CR
Repo: tegnike/aituber-kit PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-08-05T17:30:02.776Z
Learning: Applies to .env.example : 設定画面の項目を追加・更新した場合は、必要に応じて新しい環境変数を`.env.example`の適切な項目に追加してください。
Applied to files:
.env.example
🪛 dotenv-linter (4.0.0)
.env.example
[warning] 45-45: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 48-48: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 51-51: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 54-54: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 55-55: [UnorderedKey] The NEXT_PUBLIC_PNGTUBER_CHROMA_KEY_COLOR key should go before the NEXT_PUBLIC_PNGTUBER_CHROMA_KEY_ENABLED key
(UnorderedKey)
[warning] 56-56: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 59-59: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 60-60: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 60-60: [UnorderedKey] The NEXT_PUBLIC_PNGTUBER_OFFSET_X key should go before the NEXT_PUBLIC_PNGTUBER_SCALE key
(UnorderedKey)
[warning] 61-61: [QuoteCharacter] The value has quote characters (', ")
(QuoteCharacter)
[warning] 61-61: [UnorderedKey] The NEXT_PUBLIC_PNGTUBER_OFFSET_Y key should go before the NEXT_PUBLIC_PNGTUBER_SCALE key
(UnorderedKey)
🪛 LanguageTool
docs/README_zh-TW.md
[uncategorized] ~125-~125: 您的意思是“"不"驟”？
Context: ... Node.js: ^20.0.0 - npm: ^10.0.0 ### 安裝步驟 1. 將倉庫克隆到本地。 ```bash git clone http...
(BU)
[uncategorized] ~304-~304: 您的意思是“"不"驟”？
Context: ...題時，包含以下資訊將有助於我們順利響應： - 問題或新功能的詳細描述 - 重現步驟（對於錯誤） - 預期行為與實際行為 - 使用環境（瀏覽器、作業系統、Node...
(BU)
🪛 markdownlint-cli2 (0.18.1)
.claude/skills/sync-translations/SKILL.md
39-39: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
78-78: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
docs/README_zh-TW.md
3-3: Images should have alternate text (alt text)
(MD045, no-alt-text)
347-347: Bare URL used
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: claude-review
🔇 Additional comments (33)
locales/pt/translation.json (2)
288-300: コーディングガイドライン違反: 非日本語ロケールファイルの手動更新コーディングガイドラインによると、
locales/!(ja)/**/*.jsonパターンに一致するファイル（ポルトガル語を含む）は手動で更新しないこととされています。この変更が意図的なものである場合、または自動翻訳プロセスによって生成されたものである場合は、このコメントを無視してください。そうでない場合は、翻訳の自動化プロセスを使用することを検討してください。
コーディングガイドラインに基づいています。
457-480: コーディングガイドライン違反: Memory関連翻訳の手動追加上記と同様に、このMemory関連の翻訳ブロックも
locales/!(ja)/**/*.jsonガイドラインに抵触しています。翻訳内容自体は問題なさそうですが、他の言語ファイル（en、ko、zh等）と同様に、自動翻訳プロセスを通じて更新されるべきかどうかをご確認ください。
コーディングガイドラインに基づいています。
CLAUDE.md (1)
78-78: ドキュメントの改善として適切です中国語ロケールの扱いを「zh等」から「zh-CN、zh-TW等」と明示的に記載することで、どの言語ファイルを手動更新の対象外とすべきか、より明確になりました。
src/features/constants/settings.ts (2)
129-130: VoiceLanguageへの'zh-CN'追加は適切です。
VoiceLanguage型に'zh-CN'を追加することで、LANGUAGES配列との一貫性が保たれています。これにより、簡体字中国語のTTS/音声ローカライゼーション機能が有効になります。
107-108: 中国語ロケール分割の実装は適切で完全です。
'zh-CN'（簡体字）と'zh-TW'（繁体字）への分割は既に正しく実装されています。旧'zh'への参照はコードベース内に存在せず、locales/zh-CNとlocales/zh-TWディレクトリも正しく構成されているため、破壊的変更の懸念はありません。変更は承認できます。locales/ru/translation.json (1)
456-458: JSONの重複キーで値が上書きされます。
UsingAivisCloudAPIが重複しており、後勝ちで上書きされるためツールや翻訳管理に悪影響が出ます。重複エントリを削除し、AivisCloudAPIInfoは既存のUsingAivisCloudAPI直後に配置してください。🛠️ 修正案（重複キー削除）
- "UsingAivisCloudAPI": "Использовать Aivis Cloud API", - "AivisCloudAPIInfo": "Настройки Aivis Cloud API", + "AivisCloudAPIInfo": "Настройки Aivis Cloud API",Likely an incorrect or invalid review comment.
.claude/skills/sync-translations/SKILL.md (4)
1-6: スキルメタデータの定義が適切です。YAML front matterの設定が明確で、必要なツール（Read、Grep、Glob、Edit、Write、Bash、Task、TodoWrite）がすべて含まれています。
14-31: 対象言語の一覧が明確です。14言語のファイルパスが一貫したフォーマットで記載されており、対象範囲が明確に定義されています。
33-74: 実行手順とルールが明確に定義されています。翻訳ファイルの整合性を保つための適切なルールが設定されており、既存の翻訳を上書きしない、JSON構造を保持するなど、重要な注意事項が明記されています。
1-90: 翻訳ワークフローとの整合性を確認してください。学習データによると、「言語ファイルの更新は日本語（
/locales/ja/）のみ行う」および「他の言語ファイル（en、ko、zh等）は手動で更新しない」とされています。このスキルは他の言語ファイルへのキー追加を自動化しますが、実際の翻訳は別プロセスで行うとのことです。この自動化アプローチがプロジェクトの翻訳ワークフローと整合しているか確認することを推奨します。
確認ポイント：
- 構造同期（キーの追加）の自動化は許容されるか
- 日本語の値をプレースホルダーとして使用するアプローチは適切か
- 実際の翻訳プロセスとの連携方法
Based on learnings, 他の言語ファイルは手動で更新しないという方針があります。
docs/logo_licence_zh-TW.md (6)
11-15: LGTM!許可される使用範囲が明確に定義されており、適切です。
28-30: LGTM!契約変更条項は標準的で適切です。
32-34: LGTM!連絡方法が明確に記載されています。
5-5: GitHubリポジトリへのリンクは有効です。検証済み: https://github.com/tegnike/aituber-kit は HTTP 200 で正常にアクセス可能であり、リンクは正しく機能しています。
36-38: Coconalaリンクは有効です。ロゴ作成者（Ruka Design）への帰属表示が適切に行われており、リンク（https://coconala.com/users/2208636）は正常にアクセス可能です（HTTP 200）。この属性表示は問題ありません。
17-22: 商標使用条項は、オープンソースライセンスとは別の問題です。このドキュメント「商標使用条項」は、ソフトウェアコードではなく、商標およびロゴの使用を規定するものです。商標の配布を制限することは、ソフトウェアライセンスの原則とは独立した、標準的な商標保護の慣行であり、オープンソース原則と矛盾しません。オープンソースプロジェクトは、開放的なソフトウェアライセンスを採用しつつ、ブランド保護のため商標使用を制限することは一般的です。
Likely an incorrect or invalid review comment.
.env.example (2)
11-15: 言語ドキュメントの更新を承認します。中国語の簡体字（zh-CN）と繁体字（zh-TW）を明示的に区別する説明が追加されており、ユーザーにとって分かりやすくなっています。
44-45: モデルタイプの説明更新を承認します。コメントに「pngtuber」オプションが追加され、新機能に対応しています。デフォルト値は既存の安定したオプション「vrm」のままで適切です。
docs/character_model_licence_zh-TW.md (1)
1-68: LGTM! 繁体字中国語のライセンスドキュメントが適切に追加されています。ドキュメントの構成が適切で、使用条件、著作権、許可・禁止事項、免責事項を網羅しています。モデル製作者のクレジット表記も正しく、日本語の敬称（様）を維持しているのは適切です。
docs/auto_translate.md (2)
38-38: LGTM! ドキュメントの更新が適切です。対象言語の説明が
zh-CN（簡体字中国語）とzh-TW（繁体字中国語）を反映するように正しく更新されています。
75-75: LGTM! コード例の更新が適切です。
TARGET_LANGUAGESの例が新しい言語コードを反映しています。scripts/auto_translate.py (1)
38-54: LGTM! 中国語の簡体字・繁体字分離は適切に実装されています。
zhからzh-CN（簡体字中国語）とzh-TW（繁体字中国語）への分割は、BCP 47言語タグに準拠しており、より正確なローカライゼーションを実現します。対応するロケールディレクトリ（locales/zh-CN/およびlocales/zh-TW/）と翻訳ファイルが既に存在し、実装は完全です。src/components/settings/based.tsx (1)
158-159: LGTM! 言語選択オプションの追加は適切です。簡体字・繁体字中国語のオプションが正しく追加されています。
Language型にzh-CNとzh-TWが含まれており、対応するロケールファイルも存在します。ラベル形式も他の言語と一貫しています。locales/ja/translation.json (4)
129-129: {{count}} 削除の意図を確認してください他言語では件数のプレースホルダーが残っているため、日本語だけ件数が表示されなくなります。UIで件数表示が必要なら、
{{count}}を維持するか別文言にする意図を確認してください。
346-358: PNGTuber 文言追加は問題ありません新機能向けのラベル/説明が一通り揃っており、UI 表示の観点で十分です。
390-391: 画像エラー文言の追加は問題ありません画像関連のエラーメッセージが補完されており、UX向上につながります。
457-480: メモリ設定の文言追加は問題ありません設定項目と説明が揃っており、画面側の文言として十分です。
docs/README_pl.md (1)
42-43: 言語リンク先ファイルの存在確認README_zh-CN.md / README_zh-TW.md はいずれも docs/ 配下に正常に存在しており、リンク切れの問題はありません。
docs/README_ko.md (1)
42-43: 言語ナビゲーションの改善を確認しました。中国語のリンクを簡体字（zh-CN）と繁体字（zh-TW）に分割したことで、より正確なローカリゼーションが実現されています。
docs/README_zh-CN.md (2)
42-44: 言語ナビゲーションの更新を確認しました。繁体字中国語（zh-TW）への明示的なリンクが追加され、言語選択がより明確になりました。
333-334: 簡体字中国語ライセンスファイルの存在は確認済み参照されている簡体字中国語固有のライセンスファイルは既に存在しており、リンクは有効です。
./logo_licence_zh-CN.mdと./character_model_licence_zh-CN.mdは正常に配置されています。README.md (1)
41-42: 言語ナビゲーションの改善を確認しました。中国語のリンクを簡体字（README_zh-CN.md）と繁体字（README_zh-TW.md）に明確に分割したことで、ユーザーにとってより分かりやすくなりました。この変更は他の言語版READMEとも一貫性が取れています。
docs/README_zh-TW.md (1)
333-336: 追加の検証は不要 - ファイルは実際に存在します参照されている繁体字中国語のライセンスファイル（
./logo_licence_zh-TW.mdおよび./character_model_licence_zh-TW.md）は実際にdocs/ディレクトリに存在しており、リンクは正しく機能しています。Likely an incorrect or invalid review comment.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
Comment on lines
+122
to
+123
| - Node.js: ^20.0.0 | ||
| - npm: ^10.0.0 |
There was a problem hiding this comment.
開発環境バージョンの不整合（重大な問題）
このファイルではNode.js ^20.0.0とnpm ^10.0.0が指定されていますが、他のすべてのREADMEファイル（README.md、README_ko.md、README_zh-CN.md）ではNode.js ^25.2.1とnpm ^11.6.2が指定されています。
この不整合により、ユーザーが言語によって異なる開発環境をセットアップする可能性があり、混乱や環境構築の問題を引き起こします。
🔧 修正案
他のREADMEファイルと整合性を取るため、以下のいずれかの対応が必要です：
オプション1：Node.js 25.xに統一する場合
-- Node.js: ^20.0.0
-- npm: ^10.0.0
+- Node.js: ^25.2.1
+- npm: ^11.6.2オプション2（推奨）：すべてのREADMEをLTSバージョンに統一する場合
-- Node.js: ^20.0.0
-- npm: ^10.0.0
+- Node.js: ^20.18.0
+- npm: ^10.9.2オプション2を推奨する理由：
- Node.js 20.xはLTSバージョンで2026年4月まで積極的なサポートがある
- Node.js 25.xは非LTSで短期サポートのみ
- npm 11.6.2には既知の問題が報告されている
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| - Node.js: ^20.0.0 | |
| - npm: ^10.0.0 | |
| - Node.js: ^25.2.1 | |
| - npm: ^11.6.2 |
Suggested change
| - Node.js: ^20.0.0 | |
| - npm: ^10.0.0 | |
| - Node.js: ^20.18.0 | |
| - npm: ^10.9.2 |
🤖 Prompt for AI Agents
In `@docs/README_zh-TW.md` around lines 122 - 123, The README_zh-TW.md specifies
Node.js: ^20.0.0 and npm: ^10.0.0 which conflicts with other READMEs that list
Node.js: ^25.2.1 and npm: ^11.6.2; update README_zh-TW.md to match the chosen
project-wide standard (either change the "Node.js: ^20.0.0" and "npm: ^10.0.0"
lines to "Node.js: ^25.2.1" and "npm: ^11.6.2" if aligning to existing files, or
preferably update all README files to a single LTS version such as Node.js 20.x
and a compatible npm version), ensuring the same version strings are used across
README.md, README_ko.md, README_zh-CN.md, and README_zh-TW.md.
Claude Sonnet 4.5を追加
Contributor
|
Claude finished @tegnike's task —— View job PR Review - Comprehensive Analysis
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/features/constants/aiModels.ts`:
- Around line 154-155: The constant openaiAudio currently defaults to the
speech-only model 'tts-1' which is incompatible with the chat completions call
in openAIAudioChat.ts (used with openai.chat.completions.create()); update the
default value of openaiAudio in aiModels.ts to a chat-capable audio model (for
example 'gpt-4o-audio' or another chat-compatible audio model), so
openAIAudioChat.ts uses a model supported by the Chat Completions API.
🧹 Nitpick comments (1)
src/features/constants/aiModels.ts (1)
217-221:openAIAudioModelsとopenAITTSModelsの重複。
openAIAudioModels（217-221行目）とopenAITTSModels（258行目）の内容が完全に同一です：// Line 217-221 export const openAIAudioModels = ['tts-1', 'tts-1-hd', 'gpt-4o-mini-tts'] as const // Line 258 export const openAITTSModels = ['tts-1', 'tts-1-hd', 'gpt-4o-mini-tts'] as constこの2つの定数の用途が異なる場合は、それぞれに適切なモデルを設定してください。同じ用途であれば、一方を削除するか、片方から参照する形にリファクタリングすることを検討してください。
♻️ 重複を解消する提案
export const openAIAudioModels = [ 'tts-1', 'tts-1-hd', 'gpt-4o-mini-tts', ] as const -export const openAITTSModels = ['tts-1', 'tts-1-hd', 'gpt-4o-mini-tts'] as const +export const openAITTSModels = openAIAudioModelsまたは、用途が異なる場合はコメントで違いを明確化してください。
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/features/constants/aiModels.ts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: claude-review
🔇 Additional comments (3)
src/features/constants/aiModels.ts (3)
209-212: リアルタイムAPIモデル名の形式変更。日付付きのプレビューバージョンから汎用名に変更されています。OpenAI APIでこれらの識別子が有効であることを確認してください。
20-33: OpenAI APIの公式ドキュメント（2026年1月時点）によると、コード内のGPT-5.xシリーズのモデル識別子（gpt-5.2-pro、gpt-5.1-codex、gpt-5-mini等）はすべてOpenAIの公式APIでサポートされています。これらのモデル名は正確で、API呼び出し時のエラーは発生しません。
42-42: 問題なし。
claude-sonnet-4-5-20250929はAnthropicの公式APIの有効なモデル識別子です。
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
Comment on lines
+154
to
+155
| openaiAudio: 'tts-1', | ||
| openaiRealtime: 'gpt-realtime', |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
OpenAI gpt-realtime model API 2026 valid identifier
💡 Result:
Valid OpenAI Realtime model identifiers (current as of Jan 16, 2026):
- gpt-realtime. [1]
- gpt-realtime-2025-08-28 (snapshot/alias). [1]
- gpt-realtime-mini. [2]
- gpt-realtime-mini-2025-10-06 (and other mini snapshots such as 2025-12-15). [2]
Sources: OpenAI Models docs for "gpt-realtime" and "gpt-realtime-mini". [1][2]
🌐 Web query:
OpenAI tts-1 model text-to-speech specifications
💡 Result:
Here are the main public specifications for OpenAI’s tts-1 text-to-speech model (concise summary):
- Purpose: realtime / low-latency text-to-speech model (standard quality). [1][2]
- Endpoint: Speech generation via the Audio / Speech endpoint (v1/audio/speech). [1]
- Voices: ~11 built‑in voices (examples: alloy, ash, ballad, coral, echo, fable, nova, onyx, sage, shimmer). Voices optimized for English; realtime voice set may vary. [2]
- Quality/variants: tts-1 (lower latency) and tts-1‑hd (higher quality, higher latency/cost). [1][2]
- Output formats: common audio formats supported (mp3/opus/aac/flac/wav/pcm for non‑streaming responses via API implementations). (API examples and wrappers show these formats.) [2]
- Speed control: adjustable speech speed (commonly exposed range ~0.25–4.0; default 1.0 in API examples). [2]
- Pricing (public doc): listed as $15 for tts-1 and $30 for tts-1‑hd (check official pricing page for any updates). [1]
- Rate limits: documented RPM/RPD tiers on OpenAI docs (Free tier and higher paid tiers listed). [1]
Sources: OpenAI model & text-to-speech docs. [1] OpenAI Models: TTS‑1 documentation. [2] OpenAI Text-to-Speech guide (voices, controls, examples).
🏁 Script executed:
# Search for usage of openaiAudio and openaiRealtime in the codebase
rg -i "openaiAudio|openaiRealtime" --type ts --type js -B 2 -A 2Repository: tegnike/aituber-kit
Length of output: 13165
🏁 Script executed:
# Check the aiModels.ts file context around lines 154-155
cat -n src/features/constants/aiModels.ts | head -170 | tail -25Repository: tegnike/aituber-kit
Length of output: 977
openaiAudioのデフォルト値が使用APIと不一致です。
openaiAudioはopenAIAudioChat.ts内でopenai.chat.completions.create()と共に使用されていますが、デフォルトがtts-1に設定されています。しかしtts-1はテキスト読み上げ専用モデルであり、OpenAIの/v1/audio/speechエンドポイント専用です。Chat Completions APIでは使用できません。
gpt-realtimeは有効な識別子ですが（OpenAI 2026年1月現在）、openaiAudioのデフォルトはgpt-4o-audioなどのチャット対応モデルに修正が必要です。
🤖 Prompt for AI Agents
In `@src/features/constants/aiModels.ts` around lines 154 - 155, The constant
openaiAudio currently defaults to the speech-only model 'tts-1' which is
incompatible with the chat completions call in openAIAudioChat.ts (used with
openai.chat.completions.create()); update the default value of openaiAudio in
aiModels.ts to a chat-capable audio model (for example 'gpt-4o-audio' or another
chat-compatible audio model), so openAIAudioChat.ts uses a model supported by
the Chat Completions API.
- createAIRegistry関数を追加し、リクエストごとにProvider Registryを作成 - aiServiceConfigを廃止し、createProviderRegistryを使用 - getLanguageModel関数でregistry.languageModel()を使用してモデルを取得 - streamAiText/generateAiTextの引数をregistry + service形式に変更 - テストを新しいAPIに対応 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
テストで期待されていたzh→zh-TWのマッピングが 実装に欠けていたため追加 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
refactor: AI SDKのProvider Registry方式に移行
Contributor
Contributor
|
Claude finished @tegnike's task —— View job PR Review - Comprehensive AnalysisI've conducted a thorough review of this substantial develop → main PR with 100 changed files. Here's my comprehensive feedback: Review Checklist
🎯 Overall Assessment: Strong Implementation with Minor ConcernsThis is an impressive feature-rich release that adds significant value to AITuberKit. The codebase shows excellent engineering practices, comprehensive testing, and thoughtful architecture. ✅ Strengths1. Memory System (RAG-based) - Excellent Implementation
2. PNGTuber Engine - Production-Ready
3. Test Coverage - Exceptional
4. Configuration & Tooling
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
主要な変更
1. 技術スタックのメジャーアップグレード
2. 新機能: PNGTuber機能
3. 新機能: メモリ機能（RAGベース長期記憶）
4. AI SDK Provider Registry方式への移行
5. 新AIモデル追加
6. 中国語ロケールの分離
7. Claude Code / Kiro設定追加
8. 音声認識の改善
9. テスト大幅追加