Skip to content

fix(freebsd): process detection, platform guards, and CI race detector#97

Merged
cyyever merged 3 commits intomainfrom
freebsd
Mar 18, 2026
Merged

fix(freebsd): process detection, platform guards, and CI race detector#97
cyyever merged 3 commits intomainfrom
freebsd

Conversation

@cyyever
Copy link
Collaborator

@cyyever cyyever commented Mar 18, 2026

Summary

  • agentdetect: use ps -axo instead of ps -eo to list all processes on BSD systems, fixing test failures where the test binary was not visible
  • platform: skip /proc/version read on non-Linux to avoid wasted syscall
  • platform: add explicit FreeBSD case in platform detection test
  • ci: disable ASLR in FreeBSD VM so Go race detector works correctly
  • ci: enable -race flag for FreeBSD test runs
  • readme: update platform badge from FreeBSD 14+ to 15+

Test plan

  • go build ./... passes
  • go test -race ./... passes
  • Tested manually (describe below)

Security checklist

  • No new path traversal or glob bypass vectors
  • No secrets or credentials in code

cyyever added 3 commits March 18, 2026 14:05
- agentdetect: use ps -axo instead of ps -eo to list all processes on
  BSD systems, fixing test failures where the test binary was not visible
- platform: skip /proc/version read on non-Linux to avoid wasted syscall
- platform: add explicit FreeBSD case in platform detection test
- ci: disable ASLR in FreeBSD VM so Go race detector works correctly
- ci: enable -race flag for FreeBSD test runs
- readme: update platform badge from FreeBSD 14+ to 15+
- platform_test: add "freebsd" to OS switch so test actually verifies
  ShellEnvironment() returns Unix on FreeBSD
- normalizer: clarify that /dev/fd → /proc/self/fd rewrite is
  intentional for consistent rule matching across all platforms, not
  just Linux symlink resolution
@cyyever cyyever merged commit a5c0a22 into main Mar 18, 2026
15 checks passed
@cyyever cyyever deleted the freebsd branch March 18, 2026 06:56
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