cleanup: use nested CLIHandler command definitions
[pve-client.git] / pveclient
index 46d902d..70db84a 100755 (executable)
--- a/pveclient
+++ b/pveclient
@@ -1,5 +1,7 @@
 #!/usr/bin/perl
 
+package PVE::CLI::pveclient;
+
 use strict;
 use warnings;
 use Cwd 'abs_path';
@@ -7,88 +9,35 @@ use lib '/usr/share/pve-client';
 use lib '.';
 use Data::Dumper;
 
-use PVE::JSONSchema;
+use PVE::JSONSchema qw(register_standard_option get_standard_option);
 use PVE::CLIHandler;
 
 use PVE::APIClient::LWP;
 use PVE::APIClient::Helpers;
+use PVE::APIClient::Commands::config;
 use PVE::APIClient::Commands::remote;
+use PVE::APIClient::Commands::list;
 use PVE::APIClient::Commands::lxc;
 use PVE::APIClient::Commands::help;
 
 use JSON;
 
 sub call_method {
-    my ($path, $method, $args) = @_;
+    my ($remote, $path, $method, $params) = @_;
 
     die "missing API path\n" if !defined($path);
 
     my $info = PVE::APIClient::Helpers::lookup_api_method($path, $method);
-    my $param = PVE::JSONSchema::get_options($info->{parameters}, $args);
-    print Dumper($param);
+    print Dumper($params);
 
     die "implement me";
 }
 
-my $cli_class_handlers = {
-    lxc => 'PVE::APIClient::Commands::lxc',
-    remote => 'PVE::APIClient::Commands::remote',
-    help => 'PVE::APIClient::Commands::help',
-};
-
-my $cmd = shift;
-if (!defined($cmd)) {
-    PVE::APIClient::Commands::help->help({});
-    exit(-1);
-}
-
-if ($cmd eq 'get') {
-    my $method = 'GET';
-    my $path;
-    if (scalar(@ARGV) && $ARGV[0] !~ m/^\-/)  {
-       $path = shift @ARGV;
-    }
-    my $res = call_method($path, $method, \@ARGV);
-    die "implement me";
-} elsif ($cmd eq 'set') {
-    die "implement me";
-} elsif ($cmd eq 'create') {
-    die "implement me";
-} elsif ($cmd eq 'delete') {
-    die "implement me";
-} elsif (my $class = $cli_class_handlers->{$cmd}) {
-    $class->run_cli_handler();
-} elsif ($cmd eq 'bashcomplete') {
-
-    exit(0) if !(defined($ENV{COMP_LINE}) && defined($ENV{COMP_POINT}));
-
-    my $cmdlist = join('|', keys %$cli_class_handlers);
-    if ($ENV{COMP_LINE} =~ m/^(.*pveclient\s+($cmdlist)\s+)(.*)$/) {
-       my $cmd = $2;
-       my $class = $cli_class_handlers->{$cmd} || die "internal error";
-       $ENV{COMP_LINE} = "pveclient $3";
-       $ENV{COMP_POINT} = length($ENV{COMP_LINE});
-       @ARGV = ('bashcomplete', 'pveclient', $ARGV[1], $ARGV[2]);
-
-       $class->run_cli_handler();
-
-    } else {
+use base qw(PVE::CLIHandler);
 
-       my $cmdline = substr($ENV{COMP_LINE}, 0, $ENV{COMP_POINT});
-       my ($bash_command, $cur, $prev) = @ARGV;
-       $cmdline =~ s/$cur$//;
+my $cmd = $ARGV[0];
 
-       my $args = PVE::Tools::split_args($cmdline);
-
-       my @cmds = ('get', 'set', 'create', 'delete', keys %$cli_class_handlers);
-       if (scalar(@$args) == 1) {
-           foreach my $p (@cmds) {
-               print "$p\n" if $p =~ m/^$cur/;
-           }
-       }
-    }
-
-} elsif ($cmd eq 'packagedepends') {
+if ($cmd && $cmd eq 'packagedepends') {
     # experimental code to print required perl packages
     my $packages = {};
     my $dir = Cwd::getcwd;
@@ -106,8 +55,120 @@ if ($cmd eq 'get') {
        }
     }
     print join("\n", sort(keys %$packages)) . "\n";
-} else {
-    PVE::APIClient::Commands::help->help({});
+
+    exit(0);
+}
+
+my $path_properties = {};
+my $path_returns = { type => 'null' };
+
+# dynamically update schema definition for direct API call
+# like: pveclient api <get|set|create|delete|help> <remote> <path>
+if (my $info =  PVE::APIClient::Helpers::extract_path_info()) {
+    $path_properties = $info->{parameters}->{properties};
+    $path_returns = $info->{returns};
 }
 
-exit(0);
+$path_properties->{remote} = get_standard_option('pveclient-remote-name');
+$path_properties->{api_path} = {
+    description => "API path.",
+    type => 'string',
+    completion => sub {
+       my ($cmd, $pname, $cur, $args) = @_;
+       return PVE::APIClient::Helpers::complete_api_path($cur);
+    },
+};
+
+__PACKAGE__->register_method ({
+    name => 'pveclient_get',
+    path => 'pveclient_get',
+    method => 'GET',
+    description => "call API GET on <path>.",
+    parameters => {
+       additionalProperties => 0,
+       properties => $path_properties,
+    },
+    returns => $path_returns,
+    code => sub {
+       my ($param) = @_;
+
+       my $path = PVE::Tools::extract_param($param, 'api_path');
+       my $remote = PVE::Tools::extract_param($param, 'remote');
+
+       return call_method($remote, $path, 'GET', $param);
+    }});
+
+__PACKAGE__->register_method ({
+    name => 'pveclient_set',
+    path => 'pveclient_set',
+    method => 'PUT',
+    description => "call API PUT on <path>.",
+    parameters => {
+       additionalProperties => 0,
+       properties => $path_properties,
+    },
+    returns => $path_returns,
+    code => sub {
+       my ($param) = @_;
+
+       print Dumper($param);
+
+       die "implement me";
+
+    }});
+
+__PACKAGE__->register_method ({
+    name => 'pveclient_create',
+    path => 'pveclient_create',
+    method => 'PUSH',
+    description => "call API PUSH on <path>.",
+    parameters => {
+       additionalProperties => 0,
+       properties => $path_properties,
+    },
+    returns => $path_returns,
+    code => sub {
+       my ($param) = @_;
+
+       print Dumper($param);
+
+       die "implement me";
+
+    }});
+
+__PACKAGE__->register_method ({
+    name => 'pveclient_delete',
+    path => 'pveclient_delete',
+    method => 'DELETE',
+    description => "call API DELETE on <path>.",
+    parameters => {
+       additionalProperties => 0,
+       properties => $path_properties,
+    },
+    returns => $path_returns,
+    code => sub {
+       my ($param) = @_;
+
+       print Dumper($param);
+
+       die "implement me";
+
+    }});
+
+
+our $cmddef = {
+    config => $PVE::APIClient::Commands::config::cmddef,
+    list => $PVE::APIClient::Commands::list::cmddef,
+    lxc => $PVE::APIClient::Commands::lxc::cmddef,
+    remote => $PVE::APIClient::Commands::remote::cmddef,
+
+    api => {
+       get => [ __PACKAGE__, 'pveclient_get', ['remote', 'api_path']],
+       set => [ __PACKAGE__, 'pveclient_set', ['remote', 'api_path']],
+       create => [ __PACKAGE__, 'pveclient_create', ['remote', 'api_path']],
+       delete => [ __PACKAGE__, 'pveclient_delete', ['remote', 'api_path']],
+    }
+};
+
+
+__PACKAGE__->run_cli_handler();