Skip to content

#226: R7: Separate i18n translation data from logic#243

Merged
JeremyDev87 merged 5 commits intomainfrom
claude/issue-226
Feb 16, 2026
Merged

#226: R7: Separate i18n translation data from logic#243
JeremyDev87 merged 5 commits intomainfrom
claude/issue-226

Conversation

@github-actions
Copy link
Contributor

Closes #226

Summary

This PR splits the monolithic src/i18n.ts file (300 lines) into a directory module to separate translation data (~200 lines) from logic functions.

Changes:

  • Created src/i18n/ directory module
  • Moved translation map data to src/i18n/translations.ts (237 lines)
  • Moved logic functions to src/i18n/index.ts (85 lines)
  • Relocated and updated test file to src/i18n/i18n.test.ts with corrected imports
  • Deleted old src/i18n.ts file

Backward compatibility:

  • All existing imports like from "./i18n" continue to work via directory module resolution
  • All exports maintain exact same types and signatures
  • No breaking changes for consumers

Verification:

  • npm run lint - passed
  • npm run typecheck - passed
  • npm run build - passed
  • npm test - all 398 tests passed

@github-actions github-actions bot added the refactor Code refactoring label Feb 16, 2026
@github-actions github-actions bot marked this pull request as ready for review February 16, 2026 06:48
@JeremyDev87 JeremyDev87 merged commit 4548dbf into main Feb 16, 2026
6 checks passed
@JeremyDev87 JeremyDev87 deleted the claude/issue-226 branch February 16, 2026 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

R7: Separate i18n translation data from logic

1 participant