Skip to content

Fix backend-only compile marker cleanup - #7125

Open
harsh21234i wants to merge 2 commits into
reflex-dev:mainfrom
harsh21234i:fix/7089-backend-only-nocompile
Open

Fix backend-only compile marker cleanup#7125
harsh21234i wants to merge 2 commits into
reflex-dev:mainfrom
harsh21234i:fix/7089-backend-only-nocompile

Conversation

@harsh21234i

@harsh21234i harsh21234i commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Closes #7089

Summary

Fix backend-only development runs leaving .web/nocompile, causing the
next full reflex run to skip frontend compilation.

Root Cause

run_backend() created the compile-skip marker unconditionally, including
when started with --backend-only.

Changes

  • Only full-stack backend runs create .web/nocompile.
  • Backend-only runs remove any stale marker.
  • Added regression tests covering backend-only and full-stack behavior.
  • Added a bugfix news fragment.

Testing

  • git diff --check
  • Ruff formatting check
  • Added unit coverage in tests/units/utils/test_exec.py

Review in cubic

@harsh21234i
harsh21234i requested a review from a team as a code owner September 12, 2026 12:39
@greptile-apps

greptile-apps Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge because the marker lifecycle fix is directly covered for both backend-only and full-stack execution.

Summary

  • Backend-only runs now remove stale compile-skip markers.
  • Full-stack backend runs continue creating the marker.
  • Regression tests independently verify marker creation and cleanup.

Reviews (2) · Last reviewed commit: "Cover backend marker creation in regress..."

Comment thread tests/units/utils/test_exec.py Outdated

@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 3 files

Re-trigger cubic

@codspeed-hq

codspeed-hq Bot commented Sep 12, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 40 untouched benchmarks
⏩ 8 skipped benchmarks1


Comparing harsh21234i:fix/7089-backend-only-nocompile (d810313) with main (9165dc6)

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.

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.

A backend-only run leaves .web/nocompile and makes the next full run skip compilation

1 participant