fix(convert): notify DOCX uploads with mixed-case extensions

[Mr-Neutr0n]

Problem

process_message only recognized keys ending in lowercase .docx when deciding whether to send the completion or failure callback. The conversion parser accepts extensions case-insensitively, so .DOCX uploads could convert successfully while the client received no status update.

Fix

Use the same case-insensitive extension behavior for callback routing and add focused tests for mixed-case and trailing-extension cases.

Verification

• cargo test -p convert_service docx_detection --offline
• cargo fmt --all -- --check

No existing open issue or pull request covers this callback mismatch.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 0e9e5832-6631-404f-9e96-55742fd34bf0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@services/convert_service/src/process/convert.rs`:
- Around line 133-148: Move the inline test module containing
docx_detection_accepts_mixed_case_extensions and
docx_detection_requires_the_final_extension from convert.rs into
process/test.rs. Replace the inline body in convert.rs with the separate-module
declaration #[cfg(test)] mod test;, preserving the existing imports and test
assertions.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 999f1cdd-c1f3-4531-b619-c587407092cd

📥 Commits

Reviewing files that changed from the base of the PR and between 412bc88 and 7fc6bd4.

📒 Files selected for processing (1)

• services/convert_service/src/process/convert.rs

[whutchinson98]

Hi @Mr-Neutr0n thanks for your contribution!

Just a minor nit then we are good to merge

[Mr-Neutr0n]

Addressed the review: moved the DOCX callback regression tests into process/test.rs, declared the module from process/mod.rs, and verified all convert_service tests pass. Mr-Neutr0n@8d40a1f