Skip to content

feat: Add edit commit description functionality to Commit tab - #152

Merged
Ziinc merged 3 commits into
mainfrom
claude/edit-commit-description-wxthqt
Aug 5, 2026
Merged

feat: Add edit commit description functionality to Commit tab#152
Ziinc merged 3 commits into
mainfrom
claude/edit-commit-description-wxthqt

Conversation

@Ziinc

@Ziinc Ziinc commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Adds a full end-to-end flow to edit an existing commit's description
from the Commits tab: a jj-lib rewrite in jj.rs (jj_describe /
jj_get_commit_description), core + Tauri command wrappers, frontend
API bindings, and an EditCommitDescriptionDialog wired into the
commit row's action bar next to Move/Delete commit.

claude added 2 commits August 4, 2026 11:12
Adds a full end-to-end flow to edit an existing commit's description
from the Commits tab: a jj-lib rewrite in jj.rs (jj_describe /
jj_get_commit_description), core + Tauri command wrappers, frontend
API bindings, and an EditCommitDescriptionDialog wired into the
commit row's action bar next to Move/Delete commit.
… add screenshot spec

Adds the two dispatch.rs match arms so the screenshot test harness can
exercise the new edit-description commands, plus a spec that drives the
full flow through the real UI (expand commit, edit description, save)
with before/after captures.
@Ziinc Ziinc changed the title Add edit commit description functionality to Commit tab feat: Add edit commit description functionality to Commit tab Aug 5, 2026
…tion

Covers updating a commit's description, round-tripping a full multiline
message (vs. list_commits' first-line-only field), rebasing descendants
after a describe, and rejecting invalid change-ids.
@Ziinc
Ziinc merged commit 61cf75d into main Aug 5, 2026
11 checks passed
@Ziinc
Ziinc deleted the claude/edit-commit-description-wxthqt branch August 5, 2026 02:58
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.

2 participants