kaogeek / kaogeek-discord-bot Goto Github PK

Overview

Proposed steps:

1. When user is flagged, we save those user to flag table in db
2. If user is rejoin, we check against the flag table and reapply the role

• When user is unflagged, remove those user from flag table

Why you want this feature?

*This is just a quick fix for current issue*

Currently, there are some flagged users making themselves unflagged by leaving the server and rejoining.
Admins suggest that, we should reapply flag role to those who leave with the role.

Describe the bug

I plan to test command but it's already merge 😂

Anyways, my main account has DM disable and it's crash the bot.

Log:

Sticky message saved: Hi
error while update sticky message Unknown Message
\kaogeek-discord-bot\node_modules\.pnpm\@[email protected]\node_modules\@discordjs\rest\dist\index.js:640
      throw new DiscordAPIError(data, "code" in data ? data.code : data.error, status, method, url, requestData);
            ^


DiscordAPIError[50007]: Cannot send messages to this user
    at handleErrors (\kaogeek-discord-bot\node_modules\.pnpm\@[email protected]\node_modules\@discordjs\rest\dist\index.js:640:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.runRequest (\kaogeek-discord-bot\node_modules\.pnpm\@[email protected]\node_modules\@discordjs\rest\dist\index.js:1021:23)
    at Object.queueRequest (\kaogeek-discord-bot\node_modules\.pnpm\@[email protected]\node_modules\@discordjs\rest\dist\index.js:862:14)
    at EventEmitter.request (\kaogeek-discord-bot\node_modules\.pnpm\@[email protected]\node_modules\@discordjs\rest\dist\index.js:1387:22)
    at DMChannel.send (\kaogeek-discord-bot\node_modules\.pnpm\[email protected]\node_modules\discord.js\src\structures\interfaces\TextBasedChannel.js:162:15) {
  requestBody: {
    files: [],
    json: {
      content: 'Successfully created sticky message.',
      tts: false,
      nonce: undefined,
      embeds: undefined,
      components: undefined,
      username: undefined,
      avatar_url: undefined,
      allowed_mentions: undefined,
      flags: undefined,
      message_reference: undefined,
      attachments: undefined,
      sticker_ids: undefined,
      thread_name: undefined
    }
  },
  rawError: { message: 'Cannot send messages to this user', code: 50007 },
  code: 50007,
  status: 403,
  method: 'POST',
  url: 'https://discord.com/api/v10/channels/.../messages'
}

Node.js v18.16.0
 ELIFECYCLE  Command failed with exit code 1.

To Reproduce

Steps to reproduce the behavior:

1. Disable dm in discord in Privacy & Safety
   
2. try set message using ?stickao-set
3. Done, your bot will now crash

Expected behavior

Just skip DM if user disabled it, maybe.

Additional context

@samithiwat

สร้างโครงสร้างพื้นฐานสำหรับการรันบอท discord

https://github.com/kaogeek/kaogeek-discord-bot/blob/main/data/dock/db.sqlite

It looks unused to me. Someone should double check and remove.

Let's initialize the TypeScript code for the discord bot.

PS - you can create a Draft pull request to prevent duplicate work.

I would like to introduce frameworks for implementing Discord bot that is discord-nestjs. It allows us to implement the bot based on NestJS framework which provides greats decorators and separations of layers of logics. We can also implement REST api in it if we want.

Overview

migrate data storage from Keyv to Prisma

• no need to migrate data (just discard old data and use a new database file)
• use SQLite for now

Why you want this feature?

in the future we may have more complex feature that needs more structured data.

(please comment if you want to do it, so other people know someone is already working on it)

https://github.com/creatorsgarten/kaogeek-discord-bot/blob/975d08b08b95f7a089ff3ea4a67988954d3f44f0/src/events/interactionCreate.ts#L15-L17

refs:
https://discordjs.guide/interactions/modals.html#building-and-responding-with-modals
https://stackoverflow.com/questions/74444213/discord-js-interaction-already-replied-error-while-sending-modal

the isOnlyEmoji function currently has a bug. it detects numbers (e.g. 123) as an emoji. this leads to a bug:

• #99

a test case has been added:

https://github.com/creatorsgarten/kaogeek-discord-bot/blob/2c204aa04871b0a3e527d781e76e1f8119e9b0e0/test/spec/utils/isOnlyEmoji.spec.ts#L24-L26

please fix the function so that this test passes

Overview

add Vitest to project to allow unit testing some logic

• add Vitest to project
• add a dummy test to verify that Vitest is working, e.g. expect('foo').toBe('foo')
• add a CI configuration to run Vitest

Why you want this feature?

there are some complex logic, e.g. emoji detection #36. maybe these logic can have automated tests to increase confidence in the system

(please comment if you would like to help, so other people know someone is already working on it)

Overview

1. when set message for sticky in chanel
2. someone send message in chanel >> sticky message will delete and send sticky messages to the bottom always.

Why you want this feature?

1. Free StickyBot Free Version can set 3 message

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Rate-Limited

These updates are currently rate-limited. Click on a checkbox below to force their creation now.

• chore(deps): update actions/checkout action to v4
• chore(deps): update actions/setup-node action to v4
• chore(deps): update appleboy/ssh-action action to v1
• chore(deps): update dependency @types/node to v20
• chore(deps): update dependency eslint-config-prettier to v9
• chore(deps): update dependency eslint-plugin-prettier to v5
• chore(deps): update dependency eslint-plugin-unicorn to v51
• chore(deps): update dependency prettier to v3
• chore(deps): update dependency tsup to v8
• chore(deps): update dependency tsx to v4
• chore(deps): update docker/build-push-action action to v5
• chore(deps): update docker/login-action action to v3
• chore(deps): update docker/setup-buildx-action action to v3
• chore(deps): update typescript-eslint monorepo to v7 (major) (@typescript-eslint/eslint-plugin, @typescript-eslint/parser)
• chore(deps): update vitest monorepo to v1 (major) (@vitest/coverage-v8, vitest)
• fix(deps): update prisma monorepo to v5 (major) (@prisma/client, prisma)
• chore(deps): lock file maintenance
• 🔐 Create all rate-limited PRs at once 🔐

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• fix(deps): update dependencies (non-major) (@prisma/client, @trivago/prettier-plugin-sort-imports, @types/eslint, @types/humanize-ms, @types/node, @types/node-cron, @types/unzalgo, @typescript-eslint/eslint-plugin, @typescript-eslint/parser, @vitest/coverage-v8, discord.js, dotenv, eslint, eslint-config-prettier, eslint-plugin-import, node-cron, pnpm, prisma, rimraf, tsup, tsx, typescript, vite-tsconfig-paths, vitest, zod)
• chore(deps): update dependency @tsconfig/node18 to v18

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

Discussed in #24

I think it's unnecessary if disableAutoDeferReply is true

Overview

find a way to let developers see the server logs without having to ssh into a server, so they can better debug the bot

Why you want this feature?

right now when debugging issues, to view the logs, i have to ssh into @Gusb3ll’s server and run docker compose logs -f

this means only a handful people are able to debug the system because only few people can access log. giving too many people ssh access can be a security concern. therefore, if we are able to use some hosted log management solution, more people can debug the system

Suggestions

• my personal favorite would be Papertrail. it is a simple text-based logging system with a free tier. very easy to filter logs using text patterns. the free tier has 50MB/month limit, but each referral would grant an extra 50MB/month for 1 year. i am currently testing out Papertrail with my account, but i am using it with other projects too. it would be great if someone can create a new account so we can have a dedicated account for the bot (let me know so I can send the referral code). since Papertrail supports unlimited team members, we don’t need to share account passwords.

  

• another option is to use Loggly. its free plan has 200MB/day limit. however, the UI does not look as good as Papertrail. it also emphasizes structured logging, which is not what we are doing right now. but it is still a good option to consider given a much higher free tier limit.

• any other suggestions?

cc: @Gusb3ll @ST4RCHASER

Describe the bug

morning krubb refer to #141
handle more cases : when users change their minds to disable DMs

To Reproduce

Steps to reproduce the behavior:

1. turn on setting to allow bot DM
2. use stickao-set
3. turn off setting
4. use stickao-set again

Expected behavior

skip DM

Screenshots

Additional context

Overview

Some users in ก้าว Geek's discord trying to modify their nickname to start with/only "!" to be sorted at the top of the user list. After this feature rollout, some users try to replace "!" with other special characters are the same range of ASCII code (e.g., $, %, *, etc.). So blocking only the exclamation mark ("!") is not enough.

I have a suggestion for blocking all special characters (ASCII code from 33 to 47) from the nickname with the mentioned condition.

Why you want this feature?

Enhance nickname validation function more effectively according to an objective.

According to #84 (comment)

It's weird that .DS_Store is not there in .gitignore template 😂

To add to .gitignore

• .DS_Store and __MACOSX
• package-lock.json because someone insists to use npm (We already have yarn.lock in .gitignore, so group them together will be very nice)