Skip to content

script-debug: expose ScriptError codes from kernel verification API#143

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

script-debug: expose ScriptError codes from kernel verification API#143
sedited merged 1 commit intosedited:feature_script_debugfrom
febyeji:script-debug/expose-script-error

Conversation

@febyeji
Copy link
Copy Markdown

@febyeji febyeji commented Mar 5, 2026

Summary

  • Expose the C++ ScriptError_t enum as ScriptError (53 variants)
  • Replace ScriptVerifyError::Invalid with ScriptVerifyError::Script(ScriptError) for specific failure reasons

Tests

  • test_script_error_codes: validates distinct ScriptError variants using minimal synthetic scripts

@febyeji febyeji force-pushed the script-debug/expose-script-error branch from fd24729 to d63b62a Compare March 5, 2026 04:59
@febyeji febyeji changed the title Expose ScriptError codes from kernel verification API script-debug: expose ScriptError codes from kernel verification API Mar 5, 2026
@febyeji febyeji force-pushed the script-debug/expose-script-error branch from d63b62a to 0fcbcea Compare March 5, 2026 08:35
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

@sedited sedited merged commit 5e3eeb3 into sedited:feature_script_debug Mar 26, 2026
9 checks passed
@febyeji febyeji deleted the script-debug/expose-script-error 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