@telepath-computer/television-desktop 0.1.107 → 0.1.112

package/README.md

@@ -19,11 +19,8 @@ token, or a `--public` server with an empty token).
 
 ## macOS notes
 
-The first install runs a postinstall script that renames the bundled
-`Electron.app` to `Television.app` (bundle directory + inner executable +
-`Info.plist` + bundle icon). This is what gives the Dock and menu bar
-"Television" instead of "Electron".
-
-If you install with `--ignore-scripts`, the rename is skipped — the app
-still works but will be labelled "Electron" everywhere macOS reads the
-bundle.
+The first launch renames the bundled `Electron.app` to `Television.app`
+(bundle directory + inner executable + `Info.plist` + bundle icon). This
+is what gives the Dock and menu bar "Television" instead of "Electron".
+First launch is a few seconds slower; every launch after takes the fast
+path.

package/bin/tv-desktop.cjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-const { spawn } = require("node:child_process");
+const { spawn, spawnSync } = require("node:child_process");
 const path = require("node:path");
 const fs = require("node:fs");
 
@@ -11,6 +11,17 @@ function resolveExecutable() {
   if (process.platform === "darwin") {
     const renamed = path.join(distDir, "Television.app", "Contents", "MacOS", "Television");
     if (fs.existsSync(renamed)) return renamed;
+    // npm runs our package's postinstall before electron's postinstall has
+    // downloaded the Electron.app, so we can't rely on a postinstall hook.
+    // Rename lazily on first launch (idempotent); subsequent launches take
+    // the fast path above.
+    const upstream = path.join(distDir, "Electron.app");
+    if (fs.existsSync(upstream)) {
+      spawnSync(process.execPath, [path.join(appDir, "scripts", "rename-bundle.cjs")], {
+        stdio: "inherit",
+      });
+      if (fs.existsSync(renamed)) return renamed;
+    }
   }
   return require("electron");
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@telepath-computer/television-desktop",
   "productName": "Television",
-  "version": "0.1.107",
+  "version": "0.1.112",
   "type": "module",
   "main": "dist/electron.cjs",
   "bin": {
@@ -11,14 +11,13 @@
     "dist/**",
     "bin/**",
     "assets/**",
-    "scripts/postinstall.cjs"
+    "scripts/rename-bundle.cjs"
   ],
   "exports": {
     ".": "./src/index.ts"
   },
   "scripts": {
     "build": "node build.mjs",
-    "postinstall": "node scripts/postinstall.cjs",
     "prepublishOnly": "node build.mjs",
     "type-check": "tsc -p tsconfig.json --noEmit"
   },

package/scripts/{postinstall.cjs → rename-bundle.cjs}

@@ -4,7 +4,13 @@
 // "Television" instead of "Electron". macOS reads the running-app name
 // from the bundle's filesystem identity; just editing Info.plist isn't
 // enough — the .app directory and Contents/MacOS/<exe> have to match.
-// Scoped to this package's resolved electron install.
+// Scoped to this package's resolved electron install. Idempotent; no-ops
+// once the rename is complete.
+//
+// Invoked lazily on first launch by bin/tv-desktop.cjs. We can't use an
+// npm `postinstall` hook because npm runs our package's postinstall
+// before electron's postinstall has downloaded Electron.app — the rename
+// has nothing to rename at install time.
 
 const { execFileSync } = require("node:child_process");
 const path = require("node:path");