Skip to content

fix: reword player messages in character - #23

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
Only a trained Physician can perform surgery. You are not skilled enough to perform surgery. Only a trained Physician can operate.
Vision cleared! Skill fail chance reduced for next move. Vision cleared! Your next move will be steadier.

Live config

Deployed servers keep their own copy of this file, so Dev and Main need a matching edit when this ships: messages.yml: command-no-permission and vision-cleared.

Testing

mvn -o verify passes: 7 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: 83faf745-4928-492b-95f5-45980632aa03

📥 Commits

Reviewing files that changed from the base of the PR and between 79f1d94 and 8a365a4.

📒 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

  • Text Updates
    • Updated permission messaging to describe players as not skilled enough to perform surgery.
    • Changed the vision-cleared message to say the next move will be steadier.

Walkthrough

Two player-facing messages have revised wording: the surgery permission message now refers to skill, and the vision message says the next move will be steadier.

Changes

Player-facing messages

Layer / File(s) Summary
Revise player-facing message wording
src/main/resources/messages.yml
The command-no-permission message now says the player is not skilled enough to perform surgery. The vision-cleared message says the next move will be steadier.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: justinasla

Merge Risk: ⚪ Minimal · up to 8a365

This is a wording-only change to two player-facing messages, with no change to permission checks or surgery behavior. The risk of merging is minimal. Servers with their own messages.yml copy need matching edits to see the new text.

Architecture Summary

Architecture risk: 🔵 Low · up to 8a365

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: command-no-permission replaces the trained-Physician-only wording with a message that the player is not skilled enough to perform surgery.
  • observed — Modified behavior in src/main/resources/messages.yml: vision-cleared replaces the claim that skill fail chance is reduced for the next move with a message that the next move will be steadier.
  • 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 a5e2938 into main Oct 1, 2026
2 checks passed
@ryanbarlow97
ryanbarlow97 deleted the copy/rp-wording branch October 1, 2026 11:37
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