← Back to PRs

#20319: fix(whisper-skill): set execute bit on transcribe.sh (fixes #9303)

by nikolasdehor open 2026-02-18 19:47 View on GitHub →
size: XS
## Summary - `skills/openai-whisper-api/scripts/transcribe.sh` was committed with mode `100644` (no execute bit), causing a `permission denied` error when the skill tried to run it - This PR changes the git mode to `100755` — a one-character/one-tree-entry fix, no content change - Fixes #9303 ## Root cause The file was added to the repo without the execute bit set. npm preserves git mode bits when publishing, so the installed file inherits the wrong permissions on every fresh install. A manual `chmod +x` workaround is required today. ## Test plan - [ ] Install `openai-whisper-api` skill from a build of this branch - [ ] Verify `ls -l` shows `-rwxr-xr-x` on `transcribe.sh` - [ ] Send an audio file to the gateway and confirm transcription succeeds without permission errors ## Notes - No content changes — only the file mode changes from `100644` → `100755` - Other skills that ship `.sh` scripts should be audited for the same issue (see #9303 discussion) Fixes #9303 <!-- greptile_comment --> <h3>Greptile Summary</h3> Sets the execute bit on `skills/openai-whisper-api/scripts/transcribe.sh` (mode `100644` → `100755`) to fix "permission denied" errors when the Whisper skill invokes the script. No content changes — only the git file mode is updated. - Fixes the root cause of #9303: the script was originally committed without execute permissions, and git/npm preserve file modes on checkout/install - The script is invoked via Node's `execFile` in the Telegram voice-note transcription path (`src/telegram/bot-message-context.ts`), making correct permissions critical for runtime - **Note**: `skills/video-frames/scripts/frame.sh` has the same issue (mode `100644`) and should be fixed in a follow-up <h3>Confidence Score: 5/5</h3> - This PR is safe to merge — it only changes the file permission bit with zero content modifications. - The change is a single file mode update (100644 → 100755) with no code modifications. It directly fixes a documented bug (permission denied on script execution). There is no risk of regression or unintended side effects. - No files require special attention. The only changed file is a permission-bit-only update. <sub>Last reviewed commit: 8881394</sub> <!-- greptile_other_comments_section --> <sub>(3/5) Reply to the agent's comments like "Can you suggest a fix for this @greptileai?" or ask follow-up questions!</sub> <!-- /greptile_comment -->

Most Similar PRs