Skip to content

feat(#6315): add Git History client (2/4) - #24

Open
Defmon3 wants to merge 2 commits into
stack/6315-series-serverfrom
stack/6315-series-history
Open

Defmon3 wants to merge 2 commits into
stack/6315-series-serverfrom
stack/6315-series-history

Conversation

@Defmon3

@Defmon3 Defmon3 commented Sep 18, 2026

Copy link
Copy Markdown
Owner

Series and review order

Part Review Base
1 Server API Bilal's feat/issues-page
2 History client Part 1
3 Repository pane Part 2
4 Issues integration Part 3

These are open review units for one proposal, stacked on pingdotgg/t3code#6315. Part 1 targets Bilal's fork; parts 2–4 target the preceding branch in our fork. The complete series is available at the final branch. The series is open for review; feedback on product direction and implementation is welcome.

Short demo: History paging, Pull Requests, Issues, and retained History state.

Prior discussion: the original proposal and Bilal's invitation to stack it. This records the invitation, not upstream acceptance of the design.

What changed

Adds a History surface for browsing a virtualized commit graph, branches and tags, commit details, changed files and diffs. Favourites use environment and repository identity.

Why

Users can inspect repository history alongside their coding thread without leaving the app.

Dependency and review

Part 2 of 4, based on stack/6315-series-server. Review only this PR's incremental diff. It is hosted in this fork so its base branch exists without requiring write access to Bilal's fork. It is not a standalone upstream merge request.

History is still a separate surface here. The next layer combines it with Pull Requests; the fourth integrates Issues.

Verification

This exact cumulative layer passed dev-server checks for 100-to-200 commit paging, commit details, rendered file diffs, ref filtering with an empty result, and favourite persistence after reload. No page exceptions were captured. Earlier focused History paging/ref checks passed 39 tests and the layer passed web typecheck; those checks were not repeated for the server-only date-field removal.

UI

Before: the earlier base UI offers the existing right-panel surfaces without History (archived baseline screenshot).
Before: existing right-panel surfaces

After: History list:
History list

Selected file diff:
Commit file diff

Known limitation: a favourite appears once its local-ref page is loaded; deleted branch names remain saved. Partial or filtered results do not prune favourites.

The old proposal and its images remain available as historical context.

Developed with Astra 6 in Codex/T3 Code and reviewed with Fable 5.1.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sorry @Defmon3, your pull request is larger than the review limit of 150,000 diff characters

@Defmon3
Defmon3 force-pushed the stack/6315-series-history branch from cd3ad6e to c021011 Compare September 21, 2026 22:35

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant