Repository navigation
fix(hermes): read Stop's prompt as the input row's first ❯ - #1161
Merged
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly and concisely describes the main change: Hermes Stop now reads the prompt from the input row’s first ❯. |
| Description check | |
The description includes the required Change and Validation sections. It explains the failure scenario, resulting behavior, regression coverage, and the successful check result. |
| Docstring Coverage | |
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
devswha
added a commit
that referenced
this pull request
Oct 15, 2026
## Change Release 0.4.9 from main at 6e575a8 (v0.4.8 + 193 commits), plus one blocker fix found in pre-release QA. - **`fix(terminal)` (ededf39)**: an open terminal re-attaches by itself after herdr restarts or updates. Before, the pane kept "terminal ended" and dropped typed input until another pane was picked (also in v0.4.8). The renderer now keys its attachment on the pane's `terminal_id`, which herdr changes on restore, and a cached tab waits until it is shown again. Locked by `scripts/herdr-restart-regression.ts` (fails on main, passes here), added to the browser lane. - `package.json` and `herdr-plugin.toml`: 0.4.9. - CHANGELOG: `[Unreleased]` and 147 changelog.d fragments become `[0.4.9] - 2026-10-16`, with `by @login` for outside contributors (22 PRs) and #1164 by @aisensiy carried in #1169. The reattach entry has no PR link yet: it is this PR's. - `release-summaries.json`: 0.4.9 in en/ko/ja/zh (7 new, 7 improved, 8 fixed in each). - `REMOTE_BUNDLE_VERSION` 27 -> 28: the bundle's inputs changed since remote-v27. - `telemetry/deployed.json`: the Worker was redeployed (version 431d6e07) for the settings schema changes since v0.4.8 (#1120, #1123, #1135); before it, this build's install event got HTTP 400. ## Release notes (as installs will show them) ### New features - New workspace → Browse makes a new folder and starts in it. - Shortcuts record chords with their exact modifiers; New tab and splits can be bound. - Pin workspaces, worktrees and agents from a row's menu or a right-click. - A Jump to bottom button returns a terminal scrolled up into history to the live end. - Pick a skill anywhere in a message, not only at its start. - Connect to remote PCs from a Windows PC. - Mod+Shift+E puts the cursor in the pane's input. ### Improvements - Split view halves are independent, and both may show the same tab. - Phones start in direct typing; the key bar's keyboard key lights for the input line. - Claude's whole plan shows from its plan file, and its answer review reads correctly. - Send shows progress until the agent takes the message, and Stop always goes first. - Recent tabs stay ready, so switching back shows the chat at once. - Sign out moved into Settings behind a confirm; menus and settings were trimmed. - A workspace or tab reopens on the pane you last saw, also after a reload. ### Bug fixes - Open terminals reconnect by themselves after herdr restarts or updates. - With herdr 0.9.4 or later, a second device no longer types stray characters into a pane. - Typing Hangul from an iPad hardware keyboard composes syllables. - Opening an Android keyboard no longer resizes the shared terminal. - When the app's folder is replaced under it, it asks for a restart, not an endless reload. - Hermes: Stop ends a running turn, and Chat will not send over text typed in its input. - The sidebar no longer breaks on a worktree without a branch. - A click on a lone pane's top right corner reaches the program there. The full changelog is the [`[0.4.9]` section of CHANGELOG.md on this branch](https://github.com/devswha/herdr-web-ui/blob/chore/release-0.4.9/CHANGELOG.md) (too long for a PR body). ## Verification Scenarios ran on c5f297e. The two commits after it (a978847, 14387e5) change only CHANGELOG.md, from the gate review: links for #863, #864, #869, #870, #972 and #1161, #811 moved from 0.4.7 (where a merge put it) into 0.4.9, and entries no longer name the removed Files at once setting or Find. No runtime behaviour differs from what was verified. QA plan with scenarios and pass conditions: user flows install → first run → daily use → errors, on Linux, macOS, Windows and remote PCs. Each scenario ran twice: a first pass on main 6e575a8, then an independent re-run by separate agents on this head (c5f297e), checked artifact by artifact. Browser checks drove a real Chrome at 1440 px and 390 px (iPhone emulation), light and dark. Local herdr was 0.9.5 (CI pins 0.9.3), so this also covers the newer herdr users have. | ID | Flow | Where | Result on c5f297e | |---|---|---|---| | L1 | Fresh install: README plugin path and `install.sh` one-liner | Clean Ubuntu 24.04 (Docker) | PASS: both exit 0, health ok, startup hook after a herdr restart | | M1 | Plugin install, start, first terminal | MacBook, macOS 26.6, herdr 0.9.5 | PASS: installed in ~1 min, health ok, `echo` runs | | W1(a) | Fresh native install | windows-latest ("Real Windows install" check) | PASS (CI) | | W1(b) | Plugin install on an existing herdr 0.9.3, mirrored pane | Windows 11 PC | PASS; installer unit test PASS | | L2 | First run: workspace list, new workspace, type in a pane | Linux, herdr 0.9.5 | PASS | | L3 | Daily use: split halves on one tab, Browse → New folder, pin, record a shortcut, Sign out confirm, phone direct typing and input-line key | Linux | PASS 7/7 | | L4 | herdr down, server restart, typing while disconnected (kept as a draft), token-protected server refuses a fresh browser | Linux | PASS | | FIX1 | herdr stop + start with a pane open: re-attaches without re-pick or reload | Linux | PASS (screenshots below) | | R1 | Remote PC over SSH: bundle smoke, key + password logins, desktop and phone | Linux sshd, Debian container | PASS | | L5 | In-place update and PWA update | Linux | PASS for the product; `update-browser-qa.ts` needs a fixture body after a fold (#1178) | | C1 | Integration suite on herdr 0.9.5 + settings/split/shortcut/chat browser scripts | Linux | 521 pass / 0 fail; `chat-browser-qa.ts` script drift (#1178) | | T1 | 1,484/2,000-char one-line messages into real Claude Code | Linux; macOS | Linux PASS; **macOS input line: known issue #1174, see below** | CI on this head: Fast checks, Integration and browser, Native Windows install, Real Windows install, macOS session identity and the remote bundle builds all pass. Local `bun run check fast`: 3,825 unit tests pass; `bun scripts/release-guards.ts` passes. ### Known issue shipped: #1174 (not a regression) On macOS, a single-line message of about 1.5 KB sent through the terminal **input line** can reach Claude Code without its first 1,022 characters (5/5 in the re-run; the first pass kept it whole). It predates 0.4.8. Two fixes were built and rejected on runtime evidence: bracketed paste over `pane.send_text` corrupts plain shells, and sending through the attach stream lets a following Stop overtake the message. A safe fix needs a program-aware paste RPC from herdr or a shared delivery barrier. Direct typing, which phones start in from this release, and the desktop terminal delivered the same messages whole. Details in #1174. **Shipping with it is the maintainer's call at merge**: the QA plan counted it as a blocker, and no safe fix exists in this release. ### Follow-ups filed #1177 (Homebrew Bun hint), #1178 (three QA scripts outside CI drifted), #1179 (small UI findings). ### Screenshots (from the re-run on this head) **Reattach after a herdr restart (FIX1)** <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/FIX1/restarted-1440-dark.png" alt="pane re-attached after herdr restart, desktop" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/FIX1/restarted-390-light.png" alt="phone" width="240"> **First run (L2)** <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L2/echo-terminal-1440-dark.png" alt="first terminal" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L2/echo-terminal-390-light.png" alt="phone" width="240"> **Daily use (L3)** <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L3/split-03-typed-in-active-half-both-draw-1440-dark.png" alt="split halves on one tab" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L3/newfolder-03-created-directory-follows-1440-light.png" alt="Browse, new folder" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L3/pin-02-workspace-pinned-1440-dark.png" alt="pinned workspace" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L3/signout-02-confirm-dialog-1440-dark.png" alt="sign out confirm" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L3/phone-02-input-line-key-lit-390-dark.png" alt="phone input line key lit" width="240"> **Errors (L4)** <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L4/herdr-stopped-disconnected-1440-dark.png" alt="herdr stopped" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L4/server-restarted-reconnected-draft-offered-1440-dark.png" alt="draft offered after reconnect" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L4/server-killed-typed-draft-kept-390-light.png" alt="phone draft kept" width="240"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L4/unauth-gate-signin-1440-dark.png" alt="token gate" width="560"> **macOS (M1) and Windows (W1)** <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/M1/06-echo-1440-dark.png" alt="macOS" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/W1/echo-1440-dark.png" alt="Windows mirrored pane" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/W1/echo-390-light.png" alt="Windows, phone" width="240"> **Remote PC (R1), Claude long message on Linux (T1), update notes (L5)** <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/R1/machines-1440-dark.png" alt="remote PCs" width="560"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/R1/add-pc-390-light.png" alt="add PC, phone" width="240"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/T1/claude-1484-390-dark.png" alt="1,484 chars into Claude" width="240"> <img src="https://raw.githubusercontent.com/devswha/herdr-web-ui/9bc3d185945572068f3fe9746e646312a009618c/L5/notes-desktop.png" alt="release notes beside the update" width="560"> Screenshots live on the `qa/v0.4.9-evidence` branch (not for merging; delete it after this release). ## Release After this merges and its CI on `main` passes: push `remote-v28` on the merged commit, wait for the Remote PC bundles workflow, then run the Release workflow with `0.4.9`. None of that was done here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change
Follow-up to #1148, the same fix as #1149 applied to Stop.
hermesInterruptibletook the row's last❯for Hermes' prompt. A draft that itself contains❯and Hermes' busy hint (❯ note ❯ Ctrl+C to interrupt…) therefore read as a running turn, and Stop would send Ctrl+C. On an idle Hermes, that clears the user's draft, or exits Hermes when the input is empty. The prompt is now the row's first❯, and anything after it is input.Validation
hermesStop.test.tscase (a draft quoting the TUI hint, and one quoting the classic hint) fails onmainand passes with this change. The existing cases with real captured rows, both interfaces' busy and idle rows, still pass.bun run check fast: ok.