From 027ed5276822ad099ebce31efe5473ce2b1d802d Mon Sep 17 00:00:00 2001
From: josh <josh@thewrightserver.net>
Date: Sat, 28 Mar 2026 20:46:49 -0400
Subject: [PATCH] fix: config is already a parsed object from the jobs API
 response

maskJob parses job.config before returning it, so calling JSON.parse
on it again threw an exception. The catch returned false for every
job, so relevant was always empty and _waitForOnCreateJobs returned
immediately without polling.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 js/ui.js | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/js/ui.js b/js/ui.js
index b691475..64930fa 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -318,9 +318,7 @@ async function _snapshotJobBaseline() {
 
 async function _waitForOnCreateJobs(baseline) {
   const jobs = await fetch('/api/jobs').then(r => r.json());
-  const relevant = jobs.filter(j => {
-    try { return JSON.parse(j.config || '{}').run_on_create; } catch { return false; }
-  });
+  const relevant = jobs.filter(j => (j.config ?? {}).run_on_create);
   if (!relevant.length) return;
 
   const deadline = Date.now() + 30_000;