Skip to content

fix(app): browse subfolders in the open project dialog - #42786

Closed
SinghAtithi wants to merge 1 commit into
anomalyco:devfrom
SinghAtithi:browse-subfolders
Closed

SinghAtithi wants to merge 1 commit into
anomalyco:devfrom
SinghAtithi:browse-subfolders

Conversation

@SinghAtithi

@SinghAtithi SinghAtithi commented Aug 15, 2026 •

Copy link
Copy Markdown

Issue for this PR

Fixes #42784

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

The directory picker used in the web UI (server mode) lists only the direct children of the home folder. Folders can't be opened, so subfolders are only reachable through the Tab autocomplete in the search box, which isn't discoverable.

Folder rows now have a chevron that navigates into that folder (the list refreshes to show its children), and an up button next to the search box goes back one level. Clicking a row still selects the folder, as before. The desktop tree picker is untouched.

How did you verify your code works?

Ran the backend and app dev servers locally and clicked through the dialog: chevron enters a folder and lists its children, the up button returns to the parent, and row click still selects. Also ran bun typecheck and the picker + i18n parity tests, all passing.

Screenshots / recordings

Before:

directory-picker-before

After:

directory-picker-after

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@github-actions github-actions Bot added needs:compliance This means the issue will auto-close after 2 hours. and removed needs:compliance This means the issue will auto-close after 2 hours. labels Aug 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

@Enough1122

Copy link
Copy Markdown

AI code review — automated review for reference; please use your judgment.

Scope: web directory picker gains a chevron to enter subfolders plus an up-level button next to the search box; desktop tree picker untouched.

  • Verified both new i18n keys (dialog.directory.parent, common.open) already exist across locales, so no missing-translation fallout despite only one file changing.
  • The chevron correctly stopPropagation()s so row-click selection still works; Windows drive normalization via normalizePickerDrive is reused rather than reimplemented — good.
  • CI risk: the file now lacks a trailing newline at EOF (\ No newline at end of file), which will likely fail the repo's prettier/format checks. Worth fixing before merge.
  • Accessibility: the chevron is a <span role="button" tabindex={-1}> with click-only handling — keyboard users can't enter a folder. Consider a real <button> (or keydown handler for Enter/Space) and tabindex={0}.
  • The up button is enabled whenever current() is non-empty, but current() derives from the raw filter text — typing a plain search term (not a path) yields a nonsense parent target. The empty-target guard makes it a harmless no-op, but disabling unless the filter looks path-like might be cleaner.
  • navigate() communicates the target directory by setting the filter to a trailing-slash path — it relies on an implicit convention of createDirectorySearch; a brief comment would help future readers.

@github-actions

Copy link
Copy Markdown
Contributor

Automated PR Cleanup

Thank you for contributing to opencode.

Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions.

This PR was closed because it matched the following cleanup criteria:

  • The PR was created more than 1 month ago
  • The PR had fewer than 2 positive reactions
  • Positive reactions are counted as thumbs-up, heart, celebration, or rocket reactions on the PR

PRs created within the last month are not affected by this cleanup.

If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate.

Thanks again for taking the time to contribute.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot browse into subfolders in the Add project dialog (web UI)

2 participants