refactor: unify parameter extraction with shared helpers

Add extractItemBoolean to ItemFieldExtractors for handler-context
boolean extraction. Replace inline jsonObject["key"]?.jsonPrimitive
?.booleanOrNull patterns with optionalBoolean() across 5 tool files.
Replace inline offset extraction with optionalInt() in QueryItemsTool.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: jpicklyk

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/items/CreateItemHandler.kt

@@ -50,7 +50,7 @@ class CreateItemHandler(
                 val statusLabel = extractItemString(itemObj, "statusLabel")
                 val priorityStr = extractItemString(itemObj, "priority")
                 val complexity = extractItemInt(itemObj, "complexity")
-                val requiresVerification = itemObj["requiresVerification"]?.let { (it as? JsonPrimitive)?.booleanOrNull } ?: false
+                val requiresVerification = extractItemBoolean(itemObj, "requiresVerification") ?: false
                 val metadata = extractItemString(itemObj, "metadata")
                 val tags = extractItemString(itemObj, "tags")
 

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/items/ItemFieldExtractors.kt

@@ -35,6 +35,17 @@ internal fun extractItemStringAllowNull(obj: JsonObject, name: String, existing:
     return if (content.isBlank()) null else content
 }
 
+/**
+ * Extracts an integer field from a JsonObject item. Returns null if absent or not parseable.
+ */
+/**
+ * Extracts a boolean field from a JsonObject item. Returns null if absent or not parseable.
+ */
+internal fun extractItemBoolean(obj: JsonObject, name: String): Boolean? {
+    val value = obj[name] as? JsonPrimitive ?: return null
+    return value.booleanOrNull ?: value.content.toBooleanStrictOrNull()
+}
+
 /**
  * Extracts an integer field from a JsonObject item. Returns null if absent or not parseable.
  */

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/items/ManageItemsTool.kt

@@ -143,7 +143,7 @@ Unified write operations for WorkItems (create, update, delete).
             )
             "delete" -> deleteHandler.execute(
                 requireJsonArray(params, "ids"),
-                params.jsonObject["recursive"]?.jsonPrimitive?.booleanOrNull ?: false,
+                optionalBoolean(params, "recursive", false),
                 context
             )
             else -> errorResponse("Invalid operation: $operation", ErrorCodes.VALIDATION_ERROR)

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/items/QueryItemsTool.kt

@@ -237,7 +237,7 @@ Operations: get, search, overview
 
     private suspend fun executeGet(params: JsonElement, context: ToolExecutionContext): JsonElement {
         val idStr = requireString(params, "id")
-        val includeAncestors = params.jsonObject["includeAncestors"]?.jsonPrimitive?.booleanOrNull ?: false
+        val includeAncestors = optionalBoolean(params, "includeAncestors", false)
 
         // Try parsing as a full UUID first (fast path — avoids prefix resolution overhead)
         val item = if (idStr.length == 36) {
@@ -340,8 +340,8 @@ Operations: get, search, overview
         val sortBy = optionalString(params, "sortBy")
         val sortOrder = optionalString(params, "sortOrder")
         val limit = optionalInt(params, "limit") ?: 50
-        val offset = params.jsonObject["offset"]?.jsonPrimitive?.intOrNull?.coerceAtLeast(0) ?: 0
-        val includeAncestors = params.jsonObject["includeAncestors"]?.jsonPrimitive?.booleanOrNull ?: false
+        val offset = (optionalInt(params, "offset") ?: 0).coerceAtLeast(0)
+        val includeAncestors = optionalBoolean(params, "includeAncestors", false)
 
         // Parse role
         val role = roleStr?.let {

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/items/UpdateItemHandler.kt

@@ -77,7 +77,7 @@ class UpdateItemHandler(
                 val newStatusLabel = extractItemStringAllowNull(itemObj, "statusLabel", existing.statusLabel)
                 val newPriorityStr = extractItemString(itemObj, "priority")
                 val newComplexity = extractItemInt(itemObj, "complexity")
-                val newRequiresVerification = itemObj["requiresVerification"]?.let { (it as? JsonPrimitive)?.booleanOrNull }
+                val newRequiresVerification = extractItemBoolean(itemObj, "requiresVerification")
                 val newMetadata = extractItemStringAllowNull(itemObj, "metadata", existing.metadata)
                 val newTags = extractItemStringAllowNull(itemObj, "tags", existing.tags)
 

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/workflow/GetBlockedItemsTool.kt

@@ -101,7 +101,7 @@ Items in TERMINAL role are never included.
     override suspend fun execute(params: JsonElement, context: ToolExecutionContext): JsonElement {
         val parentId = extractUUID(params, "parentId", required = false)
         val includeDetails = optionalBoolean(params, "includeItemDetails", false)
-        val includeAncestors = params.jsonObject["includeAncestors"]?.jsonPrimitive?.booleanOrNull ?: false
+        val includeAncestors = optionalBoolean(params, "includeAncestors", false)
 
         val workItemRepo = context.workItemRepository()
         val depRepo = context.dependencyRepository()

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/workflow/GetContextTool.kt

@@ -91,7 +91,7 @@ Parameters:
     override suspend fun execute(params: JsonElement, context: ToolExecutionContext): JsonElement {
         val itemId = extractUUID(params, "itemId", required = false)
         val sinceInstant = parseInstant(params, "since")
-        val includeAncestors = params.jsonObject["includeAncestors"]?.jsonPrimitive?.booleanOrNull ?: false
+        val includeAncestors = optionalBoolean(params, "includeAncestors", false)
         val transitionLimit = params.jsonObject["limit"]?.jsonPrimitive?.intOrNull?.coerceIn(1, 200) ?: 50
 
         return when {

current/src/main/kotlin/io/github/jpicklyk/mcptask/current/application/tools/workflow/GetNextItemTool.kt

@@ -78,7 +78,7 @@ Parameters:
         val parentId = extractUUID(params, "parentId", required = false)
         val limit = optionalInt(params, "limit") ?: 1
         val includeDetails = optionalBoolean(params, "includeDetails", defaultValue = false)
-        val includeAncestors = params.jsonObject["includeAncestors"]?.jsonPrimitive?.booleanOrNull ?: false
+        val includeAncestors = optionalBoolean(params, "includeAncestors", false)
 
         val workItemRepo = context.workItemRepository()
         val dependencyRepo = context.dependencyRepository()