zsp/everything-claude-code
1
0
Fork 0
mirror of https://github.com/affaan-m/everything-claude-code.git synced 2026-04-07 17:53:32 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: use AppleScript-safe escaping and reduce spawnSync timeout

Browse Source 
- Replace JSON.stringify with curly quote substitution for AppleScript
  compatibility (AppleScript does not support \" backslash escapes)
- Reduce spawnSync timeout from 5000ms to 3000ms to leave headroom
  within the 5s hook deadline
This commit is contained in:
Jonghyeok Park
2026-03-24 10:36:00 +09:00
parent 445ae5099d
commit d3699f9010
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
scripts/hooks/desktop-notify.js
View File

@@ -40,11 +40,14 @@ function extractSummary(message) {
/** /**
* Send a macOS notification via osascript. * Send a macOS notification via osascript.
* Uses spawnSync with an argument array to avoid shell injection. * AppleScript strings do not support backslash escapes, so we replace
* double quotes with curly quotes and strip backslashes before embedding.
*/ */
function notifyMacOS(title, body) { function notifyMacOS(title, body) {
const script = `display notification ${JSON.stringify(body)} with title ${JSON.stringify(title)}`; const safeBody = body.replace(/\\/g, '').replace(/"/g, '\u201C');
spawnSync('osascript', ['-e', script], { stdio: 'ignore', timeout: 5000 }); const safeTitle = title.replace(/\\/g, '').replace(/"/g, '\u201C');
const script = `display notification "${safeBody}" with title "${safeTitle}"`;
spawnSync('osascript', ['-e', script], { stdio: 'ignore', timeout: 3000 });
} }
// TODO: future platform support // TODO: future platform support