fix: simplify Kimi response parser

[riderx]

What changed

• Refactor Kimi/OpenAI-style response parsing into small helpers to satisfy Sonar reliability rules.
• Remove the redundant redirect response conditional flagged by Sonar.

Why

PR #610 was merged before the Sonar fixes were included. This follow-up keeps the merged behavior and fixes the quality gate failure.

Validation

• bun run ci:verify:translation

Summary by CodeRabbit

• Bug Fixes

  • Improved robustness of AI text extraction with safer parsing of responses, better handling of nested content structures.

• Refactor

  • Simplified response body construction logic for cleaner codebase maintenance.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: be9be48b-b7cc-4d61-992d-da7c4bb9ec42

📥 Commits

Reviewing files that changed from the base of the PR and between 0cb26b5 and 679bf04.

📒 Files selected for processing (1)

• apps/translation-worker/src/index.ts

---

📝 Walkthrough

Walkthrough

Refactored AI text extraction logic in the translation worker with new helper functions for safer, more flexible response parsing. Simplified response body construction. No exported function signatures changed.

Changes

Cohort / File(s)	Summary
Translation Worker Refactoring apps/translation-worker/src/index.ts	Introduced reusable helper functions (recordOf, content/choice extractors) to safely parse varying AI response formats. Replaced deeply nested parsing logic with clearer, more maintainable extraction flow. Simplified response body handling in temporaryEnglishRedirectResponse.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 With helper hands, the parsing flows so clean,
No more deep nesting in the response machine!
Choice by choice, content we find with care,
Refactored whisker-smooth through tangled air! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: simplify Kimi response parser' directly and accurately reflects the main change—simplifying the Kimi response parser by refactoring nested parsing logic into reusable helpers.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/fix-kimi-worker-sonar

---

_{Review rate limit: 0/5 reviews remaining, refill in 58 minutes and 14 seconds.}

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud