Skip to content

chore: sync default alloy base bonuses with the live server - #24

Merged
XxFran10xX merged 1 commit into
mainfrom
chore/sync-base-bonus-defaults
Sep 27, 2026
Merged

XxFran10xX merged 1 commit into
mainfrom
chore/sync-base-bonus-defaults

Conversation

@XxFran10xX

@XxFran10xX XxFran10xX commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Matches the default alloy-forge.base-bonus-percent values in config.yml to the values now on TFMCMain01. Staff tuned these by hand after #23:

Base Before After
steel_ingot 10 8
bronze_ingot 15 12
abyssalite_ingot 20 16
mythril_ingot 25 24
refined_maplewood — 8
refined_elderwood — 16
refined_demonwood — 24

Default config only. There is no code change, and nothing changes on servers that already have a config.yml.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Balance Changes
    • Reduced alloy forge base bonuses for steel, bronze, abyssalite, and mythril.
    • Added base bonuses for refined maplewood, refined elderwood, and refined demonwood.

Match the default base-bonus-percent values to TFMCMain01: steel 8,
bronze 12, abyssalite 16, mythril 24, and the refined wood bases
(maplewood 8, elderwood 16, demonwood 24).

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
@coderabbitai

coderabbitai Bot commented Sep 27, 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: c645f035-d815-456f-a494-060c4fa86a9f

📥 Commits

Reviewing files that changed from the base of the PR and between 2efc4f0 and b5ff029.

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

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


📝 Walkthrough

Walkthrough

The alloy forge configuration now uses reduced base bonuses for steel, bronze, abyssalite, and mythril. It adds base bonuses for refined maplewood, refined elderwood, and refined demonwood.

Changes

Alloy Forge Base Bonuses

Layer / File(s) Summary
Update configured base bonuses
src/main/resources/config.yml
The configured bonuses for steel, bronze, abyssalite, and mythril were reduced. Bonuses were added for refined maplewood, refined elderwood, and refined demonwood.

Priority: ⬇️ Low

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

Change: Other

Suggested reviewers: ryanbarlow97

Merge Risk: ⚪ Minimal · up to b5ff0

The configured bonuses match the intended updates, and no concrete merge-blocking risk is identified.

Architecture Summary

Architecture risk: 🔵 Low · up to b5ff0

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/config.yml: The configured base bonuses for steel, bronze, abyssalite, and mythril ingots were reduced; refined maplewood, refined elderwood, and refined demonwood were added with bonuses of 8.0%, 16.0%, and 24.0%, respectively.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: updating default alloy base bonuses to match the live server.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

A rabbit checks the forge at night
Steel and bronze now glow less bright
Three woods join the bonus row
Maple, elder, demonwood in tow
The config rests, its numbers neat
The rabbit hops on quiet feet

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

@XxFran10xX
XxFran10xX merged commit 9bf45a7 into main Sep 27, 2026
2 checks passed
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