Skip to content

fix: refuse rewriting through symbolic links - #4

Merged
hc-ui merged 2 commits into
mainfrom
cursor/refuse-symlink-writes-aa74-rebased
Aug 26, 2026
Merged

hc-ui merged 2 commits into
mainfrom
cursor/refuse-symlink-writes-aa74-rebased

Conversation

@hc-ui

@hc-ui hc-ui commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Same fix as #3, rebased onto current main after #1/#2 so it can merge.

  • Named file symlinks are analyzed; fix --apply and attributes --write refuse to write through a symlink.
  • Keeps the OSError handling from main.
  • Tests cover both the symlink guard and the permission-error path.

@hc-ui
hc-ui merged commit ecfcda1 into main Aug 26, 2026
16 checks passed
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