Server side works

ConfusedPolarBear.Plugin.IntroSkipper/Configuration/PluginConfiguration.cs

@@ -1,23 +1,11 @@
+using System;
+using System.Collections.Generic;
+using System.Collections.ObjectModel;
+
 using MediaBrowser.Model.Plugins;
 
 namespace ConfusedPolarBear.Plugin.IntroSkipper.Configuration;
 
-/// <summary>
-/// The configuration options.
-/// </summary>
-public enum SomeOptions
-{
-    /// <summary>
-    /// Option one.
-    /// </summary>
-    OneOption,
-
-    /// <summary>
-    /// Second option.
-    /// </summary>
-    AnotherOption
-}
-
 /// <summary>
 /// Plugin configuration.
 /// </summary>
@@ -28,30 +16,38 @@ public class PluginConfiguration : BasePluginConfiguration
     /// </summary>
     public PluginConfiguration()
     {
-        // set default options here
-        Options = SomeOptions.AnotherOption;
-        TrueFalseSetting = true;
-        AnInteger = 2;
-        AString = "string";
+        AnalysisResults = new Collection<Intro>();
     }
 
     /// <summary>
-    /// Gets or sets a value indicating whether some true or false setting is enabled..
+    /// Save timestamps to disk.
     /// </summary>
-    public bool TrueFalseSetting { get; set; }
+    public void SaveTimestamps()
+    {
+        AnalysisResults.Clear();
 
-    /// <summary>
-    /// Gets or sets an integer setting.
-    /// </summary>
-    public int AnInteger { get; set; }
-
-    /// <summary>
-    /// Gets or sets a string setting.
-    /// </summary>
-    public string AString { get; set; }
-
-    /// <summary>
-    /// Gets or sets an enum option.
-    /// </summary>
-    public SomeOptions Options { get; set; }
+        foreach (var intro in Plugin.Instance!.Intros)
+        {
+            AnalysisResults.Add(intro.Value);
+        }
+
+        Plugin.Instance!.SaveConfiguration();
+    }
+
+    /// <summary>
+    /// Restore previous analysis results from disk.
+    /// </summary>
+    public void RestoreTimestamps()
+    {
+        // Since dictionaries can't be easily serialized, analysis results are stored on disk as a list.
+        foreach (var intro in AnalysisResults)
+        {
+            Plugin.Instance!.Intros[intro.EpisodeId] = intro;
+        }
+    }
+
+    /// <summary>
+    /// Previous analysis results.
+    /// </summary>
+    public Collection<Intro> AnalysisResults { get; private set; }
 }

ConfusedPolarBear.Plugin.IntroSkipper/Controllers/SkipIntroController.cs

@@ -1,17 +1,7 @@
 using System;
-using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
-using System.Globalization;
-using System.Linq;
 using System.Net.Mime;
-using System.Text.Json;
-using Jellyfin.Data.Entities;
-using Jellyfin.Extensions.Json;
-using MediaBrowser.Controller.Library;
-using MediaBrowser.Model.Dto;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
-using Microsoft.Extensions.Logging;
 
 namespace ConfusedPolarBear.Plugin.IntroSkipper.Controllers;
 
@@ -33,18 +23,18 @@ public class SkipIntroController : ControllerBase
     /// <summary>
     /// Returns the timestamps of the introduction in a television episode.
     /// </summary>
-    /// <param name="episodeId">ID of the episode. Required.</param>
+    /// <param name="id">ID of the episode. Required.</param>
     /// <response code="200">Episode contains an intro.</response>
     /// <response code="404">Failed to find an intro in the provided episode.</response>
     [HttpGet("Episode/{id}/IntroTimestamps")]
-    public ActionResult<Intro> GetIntroTimestamps([FromRoute] Guid episodeId)
+    public ActionResult<Intro> GetIntroTimestamps([FromRoute] Guid id)
     {
-        if (!Plugin.Instance!.Intros.ContainsKey(episodeId))
+        if (!Plugin.Instance!.Intros.ContainsKey(id))
         {
             return NotFound();
         }
 
-        var intro = Plugin.Instance!.Intros[episodeId];
+        var intro = Plugin.Instance!.Intros[id];
 
         // Check that the episode was analyzed successfully.
         if (!intro.Valid)

ConfusedPolarBear.Plugin.IntroSkipper/Data/Intro.cs

@@ -6,8 +6,12 @@ namespace ConfusedPolarBear.Plugin.IntroSkipper;
 /// Result of fingerprinting and analyzing two episodes in a season.
 /// All times are measured in seconds relative to the beginning of the media file.
 /// </summary>
-[Serializable]
 public class Intro {
+    /// <summary>
+    /// Episode ID.
+    /// </summary>
+    public Guid EpisodeId { get; set; }
+
     /// <summary>
     /// If this introduction is valid or not. Invalid results should not be returned through the API.
     /// </summary>

ConfusedPolarBear.Plugin.IntroSkipper/Plugin.cs

@@ -1,5 +1,4 @@
 using System;
-using System.Collections.ObjectModel;
 using System.Collections.Generic;
 using System.Globalization;
 using ConfusedPolarBear.Plugin.IntroSkipper.Configuration;
@@ -11,10 +10,25 @@ using MediaBrowser.Model.Serialization;
 namespace ConfusedPolarBear.Plugin.IntroSkipper;
 
 /// <summary>
-/// The main plugin.
+/// Intro skipper plugin. Uses audio analysis to find common sequences of audio shared between episodes.
 /// </summary>
 public class Plugin : BasePlugin<PluginConfiguration>, IHasWebPages
 {
+    /// <summary>
+    /// Initializes a new instance of the <see cref="Plugin"/> class.
+    /// </summary>
+    /// <param name="applicationPaths">Instance of the <see cref="IApplicationPaths"/> interface.</param>
+    /// <param name="xmlSerializer">Instance of the <see cref="IXmlSerializer"/> interface.</param>
+    public Plugin(IApplicationPaths applicationPaths, IXmlSerializer xmlSerializer)
+        : base(applicationPaths, xmlSerializer)
+    {
+        Intros = new Dictionary<Guid, Intro>();
+        AnalysisQueue = new Dictionary<Guid, List<QueuedEpisode>>();
+        Instance = this;
+
+        Configuration.RestoreTimestamps();
+    }
+
     /// <summary>
     /// Results of fingerprinting all episodes.
     /// </summary>
@@ -30,29 +44,13 @@ public class Plugin : BasePlugin<PluginConfiguration>, IHasWebPages
     /// </summary>
     public int TotalQueued { get; set; }
 
-    /// <summary>
-    /// Initializes a new instance of the <see cref="Plugin"/> class.
-    /// </summary>
-    /// <param name="applicationPaths">Instance of the <see cref="IApplicationPaths"/> interface.</param>
-    /// <param name="xmlSerializer">Instance of the <see cref="IXmlSerializer"/> interface.</param>
-    public Plugin(IApplicationPaths applicationPaths, IXmlSerializer xmlSerializer)
-        : base(applicationPaths, xmlSerializer)
-    {
-        Intros = new Dictionary<Guid, Intro>();
-        AnalysisQueue = new Dictionary<Guid, List<QueuedEpisode>>();
-
-        Instance = this;
-    }
-
     /// <inheritdoc />
     public override string Name => "Intro Skipper";
 
     /// <inheritdoc />
     public override Guid Id => Guid.Parse("c83d86bb-a1e0-4c35-a113-e2101cf4ee6b");
 
-    /// <summary>
-    /// Gets the current plugin instance.
-    /// </summary>
+    /// <inheritdoc />
     public static Plugin? Instance { get; private set; }
 
     /// <inheritdoc />

ConfusedPolarBear.Plugin.IntroSkipper/ScheduledTasks/FingerprinterTask.cs

@@ -119,7 +119,7 @@ public class FingerprinterTask : IScheduledTask {
                 }
             }
 
-            // TODO: after every season completes, serialize fingerprints to disk somewhere
+            Plugin.Instance!.Configuration.SaveTimestamps();
         }
 
         return Task.CompletedTask;
@@ -248,10 +248,11 @@ public class FingerprinterTask : IScheduledTask {
         // and that it disappears 10 seconds after the intro begins.
         Plugin.Instance!.Intros[episode] = new Intro()
         {
+            EpisodeId = episode,
             Valid = (introStart > 0) && (introEnd > 0),
             IntroStart = introStart,
             IntroEnd = introEnd,
-            ShowSkipPromptAt = Math.Min(0, introStart - 5),
+            ShowSkipPromptAt = Math.Max(0, introStart - 5),
             HideSkipPromptAt = introStart + 10
         };
     }