Skip to content

feat(code): tree view for changed files list#1494

Open
adboio wants to merge 1 commit into04-03-feat_code_use_remote_files_for_cloud_review_panelfrom
posthog-code/changes-tree-view
Open

feat(code): tree view for changed files list#1494
adboio wants to merge 1 commit into04-03-feat_code_use_remote_files_for_cloud_review_panelfrom
posthog-code/changes-tree-view

Conversation

@adboio
Copy link
Copy Markdown
Contributor

@adboio adboio commented Apr 3, 2026

Problem

changed files would be easier to parse as a tree

closes #1197

Changes

  • extract some stuff from the existing file tree view
  • make changed files render as a tree
chat view review panel
Screenshot 2026-04-03 at 1.21.38 PM.png Screenshot 2026-04-03 at 1.21.41 PM.png

How did you test this?

manually

Copy link
Copy Markdown
Contributor Author

adboio commented Apr 3, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@adboio adboio marked this pull request as ready for review April 3, 2026 20:22
@adboio adboio requested a review from a team April 3, 2026 20:22
Replace flat file list in ChangesPanel with a collapsible folder tree.
Directories with a single child are compacted (e.g. "src/utils").

Extract shared TreeDirectoryRow and TreeFileRow components used by both
the changes tree and the repo file tree (FileTreePanel), deduplicating
directory and file row rendering.

Generated-By: PostHog Code
Task-Id: 60c3af6f-6f10-4722-a2c4-7a2a15303ba1
@adboio adboio force-pushed the 04-03-feat_code_use_remote_files_for_cloud_review_panel branch from 488611e to 3398bc6 Compare April 3, 2026 21:59
@adboio adboio force-pushed the posthog-code/changes-tree-view branch from b1740de to 2d01726 Compare April 3, 2026 21:59
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.

1 participant