[BUG] deploy-crons.py silently drops model/provider changes when prompt unchanged #375

Closed
opened 2026-04-13 21:21:40 +00:00 by Rockachopa · 3 comments
Owner

Bug

In deploy-crons.py lines 91-92, update check compares ONLY prompt and schedule. If model/provider changes but prompt stays the same, the job is marked unchanged and NOT updated.

Impact

The provider migration changes to cron-jobs.yaml were applied to YAML but deploy-crons.py would NOT deploy them to jobs.json because the prompts didn't change. Model/provider updates are silently dropped.

Fix

Include model and provider in the comparison:

if (cur.get('prompt') != normalized.get('prompt') or
cur.get('schedule') != normalized.get('schedule') or
cur.get('model') != normalized.get('model') or
cur.get('provider') != normalized.get('provider')):
## Bug In deploy-crons.py lines 91-92, update check compares ONLY prompt and schedule. If model/provider changes but prompt stays the same, the job is marked unchanged and NOT updated. ## Impact The provider migration changes to cron-jobs.yaml were applied to YAML but deploy-crons.py would NOT deploy them to jobs.json because the prompts didn't change. Model/provider updates are silently dropped. ## Fix Include model and provider in the comparison: ``` if (cur.get('prompt') != normalized.get('prompt') or cur.get('schedule') != normalized.get('schedule') or cur.get('model') != normalized.get('model') or cur.get('provider') != normalized.get('provider')): ```
Author
Owner

🔥 BURN DISPATCH v2 — BURNONE pane 3

Timmy multi-threaded burn. Gitea-first workflow.

🔥 **BURN DISPATCH v2** — BURNONE pane 3 Timmy multi-threaded burn. Gitea-first workflow.
claude self-assigned this 2026-04-14 00:42:58 +00:00
Author
Owner

Fixed in PR #447.

Added --deploy mode with _jobs_changed() that compares prompt, schedule, model, and provider — so model/provider changes are never silently dropped.

See: #447

Fixed in PR #447. Added `--deploy` mode with `_jobs_changed()` that compares prompt, schedule, model, and provider — so model/provider changes are never silently dropped. See: https://forge.alexanderwhitestone.com/Timmy_Foundation/hermes-agent/pulls/447
Author
Owner

Fixed in PR #448.

_jobs_changed() now compares prompt, schedule, model, and provider.

See: #448

Fixed in PR #448. `_jobs_changed()` now compares prompt, schedule, model, and provider. See: https://forge.alexanderwhitestone.com/Timmy_Foundation/hermes-agent/pulls/448
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Timmy_Foundation/hermes-agent#375