pinokiod 3.139.0 → 3.141.0

package/kernel/util.js

@@ -33,6 +33,32 @@ const platform = os.platform()
 const WINDOWS_TOAST_APP_ID = process.env.PINOKIO_WINDOWS_APP_ID || 'computer.pinokio'
 const DEFAULT_CHIME_URL_PATH = '/chime.mp3'
 const pushListeners = new Set()
+let cachedMacNotifierPath
+
+function resolveAsarPath(p) {
+  if (!p || p.includes('app.asar.unpacked')) {
+    return p
+  }
+  if (p.includes('app.asar')) {
+    return p.replace('app.asar', 'app.asar.unpacked')
+  }
+  return p
+}
+
+function getMacNotifierBinaryPath() {
+  if (cachedMacNotifierPath !== undefined) {
+    return cachedMacNotifierPath
+  }
+  try {
+    const notifierModulePath = require.resolve('toasted-notifier/notifiers/notificationcenter')
+    const binaryPath = resolveAsarPath(path.resolve(path.dirname(notifierModulePath), '../vendor/mac.noindex/terminal-notifier.app/Contents/MacOS/terminal-notifier'))
+    cachedMacNotifierPath = binaryPath
+    return cachedMacNotifierPath
+  } catch (err) {
+    cachedMacNotifierPath = null
+    return cachedMacNotifierPath
+  }
+}
 
 function registerPushListener(listener) {
   if (typeof listener !== 'function') {
@@ -63,15 +89,20 @@ function resolvePublicAssetUrl(filePath) {
     return filePath
   }
   try {
-    const absolute = path.resolve(filePath)
+    const absolute = resolveAsarPath(path.resolve(filePath))
     const publicRoot = path.resolve(__dirname, '../server/public')
-    if (absolute === publicRoot) {
+    const unpackedRoot = resolveAsarPath(publicRoot)
+    if (absolute === publicRoot || absolute === unpackedRoot) {
       return '/'
     }
-    if (absolute.startsWith(publicRoot + path.sep) || absolute === publicRoot) {
+    if (absolute.startsWith(publicRoot + path.sep)) {
       const relative = path.relative(publicRoot, absolute).replace(/\\/g, '/')
       return '/' + relative
     }
+    if (absolute.startsWith(unpackedRoot + path.sep)) {
+      const relative = path.relative(unpackedRoot, absolute).replace(/\\/g, '/')
+      return '/' + relative
+    }
   } catch (_) {
     // ignore resolution failures
   }
@@ -86,7 +117,7 @@ function soundTargetToClientUrl(target) {
     return target.value
   }
   if (target.kind === 'file') {
-    const resolved = path.resolve(target.value)
+    const resolved = resolveAsarPath(path.resolve(target.value))
     if (resolved === DEFAULT_CHIME_PATH) {
       return DEFAULT_CHIME_URL_PATH
     }
@@ -98,9 +129,11 @@ function ensureNotifierBinaries() {
   if (platform !== 'darwin') {
     return
   }
+  const binaryPath = getMacNotifierBinaryPath()
+  if (!binaryPath) {
+    return
+  }
   try {
-    const notifierModulePath = require.resolve('toasted-notifier/notifiers/notificationcenter')
-    const binaryPath = path.resolve(path.dirname(notifierModulePath), '../vendor/mac.noindex/terminal-notifier.app/Contents/MacOS/terminal-notifier')
     if (!fs.existsSync(binaryPath)) {
       return
     }
@@ -629,7 +662,7 @@ const linuxSoundCandidates = [
   { cmd: 'cvlc', args: (filePath) => ['--play-and-exit', filePath] },
 ]
 let cachedLinuxSoundPlayer
-const DEFAULT_CHIME_PATH = path.resolve(__dirname, '../server/public/chime.mp3')
+const DEFAULT_CHIME_PATH = resolveAsarPath(path.resolve(__dirname, '../server/public/chime.mp3'))
 function pickLinuxSoundPlayer() {
   if (cachedLinuxSoundPlayer !== undefined) {
     return cachedLinuxSoundPlayer
@@ -683,10 +716,10 @@ function playSoundFile(filePath) {
     return null
   }
   if (platform === 'darwin') {
-    return spawn('afplay', [filePath], { stdio: 'ignore' })
+    return spawn('afplay', [resolveAsarPath(filePath)], { stdio: 'ignore' })
   }
   if (platform === 'win32') {
-    const fileUrl = pathToFileURL(filePath).href.replace(/'/g, "''")
+    const fileUrl = pathToFileURL(resolveAsarPath(filePath)).href.replace(/'/g, "''")
     const script = [
       "$ErrorActionPreference = 'Stop'",
       "Add-Type -AssemblyName presentationCore",
@@ -704,7 +737,8 @@ function playSoundFile(filePath) {
   if (!candidate) {
     throw new Error('No supported audio player found for Linux (expected one of paplay, aplay, ffplay, mpg123, mplayer, cvlc).')
   }
-  return spawn(candidate.cmd, candidate.args(filePath), { stdio: 'ignore' })
+  const preparedPath = resolveAsarPath(filePath)
+  return spawn(candidate.cmd, candidate.args(preparedPath), { stdio: 'ignore' })
 }
 function scheduleSoundPlayback(target) {
   if (!target) {
@@ -715,7 +749,7 @@ function scheduleSoundPlayback(target) {
     try {
       let filePath
       if (target.kind === 'file') {
-        filePath = target.value
+        filePath = resolveAsarPath(target.value)
       } else if (target.kind === 'url') {
         if (!HTTP_URL_REGEX.test(target.value)) {
           throw new Error(`Unsupported sound URL: ${target.value}`)
@@ -791,17 +825,32 @@ function push(params) {
     notifyParams.title = "Pinokio"
   }
   if (!notifyParams.contentImage) {
-    notifyParams.contentImage = path.resolve(__dirname, "../server/public/pinokio-black.png")
+    notifyParams.contentImage = resolveAsarPath(path.resolve(__dirname, "../server/public/pinokio-black.png"))
+  }
+  if (isNonEmptyString(notifyParams.contentImage) && !HTTP_URL_REGEX.test(notifyParams.contentImage)) {
+    notifyParams.contentImage = resolveAsarPath(notifyParams.contentImage)
+  }
+  if (isNonEmptyString(notifyParams.image) && !HTTP_URL_REGEX.test(notifyParams.image)) {
+    notifyParams.image = resolveAsarPath(notifyParams.image)
+  }
+  if (isNonEmptyString(notifyParams.icon) && !HTTP_URL_REGEX.test(notifyParams.icon)) {
+    notifyParams.icon = resolveAsarPath(notifyParams.icon)
   }
   if (platform === 'win32') {
     // Ensure Windows toast branding aligns with Pinokio assets.
     if (!notifyParams.icon && notifyParams.contentImage) {
-      notifyParams.icon = notifyParams.contentImage
+      notifyParams.icon = resolveAsarPath(notifyParams.contentImage)
     }
     if (!notifyParams.appID && !notifyParams.appName) {
       notifyParams.appID = WINDOWS_TOAST_APP_ID
     }
   }
+  if (platform === 'darwin') {
+    const macNotifierPath = getMacNotifierBinaryPath()
+    if (macNotifierPath) {
+      notifyParams.customPath = macNotifierPath
+    }
+  }
 
   const clientSoundUrl = soundTargetToClientUrl(customSoundTarget)
   if (customSoundTarget) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pinokiod",
-  "version": "3.139.0",
+  "version": "3.141.0",
   "description": "",
   "main": "index.js",
   "scripts": {

package/server/public/tab-idle-notifier.js

@@ -480,7 +480,7 @@ const ensureTabAccessories = aggregateDebounce(() => {
     const title = tab ? tab.textContent.trim() : 'Tab activity';
     //const subtitle = title || 'Pinokio';
     //const message = state.lastInput ? `Last input: ${state.lastInput}` : 'Tab is now idle.';
-    const message = state.lastInput ? `Response to: "${state.lastInput}` : "Tab is now idle."
+    const message = state.lastInput ? `From: "${state.lastInput}"` : "Tab is now idle."
     const image = resolveTabImage(link);
 
     const payload = {

package/server/views/app.ejs

@@ -5213,7 +5213,7 @@ body.dark {
       for(let frame of hardFrames) {
         // Types of links in the tab
         //if (String(port) === "<%=port%>") {
-        if (String(port) === String(location.port) || /https:\/\/pinokio\..*localhost/.test(origin)) {
+        if (String(port) === String(location.port) || /https:\/\/.+\.localhost/.test(origin)) {
           // 1. running from <%=port%> => find a frame whose base URL matches the tab's base URL
           // Pinokio links
           let targetPath = new URL(target.href).pathname