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

perf(hooks): reduce PowerShell spawns from 3 to 1 per notification

Browse Source 
Merge findPowerShell version check and isBurntToastAvailable check
into a single notifyWindows call. Now just tries to send directly;
if it fails, tries next PowerShell path. Version field was unused.

Net effect: up to 3 spawns reduced to 1 in the happy path.
This commit is contained in:
QWsin
2026-03-30 14:12:05 +08:00
parent 3d5ae70c74
commit 99ff568c0e
1 changed files with 17 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -35,8 +35,7 @@ if (process.platform === 'linux') {
/** /**
* Find available PowerShell executable on WSL. * Find available PowerShell executable on WSL.
* Checks PowerShell 7 first, then falls back to Windows PowerShell. * Returns first accessible path, or null if none found.
* Returns { path, version } or null if none available.
*/ */
function findPowerShell() { function findPowerShell() {
if (!isWSL) return null; if (!isWSL) return null;
@@ -50,11 +49,10 @@ function findPowerShell() {
for (const path of candidates) { for (const path of candidates) {
try { try {
const result = spawnSync(path, ['-Command', '$PSVersionTable.PSVersion.Major'], const result = spawnSync(path, ['-Command', 'exit 0'],
{ stdio: ['ignore', 'pipe', 'ignore'], timeout: 3000 }); { stdio: ['ignore', 'pipe', 'ignore'], timeout: 1000 });
if (result.status === 0) { if (result.status === 0) {
const version = parseInt(result.stdout.toString().trim(), 10); return path;
return { path, version };
} }
} catch { } catch {
// continue // continue
@@ -64,17 +62,16 @@ function findPowerShell() {
} }
/** /**
* Check if BurntToast module is available on the given PowerShell path. * Send a Windows Toast notification via PowerShell BurntToast.
* Returns true on success, false on failure.
*/ */
function isBurntToastAvailable(pwshPath) { function notifyWindows(pwshPath, title, body) {
try { const safeBody = body.replace(/'/g, "''");
const result = spawnSync(pwshPath, const safeTitle = title.replace(/'/g, "''");
['-Command', 'Import-Module BurntToast -ErrorAction Stop; $true'], const command = `Import-Module BurntToast; New-BurntToastNotification -Text '${safeTitle}', '${safeBody}'`;
{ stdio: ['ignore', 'pipe', 'ignore'], timeout: 5000 }); const result = spawnSync(pwshPath, ['-Command', command],
return result.status === 0; { stdio: ['ignore', 'pipe', 'pipe'], timeout: 5000 });
} catch { return result.status === 0;
return false;
}
} }
/** /**
@@ -136,14 +133,12 @@ function run(raw) {
if (isMacOS) { if (isMacOS) {
notifyMacOS(TITLE, summary); notifyMacOS(TITLE, summary);
} else if (isWSL) { } else if (isWSL) {
// WSL: try PowerShell 7 first, then Windows PowerShell
const ps = findPowerShell(); const ps = findPowerShell();
if (ps && isBurntToastAvailable(ps.path)) { if (ps && notifyWindows(ps, TITLE, summary)) {
notifyWindows(ps.path, TITLE, summary); // notification sent successfully
} else if (ps) { } else if (ps) {
// PowerShell exists but no BurntToast module // PowerShell found but BurntToast not available
log('[DesktopNotify] Tip: Install BurntToast module to enable notifications:'); log('[DesktopNotify] Tip: Install BurntToast module to enable notifications');
log(`[DesktopNotify] "${ps.path}" -Command "Install-Module -Name BurntToast -Scope CurrentUser"`);
} else { } else {
// No PowerShell found // No PowerShell found
log('[DesktopNotify] Tip: Install BurntToast module in PowerShell for notifications'); log('[DesktopNotify] Tip: Install BurntToast module in PowerShell for notifications');