@bojackduy/opencode-learn 0.1.6 → 1.0.0

package/dist/server.js

@@ -147,6 +147,18 @@ function resolveCorrect(correctAnswer, options) {
   }
   return { indices: Array.from(new Set(indices)).sort((a, b) => a - b) };
 }
+function decodeQuizText(s) {
+  if (!s || typeof s !== "string")
+    return s;
+  if (!s.includes("\\"))
+    return s;
+  let out = s.replace(/\\r\\n/g, `
+`).replace(/\\n/g, `
+`).replace(/\\r/g, "\r").replace(/\\t/g, "\t");
+  out = out.replace(/\\"/g, '"').replace(/\\'/g, "'");
+  out = out.replace(/\\\\/g, "\\");
+  return out;
+}
 var mdLogFile = null;
 var mdLogWriteLock = Promise.resolve();
 function withMdLock(fn) {
@@ -891,9 +903,13 @@ Explanation: ${j.explanation}${note}`;
           shuffle: tool.schema.boolean().optional().describe("Default true: shuffle before display. False only if order matters.")
         },
         async execute(args, ctx) {
+          const qFixed = decodeQuizText(args.question) ?? args.question;
+          const dFixed = decodeQuizText(args.details);
+          const eFixed = decodeQuizText(args.explanation) ?? args.explanation;
+          const optsDecoded = args.options?.map((o) => ({ ...o, label: decodeQuizText(o.label) ?? o.label, description: o.description ? decodeQuizText(o.description) : o.description }));
           let options;
           try {
-            options = normalizeQuizOptions(args.options);
+            options = normalizeQuizOptions(optsDecoded);
           } catch (e) {
             return `quiz error: ${e.message}`;
           }
@@ -917,11 +933,11 @@ Explanation: ${j.explanation}${note}`;
           const payload = {
             id,
             type: "quiz",
-            question: args.question,
-            details: args.details,
+            question: qFixed,
+            details: dFixed,
             options: options.map((o, i) => ({ label: o.label, value: o.value, description: o.description, index: i + 1 })),
             correctIndices,
-            explanation: args.explanation,
+            explanation: eFixed,
             multiSelect: !!args.multiSelect,
             sessionID: ctx.sessionID,
             timestamp: Date.now()
@@ -933,7 +949,7 @@ Explanation: ${j.explanation}${note}`;
             slog("quiz write failed", String(e));
           }
           try {
-            await ctx.metadata?.({ title: `Quiz: ${args.question.slice(0, 40)}`, metadata: { pendingId: id } });
+            await ctx.metadata?.({ title: `Quiz: ${qFixed.slice(0, 40)}`, metadata: { pendingId: id } });
           } catch {}
           watchAndInject(client, directory, id, ctx.sessionID, (r) => {
             const dk = !!r?.dontKnow;
@@ -949,21 +965,21 @@ Note: ${r.note}` : "";
                 answers: r?.answers || [],
                 correct: ok,
                 correctIndices,
-                explanation: args.explanation,
+                explanation: eFixed,
                 dontKnow: dk,
                 note: r?.note
               };
               withMdLock(() => appendToMdLog(answerCalloutQuiz(details)));
             }
-            return dk ? `[quiz answered] "${args.question}" -> I don't know (genuine gap).
+            return dk ? `[quiz answered] "${qFixed}" -> I don't know (genuine gap).
 Correct: ${correctStr}
-Explanation: ${args.explanation}${note}` : `[quiz answered] "${args.question}" -> ${sel} = ${ok ? "CORRECT" : "INCORRECT"}.
+Explanation: ${eFixed}${note}` : `[quiz answered] "${qFixed}" -> ${sel} = ${ok ? "CORRECT" : "INCORRECT"}.
 Correct: ${correctStr}
-Explanation: ${args.explanation}${note}`;
+Explanation: ${eFixed}${note}`;
           });
           if (mdLogFile) {
             try {
-              await withMdLock(() => appendToMdLog(questionCallout("Quiz", args.question, args.details?.trim() || undefined, options.map((o) => ({ label: o.label })))));
+              await withMdLock(() => appendToMdLog(questionCallout("Quiz", qFixed, dFixed?.trim() || undefined, options.map((o) => ({ label: o.label })))));
             } catch {}
           }
           if (tuiAlive) {
@@ -998,9 +1014,9 @@ ${args.multiSelect ? "Select all correct (comma-separated numbers, e.g. 1,3) or
             if (trimmed === "0" || trimmed.toLowerCase() === "i don't know") {
               const msg = `User selected "I don't know" \u2014 genuine gap, not a guess.
 Correct: ${correctStr}
-Explanation: ${args.explanation}`;
+Explanation: ${eFixed}`;
               if (mdLogFile)
-                await withMdLock(() => appendToMdLog(callout("question", "Quiz \u2014 I don't know", [args.question, trimmed, `Correct: ${correctStr}`, args.explanation])));
+                await withMdLock(() => appendToMdLog(callout("question", "Quiz \u2014 I don't know", [qFixed, trimmed, `Correct: ${correctStr}`, eFixed])));
               return msg;
             }
             const nums = trimmed.split(/[,\s]+/).map((s) => parseInt(s, 10)).filter((n) => !isNaN(n) && n >= 1 && n <= options.length);
@@ -1012,30 +1028,30 @@ Explanation: ${args.explanation}`;
             const result = `User answered ${verdict}.
 Selected: ${selectedStr}
 Correct: ${correctStr}
-Explanation: ${args.explanation}`;
-            ctx.metadata?.({ title: correct ? "Quiz \u2014 correct \u2713" : "Quiz \u2014 incorrect \u2717", metadata: { correct, correctIndices, explanation: args.explanation } });
+Explanation: ${eFixed}`;
+            ctx.metadata?.({ title: correct ? "Quiz \u2014 correct \u2713" : "Quiz \u2014 incorrect \u2717", metadata: { correct, correctIndices, explanation: eFixed } });
             if (mdLogFile)
-              await withMdLock(() => appendToMdLog(callout(correct ? "success" : "failure", correct ? "Quiz \u2014 correct \u2713" : "Quiz \u2014 incorrect \u2717", [`Q: ${args.question}`, `Selected: ${selectedStr}`, `Correct: ${correctStr}`, args.explanation])));
+              await withMdLock(() => appendToMdLog(callout(correct ? "success" : "failure", correct ? "Quiz \u2014 correct \u2713" : "Quiz \u2014 incorrect \u2717", [`Q: ${qFixed}`, `Selected: ${selectedStr}`, `Correct: ${correctStr}`, eFixed])));
             return result;
           }
           const instruction = [
             `[quiz ready \u2014 awaiting user answer via \`question\` tool]`,
-            `Question: ${args.question}`,
-            args.details ? `Details: ${args.details}` : null,
+            `Question: ${qFixed}`,
+            dFixed ? `Details: ${dFixed}` : null,
             `Options (display order, already shuffled):`,
             ...options.map((o, i) => `${i + 1}. ${o.label}${o.description ? ` \u2014 ${o.description}` : ""} (value="${o.value}")`),
             `Correct indices: ${correctIndices.join(", ")} (Correct values: ${correctStr})`,
-            `Explanation (reveal AFTER answer): ${args.explanation}`,
+            `Explanation (reveal AFTER answer): ${eFixed}`,
             `Mode: ${args.multiSelect ? "multi-select (exact set)" : "single-select"}`,
             ``,
             `INSTRUCTION FOR LLM: Call the built-in \`question\` tool with:`,
             `  header: "Quiz"`,
-            `  question: "${args.question.replace(/"/g, "\\\"")}"`,
+            `  question: "${qFixed.replace(/"/g, "\\\"")}"`,
             `  options: [${options.map((o) => `{label:"${o.label.replace(/"/g, "\\\"")}", description:"${(o.description ?? "").replace(/"/g, "\\\"")}"}`).join(", ")}]`,
             `Then compare the user's selected labels to correct indices [${correctIndices.join(", ")}]. Grade as ${args.multiSelect ? "exact-set match" : "single match"}, show \u2713/\u2717, reveal Correct: ${correctStr}, and Explanation. An 'I don't know' maps to dontKnow (genuine gap).`
           ].filter(Boolean).join(`
 `);
-          ctx.metadata?.({ title: `Quiz: ${args.question.slice(0, 40)}`, metadata: { correctIndices, explanation: args.explanation, options: options.map((o, i) => ({ index: i + 1, label: o.label })) } });
+          ctx.metadata?.({ title: `Quiz: ${qFixed.slice(0, 40)}`, metadata: { correctIndices, explanation: eFixed, options: options.map((o, i) => ({ index: i + 1, label: o.label })) } });
           return instruction;
         }
       }),
@@ -1063,23 +1079,27 @@ Explanation: ${args.explanation}`;
           slog("quiz_batch isAlive", isAlive);
           const normalized = [];
           for (const q of args.quizzes) {
+            const qFixed = decodeQuizText(q.question) ?? q.question;
+            const dFixed = decodeQuizText(q.details);
+            const eFixed = decodeQuizText(q.explanation) ?? q.explanation;
+            const optsDecoded = q.options?.map((o) => ({ ...o, label: decodeQuizText(o.label) ?? o.label, description: o.description ? decodeQuizText(o.description) : o.description }));
             let opts;
             try {
-              opts = normalizeQuizOptions(q.options);
+              opts = normalizeQuizOptions(optsDecoded);
             } catch (e) {
               slog("quiz_batch normalize error", e.message);
-              return `quiz_batch error: ${e.message} in "${q.question}"`;
+              return `quiz_batch error: ${e.message} in "${qFixed}"`;
             }
             if (q.shuffle !== false)
               opts = shuffleOptions(opts);
             const { indices, error } = resolveCorrect(q.correctAnswer, opts);
             if (error) {
               slog("quiz_batch resolveCorrect error", error);
-              return `quiz_batch error: ${error} in "${q.question}"`;
+              return `quiz_batch error: ${error} in "${qFixed}"`;
             }
             if (opts.length < 2)
-              return `quiz_batch error: need 2+ options in "${q.question}"`;
-            normalized.push({ question: q.question, details: q.details, options: opts, correctIndices: indices, explanation: q.explanation, multiSelect: !!q.multiSelect });
+              return `quiz_batch error: need 2+ options in "${qFixed}"`;
+            normalized.push({ question: qFixed, details: dFixed, options: opts, correctIndices: indices, explanation: eFixed, multiSelect: !!q.multiSelect });
           }
           slog("quiz_batch normalized", normalized.length);
           try {