Skip to content

fix: reword player messages in character - #12

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
Chunk already has a node! There is already a node on this land!
You are already filled your node capacity! Your guild has already filled its node capacity!
Natural yields are disabled Your dowsing stick does not stir.
This chunk has resources of the type: … in it, with a yield of This land holds … , with a yield of
Chunk has a natural yield of This land has a natural yield of

Testing

mvn -o verify passes: 38 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: 306476d9-b3f8-42c2-a199-ea5a7c86aeaf

📥 Commits

Reviewing files that changed from the base of the PR and between afef00e and a699a61.

📒 Files selected for processing (4)
  • src/main/java/net/tfminecraft/dowsing/managers/NodeManager.java
  • src/main/java/net/tfminecraft/dowsing/managers/ResourceManager.java
  • src/main/java/net/tfminecraft/dowsing/objects/Node.java
  • src/main/java/net/tfminecraft/dowsing/utils/ItemCreator.java

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 2 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Updates
    • Clarified messages about existing nodes and guild node capacity.
    • Updated dowsing and resource messages to refer to land and natural yields rather than chunks. Values and behaviour are unchanged.

Walkthrough

Player-facing messages for node checks and dowsing results now use updated wording. The underlying checks, rejection behaviour, and displayed resource and yield values remain unchanged.

Changes

Player-facing messages

Layer / File(s) Summary
Update node and dowsing messages
src/main/java/net/tfminecraft/dowsing/managers/NodeManager.java, src/main/java/net/tfminecraft/dowsing/managers/ResourceManager.java, src/main/java/net/tfminecraft/dowsing/objects/Node.java, src/main/java/net/tfminecraft/dowsing/utils/ItemCreator.java
Node messages refer to guild capacity and land. Dowsing messages and the natural-yield item name use updated wording. Checks, rejection behaviour, and resource and yield values are unchanged.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: drefvelin

Merge Risk: ⚪ Minimal · up to a699a

This change updates player-facing wording without a reported gameplay or displayed-value change. No merge-blocking risk is evident beyond normal checks.

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

@ryanbarlow97
ryanbarlow97 merged commit 8622a49 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