Skip to content

feat(tui): add support for readline list nav (ctrl-p/ctrl-n) - #955

Merged
adamdotdevin merged 2 commits into
anomalyco:devfrom
dkarter:feat/add-support-for-readline-list-nav
Jul 14, 2025
Merged

adamdotdevin merged 2 commits into
anomalyco:devfrom
dkarter:feat/add-support-for-readline-list-nav

Conversation

@dkarter

@dkarter dkarter commented Jul 13, 2025 •

Copy link
Copy Markdown
Contributor

Motivation

I'm so used to navigating lists like auto complete suggestions / file lists with ctrl-p and ctrl-n (Readline default mappings in macOS/Linux/emacs). I'm sure a lot of other people are used to it too (Claude Code also supports it FWIW).

Changes

  • Makes ctrl-n and ctrl-p work in lists like the / command completion and @ file completion.
  • Add tests for the list module

Demo

CleanShot.2025-07-13.at.18.01.09.mp4

@dkarter
dkarter marked this pull request as ready for review July 13, 2025 23:02
@dkarter dkarter changed the title feat(tui): add support for readline list nav feat(tui): add support for readline list nav (ctrl-p/ctrl-n) Jul 13, 2025

@adamdotdevin adamdotdevin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a little change requested to the implementation, but happy to add this

@dkarter
dkarter force-pushed the feat/add-support-for-readline-list-nav branch from b3e6ece to bd3c71b Compare July 14, 2025 14:48
@dkarter
dkarter requested a review from adamdotdevin July 14, 2025 14:49
@adamdotdevin
adamdotdevin merged commit 86a2ea4 into anomalyco:dev Jul 14, 2025
@dkarter

dkarter commented Jul 14, 2025

Copy link
Copy Markdown
Contributor Author

The @adamdotdevin changelog for v0.3.4 says this is included in the release but the diff doesn't contain the commit https://github.com/sst/opencode/releases/tag/v0.3.4

@dkarter
dkarter deleted the feat/add-support-for-readline-list-nav branch July 14, 2025 18:53
andreipromarketing-dev pushed a commit to andreipromarketing-dev/opencode that referenced this pull request Apr 7, 2026
CodeRabbit fixes (6 comments):
- All 4 skills: renamed 'When to Activate' → 'When to Use', added 'How It Works' and 'Examples' sections
- CDSS: DoseValidationResult.suggestedRange now typed as '| null'
- PHI: hyphenated 'Non-patient-sensitive'

Cubic fixes (7 issues):
- P1: CDSS weight-based check now BLOCKS when weight missing (was false-negative pass)
- P1: EMR medication safety clarified — critical = hard block, override requires documented reason
- P1: PHI logging guidance clarified — use opaque UUIDs only, not medical record numbers
- P2: CDSS validateDose now uses age and renal function params (ageAdjusted, renalAdjusted rules)
- P2: Eval CI example now enforces 95% threshold with jq + bc calculation
- P2: Eval CI example now includes --coverage --coverageThreshold on CDSS suite
- P2: CDSS suggestedRange null type fixed (same as CodeRabbit)
andreipromarketing-dev pushed a commit to andreipromarketing-dev/opencode that referenced this pull request Apr 7, 2026
…-patterns

feat: Healthcare domain — 4 skills + 1 agent for health-tech applications
bussard76 pushed a commit to bussard76/openwork that referenced this pull request May 12, 2026
* Refine share preview OG skill card

* Tighten share preview OG styling

* Polish share preview fallback copy

* Serve share preview OG images as PNG

* Default share links to HTML for crawlers

* Remove legacy share OG alias

* Always serve share pages as HTML

* Use direct OG image route

* Use server-safe fonts in OG images

* Fix OG SVG font-family attributes

* Remove unused share proxy

* Remove dead share JSON negotiation helper

---------

Co-authored-by: jcllobet <[email protected]>
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
meSingh pushed a commit to meSingh/opencode that referenced this pull request Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants