+
${esc(t('cron_schedule_label') || 'Cron expression')}
${esc(t('cron_schedule_hint') || "Cron expression or shorthand like 'every 1h'.")}
${esc(t('cron_schedule_once_warning') || "Duration forms like '30m' run once and are removed after running. Use 'every 30m' to keep a recurring job.")}
diff --git a/static/style.css b/static/style.css
index b16d135cb..4fbea1c0d 100644
--- a/static/style.css
+++ b/static/style.css
@@ -6318,6 +6318,8 @@ main.main > .main-view:not([id="mainChat"]):not([id="mainSettings"]) .main-view-
.detail-form-row > label{font-size:12px;font-weight:500;color:var(--muted);letter-spacing:.01em;}
.detail-form-row input[type="text"],
.detail-form-row input[type="password"],
+.detail-form-row input[type="number"],
+.detail-form-row input[type="time"],
.detail-form-row input:not([type]),
.detail-form-row select,
.detail-form-row textarea{
@@ -6329,10 +6331,23 @@ main.main > .main-view:not([id="mainChat"]):not([id="mainSettings"]) .main-view-
}
.detail-form-row input[type="text"]:focus,
.detail-form-row input[type="password"]:focus,
+.detail-form-row input[type="number"]:focus,
.detail-form-row input:not([type]):focus,
.detail-form-row select:focus,
.detail-form-row textarea:focus{border-color:var(--accent,var(--accent-bg-strong,var(--border)));}
.detail-form-row input:disabled{opacity:.6;cursor:not-allowed;}
+.cron-schedule-preset-shell{display:flex;flex-wrap:wrap;gap:10px;align-items:center;}
+.cron-schedule-preset-shell > select.cron-schedule-preset-select{width:auto;min-width:150px;flex:0 0 auto;}
+.cron-schedule-preset-params{display:flex;flex-wrap:wrap;gap:8px 10px;align-items:center;min-width:0;}
+.cron-schedule-preset-field{display:inline-flex;flex-direction:row;align-items:center;gap:6px;}
+.cron-schedule-preset-conj{font-size:13px;color:var(--muted);white-space:nowrap;}
+.cron-schedule-preset-field select,.cron-schedule-preset-field input[type="time"]{width:auto;min-width:0;}
+.cron-schedule-preset-field select{min-width:120px;}
+#cronFormScheduleMonthDay{min-width:72px;}
+.cron-schedule-preset-field input[type="time"]{min-width:120px;}
+.cron-schedule-preset-field input[type="number"]{width:64px;}
+.cron-schedule-preset-time-hint{flex-basis:100%;margin-top:2px;}
+.cron-schedule-preview{font-family:'SF Mono',ui-monospace,SFMono-Regular,Menlo,monospace;color:var(--text);opacity:.85;}
.detail-form-row textarea{resize:vertical;font-family:'SF Mono',ui-monospace,SFMono-Regular,Menlo,monospace;font-size:12px;}
.detail-form-row .detail-form-hint{font-size:11px;color:var(--muted);line-height:1.5;}
.detail-form-warning{font-size:11px;line-height:1.5;border:1px solid rgba(245,158,11,.35);background:rgba(245,158,11,.1);color:rgba(245,158,11,.98);border-radius:8px;padding:8px 10px;}
diff --git a/tests/test_issue2031_cron_once_visibility.py b/tests/test_issue2031_cron_once_visibility.py
index 3ea70781e..dc8adcff9 100644
--- a/tests/test_issue2031_cron_once_visibility.py
+++ b/tests/test_issue2031_cron_once_visibility.py
@@ -81,19 +81,28 @@ def test_cron_schedule_preset_matching():
script = _cron_schedule_source() + r"""
const cases = {
hourly: _cronSchedulePresetIdForValue("every 1h"),
- daily: _cronSchedulePresetIdForValue("0 9 * * *"),
- weekdays: _cronSchedulePresetIdForValue("0 9 * * 1-5"),
- weekly: _cronSchedulePresetIdForValue("0 9 * * 1"),
- monthly: _cronSchedulePresetIdForValue("0 9 1 * *"),
+ hourlyCron: _cronSchedulePresetIdForValue("15 * * * *"),
+ daily: _cronSchedulePresetIdForValue("30 7 * * *"),
+ weekdays: _cronSchedulePresetIdForValue("5 22 * * 1-5"),
+ weekly: _cronSchedulePresetIdForValue("45 8 * * 3"),
+ monthly: _cronSchedulePresetIdForValue("10 6 15 * *"),
empty: _cronSchedulePresetIdForValue(""),
- trimMatch: _cronSchedulePresetIdForValue(" 0 9 * * * "),
+ trimMatch: _cronSchedulePresetIdForValue(" 30 7 * * * "),
custom: _cronSchedulePresetIdForValue("0 9 * * * 0"),
+ shorthand: _cronSchedulePresetIdForValue("@daily"),
+ invalidDaily: _cronSchedulePresetIdForValue("99 99 * * *"),
+ invalidHourly: _cronSchedulePresetIdForValue("60 * * * *"),
+ ambiguousSunday: _cronSchedulePresetIdForValue("0 9 * * 7"),
+ invalidWeekly: _cronSchedulePresetIdForValue("0 9 * * 8"),
+ invalidMonthDayZero: _cronSchedulePresetIdForValue("0 9 0 * *"),
+ invalidMonthly: _cronSchedulePresetIdForValue("0 9 32 * *"),
};
console.log(JSON.stringify(cases));
"""
cases = json.loads(_run_node(script))
assert cases["hourly"] == "hourly"
+ assert cases["hourlyCron"] == "hourly"
assert cases["daily"] == "daily"
assert cases["weekdays"] == "weekdays"
assert cases["weekly"] == "weekly"
@@ -101,16 +110,306 @@ console.log(JSON.stringify(cases));
assert cases["empty"] == "custom"
assert cases["trimMatch"] == "daily"
assert cases["custom"] == "custom"
+ assert cases["shorthand"] == "custom"
+ assert cases["invalidDaily"] == "custom"
+ assert cases["invalidHourly"] == "custom"
+ assert cases["ambiguousSunday"] == "weekly"
+ assert cases["invalidWeekly"] == "custom"
+ assert cases["invalidMonthDayZero"] == "custom"
+ assert cases["invalidMonthly"] == "custom"
def test_cron_schedule_preset_controls_sync_raw_and_preset_paths():
script = _cron_schedule_source() + r"""
const elements = {};
function $(id) { return elements[id]; }
-function makeElement() {
+function makeElement(initialValue = '') {
return {
- value: '',
- style: {},
+ value: initialValue,
+ style: { display: '' },
+ listeners: {},
+ addEventListener(type, handler) {
+ (this.listeners[type] || (this.listeners[type] = [])).push(handler);
+ },
+ dispatchEvent(eventType) {
+ const handlers = this.listeners[eventType] || [];
+ for (const handler of handlers) handler({ type: eventType, target: this });
+ },
+ };
+}
+function t(key) {
+ const dict = {
+ cron_schedule_preset_label: 'Schedule',
+ cron_schedule_preset_hourly: 'Hourly',
+ cron_schedule_preset_daily: 'Daily',
+ cron_schedule_preset_weekdays: 'Weekdays (Mon-Fri)',
+ cron_schedule_preset_weekly: 'Weekly',
+ cron_schedule_preset_monthly: 'Monthly',
+ cron_schedule_preset_custom: 'Custom',
+ };
+ return dict[key];
+}
+function esc(value) { return value == null ? '' : String(value); }
+
+[
+ 'cronFormSchedule',
+ 'cronFormSchedulePreset',
+ 'cronFormSchedulePresetParams',
+ 'cronFormScheduleCustomRow',
+ 'cronFormScheduleTimeField',
+ 'cronFormScheduleMinuteField',
+ 'cronFormScheduleWeekdayField',
+ 'cronFormScheduleMonthDayField',
+ 'cronFormScheduleTime',
+ 'cronFormScheduleMinute',
+ 'cronFormScheduleWeekday',
+ 'cronFormScheduleMonthDay',
+ 'cronFormSchedulePreview',
+].forEach((id) => {
+ elements[id] = makeElement();
+});
+elements.cronFormScheduleOnceWarning = { style: { display: 'none' } };
+elements.cronFormSchedulePresetParams.style.display = 'none';
+
+_initCronSchedulePresetControls();
+
+// Split "HH:MM" from the time picker so assertions can read hour/minute separately.
+function timeHour() { const v = String(elements.cronFormScheduleTime.value || ''); return v.includes(':') ? v.split(':')[0].replace(/^0/, '') || '0' : ''; }
+function timeMin() { const v = String(elements.cronFormScheduleTime.value || ''); return v.includes(':') ? String(parseInt(v.split(':')[1], 10)) : ''; }
+
+function snapshot() {
+ return {
+ preset: elements.cronFormSchedulePreset.value,
+ schedule: elements.cronFormSchedule.value,
+ paramsDisplay: elements.cronFormSchedulePresetParams.style.display,
+ customRowDisplay: elements.cronFormScheduleCustomRow.style.display,
+ timeFieldDisplay: elements.cronFormScheduleTimeField.style.display,
+ minuteFieldDisplay: elements.cronFormScheduleMinuteField.style.display,
+ weekdayFieldDisplay: elements.cronFormScheduleWeekdayField.style.display,
+ monthDayFieldDisplay: elements.cronFormScheduleMonthDayField.style.display,
+ time: elements.cronFormScheduleTime.value,
+ hour: timeHour(),
+ minute: timeMin(),
+ minuteBox: elements.cronFormScheduleMinute.value,
+ weekday: elements.cronFormScheduleWeekday.value,
+ monthDay: elements.cronFormScheduleMonthDay.value,
+ warning: elements.cronFormScheduleOnceWarning.style.display,
+ kind: _cronScheduleKindForInput(elements.cronFormSchedule.value),
+ };
+}
+
+// Hourly: raw sync + editing the standalone minute box.
+elements.cronFormSchedule.value = '15 * * * *';
+elements.cronFormSchedule.dispatchEvent('change');
+const hourlySync = snapshot();
+elements.cronFormSchedulePreset.value = 'hourly';
+elements.cronFormSchedulePreset.dispatchEvent('change');
+elements.cronFormScheduleMinute.value = '15';
+elements.cronFormScheduleMinute.dispatchEvent('change');
+const hourlyWrite = snapshot();
+
+// Daily: raw sync populates the time picker; editing the time regenerates.
+elements.cronFormSchedule.value = '30 7 * * *';
+elements.cronFormSchedule.dispatchEvent('change');
+const dailySync = snapshot();
+elements.cronFormScheduleTime.value = '07:30';
+elements.cronFormScheduleTime.dispatchEvent('change');
+const dailyWrite = snapshot();
+
+elements.cronFormSchedule.value = '5 22 * * 1-5';
+elements.cronFormSchedule.dispatchEvent('change');
+const weekdaysSync = snapshot();
+
+elements.cronFormSchedule.value = '45 8 * * 3';
+elements.cronFormSchedule.dispatchEvent('change');
+const weeklySync = snapshot();
+elements.cronFormSchedule.value = '0 9 * * 7';
+elements.cronFormSchedule.dispatchEvent('change');
+const weeklyRawSundaySync = snapshot();
+elements.cronFormScheduleWeekday.value = '3';
+elements.cronFormScheduleWeekday.dispatchEvent('change');
+elements.cronFormScheduleTime.value = '08:45';
+elements.cronFormScheduleTime.dispatchEvent('change');
+const weeklyWrite = snapshot();
+
+elements.cronFormSchedule.value = '10 6 15 * *';
+elements.cronFormSchedule.dispatchEvent('change');
+const monthlySync = snapshot();
+elements.cronFormScheduleMonthDay.value = '15';
+elements.cronFormScheduleMonthDay.dispatchEvent('change');
+elements.cronFormScheduleTime.value = '06:10';
+elements.cronFormScheduleTime.dispatchEvent('change');
+const monthlyWrite = snapshot();
+
+elements.cronFormSchedule.value = '30 7 * * *';
+elements.cronFormSchedule.dispatchEvent('change');
+const dailyAfterMonthlySync = snapshot();
+
+// Monthly clamp: out-of-range monthDay clamps to 31 on change.
+elements.cronFormSchedulePreset.value = 'monthly';
+elements.cronFormSchedulePreset.dispatchEvent('change');
+elements.cronFormScheduleTime.value = '23:00';
+elements.cronFormScheduleMonthDay.value = '99';
+elements.cronFormScheduleMonthDay.dispatchEvent('change');
+const normalizedMonthlyWrite = snapshot();
+
+elements.cronFormSchedule.value = '@daily';
+elements.cronFormSchedule.dispatchEvent('change');
+const shorthandCustom = snapshot();
+
+elements.cronFormSchedule.value = 'advanced: cron expression';
+elements.cronFormSchedule.dispatchEvent('change');
+const unsupportedCustom = snapshot();
+
+elements.cronFormSchedulePreset.value = 'custom';
+elements.cronFormSchedulePreset.dispatchEvent('change');
+const customSelectionPreserved = snapshot();
+
+console.log(JSON.stringify({
+ hourlySync,
+ hourlyWrite,
+ dailySync,
+ dailyWrite,
+ weekdaysSync,
+ weeklySync,
+ weeklyRawSundaySync,
+ weeklyWrite,
+ monthlySync,
+ monthlyWrite,
+ dailyAfterMonthlySync,
+ normalizedMonthlyWrite,
+ shorthandCustom,
+ unsupportedCustom,
+ customSelectionPreserved,
+}));
+"""
+ result = json.loads(_run_node(script))
+
+ assert result["hourlySync"]["preset"] == "hourly"
+ assert result["hourlySync"]["schedule"] == "15 * * * *"
+ assert result["hourlySync"]["paramsDisplay"] == ""
+ assert result["hourlySync"]["customRowDisplay"] == "none"
+ assert result["hourlySync"]["timeFieldDisplay"] == "none"
+ assert result["hourlySync"]["minuteFieldDisplay"] == ""
+ assert result["hourlySync"]["kind"] == "cron"
+ assert result["hourlyWrite"]["schedule"] == "15 * * * *"
+ assert result["hourlyWrite"]["kind"] == "cron"
+
+ assert result["dailySync"]["preset"] == "daily"
+ assert result["dailySync"]["schedule"] == "30 7 * * *"
+ assert result["dailySync"]["timeFieldDisplay"] == ""
+ assert result["dailySync"]["minuteFieldDisplay"] == "none"
+ assert result["dailySync"]["weekdayFieldDisplay"] == "none"
+ assert result["dailySync"]["monthDayFieldDisplay"] == "none"
+ assert result["dailySync"]["customRowDisplay"] == "none"
+ assert result["dailySync"]["time"] == "07:30"
+ assert result["dailyWrite"]["schedule"] == "30 7 * * *"
+ assert result["dailyWrite"]["kind"] == "cron"
+
+ assert result["weekdaysSync"]["preset"] == "weekdays"
+ assert result["weekdaysSync"]["schedule"] == "5 22 * * 1-5"
+ assert result["weekdaysSync"]["timeFieldDisplay"] == ""
+ assert result["weekdaysSync"]["minuteFieldDisplay"] == "none"
+ assert result["weekdaysSync"]["weekdayFieldDisplay"] == "none"
+ assert result["weekdaysSync"]["monthDayFieldDisplay"] == "none"
+ assert result["weekdaysSync"]["time"] == "22:05"
+
+ assert result["weeklySync"]["preset"] == "weekly"
+ assert result["weeklySync"]["schedule"] == "45 8 * * 3"
+ assert result["weeklySync"]["weekdayFieldDisplay"] == ""
+ assert result["weeklySync"]["timeFieldDisplay"] == ""
+ assert result["weeklySync"]["monthDayFieldDisplay"] == "none"
+ assert result["weeklySync"]["weekday"] == "3"
+ assert result["weeklySync"]["time"] == "08:45"
+ assert result["weeklyRawSundaySync"]["preset"] == "weekly"
+ assert result["weeklyRawSundaySync"]["schedule"] == "0 9 * * 7"
+ assert result["weeklyRawSundaySync"]["weekday"] == "0"
+ assert result["weeklyWrite"]["schedule"] == "45 8 * * 3"
+ assert result["weeklyWrite"]["kind"] == "cron"
+
+ assert result["monthlySync"]["preset"] == "monthly"
+ assert result["monthlySync"]["schedule"] == "10 6 15 * *"
+ assert result["monthlySync"]["monthDayFieldDisplay"] == ""
+ assert result["monthlySync"]["timeFieldDisplay"] == ""
+ assert result["monthlySync"]["weekdayFieldDisplay"] == "none"
+ assert result["monthlySync"]["monthDay"] == "15"
+ assert result["monthlySync"]["time"] == "06:10"
+ assert result["monthlyWrite"]["schedule"] == "10 6 15 * *"
+ assert result["monthlyWrite"]["kind"] == "cron"
+
+ assert result["dailyAfterMonthlySync"]["preset"] == "daily"
+ assert result["dailyAfterMonthlySync"]["schedule"] == "30 7 * * *"
+ assert result["dailyAfterMonthlySync"]["time"] == "07:30"
+ assert result["dailyAfterMonthlySync"]["monthDayFieldDisplay"] == "none"
+
+ assert result["normalizedMonthlyWrite"]["schedule"] == "0 23 31 * *"
+ assert result["normalizedMonthlyWrite"]["time"] == "23:00"
+ assert result["normalizedMonthlyWrite"]["monthDay"] == "31"
+ assert result["normalizedMonthlyWrite"]["kind"] == "cron"
+
+ assert result["shorthandCustom"]["preset"] == "custom"
+ assert result["shorthandCustom"]["schedule"] == "@daily"
+ assert result["shorthandCustom"]["paramsDisplay"] == "none"
+ assert result["shorthandCustom"]["customRowDisplay"] == ""
+
+ assert result["unsupportedCustom"]["preset"] == "custom"
+ assert result["unsupportedCustom"]["schedule"] == "advanced: cron expression"
+ assert result["unsupportedCustom"]["paramsDisplay"] == "none"
+ assert result["unsupportedCustom"]["customRowDisplay"] == ""
+
+ assert result["customSelectionPreserved"]["preset"] == "custom"
+ assert result["customSelectionPreserved"]["schedule"] == "advanced: cron expression"
+ assert result["customSelectionPreserved"]["paramsDisplay"] == "none"
+ assert result["customSelectionPreserved"]["customRowDisplay"] == ""
+
+
+def test_cron_schedule_raw_warning_listener_survives_missing_preset_params():
+ script = _cron_schedule_source() + r"""
+const elements = {};
+function $(id) { return elements[id]; }
+function makeElement(initialValue = '') {
+ return {
+ value: initialValue,
+ style: { display: '' },
+ listeners: {},
+ addEventListener(type, handler) {
+ (this.listeners[type] || (this.listeners[type] = [])).push(handler);
+ },
+ dispatchEvent(eventType) {
+ const handlers = this.listeners[eventType] || [];
+ for (const handler of handlers) handler({ type: eventType, target: this });
+ },
+ };
+}
+elements.cronFormSchedule = makeElement();
+elements.cronFormSchedulePreset = makeElement('custom');
+elements.cronFormScheduleOnceWarning = { style: { display: 'none' } };
+
+_initCronSchedulePresetControls();
+elements.cronFormSchedule.value = '30m';
+elements.cronFormSchedule.dispatchEvent('input');
+
+console.log(JSON.stringify({
+ warning: elements.cronFormScheduleOnceWarning.style.display,
+ inputListeners: (elements.cronFormSchedule.listeners.input || []).length,
+ presetChangeListeners: (elements.cronFormSchedulePreset.listeners.change || []).length,
+}));
+"""
+ result = json.loads(_run_node(script))
+
+ assert result["warning"] == ""
+ assert result["inputListeners"] == 1
+ assert result["presetChangeListeners"] == 1
+
+
+def test_cron_schedule_custom_selection_preserves_raw_schedule_exactly():
+ script = _cron_schedule_source() + r"""
+const elements = {};
+function $(id) { return elements[id]; }
+function makeElement(initialValue = '') {
+ return {
+ value: initialValue,
+ style: { display: '' },
listeners: {},
addEventListener(type, handler) {
(this.listeners[type] || (this.listeners[type] = [])).push(handler);
@@ -135,64 +434,42 @@ function t(key) {
}
function esc(value) { return value == null ? '' : String(value); }
-elements.cronFormSchedule = makeElement();
-elements.cronFormSchedulePreset = makeElement();
-elements.cronFormScheduleOnceWarning = { style: {} };
+[
+ 'cronFormSchedule',
+ 'cronFormSchedulePreset',
+ 'cronFormSchedulePresetParams',
+ 'cronFormScheduleHourField',
+ 'cronFormScheduleMinuteField',
+ 'cronFormScheduleWeekdayField',
+ 'cronFormScheduleMonthDayField',
+ 'cronFormScheduleHour',
+ 'cronFormScheduleMinute',
+ 'cronFormScheduleWeekday',
+ 'cronFormScheduleMonthDay',
+].forEach((id) => {
+ elements[id] = makeElement();
+});
+elements.cronFormScheduleOnceWarning = { style: { display: 'none' } };
+elements.cronFormSchedulePresetParams.style.display = 'none';
_initCronSchedulePresetControls();
-elements.cronFormSchedule.value = "0 9 * * *";
+elements.cronFormSchedule.value = 'advanced: cron expression';
elements.cronFormSchedule.dispatchEvent('input');
-const selectAfterDaily = elements.cronFormSchedulePreset.value;
-const warningAfterDaily = elements.cronFormScheduleOnceWarning.style.display;
-
-elements.cronFormSchedule.value = "30m";
-elements.cronFormSchedule.dispatchEvent('input');
-const selectAfterOnce = elements.cronFormSchedulePreset.value;
-const warningAfterOnce = elements.cronFormScheduleOnceWarning.style.display;
-
-elements.cronFormSchedulePreset.value = 'hourly';
+elements.cronFormSchedulePreset.value = 'custom';
elements.cronFormSchedulePreset.dispatchEvent('change');
-const presetWriteValue = elements.cronFormSchedule.value;
-const selectAfterPresetWrite = elements.cronFormSchedulePreset.value;
-
-elements.cronFormSchedulePreset.value = "custom";
-elements.cronFormSchedulePreset.dispatchEvent('change');
-const preservedExactPresetValue = elements.cronFormSchedule.value;
-const selectAfterExactPresetCustom = elements.cronFormSchedulePreset.value;
-
-elements.cronFormSchedule.value = "advanced: cron expression";
-elements.cronFormSchedule.dispatchEvent('input');
-elements.cronFormSchedulePreset.value = "custom";
-elements.cronFormSchedulePreset.dispatchEvent('change');
-const preservedCustomValue = elements.cronFormSchedule.value;
-const selectAfterCustom = elements.cronFormSchedulePreset.value;
console.log(JSON.stringify({
- selectAfterDaily,
- warningAfterDaily,
- selectAfterOnce,
- warningAfterOnce,
- presetWriteValue,
- selectAfterPresetWrite,
- preservedExactPresetValue,
- selectAfterExactPresetCustom,
- preservedCustomValue,
- selectAfterCustom,
+ preset: elements.cronFormSchedulePreset.value,
+ schedule: elements.cronFormSchedule.value,
+ paramsDisplay: elements.cronFormSchedulePresetParams.style.display,
}));
"""
result = json.loads(_run_node(script))
- assert result["selectAfterDaily"] == "daily"
- assert result["warningAfterDaily"] == "none"
- assert result["selectAfterOnce"] == "custom"
- assert result["warningAfterOnce"] == ""
- assert result["presetWriteValue"] == "every 1h"
- assert result["selectAfterPresetWrite"] == "hourly"
- assert result["preservedExactPresetValue"] == "every 1h"
- assert result["selectAfterExactPresetCustom"] == "custom"
- assert result["preservedCustomValue"] == "advanced: cron expression"
- assert result["selectAfterCustom"] == "custom"
+ assert result["preset"] == "custom"
+ assert result["schedule"] == "advanced: cron expression"
+ assert result["paramsDisplay"] == "none"
def test_cron_form_surfaces_one_shot_warning_copy_markers_and_preset_markup():
@@ -202,20 +479,44 @@ def test_cron_form_surfaces_one_shot_warning_copy_markers_and_preset_markup():
assert "id=\"cronFormScheduleOnceWarning\"" in panels
assert "id=\"cronFormSchedulePreset\"" in panels
+ assert "id=\"cronFormSchedulePresetParams\"" in panels
+ assert "id=\"cronFormScheduleCustomRow\"" in panels
+ assert "id=\"cronFormScheduleTime\"" in panels
+ assert "type=\"time\"" in panels
+ assert "id=\"cronFormScheduleMinute\"" in panels
+ assert "id=\"cronFormScheduleWeekday\"" in panels
+ assert "id=\"cronFormScheduleMonthDay\"" in panels
+ assert "id=\"cronFormSchedulePreview\"" in panels
assert "cron_schedule_once_warning" in panels
assert "_cronSchedulePresetIdForValue" in panels
assert "_cronSchedulePresetOptionHtml" in panels
assert "_initCronSchedulePresetControls" in panels
- assert "addEventListener('input', _syncCronSchedulePresetAndWarning" in panels
+ # Raw-cron `input` only updates the warning/preview (NOT preset re-detection),
+ # so a partial custom value that transiently matches a preset can't hide the
+ # focused raw field mid-typing (#5554). Preset re-sync runs on change/init.
+ assert "addEventListener('input', _syncCronScheduleWarning" in panels
assert "addEventListener('change', _syncCronSchedulePresetAndWarning" in panels
assert "addEventListener('change', _applyCronSchedulePresetSelection" in panels
+ # UX fix: field edits regenerate the expression on `input` WITHOUT writing the
+ # clamped value back into the field being typed (only clamp on `change`).
+ assert "addEventListener('input', _regenCronScheduleFromFields" in panels
assert ".cron-once-warning" in style
- assert "id: 'hourly'" in panels
- assert "id: 'daily'" in panels
- assert "id: 'weekdays'" in panels
- assert "id: 'weekly'" in panels
- assert "id: 'monthly'" in panels
- assert "id: 'custom'" in panels
+ assert ".cron-schedule-preset-shell" in style
+ assert ".cron-schedule-preset-params" in style
+ assert ".cron-schedule-preset-field" in style
+ assert ".cron-schedule-preset-time-hint" in style
+ assert ".cron-schedule-preview" in style
+ # Time input is themed to match the other form controls (not bare native chrome).
+ assert "input[type=\"time\"]" in style
+ assert "cron_schedule_time_hint" in panels
+ assert "fields: ['minute']" in panels
+ assert "fields: ['time']" in panels
+ assert "fields: ['weekday', 'time']" in panels
+ assert "fields: ['monthDay', 'time']" in panels
+ # Custom is last in the preset order (after the common frequencies).
+ custom_idx = panels.index("id: 'custom'")
+ monthly_idx = panels.index("id: 'monthly'")
+ assert monthly_idx < custom_idx
assert "Duration forms like '30m' run once" in i18n