umputun / revdiff

90%

master: 90%

LAST BUILD ON BRANCH compare-flag

branch: compare-flag

CHANGE BRANCH

x

Reset

Sync Branches

• compare-flag
• add-collapsed-flag
• all-files-mode
• annotation-list-popup
• annotation-rowcache
• ci-shellcheck-yaml
• code-smells-cleanup
• codex-plugin
• collapsed-diff-mode
• commit-info-popup
• community-themes
• compact-diff-mode
• compact-improve
• custom-keybindings
• daulet/new_file
• daulet/word_highlight
• dependabot/go_modules/github.com/alecthomas/chroma/v2-2.24.1
• dependabot/go_modules/github.com/charmbracelet/x/ansi-0.11.7
• dependabot/go_modules/github.com/mattn/go-runewidth-0.0.23
• diff-search
• discard-quit
• docs/pi-package-review-fixes
• eager-commit-fetch
• exit-code-on-annotations
• external-editor-annotations
• feat/annotation-marker
• feat/cmux-support
• feat/history-skill-fallback
• feat/jj-support
• feat/launch-script-improvements
• feat/navigation-scrollbar
• feat/one-light-chroma-style
• feat/open-editor-keymap
• feat/opencode-config-docs
• feat/overlay-mouse-support
• feat/preload-annotations
• feat/review-info-overlay
• feat/reviewed-files-and-status-indicators
• feat/scroll-indicators
• feat/scrollbar-thumb
• feat/stdin-scratch-buffer
• feat/toggle-tree-pane
• feat/untracked-flag
• feat/wheel-scroll-viewport-only
• feat/word-diff-highlighting
• feat/word-diff-opt-in
• feat/zed-revdiff-task
• feature/annotation-navigation
• feature/binary-file-detection
• feature/layout-agnostic-keys
• feature/layout-agnostic-keys-v2
• feature/new-file-and-untracked-support
• feature/range-hunk-annotations
• fix-ctrl-du-half-page
• fix-file-annotation-bugs
• fix-initial-loading-flash
• fix-marketplace-name-in-docs
• fix-prefix-fg-on-highlighted-lines
• fix-tmux-title-compat
• fix-tree-wheel-single-step
• fix-wheel-scroll-coalesce
• fix-wrap-right-padding
• fix/124-page-cursor-relative-position
• fix/center-hunk-in-viewport
• fix/cmux-ghostty-detection
• fix/cmux-send-stdout-leak
• fix/diff-viewport-visual-height-scroll
• fix/ghostty-improvements
• fix/issue-203-zellij-popup-size
• fix/kitty-ssh-remote-launch
• fix/launcher-argument-quoting
• fix/only-dot-prefix
• fix/overlay-path-inheritance
• fix/pi-revdiff-review-trigger
• fix/staged-only-empty-diff
• fix/subdirectory-support
• fix/theme-persist-ini-section
• global-hunk-navigation
• hunk-keyword-expansion
• include-filter
• intraline-word-diff
• iterm2-osascript
• launcher-override-chain
• leader-chord-keybindings
• line-numbers
• markdown-toc-navigation
• master
• mercurial-support
• mouse-support
• move-code-to-app
• only-filter
• overlay-extraction
• refs/tags/v0.1.0
• refs/tags/v0.1.1
• refs/tags/v0.10.0
• refs/tags/v0.11.0
• refs/tags/v0.12.0
• refs/tags/v0.13.0
• refs/tags/v0.14.0
• refs/tags/v0.14.1
• refs/tags/v0.15.0
• refs/tags/v0.15.1
• refs/tags/v0.15.2
• refs/tags/v0.15.3
• refs/tags/v0.16.0
• refs/tags/v0.16.1
• refs/tags/v0.17.0
• refs/tags/v0.18.0
• refs/tags/v0.18.1
• refs/tags/v0.19.0
• refs/tags/v0.19.1
• refs/tags/v0.19.2
• refs/tags/v0.2.0
• refs/tags/v0.2.1
• refs/tags/v0.2.2
• refs/tags/v0.2.3
• refs/tags/v0.2.4
• refs/tags/v0.20.0
• refs/tags/v0.21.0
• refs/tags/v0.22.0
• refs/tags/v0.23.0
• refs/tags/v0.24.0
• refs/tags/v0.25.0
• refs/tags/v0.25.1
• refs/tags/v0.26.0
• refs/tags/v0.26.1
• refs/tags/v0.27.0
• refs/tags/v0.27.1
• refs/tags/v0.28.0
• refs/tags/v0.3.0
• refs/tags/v0.4.0
• refs/tags/v0.4.1
• refs/tags/v0.4.2
• refs/tags/v0.5.0
• refs/tags/v0.6.0
• refs/tags/v0.7.0
• refs/tags/v0.7.1
• refs/tags/v0.7.2
• refs/tags/v0.8.0
• refs/tags/v0.9.0
• refs/tags/v1.0.0
• refs/tags/v1.1.0
• refs/tags/v1.1.1
• refs/tags/v1.2.0
• refs/tags/v1.2.1
• refs/tags/v1.3.0
• refs/tags/v1.4.0
• refs/tags/v1.4.1
• reload
• revert/197-opencode-symlink
• revert/opencode-setup-symlink
• review-findings
• review-history
• search-history
• sidepane-extraction
• single-column-line-numbers-full-context
• single-file-mode
• skill-opencode-timeout-fix
• skill-scripts-hg-jj
• skill-use-claude-skill-dir
• status-bar-line-number
• status-line-help-overlay
• structural-refactor
• style-extraction
• theme-support
• two-ref-diff
• ui-package-split
• update-pi-extension-agent-tool
• vim-motion-preset
• widen-annot-list-popup
• word-wrap
• worddiff-extraction
• worktree-bugfix
• worktree-no-git-file-review
• wrap-marker-style

Build # 25236122332

Build Type

Pull #163

github

Committed by rashpile

Commit Message

fix: round-2 review findings — stale launcher safety + tighter test/prompt

- swap compare-mode arg order from (old, new) to (new, old) in both the
  bundled launcher's 2-arg branch and the planning hook so that stale
  user-layer overrides at ${CLAUDE_PLUGIN_DATA}/scripts/ — copies of
  master's 1-arg launcher made before compare mode shipped — silently
  degrade to --only review of the NEW revision instead of opening the
  OLD one the user already reviewed. revdiff itself still receives
  --compare-old=<OLD> --compare-new=<NEW>; the launcher relabels locally.
- tighten the deny-reason text so the agent is explicitly told to ignore
  any other plan-rev-*.md marker it may have seen earlier in the
  conversation, addressing intra-session chain-mixup at the prompt layer.
- replace assert.NotEmpty on opts.compareAbsOld/New with equality against
  filepath.Abs(input) so the test would actually catch a swapped-fields
  bug rather than just confirming the cache is populated.
- document the new launcher contract (new-revision first) and the
  graceful-degradation rationale in the planning plugin README.

Pull Request Pull Request #163: feat: `--compare=old:new` — two-file diff for rolling agent reviews

Coverage Stats

110 of 139 new or added lines in 7 files covered. (79.14%)

57 existing lines in 5 files now uncovered.

9624 of 10730 relevant lines covered (89.69%)

3210.54 hits per line