Skip to content

Fix watcher stat mtime probing#163

Open
benawad wants to merge 3 commits into
kunchenguid:mainfrom
benawad:fm/watcher-stat-path-fix-m2-nm
Open

Fix watcher stat mtime probing#163
benawad wants to merge 3 commits into
kunchenguid:mainfrom
benawad:fm/watcher-stat-path-fix-m2-nm

Conversation

@benawad

@benawad benawad commented Jun 30, 2026

Copy link
Copy Markdown

Summary

  • add a shared validated stat helper for watcher/guard/wake mtime and signature reads
  • prefer BSD /usr/bin/stat on Darwin while rejecting nonnumeric GNU filesystem-stat output
  • fix BSD fm_path_sig to use integer %m instead of float %Fm and add Darwin coverage

Validation

  • no-mistakes run 01KWD0AC18D09ZSFGXFAWT21TJ passed review, test, document, and lint on head 4f6f939
  • no-mistakes push to upstream failed with 403 for benawad on kunchenguid/firstmate, so this PR is opened from the authorized fork benawad/firstmate
  • manual pre-gate validation also passed: bash -n bin/.sh tests/.sh; for test_script in tests/*.test.sh; do bash ""; done

Pipeline

Updates from git push no-mistakes

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