Skip to content

Add ports column to status command#190

Open
Fabian-Fynn wants to merge 3 commits into
mainfrom
add-ports-to-status
Open

Add ports column to status command#190
Fabian-Fynn wants to merge 3 commits into
mainfrom
add-ports-to-status

Conversation

@Fabian-Fynn

Copy link
Copy Markdown
Collaborator

Description

Show the TCP ports each running script is listening on, detected at runtime via lsof over the script's process tree.

Screenshot

image

Show the TCP ports each running script is listening on, detected at
runtime via lsof over the script's process tree.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Fabian-Fynn Fabian-Fynn requested a review from nsams June 16, 2026 15:50
@nsams

nsams commented Jun 16, 2026

Copy link
Copy Markdown
Member

good idea, that seems to be very useful!

  • have you considered using an npm package instead?
  • did you measure the performance impact of this?
  • too bad it doesn't work for docker, but I'd say that is acceptable

Fabian-Fynn and others added 2 commits June 17, 2026 15:05
Replace the manual lsof invocation and output parsing with the pid-port
package, which wraps lsof/netstat and adds cross-platform support.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Store detected ports on the Script instance and only probe until ports
are found, then reuse the stored value. The cache is reset whenever a
new process spawns so a restarted process is re-detected.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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