claudecodeui icon indicating copy to clipboard operation
claudecodeui copied to clipboard

Incorrect Executable Permissions on Non-Script Files

Open Difocd opened this issue 5 months ago • 0 comments

Description

In commit 845d5346eb383ebd9ec03e876f8fa35c345ea0b0 ("Update file permissions to executable for multiple files..."), all files in the repository were set to executable (100755) in the Git index. This includes non-executable files such as README.md, LICENSE, .json files, .png and .svg assets, and .jsx components, which typically should have non-executable permissions (100644) in a frontend project like claudecodeui.

Expected Behavior

  • Only files intended to be executed should have 100755 permissions.
  • Non-executable files (e.g., README.md, LICENSE, .json, .png, .svg, .jsx, .css) should have 100644 permissions.

Suggested Fix

# Reset permissions for non-script files (e.g., .md, .json, .png, .svg, .html, .css, .jsx)
git ls-files -s | grep 100755 | grep -E '\.(md|json|png|svg|html|css|jsx)$' | awk '{print $4}' | xargs git update-index --chmod=-x
git commit -m "Remove executable permissions from non-script files"
git push

Questions

  • Was the decision to set all files to executable in commit 845d5346 intentional? If so, could you clarify the reason?
  • Which specific files need to remain executable?

Difocd avatar Jul 23 '25 03:07 Difocd