Skip to content

Commit bea5940

Browse files
k-ibarakiclaude
andcommitted
style: address AI review feedback (import and docstring)
## 修正内容 ### 1. import文の整理(tests/test_sharepoint_excel.py) - `from unittest.mock import patch` をメソッド内からファイル先頭に移動 - PEP 8に準拠:`from unittest.mock import Mock, patch` に統合 ### 2. docstringの改善(src/sharepoint_excel.py) - `_build_merged_cell_cache`メソッドのdocstringを拡充 - `effective_cell_range`パラメータの説明を追加: - 正規化・拡張済みのセル範囲であることを明記 - 形式("A1:D10")と挙動(Noneの場合)を説明 - 返り値の説明を追加 ## 背景 PR #40 のAIレビュー指摘(Gemini, Copilot)に対応: - import文の位置がPEP 8に準拠していない - `effective_cell_range`パラメータの意味が不明確 これらは軽微な問題だが、コードの可読性と保守性を向上させるため対応。 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 8a4f699 commit bea5940

2 files changed

Lines changed: 9 additions & 3 deletions

File tree

src/sharepoint_excel.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -439,6 +439,14 @@ def _build_merged_cell_cache(
439439
マージセル情報をキャッシュして返す(パフォーマンス最適化)
440440
- 「今回返す予定の範囲」を先に確定し、その範囲と交差する結合だけを部分展開する
441441
- アンカー値は左上→無ければ結合範囲内の実在セルのみから最小(row,col)を選ぶ
442+
443+
Args:
444+
sheet: openpyxl Worksheet
445+
effective_cell_range: 正規化・拡張済みのセル範囲(例: "A1:D10")
446+
Noneの場合はsheet.dimensionsを使用
447+
448+
Returns:
449+
(merged_cell_map, merged_anchor_value_map, merged_ranges)のタプル
442450
"""
443451
merged_cell_map: dict[str, str] | None = None
444452
merged_anchor_value_map: dict[str, Any] | None = None

tests/test_sharepoint_excel.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import datetime
22
import json
33
from io import BytesIO
4-
from unittest.mock import Mock
4+
from unittest.mock import Mock, patch
55
from zipfile import BadZipFile
66

77
import pytest
@@ -849,8 +849,6 @@ def test_no_duplicate_range_normalization(self):
849849
課題3-2の対応:_parse_sheetと_build_merged_cell_cacheで
850850
重複していた計算が1回のみになったことを検証
851851
"""
852-
from unittest.mock import patch
853-
854852
# テスト用Excelを作成(結合セルあり)
855853
wb = Workbook()
856854
ws = wb.active

0 commit comments

Comments
 (0)