Skip to content

fix(i18n): drop the duplicate ZH_CN "Idle" key - #729

Merged
Ark0N merged 1 commit into
Ark0N:masterfrom
timkjr:fix/i18n-duplicate-idle
Oct 28, 2026
Merged

Ark0N merged 1 commit into
Ark0N:masterfrom
timkjr:fix/i18n-duplicate-idle

Conversation

@timkjr

@timkjr timkjr commented Oct 27, 2026

Copy link
Copy Markdown

The Approvals Inbox strings added a second Idle: '空闲' to the ZH_CN table in src/web/public/i18n.js, next to the original among the status words. Both entries have the same value, so nothing changes on screen, but every build warns:

[WARNING] Duplicate key "Idle" in object literal [duplicate-object-key]

This removes the later copy. After the change, npm run build has no warnings, and the i18n tests and format:check pass.

🤖 Generated with Claude Code

The Approvals Inbox strings added a second `Idle: '空闲'` to the ZH_CN
table, beside the original in the status words. Same value, so nothing
changed on screen, but every build warned:

  [WARNING] Duplicate key "Idle" in object literal [duplicate-object-key]
@Ark0N

Ark0N commented Oct 27, 2026

Copy link
Copy Markdown
Owner

Thanks @timkjr, and welcome! This removes the second Idle: '空闲' entry that the Approvals Inbox strings added to the ZH_CN table in src/web/public/i18n.js, which silences the duplicate-key warning on every build.

I checked that the translation table is identical afterwards (same keys, same order, same values), and typecheck, lint, the frontend syntax and format checks and the i18n tests all pass. Nothing to change here; this is good to merge once CI finishes.

@Ark0N
Ark0N merged commit 20cd022 into Ark0N:master Oct 28, 2026
2 checks passed
@Ark0N

Ark0N commented Oct 28, 2026

Copy link
Copy Markdown
Owner

Merged in 1.47.0 (https://github.com/Ark0N/Codeman/releases/tag/codeman%401.47.0). Thank you, the build is quiet again.

@timkjr
timkjr deleted the fix/i18n-duplicate-idle branch October 28, 2026 15:25
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.

2 participants