Fix Ctrl+Backspace not deleting word in terminal by mathnogueira · Pull Request #2 · stablyai/orca

mathnogueira · 2026-03-19T23:12:26Z

Summary

Intercept Ctrl+Backspace in the terminal pane keydown handler and send \x17 (backward-kill-word) to the PTY via sendKeyInput
Restty/xterm.js doesn't map this shortcut by default, so the key was being ignored or producing an invalid character

Test plan

Open a terminal pane and type a multi-word command
Press Ctrl+Backspace and verify the previous word is deleted
Verify regular Backspace still works as expected
Verify Ctrl+W still closes split panes (not affected)

🤖 Generated with Claude Code

Restty does not map Ctrl+Backspace to backward-kill-word by default. Intercept the key combination at the capture phase and send \x17 via sendKeyInput so it reaches the PTY process directly. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

…ent status) (#1678) * feat(agent-hooks): introduce relay wire envelope + connectionId stamping Adds the shared `agent-hook-relay.ts` module with the `agent.hook` JSON-RPC notification envelope, the `agent_hook.requestReplay` / `agent_hook.installPlugins` method names, and the `ORCA_FEATURE_REMOTE_AGENT_HOOKS` flag helper. Promotes `AgentHookSource` to `shared/` so the relay can import it without dragging Electron in. Threads a `connectionId: string | null` field through `AgentHookEventPayload`, the `agentStatus:set` IPC contract, and the renderer-bound preload listener. Local hook posts stamp `null`; the relay-forwarded path will stamp from `mux` identity in a later commit. Renderer uses the stamp for stale-event filtering when an SSH connection tears down with notifications still in flight. See docs/design/agent-status-over-ssh.md §1, §5, §8 (commit #1). Co-authored-by: Orca <help@stably.ai> * refactor(agent-hooks): extract shared listener; add relay-side adapter Extracts the listener internals (request parsing, payload normalization, endpoint-file writing, per-CLI extractors, warn-once Sets, slowloris timer helper, request size cap, paneKey caches) from `src/main/agent-hooks/server.ts` into a new transport-agnostic `src/shared/agent-hook-listener.ts`. The shared module uses only Node builtins (no Electron) so it is safe to import from `src/relay/`. Adds `src/relay/agent-hook-server.ts` — a thin HTTP-loopback adapter that wires the shared listener to a `forward(envelope)` callback so `relay.ts` can re-emit each parsed payload as an `agent.hook` JSON-RPC notification on the existing SshChannelMultiplexer. The adapter owns: - 127.0.0.1:0 socket + bearer-token auth, identical shape to the local server - per-paneKey last-payload cache + replayCachedPayloadsForPanes() for the request-driven replay path used after `--connect` reattach (see §5 Path 3) - clearPaneState(paneKey) for PTY-exit eviction (symmetric with local server) - buildPtyEnv() / endpoint-file writing for relay-spawned PTYs Orca's `AgentHookServer` is now a ~200-LoC adapter over the shared listener that owns the IPC fanout, listener replay, and `ingestRemote(envelope, connId)` entry point that bypasses the HTTP path for relay-forwarded events. See docs/design/agent-status-over-ssh.md §3, §8 (commit #2). Co-authored-by: Orca <help@stably.ai> * fix(preload): expose connectionId on agentStatus.onSet type src/preload/index.ts already passes through `connectionId?: string | null` from main, but the PreloadApi declaration in api-types.ts was missing the field. Align the type with the runtime contract so renderer call sites can read connectionId without an `as` cast. Co-authored-by: Orca <help@stably.ai> * fix(agent-hooks): harden ingestRemote + relay replay; review-driven cleanup - ingestRemote: re-run normalizeAgentStatusPayload at trust boundary; trim+validate connectionId/paneKey/tabId/worktreeId - relay: preserve source/env/version through replay via sidecar map; drop sourceFromAgentType fallback that mis-tagged unknown agents - shared listener: exhaustive switch+never on AgentHookSource dispatch chains; extractPromptText returns trimmed values; export MAX_PANE_KEY_LEN - preload: tighten connectionId from optional to required (always sent) - main IPC: reorder spread so explicit envelope fields win on collision Co-authored-by: Orca <help@stably.ai> * chore(docs): drop agent-status-over-ssh design doc from PR The design RFC was useful for authoring this PR series but doesn't belong in-tree — keeping it here would freeze line-number references and design prose against future churn. Folding it into the PR description instead. Co-authored-by: Orca <help@stably.ai> * chore(agent-hooks): widen ingestRemote type for env/version (PR2 prep) Declares `env?: string` and `version?: string` on the `ingestRemote` envelope parameter so PR2 only needs to add the `warnOnHookEnvOrVersionMismatch` callsite, not also widen the type. The fields are forwarded verbatim from the agent CLI POST body on the remote and let Orca's warn-once cross-build / dev-vs-prod diagnostics fire identically on remote-sourced events. Type-only addition; no runtime consumer in this PR. Co-authored-by: Orca <help@stably.ai> --------- Co-authored-by: Orca <help@stably.ai>

importFromSshConfig only inserted unknown hosts and never updated an existing target, so a rotated port (or any changed config field) was silently ignored on re-import. Make it an upsert: config-sourced targets are refreshed in place, genuinely new hosts are inserted, and the SSH Manage pane auto-syncs when it opens so changes appear without a manual Import click. A new optional SshTarget.source ('ssh-config' | 'manual') protects user-created and user-edited targets from being overwritten by sync — editing a target in the UI detaches it from config-sync. Unchanged targets are skipped so a repeat sync performs no disk write. Editing a target now also carries cleared optional fields (identity file, proxy command, jump host) so removing a config-derived value actually deletes it instead of silently keeping the stale value through the partial merge. Resolves item #1 of stablyai#4684 (item #2 was fixed in stablyai#4860). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* fix(ssh): sync ~/.ssh/config on import and Manage open importFromSshConfig only inserted unknown hosts and never updated an existing target, so a rotated port (or any changed config field) was silently ignored on re-import. Make it an upsert: config-sourced targets are refreshed in place, genuinely new hosts are inserted, and the SSH Manage pane auto-syncs when it opens so changes appear without a manual Import click. A new optional SshTarget.source ('ssh-config' | 'manual') protects user-created and user-edited targets from being overwritten by sync — editing a target in the UI detaches it from config-sync. Unchanged targets are skipped so a repeat sync performs no disk write. Editing a target now also carries cleared optional fields (identity file, proxy command, jump host) so removing a config-derived value actually deletes it instead of silently keeping the stale value through the partial merge. Resolves item #1 of #4684 (item #2 was fixed in #4860). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(ssh): preserve legacy manual targets during config sync Co-authored-by: Orca <help@stably.ai> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Co-authored-by: Jinwoo-H <jinwoo0825@gmail.com> Co-authored-by: Orca <help@stably.ai>

…-word

…2e; doc wire-format Adds unit guards that the floating sentinel resolves consistently on activate/close (no selector_not_found) and a Playwright e2e driving the sentinel through the real resolver. Augments the resolveWorktreeSelector comment to document that the client always sends the id: form via toRuntimeWorktreeSelector (no logic change). Per peer-review items #1 (doc) and #2. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

…w (4 fixes) NotebookLM peer review verdict: CHANGES-NEEDED. Applied the genuine findings (assessed against actual Electron/Node behavior; teardown was the only hypothesis NLM rated already-Established): - fix #7: also override XDG_RUNTIME_DIR (scratch, mode 0700) -- Chromium/ Electron IPC sockets live there (/run/user/UID, shared w/ prod) and are NOT covered by app.getPath('userData'). - fix #8: PRE-7b df -h /tmp gate -- --appimage-extract-and-run unpacks the full ~400-600MB squashfs into shared /tmp; a full /tmp could hang prod. - fix #9: correct Step 4 + acceptance #2 -- under the HOME override, homedir() resolves to the scratch home, so expect that cwd (avoids a false-negative wrong-cwd failure). - fix #10: documented prod-recovery pointer for Lesley (systemctl restart / orca-serve-version restore); STOP-first posture retained per the never-hotfix / deviation=STOP governance. Execution remains GATED on Lesley's approval. Notebook: cf8d676b-fb68-4f01-9388-922b83568759 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Remote-runtime and SSH worktrees are fetched/created/reparented through the owning host, which reports them from its own perspective (hostId defaults to "local"). After the per-worktree hostId override in stablyai#2, that bogus "local" overrode the repo's runtime owner, so terminals/sessions for a remote worktree resolved to the local machine instead of the remote (clicking an omarchy-office worktree opened a shell on the local omarchy-thinkpad). Re-stamp every runtime worktree payload with the repo's execution host via a shared withRepoHostId helper, applied at all worktreesByRepo ingress points: toVisibleWorktrees (fetch), createWorktree, and applyWorktreeLineageUpdate. Local-owned repos are left untouched, so an explicit local worktree still overrides a runtime repo owner. Mirrors how repos already get repoWithFetchedOwner.

PR stablyai#2 gated the global session-tab sync to web clients only, removing the desktop path that eagerly populated remote projects. With no on-connect repo fetch left, remote projects only appeared after the user opened the Add-Project dropdown (which calls fetchRuntimeEnvironmentRepos directly). Seed an initial repo/worktree/lineage refresh for every runtime environment that is already connected when useIpcEvents mounts, and for each one that becomes connected afterward. Reuses the debounced/throttled refresh scheduler, and works regardless of whether the remote server emits client events.

* Fix idle remote runtime session work * fix: stamp remote/SSH worktrees with their repo execution host Remote-runtime and SSH worktrees are fetched/created/reparented through the owning host, which reports them from its own perspective (hostId defaults to "local"). After the per-worktree hostId override in #2, that bogus "local" overrode the repo's runtime owner, so terminals/sessions for a remote worktree resolved to the local machine instead of the remote (clicking an omarchy-office worktree opened a shell on the local omarchy-thinkpad). Re-stamp every runtime worktree payload with the repo's execution host via a shared withRepoHostId helper, applied at all worktreesByRepo ingress points: toVisibleWorktrees (fetch), createWorktree, and applyWorktreeLineageUpdate. Local-owned repos are left untouched, so an explicit local worktree still overrides a runtime repo owner. Mirrors how repos already get repoWithFetchedOwner. * fix: auto-discover remote runtime projects on connect PR #2 gated the global session-tab sync to web clients only, removing the desktop path that eagerly populated remote projects. With no on-connect repo fetch left, remote projects only appeared after the user opened the Add-Project dropdown (which calls fetchRuntimeEnvironmentRepos directly). Seed an initial repo/worktree/lineage refresh for every runtime environment that is already connected when useIpcEvents mounts, and for each one that becomes connected afterward. Reuses the debounced/throttled refresh scheduler, and works regardless of whether the remote server emits client events. * fix: tighten idle runtime refresh behavior Co-authored-by: Orca <help@stably.ai> --------- Co-authored-by: Jinwoo-H <jinwoo0825@gmail.com> Co-authored-by: Orca <help@stably.ai>

nwparker merged commit f4840ab into main Mar 20, 2026

shuangly mentioned this pull request Apr 4, 2026

Worktree sidebar doesn't show new PR after first PR is merged #275

Closed

shuangly mentioned this pull request Apr 20, 2026

Add "Open in host browser" option for links (or provide Orca CLI) #854

Closed

xiaolai mentioned this pull request Apr 23, 2026

NLPM audit findings: 1 bug fix + 2 security improvements #976

Closed

akapug mentioned this pull request May 1, 2026

orca v1.3.24: missing single-instance lock corrupts orca-runtime.json + endpoint.env on every relaunch — CLI sees stale daemon metadata #1312

Closed

nwparker mentioned this pull request May 2, 2026

fix(release): grant actions:write + drop broken auto-merge on tap #1336

Merged

1 of 3 tasks

Jinwoo-H mentioned this pull request May 3, 2026

fix: zombie worktree PTY cleanup (issue #218) #1376

Merged

4 of 8 tasks

nengqi mentioned this pull request May 3, 2026

[Feature]: Settings UI for HTTP proxy — env-var bridge in v1.3.32 doesn't cover GUI-launched app #1378

Closed

Jinwoo-H mentioned this pull request May 4, 2026

feat(orchestration): fix check --wait + read/unread semantics #1389

Closed

3 of 5 tasks

buf0-bot Bot mentioned this pull request May 5, 2026

fix: pr-bug-scan findings from #1385 #1462

Merged

BradleyGoulding mentioned this pull request May 18, 2026

[Bug]: ssh terminal state lost after >5min Orca disconnect #2248

Closed

This was referenced Jun 7, 2026

[Bug]: Windows app slow to launch (~2x) and stalls on every keystroke — main process synchronously spawns powershell.exe to re-harden env-store ACLs #4839

Closed

[Feedback]: remote server sync and directory refresh UX #4684

Closed

LesleyMurfin mentioned this pull request Jun 8, 2026

fix(ssh): sync ~/.ssh/config on import and Manage open #4899

Merged

5 tasks done

nwparker mentioned this pull request Jun 9, 2026

test(e2e): stabilize two flaky release-cut e2e specs #4937

Closed

1 of 3 tasks

Jinwoo-H mentioned this pull request Jun 13, 2026

Run the file watcher off the main process to fix the serve deadlock (#5308) #5310

Merged

davideiffert mentioned this pull request Jun 15, 2026

[Bug]: Main-process crash in native file watcher (watcher.node Watcher::findCallback / napi_strict_equals; worker-teardown napi_fatal_error) — 3x in one day on macOS 1.4.68 #5377

Closed

thomaszdxsn pushed a commit to thomaszdxsn/orca that referenced this pull request Jun 15, 2026

Merge pull request stablyai#2 from stablyai/fix/ctrl-backspace-delete…

5a2db2c

…-word

NirajRChaudhari mentioned this pull request Jun 24, 2026

[Bug]: Jira Integration Shows "No Jira Issues Found" #6280

Closed

SpykeRel04D mentioned this pull request Oct 1, 2026

[Bug]: Main process freezes 4–6 s on window focus: Cursor state.vscdb read synchronously on the main thread #24360

Closed

pullfrog Bot mentioned this pull request Oct 5, 2026

[Feature]: Attach an existing tmux session started outside Orca as a terminal pane #25344

Open

bbingz mentioned this pull request Oct 8, 2026

fix(new-workspace): accept an all-digits name instead of treating it as an issue number #26378

Closed

7 of 9 tasks

This was referenced Oct 8, 2026

feat(orcad): add --help/-h usage text #26621

Closed

fix(web): answer the mobile API from host RPCs instead of stubs #26622

Open

fix(web): mount offline instead of no-save degraded mode when the paired host is unreachable at startup #26623

Closed

This was referenced Oct 9, 2026

fix(composer): preserve explicit workspace names after selecting tasks #26937

Merged

fix(composer): ignore delayed PR/MR lookups after you change the form #26954

Merged

AmoabaKelvin mentioned this pull request Oct 14, 2026

feat(native-chat): mark each attached image as [Image #N] where it sits in the prompt #28736

Merged

8 of 9 tasks

SudoAI-DEV mentioned this pull request Oct 15, 2026

[Feature]: Mobile notification should deep-link to the specific agent that finished, not the first agent in the project #29141

Closed

AmoabaKelvin mentioned this pull request Oct 16, 2026

fix(pr): show a branch's own open PR, not a newer closed or same-named one #29330

Closed

8 of 9 tasks

ting7777 mentioned this pull request Oct 18, 2026

Add Gitea issue management to Tasks ting7777/orca#1

Open

10 tasks done

AmoabaKelvin mentioned this pull request Oct 22, 2026

fix(native-chat): keep every image chip when several images are dropped at once #32022

Merged

2 tasks done

eatmoreduck mentioned this pull request Oct 24, 2026

[Bug]: Per-account Claude folder created with no links to ~/.claude for one account — plugins missing and session history stored outside ~/.claude/projects #33170

Closed

brennanb2025 mentioned this pull request Oct 25, 2026

test(native-chat): cover mixed drops through question submission #33632

Merged

nwparker mentioned this pull request Oct 27, 2026

perf(wsl): return orcad's startup compile memory with a malloc mmap threshold #34187

Merged

7 of 9 tasks

AmoabaKelvin mentioned this pull request Oct 27, 2026

fix(mobile): use desktop's rules for queued messages, turn status, Stop and the chat composer #34000

Merged

6 of 8 tasks

This was referenced Oct 30, 2026

fix(plugins): protect every security and destructive string from language packs #36219

Merged

Components (TSX) plugin panels #36814

Merged

kimhongsu mentioned this pull request Nov 6, 2026

Update installer swaps app.asar under the running renderer, corrupting lazy chunks (root cause of the corrupt-chunk crash cluster) #13171

Open

AmoabaKelvin mentioned this pull request Nov 6, 2026

perf(mobile): trim the page bundle back under its old budget #39182

Merged

10 of 11 tasks

Navigation Menu

Repository navigation

Fix Ctrl+Backspace not deleting word in terminal - #2

Fix Ctrl+Backspace not deleting word in terminal#2
nwparker merged 1 commit into
mainfrom
fix/ctrl-backspace-delete-word

mathnogueira commented Mar 19, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

mathnogueira commented Mar 19, 2026

Summary

Test plan

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants