pulsemcp-cms-admin-mcp-server 0.7.1 → 0.7.3

package/build/shared/src/pulsemcp-admin-client/lib/save-results-for-mirror.js

@@ -7,7 +7,7 @@ export async function saveResultsForMirror(apiKey, baseUrl, params) {
             exam_id: r.exam_id,
             result: {
                 status: r.status,
-                ...(r.data && { data: r.data }),
+                ...(r.data || {}),
             },
         })),
     };

package/build/shared/src/tools/save-results-for-mirror.js

@@ -98,14 +98,27 @@ Typical workflow:
                     // The exam_id may live at the top level of the stream line OR inside
                     // line.data (the actual result payload). Prefer the data payload to
                     // avoid reading from potentially incomplete display metadata.
+                    //
+                    // The real proctor API returns line.data as a metadata wrapper:
+                    //   { mirror_id, exam_id, status, result: { status, output: {...} } }
+                    // The actual output lives inside line.data.result. When we pass the
+                    // entire line.data as `data`, the output ends up nested too deeply
+                    // (result.data.result.output) and the backend saves empty output.
+                    // Use line.data.result when present so that `output` is at the
+                    // expected depth (result.data.output).
                     results = stored.lines
                         .filter((line) => line.type === 'exam_result')
                         .map((line) => {
                         const data = line.data;
+                        // Prefer the nested result object (contains output, input, etc.)
+                        // over the full data wrapper (contains metadata like mirror_id)
+                        const resultData = data?.result && typeof data.result === 'object' && !Array.isArray(data.result)
+                            ? data.result
+                            : data;
                         return {
                             exam_id: extractExamId(line),
                             status: extractStatus(line),
-                            ...(data ? { data } : {}),
+                            ...(resultData ? { data: resultData } : {}),
                         };
                     });
                     if (!runtimeId) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pulsemcp-cms-admin-mcp-server",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "description": "Local implementation of PulseMCP CMS Admin MCP server",
   "mcpName": "com.pulsemcp.servers/pulsemcp-cms-admin",
   "main": "build/index.js",

package/shared/pulsemcp-admin-client/lib/save-results-for-mirror.js

@@ -7,7 +7,7 @@ export async function saveResultsForMirror(apiKey, baseUrl, params) {
             exam_id: r.exam_id,
             result: {
                 status: r.status,
-                ...(r.data && { data: r.data }),
+                ...(r.data || {}),
             },
         })),
     };

package/shared/tools/save-results-for-mirror.js

@@ -98,14 +98,27 @@ Typical workflow:
                     // The exam_id may live at the top level of the stream line OR inside
                     // line.data (the actual result payload). Prefer the data payload to
                     // avoid reading from potentially incomplete display metadata.
+                    //
+                    // The real proctor API returns line.data as a metadata wrapper:
+                    //   { mirror_id, exam_id, status, result: { status, output: {...} } }
+                    // The actual output lives inside line.data.result. When we pass the
+                    // entire line.data as `data`, the output ends up nested too deeply
+                    // (result.data.result.output) and the backend saves empty output.
+                    // Use line.data.result when present so that `output` is at the
+                    // expected depth (result.data.output).
                     results = stored.lines
                         .filter((line) => line.type === 'exam_result')
                         .map((line) => {
                         const data = line.data;
+                        // Prefer the nested result object (contains output, input, etc.)
+                        // over the full data wrapper (contains metadata like mirror_id)
+                        const resultData = data?.result && typeof data.result === 'object' && !Array.isArray(data.result)
+                            ? data.result
+                            : data;
                         return {
                             exam_id: extractExamId(line),
                             status: extractStatus(line),
-                            ...(data ? { data } : {}),
+                            ...(resultData ? { data: resultData } : {}),
                         };
                     });
                     if (!runtimeId) {