[oeddyo]

Summary

Fixes #428.

On Android, a MediaCodec.CodecException thrown during Video.compress usually has an empty message, so the promise rejects with nothing useful (CodecException: or just start failed). Every codec failure looks the same from JS. Before #424 that is how the Exynos NO_MEMORY failures reached us: empty, with the cause only in logcat.

CodecException.getDiagnosticInfo() already names the error (android.media.MediaCodec.error_neg_12 is NO_MEMORY). This PR appends it to the failure message at the two catch sites that wrap codec work in Compressor.start:

before: ""                 after: "android.media.MediaCodec.error_neg_12"
before: "start failed"     after: "start failed: android.media.MediaCodec.error_neg_12"

Messages from other exceptions are unchanged. The finishMovie() catch only touches the muxer, so it is left alone.

Changelog

[ANDROID] [FIXED] - Video compression failures now name the MediaCodec error instead of an empty message

Test Plan

• Ran local JS PR gate: yarn test:pr (passes, 13/13)
• yarn build:android (BUILD SUCCESSFUL)
• Ran yarn test:harness:android on an arm64 API emulator: 8/9 passed. The one failure, compresses a real image fixture through the exported Image API (expected 16 to be less than or equal to 8), fails the same way on unmodified main with the same emulator, so it is unrelated. The harness has no Video.compress failure case, so this is a regression check only.

The CodecException branch was not runtime-verified: I couldn't force a real CodecException on an emulator. The diagnosticInfo format comes from AOSP MediaCodec.CodecException ("android.media.MediaCodec.error_" + (errorCode < 0 ? "neg_" : "") + Math.abs(errorCode)), available since API 21 (minSdk is 24).

🤖 Generated with Claude Code

[Copilot]

🟢 Approval recommended

The narrow error-formatting change addresses the reported issue without altering compression behavior, and no blocking issues remain.

No open issues 🎉

What changed in this PR

Improves Android video compression errors for #428 by including codec diagnostics in failure messages, making failures such as Exynos NO_MEMORY identifiable from JavaScript.

Changes:

• Adds a shared formatter that includes MediaCodec.CodecException.diagnosticInfo at both codec-work catch sites.
• Preserves non-codec exception messages and leaves compression logic unchanged.
• Records the fix in the triage log.

The codec-exception branch was not runtime-verified, as noted in the test plan.

File	Description
TRIAGE.md	Records #428 and its diagnostic-message fix.
android/​src/​main/​java/​com/​reactnativecompressor/​Video/​VideoCompressor/​compressor/​Compressor.kt	Adds codec diagnostics to both relevant failure paths.

_{Review effort: Balanced}