fix(gateway): add missing @Inject() decorators causing silent startup hang #109

Merged
jason.woltje merged 1 commits from fix/gateway-silent-exit into main 2026-03-15 01:52:01 +00:00
Owner

Summary

Three constructor parameters were missing explicit @Inject() decorators. Since tsx/esbuild doesn't support emitDecoratorMetadata, NestJS couldn't resolve them, causing the DI container to hang forever during onModuleInit with no error output.

Fixed:

  • SummarizationService: @Inject(EmbeddingService)
  • CronService: @Inject(SummarizationService)
  • SkillsController: @Inject(SkillsService)

Also includes the dotenv fix (PR #108) for loading .env from monorepo root.

Test plan

  • pnpm --filter @mosaic/gateway exec tsx src/main.ts → Gateway
## Summary Three constructor parameters were missing explicit @Inject() decorators. Since tsx/esbuild doesn't support emitDecoratorMetadata, NestJS couldn't resolve them, causing the DI container to hang forever during onModuleInit with no error output. Fixed: - SummarizationService: @Inject(EmbeddingService) - CronService: @Inject(SummarizationService) - SkillsController: @Inject(SkillsService) Also includes the dotenv fix (PR #108) for loading .env from monorepo root. ## Test plan - [ ] pnpm --filter @mosaic/gateway exec tsx src/main.ts → Gateway
jason.woltje added 1 commit 2026-03-15 01:51:55 +00:00
fix(gateway): add missing @Inject() decorators causing silent startup hang
All checks were successful
ci/woodpecker/push/ci Pipeline was successful
4c37162b55
tsx/esbuild doesn't support emitDecoratorMetadata, so NestJS can't
infer constructor parameter types at runtime. Three services were
missing explicit @Inject() decorators:

- SummarizationService: EmbeddingService parameter
- CronService: SummarizationService parameter
- SkillsController: SkillsService parameter

Without these, NestJS DI hangs forever during onModuleInit resolution
with no error output — the process silently exits (or hangs with a
keepAlive).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jason.woltje merged commit 6d2b81f6e4 into main 2026-03-15 01:52:01 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#109