@glassnote/client 2.4.5 → 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á.
@@ -204,10 +229,10 @@ function addToPathUnix() {
 	return { touched, line };
 }
 
-function addToPathWindows() {
+function addToPathWindows(dir = mode.BIN_DIR) {
 	// Nada de `setx`: trunca el PATH a 1024 caracteres y se come el de la gente.
 	const ps = [
-		"$dir = '" + mode.BIN_DIR.replace(/'/g, "''") + "'",
+		"$dir = '" + dir.replace(/'/g, "''") + "'",
 		"$cur = [Environment]::GetEnvironmentVariable('Path','User')",
 		"if ($null -eq $cur) { $cur = '' }",
 		"if (($cur -split ';') -notcontains $dir) {",
@@ -220,7 +245,22 @@ function addToPathWindows() {
 		encoding: 'utf8',
 	});
 	const added = res.status === 0 && String(res.stdout).includes('added');
-	return { touched: added ? ['PATH de usuario (sin administrador)'] : [], line: mode.BIN_DIR };
+	return { touched: added ? ['PATH de usuario (sin administrador)'] : [], line: dir };
+}
+
+// El shim `glassnote.cmd` que crea npm invoca `node` del PATH. Cuando el Node lo bajó el
+// instalador a ~/.glassnote, ese directorio solo quedaba en el PATH de la sesión de
+// PowerShell que corrió la instalación: en una terminal nueva —o en cualquier proceso
+// posterior— `glassnote` moría con «"node" no se reconoce como un comando». Se agrega el
+// directorio del node al PATH de usuario, igual que BIN_DIR.
+function addNodeToPathWindows() {
+	if (!IS_WIN) return null;
+	const nodeBinary = mode.resolveNodeBinary(process.execPath);
+	// Solo el node que trajimos nosotros: el del sistema ya está donde tiene que estar.
+	if (!nodeBinary || !mode.isInside(nodeBinary, mode.HOME_DIR)) return null;
+	const dir = path.dirname(nodeBinary);
+	const res = addToPathWindows(dir);
+	return res.touched.length ? dir : null;
 }
 
 function removeFromPathUnix() {
@@ -253,7 +293,10 @@ function removeFromPathWindows() {
 		"$dir = '" + mode.BIN_DIR.replace(/'/g, "''") + "'",
 		"$cur = [Environment]::GetEnvironmentVariable('Path','User')",
 		"if ($null -eq $cur) { Write-Output 'absent'; exit }",
-		"$new = (($cur -split ';') | Where-Object { $_ -ne '' -and $_ -ne $dir }) -join ';'",
+		// Se quita cualquier entrada bajo ~/.glassnote, no solo BIN_DIR: el install tambien
+		// agrega el directorio del node que bajo, y dejarlo seria una ruta muerta.
+		"$home2 = '" + mode.HOME_DIR.replace(/'/g, "''") + "'",
+		"$new = (($cur -split ';') | Where-Object { $_ -ne '' -and $_ -ne $dir -and -not $_.StartsWith($home2, [StringComparison]::OrdinalIgnoreCase) }) -join ';'",
 		'if ($new -ne $cur) {',
 		"  [Environment]::SetEnvironmentVariable('Path', $new, 'User')",
 		"  Write-Output 'removed'",
@@ -457,6 +500,8 @@ function cmdInstall(args) {
 	if (!noPath) {
 		pathInfo = IS_WIN ? addToPathWindows() : addToPathUnix();
 		if (pathInfo.touched.length) step(`agregado al PATH en: ${pathInfo.touched.join(' ')}`);
+		const nodeDir = addNodeToPathWindows();
+		if (nodeDir) step(`agregado al PATH: ${nodeDir} (lo necesita el comando glassnote)`);
 	}
 
 	log('');
@@ -512,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.5",
+  "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": {