ci: declare Trade Everything as an embedded CurseForge relation - #834
Open
bh679 wants to merge 13 commits into
Open
ci: declare Trade Everything as an embedded CurseForge relation#834bh679 wants to merge 13 commits into
bh679 wants to merge 13 commits into
Conversation
…lot) Wire tradeeverything 0.2.0 as the fifth external required sibling dep (AIN/AIS/PMOB/ECP contract): gradle.properties version+floor, shared bh679 Ivy fetch, neoforge.mods.toml required range [0.2.0,), deptest te key + cases, docs prose 4->5 siblings, changelog entry. Modpack entry + release.yml platform slugs deliberately deferred until the Modrinth/CurseForge listings for TE exist — DT must not release with this dependency before then. Co-Authored-By: Claude Opus 4.8 <[email protected]>
…gs approved DP-style temporary bundling: jarJar + versionRange keyed to tradeeverything_version, no _min_version floor, deptest reverted to the four external siblings (a bundled mod can never go missing). Un-bundle to the external contract once Modrinth/CurseForge approve the listings. Co-Authored-By: Claude Opus 4.8 <[email protected]>
…d roll paused) Villager level now deterministically equals maxLevelForTier(currentTier) instead of a weighted roll capped by it — trader quality tracks train progression 1:1. Weighted spread retained for the difficulty-disabled path and easy to restore. Co-Authored-By: Claude Opus 4.8 <[email protected]>
New compat.TradeEverythingBridge registers a TE ItemValueProvider valuing every written book at 8 sixteenths (x0.75 margin = exactly 6 coal/paper per book). Installed behind the ECP-style ModList+Throwable guard. Co-Authored-By: Claude Opus 4.8 <[email protected]>
…50% one level below cap - Bundle tradeeverything 0.2.1: exception-guarded trading injects (fixes the trade-menu wedge), villager-own-rate payout pricing, Trade Anything row click fix - Train villager level: cap or cap-1 at 50/50 (floored at 1) Co-Authored-By: Claude Opus 4.8 <[email protected]>
…, stats New difficulty/TradeGearScaler post-processes every train-villager offer: - From carriage 30 (config villagerTradeScalingMinCarriage), sold gear is stripped and re-enchanted at power 15/25/35... (+1 step per 5 difficulty tiers, config villagerTradeScalingTiersPerStep, cap 60 — past vanilla's 30-level table) - Emerald costs gain the enchant premium (2^(level-1) emeralds per enchant, the same curve TE values enchants at, so buy/sell prices agree); totals past 64 convert to emerald-block costs (payable via TE block breaking) - AIS stats + AIN names applied at the villager's own carriage tier — same decorator pipeline as mob gear and chest loot - Villager level cap + all scaling now key off the villager's OWN carriage (TrainMembership.carriageIndexOf -> DifficultyProgression.positionTier), not player progress: deeper carriages host stronger traders Config: villagerTradeScalingEnabled/MinCarriage/TiersPerStep in [difficulty]. Tests: enchant power curve, carriage-tag parsing. Co-Authored-By: Claude Opus 4.8 <[email protected]>
…reeze Daemon sampler logs the server thread's live stack when a tick stalls past 5s (re-dump every 10s while stuck, recovery line with total stall time). Two in-field freezes left no trace once the process died; the next one becomes an attributable stack in latest.log. Pure diagnostics. Co-Authored-By: Claude Opus 4.8 <[email protected]>
Co-Authored-By: Claude Opus 4.8 <[email protected]>
Stat tier = posTier + rand(0..max(1, posTier/5)) — shop gear is slightly aspirational: tier 1 -> +0-1, 10 -> +0-2, 20 -> +0-4, 40 -> +0-8. Co-Authored-By: Claude Opus 4.8 <[email protected]>
… review) Co-Authored-By: Claude Opus 4.8 <[email protected]>
Co-Authored-By: Claude Opus 4.8 <[email protected]>
… re-append changelog entries)
TE is jarJar-bundled in DT (like DiscordPresence). Now that its CurseForge project is live, link/credit it from DT's CF listing via trade-everything(embedded). Kept off the Modrinth dependencies list until its Modrinth listing clears review. Co-Authored-By: Claude Opus 4.8 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Now that the Trade Everything CurseForge project (
1622962) is live, link/credit it from Dungeon Train's CurseForge listing.TE is still jarJar-bundled inside DT (same as DiscordPresence), so the relation type is
embedded— CurseForge links the project page without telling the app to fetch a separate download.trade-everything(embedded)tocurseforge-dependenciesinrelease.ymldependencieslist yet — the Modrinth listing is still under review, and referencing a not-yet-live project would fail the publish. Comment notes to addtrade-everything(embedded)there once approved.(required)and jarJar is dropped (wiring preserved in feat: require Trade Everything sibling mod (villager Trade Anything slot) #828 history).Takes effect on the next DT release. No jar/gameplay change.
🤖 Generated with Claude Code