Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: modem-dev/hunk
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: db58310
Choose a base ref
...
head repository: modem-dev/hunk
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 1cc2102
Choose a head ref
  • 17 commits
  • 29 files changed
  • 3 contributors

Commits on Mar 23, 2026

  1. docs: tighten README and simplify session docs (#74)

    * docs: tighten README and simplify session docs
    
    * docs: mention OpenTUI and Pierre
    
    * docs: restore quickstart usage headings
    
    * docs: clarify git-style review entrypoints
    
    * docs: tighten git workflow wording
    
    * docs: shorten git workflow intro
    
    * docs: tighten feature comparison table
    
    * docs: add diff-so-fancy to comparison table
    
    * docs: simplify comparison summary
    benvinegar authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    4b9a639 View commit details
    Browse the repository at this point in the history
  2. Remove logo

    benvinegar authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    a750e5e View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    0e4b9ee View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    378b8b9 View commit details
    Browse the repository at this point in the history
  5. docs: group README setup under Advanced (#82)

    * docs: group README setup under Advanced
    
    * docs: format Hunk skill prompt as block
    benvinegar authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    d4b80f4 View commit details
    Browse the repository at this point in the history
  6. perf: benchmark and optimize large split review streams (#76)

    * test: benchmark large split review streams
    
    * test: flush benchmark highlight work
    
    * Baseline note-enabled large split-stream scroll benchmark on PR branch before optimization; current 18-tick note-enabled scroll cost is 16406.23ms and still shows act warnings from deferred highlight work.\n\nResult: {"status":"keep","note_scroll_ticks_ms":16406.23}
    
    * Scope visible agent notes to the selected hunk and keep placeholder windowing active for the rest of the review stream; note-enabled 18-tick scroll drops from 16406.23ms to 647.48ms while the interaction test now follows notes as focus moves.\n\nResult: {"status":"keep","note_scroll_ticks_ms":647.48}
    
    * Count visible inline note rows in split-mode section metrics so placeholder windowing keeps accurate heights for the selected note-bearing file; note-enabled 18-tick scroll drops from 647.48ms to 231.25ms and the section-height helper now has note-row coverage.\n\nResult: {"status":"keep","note_scroll_ticks_ms":231.25}
    
    * Validation rerun after note-height-aware windowing confirms the win: note-enabled 18-tick scroll improves further to 197.32ms, though the benchmark still occasionally surfaces deferred Pierre highlight act warnings that should be cleaned up separately.\n\nResult: {"status":"keep","note_scroll_ticks_ms":197.32}
    
    * Baseline after restoring viewport-scoped visible notes rather than selected-hunk-only notes; under the broader visible-note policy the current 18-tick large-stream scroll cost is 2009.15ms.\n\nResult: {"status":"keep","note_scroll_ticks_ms":2009.15}
    
    * Cache note-aware section metrics by visible note set so viewport-scoped visible notes reuse placeholder height plans across scroll ticks; under the restored always-visible note policy, the 18-tick large-stream scroll cost drops from 2009.15ms to 301.54ms while viewport-visible notes stay on screen.\n\nResult: {"status":"keep","note_scroll_ticks_ms":301.54}
    
    * chore: remove autoresearch log
    benvinegar authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    2fc2308 View commit details
    Browse the repository at this point in the history
  7. fix: daemon autostart for prebuilt npm binaries (#84)

    * fix: daemon autostart for prebuilt npm binaries
    
    Bun-compiled single-file executables set process.execPath to the
    embedded Bun runtime, not the binary itself. When the daemon launcher
    tried to spawn `process.execPath mcp serve`, it was invoking bare Bun
    with invalid args instead of the prebuilt hunk binary.
    
    Fix: when argv[0] differs from execPath, use argv[0] as the command —
    that's the actual compiled binary the user invoked.
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    
    * style: format daemon launcher test
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    
    ---------
    
    Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
    benvinegar and claude authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    13fd1bb View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    05c2cc7 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    90d9bef View commit details
    Browse the repository at this point in the history
  10. docs: add lumen to README comparison (#85)

    * docs: add lumen to README comparison
    
    * docs: link comparison table headers
    benvinegar authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    e36f63c View commit details
    Browse the repository at this point in the history
  11. fix: detect $bunfs virtual paths for daemon autostart in Bun binaries (

    …#86)
    
    * fix: detect $bunfs virtual paths for daemon autostart in Bun binaries
    
    The previous fix (PR #84) assumed argv[0] was the real binary and
    execPath was the embedded Bun runtime, but it's the opposite: Bun
    compiled binaries report argv as ["bun", "/$bunfs/root/<name>", ...]
    with execPath pointing to the real binary on disk. The old code matched
    the $bunfs virtual path against SCRIPT_ENTRYPOINT_PATTERN (because it
    contains "/"), spawning `bun /$bunfs/root/hunk mcp serve` — a path
    that doesn't exist outside the binary.
    
    Fix: detect the /$bunfs/ prefix in argv[1] and use execPath directly.
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    
    * style: format daemon launcher test
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
    
    ---------
    
    Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
    benvinegar and claude authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    0047371 View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    2e28108 View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    cc8c69e View commit details
    Browse the repository at this point in the history
  14. fix: publish prerelease tags to npm beta (#87)

    * fix: publish prerelease tags to npm beta
    
    * fix: narrow prerelease tag detection
    benvinegar authored Mar 23, 2026
    Configuration menu
    Copy the full SHA
    ec6dd52 View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    1f06574 View commit details
    Browse the repository at this point in the history
  16. Configuration menu
    Copy the full SHA
    ba09e75 View commit details
    Browse the repository at this point in the history
  17. Configuration menu
    Copy the full SHA
    1cc2102 View commit details
    Browse the repository at this point in the history
Loading