Skip to content

feat!: remove Jujutsu support#4

Merged
amix merged 1 commit into
mainfrom
amix/remove-jujutsu
May 16, 2026
Merged

feat!: remove Jujutsu support#4
amix merged 1 commit into
mainfrom
amix/remove-jujutsu

Conversation

@amix
Copy link
Copy Markdown
Owner

@amix amix commented May 16, 2026

Context

dunk is a personal Git diff tool. The jj backend doubled the VCS surface (loaders, branch-review, watch, config detection, error translation) for a path that isn't used. Dropping it keeps things simple.

What was changed

  • Deleted src/core/jj.ts and its tests; removed jj loaders, watch signatures, and the jj branch-review revset path.
  • Removed the vcs config key, VcsMode type, and jj workspace auto-detection. findRepoRoot keys off .git only.
  • dunk stash show no longer needs a jj guard; CLI help and the "not a repo" error drop jj wording.
  • Updated README (removed the Jujutsu section and vcs config row) and CHANGELOG.

Out of scope

  • Immutable [0.13.0] changelog entry left as-is; removal is noted under [Unreleased].

Refs

Verified: bun run typecheck, bun test (339 pass), bun run lint, bun run test:integration (21 pass), and a real TTY smoke of dunk diff/dunk show. Doistbot isn't installed locally.

🤖 Generated with Claude Code

dunk now targets Git only. Keeping the VCS layer to a single backend
removes the jj loaders, the `vcs` config key and auto-detection, the
jj branch-review revset path, and jj watch signatures — less surface
to maintain for a personal tool. jj users can run a colocated Git
checkout.

BREAKING CHANGE: the `vcs` config option is gone and jj workspaces are
no longer auto-detected.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@amix amix marked this pull request as ready for review May 16, 2026 16:20
@amix amix merged commit 14f426f into main May 16, 2026
2 checks passed
@amix amix mentioned this pull request May 16, 2026
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