@agent-native/core 0.134.1 → 0.135.1

package/src/templates/chat/app/routes/settings.tsx

@@ -3,6 +3,8 @@ import { LanguagePicker, useT } from "@agent-native/core/client/i18n";
 import { TeamPage } from "@agent-native/core/client/org";
 import {
   AccountSettingsCard,
+  SettingsGroup,
+  SettingsRow,
   SettingsTabsPage,
   useAgentSettingsTabs,
   type SettingsSearchEntry,
@@ -10,14 +12,6 @@ import {
 import { useSetPageTitle } from "@agent-native/toolkit/app-shell";
 import { useMemo } from "react";
 
-import {
-  Card,
-  CardContent,
-  CardDescription,
-  CardHeader,
-  CardTitle,
-} from "@/components/ui/card";
-import { Label } from "@/components/ui/label";
 import { APP_TITLE } from "@/lib/app-config";
 
 import changelog from "../../CHANGELOG.md?raw";
@@ -55,20 +49,18 @@ export default function SettingsRoute() {
             {t("settings.description")}
           </p>
 
-          <Card id="language" className="scroll-mt-16">
-            <CardHeader>
-              <CardTitle className="text-base">
-                {t("settings.languageTitle")}
-              </CardTitle>
-              <CardDescription>
-                {t("settings.languageDescription")}
-              </CardDescription>
-            </CardHeader>
-            <CardContent className="max-w-xs space-y-1.5">
-              <Label>{t("settings.languageLabel")}</Label>
-              <LanguagePicker label={t("settings.languageLabel")} />
-            </CardContent>
-          </Card>
+          <SettingsGroup>
+            <SettingsRow
+              id="language"
+              label={t("settings.languageTitle")}
+              description={t("settings.languageDescription")}
+              control={
+                <div className="w-56">
+                  <LanguagePicker label={t("settings.languageLabel")} />
+                </div>
+              }
+            />
+          </SettingsGroup>
         </div>
       }
       team={

package/src/templates/chat/vitest.config.ts

@@ -0,0 +1,9 @@
+import baseConfig from "@agent-native/core/vitest-config";
+import { mergeConfig } from "vitest/config";
+
+import viteConfig from "./vite.config";
+
+// Kept separate from vite.config.ts so the production config never imports
+// vitest: `agent-native build` loads vite.config.ts in installs where vitest
+// is absent.
+export default mergeConfig(viteConfig, baseConfig);

package/src/triggers/actions/list-automations.ts

@@ -5,7 +5,12 @@ import {
   listAutomationDefinitions,
   type AutomationScope,
 } from "../../automations/service.js";
-import { describeCron, isValidCron, nextOccurrence } from "../../jobs/cron.js";
+import {
+  describeCron,
+  effectiveTimezone,
+  isValidCron,
+  nextOccurrence,
+} from "../../jobs/cron.js";
 
 const scopeSchema = z.enum(["personal", "organization"]);
 
@@ -13,15 +18,23 @@ function nextRun(
   meta: Awaited<ReturnType<typeof listAutomationDefinitions>>[number]["meta"],
 ): string | null {
   if (!meta.enabled) return null;
-  if (meta.nextRun) return meta.nextRun;
-  if (
+  const scheduled = Boolean(
     meta.triggerType === "schedule" &&
     meta.schedule &&
-    isValidCron(meta.schedule)
-  ) {
-    return nextOccurrence(meta.schedule).toISOString();
+    isValidCron(meta.schedule),
+  );
+  // A stored `nextRun` in the past means the dispatcher kept declining to run
+  // this automation, not that it is overdue. Report the real next occurrence
+  // and let `lastError` carry the reason it keeps being passed over.
+  if (meta.nextRun) {
+    const stored = new Date(meta.nextRun).getTime();
+    if (!Number.isFinite(stored) || stored > Date.now() || !scheduled) {
+      return meta.nextRun;
+    }
   }
-  return null;
+  return scheduled
+    ? nextOccurrence(meta.schedule!, undefined, meta.timezone).toISOString()
+    : null;
 }
 
 export interface AutomationActionItem {
@@ -32,11 +45,13 @@ export interface AutomationActionItem {
   triggerType: "event" | "schedule";
   event: string | null;
   schedule: string | null;
+  timezone: string | null;
   scheduleDescription: string | null;
   condition: string | null;
   body: string;
   enabled: boolean;
   lastRun: string | null;
+  lastCheck: string | null;
   lastStatus: string | null;
   lastError: string | null;
   nextRun: string | null;
@@ -76,11 +91,15 @@ export default defineAction({
       triggerType: meta.triggerType,
       event: meta.event ?? null,
       schedule: meta.schedule || null,
-      scheduleDescription: meta.schedule ? describeCron(meta.schedule) : null,
+      timezone: meta.schedule ? effectiveTimezone(meta.timezone) : null,
+      scheduleDescription: meta.schedule
+        ? describeCron(meta.schedule, effectiveTimezone(meta.timezone))
+        : null,
       condition: meta.condition ?? null,
       body,
       enabled: meta.enabled,
       lastRun: meta.lastRun ?? null,
+      lastCheck: meta.lastCheck ?? null,
       lastStatus: meta.lastStatus ?? null,
       lastError: meta.lastError ?? null,
       nextRun: nextRun(meta),

package/src/triggers/actions/manage-automation.ts

@@ -16,8 +16,10 @@ export default defineAction({
     name: z.string().min(1),
     scope: z.enum(["personal", "organization"]).default("personal"),
     enabled: z.boolean().optional(),
+    schedule: z.string().min(1).optional(),
+    timezone: z.string().min(1).optional(),
   }),
-  run: async ({ operation, name, scope, enabled }, ctx) => {
+  run: async ({ operation, name, scope, enabled, schedule, timezone }, ctx) => {
     const userEmail = ctx?.userEmail;
     if (!userEmail) throw new Error("Not authenticated.");
     const actor = { userEmail, orgId: ctx?.orgId };
@@ -27,20 +29,29 @@ export default defineAction({
       await refreshEventSubscriptions();
       return { deleted: true, name };
     }
-    if (enabled === undefined) {
-      throw Object.assign(new Error("enabled is required for update."), {
-        statusCode: 400,
-      });
+    if (
+      enabled === undefined &&
+      schedule === undefined &&
+      timezone === undefined
+    ) {
+      throw Object.assign(
+        new Error("enabled, schedule, or timezone is required for update."),
+        { statusCode: 400 },
+      );
     }
     const definition = await updateAutomation(actor, {
       name,
       scope,
-      enabled,
+      ...(enabled === undefined ? {} : { enabled }),
+      ...(schedule === undefined ? {} : { schedule }),
+      ...(timezone === undefined ? {} : { timezone }),
     });
     await refreshEventSubscriptions();
     return {
       name,
       enabled: definition.meta.enabled,
+      schedule: definition.meta.schedule || null,
+      timezone: definition.meta.timezone ?? null,
       nextRun: definition.meta.nextRun ?? null,
     };
   },

package/src/triggers/actions.ts

@@ -18,7 +18,7 @@ import {
   type AutomationScope,
 } from "../automations/service.js";
 import { listEvents } from "../event-bus/index.js";
-import { describeCron } from "../jobs/cron.js";
+import { describeCron, effectiveTimezone } from "../jobs/cron.js";
 import {
   getIntegrationRequestContext,
   getRequestOrgId,
@@ -71,7 +71,10 @@ async function handleList(
       triggerType: meta.triggerType,
       event: meta.event ?? null,
       schedule: meta.schedule || null,
-      scheduleDescription: meta.schedule ? describeCron(meta.schedule) : null,
+      timezone: meta.timezone ? effectiveTimezone(meta.timezone) : null,
+      scheduleDescription: meta.schedule
+        ? describeCron(meta.schedule, effectiveTimezone(meta.timezone))
+        : null,
       condition: meta.condition ?? null,
       mode: meta.mode,
       domain: meta.domain ?? null,
@@ -127,6 +130,7 @@ async function handleDefine(
         triggerType: automationTriggerType(args.trigger_type),
         body: typeof args.body === "string" ? args.body : "",
         schedule: typeof args.schedule === "string" ? args.schedule : undefined,
+        timezone: typeof args.timezone === "string" ? args.timezone : undefined,
         event: typeof args.event === "string" ? args.event : undefined,
         condition:
           typeof args.condition === "string" ? args.condition : undefined,
@@ -164,6 +168,7 @@ async function handleDefine(
       triggerType: definition.meta.triggerType,
       event: definition.meta.event ?? null,
       schedule: definition.meta.schedule || null,
+      timezone: definition.meta.timezone ?? null,
       nextRun: definition.meta.nextRun ?? null,
       createdBy: definition.meta.createdBy,
       runAs: definition.meta.runAs,
@@ -208,6 +213,7 @@ async function handleUpdate(
               : null,
         body: typeof args.body === "string" ? args.body : undefined,
         schedule: typeof args.schedule === "string" ? args.schedule : undefined,
+        timezone: typeof args.timezone === "string" ? args.timezone : undefined,
         model:
           args.model === undefined
             ? undefined
@@ -225,6 +231,7 @@ async function handleUpdate(
       triggerType: definition.meta.triggerType,
       enabled: definition.meta.enabled,
       schedule: definition.meta.schedule || null,
+      timezone: definition.meta.timezone ?? null,
       nextRun: definition.meta.nextRun ?? null,
       createdBy: definition.meta.createdBy,
       runAs: definition.meta.runAs,
@@ -305,8 +312,8 @@ export function createAutomationToolEntries(
 
 - **list-events**: List all registered event types that automations can subscribe to. Returns event names, descriptions, and payload schemas. Call this BEFORE defining an automation to discover available events.
 - **list**: List all automations (triggers). Shows trigger, status, model, MCP allowlist, and delivery metadata. Optional params: scope, domain, enabled_only.
-- **define**: Create a new automation. IMPORTANT: Always confirm with the user before calling — show them a summary of what will be created. Required params: name, trigger_type, body. Optional: scope, event, schedule, condition, mode, domain, delegated_policy_id, model, mcpTools.
-- **update**: Update an existing automation's settings without changing its creator (enabled, schedule, condition, body, policy, model, MCP allowlist). Required param: name. Use the same scope it was created in.
+- **define**: Create a new automation. IMPORTANT: Always confirm with the user before calling — show them a summary of what will be created. Required params: name, trigger_type, body. Optional: scope, event, schedule, timezone, condition, mode, domain, delegated_policy_id, model, mcpTools.
+- **update**: Update an existing automation's settings without changing its creator (enabled, schedule, timezone, condition, body, policy, model, MCP allowlist). Required param: name. Use the same scope it was created in.
 - **delete**: Delete an automation. Always confirm with the user first. Required param: name.
 - **fire-test**: Fire a test event to validate automations. Emits a test.event.fired event. Optional param: data (JSON string).`,
         parameters: {
@@ -339,6 +346,11 @@ export function createAutomationToolEntries(
               description:
                 "For event triggers: the event name to subscribe to. Call with action=list-events first to see available events.",
             },
+            timezone: {
+              type: "string",
+              description:
+                "IANA timezone the cron clock time is read in, e.g. 'America/New_York'. Optional; defaults to the user's saved scheduling timezone, then the caller's browser zone. Always pass this when the user names a time of day.",
+            },
             schedule: {
               type: "string",
               description:

package/src/triggers/dispatcher.ts

@@ -82,6 +82,31 @@ const _eventSubscriptions = new Map<string, string>();
 const _dispatchingTriggers = new Set<string>();
 let _deps: TriggerDispatcherDeps | null = null;
 
+/**
+ * Record that a tick evaluated this trigger and declined to dispatch it.
+ * `lastRun` stays untouched — nothing ran — and an unchanged outcome is not
+ * re-persisted, so a permanently blocked trigger neither reports phantom runs
+ * nor rewrites its resource on every matching event.
+ */
+async function recordTriggerSkip(
+  resource: Resource,
+  meta: TriggerFrontmatter,
+  body: string,
+  status: "skipped" | "error",
+  reason: string | undefined,
+): Promise<void> {
+  const unchanged = meta.lastStatus === status && meta.lastError === reason;
+  meta.lastCheck = new Date().toISOString();
+  meta.lastStatus = status;
+  meta.lastError = reason;
+  if (unchanged) return;
+  await resourcePut(
+    resource.owner,
+    resource.path,
+    buildTriggerContent(meta, body),
+  );
+}
+
 /**
  * Initialize the trigger dispatcher. Call once at server startup.
  * Loads all event-triggered jobs and subscribes to their events.
@@ -173,25 +198,22 @@ async function handleEvent(
             meta,
           );
         } catch {
-          meta.lastRun = new Date().toISOString();
-          meta.lastStatus = "skipped";
-          meta.lastError =
-            "Could not verify the automation execution identity.";
-          await resourcePut(
-            resource.owner,
-            resource.path,
-            buildTriggerContent(meta, body),
+          await recordTriggerSkip(
+            resource,
+            meta,
+            body,
+            "skipped",
+            "Could not verify the automation execution identity.",
           );
           continue;
         }
         if (!resolved.ok) {
-          meta.lastRun = new Date().toISOString();
-          meta.lastStatus = "skipped";
-          meta.lastError = resolved.reason;
-          await resourcePut(
-            resource.owner,
-            resource.path,
-            buildTriggerContent(meta, body),
+          await recordTriggerSkip(
+            resource,
+            meta,
+            body,
+            "skipped",
+            resolved.reason,
           );
           continue;
         }
@@ -206,13 +228,12 @@ async function handleEvent(
       const userApiKey = await getOwnerActiveApiKey(owner);
       const apiKey = userApiKey || _deps.apiKey;
       if (!apiKey) {
-        meta.lastRun = new Date().toISOString();
-        meta.lastStatus = "error";
-        meta.lastError = "No API key is available for this automation";
-        await resourcePut(
-          resource.owner,
-          resource.path,
-          buildTriggerContent(meta, body),
+        await recordTriggerSkip(
+          resource,
+          meta,
+          body,
+          "error",
+          "No API key is available for this automation",
         );
         console.warn(`[triggers] ${meta.lastError}: "${resource.path}"`);
         continue;
@@ -221,14 +242,7 @@ async function handleEvent(
       // Evaluate condition
       const matches = await evaluateCondition(meta.condition, payload, apiKey);
       if (!matches) {
-        meta.lastRun = new Date().toISOString();
-        meta.lastStatus = "skipped";
-        meta.lastError = undefined;
-        await resourcePut(
-          resource.owner,
-          resource.path,
-          buildTriggerContent(meta, body),
-        );
+        await recordTriggerSkip(resource, meta, body, "skipped", undefined);
         continue;
       }
 

package/src/triggers/routes.ts

@@ -7,7 +7,12 @@ import {
 
 import { canUpdateAutomationResource } from "../automations/service.js";
 import { getDbExec } from "../db/client.js";
-import { nextOccurrence, describeCron, isValidCron } from "../jobs/cron.js";
+import {
+  nextOccurrence,
+  describeCron,
+  effectiveTimezone,
+  isValidCron,
+} from "../jobs/cron.js";
 import {
   buildJobResourceContent,
   parseJobResource,
@@ -88,10 +93,10 @@ function normalizeAutomationPath(input: SetAutomationEnabledInput): string {
   return path;
 }
 
-function scheduleDescription(schedule: string | undefined): string | undefined {
+function scheduleDescription(schedule?: string, timezone?: string) {
   if (!schedule) return undefined;
   try {
-    return describeCron(schedule);
+    return describeCron(schedule, effectiveTimezone(timezone));
   } catch {
     return schedule;
   }
@@ -106,7 +111,11 @@ function nextRunForMeta(meta: TriggerFrontmatter): string | undefined {
     isValidCron(meta.schedule)
   ) {
     try {
-      return nextOccurrence(meta.schedule).toISOString();
+      return nextOccurrence(
+        meta.schedule,
+        undefined,
+        meta.timezone,
+      ).toISOString();
     } catch {
       return undefined;
     }
@@ -194,7 +203,7 @@ async function resourceToAutomationItem(
     triggerType: meta.triggerType,
     event: meta.event,
     schedule: meta.schedule || undefined,
-    scheduleDescription: scheduleDescription(meta.schedule),
+    scheduleDescription: scheduleDescription(meta.schedule, meta.timezone),
     condition: meta.condition,
     mode: meta.mode,
     domain: meta.domain,
@@ -292,7 +301,11 @@ export async function setAutomationEnabledForOwner(
     meta.schedule &&
     isValidCron(meta.schedule)
   ) {
-    parsed.meta.nextRun = nextOccurrence(meta.schedule).toISOString();
+    parsed.meta.nextRun = nextOccurrence(
+      meta.schedule,
+      undefined,
+      meta.timezone,
+    ).toISOString();
   }
 
   const updatedContent = buildJobResourceContent(parsed.meta, parsed.body);

package/src/vitest-config.ts

@@ -0,0 +1,68 @@
+import type { ViteUserConfig } from "vitest/config";
+
+/**
+ * Base vitest config that caps a suite's worker pool at a share of the machine.
+ *
+ * Vitest sizes its pool at `cores - 1` with no awareness of peer processes, so
+ * several suites running at once — concurrent agent sessions, or a workspace
+ * running packages in parallel — each try to claim the whole box. Set
+ * `VITEST_CONCURRENCY` to a percentage or a worker count to change the lid; a
+ * package that needs its own value sets `test.maxWorkers` in its own config,
+ * which wins the merge.
+ *
+ * This ships from core rather than living at the monorepo root because template
+ * directories are scaffolded out verbatim as standalone apps — a config that
+ * reaches above the template would resolve to nothing in the generated app.
+ *
+ * Keep this module free of runtime imports so loading it costs a config file
+ * nothing beyond reading the environment.
+ */
+
+const DEFAULT_MAX_WORKERS = "25%";
+const ENV_KEYS = ["VITEST_CONCURRENCY", "AGENT_NATIVE_VITEST_CONCURRENCY"];
+
+/**
+ * Resolve the worker lid from the environment, throwing on anything that is
+ * neither a percentage nor a worker count. A bad value must not quietly become
+ * the default — that reads as "configured" while running at some other size.
+ */
+export function resolveMaxWorkers(
+  env: NodeJS.ProcessEnv = process.env,
+): string | number {
+  const rawMaxWorkers = env.VITEST_MAX_WORKERS;
+  if (rawMaxWorkers?.includes("%")) {
+    throw new Error(
+      `VITEST_MAX_WORKERS=${rawMaxWorkers} is not a percentage to vitest — it parses as ` +
+        `${Number.parseInt(rawMaxWorkers)} workers. Use VITEST_CONCURRENCY for percentages, ` +
+        `or an integer for VITEST_MAX_WORKERS.`,
+    );
+  }
+
+  const key = ENV_KEYS.find((name) => env[name]);
+  if (!key) return DEFAULT_MAX_WORKERS;
+
+  const value = env[key]!.trim();
+  if (/^\d+%$/.test(value)) {
+    const percent = Number.parseInt(value);
+    if (percent < 1 || percent > 100) {
+      throw new Error(`${key}=${value} is out of range — use 1% to 100%.`);
+    }
+    return value;
+  }
+
+  const count = Number(value);
+  if (!Number.isInteger(count) || count < 1) {
+    throw new Error(
+      `${key}=${value} is not a percentage like "25%" or a worker count like "2".`,
+    );
+  }
+  return count;
+}
+
+const vitestBaseConfig: ViteUserConfig = {
+  test: {
+    maxWorkers: resolveMaxWorkers(),
+  },
+};
+
+export default vitestBaseConfig;