@andypai/agent-kanban 0.6.1 → 0.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andypai/agent-kanban",
-  "version": "0.6.1",
+  "version": "0.6.3",
   "description": "Agent-friendly kanban board CLI. Manage tasks via bash commands, parse structured JSON output.",
   "homepage": "https://github.com/abpai/agent-kanban#readme",
   "repository": {

package/src/__tests__/api.test.ts

@@ -37,6 +37,54 @@ describe('handleRequest', () => {
     expect(result.mutated).toBe(false)
   })
 
+  test('returns the error envelope for a malformed JSON body', async () => {
+    const req = new Request('http://localhost/api/tasks', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: '{ not valid json',
+    })
+    const result = await handleRequest(provider, req)
+    const body = (await result.response.json()) as {
+      ok: boolean
+      error: { code: string; message: string }
+    }
+
+    expect(result.response.status).toBe(400)
+    expect(result.mutated).toBe(false)
+    expect(body.ok).toBe(false)
+    expect(body.error.code).toBe('INVALID_REQUEST_BODY')
+  })
+
+  test('still returns MISSING_ARGUMENT when a valid body omits a required field', async () => {
+    const req = new Request('http://localhost/api/tasks', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ description: 'no title' }),
+    })
+    const result = await handleRequest(provider, req)
+    const body = (await result.response.json()) as {
+      ok: boolean
+      error: { code: string }
+    }
+
+    expect(result.response.status).toBe(400)
+    expect(result.mutated).toBe(false)
+    expect(body.error.code).toBe('MISSING_ARGUMENT')
+  })
+
+  test('rejects an invalid limit query parameter through the envelope', async () => {
+    const req = new Request('http://localhost/api/tasks?limit=-5', { method: 'GET' })
+    const result = await handleRequest(provider, req)
+    const body = (await result.response.json()) as {
+      ok: boolean
+      error: { code: string }
+    }
+
+    expect(result.response.status).toBe(400)
+    expect(body.ok).toBe(false)
+    expect(body.error.code).toBe('INVALID_ARGUMENT')
+  })
+
   test('marks successful task creation as mutated', async () => {
     const req = new Request('http://localhost/api/tasks', {
       method: 'POST',

package/src/__tests__/conflict.test.ts

@@ -1,6 +1,6 @@
 import { beforeEach, describe, expect, test } from 'bun:test'
 import { Database } from 'bun:sqlite'
-import { initSchema, seedDefaultColumns, resolveColumn } from '../db'
+import { initSchema, seedDefaultColumns, resolveColumn, bulkMoveAll } from '../db'
 import { LocalProvider } from '../providers/local'
 import { ErrorCode, KanbanError } from '../errors'
 
@@ -62,6 +62,30 @@ describe('local provider conflict detection', () => {
     expect(v2.version).toBe('2')
   })
 
+  test('bulkMoveAll bumps the task version so a stale expectedVersion conflicts', async () => {
+    const created = await provider.createTask({ title: 'T1', column: 'backlog' })
+    expect(created.version).toBe('0')
+
+    const { moved } = bulkMoveAll(db, 'backlog', 'in-progress')
+    expect(moved).toBe(1)
+
+    // The move is a real mutation, so the version must change.
+    const after = await provider.getTask(created.id)
+    expect(after.version).toBe('1')
+
+    let err: unknown
+    try {
+      await provider.updateTask(created.id, {
+        title: 'stale',
+        expectedVersion: created.version ?? undefined,
+      })
+    } catch (e) {
+      err = e
+    }
+    expect(err).toBeInstanceOf(KanbanError)
+    expect((err as KanbanError).code).toBe(ErrorCode.CONFLICT)
+  })
+
   // moveTask(id, column) intentionally has no expectedVersion parameter, so moves
   // are last-write-wins by design (drag-and-drop has no loaded version to check).
   // This locks in that contract so a move is never rejected for a stale version.