Skip to content

fix: reword player messages in character - #28

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.

Both skill checks share one constant, Permissions.NOT_SKILLED.

Before and after

Colour codes are omitted; the code keeps them.

Before After
You do not have permission to use woodworking. You are not skilled enough to work wood.
This item cannot be used for woodworking hits You cannot work the piece with that tool

Testing

mvn -o verify passes: 31 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: 9221d3c2-38d8-4a6d-a6c0-940eca7c7158

📥 Commits

Reviewing files that changed from the base of the PR and between 0106e49 and 1cfed93.

📒 Files selected for processing (3)
  • src/main/java/net/tfminecraft/woodworking/command/Permissions.java
  • src/main/java/net/tfminecraft/woodworking/station/StationManager.java
  • src/test/java/net/tfminecraft/woodworking/StationManagerTest.java

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


📝 Summary

Summary by CodeRabbit

  • Bug Fixes
    • Updated station-use messages to clarify when players lack the required woodworking skill or cannot work a piece.
    • Denied interactions remain cancelled, with notifications shown according to the existing cooldown behaviour.

Walkthrough

The permission-denial and unusable-tool messages now use updated wording. Tests check the updated messages. The permission-denial cancellation and cooldown flow remains unchanged.

Changes

Woodworking message updates

Layer / File(s) Summary
Skill-denial message
src/main/java/net/tfminecraft/woodworking/command/Permissions.java, src/main/java/net/tfminecraft/woodworking/station/StationManager.java, src/test/java/net/tfminecraft/woodworking/StationManagerTest.java
Permissions defines NOT_SKILLED, and the permission-denial paths use the updated message. The interaction test expects Permissions.NOT_SKILLED.
Unusable-tool message
src/main/java/net/tfminecraft/woodworking/station/StationManager.java, src/test/java/net/tfminecraft/woodworking/StationManagerTest.java
The unusable-hit message now says “You cannot work the piece with that tool”. The test checks the updated wording.

Priority: ⬇️ Low

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

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 1cfed

The changes update two gameplay messages without a supported indication of altered interaction behavior. No actionable merge risk remains.

  • 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: 3
  • Waived: $0.75 (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 703e56a 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