From 8ac6800d47dde9201fb808daa49a86547812aa4c Mon Sep 17 00:00:00 2001
From: John Mizerek <mizerek@gmail.com>
Date: Mon, 9 Mar 2026 19:37:42 -0700
Subject: [PATCH] Use Jackson parser via file for Claude JSON response

Lucee's deserializeJSON fails on certain Claude outputs and the error
bypasses inner cftry/cfcatch. Parse via Jackson from file instead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 api/setup/analyzeMenuUrl.cfm | 37 ++++++++++++++++++------------------
 1 file changed, 19 insertions(+), 18 deletions(-)

diff --git a/api/setup/analyzeMenuUrl.cfm b/api/setup/analyzeMenuUrl.cfm
index f569f9a..63fb53e 100644
--- a/api/setup/analyzeMenuUrl.cfm
+++ b/api/setup/analyzeMenuUrl.cfm
@@ -2240,30 +2240,31 @@
     <cfset responseText = replace(responseText, chr(8212), "-", 'all')>
     <cfset responseText = replace(responseText, chr(8230), "...", 'all')>
 
-    <!--- Try to parse JSON with error handling --->
+    <!--- Write raw response to temp file, then parse via Java to avoid Lucee deserializeJSON issues --->
+    <cfset tempJsonFile = expandPath("/uploads/temp_claude_response.json")>
+    <cfset fileWrite(tempJsonFile, responseText, "utf-8")>
     <cftry>
-        <cfset menuData = deserializeJSON(responseText)>
+        <cfset objectMapper = createObject("java", "com.fasterxml.jackson.databind.ObjectMapper")>
+        <cfset javaFile = createObject("java", "java.io.File").init(tempJsonFile)>
+        <cfset javaMap = objectMapper.readValue(javaFile, createObject("java", "java.util.LinkedHashMap").getClass())>
+        <cfset menuData = deserializeJSON(objectMapper.writeValueAsString(javaMap))>
         <cfcatch type="any">
-            <!--- Lucee failed - try Jackson (bundled with Lucee) which is more lenient --->
+            <!--- Jackson also failed — try Lucee as fallback --->
             <cftry>
-                <cfset objectMapper = createObject("java", "com.fasterxml.jackson.databind.ObjectMapper")>
-                <cfset javaMap = objectMapper.readValue(responseText, createObject("java", "java.util.LinkedHashMap").getClass())>
-                <cfset menuData = deserializeJSON(objectMapper.writeValueAsString(javaMap))>
-            <cfcatch type="any">
-                <!--- Both parsers failed — save raw response to file for debugging --->
-                <cfset debugFile = expandPath("/uploads/debug_claude_response.json")>
-                <cfset fileWrite(debugFile, responseText)>
-                <cfset response["success"] = false>
-                <cfset response["error"] = "JSON parse error - raw response saved to debug file">
-                <cfset response["DEBUG_RAW_RESPONSE"] = left(responseText, 3000)>
-                <cfset response["DEBUG_RESPONSE_LENGTH"] = len(responseText)>
-                <cfcontent type="application/json" reset="true">
-                <cfoutput>#serializeJSON(response)#</cfoutput>
-                <cfabort>
-            </cfcatch>
+                <cfset menuData = deserializeJSON(responseText)>
+                <cfcatch type="any">
+                    <cfset response["OK"] = false>
+                    <cfset response["MESSAGE"] = "JSON parse error">
+                    <cfset response["DEBUG_RAW_RESPONSE"] = left(responseText, 3000)>
+                    <cfcontent type="application/json" reset="true">
+                    <cfoutput>#serializeJSON(response)#</cfoutput>
+                    <cfabort>
+                </cfcatch>
             </cftry>
         </cfcatch>
     </cftry>
+    <!--- Clean up temp file --->
+    <cftry><cfset fileDelete(tempJsonFile)><cfcatch></cfcatch></cftry>
 
     <!--- Debug: save raw Claude response before processing --->
     <cfset response["DEBUG_RAW_CLAUDE"] = responseText>