Skip to content

Fix reflex-hosting-cli on v0.7.6.post - #6963

Merged
masenf merged 2 commits into
mainfrom
masenf/reflex-hosting-cli-0.7.6-compat
Aug 28, 2026
Merged

Fix reflex-hosting-cli on v0.7.6.post#6963
masenf merged 2 commits into
mainfrom
masenf/reflex-hosting-cli-0.7.6-compat

Conversation

@masenf

@masenf masenf commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Logic was comparing PEP 440 versions, which had an unintended effect for pre and post release version markers. The new "release" based check ignores pre/post/alpha releases and only compares against the version tuple (0, 7, 7).

Verified empirically with published version numbers.

Review in cubic

Logic was comparing PEP 440 versions, which had an unintended effect for pre
and post release version markers. The new "release" based check ignores
pre/post/alpha releases and only compares against the version tuple (0, 7, 7)
@masenf
masenf requested a review from a team as a code owner August 28, 2026 05:56
@greptile-apps

greptile-apps Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR changes hosting CLI export compatibility detection to compare the installed Reflex release tuple against 0.7.7, preventing post-release markers such as 0.7.6.post1 from selecting the wrong callback signature.

  • Uses the six-argument export callback for release tuples before 0.7.7.
  • Uses the seven-argument callback beginning with the 0.7.7 release tuple.
  • Adds a bugfix news entry.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
packages/reflex-hosting-cli/src/reflex_cli/v2/cli.py Replaces full PEP 440 ordering with release-tuple comparison for both backend and frontend export callback selection.
packages/reflex-hosting-cli/news/6963.bugfix.md Documents restored compatibility with Reflex 0.7.6.post1.

Reviews (2): Last reviewed commit: "Fix version comparison bug for reflex-0...." | Re-trigger Greptile

Comment thread packages/reflex-hosting-cli/src/reflex_cli/v2/cli.py

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

@codspeed-hq

codspeed-hq Bot commented Aug 28, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 27 untouched benchmarks
⏩ 8 skipped benchmarks1


Comparing masenf/reflex-hosting-cli-0.7.6-compat (cb69e69) with main (f7c848f)

Open in CodSpeed

Footnotes

  1. 8 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@masenf
masenf merged commit eb65356 into main Aug 28, 2026
111 checks passed
@masenf
masenf deleted the masenf/reflex-hosting-cli-0.7.6-compat branch August 28, 2026 23:43
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