Skip to content

refactor: move I18N_COOKIE_NAME to @tooling/internationalization package#96

Merged
adelrodriguez merged 1 commit intomainfrom
02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package
Feb 19, 2026
Merged

refactor: move I18N_COOKIE_NAME to @tooling/internationalization package#96
adelrodriguez merged 1 commit intomainfrom
02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package

Conversation

@adelrodriguez
Copy link
Collaborator

@adelrodriguez adelrodriguez commented Feb 19, 2026

Move I18N_COOKIE_NAME to @tooling/internationalization package

This PR moves the I18N_COOKIE_NAME constant from the utils package to the internationalization tooling package, creating a proper export structure for the internationalization package.

Changes:

  • Removed I18N_COOKIE_NAME from packages/utils/src/constants/app.ts
  • Added proper TypeScript configuration to the internationalization package
  • Created a new export in tooling/internationalization/src/index.ts for the constant
  • Updated imports in the app to use the new location
  • Added the internationalization package as a dependency to the app

Greptile Summary

This PR refactors the I18N_COOKIE_NAME constant from the utils package to the internationalization tooling package, improving code organization by consolidating internationalization-related constants.

Key changes:

  • Created new export structure for @tooling/internationalization package with proper TypeScript configuration
  • Moved I18N_COOKIE_NAME constant to tooling/internationalization/src/index.ts
  • Updated import in apps/app/vite.config.ts to use new package location
  • Added @tooling/internationalization as devDependency in app package
  • Removed constant from packages/utils/src/constants/app.ts

Review findings:

  • All imports correctly updated - no stale references found
  • TypeScript configuration follows established pattern from other tooling packages
  • Package exports properly configured with type: "module" and exports field
  • Workspace configuration already includes tooling/* pattern
  • No breaking changes - only the one usage location was updated correctly

Confidence Score: 5/5

  • This PR is safe to merge with no identified risks
  • Clean refactoring with proper package structure, all imports updated correctly, follows established patterns from existing tooling packages, no logic changes or breaking changes
  • No files require special attention

Important Files Changed

Filename Overview
tooling/internationalization/src/index.ts New file exporting I18N_COOKIE_NAME constant, properly structured
tooling/internationalization/package.json Added proper exports, type module, and tsconfig dependency
tooling/internationalization/tsconfig.json New tsconfig extending internal-package config, matches other tooling packages
packages/utils/src/constants/app.ts Removed I18N_COOKIE_NAME export, leaving APP_NAME and APP_ID intact
apps/app/vite.config.ts Updated import to use @tooling/internationalization package

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["`packages/utils/src/constants/app.ts`"] -->|"Removed I18N_COOKIE_NAME"| B["`tooling/internationalization/src/index.ts`"]
    B -->|"Exports I18N_COOKIE_NAME"| C["`tooling/internationalization/package.json`"]
    C -->|"Added exports field"| D["`apps/app/vite.config.ts`"]
    D -->|"Updated import"| E["Uses I18N_COOKIE_NAME in paraglide plugin"]
    F["`apps/app/package.json`"] -->|"Added devDependency"| C
    
    style B fill:#90EE90
    style C fill:#90EE90
    style A fill:#FFB6C1
    style D fill:#87CEEB
    style F fill:#87CEEB
Loading

Last reviewed commit: 89c9dd4

@coderabbitai
Copy link

coderabbitai bot commented Feb 19, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package

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.

@adelrodriguez adelrodriguez marked this pull request as ready for review February 19, 2026 03:23
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_storage_types_from_utils_to_storage_package branch from fda02a8 to 351fcfe Compare February 19, 2026 03:26
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch from 6741b4e to 4dc6b5f Compare February 19, 2026 03:26
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_storage_types_from_utils_to_storage_package branch from 351fcfe to 4e09225 Compare February 19, 2026 03:29
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch from 4dc6b5f to 8224d45 Compare February 19, 2026 03:29
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_storage_types_from_utils_to_storage_package branch from 4e09225 to 1593a41 Compare February 19, 2026 03:33
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch 2 times, most recently from e488cf2 to 85e4f22 Compare February 19, 2026 03:38
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_storage_types_from_utils_to_storage_package branch 3 times, most recently from cbaf5a1 to c8a3d5d Compare February 19, 2026 03:53
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch 2 times, most recently from 0a35984 to bebe06f Compare February 19, 2026 03:53
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_storage_types_from_utils_to_storage_package branch from c8a3d5d to 025d97b Compare February 19, 2026 03:59
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch from bebe06f to ef357e0 Compare February 19, 2026 03:59
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_storage_types_from_utils_to_storage_package branch 2 times, most recently from 407dc33 to 100bcd4 Compare February 19, 2026 03:59
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch from ef357e0 to fcc2d1a Compare February 19, 2026 03:59
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_storage_types_from_utils_to_storage_package branch from 100bcd4 to 68bf454 Compare February 19, 2026 04:07
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch from fcc2d1a to 3535399 Compare February 19, 2026 04:07
@adelrodriguez adelrodriguez changed the base branch from 02-18-refactor_move_storage_types_from_utils_to_storage_package to graphite-base/96 February 19, 2026 04:11
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch from 3535399 to 09e50e2 Compare February 19, 2026 04:11
@graphite-app graphite-app bot changed the base branch from graphite-base/96 to main February 19, 2026 04:11
@adelrodriguez adelrodriguez force-pushed the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch from 09e50e2 to 89c9dd4 Compare February 19, 2026 04:11
@adelrodriguez adelrodriguez merged commit 5efceb8 into main Feb 19, 2026
8 checks passed
@adelrodriguez adelrodriguez deleted the 02-18-refactor_move_i18n_cookie_name_to_tooling_internationalization_package branch February 19, 2026 04:14
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.

1 participant

Comments