Skip to content

feat(server): add Jellyfin adapter, controller and unified API#13

Closed
enoch85 wants to merge 1 commit into
jellyfin/3-plex-adapterfrom
jellyfin/4-jellyfin-complete
Closed

feat(server): add Jellyfin adapter, controller and unified API#13
enoch85 wants to merge 1 commit into
jellyfin/3-plex-adapterfrom
jellyfin/4-jellyfin-complete

Conversation

@enoch85

@enoch85 enoch85 commented Jan 2, 2026

Copy link
Copy Markdown
Owner

Jellyfin Implementation:

  • Add JellyfinService implementing IMediaServerService
  • Add JellyfinMapper for converting Jellyfin API responses
  • Add Jellyfin constants, types, and module
  • Add comprehensive Jellyfin mapper tests (477+ lines)
  • Add Jellyfin adapter service tests (256+ lines)

Unified Controller:

  • Add MediaServerController with unified /api/media-server endpoints
  • Add MediaServerSetupGuard for auth protection
  • Add controller tests (380+ lines)

Factory Updates:

  • Update MediaServerFactory to support Jellyfin
  • Factory now routes to correct adapter based on settings

This completes the media server abstraction layer.


PR 4 of 13 - Jellyfin Integration Series
Depends on: #12

Jellyfin Implementation:
- Add JellyfinService implementing IMediaServerService
- Add JellyfinMapper for converting Jellyfin API responses
- Add Jellyfin constants, types, and module
- Add comprehensive Jellyfin mapper tests (477+ lines)
- Add Jellyfin adapter service tests (256+ lines)

Unified Controller:
- Add MediaServerController with unified /api/media-server endpoints
- Add MediaServerSetupGuard for auth protection
- Add controller tests (380+ lines)

Factory Updates:
- Update MediaServerFactory to support Jellyfin
- Factory now routes to correct adapter based on settings

This completes the media server abstraction layer.

enoch85 commented Jan 3, 2026

Copy link
Copy Markdown
Owner Author

Closing in favor of consolidated PR #24. Branch kept for reference.

@enoch85 enoch85 closed this Jan 3, 2026
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