@convex-dev/workpool 0.4.6 → 0.4.7-alpha.0

package/src/component/recovery.test.ts

@@ -126,7 +126,7 @@ describe("recovery", () => {
         const scheduledId = await makeDummyScheduledFunction(ctx, workId);
 
         // Delete the work to simulate it not being found
-        await ctx.db.delete(workId);
+        await ctx.db.delete("work", workId);
 
         return [workId, scheduledId];
       });
@@ -160,9 +160,9 @@ describe("recovery", () => {
         const scheduledId = await makeDummyScheduledFunction(ctx, workId);
 
         // Update the work to have a different attempt number
-        const work = await ctx.db.get(workId);
+        const work = await ctx.db.get("work", workId);
         if (work) {
-          await ctx.db.patch(work._id, { attempts: 5 });
+          await ctx.db.patch("work", work._id, { attempts: 5 });
         }
 
         return [workId, scheduledId];

package/src/component/recovery.ts

@@ -54,7 +54,7 @@ export async function recoveryHandler(
       console.debug(`${preamble} already in pendingCompletion, skipping`);
       continue;
     }
-    const work = await ctx.db.get(job.workId);
+    const work = await ctx.db.get("work", job.workId);
     if (work === null) {
       // Completion already executed w/o retries, no need to do anything.
       console.warn(`${preamble} work not found, skipping`);
@@ -65,7 +65,10 @@ export async function recoveryHandler(
       console.warn(`${preamble} attempts mismatch, skipping`);
       continue;
     }
-    const scheduled = await ctx.db.system.get(job.scheduledId);
+    const scheduled = await ctx.db.system.get(
+      "_scheduled_functions",
+      job.scheduledId,
+    );
     if (scheduled === null) {
       console.warn(`${preamble} not found in _scheduled_functions`);
       completionJobs.push({

package/src/component/schema.ts

@@ -18,6 +18,7 @@ export default defineSchema({
   internalState: defineTable({
     // Ensure that only one main is running at a time.
     generation: v.int64(),
+    // Track where we've scanned to, so we skip tombstones on re-scan.
     segmentCursors: v.object({
       incoming: segment,
       completion: segment,
@@ -42,7 +43,7 @@ export default defineSchema({
     ),
   }),
 
-  // Singleton, written by `updateRunStatus` when running, by client or worker otherwise.
+  // Singleton, written by `main` when scheduling, by client or worker otherwise.
   // Safe to read from kickLoop, since it should update infrequently.
   runStatus: defineTable({
     state: v.union(