Skip to content

fix: group tree view by process name instead of PID#101

Merged
productdevbook merged 1 commit intomainfrom
fix/tree-view-grouping
Mar 9, 2026
Merged

fix: group tree view by process name instead of PID#101
productdevbook merged 1 commit intomainfrom
fix/tree-view-grouping

Conversation

@productdevbook
Copy link
Copy Markdown
Owner

Summary

  • Changes tree view grouping from PID-based to process name-based
  • Two node servers on different ports now appear under one "node" group instead of two separate groups
  • ProcessGroup model updated: id is now the process name (String), added pids: [Int] array
  • PID display shows single PID or "N PIDs" count with tooltip listing all PIDs
  • Updated all grouping locations: PortTableView, MenuBarView, MenuBarPortList, PortGroupingService

Fixes #85

Test plan

  • Start two servers with the same process name (e.g., two node instances) on different ports
  • Verify they appear under one group in tree view
  • Verify expanding the group shows all ports
  • Verify PID column shows "2 PIDs" with tooltip
  • Verify menu bar tree view also groups correctly
  • Verify single-PID processes still show the PID number

🤖 Generated with Claude Code

Previously, two instances of the same process (e.g., two node servers)
were shown as separate groups because grouping was by PID. Now they
are grouped by process name, so all ports from the same process appear
under one collapsible group.

Fixes #85

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@productdevbook productdevbook merged commit 78a1c05 into main Mar 9, 2026
2 checks passed
@productdevbook productdevbook deleted the fix/tree-view-grouping branch March 9, 2026 06:57
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.

View ports

1 participant