refactor: Organize Excel helper classes into modular structure by k-ibaraki · Pull Request #62 · ncdcdev/sharepoint-docs-mcp

k-ibaraki · 2026-02-11T07:51:29Z

概要

SharePoint Excel パーサーの保守性向上のため、内部実装をモジュール化してフォルダ構造を整理しました。

変更内容

ファイル構造の整理

新規ディレクトリ:

src/excel/ - Excel処理ヘルパークラス
tests/excel/ - Excel関連の単体テスト

抽出されたヘルパークラス:

ExcelRangeCalculator (range_calculator.py) - セル範囲の計算・変換・検証
ExcelMergedCellHandler (merged_cell_handler.py) - マージセル情報の構築と管理
ExcelPaneManager (pane_manager.py) - 固定行列（freeze_panes）の処理
ExcelStyleExtractor (style_extractor.py) - セルスタイル情報の抽出

コード改善

ファイルサイズ削減: sharepoint_excel.py が 1,114行 → 680行（39%削減）
責務の明確化: 各クラスが単一の関心事を持つように分離
テスト容易性: 76個の新規単体テストを追加（高カバレッジ: 81-100%）
保守性向上: モジュール化により変更の影響範囲を局所化

機能不変の保証

✅ 公開APIは完全に互換性維持

sharepoint_excel MCPツールのパラメータ・戻り値は一切変更なし
全てのドキュメントと使用例がそのまま有効

✅ 全テスト合格

既存統合テスト: 66個 ✅
新規単体テスト: 76個 ✅
合計: 142テスト全てPASS

✅ 品質チェック合格

型チェック (ty) ✅
リント (ruff) ✅
フォーマット ✅

実装方針

YAGNI原則: 過度な抽象化を避け、明確な責務分離のみ実施
シンプルさ優先: 全て staticmethod で依存性注入不要
機能不変: コピー＆移動のみでロジック変更なし
段階的実装: 各フェーズで動作確認してコミット

構造改善の効果

Before

src/
├── sharepoint_excel.py  (1,114行 - 複数の関心事が混在)
tests/
├── test_sharepoint_excel.py (統合テストのみ)

After

src/
├── sharepoint_excel.py  (680行 - 主要ロジックに集中)
└── excel/
    ├── __init__.py
    ├── range_calculator.py
    ├── merged_cell_handler.py
    ├── pane_manager.py
    └── style_extractor.py

tests/
├── test_sharepoint_excel.py (統合テスト)
└── excel/
    ├── test_excel_range_calculator.py
    ├── test_excel_merged_cell_handler.py
    ├── test_excel_pane_manager.py
    └── test_excel_style_extractor.py

レビューポイント

✅ 機能的な変更がないこと（全テスト合格で確認済み）
✅ ファイル構造が論理的に整理されていること
✅ 新規単体テストで各ヘルパークラスが適切にテストされていること
✅ ドキュメントの更新が不要であること（公開API不変のため）

🤖 Generated with Claude Code

Move Excel helper classes to dedicated subdirectory for better organization: - ExcelRangeCalculator -> src/excel/range_calculator.py - ExcelMergedCellHandler -> src/excel/merged_cell_handler.py - ExcelPaneManager -> src/excel/pane_manager.py - ExcelStyleExtractor -> src/excel/style_extractor.py Add src/excel/__init__.py to export all helper classes. Update all imports in src/ and tests/ to use new module structure. Benefits: - Clearer separation of Excel-related utilities - Better project structure (src/excel/ for Excel helpers) - Maintains all existing functionality (142 tests pass) - Improved code organization without functional changes Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Move Excel-related test files to dedicated subdirectory: - test_excel_range_calculator.py -> tests/excel/ - test_excel_merged_cell_handler.py -> tests/excel/ - test_excel_pane_manager.py -> tests/excel/ - test_excel_style_extractor.py -> tests/excel/ Benefits: - Test structure mirrors source code structure (src/excel/ <-> tests/excel/) - Better organization and discoverability - All 142 tests continue to pass (76 unit + 66 integration) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

gemini-code-assist · 2026-02-11T07:51:50Z

Summary of Changes

Hello @k-ibaraki, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

このプルリクエストは、SharePoint Excelパーサーの内部実装をモジュール化することで、コードベースの保守性とテスト容易性を大幅に向上させることを目的としています。主要なExcel処理ロジックを独立したヘルパークラスに分離し、それぞれの責務を明確にしました。これにより、コードの可読性が向上し、将来的な機能追加や変更が容易になります。既存の公開APIには影響がなく、全てのテストがパスしているため、機能的な変更はありません。

Highlights

モジュール構造の整理: Excel処理に関連するヘルパークラスが、src/excel/ ディレクトリに新しく抽出され、モジュール化されました。
責務の明確化とコードサイズの削減: sharepoint_excel.py からマージセル処理、固定行列管理、範囲計算、スタイル抽出などのロジックが分離され、ファイルサイズが約39%削減されました。
テスト容易性の向上: 抽出された各ヘルパークラスに対して76個の新規単体テストが追加され、コードカバレッジが向上しました。
公開APIの互換性維持: リファクタリングは内部実装に限定され、既存の公開APIのパラメータや戻り値は一切変更されていません。

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog

src/excel/init.py
- Excelヘルパーモジュールを初期化し、新しいヘルパークラスをエクスポートするファイルを追加しました。
src/excel/merged_cell_handler.py
- マージセル情報の構築、キャッシュ、値の伝播を処理する新しいExcelMergedCellHandlerクラスを追加しました。
src/excel/pane_manager.py
- Excelの固定行列（freeze_panes）情報の取得、フォーマット、検証を処理する新しいExcelPaneManagerクラスを追加しました。
src/excel/range_calculator.py
- Excelのセル範囲の計算、変換、検証を処理する新しいExcelRangeCalculatorクラスを追加しました。
src/excel/style_extractor.py
- セルのスタイル情報（色、サイズ）の抽出と変換を処理する新しいExcelStyleExtractorクラスを追加しました。
src/sharepoint_excel.py
- Excel関連のユーティリティメソッドを削除し、新しく作成されたsrc/excelモジュールから対応するヘルパークラスをインポートするように変更しました。
- コードの行数を1,114行から680行に削減しました。
tests/excel/test_excel_merged_cell_handler.py
- ExcelMergedCellHandlerクラスの単体テストを追加しました。
tests/excel/test_excel_pane_manager.py
- ExcelPaneManagerクラスの単体テストを追加しました。
tests/excel/test_excel_range_calculator.py
- ExcelRangeCalculatorクラスの単体テストを追加しました。
tests/excel/test_excel_style_extractor.py
- ExcelStyleExtractorクラスの単体テストを追加しました。
tests/test_sharepoint_excel.py
- リファクタリングされたSharePointExcelParserに合わせて既存のテストを更新し、新しいヘルパークラスを使用するように変更しました。
uv.lock
- sharepoint-docs-mcpパッケージのバージョンを0.4.0から0.5.0に更新しました。

Activity

既存の統合テスト66個が全てパスしました。
新しく追加された単体テスト76個が全てパスしました。
型チェック (ty)、リンティング (ruff)、フォーマットチェックが全てパスしました。
このプルリクエストはClaude Codeによって生成されました。

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

素晴らしいリファクタリングです。巨大なクラスを責務の小さいヘルパークラス群にうまく分割できており、保守性とテスト性が大幅に向上していることが新しい単体テストからも伺えます。構造も分かりやすく、実装方針に沿った堅実な変更だと感じました。
いくつか、新しいヘルパークラスとテストに関して改善提案がありますので、ご確認ください。

Copilot

Pull request overview

SharePoint Excel パーサーの内部実装をヘルパークラスへ分割し、フォルダ構成も含めて整理したことで、責務分離とテスト容易性を大きく改善するPRです。

Changes:

SharePointExcelParser 内の範囲計算・マージセル・freeze_panes・スタイル抽出を src/excel/ 配下のモジュールへ分離
既存統合テストの一部をヘルパークラス参照に更新し、新規ユニットテスト（tests/excel/）を追加
パッケージバージョン更新に伴い uv.lock を更新

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
uv.lock	パッケージバージョン更新（0.4.0 → 0.5.0）を反映
src/sharepoint_excel.py	Excel処理の一部ロジックをヘルパーへ委譲し、責務を軽量化
src/excel/init.py	抽出したヘルパークラスの公開エントリポイントを追加
src/excel/range_calculator.py	セル範囲の計算・正規化・検証を集約
src/excel/merged_cell_handler.py	マージセル情報の部分展開・アンカー値決定を集約
src/excel/pane_manager.py	freeze_panes の取得・整形・上限検証を集約
src/excel/style_extractor.py	スタイル（色・サイズ）抽出を集約
tests/test_sharepoint_excel.py	既存統合テストの一部を新ヘルパー利用に更新
tests/excel/test_excel_range_calculator.py	`ExcelRangeCalculator` のユニットテスト追加
tests/excel/test_excel_merged_cell_handler.py	`ExcelMergedCellHandler` のユニットテスト追加
tests/excel/test_excel_pane_manager.py	`ExcelPaneManager` のユニットテスト追加
tests/excel/test_excel_style_extractor.py	`ExcelStyleExtractor` のユニットテスト追加

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Apply all suggested improvements from code review: 1. Remove duplicate dict initialization in merged_cell_handler.py - Remove redundant None initialization (lines 39-40) - Add type hints at actual initialization (lines 73-74) 2. Narrow exception handling in range_calculator.py - Change `except Exception:` to `except ValueError:` for better specificity 3. Restore backward compatibility in calculate_range_size - Add try-except block to return (0, 0) on error - Maintain original behavior for invalid ranges - Add logging for debugging 4. Add defensive check for negative frozen_rows - Prevent negative values in pane_manager.py - Return (True, 0) for negative input 5. Restore duplicate normalization check in tests - Add mock-based call count verification - Ensure ExcelRangeCalculator methods called only once - Verify no performance regression from refactoring All 220 tests pass. Quality checks pass. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Self-review findings and fixes: 1. Fix validate_frozen_rows docstring accuracy - Corrected is_valid description: only False when exceeds limit - Clarified validated_frozen_rows behavior for negative values - Implementation and documentation now match 2. Improve expand_axis_range documentation - Add concrete examples first for better understanding - Add Args and Returns sections - Replace abstract description with clear examples Quality metrics maintained: - All 220 tests pass - Coverage: 81-100% (helper classes) - Type check, lint, format: all pass Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

k-ibaraki and others added 3 commits February 11, 2026 16:32

Merge branch 'main' into refactor/organize-excel-helpers

6b3f2bc

Copilot AI review requested due to automatic review settings February 11, 2026 07:51

Copilot started reviewing on behalf of k-ibaraki February 11, 2026 07:51 View session

gemini-code-assist Bot reviewed Feb 11, 2026

View reviewed changes

Comment thread src/excel/merged_cell_handler.py Outdated

Comment thread src/excel/range_calculator.py Outdated

Comment thread tests/test_sharepoint_excel.py

Copilot AI reviewed Feb 11, 2026

View reviewed changes

k-ibaraki and others added 2 commits February 11, 2026 17:01

k-ibaraki merged commit 240ac28 into main Feb 11, 2026
1 check passed

k-ibaraki deleted the refactor/organize-excel-helpers branch February 11, 2026 08:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor: Organize Excel helper classes into modular structure#62

refactor: Organize Excel helper classes into modular structure#62
k-ibaraki merged 5 commits into
mainfrom
refactor/organize-excel-helpers

k-ibaraki commented Feb 11, 2026

Uh oh!

gemini-code-assist Bot commented Feb 11, 2026

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

k-ibaraki commented Feb 11, 2026

概要

変更内容

ファイル構造の整理

コード改善

機能不変の保証

実装方針

構造改善の効果

Before

After

関連Issue

レビューポイント

Uh oh!

gemini-code-assist Bot commented Feb 11, 2026

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants