@glassnote/client 2.4.6 → 2.4.7

package/bin/glassnote.js

@@ -27,8 +27,15 @@ const MARK_B = '# <<< glassnote <<<';
 // El PATH que se le da a npm lleva delante nuestro bin: si install.sh tuvo que bajar
 // Node, sus enlaces viven ahí y de otro modo npm no existiría para este proceso.
 function npmEnv() {
+	// El directorio del node va en el PATH, no solo BIN_DIR: npm lanza los scripts de
+	// instalacion por cmd.exe (`cmd /c node install.js`, el postinstall de electron) y
+	// esos hijos buscan `node` en el PATH. Sin esto la actualizacion moria a medias con
+	// «"node" no se reconoce como un comando», dejando el paquete sin binario.
+	const nodeBinary = mode.resolveNodeBinary(process.execPath);
+	const dirs = [mode.BIN_DIR];
+	if (nodeBinary) dirs.push(path.dirname(nodeBinary));
 	return Object.assign({}, process.env, {
-		PATH: `${mode.BIN_DIR}${path.delimiter}${process.env.PATH || ''}`,
+		PATH: `${dirs.join(path.delimiter)}${path.delimiter}${process.env.PATH || ''}`,
 		// Cache propia, bajo ~/.glassnote. Un ~/.npm con archivos de root —herencia de
 		// cualquier `sudo npm` viejo— hace que npm muera con EACCES antes de bajar nada,
 		// y el usuario tiene que andar con chown. Se pisa a propósito el npm_config_cache
@@ -103,6 +110,24 @@ function sleepSync(ms) {
 	Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, ms);
 }
 
+// Dormir un rato fijo no alcanzaba. En Windows npm no puede borrar el árbol mientras
+// quede vivo UN proceso del cliente, y electron cierra media docena de procesos hijos:
+// con los 4 segundos de antes npm entraba con la app a medio cerrar y salía con
+// «EPERM: rmdir ...\node_modules\got\dist», dejando la instalación rota. Se espera a
+// que el servidor local del cliente deje de responder, que es la señal de que ya no está.
+function esperarAQueCierre(maxMs) {
+	const tope = Date.now() + Math.max(maxMs, 30000);
+	while (Date.now() < tope) {
+		if (!clienteEnMarcha()) {
+			sleepSync(1500); // margen para que el SO suelte los handles
+			return true;
+		}
+		sleepSync(1000);
+	}
+	console.error('glassnote: el cliente sigue en marcha; se actualiza igual y puede fallar.');
+	return false;
+}
+
 const log = (...args) => console.log(...args);
 // Cada paso se dice EN CUANTO EMPIEZA: lo que hace que un instalador parezca colgado
 // no es que tarde, es que tarde sin decir en qué está.
@@ -532,7 +557,7 @@ function cmdUpdate(args) {
 	);
 	const version = positional[0] || 'latest';
 
-	if (waitMs > 0) sleepSync(waitMs);
+	if (waitMs > 0) esperarAQueCierre(waitMs);
 
 	// El cwd no puede estar dentro de lo que npm va a borrar: en Windows un directorio
 	// que es el cwd de un proceso vivo no se puede eliminar, y ese proceso es este.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glassnote/client",
-  "version": "2.4.6",
+  "version": "2.4.7",
   "description": "GlassNote — cliente de escritorio de notas superpuestas. Se instala con npx, sin instaladores nativos ni firmas.",
   "main": "main.js",
   "publishConfig": {