Remove unused methods and improve functionality in core classes#223

Merged

Splatcrafter merged 21 commits into

developfrom

bugfix/117-118-119-120-121-122-123-api-bugfixes

Mar 21, 2026

Commits on Mar 21, 2026

Fix Fixes.fixTypeEverywhere() silently returning input unchanged (#117)
Splatcrafter
committed
Fix Schema.buildTypes() parent type inheritance and thread safety (#118, #119)
Splatcrafter
committed
Implement `equals`, `hashCode`, and `toString` in `TaggedDynamic` class for improved object comparison, hashing, and debugging. (#120)
Splatcrafter
committed
Fix `Typed.encodeAndGet` to return `DataResult` and handle missing path errors gracefully (#121)
Splatcrafter
committed
Validate non-negative index in `Finder.index` method (#122)
Splatcrafter
committed
Remove unused `compose` method from reversed `Iso` implementation (#123)
Splatcrafter
committed
Update `RandomFailuresChaosIT` to handle `FixException` instead of `ChaosException`
Splatcrafter
committed
Add unit tests for negative index rejection in `Finder.index()` and `reverse()` composition in `Iso`
Splatcrafter
committed
Improve robustness of dynamic key extraction in `Rules` by adding null checks and streamlining `asString` handling
Splatcrafter
committed
Initial plan
Copilot
committed
docs: update getAt @return Javadoc to reflect error-based behavior

Copilot
and
Splatcrafter
committed
Merge pull request #224 from aether-framework/copilot/sub-pr-223
Splatcrafter
authored
Initial plan
Copilot
committed
Initial plan
Copilot
committed
Update aether-datafixers-api/src/main/java/de/splatgames/aether/datafixers/api/fix/Fixes.java

Splatcrafter
and
Copilot
authored
Merge pull request #226 from aether-framework/copilot/sub-pr-223-another-one
Splatcrafter
authored
Revert "[WIP] [WIP] Address feedback on removing unused methods"
Splatcrafter
authored
Fix Finder.index() Javadoc: negative index throws IAE at construction, not null from get()

Copilot
and
Splatcrafter
committed
Merge pull request #227 from aether-framework/revert-226-copilot/sub-pr-223-another-one
Splatcrafter
authored
Merge pull request #225 from aether-framework/copilot/sub-pr-223-again
Splatcrafter
authored
Simplify `fixTypeEverywhere` by directly returning `Optional` from `DataResult#result` and removing redundant error handling.
Splatcrafter
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Remove unused methods and improve functionality in core classes#223

Remove unused methods and improve functionality in core classes#223
Splatcrafter merged 21 commits into
developfrom
bugfix/117-118-119-120-121-122-123-api-bugfixes

Commits on Mar 21, 2026

Fix Fixes.fixTypeEverywhere() silently returning input unchanged (#117)

Fix Schema.buildTypes() parent type inheritance and thread safety (#118, #119)

Implement `equals`, `hashCode`, and `toString` in `TaggedDynamic` class for improved object comparison, hashing, and debugging. (#120)

Fix `Typed.encodeAndGet` to return `DataResult` and handle missing path errors gracefully (#121)

Validate non-negative index in `Finder.index` method (#122)

Remove unused `compose` method from reversed `Iso` implementation (#123)

Update `RandomFailuresChaosIT` to handle `FixException` instead of `ChaosException`

Add unit tests for negative index rejection in `Finder.index()` and `reverse()` composition in `Iso`

Improve robustness of dynamic key extraction in `Rules` by adding null checks and streamlining `asString` handling

Initial plan

docs: update getAt @return Javadoc to reflect error-based behavior

Merge pull request #224 from aether-framework/copilot/sub-pr-223

Initial plan

Initial plan

Update aether-datafixers-api/src/main/java/de/splatgames/aether/datafixers/api/fix/Fixes.java

Merge pull request #226 from aether-framework/copilot/sub-pr-223-another-one

Revert "[WIP] [WIP] Address feedback on removing unused methods"

Fix Finder.index() Javadoc: negative index throws IAE at construction, not null from get()

Merge pull request #227 from aether-framework/revert-226-copilot/sub-pr-223-another-one

Merge pull request #225 from aether-framework/copilot/sub-pr-223-again

Simplify `fixTypeEverywhere` by directly returning `Optional` from `DataResult#result` and removing redundant error handling.