fix(excel): Fix freeze_panes scroll position bug and update config defaults

## Bug Fix
- Replace _parse_freeze_panes with _get_frozen_panes to use pane.ySplit/xSplit
- Previously used sheet.freeze_panes (=pane.topLeftCell) which reflects scroll position
- This caused incorrect frozen_rows detection when Excel file was saved after scrolling
- Add _format_freeze_panes helper to generate freeze_panes string representation

## Tests
- Add test for scrolled position scenario (topLeftCell changed but ySplit unchanged)
- Add test for split pane (state="split") to verify it's correctly ignored
- All 32 tests pass

## Configuration Updates
- Add SHAREPOINT_EXCEL_MAX_FROZEN_ROWS to .env.example (default: 100)
- Add SHAREPOINT_EXCEL_MAX_DATA_ROWS to .env.example (default: 10000)
- Reduce SHAREPOINT_EXCEL_MAX_DATA_COLS default from 10000 to 2000
- Update setup.md and setup_ja.md with Excel processing limits documentation

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

Author: k-ibaraki

.env.example

@@ -88,6 +88,22 @@ SHAREPOINT_DEFAULT_MAX_RESULTS=20
 # Allowed file extensions / 許可するファイル拡張子
 SHAREPOINT_ALLOWED_FILE_EXTENSIONS=pdf,docx,xlsx,pptx,txt,md
 
+# ============================================================================
+# Excel Processing Limits (Optional) / Excel処理制限（オプション）
+# ============================================================================
+
+# Maximum frozen rows (header rows limit for DoS protection)
+# 最大固定行数（DoS対策のヘッダー行上限）
+# SHAREPOINT_EXCEL_MAX_FROZEN_ROWS=100
+
+# Maximum data rows (prevent processing extremely large Excel files)
+# 最大データ行数（巨大なExcelファイル処理を防止）
+# SHAREPOINT_EXCEL_MAX_DATA_ROWS=10000
+
+# Maximum data columns (prevent processing extremely wide Excel files)
+# 最大データ列数（極端に幅広いExcelファイル処理を防止）
+# SHAREPOINT_EXCEL_MAX_DATA_COLS=2000
+
 # ============================================================================
 # Tool Descriptions (Optional) / ツール説明文（オプション）
 # ============================================================================

docs/setup.md

@@ -46,6 +46,11 @@ SHAREPOINT_SITE_NAME=yoursite
 SHAREPOINT_DEFAULT_MAX_RESULTS=20
 SHAREPOINT_ALLOWED_FILE_EXTENSIONS=pdf,docx,xlsx,pptx,txt,md
 
+# Excel processing limits (optional)
+# SHAREPOINT_EXCEL_MAX_FROZEN_ROWS=100       # Maximum frozen rows (DoS protection)
+# SHAREPOINT_EXCEL_MAX_DATA_ROWS=10000       # Maximum data rows
+# SHAREPOINT_EXCEL_MAX_DATA_COLS=2000        # Maximum data columns
+
 # Tool description customization (optional)
 # SHAREPOINT_SEARCH_TOOL_DESCRIPTION=Search internal documents
 # SHAREPOINT_DOWNLOAD_TOOL_DESCRIPTION=Download files from search results

docs/setup_ja.md

@@ -46,6 +46,11 @@ SHAREPOINT_SITE_NAME=yoursite
 SHAREPOINT_DEFAULT_MAX_RESULTS=20
 SHAREPOINT_ALLOWED_FILE_EXTENSIONS=pdf,docx,xlsx,pptx,txt,md
 
+# Excel処理制限（オプション）
+# SHAREPOINT_EXCEL_MAX_FROZEN_ROWS=100       # 最大固定行数（DoS対策）
+# SHAREPOINT_EXCEL_MAX_DATA_ROWS=10000       # 最大データ行数
+# SHAREPOINT_EXCEL_MAX_DATA_COLS=2000        # 最大データ列数
+
 # ツール説明文のカスタマイズ（オプション）
 # SHAREPOINT_SEARCH_TOOL_DESCRIPTION=社内文書を検索します
 # SHAREPOINT_DOWNLOAD_TOOL_DESCRIPTION=検索結果からファイルをダウンロードします

src/config.py

@@ -63,7 +63,7 @@ def __init__(self):
             os.getenv("SHAREPOINT_EXCEL_MAX_DATA_ROWS", "10000")
         )
         self.excel_max_data_cols = int(
-            os.getenv("SHAREPOINT_EXCEL_MAX_DATA_COLS", "10000")
+            os.getenv("SHAREPOINT_EXCEL_MAX_DATA_COLS", "2000")
         )
 
         # ツール説明文のカスタマイズ

src/sharepoint_excel.py

@@ -193,7 +193,7 @@ def _parse_sheet(
         frozen_rows = 0
         frozen_cols = 0
         if include_header:
-            frozen_rows, frozen_cols = self._parse_freeze_panes(sheet.freeze_panes)
+            frozen_rows, frozen_cols = self._get_frozen_panes(sheet)
 
             # frozen_rows検証（DoS対策）
             if frozen_rows > config.excel_max_frozen_rows:
@@ -202,8 +202,10 @@ def _parse_sheet(
                     f"悪意のあるファイルまたは処理には大きすぎるファイルです。"
                 )
 
-            if sheet.freeze_panes:
-                sheet_data["freeze_panes"] = sheet.freeze_panes
+            if frozen_rows > 0 or frozen_cols > 0:
+                sheet_data["freeze_panes"] = self._format_freeze_panes(
+                    frozen_rows, frozen_cols
+                )
             sheet_data["frozen_rows"] = frozen_rows
             sheet_data["frozen_cols"] = frozen_cols
 
@@ -297,7 +299,9 @@ def _parse_sheet(
 
             if rows_to_process:
                 all_rows.extend(
-                    self._parse_rows(rows_to_process, include_formatting, merged_cell_map)
+                    self._parse_rows(
+                        rows_to_process, include_formatting, merged_cell_map
+                    )
                 )
 
         # レスポンス形式の分岐
@@ -481,38 +485,46 @@ def _calculate_range_size(self, range_str: str) -> tuple[int, int]:
             logger.warning(f"Failed to calculate range size '{range_str}': {e}")
             return (0, 0)
 
-    def _parse_freeze_panes(self, freeze_panes: str | None) -> tuple[int, int]:
+    def _get_frozen_panes(self, sheet) -> tuple[int, int]:
         """
-        freeze_panes文字列を解析して固定行数・列数を返す
+        シートのpane情報から固定行数・列数を返す（ySplit/xSplit使用）
+
+        sheet.freeze_panes（= pane.topLeftCell）はスクロール位置に依存するため、
+        正確な固定行数・列数を得るには pane.ySplit / pane.xSplit を直接参照する。
 
         Args:
-            freeze_panes: freeze_panes文字列（例: "B2", "A2", "B1", None）
+            sheet: openpyxl Worksheet
 
         Returns:
             (frozen_rows, frozen_cols)のタプル
-            例: "B2" → (1, 1)（行1と列Aが固定）
-                "A2" → (1, 0)（行1のみ固定）
-                "B1" → (0, 1)（列Aのみ固定）
-                None → (0, 0)（固定なし）
         """
-        if not freeze_panes:
-            return (0, 0)
-
         try:
-            # "B2" → ("B", 2)
-            col_letter, row = coordinate_from_string(freeze_panes)
-            # "B" → 2
-            col_index = column_index_from_string(col_letter)
-
-            # freeze_panes="B2"の場合、行2より前（行1）と列B（2列目）より前（列A）が固定
-            frozen_rows = row - 1
-            frozen_cols = col_index - 1
-
+            pane = sheet.sheet_view.pane
+            if pane is None:
+                return (0, 0)
+            if pane.state not in ("frozen", "frozenSplit"):
+                return (0, 0)
+            frozen_rows = int(pane.ySplit) if pane.ySplit else 0
+            frozen_cols = int(pane.xSplit) if pane.xSplit else 0
             return (frozen_rows, frozen_cols)
         except Exception as e:
-            logger.warning(f"Failed to parse freeze_panes '{freeze_panes}': {e}")
+            logger.warning(f"Failed to get frozen panes info: {e}")
             return (0, 0)
 
+    def _format_freeze_panes(self, frozen_rows: int, frozen_cols: int) -> str:
+        """
+        固定行数・列数からfreeze_panes文字列表現を生成
+
+        Args:
+            frozen_rows: 固定行数
+            frozen_cols: 固定列数
+
+        Returns:
+            freeze_panes文字列表現（例: "B4"）
+        """
+        col_letter = get_column_letter(frozen_cols + 1)
+        return f"{col_letter}{frozen_rows + 1}"
+
     def _expand_range_with_headers(
         self, cell_range: str, frozen_rows: int
     ) -> tuple[str | None, str]:

tests/test_sharepoint_excel.py

@@ -732,3 +732,79 @@ def test_parse_with_metadata_only_and_cell_range(self):
 
         # データ行は空リスト（metadata_onlyのため）
         assert sheet["data_rows"] == []
+
+    def test_freeze_panes_scrolled_position_does_not_affect_frozen_rows(self):
+        """スクロール後に保存されたファイルでfrozen_rowsが正しく取得されるテスト
+
+        Excel上で3行固定して行450付近にスクロールして保存すると、
+        pane.topLeftCell="A450"になるが、pane.ySplit=3は不変。
+        旧実装ではsheet.freeze_panes（=topLeftCell）を解析していたため
+        frozen_rows=449と誤判定していたバグを検証する。
+        """
+        wb = Workbook()
+        ws = wb.active
+        ws.title = "ScrolledSheet"
+
+        # ヘッダー3行 + データ行
+        for row in range(1, 11):
+            ws.cell(row=row, column=1, value=f"Row{row}")
+
+        # 3行固定を設定
+        ws.freeze_panes = "A4"
+
+        # スクロール位置を変更（pane.topLeftCellを直接操作）
+        pane = ws.sheet_view.pane
+        pane.topLeftCell = "A450"
+
+        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)
+        result_json = parser.parse_to_json("/test/scrolled.xlsx", include_header=True)
+
+        result = json.loads(result_json)
+        sheet = result["sheets"][0]
+
+        # pane.ySplit=3なので、frozen_rowsは3であるべき（449ではない）
+        assert sheet["frozen_rows"] == 3
+        assert sheet["frozen_cols"] == 0
+        assert sheet["freeze_panes"] == "A4"
+
+        # ヘッダー行は3行
+        assert len(sheet["header_rows"]) == 3
+        assert sheet["header_rows"][0][0]["value"] == "Row1"
+        assert sheet["header_rows"][2][0]["value"] == "Row3"
+
+    def test_split_pane_is_ignored(self):
+        """split pane（state="split"）は固定行として認識されないテスト"""
+        wb = Workbook()
+        ws = wb.active
+        ws.title = "SplitSheet"
+
+        ws["A1"] = "Header"
+        ws["A2"] = "Data"
+
+        # split paneを設定（frozenではなくsplit）
+        from openpyxl.worksheet.views import Pane
+
+        ws.sheet_view.pane = Pane(ySplit=3, xSplit=0, state="split")
+
+        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)
+        result_json = parser.parse_to_json("/test/split.xlsx", include_header=True)
+
+        result = json.loads(result_json)
+        sheet = result["sheets"][0]
+
+        # split paneは固定行として認識されない
+        assert sheet["frozen_rows"] == 0
+        assert sheet["frozen_cols"] == 0
+        assert "freeze_panes" not in sheet