debug: log Kanban state-change DnD flow end-to-end (#391) by BenGWeeks · Pull Request #392 · knowall-ai/zapdesk

BenGWeeks · 2026-05-15T16:13:43Z

Summary

Investigative logging for [BUG] Cannot drag a task from the "New" column to the "To Do" column on the Kanban board #391 (cannot drag New → To Do on the Kanban board). Not a fix — adds structured logs at every hop of the state-change DnD flow so we can see exactly why Azure DevOps is rejecting the transition.
The underlying AzureDevOpsService.updateTicketState was only throwing response.statusText, dropping DevOps's real error body (commonly TF237124: An invalid state transition from X to Y was requested). That's now captured and propagated all the way back to the browser console.
The /api/devops/tickets/[id]/state route now returns 400 with the real error message instead of a generic 500, so the client sees the cause.

How to use

Pull this branch, run bun dev, open the Kanban board with DevTools console open (and the server log visible).
Attempt the failing drag (e.g. New → To Do).
Capture:
- The [Kanban DnD] log line at drag-end (shows fromState/targetState/known states).
- The [Kanban] PATCH log + the [Kanban] state PATCH rejected log with status + body.
- The server-side [state PATCH] and [devops.updateTicketState] logs.
The DevOps error body in those logs will tell us whether it's a process-template transition restriction, a missing required field, or something else, and inform the real fix.

Test plan

bun run check passes locally (done).
No behavioural change on successful drags — card moves and stays.
On a failing drag, the browser console shows the DevOps error body and the card rolls back as before.

🤖 Generated with Claude Code

When a card is dragged between columns, the persisted state change can silently fail (the card snaps back) — e.g. dragging New -> To Do. The underlying updateTicketState only threw response.statusText, so the actual Azure DevOps rejection reason was lost. Add structured logging at every hop so we can see exactly why a transition is being rejected: - KanbanBoard.handleDragEnd: log itemId, fromState, targetState, whether a handler is wired, and the rollback path on failure. - /kanban handleStateChange: log the PATCH and capture the response body when the API rejects the call. - /api/devops/tickets/[id]/state route: log incoming payload, success branch, and the DevOps error; return 400 with the underlying error message instead of a generic 500 so the client sees the real reason. - AzureDevOpsService.updateTicketState: read the DevOps error body (TF237124 etc.) and include it in the thrown error. Diagnostic only. No behavioural change to successful flows. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Copilot

Pull request overview

Adds investigative logging across the Kanban state-change drag-and-drop flow (component → page handler → API route → DevOps service) to diagnose issue #391, where dragging a card from "New" to "To Do" silently fails. Also captures and propagates the underlying Azure DevOps error body instead of dropping it, and changes the API route to return 400 with the real message instead of a generic 500.

Changes:

Add structured [Kanban DnD] / [Kanban] / [state PATCH] / [devops.updateTicketState] console logs at every hop of the state-change flow.
Capture the DevOps error response body in AzureDevOpsService.updateTicketState and embed it in the thrown error.
Wrap updateTicketState calls in route.ts with try/catch, returning 400 with the real error message.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
src/lib/devops.ts	Logs PATCH request, reads error body and includes it in the thrown error.
src/components/tickets/KanbanBoard.tsx	Adds drag-end/handler/success/failure logs around `onTicketStateChange`.
src/app/kanban/page.tsx	Logs the outgoing PATCH and reads/logs the response body on failure.
src/app/api/devops/tickets/[id]/state/route.ts	Adds entry/success/failure logs and returns 400 with the upstream error message.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -42,33 +65,50 @@ export async function PATCH(request: NextRequest, { params }: { params: Promise<
      try {
        const workItem = await devopsService.getWorkItem(project.name, ticketId);
        if (workItem) {
-          const updatedWorkItem = await devopsService.updateTicketState(
-            project.name,
-            ticketId,
-            state
-          );
+          try {
+            const updatedWorkItem = await devopsService.updateTicketState(
+              project.name,
+              ticketId,
+              state
+            );
+            console.log('[state PATCH] success (fallback project lookup)', {
+              ticketId,
+              state,
+              project: project.name,
+            });

-          const ticket = workItemToTicket(updatedWorkItem, {
-            id: project.id,
-            name: project.name,
-            devOpsProject: project.name,
-            devOpsOrg: organization || '',
-            tags: [],
-            createdAt: new Date(),
-            updatedAt: new Date(),
-          });
+            const ticket = workItemToTicket(updatedWorkItem, {
+              id: project.id,
+              name: project.name,
+              devOpsProject: project.name,
+              devOpsOrg: organization || '',
+              tags: [],
+              createdAt: new Date(),
+              updatedAt: new Date(),
+            });

-          return NextResponse.json({ ticket });
+            return NextResponse.json({ ticket });
+          } catch (err) {
+            console.error('[state PATCH] DevOps rejected update (fallback)', {
+              ticketId,
+              state,
+              project: project.name,
+              error: err instanceof Error ? err.message : err,
+            });
+            const message = err instanceof Error ? err.message : 'Failed to update ticket state';
+            return NextResponse.json({ error: message }, { status: 400 });
+          }


+        body: errorBody,
+      });
+      throw new Error(
+        `Failed to update work item ${workItemId} state to "${state}": ${response.status} ${response.statusText} — ${errorBody}`


@@ -245,14 +260,30 @@ export default function KanbanBoard({
      if (onTicketStateChange) {
        setIsUpdating(true);
        try {
+          console.log('[Kanban DnD] calling onTicketStateChange', {
+            itemId: activeItemId,
+            fromState,
+            targetState,
+          });
          await onTicketStateChange(activeItemId, targetState);
+          console.log('[Kanban DnD] state change succeeded', {
+            itemId: activeItemId,
+            targetState,
+          });
        } catch (error) {
-          console.error('Failed to update item state:', error);
+          console.error('[Kanban DnD] state change failed — rolling back', {
+            itemId: activeItemId,
+            fromState,
+            targetState,
+            error,
+          });
          // Rollback on failure
          setLocalItems(sourceItems);
        } finally {
          setIsUpdating(false);
        }
+      } else {
+        console.warn('[Kanban DnD] no onTicketStateChange handler wired — change will not persist');


+      const errorBody = await response.text().catch(() => '<no body>');
+      console.error('[devops.updateTicketState] DevOps rejected PATCH', {
+        projectName,
+        workItemId,
+        state,
+        status: response.status,
+        statusText: response.statusText,
+        body: errorBody,
+      });
+      throw new Error(
+        `Failed to update work item ${workItemId} state to "${state}": ${response.status} ${response.statusText} — ${errorBody}`


Copilot AI review requested due to automatic review settings May 15, 2026 16:13

BenGWeeks requested a review from akash2017sky as a code owner May 15, 2026 16:13

Copilot started reviewing on behalf of BenGWeeks May 15, 2026 16:14 View session

Copilot AI reviewed May 15, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

debug: log Kanban state-change DnD flow end-to-end (#391)#392

debug: log Kanban state-change DnD flow end-to-end (#391)#392
BenGWeeks wants to merge 1 commit into
mainfrom
debug/391-kanban-drag-logging

BenGWeeks commented May 15, 2026 •

edited by akash2017sky

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

BenGWeeks commented May 15, 2026 • edited by akash2017sky Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

How to use

Test plan

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

BenGWeeks commented May 15, 2026 •

edited by akash2017sky

Loading