Skip to content

Fix invalid unicode escape in Korean string resource#539

Merged
utkarshdalal merged 1 commit intoutkarshdalal:masterfrom
xXJSONDeruloXx:fix/korean-string-escape
Feb 17, 2026
Merged

Fix invalid unicode escape in Korean string resource#539
utkarshdalal merged 1 commit intoutkarshdalal:masterfrom
xXJSONDeruloXx:fix/korean-string-escape

Conversation

@xXJSONDeruloXx
Copy link
Contributor

@xXJSONDeruloXx xXJSONDeruloXx commented Feb 16, 2026


Summary by cubic

Escape the apostrophes around 'none' in values-ko/strings.xml (main_pending_operation_none) to fix the aapt2 "invalid unicode escape" build error. This unblocks Android builds and keeps the intended quoted text in the Korean string.

Written for commit 4067d03. Summary will update on new commits.

Summary by CodeRabbit

  • Bug Fixes
    • Fixed formatting in Korean-language operation status text so single quotes display correctly in user-facing messages, preventing incorrect rendering and improving readability.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 16, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

A single Korean string resource (main_pending_operation_none) was edited to escape single quotes around the word "none" with backslashes in app/src/main/res/values-ko/strings.xml.

Changes

Cohort / File(s) Summary
Localization
app/src/main/res/values-ko/strings.xml
Adjusted main_pending_operation_none value to escape single quotes ('none'\'none\').

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰✨ A backslash hop, a tiny fix done right,
Quotes hugged tighter, tucked out of sight.
Korean text sings with punctuation aligned,
A rabbit’s small cheer for strings refined. 🥕

🚥 Pre-merge checks | ✅ 4
✅ 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 accurately describes the main change: escaping apostrophes in a Korean string resource to fix a build error, which directly matches the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into master

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@app/src/main/res/values-ko/strings.xml`:
- Line 786: The string resource main_pending_operation_none contains a
double-escaped single quote (\\') causing an "Invalid unicode escape sequence"
build error; replace the double-escaped sequence with a single backslash escape
(\' ) so the literal apostrophe is properly escaped in the XML string. Locate
the <string name="main_pending_operation_none"> entry and change the escape from
\\ ' to \' ensuring it matches the other entries in this file that use \' for
single quotes.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 1 file

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="app/src/main/res/values-ko/strings.xml">

<violation number="1" location="app/src/main/res/values-ko/strings.xml:786">
P2: Android string resources require apostrophes to be escaped as \' (or by wrapping the whole string in double quotes). Using \\' leaves an unescaped apostrophe and can trigger the “Apostrophe not preceded by \” error or render backslashes.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@xXJSONDeruloXx xXJSONDeruloXx force-pushed the fix/korean-string-escape branch from 1cb81f7 to 4067d03 Compare February 16, 2026 18:51
@utkarshdalal utkarshdalal merged commit 45d5b52 into utkarshdalal:master Feb 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants