Skip to content

changes: review and revert past changes as pull requests - #175

Merged
epheo merged 1 commit into
mainfrom
feat/history-review-revert
Sep 8, 2026
Merged

changes: review and revert past changes as pull requests#175
epheo merged 1 commit into
mainfrom
feat/history-review-revert

Conversation

@epheo

@epheo epheo commented Sep 7, 2026

Copy link
Copy Markdown
Owner

The Review changes tab's History lane offered no way to see what a past change did, and its revert never appeared: every forge merge is a merge commit, which both the UI and the git layer refused.

  • History rows are named by the PR they merged (title, number, link parsed from the Forgejo merge subject) and attributed to the proposer, not the merger.
  • Selecting a row renders the commit as the same semantic items a staged draft shows (GET /api/projects/{project}/history/{hash}): VM field diffs, whole-manifest edits for other kinds, creates and deletes, manifest text.
  • Revert works on merge commits, against the base branch side, so it undoes the whole PR. It opens a PR titled Revert "<title>" (#N) whose body carries the diff against current main. The pane warns when later changes to the same files would be undone, and disables the action when main already matches the pre-change state.

Verified: go test ./..., svelte-check, prettier, 74 vitest, 47 Playwright (one new spec on the fixture backend).

History rows are named by the PR they merged; selecting one shows the
same field diff a staged item has. Revert now covers merge commits (the
whole PR) and opens a PR whose body carries the diff against current
main, warning when later changes to the same files are undone.
@epheo
epheo force-pushed the feat/history-review-revert branch from bd88bcc to 959d52c Compare September 8, 2026 12:39
@epheo
epheo merged commit 25d9b00 into main Sep 8, 2026
7 checks passed
@epheo
epheo deleted the feat/history-review-revert branch September 8, 2026 12:52
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