fix: address AI review comments

- Clarify "Header detection" wording in tool description to specify frozen_rows=0 condition
- Fix test_include_frozen_rows_single_cell docstring to match actual behavior (header + specified cell)
- Add condition to skip header_detection warning when expand_axis_range=True
- Add test case for expand_axis_range=True (no warning expected)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: k-ibaraki

src/server.py

@@ -552,7 +552,8 @@ def register_tools():
                 "Response includes cell data in 'rows' (value and coordinate) and structural information "
                 "(sheet name, dimensions, frozen_rows, frozen_cols, freeze_panes when present, merged_ranges when merged cells exist). "
                 "Cell styles (include_cell_styles, default: false): background colors and sizes. Use only for color-coded data extraction. "
-                "Header detection: Cannot be auto-detected from frozen_rows. "
+                "Header detection: For sheets with frozen_rows > 0, headers are automatically included with include_frozen_rows=True (default). "
+                "For sheets with frozen_rows=0, headers are not automatically included and context may be unclear. "
                 "ALWAYS read exactly 5 rows for header check: 'A1:Z5' (NOT 'A1:Z50' or more). "
                 "Prefer 'query' search when possible to locate data first. "
                 "Workflow: 1) Search OR read 'A1:Z5' for header check, "

src/sharepoint_excel.py

@@ -453,8 +453,9 @@ def _parse_sheet(
         sheet_data["frozen_rows"] = frozen_rows
         sheet_data["frozen_cols"] = frozen_cols
 
-        # frozen_rows=0 かつ cell_range指定時の警告
-        if frozen_rows == 0 and cell_range:
+        # frozen_rows=0 かつ cell_range指定時、expand_axis_range=Falseの場合のみ警告
+        # expand_axis_range=Trueの場合は1行目/A列が含まれるため警告不要
+        if frozen_rows == 0 and cell_range and not expand_axis_range:
             sheet_data["header_detection"] = {
                 "status": "no_frozen_rows",
                 "frozen_rows": 0,

tests/test_sharepoint_excel.py

@@ -1200,7 +1200,7 @@ def test_include_frozen_rows_range_includes_headers(self):
         assert a2_count == 1
 
     def test_include_frozen_rows_single_cell(self):
-        """単一セル指定時にデフォルト（expand_axis_range=False）ではヘッダーのみ追加"""
+        """単一セル指定時にデフォルト（expand_axis_range=False）では軸拡張せずヘッダー + 指定セルが追加される"""
         # frozen_rows=2のExcelファイルを作成
         excel_bytes = self._create_frozen_rows_excel("A3")
         self.mock_download_client.download_file.return_value = excel_bytes
@@ -1406,6 +1406,38 @@ def test_header_detection_not_added_when_frozen_rows_exist(self):
         # header_detection は付与されない（frozen_rows > 0）
         assert "header_detection" not in sheet_data
 
+    def test_header_detection_not_added_when_expand_axis_range_true(self):
+        """expand_axis_range=True の場合は header_detection が付与されない"""
+        wb = Workbook()
+        ws = wb.active
+        ws.title = "TestSheet"
+        ws["A1"] = "Header1"
+        ws["B1"] = "Header2"
+        ws["A5"] = "Data1"
+        ws["B5"] = "Data2"
+
+        excel_bytes = BytesIO()
+        wb.save(excel_bytes)
+        excel_bytes.seek(0)
+
+        self.mock_download_client.download_file.return_value = excel_bytes.getvalue()
+
+        parser = SharePointExcelParser(self.mock_download_client)
+
+        # frozen_rows=0 だが expand_axis_range=True なので警告不要
+        result_json = parser.parse_to_json(
+            "test.xlsx", sheet_name="TestSheet", cell_range="A5:B5", expand_axis_range=True
+        )
+
+        result = json.loads(result_json)
+        sheet_data = result["sheets"][0]
+
+        # frozen_rows=0
+        assert sheet_data["frozen_rows"] == 0
+
+        # header_detection は付与されない（expand_axis_range=True で1行目が含まれる）
+        assert "header_detection" not in sheet_data
+
     def test_include_frozen_rows_with_merged_cells(self):
         """マージセルとの統合が正しく機能すること"""
         # frozen_rows=2、マージセル付きのExcelファイルを作成