till/managarten
1
0
Fork 0
mirror of https://github.com/Memo-2023/mana-monorepo.git synced 2026-05-14 22:01:09 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

fix(bot-services): remove SessionModule import from I18nModule to prevent DI conflicts

Browse source 
The I18nModule was importing SessionModule.forRoot() internally, which
conflicted with bots that import SessionModule with different options
(e.g., storageMode: 'redis').

Since I18nService uses @Optional() for SessionService, the module should
not force its own SessionModule import. Users who want per-user language
preferences should import SessionModule in their app before I18nModule.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Till-JS 2026-02-17 12:46:33 +01:00
parent 9faa1fcc68
commit 9b01c094c1
1 changed files with 12 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

18
packages/bot-services/src/i18n/i18n.module.ts
View file

@ -2,23 +2,29 @@ import { Module, DynamicModule, Global } from '@nestjs/common';
import { ConfigModule } from '@nestjs/config';
import { I18nService, I18N_OPTIONS } from './i18n.service';
import { I18nOptions } from './types';
import { SessionModule } from '../session/session.module';
/**
* I18n Module for Matrix Bots
*
* Provides multi-language support with per-user language preferences.
*
* NOTE: SessionService is optional. If you want per-user language preferences,
* import SessionModule in your app before I18nModule. Otherwise, the default
* language will be used for all users.
*
* @example
* ```typescript
* // Basic usage (uses SessionModule and ConfigModule)
* // Basic usage (uses default language for all users)
* @Module({
* imports: [I18nModule.forRoot()],
* })
*
* // With custom default language
* // With per-user preferences (requires SessionModule)
* @Module({
* imports: [I18nModule.forRoot({ defaultLanguage: 'en' })],
* imports: [
* SessionModule.forRoot({ storageMode: 'redis' }),
* I18nModule.forRoot({ defaultLanguage: 'en' }),
* ],
* })
* ```
*/
@ -31,7 +37,7 @@ export class I18nModule {
static forRoot(options?: I18nOptions): DynamicModule {
return {
module: I18nModule,
imports: [ConfigModule, SessionModule.forRoot()],
imports: [ConfigModule],
providers: [
{
provide: I18N_OPTIONS,
@ -53,7 +59,7 @@ export class I18nModule {
}): DynamicModule {
return {
module: I18nModule,
imports: [...(options.imports || []), ConfigModule, SessionModule.forRoot()],
imports: [...(options.imports || []), ConfigModule],
providers: [
{
provide: I18N_OPTIONS,