@quark-fw/plugin-cron 0.1.19 → 0.1.20

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@quark-fw/plugin-cron",
-	"version": "0.1.19",
+	"version": "0.1.20",
 	"main": "src/index.ts",
 	"types": "src/index.ts",
 	"type": "module",
@@ -23,12 +23,12 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@quark-fw/atom": "0.1.19",
-		"@quark-fw/plugin": "0.1.19"
+		"@quark-fw/atom": "0.1.20",
+		"@quark-fw/plugin": "0.1.20"
 	},
 	"peerDependencies": {
-		"@quark-fw/core": "0.1.19",
-		"@quark-fw/plugin-redis": "0.1.19",
+		"@quark-fw/core": "0.1.20",
+		"@quark-fw/plugin-redis": "0.1.20",
 		"ioredis": "^5.9.3"
 	},
 	"devDependencies": {

package/src/CronScheduler.test.ts

@@ -78,6 +78,22 @@ describe("CronScheduler", () => {
 		expect((await cron.status())[0]!.stale).toBe(true);
 	});
 
+	test("не запускавшаяся задача протухает только спустя maxAge после регистрации", async () => {
+		const { client } = fakeRedis();
+		let now = at("2026-10-06T12:00:00Z");
+		const cron = new CronScheduler({
+			client: () => client,
+			now: () => now,
+			prefix: "t:",
+		});
+		cron.schedule("job", "0 3 * * *", async () => {}, {
+			maxAgeMs: 3_600_000,
+		});
+		expect((await cron.status())[0]!.stale).toBe(false);
+		now = at("2026-10-06T13:30:00Z");
+		expect((await cron.status())[0]!.stale).toBe(true);
+	});
+
 	test("лок в Redis: второй инстанс не запускает ту же задачу", async () => {
 		const { client } = fakeRedis();
 		const a = new CronScheduler({ client: () => client, prefix: "t:" });

package/src/CronScheduler.ts

@@ -43,7 +43,11 @@ export class CronScheduler {
 	private readonly onError: (job: string, error: unknown) => void;
 	private readonly jobs = new Map<
 		string,
-		CronJob & { fields: CronFields; firedMinute?: string }
+		CronJob & {
+			fields: CronFields;
+			firedMinute?: string;
+			registeredAt: number;
+		}
 	>();
 	private readonly running = new Set<string>();
 	private timer: NodeJS.Timeout | undefined;
@@ -78,6 +82,7 @@ export class CronScheduler {
 			handler,
 			options: { lockTtlMs: DEFAULT_LOCK_TTL, ...options },
 			fields: parseCron(expression),
+			registeredAt: this.now().getTime(),
 		});
 		if (options.runOnStart) void this.run(name);
 	}
@@ -189,12 +194,17 @@ export class CronScheduler {
 				this.onError(job.name, error);
 			}
 			const lastSuccessAt = raw.lastSuccessAt || null;
+			// протухла: успех старше maxAge, а если успеха ещё не было — задача
+			// зарегистрирована дольше maxAge назад (свежий контейнер не виноват,
+			// что расписание ещё не наступило)
 			const stale =
 				job.options.maxAgeMs === undefined
 					? undefined
-					: !lastSuccessAt ||
-						this.now().getTime() -
-							new Date(lastSuccessAt).getTime() >
+					: lastSuccessAt
+						? this.now().getTime() -
+								new Date(lastSuccessAt).getTime() >
+							job.options.maxAgeMs
+						: this.now().getTime() - job.registeredAt >
 							job.options.maxAgeMs;
 			result.push({
 				name: job.name,