From 846d217dcaa9400f2f14bf809c0532101d415b67 Mon Sep 17 00:00:00 2001
From: BoulderBadgeDad <batmanesignal@gmail.com>
Date: Thu, 25 Jun 2026 22:38:26 -0700
Subject: [PATCH] video automations: order the System list as a logical
 pipeline
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

the API returns system automations newest-created-first, so a re-seeded row
(e.g. the just-migrated movie/episode processors) jumps to the top and jumbles
the grouping. the video page now re-sorts its System list by an explicit order:
scans (fill) → processors (drain) → library scan/sync → maintenance. scoped to
the video page only (music ordering untouched); unknown/future actions fall to
the end keeping API order. string-contract test pins the scan<proc<maint order.
---
 tests/test_video_automations_builder.py | 14 ++++++++++++++
 webui/static/video/video-automations.js | 25 ++++++++++++++++++++++++-
 2 files changed, 38 insertions(+), 1 deletion(-)

diff --git a/tests/test_video_automations_builder.py b/tests/test_video_automations_builder.py
index 1e2e874f..506a20f5 100644
--- a/tests/test_video_automations_builder.py
+++ b/tests/test_video_automations_builder.py
@@ -99,3 +99,17 @@ def test_generic_config_renderer_is_video_gated():
 
 def test_video_page_exposes_reload_hook():
     assert 'window._reloadVideoAutomations = load' in _VAUTO
+
+
+# --- the System list is shown in a logical pipeline order -----------------
+
+def test_video_system_automations_are_sorted_by_pipeline_order():
+    # The API returns newest-created-first (jumbled); the page re-sorts by an
+    # explicit order so it reads scans → processors → library → maintenance.
+    assert 'sortSystem(all.filter(isVideoAutomation))' in _VAUTO
+    assert '_SYS_ORDER' in _VAUTO
+    # the order must put the watchlist SCANS before the wishlist PROCESSORS
+    scan = _VAUTO.index("'video_scan_watchlist_people'")
+    proc = _VAUTO.index("'video_process_movie_wishlist'")
+    maint = _VAUTO.index("'video_backup_database'")
+    assert scan < proc < maint
diff --git a/webui/static/video/video-automations.js b/webui/static/video/video-automations.js
index 299e7d0f..b977a5c2 100644
--- a/webui/static/video/video-automations.js
+++ b/webui/static/video/video-automations.js
@@ -25,6 +25,29 @@
         return a && a.owned_by === 'video';
     }
 
+    // The API returns system automations newest-created-first, which jumbles them (a
+    // re-seeded row jumps to the top). Impose a logical top-to-bottom pipeline order
+    // instead. Scoped to THIS page — the music page's ordering is untouched. Unknown /
+    // future action types fall to the end, keeping their API order (stable sort).
+    var _SYS_ORDER = [
+        // Stage 1 — scans that FILL the wishlist
+        'video_scan_watchlist_people', 'video_scan_watchlist_channels',
+        'video_scan_watchlist_playlists', 'video_add_airing_episodes',
+        // Stage 2 — processors that DRAIN it (download)
+        'video_process_movie_wishlist', 'video_process_episode_wishlist', 'video_process_youtube_wishlist',
+        // Library scan / sync
+        'video_scan_server', 'video_update_database', 'video_deep_scan_tv', 'video_deep_scan_movies',
+        // Maintenance
+        'video_clean_search_history', 'video_clean_completed_downloads', 'video_full_cleanup', 'video_backup_database',
+    ];
+    function _sysOrderIndex(a) {
+        var i = _SYS_ORDER.indexOf(a && a.action_type);
+        return i === -1 ? _SYS_ORDER.length : i;
+    }
+    function sortSystem(list) {
+        return list.slice().sort(function (a, b) { return _sysOrderIndex(a) - _sysOrderIndex(b); });
+    }
+
     function renderSystem(sys) {
         var host = document.querySelector('[data-vauto-list]'); if (!host) return;
         var emptyEl = document.querySelector('[data-vauto-empty]');
@@ -71,7 +94,7 @@
     function load() {
         return getJSON('/api/automations').then(function (d) {
             var all = Array.isArray(d) ? d : (d && d.automations) || [];
-            var sys = all.filter(isVideoAutomation);
+            var sys = sortSystem(all.filter(isVideoAutomation));
             renderStats(sys);
             renderSystem(sys);
             renderHubOnce();