Skip to content

Conversation

sovdeeth
Copy link
Member

Problem

Often users can get confused on issues where they expect to have a number but actually have a string, or vice-versa, since the broadcasted values look identical.

Solution

This expression turns inputs into strings, but also adds the type of the value to the string. Text is also quoted for clarity:

image

I kept this simple for now, but more debug options (like conversions, debug info on expressions, etc) could be added later or based on verbosity settings. I intentionally did not use Classes.getDebugMessage because it was not very user-friendly:
[long:1]. This has the drawback of not having the built-in debug messages some types include, but the plus side of being easier to read and more standardized between types.

Testing Completed

Manual confirmation.

Supporting Information

This is pretty bare-bones, but I didn't want to get all that complex with it right now. I know more could be added, but I'd rather save that complexity for another pr.


Completes: none
Related: none

@sovdeeth sovdeeth requested a review from a team as a code owner September 25, 2025 19:55
@sovdeeth sovdeeth requested review from cheeezburga and Burbulinis and removed request for a team September 25, 2025 19:55
@sovdeeth sovdeeth added the feature Pull request adding a new feature. label Sep 25, 2025
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Sep 25, 2025
Copy link
Member

@Efnilite Efnilite left a comment

Choose a reason for hiding this comment

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

lgtm 🔥

@github-project-automation github-project-automation bot moved this to Awaiting Merge in 2.13 Releases Sep 26, 2025
@skriptlang-automation skriptlang-automation bot added feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. and removed needs reviews A PR that needs additional reviews labels Sep 26, 2025
@sovdeeth sovdeeth merged commit c22ee9e into SkriptLang:dev/feature Oct 2, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.13 Releases Oct 2, 2025
@skriptlang-automation skriptlang-automation bot added completed The issue has been fully resolved and the change will be in the next Skript update. and removed feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. labels Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. feature Pull request adding a new feature.
Projects
Status: Done - Awaiting Release
Development

Successfully merging this pull request may close these issues.

3 participants