@strav/queue 1.0.0-alpha.4 → 1.0.0-alpha.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@strav/queue",
-  "version": "1.0.0-alpha.4",
+  "version": "1.0.0-alpha.5",
   "description": "Strav queue layer — Job + JobRegistry + dispatch contract; backends ship as drivers",
   "type": "module",
   "main": "./src/index.ts",
@@ -19,9 +19,9 @@
     "access": "public"
   },
   "dependencies": {
-    "@strav/cli": "1.0.0-alpha.4",
-    "@strav/database": "1.0.0-alpha.4",
-    "@strav/kernel": "1.0.0-alpha.4"
+    "@strav/cli": "1.0.0-alpha.5",
+    "@strav/database": "1.0.0-alpha.5",
+    "@strav/kernel": "1.0.0-alpha.5"
   },
   "peerDependencies": {
     "@types/bun": ">=1.3.14"

package/src/database_queue.ts

@@ -138,7 +138,10 @@ export class DatabaseQueue implements Queue {
       ) VALUES (
         $1, $2, $3, $4::jsonb, 0, $5, ${availableAtFragment}, now(), now()
       )`,
-      [jobId, queue, jobClass.jobName, JSON.stringify(payload), maxAttempts],
+      // `payload` defaults to `{}` for jobs that don't take input — without
+      // this fallback `JSON.stringify(undefined)` yields `undefined`, which
+      // arrives as NULL and trips the column's NOT NULL constraint.
+      [jobId, queue, jobClass.jobName, JSON.stringify(payload ?? {}), maxAttempts],
     )
     return jobId
   }

package/src/worker.ts

@@ -237,6 +237,11 @@ export class Worker {
    * claim is durable + safe against concurrent Workers.
    */
   private async claim(): Promise<JobRow | null> {
+    // Bun.SQL 1.3.x's positional bind path doesn't auto-wrap JS arrays as
+    // Postgres array literals — passing `string[]` to a `::text[]` parameter
+    // arrives as the joined string ("default") and Postgres rejects it as a
+    // malformed array literal. Format the literal explicitly: `{"a","b"}`.
+    const queuesLiteral = `{${this.queues.map((q) => `"${q.replace(/"/g, '\\"')}"`).join(',')}}`
     return this.db.transaction(async (tx) => {
       const row = await tx.queryOne<JobRow>(
         `SELECT id, queue, job_name, payload, attempts, max_attempts
@@ -247,9 +252,15 @@ export class Worker {
          ORDER BY id
          LIMIT 1
          FOR UPDATE SKIP LOCKED`,
-        [this.queues],
+        [queuesLiteral],
       )
       if (!row) return null
+      // Bun.SQL 1.3.x returns jsonb as a raw string from the positional
+      // bind path — parse it back so JobContext.payload is the original
+      // object the job dispatched.
+      if (typeof row.payload === 'string') {
+        row.payload = JSON.parse(row.payload)
+      }
       await tx.execute(
         `UPDATE "strav_jobs"
          SET reserved_at = now(), attempts = attempts + 1, updated_at = now()