✨ enhance: add context logging and clarify math expression

microsoft · Oct 18, 2024 · 8bc9baf · 8bc9baf
1 parent 2d4e1ab
commit 8bc9baf
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/docs/src/content/docs/reference/scripts/system.mdx b/docs/src/content/docs/reference/scripts/system.mdx
@@ -1918,14 +1918,17 @@ defTool(
         properties: {
             expression: {
                 type: "string",
-                description: "Math expression to evaluate using mathjs format.",
+                description:
+                    "Math expression to evaluate using mathjs format. Use ^ for power operator.",
             },
         },
         required: ["expression"],
     },
     async (args) => {
-        const { expression } = args
-        return "" + ((await parsers.math(expression)) ?? "?")
+        const { context, expression } = args
+        const res = String((await parsers.math(expression)) ?? "?")
+        context.log(`math: ${expression} => ${res}`)
+        return res
     }
 )
 

diff --git a/packages/core/src/genaisrc/system.math.genai.js b/packages/core/src/genaisrc/system.math.genai.js
@@ -11,13 +11,16 @@ defTool(
         properties: {
             expression: {
                 type: "string",
-                description: "Math expression to evaluate using mathjs format.",
+                description:
+                    "Math expression to evaluate using mathjs format. Use ^ for power operator.",
             },
         },
         required: ["expression"],
     },
     async (args) => {
-        const { expression } = args
-        return "" + ((await parsers.math(expression)) ?? "?")
+        const { context, expression } = args
+        const res = String((await parsers.math(expression)) ?? "?")
+        context.log(`math: ${expression} => ${res}`)
+        return res
     }
 )