Merge pull request #5004 from IntersectMBO/aniketd/json-roundtrip-conway-genesis

Remove delegators field from conway-genesis JSON

Author: aniketd

libs/cardano-ledger-core/CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## 1.18.0.0
 
+* Remove `delegators` field from JSON serialiser for `DRepState` for correct round-tripping. #5004
 * Change `TxBody` to an associated `data` family
 * Remove `HeapWords` instances for: #5001
   - `Coin`

libs/cardano-ledger-core/src/Cardano/Ledger/DRep.hs

@@ -186,7 +186,8 @@ instance ToJSON DRepState where
           object $
             [ "expiry" .= toJSON drepExpiry
             , "deposit" .= toJSON drepDeposit
-            , "delegators" .= toJSON drepDelegs
+            -- Since the corresponding `FromJSON` instance ignores the `delegators` field,
+            -- we omit it from the `ToJSON` instance, ensuring that round-tripping behaves as expected.
             ]
               ++ ["anchor" .= toJSON anchor | SJust anchor <- [drepAnchor]]
 
@@ -196,7 +197,7 @@ instance FromJSON DRepState where
       <$> o .: "expiry"
       <*> o .:? "anchor" .!= SNothing
       <*> o .: "deposit"
-      -- Construction of DRep state with deleagations is intentionally prohibited, since
+      -- Construction of DRep state with delegations is intentionally prohibited, since
       -- there is a requirement to retain the invariant of delegations in the UMap
       <*> pure mempty