Skip to content

🚀 release: v0.1.20 alpha #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Apr 1, 2025
Merged

🚀 release: v0.1.20 alpha #8

merged 15 commits into from
Apr 1, 2025

Conversation

warengonzaga
Copy link
Member

This pull request includes several updates to improve documentation, add new features, and enhance existing functionalities. The most important changes include adding a code of conduct, contributing guidelines, and security policy, updating command implementations, and modifying the project version.

Documentation Enhancements:

  • Added a comprehensive code_of_conduct.md file to establish community guidelines and standards.
  • Introduced a contributing.md file to guide contributors on how to participate in the project.
  • Added a security.md file to outline the process for reporting security vulnerabilities.

Command Improvements:

  • Enhanced the ping.js command to display bot latency and API ping metrics using an embedded message.
  • Updated the server.js command to provide detailed server information in an embedded format.
  • Improved the user.js command to show detailed user information with embedded messages.
  • Added a new version.js command to display the current bot version retrieved from package.json.

Project Version Update:

  • Updated the project version in package.json from 0.1.8 to 0.1.20-alpha.

@warengonzaga warengonzaga added release Release Status (PR) maintainers only Assigned to Project Maintainers (Issue/PR) labels Apr 1, 2025
@warengonzaga warengonzaga self-assigned this Apr 1, 2025
@warengonzaga warengonzaga requested a review from Copilot April 1, 2025 12:20
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request enhances documentation and refines bot functionalities by updating commands and the Unthread service. Key changes include the addition of new documentation files, improvements to the Unthread service with polling logic for ticket creation, and the conversion of several commands to use embedded messages for a richer user experience.

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/services/unthread.js Added documentation comments and polling logic for friendlyId on ticket creation; updated webhook event handling.
src/index.js Refactored middleware usage for signature verification and webhook endpoint definition.
src/events/interactionCreate.js Updated error handling and thread creation for ticket creation with embedded messaging.
src/commands/utilities/version.js Introduced a new command to display the current bot version from package.json.
src/commands/utilities/user.js Updated the user command to display detailed user information using an embed.
src/commands/utilities/server.js Updated the server command to display server metrics via an embed.
src/commands/utilities/ping.js Enhanced the ping command with API latency and WebSocket heartbeat metrics embed.
Documentation Files Added or updated documentation including security, contributing guidelines, code of conduct, README, and funding info.
Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (2)

src/events/interactionCreate.js:29

  • ChannelType is used but not imported from 'discord.js'. Please import ChannelType to avoid runtime errors.
type: ChannelType.PrivateThread,

src/events/interactionCreate.js:23

  • [nitpick] Throwing an error here may lead to an unhandled promise rejection. Consider handling this scenario gracefully and providing a user-friendly error message.
if (!ticket.friendlyId) { throw new Error('Ticket was created but no friendlyId was provided'); }

@warengonzaga warengonzaga requested a review from Copilot April 1, 2025 12:31
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request delivers documentation updates, new features, and improvements to existing commands for the Discord bot.

  • Documentation updates: Added code of conduct, contributing guidelines, and security policy files.
  • Command improvements: Updated and enhanced the ping, server, user, and version commands, and improved webhook handling and ticket polling in the Unthread service.
  • Project version update: Upgraded package.json version from 0.1.8 to 0.1.20-alpha.

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/services/unthread.js Enhanced Unthread service with improved error handling, logging, and polling for ticket details; potential undefined thread variable issue in webhook handler.
src/index.js Introduced JSON middleware with raw body capture and reorganized webhook route.
src/events/interactionCreate.js Updated error handling in ticket creation and thread creation logic.
src/commands/utilities/version.js Added version command to display current bot version via an embed.
src/commands/utilities/user.js Refactored user command to use embedded messages and detailed timestamp formatting.
src/commands/utilities/server.js Refactored server command to use embedded messages with formatted server details.
src/commands/utilities/ping.js Enhanced the ping command with embedded messages showing latency metrics.
security.md Added security policy file.
contributing.md Added contributing guidelines.
code_of_conduct.md Added Contributor Covenant code of conduct document.
README.md Updated badges in the README file.
.github/funding.yml Introduced funding configuration file for project sponsorship.
Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (2)

src/services/unthread.js:257

  • Variable 'discordThread' is used without being defined. Please ensure that the Discord thread is properly retrieved or passed into the webhook handler.
await discordThread.send({ embeds: [closedEmbed] });

src/services/unthread.js:286

  • Variable 'discordThread' is referenced but not defined in this scope. Validate that the proper thread object is available before attempting to send a message.
await discordThread.send({ embeds: [reopenedEmbed] });

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@warengonzaga warengonzaga requested a review from Copilot April 1, 2025 12:34
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR prepares the v0.1.20-alpha release by updating documentation (code of conduct, contributing guidelines, and security policy), enhancing several command implementations, and updating the project version.

  • Updated documentation with community guidelines and contribution instructions
  • Improved command functionalities (ping, server, user, version)
  • Modified Unthread service functionality for ticket creation and webhook event handling

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/services/unthread.js Added detailed ticket creation logic with polling and webhook handling
src/index.js Updated Express middleware setup and webhook route definition
src/events/interactionCreate.js Enhanced ticket creation flow with improved error handling
src/commands/utilities/version.js Added a version command that displays bot version from package.json
src/commands/utilities/user.js Reworked the user command to display detailed user info with embeds
src/commands/utilities/server.js Reworked the server command to display server info via an embed
src/commands/utilities/ping.js Enhanced the ping command to show API latency and WebSocket heartbeat
security.md Added security policy file
contributing.md Added contributing guidelines
code_of_conduct.md Added community code of conduct guidelines
README.md Updated README badges
.github/funding.yml Added funding configuration for project support
Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (3)

src/services/unthread.js:134

  • [nitpick] Consider renaming 'maxRetries' to 'ticketPollingRetries' to clearly indicate its purpose in the ticket creation polling mechanism.
const maxRetries = 5;         // Maximum number of attempts to fetch the ticket

src/events/interactionCreate.js:23

  • Ensure that 'ticket' is defined before accessing 'ticket.friendlyId' in the catch block, as an error during ticket creation could leave 'ticket' undefined and cause a runtime exception.
if (!ticket.friendlyId) {

contributing.md:7

Please report unacceptable behavior to <sudo@wgtechlabs.com>.

@warengonzaga warengonzaga merged commit 4547d97 into main Apr 1, 2025
@warengonzaga warengonzaga changed the title 🚀 release: v0.1.20-alpha 🚀 release: v0.1.20 alpha Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainers only Assigned to Project Maintainers (Issue/PR) release Release Status (PR)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant