Repository navigation
chore(langfuse): move integration to plugins repository - #418
Yangtze-Seventh wants to merge 1 commit into
Conversation
gloryfromca
left a comment
There was a problem hiding this comment.
Blocking: Land a mergeable companion migration before deleting the current integration.
The revision deletes the only runnable/documented Langfuse integration from EverOS, but EverMind-AI/plugins#10 is still open and currently conflicted; plugins main does not contain langfuse/ yet. If this PR lands first, released main has no replacement for the removed replay/demo/recording workflow. Please merge (or otherwise make available) the companion migration first, then this cleanup can safely follow.
I covered CLAUDE.md and all applicable .claude/rules, the complete origin/main...HEAD deletion diff, repository callers/references and history, backward compatibility of the example workflow, unchanged tests, and the DDD/observability architecture. I refuted code-level concerns: all core Langfuse-compatible OpenTelemetry settings, attributes, score delivery, and their tests remain unchanged; the five non-README artifacts are byte-identical in the companion branch (with intentional README/replay path updates). uv run --frozen pytest -q tests/unit/test_config/test_observability_settings.py tests/unit/test_core/test_observability tests/unit/test_core/test_lifespan/test_tracing_lifespan.py tests/unit/test_service/test_memorize_span.py: 53 passed. Existing PR CI is green.
| @@ -1,135 +0,0 @@ | |||
| # EverOS × Langfuse (native OpenTelemetry) | |||
There was a problem hiding this comment.
The companion migration must be merged before this deletion, otherwise main loses the documented runnable Langfuse integration while the replacement remains unavailable.
EverMind-AI/plugins#10 has not landed and currently reports merge conflicts, while plugins main has no langfuse/ directory. Merging this deletion first therefore makes the replay, live demo, and recording fixture unavailable to users of released EverOS, instead of moving them. Please make the replacement available before removing this source.
There was a problem hiding this comment.
Two more things before this goes in:
1. Langfuse's official docs link here. The EverOS integration page on langfuse.com (https://langfuse.com/integrations/other/everos) points to this directory in four places:
- the span/score reference (
examples/langfuse/README.md) - the recording + replay script (
examples/langfuse/) - the live demo instructions (
python demo.py # from examples/langfuse/) - the Resources list
Deleting the directory breaks all four. Their page also relies on our README as the reference for span and score names. So we need to either get the Langfuse docs updated, or keep a README here (see below).
2. Keep a lightweight example entry point in the main repo. Users usually find use cases through the main EverOS repo, and many don't know the plugins repo exists. My suggestion:
- Keep
examples/langfuse/README.mdhere: what the integration does, setup, the span/score reference, and a link to the full example inEverMind-AI/plugins/langfuse. - Move the runnable scripts (
.py) and the recording fixtures (.json) to the plugins repo.
That keeps the main repo light and the use case discoverable. The Langfuse links keep working too, except the demo.py command, which we'd ask them to update to the new path.
Summary
examples/langfusedirectory after migrating it toEverMind-AI/plugins/langfuseCompanion migration: EverMind-AI/Plugins#10
Scope verification
examples/langfuseexamples/langfuseoutside historical CHANGELOG entriesTests
PYTHONPATH=src python -m pytest -q tests/unit/test_config/test_observability_settings.py tests/unit/test_core/test_observability tests/unit/test_core/test_lifespan/test_tracing_lifespan.py tests/unit/test_service/test_memorize_span.py