Fix TransactionBuilder failing to debug P2PKH-only transactions

[mainnet-pat]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
cashscript	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 5, 2025 9:48am

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.40%. Comparing base (248870b) to head (621f6df).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #318      +/-   ##
==========================================
+ Coverage   97.36%   97.40%   +0.03%     
==========================================
  Files          78       78              
  Lines       17920    17926       +6     
  Branches     1144     1336     +192     
==========================================
+ Hits        17448    17461      +13     
+ Misses        470      456      -14     
- Partials        2        9       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[rkalis]

Thanks pat for this PR!

We rebased and added some extra (skipped) tests.

We realised that it would be nice to have proper evaluation also for P2PKH inputs, but that is currently not how the debugging tools work, so just skipping the debugging if it's only P2PKH (like you did here) makes a lot of sense.

We did add some skipped tests and we have an issue (#323) that discusses improving the handling of P2PKH inputs in the future.