feat(tui): fuzzy search on skill descriptions, recently used skills, skill detail view#32454
Open
Btema2 wants to merge 8 commits into
Open
feat(tui): fuzzy search on skill descriptions, recently used skills, skill detail view#32454Btema2 wants to merge 8 commits into
Btema2 wants to merge 8 commits into
Conversation
added 7 commits
June 15, 2026 17:50
…scription search Add recently-used skills tracking across TUI, CLI, and web interfaces. Skills are recorded on selection and persisted to disk with 30-day TTL. - New recent-skills.ts utility with in-memory cache and fire-and-forget persistence - TUI DialogSkill: 'Recently Used' category at top of skill list - CLI RunSkillSelectBody: recent skills pinned at top, right arrow drill-down to show skill description in-place, left arrow to return - Web SlashPopover: 'Recently Used' section above full skill list - Fuzzy search now includes description field across TUI and web Right arrow dispatch is confirmed safe: useKeyboard global listeners fire before the <input> element's own keypress handler via emitWithPriority.
Reverts the detail view / drill-down panel that was added alongside the recently-used and fuzzy-description-search features. Keyboard shortcuts (arrow keys, tab, ctrl+e, ctrl+o) all failed to fire reliably due to opentui's global event propagation consuming keys before panel-level handlers ran; stale container images compounded debugging. Keeps working features: recently-used skills section and fuzzy search on skill descriptions. Adds drill-down-menu-try1.md documenting the failed approaches so future attempts don't repeat the same dead ends.
- Add DialogSkillDetail with compact header, scrollable template, Enter binding - DialogSkillDetail pushed via dialog.push() with arrow function wrapper - DialogSelect gains trailing/onTrailingClick on Option for › chevron - DialogSelect exposes selectedIndex/onIndexChange props - DialogProvider adds push()/pop() API with local clearAll/popDialog helpers - Arrow function wrapper on dialog.push() fixes SolidJS reactivity loss - Escape/ctrl+c delegate to popDialog() for consistent back navigation
- Use Bun.file().text() and Bun.write() over fs/promises readFile/writeFile - Replace try/catch with .catch() chains - Replace module-level let with const object wrapper - Remove redundant createMemo in dialog-skill and dialog-skill-detail - Fix double recordSkillUsage call (footer.view called it, RunSkillSelectBody already does)
github-actions
Bot
added
needs:compliance
Contributor
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
This was referenced Jun 19, 2026
Author
|
Hi. Sorry for bothering, but will that PR be reviewied? Do I need to do some clrification or changes? |
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
1 participant
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.
Issue for this PR
Closes #32453
Type of change
What does this PR do?
Adds three small improvements to the slash skills menu:
ctrl+oin the TUI skills dialog shows the full description and template content for the selected skill. Press Enter to insert it, Esc to go back to the list.Recent skill usage is stored in
~/.local/share/opencode/recent-skills.json(usesGlobal.Path.state), capped at 50 entries with a 30-day TTL.How did you verify your code works?
Ran
bun devin the repo, opened the skills dialog via/, verified:ctrl+oopens the detail view with description and template visiblebun turbo typecheckScreenshots / recordings
Checklist