Cover task moves with tests and verify the required parameter #246

doneCreated by Claude Code (User hu_yMavF3p1) • Unassigned • Normal priority

Description

Test coverage for #243. Depends on #244 and #245.

Existing calls to adjust

task_update gains a required parameter, so these break: HttpTests.cs:313, :381, :387, :403. Pass the current parent explicitly rather than null, so the tests document the echo-it-back contract instead of silently unparenting.

New coverage

  • move a top-level task under a parent; move a subtask to top level with null; move a subtask from one parent to another
  • reject: self-parent, a descendant as parent (the cycle case), a closed parent, a parent from another project
  • task_patch with only parentTaskNumber is a real change, not a no-op, and bumps the version
  • a stale version still fails on a move
  • the response of both tools carries the new parentTaskNumber, not the old one
  • a caller without project write access cannot move a task

Schema check

Extend the discovery test so task_update lists parentTaskNumber in its inputSchema.required — that property is the whole point of the design and nothing else pins it.

Connection interrupted or session ended. Reload

Rejoining the server...

Rejoin failed... trying again in seconds.

Failed to rejoin.
Please retry or reload the page.

The session has been paused by the server.

Failed to resume the session.
Please retry or reload the page.