Quaver.Shared/Screens/Gameplay/UI/EpilepsyWarning.cs:37

• When map is null, ShouldDisplayWarning returns true, which will display the epilepsy warning unconditionally. Since the warning is intended to fire only on maps that actually use SVs (per the PR's "Only show on SV maps" requirement), the safe default when the map cannot be inspected should be false. As written, any code path that ends up constructing the warning before Screen.Map is populated will show the warning on a non-SV map.

        if (map == null)
            return true;

Quaver.Shared/Screens/Gameplay/UI/EpilepsyWarning.cs:40

• map.Tags is a free-form string, and Contains("sv", IgnoreCase) is a substring match on the entire tag blob. This will match any tag (or word in artist/source/etc. embedded in tags) that happens to contain the letters "sv" — e.g. tags like "psv", "tsv", "service", "lsvg" — producing false positives. Consider splitting the tags on the project's standard delimiter (space/comma) and comparing each token with case-insensitive equality to "sv".

        var hasSvTag = map.Tags?.Contains("sv", StringComparison.InvariantCultureIgnoreCase) ??
                       false;

Quaver.Shared/Screens/Gameplay/UI/EpilepsyWarning.cs:70

• ShouldDisplayWarning() is evaluated only once in the constructor and stored in Visible. However, Update continues to run the lerp logic on every frame for the entire gameplay session even after the warning has fully faded out (and on every frame, regardless of whether the warning was ever displayed). Consider early-returning when Visible is false, and/or destroying/disabling the sprite once Screen.Timing.Time >= -500 and Alpha is effectively zero, to avoid pointless per-frame work and an always-rendered fully-transparent fullscreen sprite.

    public override void Update(GameTime gameTime)
    {
        base.Update(gameTime);

        var dt = gameTime.ElapsedGameTime.TotalMilliseconds;

        // Fade in on map start
        if (Screen.Timing.Time < -500)
        {
            var alpha = MathHelper.Lerp(Alpha, 1, (float)Math.Min(dt / AnimationScale, 1));

            Alpha = alpha;
        }
        else
        {
            var alpha = MathHelper.Lerp(Alpha, 0, (float)Math.Min(dt / AnimationScale, 1));

            Alpha = alpha;
        }
    }