I haven't opened the code, so this review is based only on the facts you gave. I can't cite files or line numbers yet, and step 1 of the plan is a code audit to find them.
Assumptions to confirm
- Languages: German (
de), Japanese (ja) and Arabic (ar), plus English. Arabic means RTL has to be built in from the start. - Arabic region: The choice affects the calendar, numerals and currency. For example,
ar-SAdefaults to the Islamic calendar, whilear-EGandar-AEuse Gregorian. I recommendar-AEorar-EGwith Latin digits unless the product team says otherwise. - Currency and time zone: These should stay independent of language.
- Translations: I'm assuming they come from people or a vendor, so the catalog needs translator context.
- Charts: I don't know if the dashboard has charts, and I'd check. Axes, legends and tooltips would need RTL handling too.
Review findings
Hard-coded text
- Inline strings in components. Every label, placeholder, tooltip,
aria-label, error message and page title needs a catalog key. Fix: adopt a library (react-intl or i18next with ICU), use stable keys, and add a translator description for each key. count === 1plurals. This is wrong for Arabic, which has six plural categories (zero, one, two, few, many, other). It is also wrong for Japanese, which has only "other". Fix: use ICU{count, plural, ...}messages. Arabic needs all six forms written out.- Likely string concatenation. Inline strings usually come with
count + " items in " + namepatterns. Fix: whole messages with named placeholders. I'd grep for+ "and template literals in JSX. - Ambiguous one-word labels. Words like "Open" or "Filter" can be a verb or a noun. Fix: separate keys per meaning.
Formatting
- Hand-written
MM/DD/YYYY. This is wrong for all three new languages. Fix:Intl.DateTimeFormat(locale, options), with dates stored as ISO or timestamps. Check whether the data layer holds already-formatted strings. - Numbers, currency, percentages. These are probably formatted by hand too. Fix: use
Intl.NumberFormat, with currency as an explicit code and not derived from the language. - Other formatters. Relative time ("3 days ago"), lists, and sorting should use
Intl.RelativeTimeFormat,Intl.ListFormatandIntl.Collator. - Language and region. Keep these as separate settings.
Layout and truncation
- Fixed pixel widths on buttons and tabs. German will overflow them and clip. Fix: remove the widths and use
min-width, padding and flex or grid withgap. Where truncation is unavoidable, use an ellipsis plus a tooltip or accessible name. - Fixed heights. Check for these as well, since text will wrap to more lines.
- Verification. Use pseudo-localization (about 40% longer, accented, in brackets) to find every clipped element before real translations exist.
Scripts and fonts
- No
langon the document. Set<html lang>per locale. This is needed for Japanese glyph selection, since the Han characters render differently underja. - Font stack. Add Japanese and Arabic fallbacks; a Latin-only stack will give inconsistent system fallbacks. Check that the bold weights exist for those scripts.
- Typography.
- Give Japanese more line-height.
- Remove any italic emphasis, any forced uppercase, and any
letter-spacingon Arabic. Letter spacing breaks Arabic joining. - Don't truncate strings by character index; use
Intl.Segmenterif any code does.
RTL
margin-leftandtext-align: left. These will break the Arabic layout. Fix: switch tomargin-inline-start,padding-inline,text-align: start,inset-inline-*andborder-inline-*. Do this by codebase-wide search and replace, then review each hit. Also check forfloat,left:/right:, andtranslateX.dir. Setdir="rtl"on the root for Arabic.- Icons. Audit them one by one. Mirror chevrons, back and next arrows, and progress bars. Don't mirror logos, play/pause icons, clocks or numerals.
- Mixed-direction content. Wrap user-generated or unknown-direction text in
<bdi>. Give email, URL and code fieldsdir="ltr". - Charts. If there are any, decide deliberately whether to mirror the time axis. Numerals and axis values stay LTR.
Routing and switching
- No language switcher. Add one labeled in each language's own name (English, Deutsch, 日本語, العربية), with no flags.
- Locale order. Use saved choice, then URL, then browser
Accept-Language, then default. Don't force a redirect. - Persistence. Remember the choice.
- URLs and SEO. If the dashboard is public, use a path prefix per language with
hreflang,x-defaultand canonical URLs. If it's behind a login, a saved preference may be enough, but I'd confirm. - Fallback. Missing translations fall back to English and log the gap. Raw keys should never show.
Plan
- Audit (about 1–2 days). Grep for hard-coded strings,
count ===, manual date code,width:in px, andleft/right/margin-left. This gives a real inventory and effort estimate. Confirm the Arabic region and numeral choice, the translation source, and whether the app has charts or is public. - Foundation.
- Pick the i18n library and set up the message catalog, with the
encatalog as the source. - Add a locale provider, a
lang/dirsetter and the switcher. - Add the font stacks and a pseudo-locale.
- Add a lint rule against literal JSX text (e.g.
eslint-plugin-i18next).
- Pick the i18n library and set up the message catalog, with the
- Layout and CSS (can run in parallel with step 4). Remove fixed widths and convert to logical properties. Make buttons and tabs flexible, and audit icons for mirroring. Verify with the pseudo-locale and with Arabic under
dir="rtl". - Extract strings. Move them component by component into the catalog, with translator notes and ICU plurals. Replace concatenation with whole messages.
- Formatting layer. Make one helper module wrapping
Intl.*for dates, numbers, currency, lists and relative time. Replace every manual format and store dates as ISO or timestamps. - Translation. Send the catalog to translators with context, including the six Arabic plural forms. Flag legal text, examples and region-specific content for review.
- QA and verification. Run every screen in
en,de,ja,arat phone and desktop widths. Test plurals with counts of 0, 1, 2, 5 and a large number (Arabic needs 0, 1, 2, 3–10, 11–99 and 100+ to cover all forms). Compare dates and numbers across two regions. Checklanganddirin the DOM. Search again for leftover hard-coded strings.
Order and risk: the CSS conversion (step 3) and string extraction (step 4) are the largest pieces of work. RTL is the riskiest because it touches every component, so do it early and not at the end.
Not checked: I haven't looked at the codebase, run anything, or tested any real RTL or translated text. All of the above comes from the facts you listed.
If you'd like, I can start with step 1 once you point me at the project directory. The working directory appears to be empty.
Real run in an isolated folder with only this skill installed. Only the Skill and Read tools were enabled, so no command was run and no file was written. The scenario is fictional and the model had no access to the code. The model proposed a plan and listed what it could not check.
What it does
Separates internationalization (the UI can be localized) from localization (the translations) and covers the first. Six rules: no user-visible text in code (whole messages with named placeholders, never joined fragments, and standard plural handling instead of count === 1); format dates, numbers, currencies, lists and relative time with the locale-aware Intl APIs; build layouts that survive text of different lengths using logical CSS properties and pseudo-localization; handle scripts, fonts and line breaking (lang, zh-Hans and zh-Hant, font fallbacks, CJK behavior); support right-to-left layouts (dir, mirroring what is directional and not what is not, bidirectional isolation); and design language switching, locale order, URLs, hreflang and fallbacks.
Also covers
Keeping language, region, currency and time zone independent; storing dates as timestamps; flexible name and address forms; translator context for message keys; and a checklist to verify before calling it done.
Good for
Adding a second language, adding German, Japanese or Arabic to an English-only product, and reviewing a UI that "breaks when translated".
Low risk: pure instructions with no scripts, no network access and no file writes. It does not translate anything and does not judge translation quality; machine-made or unreviewed translations need review by fluent speakers, and legal, medical or regional content needs local review. It cannot see your code or run it, so it lists what it did not check, such as real right-to-left rendering. Some `Intl` features need a support check for your browsers. Original skill by AIBars (MIT). Tried once on an invented React dashboard.