Skip to content

Conversation

@zharinov
Copy link
Member

@zharinov zharinov commented Jan 2, 2026

Summary

  • Add is_optional() helper to TypeDef for checking Optional wrapper types
  • Add unwrap_optional() to QTypeTable that returns (inner_type, is_optional) tuple
  • Update typescript.rs to use the public API instead of private implementation
  • Alphabetize module imports in lib.rs

This refactor moves the optional-unwrapping logic from typegen into the bytecode module where it can be reused by other consumers.

Test plan

  • All existing tests pass (cargo test -p plotnik-lib)

@zharinov zharinov enabled auto-merge (squash) January 2, 2026 00:09
@zharinov zharinov merged commit d78b3b0 into master Jan 2, 2026
4 checks passed
@zharinov zharinov deleted the refactor/unwrap-optional branch January 2, 2026 00:10
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