Skip to content

Preserve lazy runtime types during reconstruction - #4

Merged
trissim merged 1 commit into
mainfrom
codex/preserve-lazy-reconstruction-identity
Jul 30, 2026
Merged

Preserve lazy runtime types during reconstruction#4
trissim merged 1 commit into
mainfrom
codex/preserve-lazy-reconstruction-identity

Conversation

@trissim

@trissim trissim commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • preserve a registered lazy dataclass runtime type while reconstructing raw ObjectState parameter snapshots
  • prevent unresolved inheritance sentinels from being validated through an annotated concrete base class
  • add a constrained-field regression and bump ObjectState to 1.0.22

Architecture

The fix remains domain-blind. ObjectState queries the existing lazy-type registry and preserves the runtime owner only for raw reconstruction. Resolved reconstruction continues to normalize lazy types through their registered concrete base.

Verification

  • 159 ObjectState tests passed
  • exact OpenHCS GlobalPipelineConfig / LazyNapariStreamingConfig reconstruction smoke passed
  • Ruff added-lines gate passed
  • wheel and sdist built; Twine checks passed

Copilot AI review requested due to automatic review settings July 30, 2026 19:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@augmentcode

augmentcode Bot commented Jul 30, 2026

Copy link
Copy Markdown

PR Risk Analyzer Agent🛡️

👀 Human Input Needed
A pair-review briefing is ready for you.

→ Pair Review Briefing

@trissim
trissim merged commit 1b433ed into main Jul 30, 2026
11 checks passed
@trissim
trissim deleted the codex/preserve-lazy-reconstruction-identity branch July 30, 2026 19:46

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Deep Code Review Agent🐛

Review completed with 0 suggestions.

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