[PR #16006] [CLOSED] Claude/fix issues 6 si wf #14493

Closed
opened 2026-02-07 07:30:42 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/16006
Author: @Piravlos
Created: 1/12/2026
Status: Closed

Base: masterHead: claude/fix-issues-6SiWf


📝 Commits (9)

  • b260cb0 Add missing StringComparison to StartsWith in FfProbeKeyframeExtractor
  • 9e31412 Merge pull request #1 from Piravlos/claude/fix-bug-patterns-HxzaF
  • fd1a87a Optimize NVIDIA NVENC transcoding with modern encoding features
  • 8521868 Merge pull request #2 from Piravlos/claude/optimize-nvidia-transcoding-XJq4C
  • 6bb4b24 Optimize performance with O(1) lookups and reduced allocations
  • 8fff4d1 Merge pull request #3 from Piravlos/claude/optimize-performance-6Xupp
  • 8d77c14 Optimize performance with reduced allocations and improved algorithms
  • 6897aa6 Merge pull request #4 from Piravlos/claude/optimize-performance-dWVNT
  • 2ed3d63 Fix multiple codebase issues

📊 Changes

16 files changed (+387 additions, -120 deletions)

View changed files

📝 Emby.Naming/Common/NamingOptions.cs (+3 -2)
📝 Emby.Server.Implementations/Dto/DtoService.cs (+99 -43)
📝 Emby.Server.Implementations/Library/LibraryManager.cs (+43 -12)
📝 Emby.Server.Implementations/Library/MediaSourceManager.cs (+1 -1)
📝 Emby.Server.Implementations/Library/SearchEngine.cs (+17 -16)
📝 Emby.Server.Implementations/Library/UserViewManager.cs (+27 -10)
📝 Emby.Server.Implementations/Playlists/PlaylistManager.cs (+1 -1)
📝 Jellyfin.Api/Controllers/PluginsController.cs (+23 -3)
📝 MediaBrowser.Controller/Entities/Folder.cs (+11 -2)
📝 MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs (+82 -0)
📝 MediaBrowser.Model/Configuration/EncodingOptions.cs (+51 -0)
📝 MediaBrowser.Model/Dlna/StreamBuilder.cs (+5 -10)
📝 src/Jellyfin.MediaEncoding.Keyframes/FfProbe/FfProbeKeyframeExtractor.cs (+1 -1)
📝 src/Jellyfin.Networking/Manager/NetworkManager.cs (+21 -17)
📝 tests/Jellyfin.Naming.Tests/Video/CleanDateTimeTests.cs (+1 -1)
📝 tests/Jellyfin.Naming.Tests/Video/CleanStringTests.cs (+1 -1)

📄 Description

Changes

Issues


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/16006 **Author:** [@Piravlos](https://github.com/Piravlos) **Created:** 1/12/2026 **Status:** ❌ Closed **Base:** `master` ← **Head:** `claude/fix-issues-6SiWf` --- ### 📝 Commits (9) - [`b260cb0`](https://github.com/jellyfin/jellyfin/commit/b260cb0d7fe789bab3944922e0b7f693a4cb1fc7) Add missing StringComparison to StartsWith in FfProbeKeyframeExtractor - [`9e31412`](https://github.com/jellyfin/jellyfin/commit/9e314124fc769d06c1167bfbebfa80719931890f) Merge pull request #1 from Piravlos/claude/fix-bug-patterns-HxzaF - [`fd1a87a`](https://github.com/jellyfin/jellyfin/commit/fd1a87a24299a6ec31e8a1b11a0f6a85e96bce20) Optimize NVIDIA NVENC transcoding with modern encoding features - [`8521868`](https://github.com/jellyfin/jellyfin/commit/852186873cfc8b3dcfd427bfbdeba939179ecd27) Merge pull request #2 from Piravlos/claude/optimize-nvidia-transcoding-XJq4C - [`6bb4b24`](https://github.com/jellyfin/jellyfin/commit/6bb4b242cbf9666a7e49d6646b446c913a5aebfe) Optimize performance with O(1) lookups and reduced allocations - [`8fff4d1`](https://github.com/jellyfin/jellyfin/commit/8fff4d1896bbba771185a9732d3a57aff3e964b0) Merge pull request #3 from Piravlos/claude/optimize-performance-6Xupp - [`8d77c14`](https://github.com/jellyfin/jellyfin/commit/8d77c14a12e8f4948d58aac6cfdd24db8f1d514d) Optimize performance with reduced allocations and improved algorithms - [`6897aa6`](https://github.com/jellyfin/jellyfin/commit/6897aa6622d228a3afd9b039c2a6603e7b59d1de) Merge pull request #4 from Piravlos/claude/optimize-performance-dWVNT - [`2ed3d63`](https://github.com/jellyfin/jellyfin/commit/2ed3d6305665b0edd564945a4e6f29e6bacbddd2) Fix multiple codebase issues ### 📊 Changes **16 files changed** (+387 additions, -120 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Naming/Common/NamingOptions.cs` (+3 -2) 📝 `Emby.Server.Implementations/Dto/DtoService.cs` (+99 -43) 📝 `Emby.Server.Implementations/Library/LibraryManager.cs` (+43 -12) 📝 `Emby.Server.Implementations/Library/MediaSourceManager.cs` (+1 -1) 📝 `Emby.Server.Implementations/Library/SearchEngine.cs` (+17 -16) 📝 `Emby.Server.Implementations/Library/UserViewManager.cs` (+27 -10) 📝 `Emby.Server.Implementations/Playlists/PlaylistManager.cs` (+1 -1) 📝 `Jellyfin.Api/Controllers/PluginsController.cs` (+23 -3) 📝 `MediaBrowser.Controller/Entities/Folder.cs` (+11 -2) 📝 `MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs` (+82 -0) 📝 `MediaBrowser.Model/Configuration/EncodingOptions.cs` (+51 -0) 📝 `MediaBrowser.Model/Dlna/StreamBuilder.cs` (+5 -10) 📝 `src/Jellyfin.MediaEncoding.Keyframes/FfProbe/FfProbeKeyframeExtractor.cs` (+1 -1) 📝 `src/Jellyfin.Networking/Manager/NetworkManager.cs` (+21 -17) 📝 `tests/Jellyfin.Naming.Tests/Video/CleanDateTimeTests.cs` (+1 -1) 📝 `tests/Jellyfin.Naming.Tests/Video/CleanStringTests.cs` (+1 -1) </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** <!-- Describe your changes here in 1-5 sentences. --> **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 07:30:42 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#14493