Skip to content

feat: add exported macOS symbolic hotkeys plist - #7

Open
sapoepsilon wants to merge 1 commit into
mainfrom
feat/macos-hotkeys
Open

sapoepsilon wants to merge 1 commit into
mainfrom
feat/macos-hotkeys

Conversation

@sapoepsilon

Copy link
Copy Markdown
Owner

Summary

setup.sh already knows about macos/symbolichotkeys.plist on both sides of the round trip:

  • setup.sh:181 — defaults export com.apple.symbolichotkeys "$SHORTCUTS_PLIST" (and prints "commit it")
  • setup.sh:185 — defaults import com.apple.symbolichotkeys "$SHORTCUTS_PLIST"

…but the exported plist was never actually committed, so a fresh machine had nothing to restore from. This adds it.

Contents are just disabled Apple symbolic hotkeys — no secrets or personal paths.

Also included

  • .gitattributes — *.plist binary. The existing * text=auto relies on Git's binary heuristic to avoid line-ending normalization on the bplist. The heuristic gets it right today (the file has NUL bytes), but an explicit rule makes it a guarantee rather than a detection outcome.
  • .gitignore — *.log, to keep stray tool logs (an excalidraw.log from an MCP server had landed in the tree) out of the repo.

Test plan

  • defaults import com.apple.symbolichotkeys macos/symbolichotkeys.plist on a fresh machine restores the shortcut state
  • git check-attr binary macos/symbolichotkeys.plist reports binary: set

🤖 Generated with Claude Code

https://claude.ai/code/session_01PWJyJ8YZud1bCVfaETKpLH

setup.sh already exports com.apple.symbolichotkeys to
macos/symbolichotkeys.plist and imports it back on restore, but the
exported plist itself was never committed, so a fresh machine had
nothing to restore from.

Mark *.plist as binary in .gitattributes so the `* text=auto` rule can
never line-ending normalize it, and ignore *.log to keep stray tool
logs out of the tree.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PWJyJ8YZud1bCVfaETKpLH
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