@happy-nut/monacori 0.1.20 → 0.1.22

package/package.json

@@ -1,8 +1,9 @@
 {
   "name": "@happy-nut/monacori",
-  "version": "0.1.20",
+  "version": "0.1.22",
   "description": "Validation control plane for AI-generated code changes.",
   "type": "module",
+  "main": "dist/app-main.js",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/happy-nut/monacori.git"
@@ -35,7 +36,9 @@
     "prepare": "npm run build",
     "smoke": "npm run build && node dist/cli.js --help",
     "pretest": "npm run build",
-    "test": "node --test test/*.test.mjs"
+    "test": "node --test test/*.test.mjs",
+    "dist:mac": "npm run build && electron-packager . Monacori --platform=darwin --arch=arm64 --icon=assets/icon.icns --app-bundle-id=dev.happynut.monacori --app-category-type=public.app-category.developer-tools --out=release --overwrite --no-asar --ignore=/src/ --ignore=/test/ --ignore=/release --ignore=/.git --ignore=/.github --ignore=/.omc --ignore=/openspec --ignore=/node_modules/electron/ --ignore=/node_modules/@electron --ignore=/node_modules/typescript --ignore=/node_modules/jsdom",
+    "dist:mac:dmg": "npm run dist:mac && create-dmg --overwrite --no-code-sign --dmg-title=Monacori 'release/Monacori-darwin-arm64/Monacori.app' release/ || true"
   },
   "keywords": [
     "ai",
@@ -47,8 +50,10 @@
   "author": "happy-nut <happynut.dev@gmail.com>",
   "license": "MIT",
   "devDependencies": {
+    "@electron/packager": "^20.0.1",
     "@types/node": "^22.15.21",
     "jsdom": "^29.1.1",
+    "terser": "^5.48.0",
     "typescript": "^5.8.3"
   },
   "engines": {
@@ -62,5 +67,8 @@
     "electron": "^42.4.1",
     "highlight.js": "^11.11.1",
     "node-pty": "^1.1.0"
+  },
+  "optionalDependencies": {
+    "create-dmg": "^8.1.0"
   }
 }

package/scripts/patch-electron-name.mjs

@@ -29,36 +29,45 @@ function main() {
   const oldExe = join(macosDir, "Electron");
   const newExe = join(macosDir, APP_NAME);
   const pathTxt = join(root, "path.txt");
-  if (!existsSync(plistPath)) return;
+  if (!existsSync(plistPath)) {
+    console.warn('monacori: Electron.app not found at ' + appDir + ' — skipping rebrand (Dock/menu may show "Electron")');
+    return;
+  }
 
   try {
+    let changed = false;
     // 1. Bundle metadata: name, display name, AND executable -> monacori.
     const before = readFileSync(plistPath, "utf8");
     const after = before
       .replace(/(<key>CFBundleName<\/key>\s*<string>)[^<]*(<\/string>)/, "$1" + APP_NAME + "$2")
       .replace(/(<key>CFBundleDisplayName<\/key>\s*<string>)[^<]*(<\/string>)/, "$1" + APP_NAME + "$2")
       .replace(/(<key>CFBundleExecutable<\/key>\s*<string>)[^<]*(<\/string>)/, "$1" + APP_NAME + "$2");
-    if (after !== before) writeFileSync(plistPath, after);
+    if (after !== before) { writeFileSync(plistPath, after); changed = true; }
 
     // 2. Rename the executable so the directly-spawned process is "monacori" (idempotent).
-    if (existsSync(oldExe) && !existsSync(newExe)) renameSync(oldExe, newExe);
+    if (existsSync(oldExe) && !existsSync(newExe)) { renameSync(oldExe, newExe); changed = true; }
 
     // 3. Repoint electron's path.txt at the renamed binary so require("electron") resolves it.
     if (existsSync(pathTxt)) {
       const pt = readFileSync(pathTxt, "utf8");
       const fixed = pt.replace("MacOS/Electron", "MacOS/" + APP_NAME);
-      if (fixed !== pt) writeFileSync(pathTxt, fixed);
+      if (fixed !== pt) { writeFileSync(pathTxt, fixed); changed = true; }
     }
-    // Refresh LaunchServices so the Dock / Cmd+Tab show "monacori" instead of a cached "Electron".
-    // Without this, macOS keeps the previously-registered bundle name even after the plist is patched.
-    spawnSync(
-      "/System/Library/Frameworks/CoreServices.framework/Versions/A/Frameworks/LaunchServices.framework/Versions/A/Support/lsregister",
-      ["-f", appDir],
-      { stdio: "ignore" },
-    );
-    console.log('monacori: branded Electron app + executable as "' + APP_NAME + '"');
-  } catch {
-    // read-only / permission-denied environments — harmless, this is a convenience step
+
+    // Only when something actually changed: refresh LaunchServices so the Dock / Cmd+Tab show "monacori"
+    // instead of a cached "Electron". Skipping it when already-branded keeps the startup re-run cheap.
+    if (changed) {
+      spawnSync(
+        "/System/Library/Frameworks/CoreServices.framework/Versions/A/Frameworks/LaunchServices.framework/Versions/A/Support/lsregister",
+        ["-f", appDir],
+        { stdio: "ignore" },
+      );
+      console.log('monacori: branded Electron app + executable as "' + APP_NAME + '"');
+    }
+  } catch (e) {
+    // Surface the reason (perms / read-only) instead of failing SILENTLY — otherwise the Dock/Cmd+Tab/menu
+    // keep showing "Electron" with no hint why. Non-fatal: app-main.ts re-runs this at startup.
+    console.warn('monacori: could not rebrand the Electron app to "' + APP_NAME + '". Dock/Cmd+Tab/menu may stay "Electron". Reason: ' + (e && e.message ? e.message : e));
   }
 }