dtourolleandClaude Opus 5.5
582c94f5f9
The single net8.0 build compiled against Jellyfin 10.9.11 still loads on
10.11 and 12, so channels and browsing keep working, but 10.11 changed
APIs in ways that only fail at runtime. A binary audit of the DLL against
the real server assemblies found 9 broken references on both 10.11 and
12 (none on 10.9/10.10):
- TaskTriggerInfo.Type became an enum. GetDefaultTriggers throws, so
Jellyfin silently gives every SRF task a 24h fallback trigger. The
recording scheduler therefore ran about once a day and scheduled
recordings never started.
- ILibraryManager.GetItemList changed its return type: the expiration
check crashed on every run.
- Jellyfin.Data.Entities.User moved (IUserManager.Users, GetUserData,
SaveUserData, PlaybackProgressEventArgs.Users): resume cleanup returned
500 and the playback-stop guard threw on every stop.
Jellyfin 12 also rejects the legacy X-Emby-Token header with an empty
401, which the recordings UI surfaced as "Unexpected end of JSON input".
The pages now send Authorization: MediaBrowser Token="...", verified on
10.9, 10.11 and 12.
The plugin now multi-targets net8.0/net9.0/net10.0 against Jellyfin
10.9.11/10.11.0/12.0.0 (the oldest package of each line, so each DLL
loads on every patch release of it), with JELLYFIN_10_11_OR_GREATER and
JELLYFIN_12_OR_GREATER for the differences. Trigger construction moves
into Utilities/TaskTriggers so the version switch lives in one place.
.gitea/scripts/build-plugins.sh builds one zip per generation. jprm has
no targetAbi flag, so it rewrites build.yaml per flavour and restores
it. Jellyfin installs the highest version whose targetAbi it satisfies,
so the flavour code goes into the last version segment:
release v1.2.0 -> 1.2.0.9 / 1.2.0.11 / 1.2.0.12
nightly -> 1.0.<date>.<run>09 / 11 / 12
The PR, nightly and release workflows use the script and publish one
manifest entry per zip. Verified end to end against a three-entry
manifest: 10.11 installs the .11 build, 12.1 installs the .12 build.
The existing .NET 10 builder image builds all three unchanged.
Supporting changes: the test project multi-targets too, because jprm
publishes the whole solution for one framework at a time, and it drops
its Microsoft.Extensions.Logging 8.0.1 pin, which is a NU1605 downgrade
under the newer Jellyfin packages. CA1873 (new in the .NET 10 analyzers)
is silenced beside CA1848. The release manifest step checks out
origin/master like the nightly does since 7e1973a.
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
111 lines
6.6 KiB
XML
111 lines
6.6 KiB
XML
<?xml version="1.0" encoding="utf-8"?>
|
|
<RuleSet Name="Rules for Jellyfin.Server" Description="Code analysis rules for Jellyfin.Server.csproj" ToolsVersion="14.0">
|
|
<Rules AnalyzerId="StyleCop.Analyzers" RuleNamespace="StyleCop.Analyzers">
|
|
<!-- disable warning SA1009: Closing parenthesis should be followed by a space. -->
|
|
<Rule Id="SA1009" Action="None" />
|
|
<!-- disable warning SA1011: Closing square bracket should be followed by a space. -->
|
|
<Rule Id="SA1011" Action="None" />
|
|
<!-- disable warning SA1101: Prefix local calls with 'this.' -->
|
|
<Rule Id="SA1101" Action="None" />
|
|
<!-- disable warning SA1108: Block statements should not contain embedded comments -->
|
|
<Rule Id="SA1108" Action="None" />
|
|
<!-- disable warning SA1118: Parameter must not span multiple lines. -->
|
|
<Rule Id="SA1118" Action="None" />
|
|
<!-- disable warning SA1128:: Put constructor initializers on their own line -->
|
|
<Rule Id="SA1128" Action="None" />
|
|
<!-- disable warning SA1130: Use lambda syntax -->
|
|
<Rule Id="SA1130" Action="None" />
|
|
<!-- disable warning SA1200: 'using' directive must appear within a namespace declaration -->
|
|
<Rule Id="SA1200" Action="None" />
|
|
<!-- disable warning SA1202: 'public' members must come before 'private' members -->
|
|
<Rule Id="SA1202" Action="None" />
|
|
<!-- disable warning SA1204: Static members must appear before non-static members -->
|
|
<Rule Id="SA1204" Action="None" />
|
|
<!-- disable warning SA1309: Fields must not begin with an underscore -->
|
|
<Rule Id="SA1309" Action="None" />
|
|
<!-- disable warning SA1413: Use trailing comma in multi-line initializers -->
|
|
<Rule Id="SA1413" Action="None" />
|
|
<!-- disable warning SA1512: Single-line comments must not be followed by blank line -->
|
|
<Rule Id="SA1512" Action="None" />
|
|
<!-- disable warning SA1515: Single-line comment should be preceded by blank line -->
|
|
<Rule Id="SA1515" Action="None" />
|
|
<!-- disable warning SA1600: Elements should be documented -->
|
|
<Rule Id="SA1600" Action="None" />
|
|
<!-- disable warning SA1602: Enumeration items should be documented -->
|
|
<Rule Id="SA1602" Action="None" />
|
|
<!-- disable warning SA1633: The file header is missing or not located at the top of the file -->
|
|
<Rule Id="SA1633" Action="None" />
|
|
</Rules>
|
|
|
|
<Rules AnalyzerId="Microsoft.CodeAnalysis.NetAnalyzers" RuleNamespace="Microsoft.Design">
|
|
<!-- error on CA1305: Specify IFormatProvider -->
|
|
<Rule Id="CA1305" Action="Error" />
|
|
<!-- error on CA1725: Parameter names should match base declaration -->
|
|
<Rule Id="CA1725" Action="Error" />
|
|
<!-- error on CA1725: Call async methods when in an async method -->
|
|
<Rule Id="CA1727" Action="Error" />
|
|
<!-- error on CA1843: Do not use 'WaitAll' with a single task -->
|
|
<Rule Id="CA1843" Action="Error" />
|
|
<!-- error on CA2016: Forward the CancellationToken parameter to methods that take one
|
|
or pass in 'CancellationToken.None' explicitly to indicate intentionally not propagating the token -->
|
|
<Rule Id="CA2016" Action="Error" />
|
|
<!-- error on CA2254: Template should be a static expression -->
|
|
<Rule Id="CA2254" Action="Error" />
|
|
|
|
<!-- disable warning CA1014: Mark assemblies with CLSCompliantAttribute -->
|
|
<Rule Id="CA1014" Action="Info" />
|
|
<!-- disable warning CA1024: Use properties where appropriate -->
|
|
<Rule Id="CA1024" Action="Info" />
|
|
<!-- disable warning CA1031: Do not catch general exception types -->
|
|
<Rule Id="CA1031" Action="Info" />
|
|
<!-- disable warning CA1032: Implement standard exception constructors -->
|
|
<Rule Id="CA1032" Action="Info" />
|
|
<!-- disable warning CA1040: Avoid empty interfaces -->
|
|
<Rule Id="CA1040" Action="Info" />
|
|
<!-- disable warning CA1062: Validate arguments of public methods -->
|
|
<Rule Id="CA1062" Action="Info" />
|
|
<!-- TODO: enable when false positives are fixed -->
|
|
<!-- disable warning CA1508: Avoid dead conditional code -->
|
|
<Rule Id="CA1508" Action="Info" />
|
|
<!-- disable warning CA1716: Identifiers should not match keywords -->
|
|
<Rule Id="CA1716" Action="Info" />
|
|
<!-- disable warning CA1720: Identifiers should not contain type names -->
|
|
<Rule Id="CA1720" Action="Info" />
|
|
<!-- disable warning CA1724: Type names should not match namespaces -->
|
|
<Rule Id="CA1724" Action="Info" />
|
|
<!-- disable warning CA1805: Do not initialize unnecessarily -->
|
|
<Rule Id="CA1805" Action="Info" />
|
|
<!-- disable warning CA1812: internal class that is apparently never instantiated.
|
|
If so, remove the code from the assembly.
|
|
If this class is intended to contain only static members, make it static -->
|
|
<Rule Id="CA1812" Action="Info" />
|
|
<!-- disable warning CA1822: Member does not access instance data and can be marked as static -->
|
|
<Rule Id="CA1822" Action="Info" />
|
|
<!-- disable warning CA2000: Dispose objects before losing scope -->
|
|
<Rule Id="CA2000" Action="Info" />
|
|
<!-- disable warning CA2253: Named placeholders should not be numeric values -->
|
|
<Rule Id="CA2253" Action="Info" />
|
|
<!-- disable warning CA5394: Do not use insecure randomness -->
|
|
<Rule Id="CA5394" Action="Info" />
|
|
|
|
<!-- disable warning CA1054: Change the type of parameter url from string to System.Uri -->
|
|
<Rule Id="CA1054" Action="None" />
|
|
<!-- disable warning CA1055: URI return values should not be strings -->
|
|
<Rule Id="CA1055" Action="None" />
|
|
<!-- disable warning CA1056: URI properties should not be strings -->
|
|
<Rule Id="CA1056" Action="None" />
|
|
<!-- disable warning CA1303: Do not pass literals as localized parameters -->
|
|
<Rule Id="CA1303" Action="None" />
|
|
<!-- disable warning CA1308: Normalize strings to uppercase -->
|
|
<Rule Id="CA1308" Action="None" />
|
|
<!-- disable warning CA1848: Use the LoggerMessage delegates -->
|
|
<Rule Id="CA1848" Action="None" />
|
|
<!-- disable warning CA1873: Avoid potentially expensive logging (new in the .NET 10 analyzers; same family as CA1848) -->
|
|
<Rule Id="CA1873" Action="None" />
|
|
<!-- disable warning CA2101: Specify marshaling for P/Invoke string arguments -->
|
|
<Rule Id="CA2101" Action="None" />
|
|
<!-- disable warning CA2234: Pass System.Uri objects instead of strings -->
|
|
<Rule Id="CA2234" Action="None" />
|
|
</Rules>
|
|
</RuleSet>
|