I created Issue #68 to document this PR. (I can't find the button to link the PR to the Issue, so I am just mentioning it in a comment.)

@agnostic-apollo

This is a sample command in a walkthrough creating the directory with no contents, so the proposed change to the chmod would (in theory) have no effect, since it only affects the directory and its current contents.

The example command is, however, written in a way such that it won't cause an error if the directory exists (mkdir -p), and the recursive flag on chmod would indeed affect any existing files in that case.

Perhaps the tutorial's sample commands should be rewritten to not be so graceful, so as to prevent making any existing files executable?

To be really safe, I think the app itself would need to be modified to scan for permissions and refuse to proceed if unsafe executable flags are set.

Well, if the directory didn't exist, then obv chmod won't do any good, and directory should automatically be 700 too. I added permissions since users are quite often running random commands from the internet including permission and ownership mess ups, so providing commands that users followed would help in fixing those so that issues aren't opened for them. Checking or fixing permissions recursively while showing choosers is a bit much for the app, and java probably doesn't have good support for such granular permission stuff, only for user, not for group and others.

It's our responsibility to just tell user what correct permissions are, specially considering some run selinux in permissive mode too. Rest is on them.

Images have no reason to be executable, so any existing files under existing directory would get fixed.

Script execution permissions are in fact checked and only fixed if its under shortcuts directory. Termux:Tasker allows scripts outside tasker directory, so that directory check is more useful for it.

https://github.com/termux/termux-widget/blob/v0.13.0/app/src/main/java/com/termux/widget/TermuxWidgetProvider.java#L195

Perhaps the tutorial's sample commands should be rewritten to not be so graceful, so as to prevent making any existing files executable?

I didn't get you, can you explain.

Sorry, forgot about this. Merging. Thanks again!