Share common transaction-body arguments across builder commands
Maintainers usually reply within 1 day
@Jimbo4350 is already working on this.
Since Oct 13, 2026.
Assessment
This issue has not been assessed yet.
Description
SubTransactionBuildRawCmdArgs repeats most transaction-body fields from TransactionBuildRawCmdArgs. Adding or changing a shared body field therefore requires parallel edits to the argument records and parser chains. TransactionBuildCmdArgs and TransactionBuildEstimateCmdArgs also repeat these fields.
Introduce a shared record for common transaction-body arguments and reuse the corresponding parser components across the applicable builders. Cover the top-level build, build-raw, build-estimate, and sub-transaction build-raw commands where their fields overlap, while retaining command-specific fields and restrictions.
Preserve option names, grouping/order, era restrictions, and transaction-building behavior. Update affected runners and retain test coverage for each command.
Deferred from PR #1453: review comment. Jordan marked this non-critical and suitable for a future PR.
- Dominant language
- Haskell
- Stars
- 72
- Forks
- 25
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 10
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from IntersectMBO/cardano-cli
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
IntersectMBO/cardano-cli#1306 · 2 comments ·
Maintainers usually reply within 1 day
-
Clarify the origin and intended use of Plutus V2/V4 test fixturesPossibly taken @Jimbo4350 claimed this 1 day ago. Open
IntersectMBO/cardano-cli#1464 · 1 assignee ·
Maintainers usually reply within 1 day
-
Share body construction and input reading between transaction and sub-transaction buildersPossibly taken @Jimbo4350 claimed this 1 day ago. Open
IntersectMBO/cardano-cli#1463 · 1 assignee ·
Maintainers usually reply within 1 day
-
Move Transaction.Internal.Common helpers into focused modulesPossibly taken @Jimbo4350 claimed this 1 day ago. Open
IntersectMBO/cardano-cli#1461 · 1 assignee ·
Maintainers usually reply within 1 day
-
Ensure Plutus V4 scripts can be deserialised by cardano-cliPossibly taken @carbolymer claimed this 21 days ago. Openenhancement
IntersectMBO/cardano-cli#1448 · 1 comment · 1 assignee ·
Maintainers usually reply within 1 day
All issues in IntersectMBO/cardano-cli
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 3 days
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
simplex-chat/simplex-chat#7714 · 1 reaction ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 2 days
-
component: hls-eval-plugin status: needs triage type: bug
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
haskell/haskell-language-server#5144 · 2 comments ·
Maintainers usually reply within 1 day
-
[BUG]: -std selection does not work with COBOL -> GnuCOBOL 1.1Possibly taken @Anandb71 claimed this today. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
compiler-explorer/compiler-explorer#9272 ·
Maintainers usually reply within 1 day