ENG-9166 feat(otel): compile spans (2/3)

[FarhanAliRaza]

Part of #6227 (ENG-9166). Stack: 1/3 → 2/3 (this) → 3/3. Base is farhan/eng-9166-reflex-otel (1/3); the diff is one commit.

What

• App._compile runs inside a reflex.compile span (reflex.compile.trigger, reflex.compile.dry_run); the compile stages get explicit child spans in compiler.py: reflex.compile.evaluate_pages, .pages, .copy_assets, .install_frontend_packages, .write.
• otel.span() / otel.compile_span() helpers (no-op nullcontext when disabled; compile is not a hot path).

Log correlation (trace_id/span_id on JSON log records) was split out of this stack: it needs the JsonHandler from the logging pipeline (#6863) and will follow as a small PR once that lands (#6903).

Tests

test_otel.py (span helpers, compile_span attributes/nesting), test_app.py::test_compile_emits_stage_spans (real compile, stages nest under reflex.compile).

[linear-code]

ENG-9166

[greptile-apps]

Greptile Summary

The PR instruments application compilation with an OpenTelemetry root span and nested stage spans while remaining inert when tracing is disabled.

• Adds generic and compile-specific span context-manager helpers.
• Wraps App._compile in a compile span carrying trigger and dry-run attributes.
• Instruments page evaluation, page compilation, asset copying, package installation, and disk writes.
• Adds unit and integration coverage for span attributes, nesting, and disabled behavior.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
packages/reflex-base/src/reflex_base/otel.py	Adds no-op-aware generic and compile span helpers with compile metadata attributes.
reflex/app.py	Wraps the existing compile and telemetry-accounting flow in the new root compile span.
reflex/compiler/compiler.py	Adds child spans around the major compile stages without changing their underlying operations.
tests/units/reflex_base/test_otel.py	Covers disabled helpers, compile attributes, and parent-child span relationships.
tests/units/test_app.py	Verifies a real compile emits root and stage spans with the expected hierarchy.

_{Reviews (4): Last reviewed commit: "feat(otel): compile spans" | Re-trigger Greptile}

[codspeed-hq]

Merging this PR will not alter performance

✅ 26 untouched benchmarks
⏩ 8 skipped benchmarks¹

---

_{Comparing farhan/eng-9166-reflex-otel-3 (acf9a22) with farhan/eng-9166-reflex-otel (b0d7618)}

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. ↩

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e88ec4f6ef

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cubic-dev-ai]

All reported issues were addressed across 28 files

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[Alek99]

The initial compile trace works locally, but the common backend-startup/stateful-marker path omits one advertised stage.

[Alek99]

[P2] Include the stateful-marker backend evaluation in reflex.compile.evaluate_pages

When .web/backend exists and stateful_pages.json is present, the earlier branch evaluates each marked page with _compile_page(..., save_page=False) and returns at line 1171 before this span is entered. I reproduced that normal backend_startup path with an OTLP exporter: only the reflex.compile root was emitted even though page evaluation ran. Please wrap the marker-driven loop in the same stage span (or factor both paths through a shared helper) and cover the early-return marker path.