fix: add types to lists, and fix doc blocks

Author: christyjacob4

example.php

@@ -220,6 +220,7 @@ function getSSLPage($url) {
 
     $sdk
         ->setName('NAME')
+        ->setVersion('7.2.0')
         ->setDescription('Repo description goes here')
         ->setShortDescription('Repo short description goes here')
         ->setURL('https://example.com')

src/SDK/Language/Python.php

@@ -231,16 +231,26 @@ public function getTypeName(array $parameter, array $spec = []): string
         if (!empty($parameter['enumValues'])) {
             return \ucfirst($parameter['name']);
         }
-        return match ($parameter['type'] ?? '') {
-            self::TYPE_FILE => 'InputFile',
-            self::TYPE_NUMBER,
-            self::TYPE_INTEGER => 'float',
-            self::TYPE_BOOLEAN => 'bool',
-            self::TYPE_STRING => 'str',
-            self::TYPE_ARRAY => 'list',
-            self::TYPE_OBJECT => 'dict',
-            default => $parameter['type'],
-        };
+        switch ($parameter['type'] ?? '') {
+            case self::TYPE_FILE:
+                return 'InputFile';
+            case self::TYPE_NUMBER:
+            case self::TYPE_INTEGER:
+                return 'float';
+            case self::TYPE_BOOLEAN:
+                return 'bool';
+            case self::TYPE_STRING:
+                return 'str';
+            case self::TYPE_ARRAY:
+                if (!empty(($parameter['array'] ?? [])['type']) && !\is_array($parameter['array']['type'])) {
+                    return 'list[' . $this->getTypeName($parameter['array']) . ']';
+                }
+                return 'list[str]';
+            case self::TYPE_OBJECT:
+                return 'dict';
+            default:
+                return $parameter['type'];
+        }
     }
 
     /**

templates/python/package/client.py.twig

@@ -13,7 +13,7 @@ class Client:
         self._endpoint = '{{spec.endpoint}}'
         self._global_headers = {
             'content-type': '',
-            'user-agent' : '{{spec.title | caseUcfirst}}{{ language.name | caseUcfirst }}SDK/{{ sdk.version }} (${os.uname().sysname}; ${os.uname().version}; ${os.uname().machine})',
+            'user-agent' : f'{{spec.title | caseUcfirst}}{{ language.name | caseUcfirst }}SDK/{{ sdk.version }} ({os.uname().sysname}; {os.uname().version}; {os.uname().machine})',
             'x-sdk-name': '{{ sdk.name }}',
             'x-sdk-platform': '{{ sdk.platform }}',
             'x-sdk-language': '{{ language.name | caseLower }}',

templates/python/package/services/service.py.twig

@@ -28,6 +28,7 @@ class {{ service.name | caseUcfirst }}(Service):
 {% for method in service.methods %}
 
     def {{ method.name | caseSnake }}(self{% if method.parameters.all|length > 0 %}, {% endif %}{% for parameter in method.parameters.all %}{{ parameter.name | escapeKeyword | caseSnake }}: {{ parameter | getPropertyType(method) | raw }}{% if not parameter.required %} = None{% endif %}{% if not loop.last %}, {% endif %}{% endfor %}{% if 'multipart/form-data' in method.consumes %}, on_progress = None{% endif %}):
+        """{{method.title}}"""
 
         api_path = '{{ method.path }}'
 {{ include('python/base/params.twig') }}