Problem

From verification of #3, #4:
「MCP 層的測試只驗 schema，沒有驗 handler 行為。ServerDumpChatToolTests.swift:71 只確認 get_chat_history 有六個欄位，沒有驗證 from_message_id == 0 時是否真的自動把 max_messages = limit，也沒有驗證 since_date / until_date / max_messages 是否真的傳進 TDLibClient」
— Sources: codex + team:regression

目前 get_chat_history 的 MCP 層只有 schema shape test，沒有 handler 行為 test。#3 的核心 fix（fromMsgId==0 時自動帶 maxMessages=limit）完全沒有 unit test 保護，只靠間接的 E2E test。

Type

enhancement

Expected

加入 unit tests 驗證 MCP handler 行為（至少不依賴真實 TDLib 連線）：

• fromMsgId == 0 且無 max_messages → TDLibClient.getChatHistory 被呼叫時帶 maxMessages == limit
• fromMsgId > 0 且無 max_messages → maxMessages == nil（backward-compat path）
• 傳入 since_date / until_date → 正確解析為 Date 並傳入
• 傳入 max_messages → 覆蓋上面的隱式默認

Actual

只有 testGetChatHistoryProperties 驗證 schema 有 6 個 properties。

Impact

• get_chat_history: from_message_id=0 首次呼叫只回傳 1 則訊息 #3 的 fix 如果未來被改壞（例如有人把 fromMsgId == 0 ? limit : nil 移除），沒有 unit test 會 fail
• 回歸保護不足

Technical Approach

可能需要 mock 或 fake TDLibClient 來驗證 handler 傳給 library 層的參數。參考 TDLibClientBackwardCompatTests 的 pattern（pure compilation + signature tests）。

Code Reference

• Handler: Sources/CheTelegramAllMCPCore/Server.swift:345-363
• Existing schema test: Tests/CheTelegramAllMCPTests/ServerDumpChatToolTests.swift:71-82

Related: #3, #4

Current Status

Phase: closed
Last updated: 2026-04-26 by idd-close

Key Decisions

• Pure-function extraction strategy validated by full 6-AI review
• 11 regression tests lock get_chat_history: from_message_id=0 首次呼叫只回傳 1 則訊息 #3/get_chat_history: MCP 層未暴露 sinceDate/untilDate/maxMessages 參數 #4/日期處理改善: until_date 語義偏差 + parseISODate 靜默 nil #5/max_messages 邊界防護: 加硬上限 + 0/負數處理 #6 behaviors
• Handler 35 → 16 lines (responsibility separation)
• 5 non-blocking findings → follow-up issues Test: assert MCP handler error message text (contract test) #11/Test: backward-compat numeric-string chat_id / from_message_id parsing #12/Refactor dump_chat_to_markdown handler to use parseGetChatHistoryArgs pattern #13/Test: handler glue integration (errorResult conversion + tdlib wiring) #14/Test hardening: int64 helper DRY + boundary + TZ assertions #15

Scope Changes

(none)

Follow-up

• Test: assert MCP handler error message text (contract test) #11 error text 斷言測試
• Test: backward-compat numeric-string chat_id / from_message_id parsing #12 numeric-string backward-compat 測試
• Refactor dump_chat_to_markdown handler to use parseGetChatHistoryArgs pattern #13 dump_chat_to_markdown 同步 refactor (HIGH)
• Test: handler glue integration (errorResult conversion + tdlib wiring) #14 handler glue integration tests
• Test hardening: int64 helper DRY + boundary + TZ assertions #15 int64Arg helper DRY + 10_001 boundary + TZ assertions

Commits

• f8a7d51 refactor: extract parseGetChatHistoryArgs pure function + handler tests (MCP 層補 handler 行為 unit test（get_chat_history） #7)