discord.js - Imagine an app•5w ago

Problem with channel fetching

Hello. So Ive been fighting with this for a month now and I don't know where issue is, so im trying my luck here. I have made a reminders system. Reminders has channel ID and user ID saved in db. I have functions running every 5 seconds that check if there are any reminders to send. This function fetches the discord user, this works fine and as intendent. But problem I have is with the channel fetch. I'm using .broadcastEval function, inside first I try to fetch the channel from cache, if this fails, it try to fetch normally from API. But sometimes this process fails and no shards could find that channel and rather send the reminder into DM. The problem is, that the channel is same in both cases. I tried to use the console logs for debugging, but its just logs that none of the shards could find the channel, but for next reminder with same channel, it could find the channel perfectly fine. What can be issue here? Or what other alternatives can I try? Someone told me that I shouldnt be fetching channels when using sharding, but then how can I make this reminder system possible?

const results = await client.shard.broadcastEval(
        async (client, { channelId, reminderText, userId }) => {
          try {
            // Try to get the channel from cache first
            let channel = client.channels.cache.get(channelId);

            // If not in cache, try to fetch
            if (!channel) {
              try {
                channel = await client.channels.fetch(channelId, { force: true }).catch(() => null);
              } catch (error) {
                return { success: false, error: error.message, shardId: client.shard.ids[0] };
              }
            }

            // If channel not found or not accessible by this shard, report failure
            if (!channel) {
              return { success: false, reason: 'Channel not found', shardId: client.shard.ids[0] };
            }

            // Check permissions - check for DM channels without using ChannelType enum
            if (
              channel.type === 'DM' ||
              channel.isDMBased?.() ||
              channel.permissionsFor?.(client.user)?.has(PermissionsBitField.Flags.SendMessages)
            ) {
              // Fetch user to mention them properly in the channel
              const user = await client.users.fetch(userId).catch(() => null);
              if (!user) {
                return { success: false, reason: 'User not found', shardId: client.shard.ids[0] };
              }

              // Send the reminder
              await channel.send(`${user}, here is your reminder: "${reminderText}"`);
              return { success: true, shardId: client.shard.ids[0] };
            } else {
              return {
                success: false,
                reason: 'Missing permissions',
                shardId: client.shard.ids[0],
              };
            }
          } catch (error) {
            return { success: false, error: error.message, shardId: client.shard.ids[0] };
          }
        },
        {
          context: {
            channelId: reminder.channel,
            reminderText: reminder.reminder,
            userId: user.id,
          },
        },
      );

const results = await client.shard.broadcastEval(
        async (client, { channelId, reminderText, userId }) => {
          try {
            // Try to get the channel from cache first
            let channel = client.channels.cache.get(channelId);

            // If not in cache, try to fetch
            if (!channel) {
              try {
                channel = await client.channels.fetch(channelId, { force: true }).catch(() => null);
              } catch (error) {
                return { success: false, error: error.message, shardId: client.shard.ids[0] };
              }
            }

            // If channel not found or not accessible by this shard, report failure
            if (!channel) {
              return { success: false, reason: 'Channel not found', shardId: client.shard.ids[0] };
            }

            // Check permissions - check for DM channels without using ChannelType enum
            if (
              channel.type === 'DM' ||
              channel.isDMBased?.() ||
              channel.permissionsFor?.(client.user)?.has(PermissionsBitField.Flags.SendMessages)
            ) {
              // Fetch user to mention them properly in the channel
              const user = await client.users.fetch(userId).catch(() => null);
              if (!user) {
                return { success: false, reason: 'User not found', shardId: client.shard.ids[0] };
              }

              // Send the reminder
              await channel.send(`${user}, here is your reminder: "${reminderText}"`);
              return { success: true, shardId: client.shard.ids[0] };
            } else {
              return {
                success: false,
                reason: 'Missing permissions',
                shardId: client.shard.ids[0],
              };
            }
          } catch (error) {
            return { success: false, error: error.message, shardId: client.shard.ids[0] };
          }
        },
        {
          context: {
            channelId: reminder.channel,
            reminderText: reminder.reminder,
            userId: user.id,
          },
        },
      );

discord.js - 14.14.1 nodejs - 20.15.1

12 Replies

d.js toolkit•5w ago

- What's your exact discord.js npm list discord.js and node node -v version? - Not a discord.js issue? Check out #other-js-ts. - Consider reading #how-to-get-help to improve your question! - Explain what exactly your issue is. - Post the full error stack trace, not just the top part! - Show your code! - Issue solved? Press the button! - ✅ Marked as resolved by OP

Syjalo•5w ago

Fetching doesn't depend on the shard. It will fetch the same channel from the REST API for all shards Make it simple. You can fetch the user, just in one shard. Then you can use user.createDM() to get the channel.

d.js docs•5w ago

:method: User#createDM() [email protected] Creates a DM channel between the client and the user.

GattiOP•5w ago

thanks ill check it out But looking at it right now, this isnt really what i wanted. This createDM works only for DMs am i wrong? What I wanted is to send the reminder into the channel where reminder was created using channel ID This DMing user is just if the channel fetching fails for example, if the bot doesnt have permission or other things but as I said the problem is shards sometimes fetch the channel and sometimes dont. And we talking about same channel or if i dont use the broadcast eval function at all and directly fetch the channel will it work? I just tried it and it doesnt work Okay after some investigation, I got this as error: Failed to fetch channel DiscordAPIError[50001]: Missing Access But how can it be missing access when its same channel that it accessed before?

NyR•5w ago

Show the full error, the bot may not have permission to view the channel. Also the way you are getting the channel is wrong. The channel will only be cache on the shard that handles it. So for all the other shard, it won't be at that point you try to fetch the channel which will return an actual channel since it returns from the api. Essentially duplicating the send action for all the shard Hence why it is not recommended to fetch channels when sharding

GattiOP•5w ago

In the other part Im running that function only on one shard

NyR•5w ago

broadcastEval will run that on all shard though

GattiOP•5w ago

But, I managed to fix it by not using functions from discordjs and rather made function that directly using discord API to send the message to the channel

NyR•5w ago

Then you should show your updated code and also the full error you get

GattiOP•5w ago

yeah i realized that after i sent this, but in this context it doesnt really matter I put there console log, to show the error and this is what i got: Failed to fetch channel DiscordAPIError[50001]: Missing Access I just used the post method /channels/{channelId}/messages

NyR•5w ago

The error could be due to not having ViewChannel permission. Verify bot actually has it. Permissions are implicit on discord, SendMessages will be denied when bot can't see the channel

GattiOP•5w ago

I stated multiple times that it was tested on same channel, same channel id. One person got that reminder in the channel other didnt, and they dont the reminder command in the same channel

Gaming

Programming

Problem with channel fetching

Did you find this page helpful?