Skip to content

feat(economy): consolidated semantic gil telemetry#122

Draft
Shuu-37 wants to merge 1 commit into
betafrom
economy
Draft

feat(economy): consolidated semantic gil telemetry#122
Shuu-37 wants to merge 1 commit into
betafrom
economy

Conversation

@Shuu-37

@Shuu-37 Shuu-37 commented Jul 13, 2026

Copy link
Copy Markdown

Moves the semantic gil telemetry module to the consolidated economy branch. This replaces the earlier feature-branch PR.

@github-actions

Copy link
Copy Markdown

✨ Hello and thanks for the PR! ✨

🤖: This is a friendly automated reminder that the maintainers won't look at your PR until you've properly completed all of the checkboxes in the pre-filled template.

@Shuu-37
Shuu-37 marked this pull request as draft July 13, 2026 17:52
@zach2good

zach2good commented Jul 13, 2026

Copy link
Copy Markdown

Just because I saw it in passing, I'll throw in an opinion:

There's some useful ideas here, but as mentioned, it's otherwise slop.

Moving gil handling per player into its own container and really locking down how the rest of the code interacts with it is a good idea. Then we can add logging, and turn that into telemetry with origin enums, etc.

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