From 0c600befaf7b0be059f5b6de3c97674c81a15b6c Mon Sep 17 00:00:00 2001
From: merencia <lucas.merencia@gmail.com>
Date: Tue, 23 Jun 2026 09:49:15 -0300
Subject: [PATCH] feat: accept every/the prefixes on ordinal weekdays

"every last monday of the month" and "the last monday of the month" previously
threw, because parseEvery and the the-clause only handled the bare ordinal form.
Both now delegate to the ordinal-weekday handler and produce the same 1L / 1#1 / L.
"every second" stays the unit: the ordinal path requires a following weekday.
---
 README.md           |  2 +-
 src/to-cron.ts      | 11 ++++++++++-
 test/golden.test.ts |  7 +++++++
 3 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 6faa514..ad86ed3 100644
--- a/README.md
+++ b/README.md
@@ -72,7 +72,7 @@ every <frequency> [at <time>] [on <day>] [in <month>]
 - weekdays: `every monday`, `on friday`, `every weekday`, `every weekend`
 - day-of-month: `on day 15`, `the 15th of every month`
 - months: `in march`, `every january`
-- ordinals (node-cron `L` / `#`): `last day of the month`, `last friday of the month`, `first monday of the month`
+- ordinals (node-cron `L` / `#`): `last friday of the month`, `first monday of the month`, `last day of the month` (an `every` or `the` prefix works too)
 
 ### Values, lists, and ranges
 
diff --git a/src/to-cron.ts b/src/to-cron.ts
index 0dc042a..0063636 100644
--- a/src/to-cron.ts
+++ b/src/to-cron.ts
@@ -221,6 +221,11 @@ export function toCron(expression: string): string {
   function parseEvery(): void {
     i++; // consume 'every'
     const token = peek();
+    // "every last monday of the month", "every first monday", "every last day"
+    if (token in ORDINALS && (words[i + 1] in WEEKDAYS || (token === 'last' && words[i + 1] === 'day'))) {
+      parseOrdinalWeekday();
+      return;
+    }
     if (token === 'other') { i++; everyInterval(words[i++], 2); return; }
     if (isNumber(token)) { const n = Number(token); i++; everyInterval(words[i++], n); return; }
     if (UNITS.has(token)) { i++; everyUnit(token); return; }
@@ -313,7 +318,11 @@ export function toCron(expression: string): string {
     else if (token === 'on') parseOn();
     else if (token === 'in') parseIn();
     else if (token in WEEKDAYS) set('dow', readValues(DOW));
-    else if (token === 'the' || ordinalDay(token) !== null) {
+    else if (token === 'the' && words[i + 1] in ORDINALS
+      && (words[i + 2] in WEEKDAYS || (words[i + 1] === 'last' && words[i + 2] === 'day'))) {
+      i++; // consume 'the' before "last monday of the month"
+      parseOrdinalWeekday();
+    } else if (token === 'the' || ordinalDay(token) !== null) {
       if (token === 'the') i++;
       set('dom', readValues(FIELDS.day));
       consumeOfTheMonth();
diff --git a/test/golden.test.ts b/test/golden.test.ts
index c8ebb9b..64cbae8 100644
--- a/test/golden.test.ts
+++ b/test/golden.test.ts
@@ -103,6 +103,13 @@ describe('toCron — ordinals (L / #)', () => {
     ['last day of the month', '0 0 0 L * *'],
     ['last friday of the month', '0 0 0 * * 5L'],
     ['first monday of the month at 9am', '0 0 9 * * 1#1'],
+    // "every" / "the" prefixes on ordinal weekdays
+    ['every last monday of the month', '0 0 0 * * 1L'],
+    ['the last monday of the month', '0 0 0 * * 1L'],
+    ['every first monday of the month', '0 0 0 * * 1#1'],
+    ['every second tuesday of the month', '0 0 0 * * 2#2'],
+    ['every last day of the month', '0 0 0 L * *'],
+    ['the last day of the month', '0 0 0 L * *'],
   ]);
 });