fix: MoveTool drag translate keeps three-point arcs

[grootstebozewolf]

HOLD merge until PO. UX SIGNs the MoveTool fix from the next JAR after this PR lands a tip — do not merge.

TestBuilder-only honesty fix on #7 (cbc45d4a). Do not join #55 #52 #38. Do not revert #54 #57 #58. No new WKB / type number. Type set stays 1–7 + 8–12. Type 19 stays named fallback. No Bézier I/O type. Off #7 as a core type.

Do not retip pin JAR 61eb3377. Testers still shoot Function-tree AffineTranslation (10, 8) on that pin. Do not rebuild the guides JAR.

Canvas miss is MoveTool drag

Java2D draws arcs with Béziers (hidden canvas controls). Those points are a draw path, not the JTS geometry. The miss is MoveTool drag on logoLines in A, not the Function-tree AffineTranslation (10, 8).

MoveTool.execute (whole-geom, not Ctrl-component):

AffineTransformation trans = AffineTransformation.translationInstance(dx, dy);
GeometryComponentTransformer.transform(geom, trans)

which is geom.copy(); geom.apply(trans).

After drag, type stayed MULTICURVE but arcs lied: each later CIRCULARSTRING start was untranslated while mid/end moved.

Same apply as Function-tree — verified, not a Function-tree SIGN

Function-tree AffineTransformation.translate is:

AffineTransformation.translationInstance(dx, dy).transform(g)

and AffineTransformation.transform is also copy(); apply(this).

Same apply. Stated. Not a UX SIGN that Function-tree AffineTranslation (10, 8) is the canvas miss. Testers still shoot that function on pin JAR 61eb3377. Do not retip.

Cause

CompoundCurve.apply inherited LineString.apply, which walks only the concatenated sequence. concatMembers drops each later member’s start. Those starts are distinct Coordinate objects. Mid+end moved; start did not.

No PathIterator / ShapeWriter / CurveShapeWriter round-trip bakes cubicTo into the geom. MoveTool does not reconstruct from a Shape.

CurvePolygon inherited Polygon.apply on the flat rings. A COMPOUNDCURVE shell/hole had the same skipped-start miss.

Fix

• CompoundCurve.apply visits every member (including circular starts), then syncs the concatenated sequence in place.
• CurvePolygon.apply visits structural rings when they are not the same object as the flat views.
• MoveTool.execute javadoc: canvas drag translate; same apply as AffineTransformation.transform; translate only.

Affine translate is the signed type-honest case. This does not sign that shear or non-uniform scale still describes a circular arc.

Unchanged: Geometry.buffer / logoBuffer (still toLinear + BufferOp CHORD-PATH). Named fallback stays named. Never isApproximate()=false. OverlayNGCurve not touched.

Tests

• JTSFunctionsLogoLinesTranslateTest: MoveTool path on logoLines +(10, 8). Types kept. Every CIRCULARSTRING still 3 control points. Every control point, including member starts, is +(10, 8). Same-apply check vs AffineTransformationFunctions.translate (verification, not a Function-tree SIGN).
• CompoundCurveTranslateTest: apply honesty on COMPOUNDCURVE / MULTICURVE / CURVEPOLYGON shell and hole.

Architect SIGN: translate only. UX SIGN: MoveTool from the next JAR after this tip. HOLD merge.