fix: config is already a parsed object from the jobs API response #61

Merged
josh merged 2 commits from feat/jobs-system into dev 2026-03-28 20:47:46 -04:00
Owner

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

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>
josh added 1 commit 2026-03-28 20:47:15 -04:00
fix: config is already a parsed object from the jobs API response
All checks were successful
CI / test (pull_request) Successful in 14s
CI / build-dev (pull_request) Has been skipped
027ed52768
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>
josh added 1 commit 2026-03-28 20:47:20 -04:00
Merge branch 'dev' into feat/jobs-system
All checks were successful
CI / test (pull_request) Successful in 13s
CI / build-dev (pull_request) Has been skipped
047fd0653e
josh scheduled this pull request to auto merge when all checks succeed 2026-03-28 20:47:24 -04:00
josh merged commit cb83d11261 into dev 2026-03-28 20:47:46 -04:00
josh deleted branch feat/jobs-system 2026-03-28 20:47:46 -04:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: josh/Catalyst#61