Skip to content

docs: McpHttpResponse の body 型誤記を修正 (#107)#114

Closed
hideyukiMORI wants to merge 2 commits into
mainfrom
fix/107-mcp-http-response-docs
Closed

docs: McpHttpResponse の body 型誤記を修正 (#107)#114
hideyukiMORI wants to merge 2 commits into
mainfrom
fix/107-mcp-http-response-docs

Conversation

@hideyukiMORI
Copy link
Copy Markdown
Owner

Summary

Closes #107

Test plan

  • docs/reference/framework-modules.md の McpHttpResponse セクションが実コード(src/nene2/mcp/http_client.py)と一致していることを目視確認

🤖 Generated with Claude Code

hideyukiMORI and others added 2 commits May 20, 2026 00:20
__all__ に含まれているがドキュメント化されていなかった公開 API を EN/JA 両版に追加:

nene2.http:
- PaginationQuery — PaginationQueryParser.parse() の戻り値型
- HealthCheckProtocol / HealthStatus — ヘルスチェック契約と結果型

nene2.database:
- DatabaseHealthCheck — HealthCheckProtocol 実装、DB接続確認
- DatabaseConnectionException — DB接続不能時の例外

nene2.mcp:
- McpHttpResponse — HttpxMcpClient の戻り値型(status_code / body / is_successful())
- McpHttpClientProtocol — カスタム MCP HTTP クライアントの構造的契約

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- body: dict | list → body: str(生のレスポンステキスト)に修正
- headers: dict[str, str] フィールドを追記
- request_id() -> str | None メソッドを追記
- McpHttpClientProtocol に has_authentication() を追記
- EN / JA 両ドキュメントに適用

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
hideyukiMORI pushed a commit that referenced this pull request May 19, 2026
2026-05-20 のコード品質評価で作成した PR #114#119 を一覧化。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hideyukiMORI
Copy link
Copy Markdown
Owner Author

同等の変更が手動コミットで main に取り込み済みのため Close します(PR #143 / #144 / #150 参照)。

hideyukiMORI added a commit that referenced this pull request May 20, 2026
古い PR #114-#139 のマージ待ち記述を削除し、現在のオープン PR と
フィールドトライアル進捗を反映した内容に更新。

Co-authored-by: hideyukiMORI <info.xion.cc@gmail.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@hideyukiMORI hideyukiMORI deleted the fix/107-mcp-http-response-docs branch May 20, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: McpHttpResponse の body 型が誤記(str ではなく dict|list と記載)

1 participant