Skip to content

Add tree-mode files to the review side panel#148

Merged
lugen4ro merged 4 commits into
mainfrom
codex/sidepanel-file-tree
Jun 11, 2026
Merged

Add tree-mode files to the review side panel#148
lugen4ro merged 4 commits into
mainfrom
codex/sidepanel-file-tree

Conversation

@lugen4ro

@lugen4ro lugen4ro commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add a "tree view" to the side panel.

Right now we have only one view which is a flat one listing all the files.

Testing

  • Added unit coverage for tree building, collapsing, aggregation, and flattened rendering.
  • Added sidepanel integration coverage for the flat default and tree-configured panel.
  • make all passed.
  • Confirmed as below that the command works as intended
CleanShot2026-06-10at18.46.11-converted.mp4

Copilot AI review requested due to automatic review settings June 10, 2026 09:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

サイドパネルの Files セクションに sidepanel.file_tree オプションを追加し、デフォルトのフラット表示は維持したまま、オプトインでディレクトリツリー表示に切り替えられるようにするPRです。レビュー時のファイル把握を、ディレクトリ単位で整理して行いやすくする目的の変更です。

Changes:

  • lua/fude/ui/sidepanel/tree.lua を追加し、変更ファイル一覧からツリー構造の構築・集計・フラット化(描画用)を実装
  • サイドパネルにツリー描画 (format_files_section_tree) とセッション内トグル(t / :FudeReviewToggleFileTree)を追加
  • README / doc/fude.txt / CLAUDE.md / config defaults / テストを更新し、新モードとコマンドを反映

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tests/fude/sidepanel_tree_spec.lua ツリー構築・連結ディレクトリの圧縮・集計・フラット化のユニットテスト追加
tests/fude/sidepanel_spec.lua format_files_section_tree の描画仕様テスト追加
tests/fude/sidepanel_integration_spec.lua デフォルトは flat / 設定時は tree を使う統合テスト追加
README.md FudeReviewPanelFudeReviewToggleFileTreesidepanel.file_tree 設定例を追記
plugin/fude.lua :FudeReviewToggleFileTree コマンド追加
lua/fude/ui/sidepanel/tree.lua ツリー構築・圧縮・集計・描画順フラット化の純粋ヘルパー追加
lua/fude/ui/sidepanel.lua ツリー描画・トグル・エントリ解決の拡張、t キーマップ追加
lua/fude/config.lua sidepanel.file_tree = "flat" を defaults に追加
doc/fude.txt t キーマップと :FudeReviewToggleFileTreesidepanel.file_tree をドキュメント化
CLAUDE.md サイドパネル/ツリーの構成・関数一覧・キーマップ説明を更新

Comment thread lua/fude/ui/sidepanel.lua
Comment thread lua/fude/ui/sidepanel/tree.lua Outdated
Comment thread lua/fude/ui/sidepanel/tree.lua
@lugen4ro lugen4ro marked this pull request as ready for review June 10, 2026 09:56
@lugen4ro lugen4ro requested review from flexphere and kyu08 as code owners June 10, 2026 09:56
Copilot AI review requested due to automatic review settings June 10, 2026 09:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Comment thread lua/fude/ui/sidepanel.lua
Copilot AI review requested due to automatic review settings June 10, 2026 10:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated no new comments.

@kyu08 kyu08 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!!!!!

@lugen4ro lugen4ro merged commit 36c08fe into main Jun 11, 2026
10 checks passed
@lugen4ro lugen4ro deleted the codex/sidepanel-file-tree branch June 11, 2026 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants