Skip to content

feat(folder-picker): show up to 10 recent repositories#2725

Open
richardsolomou wants to merge 2 commits into
mainfrom
posthog-code/recent-repos-show-10
Open

feat(folder-picker): show up to 10 recent repositories#2725
richardsolomou wants to merge 2 commits into
mainfrom
posthog-code/recent-repos-show-10

Conversation

@richardsolomou

Copy link
Copy Markdown
Member

Problem

We work across a lot of repos internally, and the folder picker only surfaced the 5 most recent repositories — too limiting when switching between many.

Changes

Bumped the recent repositories shown in the folder picker's "Recent" list from 5 to 10 (getRecentFolders(10) in FolderPicker.tsx).

How did you test this?

Not manually tested beyond confirming the change. It's a one-line bump of an existing display limit.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code from a Slack thread

Bump the recent folders shown in the folder picker from 5 to 10 to better support teams working across many internal repos.

Generated-By: PostHog Code
Task-Id: c5d2c398-2356-4dcf-8bf8-26419318a723
@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit f8f3f1a.

@greptile-apps

greptile-apps Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor
Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
packages/ui/src/features/folder-picker/FolderPicker.tsx:50
The literal `10` is a magic number. Extracting it to a named constant would make the intent self-documenting and make future adjustments easier to find. Note also that `AdditionalDirectoriesButton.tsx` calls `getRecentFolders()` without an argument, so it still uses the old default of `5` — both limits now live implicitly in different files. A shared constant (or a consistent explicit call) would keep the two in sync.

```suggestion
  const MAX_RECENT_FOLDERS = 10;
  const recentFolders = getRecentFolders(MAX_RECENT_FOLDERS);
```

Reviews (1): Last reviewed commit: "feat(folder-picker): show up to 10 recen..." | Re-trigger Greptile

Comment thread packages/ui/src/features/folder-picker/FolderPicker.tsx Outdated
Change the getRecentFolders default limit from 5 to 10 instead of overriding at the call site, so all consumers benefit consistently.

Generated-By: PostHog Code
Task-Id: c5d2c398-2356-4dcf-8bf8-26419318a723
@richardsolomou richardsolomou requested a review from a team June 17, 2026 07:13
@richardsolomou richardsolomou marked this pull request as ready for review June 17, 2026 07:13
@greptile-apps

greptile-apps Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Reviews (2): Last reviewed commit: "refactor(folders): default recent folder..." | Re-trigger Greptile

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.

1 participant