Skip to content

fix: match daemon process flags exactly#6

Merged
brannn merged 1 commit into
mainfrom
fix/darwin-process-flag-matching
May 21, 2026
Merged

fix: match daemon process flags exactly#6
brannn merged 1 commit into
mainfrom
fix/darwin-process-flag-matching

Conversation

@brannn
Copy link
Copy Markdown
Contributor

@brannn brannn commented May 21, 2026

Summary

  • replace substring matching for daemon command-line flag verification with exact token-boundary matching
  • move the command-line matcher into shared process arg helpers
  • add parser tests covering separate and equals flag forms plus prefix mismatches

Test Plan

  • go test ./cmd/threadmark -run TestCommandLineHasFlagValue -count=1
  • go test ./cmd/threadmark -count=1
  • GOOS=darwin GOARCH=amd64 go test -c ./cmd/threadmark -o /tmp/threadmark-darwin.test
  • go test ./...
  • git diff --check

@brannn brannn merged commit 525bae7 into main May 21, 2026
1 check passed
@brannn brannn deleted the fix/darwin-process-flag-matching branch May 21, 2026 06:59
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