refactor(database): cleanup MikroOrmDatabase initializer #160

Merged

jonas.hanisch merged 3 commits from refactor/152-cleanup-mikro-orm-initializer into main

2026-01-24 12:41:28 +00:00

jack commented

2026-01-24 10:10:26 +00:00

Owner

Summary

Replace deprecated getMigrator() with orm.migrator property (MikroORM v6+ API)
Remove 6 unused methods: isConnected(), getEntityManager(), getOrm(), transaction(), runMigrations(), getMigrationStatus()
Reduced from 159 to 87 lines (-72 lines, -45%)

Refs #152

Test Plan

pnpm run typecheck passes
Verified via grep that none of the removed methods are called anywhere in the codebase
Only unitOfWork getter and close() method are actually used externally

## Summary - Replace deprecated `getMigrator()` with `orm.migrator` property (MikroORM v6+ API) - Remove 6 unused methods: `isConnected()`, `getEntityManager()`, `getOrm()`, `transaction()`, `runMigrations()`, `getMigrationStatus()` - Reduced from 159 to 87 lines (-72 lines, -45%) Refs #152 ## Test Plan - [x] `pnpm run typecheck` passes - [x] Verified via grep that none of the removed methods are called anywhere in the codebase - [x] Only `unitOfWork` getter and `close()` method are actually used externally

jack added 1 commit

2026-01-24 10:10:26 +00:00

refactor(database): cleanup MikroOrmDatabase initializer

CI / check (pull_request) Successful in 1s

Details

CI / build (pull_request) Successful in 42s

Details

CI / validate-plugin (pull_request) Successful in 36s

Details

a0a3321894

- Replace deprecated getMigrator() with orm.migrator property
- Remove unused isConnected() method
- Remove unused getEntityManager() method
- Remove unused getOrm() method
- Remove unused transaction() method
- Remove unused runMigrations() method
- Remove unused getMigrationStatus() method

Reduced from 159 to 87 lines (-72 lines, -45%)

Refs #152

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>