Skip to content

Conversation

@TheBjoRedCraft
Copy link
Member

@TheBjoRedCraft TheBjoRedCraft commented Nov 19, 2025

  • The default prefix has been changed to remove the internal "Surf" from every public message. The prefix can still be used for internal messages.
  • New prefixes have been added to the SurfComponentBuilder, for errors, warnings and the old surf prefix.
  • A new color have been added to the SurfComponentBuilder, white, to display white text simple.

@TheBjoRedCraft TheBjoRedCraft self-assigned this Nov 19, 2025
Copilot AI review requested due to automatic review settings November 19, 2025 15:08
Copilot finished reviewing on behalf of TheBjoRedCraft November 19, 2025 15:10
Copy link
Contributor

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 message prefix consistency by introducing a simplified default prefix and adding specialized prefix variants for different message types (errors, warnings, and internal "Surf" branding). The default PREFIX is simplified to just a spacer symbol without internal branding, while a new SURF_PREFIX maintains the original branded format for internal use.

  • Simplified the default PREFIX component to remove the "Surf" branding for public messages
  • Added three new prefix variants: SURF_PREFIX for internal messages, ERROR_PREFIX for errors, and WARNING_PREFIX for warnings
  • Added a white() color function to the SurfComponentBuilder for displaying white text

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
surf-api-core/surf-api-core-api/src/main/kotlin/dev/slne/surf/surfapi/core/api/messages/Colors.kt Defines the new prefix components (PREFIX, SURF_PREFIX, WARNING_PREFIX, ERROR_PREFIX) with updated documentation
surf-api-core/surf-api-core-api/src/main/kotlin/dev/slne/surf/surfapi/core/api/messages/builder/SurfComponentBuilder.kt Adds convenience methods for appending the new prefixes and introduces the white() color function
gradle.properties Bumps the version from 1.21.10-2.42.3 to 1.21.10-2.42.4

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

TheBjoRedCraft and others added 2 commits November 19, 2025 16:25
…surfapi/core/api/messages/Colors.kt

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants