Skip to content

fix(#128/#129/#130/#131/#132): 低優先度バグ・コード品質の一括修正#139

Closed
hideyukiMORI wants to merge 1 commit into
mainfrom
fix/128-132-low-priority-fixes
Closed

fix(#128/#129/#130/#131/#132): 低優先度バグ・コード品質の一括修正#139
hideyukiMORI wants to merge 1 commit into
mainfrom
fix/128-132-low-priority-fixes

Conversation

@hideyukiMORI
Copy link
Copy Markdown
Owner

@hideyukiMORI hideyukiMORI commented May 19, 2026

Summary

#128 — InMemoryCommentRepository ソート欠如

  • find_all_by_note() に sorted(..., key=lambda c: c.id) を追加
  • SqlAlchemyCommentRepository の ORDER BY id と挙動を一致させる

#129 — mcp.py の type: ignore に reason コメントがない

  • 15 件の # type: ignore[type-arg] 全てに # reason: mcp tool handler type stubs do not support generic dict を追加

#130 — app_env が未検証

  • @field_validator("app_env") で {"local", "test", "production"} に制限
  • タイポで本番ログ設定が変わるリスクを排除

#131 — CI Python matrix に 3.14 がない

  • ["3.12", "3.14"] に拡張（CLAUDE.md 記載の開発環境 3.14 と一致）

#132 — export_openapi.py の stale docstring

  • 削除済みの uv run export-openapi → 正しい uv run python src/scripts/export_openapi.py に修正

Test plan

  • 167 テスト全パス
  • mypy --strict パス
  • ruff check パス

Closes #128
Closes #129
Closes #130
Closes #131
Closes #132

🤖 Generated with Claude Code

@claude
fix(#128/#129/#130/#131/#132): 低優先度バグ・コード品質の一括修正
f89a41b 
- #128: InMemoryCommentRepository.find_all_by_note() に sorted() を追加し
  SQLAlchemy 実装と同じ id 昇順に揃える
- #129: mcp.py の 15 件の type: ignore[type-arg] に reason コメントを追加
- #130: app_env に @field_validator を追加し "local/test/production" に制限する
- #131: CI の Python matrix に 3.14 を追加（開発環境と一致させる）
- #132: export_openapi.py の docstring の削除済みコマンドを修正する

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hideyukiMORI hideyukiMORI mentioned this pull request May 19, 2026
Merged
3 tasks
hideyukiMORI added a commit that referenced this pull request May 19, 2026
@hideyukiMORI
Merge pull request #144 from hideyukiMORI/fix/139-low-priority-manual
8b619ef 
fix: low-priority fixes from review (PR #139 manual)
@hideyukiMORI
Copy link
Copy Markdown
Owner Author

hideyukiMORI commented May 19, 2026

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

@hideyukiMORI hideyukiMORI mentioned this pull request May 20, 2026
Merged
hideyukiMORI added a commit that referenced this pull request May 20, 2026
@hideyukiMORI @claude
docs: docs/todo/current.md を v1.8.33 現状に合わせて更新 (#429)
9da8ca4 
古い 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/128-132-low-priority-fixes branch May 20, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment