Skip to content

fix(core): compare only the executable bit in the mode drift check - #2

Merged
phense merged 1 commit into
mainfrom
fix/mode-check-executable-bit-only
Nov 13, 2026
Merged

phense merged 1 commit into
mainfrom
fix/mode-check-executable-bit-only

Conversation

@phense

@phense phense commented Nov 13, 2026

Copy link
Copy Markdown
Owner

sync-core --check compared the full permission bits of every managed file with the lock. Git records only the executable bit, so on a host with umask 0002 an intact installation (files 664/775) failed with core mode drift: CORE_NOTICES.md.

The check now compares the owner-executable bit with the locked mode. A lost or added execute bit is still reported. Adds a regression test for 664/775 copies.

Verification: python3 -m unittest discover -s tests -t . (165 tests, OK) under umask 0002.

🤖 Generated with Claude Code

Git records only the executable bit. On a host with umask 0002 every
installed copy is 664/775, so sync-core --check reported mode drift for
an intact installation.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
@phense
phense merged commit ca9f5d1 into main Nov 13, 2026
2 checks passed
@phense
phense deleted the fix/mode-check-executable-bit-only branch November 13, 2026 12:53
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