Skip to content

fix: reword player messages in character - #8

Merged
ryanbarlow97 merged 1 commit into
mainfrom
copy/rp-wording
Oct 1, 2026
Merged

ryanbarlow97 merged 1 commit into
mainfrom
copy/rp-wording

Conversation

@ryanbarlow97

Copy link
Copy Markdown
Contributor

Summary

Rewords player-facing gameplay messages that broke immersion with server terms (permissions, access, containers, entities, chunks, plugin names) so they read in character. Staff and admin commands, usage help and setup errors keep their plain wording. Wording only: no permission checks or message routing change.

Before and after

Colour codes are omitted; the code keeps them.

Before After
{prefix}Highlighted {count} remaining lure mobs for 10s. {prefix}{count} creatures remain. The lure marks them for 10 seconds.

Live config

Deployed servers keep their own copy of this file, so Dev and Main need a matching edit when this ships: messages.yml: lure-highlight.

Testing

mvn -o verify passes: 8 tests, 0 failures.

🤖 Generated with Claude Code

Replace out-of-character wording in gameplay messages (permissions,
containers, entities, chunks, plugin tags) with in-world phrasing.
Staff commands, usage help and setup errors keep their plain wording.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: df23de91-66f0-4001-9710-a089f6985d1c

📥 Commits

Reviewing files that changed from the base of the PR and between 318c455 and 96d7379.

📒 Files selected for processing (1)
  • src/main/resources/messages.yml

Limit details: You’ve used all 10 included reviews currently available.


📝 Summary

Summary by CodeRabbit

  • Updates
    • Clarified that the lure marks creatures for 10 seconds and shows how many remain.

Walkthrough

The lure-highlight message now describes how many creatures remain and states that the lure marks them for 10 seconds.

Changes

Lure highlight message

Layer / File(s) Summary
Update lure highlight message
src/main/resources/messages.yml
The message now describes the remaining creature count and a 10-second marking duration.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix

Suggested reviewers: drefvelin

Merge Risk: ⚪ Minimal · up to 96d73

The lure notification’s count and 10-second duration remain consistent with the described behavior. No actionable merge risk is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to 96d73

The change affects 1 system.

Changed systems: src

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — src (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in src/main/resources/messages.yml: lure-highlight now describes the remaining creature count and a 10-second marking duration instead of reporting how many remaining lure mobs were highlighted for 10 seconds.
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

Usage-based review receipt

  • Mode: Continue automatically
  • Reviewed files: 1
  • Waived: $0.25 (charged $0.00)
  • View usage details

Note

This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.


Comment @coderabbitai help to get the list of available commands.

@ryanbarlow97
ryanbarlow97 merged commit ce30d36 into main Oct 1, 2026
2 checks passed
@ryanbarlow97
ryanbarlow97 deleted the copy/rp-wording branch October 1, 2026 11:36
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