Skip to content

script_debug: expose SigVersion in callback#145

Merged
sedited merged 1 commit intosedited:feature_script_debugfrom
febyeji:script-debug/sigversion
Mar 26, 2026
Merged

script_debug: expose SigVersion in callback#145
sedited merged 1 commit intosedited:feature_script_debugfrom
febyeji:script-debug/sigversion

Conversation

@febyeji
Copy link
Copy Markdown

@febyeji febyeji commented Mar 5, 2026

Summary

  • Expose SigVersion (BASE/WITNESS_V0/TAPROOT/TAPSCRIPT) in the debug callback
  • Add SigVersion enum to Rust bindings with TryFrom<u8>

Tests

  • test_script_debug: asserts SigVersion::Base for P2PKH frames

Depends on #143 and #144.

@febyeji febyeji force-pushed the script-debug/sigversion branch from 4a6d93b to 24b9e8f Compare March 5, 2026 08:36
Copy link
Copy Markdown
Owner

@sedited sedited left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just needs to fix from its parent PR.

@sedited
Copy link
Copy Markdown
Owner

sedited commented Mar 26, 2026

Needs a rebase :)

@febyeji febyeji force-pushed the script-debug/sigversion branch from 24b9e8f to 7b3da27 Compare March 26, 2026 10:14
Copy link
Copy Markdown
Owner

@sedited sedited left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sorry for taking this long to get around to looking at this again. I'll also be away for the next few weeks. I'd still be very interested in a more interactive implementation of the debugger. Something where can developers can provide and edit op codes and push data on the fly, provide transactions, or just singular script fragments. and interactively step through execution like a normal debugger.

@sedited sedited merged commit d621fed into sedited:feature_script_debug Mar 26, 2026
9 checks passed
@febyeji
Copy link
Copy Markdown
Author

febyeji commented Mar 26, 2026

@sedited Thank you for the reviews! I'll work on designing the interfaces for the interactive debuggers. I will look into the features that you mentioned. (editable opcodes, push data, step-through execution etc.) Will open follow-up PRs. Enjoy your time off!

@febyeji febyeji deleted the script-debug/sigversion branch March 26, 2026 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants