Skip to content

feat(#6315): add Git History server API (1/4) - #66

Open
Defmon3 wants to merge 1 commit into
Bil0000:feat/issues-pagefrom
Defmon3:stack/6315-series-server
Open

Defmon3 wants to merge 1 commit into
Bil0000:feat/issues-pagefrom
Defmon3:stack/6315-series-server

Conversation

@Defmon3

@Defmon3 Defmon3 commented Sep 18, 2026

Copy link
Copy Markdown

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 the server API needed to browse repository commits, refs, commit details, changed files and diffs. Results use bounded pages and expiring snapshots; Git commands retain the existing concurrency limit and diagnostics.

Why

This is the backend foundation for the History and Repository proposal invited in #6315. It is based directly on Bilal's feat/issues-page at 3033fdac0.

Review scope

This is part 1 of 4. Review this layer against feat/issues-page; the following layers are History client, Repository pane, and Issues integration. They are separate review units, not additional changes in this diff. The larger two layers remain substantial despite the split.

The four general fixes for path whitespace, reconnect paging, Windows caches and diagnostic configuration remain outside this series.

Verification

Six focused ref tests passed on the final rewritten stack. Scoped lint passed with an existing schema warning; formatting passed. On this exact server-layer commit, authenticated dev-server calls returned two non-overlapping 100-commit pages, each ref namespace, commit details, 15 changed files, and a selected file diff.

No new UI in this layer. The complete proposal is demonstrated in the later layers. Local web verification does not cover every provider, mobile, Electron shell or relay mode.

Replaces the combined proposal Bil0000/t3code#6, whose original images and discussion are preserved.

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

@Defmon3
Defmon3 force-pushed the stack/6315-series-server branch from ddf51f3 to 51ca862 Compare September 21, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant