@agent-native/core 0.134.0 → 0.135.0

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;

package/corpus/templates/content/actions/delete-database-items.ts

@@ -1,89 +0,0 @@
-import { defineAction } from "@agent-native/core";
-import { writeAppState } from "@agent-native/core/application-state";
-import { assertAccess } from "@agent-native/core/sharing";
-import { inArray } from "drizzle-orm";
-
-import { getDb, schema } from "../server/db/index.js";
-import { assertNotWorkspaceCatalogDocuments } from "./_content-space-catalog-guards.js";
-import {
-  databaseRowBatchSchema,
-  renumberDatabaseRows,
-  resolveDatabaseRowsForBatch,
-} from "./_database-row-batch.js";
-import { getContentDatabaseResponse } from "./_database-utils.js";
-import { trashDocumentSubtree } from "./delete-document.js";
-
-export default defineAction({
-  description:
-    "Delete multiple page rows from a content database in one atomic batch. Use this for two or more selected/named rows instead of looping delete-document.",
-  schema: databaseRowBatchSchema,
-  run: async (args) => {
-    const db = getDb();
-    const { database, rows } = await resolveDatabaseRowsForBatch(args);
-
-    await assertAccess("document", database.documentId, "editor");
-    if (database.systemRole === "favorites") {
-      const removedItemIds = rows.map((row) => row.item.id);
-      const removedDocumentIds = rows.map((row) => row.document.id);
-      const now = new Date().toISOString();
-      await db.transaction(async (tx) => {
-        await tx
-          .delete(schema.contentDatabaseItems)
-          .where(inArray(schema.contentDatabaseItems.id, removedItemIds));
-        await renumberDatabaseRows(
-          tx as unknown as ReturnType<typeof getDb>,
-          database,
-          now,
-        );
-      });
-      await writeAppState("refresh-signal", { ts: Date.now() });
-      return {
-        ...(await getContentDatabaseResponse(database.id, {
-          limit: 100,
-          offset: 0,
-        })),
-        deletedItemIds: removedItemIds,
-        deletedDocumentIds: [],
-        deletedCount: 0,
-        removedDocumentIds,
-        removedCount: removedItemIds.length,
-      };
-    }
-    for (const row of rows) {
-      await assertAccess("document", row.document.id, "admin");
-    }
-
-    const deletedItemIds = rows.map((row) => row.item.id);
-    const deletedDocumentIds = rows.map((row) => row.document.id);
-    await assertNotWorkspaceCatalogDocuments(db, deletedDocumentIds, "deleted");
-    const now = new Date().toISOString();
-
-    await db.transaction(async (tx) => {
-      for (const row of rows) {
-        await trashDocumentSubtree(
-          tx as unknown as ReturnType<typeof getDb>,
-          row.document.id,
-          row.document.ownerEmail,
-          now,
-        );
-      }
-      await renumberDatabaseRows(
-        tx as unknown as ReturnType<typeof getDb>,
-        database,
-        now,
-      );
-    });
-
-    await writeAppState("refresh-signal", { ts: Date.now() });
-
-    return {
-      ...(await getContentDatabaseResponse(database.id, {
-        limit: 100,
-        offset: 0,
-      })),
-      deletedItemIds,
-      deletedDocumentIds,
-      deletedCount: deletedItemIds.length,
-    };
-  },
-});