@skrr-ai/cli 0.1.127 → 0.1.129

package/dist/commands/tasks/schedule.js

@@ -5,6 +5,8 @@ const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const web_url_1 = require("../../lib/web-url");
 const prompt_1 = require("../../lib/prompt");
+const iso_instant_1 = require("../../lib/iso-instant");
+const task_recurrence_summary_1 = require("../../lib/task-recurrence-summary");
 class TasksSchedule extends base_command_1.BaseCommand {
     static description = 'Schedule a task to fire at a future time (optionally recurring)';
     static examples = [
@@ -21,10 +23,11 @@ class TasksSchedule extends base_command_1.BaseCommand {
             description: 'First fire time as ISO-8601 UTC ending in Z (e.g. 2026-06-01T15:30:00Z); numeric offsets such as +00:00 are not accepted',
         }),
         cron: core_1.Flags.string({
-            description: 'Recurrence cron pattern (UTC unless --timezone is set)',
+            description: 'Recurrence cron pattern: minute hour day-of-month month day-of-week (UTC unless --timezone is set). ' +
+                'When both a day of the month and a weekday are set, it fires when EITHER matches',
         }),
         timezone: core_1.Flags.string({
-            description: 'IANA timezone for the cron pattern (e.g. America/Los_Angeles)',
+            description: 'IANA zone name for the cron pattern, e.g. Asia/Shanghai or America/Los_Angeles (not an offset such as UTC+8, and not "local")',
             dependsOn: ['cron'],
         }),
         'max-runs': core_1.Flags.integer({
@@ -32,7 +35,7 @@ class TasksSchedule extends base_command_1.BaseCommand {
             dependsOn: ['cron'],
         }),
         'end-date': core_1.Flags.string({
-            description: 'Stop recurrence after this ISO-8601 date',
+            description: 'Last day of the recurrence: YYYY-MM-DD (through the end of that day in --timezone, UTC if omitted) or an ISO-8601 instant with a zone (2026-12-31T17:00:00Z); must be after the first fire',
             dependsOn: ['cron'],
         }),
         'from-json': core_1.Flags.string({
@@ -52,8 +55,10 @@ class TasksSchedule extends base_command_1.BaseCommand {
         }
         const recurrence = flags.cron
             ? (0, prompt_1.mergeDefined)({}, {
-                pattern: flags.cron,
-                timezone: flags.timezone,
+                // Stored as typed otherwise: a padded pattern came back padded from `show --json`, and a blank
+                // `--timezone ""` printed a recurrence with no zone at all (OSK-21531).
+                pattern: flags.cron.trim(),
+                timezone: flags.timezone?.trim() || undefined,
                 maxRuns: flags['max-runs'],
                 endDate: flags['end-date'],
             })
@@ -65,6 +70,11 @@ class TasksSchedule extends base_command_1.BaseCommand {
         if (!payload.scheduledFor || typeof payload.scheduledFor !== 'string') {
             this.error('A first-fire time is required. Pass --at <ISO-8601 UTC ending in Z> or include scheduledFor in --from-json using the same format.', { exit: 1 });
         }
+        // Said here, in the user's words: the server's zod issue names the REST field (`scheduledFor`) and is the same
+        // text for a missing Z, a numeric offset and a day that does not exist (OSK-21507).
+        const instant = (0, iso_instant_1.parseUtcInstant)(payload.scheduledFor, flags.at !== undefined ? '--at' : 'scheduledFor');
+        if (!instant.ok)
+            this.error(instant.message, { exit: 2 });
         let result;
         try {
             result = await data_provider_1.dataService.scheduleTask({
@@ -83,8 +93,20 @@ class TasksSchedule extends base_command_1.BaseCommand {
         const ref = t.identifier ?? (t.taskNumber ? `#${t.taskNumber}` : '');
         this.log(`Scheduled task ${t.id ?? t._id ?? taskRef}${ref ? ` (${ref})` : ''} for ${payload.scheduledFor}`);
         if (t.recurrence?.pattern) {
-            const tz = t.recurrence.timezone ? ` (${t.recurrence.timezone})` : '';
-            this.log(`Recurrence: ${t.recurrence.pattern}${tz}`);
+            // Always name the zone: the pattern is read in UTC unless one was set, and a rule with no zone beside it
+            // is exactly what could not be told apart from a local-time one.
+            const tz = t.recurrence.timezone?.trim() || 'UTC';
+            this.log(`Recurrence: ${t.recurrence.pattern} (${tz})`);
+            const reading = (0, task_recurrence_summary_1.describeCronPattern)(t.recurrence.pattern);
+            if (reading) {
+                this.log(`            ${reading.text}`);
+                if (reading.dayOfMonthOrWeekday) {
+                    this.log('            With both a day of the month and a weekday, it fires when EITHER matches, not only when both do.');
+                }
+            }
+            // The instant the server resolved `--end-date` to, so a calendar date is never a guess.
+            if (t.recurrence.endDate)
+                this.log(`Ends: ${t.recurrence.endDate}`);
         }
     }
 }

package/dist/commands/tasks/show.js

@@ -23,6 +23,7 @@ const principal_display_1 = require("../../lib/principal-display");
 const task_economics_line_1 = require("../../lib/task-economics-line");
 const format_1 = require("../../lib/format");
 const task_recurrence_outcome_1 = require("../../lib/task-recurrence-outcome");
+const task_recurrence_summary_1 = require("../../lib/task-recurrence-summary");
 const record_output_1 = require("../../lib/record-output");
 /**
  * Where to look next — as COMMANDS, not as facts about the task.
@@ -441,6 +442,9 @@ class TasksShow extends base_command_1.BaseCommand {
             this.log(line);
         for (const line of recurrenceArmFailureLines(t, this.config.bin))
             this.log(line);
+        // That the Task recurs, by what rule and in which zone (OSK-21531): only `--json` carried it.
+        for (const line of (0, task_recurrence_summary_1.recurrenceSummaryLines)(t))
+            this.log(line);
         /*
          * Closing is the actor's call — a Task may be `done` with work still
          * outstanding, and `resolution.closure` is where the server recorded what