Merge pull request #52 from shin902/translate-comments-to-japanese

docs: ソースコードの英語コメントを日本語に翻訳

Author: shin902

src/channels/discord.test.ts

@@ -1,20 +1,20 @@
 import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest';
 
-// --- Mocks ---
+// --- モック ---
 
-// Mock registry (registerChannel runs at import time)
+// レジストリのモック（registerChannel はインポート時に実行される）
 vi.mock('./registry.js', () => ({ registerChannel: vi.fn() }));
 
-// Mock env reader (used by the factory, not needed in unit tests)
+// 環境変数リーダーのモック（ファクトリで使用されるが、ユニットテストでは不要）
 vi.mock('../env.js', () => ({ readEnvFile: vi.fn(() => ({})) }));
 
-// Mock config
+// 設定のモック
 vi.mock('../config.js', () => ({
   ASSISTANT_NAME: 'Andy',
   TRIGGER_PATTERN: /^@Andy\b/i,
 }));
 
-// Mock logger
+// ロガーのモック
 vi.mock('../logger.js', () => ({
   logger: {
     debug: vi.fn(),
@@ -24,7 +24,7 @@ vi.mock('../logger.js', () => ({
   },
 }));
 
-// --- discord.js mock ---
+// --- discord.js のモック ---
 
 type Handler = (...args: any[]) => any;
 
@@ -76,7 +76,7 @@ vi.mock('discord.js', () => {
 
     async login(_token: string) {
       this._ready = true;
-      // Fire the ready event
+      // ready イベントを発火させる
       const readyHandlers = this.eventHandlers.get('ready') || [];
       for (const h of readyHandlers) {
         h({ user: this.user });
@@ -99,10 +99,10 @@ vi.mock('discord.js', () => {
     }
   }
 
-  // Mock TextChannel type
+  // TextChannel 型のモック
   class TextChannel {}
 
-  // Mock ThreadChannel type
+  // ThreadChannel 型のモック
   class ThreadChannel {}
 
   return {
@@ -117,7 +117,7 @@ vi.mock('discord.js', () => {
 
 import { DiscordChannel, DiscordChannelOpts } from './discord.js';
 
-// --- Test helpers ---
+// --- テスト用ヘルパー ---
 
 function createTestOpts(
   overrides?: Partial<DiscordChannelOpts>,
@@ -159,7 +159,7 @@ function createMessage(overrides: {
 }) {
   const channelId = overrides.channelId ?? '1234567890123456';
   const authorId = overrides.authorId ?? '55512345';
-  const botId = '999888777'; // matches mock client user id
+  const botId = '999888777'; // モッククライアントのユーザーIDと一致
   const isThreadChannel = overrides.isThread ?? false;
 
   const mentionsMap = new Map();
@@ -216,7 +216,7 @@ async function triggerMessage(message: any) {
   for (const h of handlers) await h(message);
 }
 
-// --- Tests ---
+// --- テスト ---
 
 describe('DiscordChannel', () => {
   beforeEach(() => {
@@ -227,7 +227,7 @@ describe('DiscordChannel', () => {
     vi.restoreAllMocks();
   });
 
-  // --- Connection lifecycle ---
+  // --- 接続ライフサイクル ---
 
   describe('connection lifecycle', () => {
     it('resolves connect() when client is ready', async () => {
@@ -269,7 +269,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- Text message handling ---
+  // --- テキストメッセージ処理 ---
 
   describe('text message handling', () => {
     it('delivers message for registered channel', async () => {
@@ -436,7 +436,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- @mention translation ---
+  // --- @メンション変換 ---
 
   describe('@mention translation', () => {
     it('translates <@botId> mention to trigger format', async () => {
@@ -471,8 +471,8 @@ describe('DiscordChannel', () => {
       });
       await triggerMessage(msg);
 
-      // Should NOT prepend @Andy — already starts with trigger
-      // But the <@botId> should still be stripped
+      // @Andy を先頭に付加しない — すでにトリガーで始まっている
+      // ただし <@botId> は除去される
       expect(opts.onMessage).toHaveBeenCalledWith(
         'dc:1234567890123456',
         expect.objectContaining({
@@ -521,7 +521,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- Attachments ---
+  // --- 添付ファイル ---
 
   describe('attachments', () => {
     it('stores image attachment with placeholder', async () => {
@@ -641,7 +641,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- Reply context ---
+  // --- リプライ文脈 ---
 
   describe('reply context', () => {
     it('includes reply author in content', async () => {
@@ -665,7 +665,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- sendMessage ---
+  // --- メッセージ送信 ---
 
   describe('sendMessage', () => {
     it('sends message via channel', async () => {
@@ -701,7 +701,7 @@ describe('DiscordChannel', () => {
         new Error('Channel not found'),
       );
 
-      // Should not throw
+      // 例外を投げないこと
       await expect(
         channel.sendMessage('dc:1234567890123456', 'Will fail'),
       ).resolves.toBeUndefined();
@@ -711,10 +711,10 @@ describe('DiscordChannel', () => {
       const opts = createTestOpts();
       const channel = new DiscordChannel('test-token', new Set(), opts);
 
-      // Don't connect — client is null
+      // 接続しない — client が null
       await channel.sendMessage('dc:1234567890123456', 'No client');
 
-      // No error, no API call
+      // エラーも API 呼び出しもない
     });
 
     it('splits messages exceeding 2000 characters', async () => {
@@ -737,7 +737,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- createThread ---
+  // --- スレッド作成 ---
 
   describe('createThread', () => {
     it('creates message-linked thread when messageId is provided', async () => {
@@ -831,7 +831,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- ownsJid ---
+  // --- JID の所有判定 ---
 
   describe('ownsJid', () => {
     it('owns dc: JIDs', () => {
@@ -871,7 +871,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- setTyping ---
+  // --- タイピング表示 ---
 
   describe('setTyping', () => {
     it('sends typing indicator when isTyping is true', async () => {
@@ -897,22 +897,22 @@ describe('DiscordChannel', () => {
 
       await channel.setTyping('dc:1234567890123456', false);
 
-      // channels.fetch should NOT be called
+      // channels.fetch は呼ばれないこと
       expect(currentClient().channels.fetch).not.toHaveBeenCalled();
     });
 
     it('does nothing when client is not initialized', async () => {
       const opts = createTestOpts();
       const channel = new DiscordChannel('test-token', new Set(), opts);
 
-      // Don't connect
+      // 接続しない
       await channel.setTyping('dc:1234567890123456', true);
 
-      // No error
+      // エラーは出ない
     });
   });
 
-  // --- Channel properties ---
+  // --- チャンネルプロパティ ---
 
   describe('channel properties', () => {
     it('has name "discord"', () => {
@@ -925,7 +925,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- Allowed bot filtering ---
+  // --- 許可Botフィルタリング ---
 
   describe('allowed bot filtering', () => {
     const ALLOWED_BOT_ID = '111222333444555666';
@@ -1019,7 +1019,7 @@ describe('DiscordChannel', () => {
     });
   });
 
-  // --- Thread detection ---
+  // --- スレッド検出 ---
 
   describe('thread detection', () => {
     it('sets is_thread=true and place_type=public_thread for thread messages', async () => {
@@ -1123,7 +1123,7 @@ describe('DiscordChannel', () => {
       });
       await triggerMessage(msg);
 
-      // Should deliver even though thread itself is not registered yet
+      // スレッド自体はまだ登録されていないが配信されること
       expect(opts.onMessage).toHaveBeenCalledWith(
         `dc:${threadId}`,
         expect.objectContaining({
@@ -1143,7 +1143,7 @@ describe('DiscordChannel', () => {
             folder: 'test-server',
             trigger: '@Andy',
             added_at: '2024-01-01T00:00:00.000Z',
-            // no thread_defaults
+            // thread_defaults なし
           },
         })),
       });

src/channels/discord.ts

@@ -101,7 +101,7 @@ export class DiscordChannel implements Channel {
       const sender = message.author.id;
       const msgId = message.id;
 
-      // Determine chat name
+      // チャット名を決定する
       let chatName: string;
       if (message.guild) {
         if (isThread && message.channel.isThread()) {
@@ -116,16 +116,16 @@ export class DiscordChannel implements Channel {
         chatName = senderName;
       }
 
-      // Resolve parent JID for thread messages
+      // スレッドメッセージの親JIDを解決する
       let parentJid: string | undefined;
       if (isThread && message.channel.isThread() && message.channel.parentId) {
         parentJid = `dc:${message.channel.parentId}`;
       }
 
-      // Translate Discord @bot mentions into TRIGGER_PATTERN format.
-      // Discord mentions look like <@botUserId> — these won't match
-      // TRIGGER_PATTERN (e.g., ^@Andy\b), so we prepend the trigger
-      // when the bot is @mentioned.
+      // Discord の @bot メンションを TRIGGER_PATTERN 形式に変換する。
+      // Discord のメンションは <@botUserId> のような形式で、
+      // TRIGGER_PATTERN（例: ^@Andy\b）には一致しないため、
+      // bot が @mentioned されたときにトリガーを先頭に付加する。
       if (this.client?.user) {
         const botId = this.client.user.id;
         const isBotMentioned =
@@ -134,18 +134,18 @@ export class DiscordChannel implements Channel {
           content.includes(`<@!${botId}>`);
 
         if (isBotMentioned) {
-          // Strip the <@botId> mention to avoid visual clutter
+          // 視覚的なノイズを避けるため <@botId> メンションを除去する
           content = content
             .replace(new RegExp(`<@!?${botId}>`, 'g'), '')
             .trim();
-          // Prepend trigger if not already present
+          // トリガーがまだ先頭にない場合は付加する
           if (!TRIGGER_PATTERN.test(content)) {
             content = `@${ASSISTANT_NAME} ${content}`;
           }
         }
       }
 
-      // Handle attachments — store placeholders so the agent knows something was sent
+      // 添付ファイルを処理 — プレースホルダーを保存してエージェントに送信内容を伝える
       if (message.attachments.size > 0) {
         const attachmentDescriptions = [...message.attachments.values()].map(
           (att) => {
@@ -168,7 +168,7 @@ export class DiscordChannel implements Channel {
         }
       }
 
-      // Handle reply context — include who the user is replying to
+      // リプライ文脈を処理 — 誰への返信かを含める
       if (message.reference?.messageId) {
         try {
           const repliedTo = await message.channel.messages.fetch(
@@ -180,11 +180,11 @@ export class DiscordChannel implements Channel {
             repliedTo.author.username;
           content = `[Reply to ${replyAuthor}] ${content}`;
         } catch {
-          // Referenced message may have been deleted
+          // 参照元メッセージが削除されている可能性がある
         }
       }
 
-      // Store chat metadata for discovery
+      // 発見用にチャットメタデータを保存する
       const isGroup = message.guild !== null;
       this.opts.onChatMetadata(
         chatJid,
@@ -194,15 +194,15 @@ export class DiscordChannel implements Channel {
         isGroup,
       );
 
-      // Only deliver full message for registered groups.
-      // Exception: thread messages from channels whose parent has thread_defaults
-      // are allowed through so index.ts can auto-register the thread group.
+      // 登録済みグループのみ完全なメッセージを配信する。
+      // 例外: thread_defaults を持つ親チャンネルのスレッドメッセージは、
+      // index.ts がスレッドグループを自動登録できるよう通過させる。
       const group = this.opts.registeredGroups()[chatJid];
       if (!group) {
         if (parentJid) {
           const parentGroup = this.opts.registeredGroups()[parentJid];
           if (parentGroup?.thread_defaults) {
-            // Auto-registration candidate — fall through to deliver the message
+            // 自動登録候補 — メッセージ配信をそのまま続行する
           } else {
             logger.debug(
               { chatJid, chatName },
@@ -219,9 +219,9 @@ export class DiscordChannel implements Channel {
         }
       }
 
-      // Deliver message — startMessageLoop() will pick it up.
-      // InboundMessage extends NewMessage with Discord-specific metadata
-      // (place_type, actor_role, is_thread, parent_jid) that is callback-only and not persisted.
+      // メッセージを配信 — startMessageLoop() が取得する。
+      // InboundMessage は NewMessage を拡張し、Discord 固有のメタデータ
+      // (place_type, actor_role, is_thread, parent_jid) を含むが、これらはコールバック専用で永続化されない。
       const inbound: InboundMessage = {
         id: msgId,
         chat_jid: chatJid,
@@ -243,7 +243,7 @@ export class DiscordChannel implements Channel {
       );
     });
 
-    // Handle errors gracefully
+    // エラーを適切に処理する
     this.client.on(Events.Error, (err) => {
       logger.error({ err: err.message }, 'Discord client error');
     });
@@ -292,7 +292,7 @@ export class DiscordChannel implements Channel {
 
       const textChannel = channel as TextChannel;
 
-      // Discord has a 2000 character limit per message — split if needed
+      // Discord は1メッセージあたり2000文字の制限がある — 必要に応じて分割する
       const MAX_LENGTH = 2000;
       if (text.length <= MAX_LENGTH) {
         await textChannel.send(text);