Update tx-generator to not use deprecated cardano-api API by carbolymer · Pull Request #6602 · IntersectMBO/cardano-node

carbolymer · 2026-06-17T08:28:22Z

Summary

Migrate tx-generator to use the experimental cardano-api for transaction building and era dispatch.

Use IsEra constraint and Cardano.Api.Experimental transaction building across the tx construction and submission pipeline
Narrow era support to Conway and Dijkstra only - pre-Conway eras are no longer exercised by the benchmarking workbench
Remove pre-Conway CBOR metadata property tests that were already broken at runtime
Dijkstra metadata tests commented out pending makeUnsignedTx support in cardano-api

Checklist

Commit sequence broadly makes sense and commits have useful messages
New tests are added if needed and existing tests are updated
Any changes are noted in the CHANGELOG.md for affected package
The version bounds in .cabal files are updated
CI passes

carbolymer · 2026-06-29T14:36:00Z

Experimental API migration guide: https://gist.github.com/carbolymer/1967d68a8d4ed4b80cbbf50ee8698bf8

carbolymer · 2026-06-30T12:01:08Z

          , teRawCBOR = "X \vl1~\182\201v(\152\250A\202\157h0\ETX\248h\153\171\SI/m\186\242D\228\NAK\182(&\162"
          }
-    era = AnyCardanoEra AllegraEra
+    era = AnyCardanoEra ConwayEra


selftest is a smoke test for tx-generator so I presume we don't have to use Allegra here, and conway is fine?

Copilot

Pull request overview

This PR migrates the tx-generator benchmarking/workbench code from deprecated cardano-api APIs to the experimental transaction-building and era-dispatch APIs, while narrowing the exercised era surface to Conway and Dijkstra.

Changes:

Switch multiple tx construction/submission paths to Cardano.Api.Experimental primitives (IsEra, useEra, makeUnsignedTx, signTx, etc.).
Narrow era dispatch in benchmarking scripts/selftests to Conway/Dijkstra and update protocol-parameters file references to protocol-parameters-v10.json.
Remove/comment out pre-Conway (and currently-broken) metadata CBOR property tests, and refactor the remaining metadata sizing implementation to use experimental tx building.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
bench/tx-generator/tx-generator.cabal	Updates dependencies/ghc-options to match the experimental API migration and reduced era surface.
bench/tx-generator/test/Main.hs	Drops pre-Conway metadata property tests; keeps Conway tests and comments out Dijkstra pending support.
bench/tx-generator/src/Cardano/TxGenerator/UTxO.hs	Moves UTxO output/fund construction to `IsEra` + `obtainCommonConstraints`.
bench/tx-generator/src/Cardano/TxGenerator/Utils.hs	Adds `txIdFromTx` and updates `TxInMode` construction to experimental-era constraints.
bench/tx-generator/src/Cardano/TxGenerator/Tx.hs	Reimplements `genTx` using experimental unsigned tx building + signing.
bench/tx-generator/src/Cardano/TxGenerator/PureExample.hs	Updates the pure example to a post-Conway era and new protocol-parameters filename.
bench/tx-generator/src/Cardano/TxGenerator/Genesis.hs	Ports genesis expenditure transaction building/signing to experimental API.
bench/tx-generator/src/Cardano/TxGenerator/Fund.hs	Narrows witness-era matching to Conway/Dijkstra under `IsEra`.
bench/tx-generator/src/Cardano/Benchmarking/Script/Selftest.hs	Updates protocol-parameters filename and switches selftest era to Conway.
bench/tx-generator/src/Cardano/Benchmarking/Script/Core.hs	Narrows era dispatch and ports generator evaluation/fee estimation to experimental-era constraints.
bench/tx-generator/src/Cardano/Benchmarking/GeneratorTx/SubmissionClient.hs	Ports submission client to `IsEra` and uses `txIdFromTx` for TxId extraction.
bench/tx-generator/src/Cardano/Benchmarking/GeneratorTx/SizedMetadata.hs	Reworks metadata sizing measurement to build dummy txs via the experimental API.
bench/tx-generator/src/Cardano/Benchmarking/GeneratorTx.hs	Switches the benchmark entrypoint constraints to `IsEra` and uses `obtainCommonConstraints`.
bench/tx-generator/app/calibrate-script.hs	Ports the calibration script’s tx sizing/fee approximation logic to experimental tx building and Plutus witness construction.

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

Jimbo4350

Some more leftover types.

Jimbo4350

There are still a few sites that use the old api's types. If you want to merge what you have and deal with them in a follow up PR that's fine but let me know.

Jimbo4350 · 2026-07-08T16:29:28Z

+    let era = useEra @ConwayEra
+    anyPlutusScript <- obtainCommonConstraints era $
+      case script of
+        ScriptInAnyLang _lang (PlutusScript version (PlutusScriptSerialised sbs)) -> do


This is still an old api type and we should be using AnyScript era. However there is no helper function to make this easy so I added it here: IntersectMBO/cardano-api#1245

^ This can stay as is for now since 1245 needs to be merged and released.

Jimbo4350 · 2026-07-08T18:16:02Z

+
+      -- just placeholders
+      dummyTxIn ix = mkTxIn $ "900fc5da77a0747da53f7675cbb7d149d46779346dea2f879ab811ccc72a2162#" <> textShow @Int ix
+      dummyTxOut   = TxOut (keyAddress (Testnet (NetworkMagic 42)) keyBenchmarkInputs) (lovelaceToTxOutValue shelleyBasedEra 1_000_000) TxOutDatumNone ReferenceScriptNone


Still using the old api's TxOut here.

Jimbo4350 · 2026-07-08T18:17:07Z

+
+    let
+      witVKey = makeKeyWitness era unsignedTx (WitnessPaymentKey keyBenchmarkInputs)
+      dummyTx :: Tx ConwayEra


This is the old api's Tx type.

Jimbo4350 · 2026-07-08T18:17:35Z

+  txBodyContent = Exp.defaultTxBodyContent
+    & Exp.setTxIns [(mkTxIn "dbaff4e270cfb55612d9e2ac4658a27c79da4a5271c6f90853042d1403733810#0", AnyKeyWitnessPlaceholder)]
+    & Exp.setTxMetadata expMetadata
+  dummyTx :: Tx era


Old api's Tx type.

Jimbo4350 · 2026-07-08T18:18:22Z

-                              scriptRedeemer
-                              executionUnits
-      in return (ScriptWitness ScriptWitnessForSpending scriptWitness, script, getScriptData scriptData, scriptFee)
+    ScriptInAnyLang _ PlutusScript{} ->


Old api's ScriptInAnyLang type here.

Jimbo4350 · 2026-07-08T18:19:22Z

+    unsignedTx <- first (\err -> TxGenError $ "genTx: " ++ show err) $ makeUnsignedTx era txBodyContent
+    let witVKeys = [makeKeyWitness era unsignedTx (WitnessPaymentKey key) | key <- allKeys]
+    let tx = case signTx era [] witVKeys unsignedTx of
+                SignedTx signedLedgerTx -> ShelleyTx shelleyBasedEra signedLedgerTx


Old api's Tx type used here.

Jimbo4350 · 2026-07-08T18:19:57Z

+    mkTxOut v = case forEraMaybeEon (cardanoEra @era) of
+      Nothing -> error "mkUtxOScript: scriptDataSupportedInEra==Nothing"
+      Just tag -> TxOut
+                    plutusScriptAddr


Old api's TxOut type used here.

Jimbo4350 · 2026-07-08T20:17:49Z

 where
-  eTx         = mkGenesisTransaction (castKey inputKey) ttl fee [pseudoTxIn] [txout]
+  era = useEra @era
  txout       = TxOut addr value TxOutDatumNone ReferenceScriptNone


Old TxOut type used here.

Jimbo4350 · 2026-07-08T20:18:08Z

-    }
+mkUTxOVariant networkId key value = obtainCommonConstraints (useEra @era) $
+  let
+    mkTxOut v = TxOut (keyAddress @era networkId key) (lovelaceToTxOutValue (shelleyBasedEra @era) v) TxOutDatumNone ReferenceScriptNone


Old TxOut type used here.

carbolymer self-assigned this Jun 17, 2026

carbolymer linked an issue Jun 17, 2026 that may be closed by this pull request

Migrate tx-generator to the experimental Cardano API (Conway and later) #6592

Open

16 tasks

carbolymer force-pushed the mgalazyn/feature/update-cardano-rpc-11.0 branch from 2064644 to 0e48d61 Compare June 18, 2026 10:40

Base automatically changed from mgalazyn/feature/update-cardano-rpc-11.0 to master June 19, 2026 12:45

carbolymer force-pushed the mgalazyn/chore/update-tx-generator-to-use-experimental-api branch 3 times, most recently from adb450f to fd7f5d5 Compare June 29, 2026 13:24

carbolymer force-pushed the mgalazyn/chore/update-tx-generator-to-use-experimental-api branch 3 times, most recently from b95df49 to 892b6c9 Compare June 30, 2026 11:55

carbolymer commented Jun 30, 2026

View reviewed changes

carbolymer force-pushed the mgalazyn/chore/update-tx-generator-to-use-experimental-api branch 4 times, most recently from 24672d4 to 44e94ac Compare June 30, 2026 14:32

Jimbo4350 requested changes Jun 30, 2026

View reviewed changes

Comment thread bench/tx-generator/app/calibrate-script.hs Outdated

Comment thread bench/tx-generator/app/calibrate-script.hs Outdated

Comment thread bench/tx-generator/app/calibrate-script.hs Outdated

carbolymer requested a review from Jimbo4350 July 1, 2026 13:01

carbolymer force-pushed the mgalazyn/chore/update-tx-generator-to-use-experimental-api branch from 44e94ac to 8d46065 Compare July 1, 2026 13:01

carbolymer requested a review from Copilot July 1, 2026 13:01

Copilot started reviewing on behalf of carbolymer July 1, 2026 13:02 View session

Copilot AI reviewed Jul 1, 2026

View reviewed changes

carbolymer force-pushed the mgalazyn/chore/update-tx-generator-to-use-experimental-api branch 2 times, most recently from 2615b9d to b6acf4c Compare July 2, 2026 11:06

carbolymer marked this pull request as ready for review July 2, 2026 12:31

carbolymer requested review from a team as code owners July 2, 2026 12:31

carbolymer changed the title ~~Update tx-generator to not use deprecated cardano-api api~~ Update tx-generator to not use deprecated cardano-api API Jul 2, 2026

carbolymer commented Jul 2, 2026

View reviewed changes

Comment thread bench/tx-generator/app/calibrate-script.hs Outdated

Jimbo4350 requested changes Jul 2, 2026

View reviewed changes

Comment thread bench/tx-generator/src/Cardano/TxGenerator/Fund.hs Outdated

Comment thread bench/tx-generator/src/Cardano/TxGenerator/Genesis.hs Outdated

Update tx-generator to not use deprecated cardano-api api

4dc6a4b

carbolymer force-pushed the mgalazyn/chore/update-tx-generator-to-use-experimental-api branch from b6acf4c to 4dc6a4b Compare July 2, 2026 19:13

carbolymer requested a review from Jimbo4350 July 2, 2026 19:13

Jimbo4350 requested changes Jul 8, 2026

View reviewed changes

Jimbo4350 reviewed Jul 8, 2026

View reviewed changes

Uh oh!

Conversation

carbolymer commented Jun 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Checklist

Uh oh!

carbolymer commented Jun 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Jimbo4350 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Jimbo4350 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Jimbo4350 Jul 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

carbolymer commented Jun 17, 2026 •

edited

Loading

carbolymer commented Jun 29, 2026 •

edited

Loading

Jimbo4350 left a comment •

edited

Loading

Jimbo4350 Jul 8, 2026 •

edited

Loading