Changed the yet unused descriptor type CbFunction for callback functions.

Author: forman

chartlets.js/CHANGES.md

@@ -1,3 +1,7 @@
+## Version 0.0.x (in development)
+
+* Changed the yet unused descriptor type `CbFunction` for callback functions.
+
 ## Version 0.0.28 (from 2024/11/26)
 
 * Updated docs.

chartlets.js/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chartlets",
-  "version": "0.0.28",
+  "version": "0.0.29",
   "description": "An experimental library for integrating interactive charts into existing JavaScript applications.",
   "type": "module",
   "files": [

chartlets.js/src/lib/types/model/callback.ts

@@ -23,15 +23,19 @@ export interface JsonSchema {
 export interface CbFunction {
   name: string;
   parameters: CbParameter[];
-  returnType: JsonSchema;
+  return: CbReturn;
 }
 
 export interface CbParameter {
   name: string;
-  type?: JsonSchema;
+  schema?: JsonSchema;
   default?: unknown;
 }
 
+export interface CbReturn {
+  schema?: JsonSchema;
+}
+
 /**
  * A reference to a specific contribution.
  */

chartlets.py/CHANGES.md

@@ -2,6 +2,7 @@
 
 * Fixed a bug that prevent using annotations of type `dict` or `dict[str, T]`.
   in callback functions.
+* Changed schema of the yet unused descriptor for callback functions.
 
 
 ## Version 0.0.28 (from 2024/11/26)

chartlets.py/chartlets/callback.py

@@ -104,9 +104,7 @@ def to_dict(self) -> dict[str, Any]:
             "function": {
                 "name": self.function.__qualname__,
                 "parameters": [_parameter_to_dict(p) for p in parameters],
-                "returnType": _annotation_to_json_schema(
-                    self.signature.return_annotation
-                ),
+                "return": _return_to_dict(self.signature.return_annotation),
             }
         }
         if self.inputs:
@@ -152,11 +150,16 @@ def _parameter_to_dict(parameter: inspect.Parameter) -> dict[str, Any]:
     empty = inspect.Parameter.empty
     d = {"name": parameter.name}
     if parameter.annotation is not empty:
-        d |= {"type": _annotation_to_json_schema(parameter.annotation)}
+        d |= {"schema": _annotation_to_json_schema(parameter.annotation)}
     if parameter.default is not empty:
         d |= {"default": parameter.default}
     return d
 
+def _return_to_dict(return_annotation: Any) -> dict[str, Any]:
+    return {
+        "schema": _annotation_to_json_schema(return_annotation)
+    }
+
 
 _basic_types = {
     None: "null",

chartlets.py/chartlets/version.py

@@ -1 +1 @@
-version = "0.0.28"
+version = "0.0.29"

chartlets.py/tests/callback_test.py

@@ -15,9 +15,9 @@ def my_callback(
     b: str | int = "",
     c: bool | None = False,
     d: list[str] = (),
-    e: dict[str, Any] = (),
+    e: dict[str, Any] | None = None,
 ) -> str:
-    return f"{a}-{b}-{c}-{d}"
+    return f"{a}-{b}-{c}-{d}-{e}"
 
 
 def my_callback_2(ctx, n: int) -> tuple[list[str], str | None]:
@@ -45,29 +45,29 @@ def test_to_dict_with_no_outputs(self):
                 "function": {
                     "name": "my_callback",
                     "parameters": [
-                        {"name": "a", "type": {"type": "integer"}},
+                        {"name": "a", "schema": {"type": "integer"}},
                         {
-                            "default": "",
                             "name": "b",
-                            "type": {"type": ["string", "integer"]},
+                            "schema": {"type": ["string", "integer"]},
+                            "default": "",
                         },
                         {
-                            "default": False,
                             "name": "c",
-                            "type": {"type": ["boolean", "null"]},
+                            "schema": {"type": ["boolean", "null"]},
+                            "default": False,
                         },
                         {
-                            "default": (),
                             "name": "d",
-                            "type": {"items": {"type": "string"}, "type": "array"},
+                            "schema": {"items": {"type": "string"}, "type": "array"},
+                            "default": (),
                         },
                         {
-                            "default": (),
                             "name": "e",
-                            "type": {"additionalProperties": {}, "type": "object"},
+                            "schema": {'type': ['object', 'null']},
+                            "default": None,
                         },
                     ],
-                    "returnType": {"type": "string"},
+                    "return": {"schema": {"type": "string"}},
                 },
                 "inputs": [
                     {"id": "a", "property": "value"},
@@ -95,14 +95,14 @@ def test_to_dict_with_two_outputs(self):
             {
                 "function": {
                     "name": "my_callback_2",
-                    "parameters": [{"name": "n", "type": {"type": "integer"}}],
-                    "returnType": {
+                    "parameters": [{"name": "n", "schema": {"type": "integer"}}],
+                    "return": {"schema":{
                         "items": [
                             {"items": {"type": "string"}, "type": "array"},
                             {"type": ["string", "null"]},
                         ],
                         "type": "array",
-                    },
+                    }},
                 },
                 "inputs": [{"id": "n", "property": "value"}],
                 "outputs": [