Skip to content

feat(packaging): add langfuse optional extra#48589

Open
tt-a1i wants to merge 1 commit into
NousResearch:mainfrom
tt-a1i:codex/langfuse-optional-extra
Open

feat(packaging): add langfuse optional extra#48589
tt-a1i wants to merge 1 commit into
NousResearch:mainfrom
tt-a1i:codex/langfuse-optional-extra

Conversation

@tt-a1i

@tt-a1i tt-a1i commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Fixes #48576

Summary

  • Add a langfuse optional dependency extra for the bundled observability/langfuse plugin.
  • Regenerate uv.lock, adding only langfuse==4.9.0 and its backoff==2.2.1 transitive dependency.
  • Document the NixOS extraDependencyGroups = [ "langfuse" ] path for sealed uv2nix/container deployments.

Notes

  • hermes-agent[langfuse] is intentionally not included in [all], so default installs remain unchanged.
  • No LAZY_DEPS entry is added because the target use case is build-time inclusion in a sealed Nix venv, where runtime installation is not available.

Tests

  • /tmp/hermes-uv-lock-venv/bin/uv lock --check
  • python3.11 -c "...tomllib structure assertions..."
  • UV_PROJECT_ENVIRONMENT=/tmp/hermes-langfuse-extra-import-env-rebase /tmp/hermes-uv-lock-venv/bin/uv run --extra langfuse --locked --no-dev python -c "from langfuse import Langfuse, propagate_attributes; print(Langfuse.__name__, callable(propagate_attributes))"
  • git diff --check origin/main...HEAD

Not run

  • cd website && npm run typecheck: local website dependencies are not installed (node_modules missing; Docusaurus packages unresolved).
  • cd website && npm run build: local website dependencies are not installed (docusaurus: command not found); prebuild also reports missing Python yaml in this local environment.

@tt-a1i tt-a1i requested a review from a team June 18, 2026 18:42
@alt-glitch alt-glitch added type/feature New feature or request comp/plugins Plugin system and bundled plugins area/nix Nix flake, NixOS module, container packaging P3 Low — cosmetic, nice to have labels Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/nix Nix flake, NixOS module, container packaging comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Add a langfuse optional-dependency extra so the bundled observability/langfuse plugin is installable on NixOS (uv2nix) + container mode

2 participants