Skip to content

compile custom components before we get app wraps of the pages#5405

Merged
adhami3310 merged 2 commits intomainfrom
compile-custom-components-before-we-get-app-wraps-of-the-pages
Jun 2, 2025
Merged

compile custom components before we get app wraps of the pages#5405
adhami3310 merged 2 commits intomainfrom
compile-custom-components-before-we-get-app-wraps-of-the-pages

Conversation

@adhami3310
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Optimizes the compilation order in reflex/app.py by moving custom component compilation before page processing, ensuring components are available during the rendering phase.

  • Moved custom component compilation block before the page processing loop to prevent potential rendering issues
  • Ensures custom components are fully compiled before any recursive component traversal begins
  • Improves dependency resolution by having all components available during the initial render phase

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Jun 2, 2025

CodSpeed Performance Report

Merging #5405 will not alter performance

Comparing compile-custom-components-before-we-get-app-wraps-of-the-pages (34a33b2) with main (ecf3be8)

Summary

✅ 8 untouched benchmarks

@adhami3310 adhami3310 merged commit 31be645 into main Jun 2, 2025
41 checks passed
@adhami3310 adhami3310 deleted the compile-custom-components-before-we-get-app-wraps-of-the-pages branch June 2, 2025 22:57
adhami3310 added a commit that referenced this pull request Jun 3, 2025
* compile custom components before we get app wraps of the pages

* maybe
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