Skip to content

feat(command-menu): add Developer section with reload window and show log folder#3031

Merged
MattPua merged 2 commits into
posthog-code/command-menu-shortcutsfrom
posthog-code/command-menu-developer-section
Jun 30, 2026
Merged

feat(command-menu): add Developer section with reload window and show log folder#3031
MattPua merged 2 commits into
posthog-code/command-menu-shortcutsfrom
posthog-code/command-menu-developer-section

Conversation

@MattPua

@MattPua MattPua commented Jun 30, 2026

Copy link
Copy Markdown
Member

Summary

Adds a Developer section to the command menu with two actions:

  • Show log folder — opens ~/.posthog-code/logs in Finder/Explorer via a new os.showLogFolder tRPC mutation
  • Reload windowwindow.location.reload() with a ⌘⇧R global shortcut
CleanShot 2026-06-30 at 12 03 33@2x

Danger level: 1/5


Created with PostHog Code

MattPua added 2 commits June 30, 2026 12:00
…lder

Generated-By: PostHog Code
Task-Id: 422bdee4-0988-45ff-b47e-4337650d40b1
…, add test

Generated-By: PostHog Code
Task-Id: 422bdee4-0988-45ff-b47e-4337650d40b1
@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 7efc24d.

@greptile-apps

greptile-apps Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Comments Outside Diff (1)

  1. packages/workspace-server/src/services/os/os.test.ts, line 146-162 (link)

    P2 Missing test for showLogFolder

    openExternal has a dedicated test asserting that urlLauncher.launch is called with the supplied URL, but showLogFolder — which has analogous (and slightly more complex) logic — has none. The storagePaths mock is already wired up in createService, so adding a test here is straightforward. Omitting it leaves the path-encoding logic (see the related comment) completely uncovered.

    Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Reviews (1): Last reviewed commit: "fix(os-service): use pathToFileURL for c..." | Re-trigger Greptile

Comment thread packages/workspace-server/src/services/os/os.ts
@MattPua MattPua requested a review from charlesvien June 30, 2026 16:04
@MattPua MattPua marked this pull request as ready for review June 30, 2026 16:04
@greptile-apps

greptile-apps Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Reviews (2): Last reviewed commit: "fix(os-service): use pathToFileURL for c..." | Re-trigger Greptile

@MattPua MattPua merged commit e327ae6 into posthog-code/command-menu-shortcuts Jun 30, 2026
41 of 44 checks passed
@MattPua MattPua deleted the posthog-code/command-menu-developer-section branch June 30, 2026 16:10
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