Ungenutzter Export mikroOrmMigrations in migrations/index.ts #147

New issue

Closed

opened 2026-01-24 09:55:58 +00:00 by jack · 0 comments

jack commented

2026-01-24 09:55:58 +00:00

Owner

Problem

In packages/database/src/mikro-orm/migrations/index.ts wird mikroOrmMigrations exportiert, aber nirgendwo verwendet.

Kontext

Laut CLAUDE.md sollte mikroOrmMigrations in der MikroORM-Konfiguration verwendet werden:

// packages/database/src/mikro-orm.config.ts
import { Migration20260123000005_... } from './mikro-orm/migrations/...';

export const migrationsList = [
  // ... bestehende Migrations
  Migration20260123000005_...,
];

Es scheint, dass die Migrations direkt in mikro-orm.config.ts importiert werden, statt über den mikroOrmMigrations Array.

Analyse

Mögliche Szenarien:

Doppelte Pflege - Migrations werden sowohl in index.ts als auch in mikro-orm.config.ts eingetragen
Veralteter Export - mikroOrmMigrations sollte verwendet werden, wird aber nicht
Alternative Architektur - Die Konfiguration nutzt direkte Imports

Optionen

Option A: Export entfernen

Wenn mikro-orm.config.ts die Migrations direkt importiert, kann mikroOrmMigrations entfernt werden.

Option B: Export verwenden

mikro-orm.config.ts so ändern, dass es mikroOrmMigrations verwendet - dann gibt es nur eine Stelle zum Pflegen.

⚠️ Wichtig: Nicht einfach löschen! Prüfen wie die MikroORM-Konfiguration die Migrations lädt und ob der Export verwendet werden sollte.

Betroffene Datei

packages/database/src/mikro-orm/migrations/index.ts

## Problem In `packages/database/src/mikro-orm/migrations/index.ts` wird `mikroOrmMigrations` exportiert, aber nirgendwo verwendet. ### Kontext Laut `CLAUDE.md` sollte `mikroOrmMigrations` in der MikroORM-Konfiguration verwendet werden: ```typescript // packages/database/src/mikro-orm.config.ts import { Migration20260123000005_... } from './mikro-orm/migrations/...'; export const migrationsList = [ // ... bestehende Migrations Migration20260123000005_..., ]; ``` Es scheint, dass die Migrations direkt in `mikro-orm.config.ts` importiert werden, statt über den `mikroOrmMigrations` Array. ## Analyse Mögliche Szenarien: 1. **Doppelte Pflege** - Migrations werden sowohl in `index.ts` als auch in `mikro-orm.config.ts` eingetragen 2. **Veralteter Export** - `mikroOrmMigrations` sollte verwendet werden, wird aber nicht 3. **Alternative Architektur** - Die Konfiguration nutzt direkte Imports ## Optionen ### Option A: Export entfernen Wenn `mikro-orm.config.ts` die Migrations direkt importiert, kann `mikroOrmMigrations` entfernt werden. ### Option B: Export verwenden `mikro-orm.config.ts` so ändern, dass es `mikroOrmMigrations` verwendet - dann gibt es nur eine Stelle zum Pflegen. ⚠️ **Wichtig:** Nicht einfach löschen! Prüfen wie die MikroORM-Konfiguration die Migrations lädt und ob der Export verwendet werden sollte. ## Betroffene Datei `packages/database/src/mikro-orm/migrations/index.ts`