[BUG] - estimateTransactionKeyWitnessCount doesn't appear to consider native (simple) scripts

[sourabhxyz]

Internal/External
External

Area
Transaction building

Summary

For transactions making use of native scripts, the required number of key witness may be more than what is returned by estimateTransactionKeyWitnessCount as it doesn't consider them (native scripts). Fwiw, the key witnesses required by native script are taken from transaction_witness_set (see attached image from Shelley ledger specification) and thus the required signatories mentioned by these scripts need not have any mention in transaction_body.

Wrong estimate by estimateTransactionKeyWitnessCount leads to wrong estimate of required transaction fees when building the transaction using cardano-api's makeTransactionBodyAutoBalance unless one gives appropriate key witness estimate to it.

[carbolymer]

cc: @Jimbo4350

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.