Improve process error handling

[rob93c]

Summary by CodeRabbit

• Bug Fixes
  • Reduced ffprobe output noise during media information extraction.
  • Improved command execution by capturing standard output and error separately.
  • Timeout and failure warnings now include collected error details to help diagnose media processing issues.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ed7cff21-1ef8-423f-a2cd-d31e21926b6b

📥 Commits

Reviewing files that changed from the base of the PR and between e5e1b33 and 949722d.

📒 Files selected for processing (1)

• src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java

🚧 Files skipped from review as they are similar to previous changes (1)

• src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java

---

📝 Walkthrough

Walkthrough

Changes

Process execution now captures standard output separately from standard error and logs captured errors for timeouts and non-zero exits. Media metadata extraction invokes ffprobe with error-level logging.

Media process output handling

Layer / File(s)	Summary
Separate process output streams src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java	executeCommand reads standard output and standard error independently using virtual threads, joins both readers, updates timeout and exit handling, and documents standard-output returns.
Use ffprobe error logging src/main/java/com/github/stickerifier/stickerify/media/MediaHelper.java	retrieveMultimediaInfo changes the ffprobe verbosity option from -v quiet to -v error.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main change: improving process command error handling and timeout logging.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 7b756258-675d-4043-9f24-71a5a1a3e658

📥 Commits

Reviewing files that changed from the base of the PR and between e72b51f and e5e1b33.

📒 Files selected for processing (2)

• src/main/java/com/github/stickerifier/stickerify/media/MediaHelper.java
• src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java

[rob93c]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[MartelliEnrico]

LGTM