We still need to trigger storing the roundtripState on writeObject.

nedtwigg · nedtwigg · commit f524e9bd28bf · 2024-10-13T01:21:29.000-07:00
diff --git a/lib/src/main/java/com/diffplug/spotless/FormatterStepSerializationRoundtrip.java b/lib/src/main/java/com/diffplug/spotless/FormatterStepSerializationRoundtrip.java
@@ -64,6 +64,13 @@ protected FormatterFunc stateToFormatter(EqualityState equalityState) throws Exc
 		return equalityStateToFormatter.apply(equalityState);
 	}
 
+	private void writeObject(java.io.ObjectOutputStream out) throws IOException {
+		if (roundtripStateInternal == null) {
+			roundtripStateInternal = ThrowingEx.get(this::roundtripStateSupplier);
+		}
+		out.defaultWriteObject();
+	}
+
 	HackClone<?, ?> hackClone(boolean optimizeForEquality) {
 		return new HackClone<>(this, optimizeForEquality);
 	}

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,13 @@ protected FormatterFunc stateToFormatter(EqualityState equalityState) throws Exc`
`64`	`64`	`return equalityStateToFormatter.apply(equalityState);`
`65`	`65`	`}`
`66`	`66`
	`67`	`+ private void writeObject(java.io.ObjectOutputStream out) throws IOException {`
	`68`	`+ if (roundtripStateInternal == null) {`
	`69`	`+ roundtripStateInternal = ThrowingEx.get(this::roundtripStateSupplier);`
	`70`	`+ }`
	`71`	`+ out.defaultWriteObject();`
	`72`	`+ }`
	`73`	`+`
`67`	`74`	`HackClone<?, ?> hackClone(boolean optimizeForEquality) {`
`68`	`75`	`return new HackClone<>(this, optimizeForEquality);`
`69`	`76`	`}`