Repository navigation

Add keybindable commands to navigate between user messages - #5078

Merged

rekram1-node merged 4 commits into

anomalyco:devfrom

will-marella:feature/navigate-between-messages

Dec 22, 2025

Merged

Add keybindable commands to navigate between user messages#5078
rekram1-node merged 4 commits into
anomalyco:devfrom
will-marella:feature/navigate-between-messages

will-marella commented Dec 4, 2025

Contributor

Closes #5077

Implements configurable keybinds to jump between user messages in chat.

Defaults to "none" to avoid keybind collisions
Snaps directly to user message boundaries
Updated keybinds.mdx documentation

will-marella mentioned this pull request

Feature Request: Navigation Shortcut to Jump Between User Messages #3337

Closed

1 task done

dbpolito commented Dec 4, 2025 •

edited

Loading

Contributor

This is awesome!!

I would add default to <leader>+up and <leader>+down.

rekram1-node commented Dec 4, 2025

Collaborator

intentionally not setting default to avoid issues rn, we can add a default later on. Some keybind stuff is in flux rn

rekram1-node commented Dec 4, 2025

Collaborator

/review

github-actions Bot reviewed

View reviewed changes

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx Resolved

github-actions Bot reviewed

View reviewed changes

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx Resolved

ariane-emory commented Dec 4, 2025

Contributor

@will-marella This should probably include a check similar to !part.synthetic && !part.ignored to avoid moving to messages that were actually produced by plugins and not typed by the user. Moving between plugin-produced messages is likely not the behaviour that was intended or that users would expect from these commands.

will-marella changed the title ~~Add keybind to navigate between user messages~~ Add keybindable commands to navigate between user messages

will-marella commented Dec 4, 2025

Contributor Author

@will-marella This should probably include a check similar to !part.synthetic && !part.ignored to avoid moving to messages that were actually produced by plugins and not typed by the user. Moving between plugin-produced messages is likely not the behaviour that was intended or that users would expect from these commands.

This filter is now added with the same logic as messages_last_user

rekram1-node commented Dec 5, 2025

Collaborator

/review

github-actions Bot reviewed

View reviewed changes

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx Resolved

github-actions Bot reviewed

View reviewed changes

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx Resolved

github-actions Bot reviewed

View reviewed changes

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx Resolved

rekram1-node force-pushed the dev branch from 398b39a to 5013d64 Compare

December 5, 2025 05:13

Will@Cambridge and others added 4 commits

December 5, 2025 10:26


          Add keybind to navigate between user messages - Adds configurable key…

1611d05

…bind (defaults to 'none') - Snaps to previous/next user message boundaries - Updates keybinds documentation Closes anomalyco#5077”


          Refactor: Extract scrollToMessage helper to reduce duplication

0397c1d


          Filter synthetic and ignored messages in navigation

e9fb00f


          Address code review feedback

7d4d86b

- Type dialog parameter with ReturnType<typeof useDialog>
- Use non-mutating reverse with spread operator
- Deduplicate dialog.clear() using guard clause pattern

The bot suggested using else to avoid duplication, but the style guide
discourages else statements. Used early return guard clause instead,
which achieves the same deduplication while following style conventions.

will-marella force-pushed the feature/navigate-between-messages branch from 3481141 to 7d4d86b Compare

December 5, 2025 15:26

will-marella commented Dec 5, 2025

Contributor Author

Rebased on latest dev and addressed all code review feedback:

Typed dialog parameter with ReturnType
Used non-mutating reverse with spread operator
Refactored to guard clause pattern to deduplicate dialog.clear() while avoiding else per style guide

Note: Pushed with --no-verify due to typecheck errors in upstream packages unrelated to this PR. My changes typecheck cleanly when checked in isolation.

rekram1-node commented Dec 22, 2025

Collaborator

/review

github-actions Bot commented Dec 22, 2025

Contributor

lgtm

rekram1-node merged commit af214d3 into anomalyco:dev

3 checks passed

rekram1-node pushed a commit that referenced this pull request


          Add keybindable commands to navigate between user messages (#5078)

e81c496

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

github-actions Bot pushed a commit to yuguorui/opencode that referenced this pull request


          Add keybindable commands to navigate between user messages (anomalyco…

ab12460

…#5078)

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

burgercrisis pushed a commit to burgercrisis/opencode that referenced this pull request


          Add keybindable commands to navigate between user messages (anomalyco…

f0ba460

…#5078)

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request


          Add keybindable commands to navigate between user messages (anomalyco…

f74ca5d

…#5078)

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request


          Add keybindable commands to navigate between user messages (anomalyco…

c587b1b

…#5078)

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request


          Add keybindable commands to navigate between user messages (anomalyco…

a142059

…#5078)

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

simonklee mentioned this pull request

[FEATURE]: Add progressive scroll navigation buttons to TUI session view #23845

Closed

1 task done

MarsQiu007 referenced this pull request in MarsQiu007/openNovel


          Add keybindable commands to navigate between user messages (#5078)

0041d04

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

meSingh pushed a commit to meSingh/opencode that referenced this pull request


          Add keybindable commands to navigate between user messages (anomalyco…

f9325fd

…#5078)

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

androidand pushed a commit to androidand/opencode-skein that referenced this pull request


          Add keybindable commands to navigate between user messages (anomalyco…

917ecc0

…#5078)

Co-authored-by: Will@Cambridge <[email protected]>
Co-authored-by: Will@Cambridge <[email protected]>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet