Merge pull request #649 from nipunayf/increase-type-param-support

Improve support for type parameter recognition
ballerina-platform · Mar 5, 2025 · 54ff184 · 54ff184
2 parents 11f2e55 + 70c078c
commit 54ff184
Show file tree

Hide file tree

Showing 28 changed files with 868 additions and 493 deletions.
diff --git a/...nerator/modules/flow-model-generator-ls-extension/src/main/resources/central-index.sqlite b/...nerator/modules/flow-model-generator-ls-extension/src/main/resources/central-index.sqlite
diff --git a/...or-ls-extension/src/test/resources/node_template/config/function_call-io-fileReadCsv.json b/...or-ls-extension/src/test/resources/node_template/config/function_call-io-fileReadCsv.json
@@ -0,0 +1,135 @@
+{
+  "source": "data_mapper/main.bal",
+  "position": {
+    "line": 14,
+    "offset": 0
+  },
+  "description": "Sample diagram node",
+  "codedata": {
+    "node": "FUNCTION_CALL",
+    "org": "ballerina",
+    "module": "io",
+    "symbol": "fileReadCsv",
+    "version": "1.6.1"
+  },
+  "output": {
+    "id": "31",
+    "metadata": {
+      "label": "fileReadCsv",
+      "description": "Read file content as a CSV.\nWhen the expected data type is record[], the first entry of the csv file should contain matching headers.\n```ballerina\nstring[][]|io:Error content = io:fileReadCsv(\"./resources/myfile.csv\");\nrecord{}[]|io:Error content = io:fileReadCsv(\"./resources/myfile.csv\");\n```",
+      "icon": "https://bcentral-packageicons.azureedge.net/images/ballerina_io_1.6.1.png"
+    },
+    "codedata": {
+      "node": "FUNCTION_CALL",
+      "org": "ballerina",
+      "module": "io",
+      "symbol": "fileReadCsv",
+      "version": "1.6.1",
+      "id": 341,
+      "inferredReturnType": "returnType[]"
+    },
+    "returning": false,
+    "properties": {
+      "returnType": {
+        "metadata": {
+          "label": "returnType",
+          "description": "The type of the return value (string[] or a Ballerina record)"
+        },
+        "valueType": "TYPE",
+        "valueTypeConstraint": "string[]|map<anydata>",
+        "placeholder": "string[]|map<anydata>",
+        "optional": false,
+        "editable": true,
+        "advanced": false,
+        "codedata": {
+          "kind": "PARAM_FOR_TYPE_INFER",
+          "originalName": "returnType"
+        }
+      },
+      "path": {
+        "metadata": {
+          "label": "path",
+          "description": "The CSV file path"
+        },
+        "valueType": "EXPRESSION",
+        "valueTypeConstraint": "string",
+        "placeholder": "\"\"",
+        "optional": false,
+        "editable": true,
+        "advanced": false,
+        "codedata": {
+          "kind": "REQUIRED",
+          "originalName": "path"
+        },
+        "typeMembers": [
+          {
+            "type": "string",
+            "packageInfo": "",
+            "kind": "BASIC_TYPE",
+            "selected": false
+          }
+        ]
+      },
+      "skipHeaders": {
+        "metadata": {
+          "label": "skipHeaders",
+          "description": "Number of headers, which should be skipped prior to reading records"
+        },
+        "valueType": "EXPRESSION",
+        "valueTypeConstraint": "int",
+        "placeholder": "0",
+        "optional": true,
+        "editable": true,
+        "advanced": true,
+        "codedata": {
+          "kind": "DEFAULTABLE",
+          "originalName": "skipHeaders"
+        },
+        "typeMembers": [
+          {
+            "type": "int",
+            "packageInfo": "",
+            "kind": "BASIC_TYPE",
+            "selected": false
+          }
+        ]
+      },
+      "type": {
+        "metadata": {
+          "label": "Variable Type",
+          "description": "Type of the variable"
+        },
+        "valueType": "TYPE",
+        "value": "returnType[]",
+        "placeholder": "var",
+        "optional": false,
+        "editable": false,
+        "advanced": false,
+        "codedata": {}
+      },
+      "variable": {
+        "metadata": {
+          "label": "Variable Name",
+          "description": "Name of the variable"
+        },
+        "valueType": "IDENTIFIER",
+        "value": "returnType",
+        "optional": false,
+        "editable": true,
+        "advanced": false
+      },
+      "checkError": {
+        "metadata": {
+          "label": "Check Error",
+          "description": "Trigger error flow"
+        },
+        "valueType": "FLAG",
+        "value": true,
+        "optional": false,
+        "editable": true,
+        "advanced": true
+      }
+    },
+    "flags": 0
+  }
+}
diff --git a/...erator-ls-extension/src/test/resources/node_template/config/function_call-io-println.json b/...erator-ls-extension/src/test/resources/node_template/config/function_call-io-println.json
@@ -25,7 +25,7 @@
       "module": "io",
       "symbol": "println",
       "version": "1.6.1",
-      "id": 351
+      "id": 361
     },
     "returning": false,
     "properties": {

diff --git a/...-ls-extension/src/test/resources/node_template/config/function_call-json-parseString.json b/...-ls-extension/src/test/resources/node_template/config/function_call-json-parseString.json
@@ -25,7 +25,7 @@
       "module": "data.jsondata",
       "symbol": "parseString",
       "version": "0.2.0",
-      "id": 597,
+      "id": 618,
       "inferredReturnType": "t"
     },
     "returning": false,

diff --git a/...rator-ls-extension/src/test/resources/node_template/config/function_call-json-toJson.json b/...rator-ls-extension/src/test/resources/node_template/config/function_call-json-toJson.json
@@ -25,7 +25,7 @@
       "module": "data.jsondata",
       "symbol": "toJson",
       "version": "0.2.0",
-      "id": 600
+      "id": 621
     },
     "returning": false,
     "properties": {

diff --git a/...tor-ls-extension/src/test/resources/node_template/config/function_call-log-printInfo.json b/...tor-ls-extension/src/test/resources/node_template/config/function_call-log-printInfo.json
@@ -25,7 +25,7 @@
       "module": "log",
       "symbol": "printInfo",
       "version": "2.10.0",
-      "id": 278
+      "id": 284
     },
     "returning": false,
     "properties": {

diff --git a/...ls-extension/src/test/resources/node_template/config/function_call-time-utcFromCivil.json b/...ls-extension/src/test/resources/node_template/config/function_call-time-utcFromCivil.json
@@ -25,7 +25,7 @@
       "module": "time",
       "symbol": "utcFromCivil",
       "version": "2.5.0",
-      "id": 574
+      "id": 553
     },
     "returning": false,
     "properties": {

diff --git a/...nerator-ls-extension/src/test/resources/node_template/config/method_call_redis_close.json b/...nerator-ls-extension/src/test/resources/node_template/config/method_call_redis_close.json
@@ -28,7 +28,7 @@
       "object": "Client",
       "symbol": "close",
       "version": "3.0.2",
-      "id": 780
+      "id": 930
     },
     "returning": false,
     "properties": {

diff --git a/...-generator-ls-extension/src/test/resources/node_template/config/new_connection-covid.json b/...-generator-ls-extension/src/test/resources/node_template/config/new_connection-covid.json
@@ -26,7 +26,7 @@
       "object": "Client",
       "symbol": "init",
       "version": "1.5.1",
-      "id": 2900
+      "id": 2645
     },
     "returning": false,
     "properties": {