fix: treat ? (Quartz no-value) like * in toHuman (#9)

A ? field was rendered as a literal value, producing nonsense such as
"on day ?" or "on undefined". ? means "no constraint on this field" (Quartz /
AWS use it in day-of-month or day-of-week), so it is now normalized to *.
"0 0 9 ? * MON" now reads as "at 9am on monday".

Author: merencia

src/to-human.ts

@@ -154,9 +154,9 @@ export function toHuman(expression: string): string {
   if (parts.length !== 5 && parts.length !== 6) {
     throw new CronTranslateError(`Expected a 5- or 6-field cron expression, got ${parts.length} fields.`);
   }
-  const [sec, min, hour, dom, monthRaw, dowRaw] = parts.length === 6
-    ? parts
-    : ['0', ...parts];
+  // `?` (Quartz "no specific value") means no constraint on the field, same as `*`.
+  const fields = (parts.length === 6 ? parts : ['0', ...parts]).map((f) => (f === '?' ? '*' : f));
+  const [sec, min, hour, dom, monthRaw, dowRaw] = fields;
   const month = mapNames(monthRaw, MONTH_NUMS);
   const dow = mapNames(dowRaw, DOW_NUMS);
 

test/to-human.test.ts

@@ -69,6 +69,17 @@ describe('toHuman — accepts month/weekday names in input', () => {
   });
 });
 
+describe('toHuman — treats ? (Quartz no-value) like *', () => {
+  const cases: [string, string][] = [
+    ['0 0 9 ? * MON', 'at 9am on monday'],
+    ['0 0 9 15 * ?', 'at 9am on day 15'],
+    ['0 0 0 ? * 1-5', 'on monday to friday'],
+  ];
+  it.each(cases)('%s => %s', (cron, expected) => {
+    expect(toHuman(cron)).toBe(expected);
+  });
+});
+
 describe('toHuman — rejects malformed input', () => {
   it.each(['', 'a b c', '1 2 3 4 5 6 7'])('rejects %j', (bad) => {
     expect(() => toHuman(bad)).toThrow();