windsor-bot 0.2.6 → 0.2.8

package/dist/bot.js

@@ -211,12 +211,15 @@ export function createWindsorBot() {
             const config = getReusableConfig(message);
             if (!config)
                 return;
-            if ([...message.reactions.cache.values()].some(candidate => candidate.me))
-                return;
-            if (reusableStates.has(message.id))
+            const emoji = reaction.emoji.name ?? reaction.emoji.id ?? 'unknown';
+            logEvent('info', `Reusable reaction added: message=${message.id}, channel=${message.channelId}, emoji=${emoji}, user=${user.id}`);
+            if (reusableStates.has(message.id)) {
+                logEvent('info', `Ignoring reusable reaction for message=${message.id}; print already in progress`);
                 return;
+            }
             const state = { reaction, cancelled: false, status: 'printing' };
             reusableStates.set(message.id, state);
+            logEvent('info', `Starting reusable print for message=${message.id}`);
             await handleReusablePrint(message, config, state);
         }
         catch (err) {
@@ -237,6 +240,10 @@ export function createWindsorBot() {
                 await removeReactionSafe(message, Reaction.waiting);
                 if (state.status !== 'printing')
                     reusableStates.delete(message.id);
+                logEvent('info', `Reusable reaction removed: message=${message.id}; cancelled ${state.status} print`);
+            }
+            else {
+                logEvent('info', `Reusable reaction removed: message=${message.id}; no active print`);
             }
             await removeOwnReactionSafe(reaction);
         }
@@ -631,13 +638,13 @@ export function createWindsorBot() {
                 state.status = 'pending';
                 await reactSafe(message, Reaction.waiting);
                 enqueuePendingPrint(message, () => handleReusablePrint(message, config, state));
-                logEvent('info', `Printer unavailable for reusable message in #${message.channelId}`);
+                logEvent('info', `Printer unavailable for reusable message=${message.id} in #${message.channelId}; waiting for retry`);
                 return;
             }
             reusableStates.delete(message.id);
             await removeReactionSafe(message, Reaction.waiting);
             await replySafe(message, `⏸️ Print failed: ${err instanceof Error ? err.message : String(err)}`);
-            logEvent('error', `Reusable List print failed: ${err}`);
+            logEvent('error', `Reusable List print failed: message=${message.id}: ${err}`);
             return;
         }
         pendingPrints.delete(message.id);
@@ -648,7 +655,7 @@ export function createWindsorBot() {
         }
         await message.react(state.reaction.emoji);
         state.status = 'printed';
-        logEvent('print', `Printed reusable message from ${message.author.username}`);
+        logEvent('print', `Printed reusable message=${message.id} from ${message.author.username}; acknowledgement reaction added`);
     }
     async function retryFailedMessages(message) {
         const mapping = getCurrentConfig().channels.find(m => m.channelId === message.channelId);

package/dist/server.js

@@ -1,7 +1,7 @@
 import { createServer } from 'http';
 import { execFile } from 'child_process';
 import { build } from 'esbuild';
-import { access, readFile, stat } from 'fs/promises';
+import { access, constants, readFile, stat } from 'fs/promises';
 import { promisify } from 'util';
 import { spawn } from 'child_process';
 import { dirname, join } from 'path';
@@ -65,7 +65,7 @@ async function readPackageVersion() {
     return packageJson.version;
 }
 async function updateGlobalBot() {
-    const npmCandidates = [
+    const configuredCandidates = [
         process.env['npm_execpath'],
         join(dirname(process.execPath), '..', 'lib', 'node_modules', 'npm', 'bin', 'npm-cli.js'),
         join(dirname(process.execPath), 'npm'),
@@ -76,11 +76,17 @@ async function updateGlobalBot() {
         '/usr/local/bin/npm',
         '/opt/homebrew/bin/npm',
         '/usr/bin/npm',
-        'npm',
     ]
         .filter((candidate) => Boolean(candidate))
         .map(candidate => candidate.trim())
         .filter((candidate, index, candidates) => candidates.indexOf(candidate) === index);
+    const pathCandidates = (process.env['PATH'] ?? '')
+        .split(':')
+        .filter(Boolean)
+        .map(directory => join(directory, 'npm'));
+    const npmCandidates = [...configuredCandidates, ...pathCandidates]
+        .filter((candidate, index, candidates) => candidates.indexOf(candidate) === index);
+    logEvent('info', `Update requested; checking ${npmCandidates.length} npm candidate(s) (node=${process.execPath}, cwd=${process.cwd()})`);
     let npmCommand;
     for (const candidate of npmCandidates) {
         const command = candidate.endsWith('.js')
@@ -88,19 +94,40 @@ async function updateGlobalBot() {
             : { path: candidate, args: [] };
         try {
             // Validate the script itself, not just the Node executable used to run it.
-            if (candidate !== 'npm')
-                await access(candidate);
+            await access(candidate, constants.R_OK | (candidate.endsWith('.js') ? 0 : constants.X_OK));
             npmCommand = command;
+            logEvent('info', `Using npm candidate ${candidate}${command.args.length > 0 ? ` via ${command.path}` : ''}`);
             break;
         }
-        catch {
-            // Try the next known npm installation location.
+        catch (error) {
+            const code = error instanceof Error && 'code' in error ? String(error.code) : 'unknown';
+            logEvent('info', `npm candidate unavailable: ${candidate} (${code})`);
         }
     }
-    if (!npmCommand)
-        throw new Error('Unable to locate npm');
+    if (!npmCommand) {
+        const path = process.env['PATH'] ?? '(unset)';
+        logEvent('error', `Unable to locate an executable npm; PATH=${path}`);
+        throw new Error('Unable to locate an executable npm; see diagnostics');
+    }
     const npm = npmCommand;
-    const runNpm = (args) => execFileAsync(npm.path, [...npm.args, ...args], { env: process.env });
+    const runNpm = async (args) => {
+        const commandArgs = [...npm.args, ...args];
+        logEvent('info', `Running npm: ${npm.path} ${commandArgs.join(' ')}`);
+        try {
+            const result = await execFileAsync(npm.path, commandArgs, { env: process.env });
+            logEvent('info', `npm completed successfully: ${args.join(' ')}`);
+            return result;
+        }
+        catch (error) {
+            const details = error instanceof Error ? error : new Error(String(error));
+            const code = 'code' in details ? String(details.code) : 'unknown';
+            const stderr = 'stderr' in details && typeof details.stderr === 'string'
+                ? details.stderr.trim().slice(-1000)
+                : '';
+            logEvent('error', `npm failed (code=${code}, path=${npm.path}, args=${commandArgs.join(' ')})${stderr ? ` stderr=${stderr}` : ''}`);
+            throw details;
+        }
+    };
     await runNpm(['install', '-g', 'windsor-bot@latest']);
     const { stdout } = await runNpm(['root', '-g']);
     const packageJsonPath = join(stdout.trim(), 'windsor-bot', 'package.json');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "windsor-bot",
-  "version": "0.2.6",
+  "version": "0.2.8",
   "description": "Self-hosted Discord bot automation for household list and todo printing",
   "type": "module",
   "main": "dist/index.js",