dankgrinder 8.113.0 → 8.114.0

package/lib/commands/generic.js

@@ -50,10 +50,9 @@ async function waitForEditedMessage(channel, messageId, baselineText, timeoutMs
  * @param {object} [opts.client]
  * @returns {Promise<{result: string, coins: number}>}
  */
-async function runGeneric({ channel, waitForDankMemer, cmdString, cmdName, client, useSlash, prefix = 'pls' }) {
+async function runGeneric({ channel, waitForDankMemer, cmdString, cmdName, client, useSlash = false, prefix = 'pls' }) {
   LOG.cmd(`${c.white}${c.bold}${cmdString}${c.reset}`);
-  // cmdString = "pls farm view" or "/farm view"
-  // Build slash command: strip prefix → "farm view" → sendSlash('farm view')
+  // Strip prefix from cmdString to get just "command subcmd"
   const slashName = cmdString
     .replace(/^pls\s+/, '')
     .replace(/^\/\s*/, '')

package/lib/commands/utils.js

@@ -65,20 +65,21 @@ async function sendCommand(channel, commandName, opts = {}) {
     args = [],
   } = opts;
 
+  // Only use slash if explicitly enabled
   if (useSlash && channel?.sendSlash) {
     try {
-      // sendSlash takes botId, commandName, ...args
       await channel.sendSlash(DANK_MEMER_ID, commandName, ...args);
       return;
     } catch (e) {
-      LOG.warn(`[sendSlash] ${commandName} failed: ${e.message} — falling back to text`);
+      const msg = e instanceof Error ? e.message : String(e);
+      LOG.warn(`[sendSlash] ${commandName} failed: ${msg} — falling back to text`);
     }
   }
 
-  // Fallback: text command
+  // Fallback: always use pls prefix unless slash is explicitly enabled
   const textCmd = args.length > 0
-    ? `${prefix} ${commandName} ${args.join(' ')}`
-    : `${prefix} ${commandName}`;
+    ? `pls ${commandName} ${args.join(' ')}`
+    : `pls ${commandName}`;
   await channel.send(textCmd);
 }
 

package/lib/grinder.js

@@ -1096,7 +1096,7 @@ class AccountWorker {
   }
 
   async checkBalance(silent = false) {
-    const prefix = this.account.use_slash ? '/' : 'pls';
+    const prefix = !!this.account.use_slash ? '/' : 'pls';
     const sentAt = Date.now();
 
     const looksLikeBalance = (t) => {
@@ -1107,7 +1107,7 @@ class AccountWorker {
     };
 
     // Fast path: send command, wait 3s, read from rawLogger
-    if (this.account.use_slash && this.channel?.sendSlash) {
+    if (!!this.account.use_slash && this.channel?.sendSlash) {
       await this.channel.sendSlash(DANK_MEMER_ID, 'balance').catch(() => this.channel.send('/balance'));
     } else {
       await this.channel.send(`${prefix} bal`);
@@ -1126,7 +1126,7 @@ class AccountWorker {
     // If rawLogger didn't capture it, fall back to waitForDankMemer
     if (!text || !looksLikeBalance(text)) {
       let response = await this.waitForDankMemer(8000);
-      if (!response && this.account.use_slash) {
+      if (!response && !!this.account.use_slash) {
         await this.channel.send('pls bal');
         response = await this.waitForDankMemer(8000);
       }
@@ -1401,7 +1401,7 @@ class AccountWorker {
                    cmdName === 'crime' ? safeParseJSON(this.account.crime_answers, []) : [],
       adventureAnswers: AccountWorker.ADVENTURE_ANSWERS,
       useSlash: !!this.account.use_slash,
-      prefix: this.account.use_slash ? '/' : 'pls',
+      prefix: !!this.account.use_slash ? '/' : 'pls',
       betAmount: this._questBetOverride || (['blackjack'].includes(cmdName) ? bjBet : gambBet),
       accountId: this.account.id,
       redis,
@@ -2458,7 +2458,7 @@ class AccountWorker {
       } else {
         this._questBetOverride = quest.bet || null;
       }
-      const prefix = this.account.use_slash ? '/' : 'pls';
+      const prefix = !!this.account.use_slash ? '/' : 'pls';
 
       if (quest.loseTarget) {
         // Loss-target quest: keep playing until cumulative losses >= loseTarget
@@ -2742,7 +2742,7 @@ class AccountWorker {
       await new Promise(r => setTimeout(r, minGap - timeSinceLastCmd));
     }
 
-    const prefix = this.account.use_slash ? '/' : 'pls';
+    const prefix = !!this.account.use_slash ? '/' : 'pls';
     this.setStatus(formatCommandName(item.cmd));
 
     // Report "running" to dashboard
@@ -2869,7 +2869,7 @@ class AccountWorker {
         if (!this.busy) {
           this.log('info', 'Alert detected → running alert');
           this.busy = true;
-          const prefix = this.account.use_slash ? '/' : 'pls';
+          const prefix = !!this.account.use_slash ? '/' : 'pls';
           this.runCommand('alert', prefix).finally(() => { this.busy = false; });
         }
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dankgrinder",
-  "version": "8.113.0",
+  "version": "8.114.0",
   "description": "Dank Memer automation engine — grind coins while you sleep",
   "bin": {
     "dankgrinder": "bin/dankgrinder.js"