Skip to content

feat(chat): unify localized permissions messages - #36

Merged
lusu007 merged 3 commits into
mainfrom
feat/java-chat-unification
Oct 14, 2026
Merged

lusu007 merged 3 commits into
mainfrom
feat/java-chat-unification

Conversation

@lusu007

@lusu007 lusu007 commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Description

Uses typed categories and localized human command/login responses. Keeps machine-readable key=value diagnostics and external error text intact.

Published dependencies

Consumes published library-i18n:0.4.0 and plugin-proxy-api:2.6.0. Standalone Gradle builds, tests and formatting checks passed without composite dependency substitutions. GitHub CI and repository review requirements gate the merge.

Validation

  • 197 test cases; 0 failures/errors.
  • Relevant Gradle build/test and formatting checks passed.
  • Integration boundaries reviewed independently.

Design and release order: Minecraft Mini-CD.

Related changes

Context and level

Separates semantic subject (SystemCategory: GENERAL, FRIENDS, PARTY, MATCH, STORY) from outcome/urgency (SystemLevel: INFO, SUCCESS, WARNING, ERROR). Defaults are GENERAL + INFO. A failed friend operation retains FRIENDS with ERROR; the level colours the context label without replacing it. Localized layouts may additionally place <level>. Both metadata fields reach host renderers; message component styles and actions remain intact.

Reverified with published dependencies in standalone builds. All 13 affected consumer build scopes pass; the shared library has 42 passing tests. Overall audited source test scope: 1550 cases, 1544 passed, six existing skips. The resource-pack product remains gated on actual Minecraft client acceptance.

@lusu007
lusu007 marked this pull request as ready for review October 14, 2026 22:38
@lusu007
lusu007 merged commit 876f830 into main Oct 14, 2026
3 checks passed
@lusu007
lusu007 deleted the feat/java-chat-unification branch October 14, 2026 22:38
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.

1 participant