Skip to content

fix: make height controller bundling deterministic - #5

Merged
Error-Zhang merged 1 commit into
mainfrom
codex/fix-height-controller-bundling
Jul 19, 2026
Merged

Error-Zhang merged 1 commit into
mainfrom
codex/fix-height-controller-bundling

Conversation

@Error-Zhang

Copy link
Copy Markdown
Collaborator

Summary

  • inject the height controller from explicit source instead of runtime Function#toString output
  • execute that exact embedded source in unit tests
  • preserve shrink, deduplication, feedback freeze, and hard-cap behavior across minified production builds

Verification

  • npm run check
  • minified Vite preview: iframe/body/scrollHeight settle at 352px
  • minified viewport-feedback fixture stops at 720px with internal scrollHeight 856px

@Error-Zhang
Error-Zhang merged commit 6462f0c into main Jul 19, 2026
1 check passed
@Error-Zhang
Error-Zhang deleted the codex/fix-height-controller-bundling branch July 19, 2026 12:07
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.

1 participant