From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id BD66F1FF191 for ; Tue, 21 Oct 2025 15:50:28 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 5A57B1FC05; Tue, 21 Oct 2025 15:50:54 +0200 (CEST) From: Hannes Laimer To: pdm-devel@lists.proxmox.com Date: Tue, 21 Oct 2025 15:50:15 +0200 Message-ID: <20251021135018.88877-2-h.laimer@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20251021135018.88877-1-h.laimer@proxmox.com> References: <20251021135018.88877-1-h.laimer@proxmox.com> MIME-Version: 1.0 X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1761054614080 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.043 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: [pdm-devel] [PATCH proxmox 1/3] pve-api-types: schema2rust: generate arrays for types with format `-list` X-BeenThere: pdm-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox Datacenter Manager development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox Datacenter Manager development discussion Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pdm-devel-bounces@lists.proxmox.com Sender: "pdm-devel" Since [1] the PVE API does take in actual arrays for parameters with a format that ends in `-list`. With this we generate `Vec` for those, return types are not affected, so they will still just generate a `String`. [1] pve-common 69d9edcc ("section config: implement array support") Signed-off-by: Hannes Laimer --- pve-api-types/generator-lib/Schema2Rust.pm | 35 ++++++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-) diff --git a/pve-api-types/generator-lib/Schema2Rust.pm b/pve-api-types/generator-lib/Schema2Rust.pm index c5ac4aad..73ca4150 100644 --- a/pve-api-types/generator-lib/Schema2Rust.pm +++ b/pve-api-types/generator-lib/Schema2Rust.pm @@ -39,6 +39,7 @@ my $dedup_enum = {}; my $dedup_array_types = {}; our $__err_path = ''; +our $__list_format_as_array = undef; my sub to_doc_comment : prototype($); my sub strip_doc_comment : prototype($); @@ -504,11 +505,12 @@ my sub print_method_without_body : prototype($$$$$) { my $arg = $fields->{$name}; my $rust_name = $arg->{rust_name}; - if ($arg->{type} eq 'Option') { print {$out} " .maybe_bool_arg(\"$name\", p_$rust_name)\n"; } elsif ($arg->{is_string_list}) { print {$out} " .maybe_list_arg(\"$name\", p_$rust_name)\n"; + } elsif ($arg->{type} =~ /^Option>$/) { + print {$out} " .maybe_list_arg(\"$name\", &p_$rust_name)\n"; } elsif ($arg->{optional}) { print {$out} " .maybe_arg(\"$name\", &p_$rust_name)\n"; } else { @@ -534,7 +536,7 @@ my sub print_method_without_body : prototype($$$$$) { if ($arg->{type} eq 'Option') { print {$out} " .maybe_bool_arg(\"$name\", $rust_name)\n"; - } elsif ($arg->{is_string_list}) { + } elsif ($arg->{is_string_list} || $arg->{type} =~ /^Option>$/) { print {$out} " .maybe_list_arg(\"$name\", &$rust_name)\n"; } elsif ($arg->{optional}) { print {$out} " .maybe_arg(\"$name\", &$rust_name)\n"; @@ -809,6 +811,22 @@ my sub type_of : prototype($) { my ($schema) = @_; my $kind = $schema->{type}; + + # If type is 'string' but format ends with `-list`, treat as array + # but only for inputs to endpoints. + # + # We want a typed Vec, and since [1] the PVE API does accept + # actual arrays for parameters with a format ending in `-list`. + # This is only for inputs though, so we can only have Vec's for + # inputs, returns are still string lists. + # + # [1] pve-common 69d9edcc ("section config: implement array support") + if ($kind eq 'string' && defined($schema->{format}) && $schema->{format} =~ /-list$/) { + if (defined($__list_format_as_array) && $__list_format_as_array) { + return 'array'; + } + } + return $kind if $kind; if (exists $schema->{properties}) { @@ -1132,6 +1150,15 @@ my sub array_type : prototype($$$) { optional => undef, description => '', }; + if (!$schema->{items} && $schema->{format} =~ /-list$/) { + my $format_name = $schema->{format}; + $format_name =~ s/-list$//; + $schema->{items} = { + description => "List item of type $format_name.", + format => $format_name, + type => 'string', + }; + } my $items = delete $schema->{items} or die "missing 'items' in array schema\n"; my $description = $items->{description}; @@ -1578,6 +1605,8 @@ my sub url_parameters : prototype($) { my sub method_parameters : prototype($$$$$) { my ($def, $api_url, $param_name, $api_method, $rust_method_name) = @_; + local $__list_format_as_array = 1; + my $url_params = url_parameters($api_url); my $parameters = $api_method->{parameters} // {}; @@ -1676,6 +1705,8 @@ my sub method_parameters : prototype($$$$$) { my sub method_return_type : prototype($$$$) { my ($def, $method, $return_name, $extra) = @_; + local $__list_format_as_array = 0; + if (defined(my $returns = $extra->{'output-type'})) { $def->{output_type} = $returns; return; -- 2.47.3 _______________________________________________ pdm-devel mailing list pdm-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel