[Apeksha9389]

What does this PR do?

Reuse successful media resolutions within one LangfuseMedia.resolve_media_references() call. Repeated image or audio references in multimodal inputs now share their metadata lookup and download, including across nested messages. A new call fetches fresh content, and failed resolutions remain retryable at later positions.

The regression input has three copies of one reference. On main it makes three metadata requests and three downloads; with this change it makes one of each and produces the same resolved content. Additional tests cover distinct references, fetch failures, input preservation, call isolation and the traversal depth limit.

Type of change

• Bug fix
• New feature
• Breaking change
• Refactor
• Documentation update
• Tooling, CI, or repo maintenance

Verification

.venv/bin/python -m pytest tests/unit/test_media.py tests/unit/test_json_path.py -q
.venv/bin/ruff check .
.venv/bin/ruff format --check langfuse/media.py tests/unit/test_media.py
.venv/bin/mypy langfuse --no-error-summary

All pass on macOS/Python 3.12.14; 48 tests pass. The repeated-reference regression was confirmed failing before the implementation. Dependencies were installed with uv sync --locked; commands used the resulting .venv executables. Backend e2e and live-provider tests were not run because this change is covered by offline media-resolution tests.

Checklist

• I self-reviewed the diff using code_review.md.
• I added or updated tests for behavior changes.
• I updated the method documentation.
• I did not hand-edit generated files.
• I did not commit secrets or credentials.

AI disclosure: The change, tests and description were prepared with OpenAI Codex assistance. All reported checks were executed locally.

Confidence Score: 5/5

This PR appears safe to merge.

Summary

LangfuseMedia.resolve_media_references() now reuses successful downloads within one call.

• Repeated media references reuse successful downloads within one call.

_{Reviews (1) · Last reviewed commit: "perf(media): reuse successful resolution..." · Reviewed by Greptile}

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[CLAassistant]

All committers have signed the CLA.