fix: address code review findings

- Revert unrelated DM typing suppression in telegram-adapter.ts (out of scope)
- Fix TypeScript errors in test mocks (missing UserFromGetMe and PrivateChat properties)
- Clarify test assertion comment for /clean handler wiring

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: fitz123

bot/src/__tests__/telegram-bot.test.ts

@@ -1301,7 +1301,7 @@ describe("command handler wiring", () => {
       message: {
         message_id: updateId,
         from: { id: testChatId, is_bot: false, first_name: "Test" },
-        chat: { id: testChatId, type: "private" as const },
+        chat: { id: testChatId, type: "private" as const, first_name: "Test" },
         date: Math.floor(Date.now() / 1000),
         text,
         entities: [{ offset: 0, length: text.length, type: "bot_command" as const }],
@@ -1324,6 +1324,8 @@ describe("command handler wiring", () => {
       supports_inline_queries: false,
       can_connect_to_business: false,
       has_main_web_app: false,
+      has_topics_enabled: false,
+      allows_users_to_create_topics: false,
     };
     return bot;
   }
@@ -1343,6 +1345,6 @@ describe("command handler wiring", () => {
 
     await bot.handleUpdate(makeCommandUpdate("clean", 2));
     assert.ok(mockSM.calls.includes("destroySession"), "/clean should call destroySession");
-    assert.ok(!mockSM.calls.includes("closeSession"), "/clean should NOT call closeSession directly");
+    assert.ok(!mockSM.calls.includes("closeSession"), "/clean handler calls destroySession, not closeSession directly");
   });
 });

bot/src/telegram-adapter.ts

@@ -76,7 +76,7 @@ export function createTelegramAdapter(
     },
 
     async sendTyping(): Promise<void> {
-      if (!chatId || isDm) return;
+      if (!chatId) return;
       await ctx.api.sendChatAction(
         chatId,
         "typing",