Skip to content

OPE46-00034: OPE46-00034: QoL: Add process filtering improvements - #10

Open
andrei-hasna wants to merge 3 commits into
mainfrom
drain/ope46-00034-ms66bf7b
Open

OPE46-00034: OPE46-00034: QoL: Add process filtering improvements#10
andrei-hasna wants to merge 3 commits into
mainfrom
drain/ope46-00034-ms66bf7b

Conversation

@andrei-hasna

@andrei-hasna andrei-hasna commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Automated task drain for OPE46-00034.

Task: OPE46-00034: QoL: Add process filtering improvements

Agent verdict: DONE
Added process owner/name filters and parent-child tree rendering to monitor ps with owner-aware collection and offline coverage.

Independent review: APPROVE

The process owner/name filters and tree rendering are correctly wired through the live local, SSH, and EC2 collection paths, preserve the existing JSON row shape and pagination, and have meaningful unit and CLI coverage. No secrets or remaining out-of-scope changes were found.

Reviewer fix applied:

  • src/collectors/process-list.ts:4 originally requested GNU ps's default-width user= field. A valid owner such as systemd-network is truncated (for example, to systemd+), so monitor ps --user systemd-network returned no rows. The command now requests user:32=.
  • src/collectors/process-list.test.ts:5 adds a regression assertion that fails with the original command, and line 9 verifies preservation of the long owner value.
  • Removed the unrelated duplicate typecheck package s

The reviewer made corrections, committed separately.

Opened by the e2b-task-drain fleet (codex gpt-5.6-sol, effort xhigh).
Not auto-merged — review before landing.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

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