Remove Lombok dependency by Copilot · Pull Request #1604 · CyclopsMC/IntegratedDynamics

Copilot · 2026-02-18T18:59:30Z

Lombok Migration - Complete ✅

Summary

Successfully migrated 62 Java files from Lombok to standard Java:

1,198 insertions, 222 deletions
Zero Lombok imports remaining
Zero Lombok dependencies in build.gradle
All tests passing
No breaking API changes
Proper exception handling using Helpers.sneakyThrow

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@DaTa

Group 2 (callSuper=false): - MechanicalMachineNetworkElement.java - CoalGeneratorNetworkElement.java - EnergyBatteryNetworkElement.java - TileNetworkElement.java Group 3 (callSuper=true): - AspectVariableFacade.java - DelayVariableFacade.java - OperatorVariableFacade.java - ProxyVariableFacade.java All @DaTa annotations replaced with manual constructors, getters, equals, and hashCode. Group 2 equals/hashCode do NOT call super methods (callSuper=false). Group 3 equals/hashCode DO call super methods (callSuper=true). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@tostring

…toString() methods Migrated the following classes: - ValueBase - ValueObjectTypeBlock.ValueBlock - ValueObjectTypeEntity.ValueEntity - ValueObjectTypeFluidStack.ValueFluidStack - ValueObjectTypeIngredients.ValueIngredients - ValueObjectTypeItemStack.ValueItemStack - ValueObjectTypeRecipe.ValueRecipe - ValueTypeBoolean.ValueBoolean - ValueTypeDouble.ValueDouble - ValueTypeInteger.ValueInteger - ValueTypeList.ValueList - ValueTypeLong.ValueLong - ValueTypeOperator.ValueOperator - ValueTypeString.ValueString Each class now has a manual toString() method that returns a string in the format 'ClassName(field1=value1, field2=value2, ...)' to match Lombok's @tostring behavior. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@DaTa

This commit completes the Lombok migration by handling the special cases: Group 6 - @DaTa with @nonnull: - UpdatingAspectVariable: Added null check in setValue() for @nonnull field - LazyAspectVariable: Added getters for @Getter fields Group 7 - @DaTa classes with inner classes: - NetworkElementBase: Removed @DaTa (no fields at top level) - PartHelpers.PartStateHolder: Added equals/hashCode/toString - NetworkDataClient inner classes: Added getters for all fields Group 8 - Special cases: - Cluster: Manually implemented all Collection interface methods (was using @DeleGate) - ItemPart: Added equals/hashCode for @EqualsAndHashCode(callSuper = false) - Operators: Replaced Lombok.sneakyThrow with RuntimeException - NbtHelpers: Removed unused @slf4j annotation All Lombok annotations have been removed and replaced with standard Java. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…cies Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>

Copilot

Pull request overview

This PR successfully removes the Lombok dependency from the codebase by replacing all Lombok-generated code with manual Java implementations. The changes affect 62 files (61 Java files and 1 Gradle build file) and include:

Changes:

Replaced @Data, @Getter, @Setter, @EqualsAndHashCode(callSuper), @ToString annotations with manual implementations
Replaced @NonNull with Objects.requireNonNull() null checks
Replaced @Delegate with manual Collection interface delegation
Removed @Slf4j (logger was not used)
Replaced Lombok.sneakyThrow() with RuntimeException wrapping
Replaced @SneakyThrows with try-catch blocks
Removed Lombok dependencies from build.gradle

Reviewed changes

Copilot reviewed 61 out of 62 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
build.gradle	Removed Lombok compile and annotation processor dependencies
MechanicalMachineNetworkElement.java	Added constructor, getter, equals, hashCode methods
EnergyBatteryNetworkElement.java	Added constructor, getter, equals, hashCode methods
CoalGeneratorNetworkElement.java	Added constructor, getter, equals, hashCode methods
Cluster.java	Manually implemented all Collection interface methods (replacing @DeleGate)
PartTypePanelVariableDriven.java	Added getters and setters for state fields
AspectPropertyTypeInstance.java	Added getters
UpdatingAspectVariable.java	Added getters, setter with null check, equals, hashCode, toString
LazyAspectVariable.java	Added getters
PartTypeBase.java	Added getters
PartStateActiveVariableBase.java	Added getters and setters
RawPartData.java	Added constructor, getters, equals, hashCode
RawObserverData.java	Added constructor, getters, equals, hashCode
RawNetworkData.java	Added constructor, getters, equals, hashCode
NetworkDataClient.java	Added constructors and getters for inner classes
TileNetworkElement.java	Added constructor, getter, equals, hashCode
PositionedAddonsNetwork.java	Added getter and setter
PartNetwork.java	Added getter and setter
NetworkElementBase.java	Removed @DaTa (no fields)
EnergyNetwork.java	Added getter and setter
ValueTypeRecipeLPElementRecipeSubGui.java	Added getter and setter
RenderPattern.java	Added getter
ValueTypeRecipeLPElement.java	Added getters and setters
ValueTypeLPElementBase.java	Added getter
ProxyVariableFacade.java	Added getters, setters, equals (with super call), hashCode (with super)
OperatorVariableFacade.java	Added getters, setters, equals (with super call), hashCode (with super)
ItemPart.java	Added equals, hashCode
DelayVariableFacade.java	Removed annotations (extends ProxyVariableFacade)
AspectVariableFacade.java	Added getters, equals (with super call), hashCode (with super)
PartHelpers.java	Added constructor, getters, equals, hashCode, toString for PartStateHolder
NbtHelpers.java	Removed @slf4j
GuiElementValueTypeStringRenderPattern.java	Added getters
GuiElementValueTypeString.java	Added getters, setters, equals, hashCode, toString; fixed getDefaultInputBoolean bug
GuiElementValueTypeDropdownListRenderPattern.java	Added getters
GuiElementValueTypeDropdownList.java	Added getters, setters, equals, hashCode, toString
GuiElementValueTypeBooleanRenderPattern.java	Added getters
GuiElementValueTypeBoolean.java	Added getters, equals, hashCode, toString; fixed getDefaultInputBoolean bug
ValueTypeString.java	Added manual toString for ValueString
ValueTypeOperator.java	Added manual toString for ValueOperator
ValueTypeLong.java	Added manual toString for ValueLong
ValueTypeList.java	Added manual toString for ValueList
ValueTypeInteger.java	Added manual toString for ValueInteger
ValueTypeDouble.java	Added manual toString for ValueDouble
ValueTypeBoolean.java	Added manual toString for ValueBoolean
ValueObjectTypeRecipe.java	Added manual toString for ValueRecipe
ValueObjectTypeItemStack.java	Added manual toString for ValueItemStack
ValueObjectTypeIngredients.java	Added manual toString for ValueIngredients
ValueObjectTypeFluidStack.java	Added manual toString for ValueFluidStack
ValueObjectTypeEntity.java	Added manual toString for ValueEntity
ValueObjectTypeBlock.java	Added manual toString for ValueBlock
ValueBase.java	Added manual toString
Operators.java	Replaced Lombok.sneakyThrow with RuntimeException wrapping
BakedSingleVariableModelProvider.java	Added constructor, getter, equals, hashCode, toString
BakedMapVariableModelProvider.java	Added constructor, getter, equals, hashCode, toString
WidgetTextFieldDropdown.java	Added getters and setters
BlockEntityMultipartTicking.java	Added getters and setters
CableRenderState.java	Added constructor, getters, equals, hashCode, toString
BlockEntitySqueezer.java	Added getter
BlockEntityProxy.java	Added getters and setters
BlockEntityMaterializer.java	Added setter
BlockEntityDelay.java	Added getters and setters
BlockCable.java	Replaced @SneakyThrows with try-catch; added setter

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-18T19:45:46Z

                                ValueHelpers.validatePredicateOutput(operator, result);
                            } catch (EvaluationException e) {
-                                throw Lombok.sneakyThrow(e);
+                                throw new RuntimeException(e);


Replacing Lombok.sneakyThrow(e) with throw new RuntimeException(e) changes the exception handling behavior. The original would rethrow the checked EvaluationException as-is (bypassing checked exception rules), while the new code wraps it in a RuntimeException. This means any code catching EvaluationException specifically won't catch this anymore - it will need to catch RuntimeException and unwrap it. Consider whether this behavioral change is acceptable, or if a better solution would be to declare throws EvaluationException on the lambda/method.

Could you use Helpers.sneakyThrow instead of throwing a RuntimeException?

Fixed in commit 95d846e. Replaced throw new RuntimeException(e) with return Helpers.sneakyThrow(e) to properly rethrow the exception without wrapping.

Copilot · 2026-02-18T19:45:46Z

+        } catch (Exception e) {
+            throw new RuntimeException(e);


Similar to the previous sneakyThrow replacement, wrapping the exception in a RuntimeException changes error handling behavior. Code that was catching specific exceptions from the cache's callable will now need to catch and unwrap RuntimeException instead. Consider whether this is the intended behavior or if there's a better alternative.

Could you use Helpers.sneakyThrow instead of throwing a RuntimeException?

Fixed in commit 95d846e. Replaced throw new RuntimeException(e) with return Helpers.sneakyThrow(e) to properly rethrow the exception without wrapping.

rubensworks · 2026-02-19T14:32:33Z

+        } catch (Exception e) {
+            throw new RuntimeException(e);


Could you use Helpers.sneakyThrow instead of throwing a RuntimeException?

rubensworks · 2026-02-19T14:33:25Z

                                ValueHelpers.validatePredicateOutput(operator, result);
                            } catch (EvaluationException e) {
-                                throw Lombok.sneakyThrow(e);
+                                throw new RuntimeException(e);


Could you use Helpers.sneakyThrow instead of throwing a RuntimeException?

… exception handling Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>

coveralls · 2026-02-19T14:51:14Z

coverage: 52.616% (+0.3%) from 52.363%
when pulling 9e55222 on copilot/migrate-away-from-lombok
into 9a98fce on master-1.21.

Initial plan

cdc1341

Copilot AI assigned Copilot and rubensworks Feb 18, 2026

Copilot started work on behalf of rubensworks February 18, 2026 18:59 View session

Copilot AI and others added 4 commits February 18, 2026 19:10

Complete Lombok migration: remove all Lombok annotations and dependen…

33790bf

…cies Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Remove Lombok usage and update related files~~ Remove Lombok dependency Feb 18, 2026

Copilot AI requested a review from rubensworks February 18, 2026 19:37

Copilot finished work on behalf of rubensworks February 18, 2026 19:37

rubensworks requested a review from Copilot February 18, 2026 19:39

Copilot started reviewing on behalf of rubensworks February 18, 2026 19:40 View session

Copilot AI reviewed Feb 18, 2026

View reviewed changes

rubensworks marked this pull request as ready for review February 18, 2026 19:48

rubensworks requested changes Feb 19, 2026

View reviewed changes

Copilot started work on behalf of rubensworks February 19, 2026 14:35 View session

Replace RuntimeException wrapping with Helpers.sneakyThrow for proper…

95d846e

… exception handling Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>

Copilot AI requested a review from rubensworks February 19, 2026 14:45

Copilot finished work on behalf of rubensworks February 19, 2026 14:45

Merge branch 'master-1.21' into copilot/migrate-away-from-lombok

9e55222

rubensworks merged commit 2fd4d78 into master-1.21 Feb 19, 2026
5 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Remove Lombok dependency#1604

Remove Lombok dependency#1604
rubensworks merged 7 commits intomaster-1.21from
copilot/migrate-away-from-lombok

Copilot AI commented Feb 18, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 18, 2026

Uh oh!

rubensworks Feb 19, 2026

Uh oh!

Copilot AI Feb 19, 2026

Uh oh!

Copilot AI Feb 18, 2026

Uh oh!

rubensworks Feb 19, 2026

Uh oh!

Copilot AI Feb 19, 2026

Uh oh!

rubensworks Feb 19, 2026

Uh oh!

rubensworks Feb 19, 2026

Uh oh!

coveralls commented Feb 19, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

Copilot AI commented Feb 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Lombok Migration - Complete ✅

Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

Uh oh!

rubensworks Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

Uh oh!

rubensworks Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

rubensworks Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

rubensworks Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

coveralls commented Feb 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Copilot AI commented Feb 18, 2026 •

edited

Loading

coveralls commented Feb 19, 2026 •

edited

Loading