Refactor postgres commands to follow ObjectVerb naming pattern, fix command hierarchy, and ensure all commands end with verbs (#866)

Copilot · jongio · web-flow · commit 43b7cfea6db1 · 2025-08-07T14:14:19.000Z
* Initial plan * Implement postgres server param set pattern - refactor from setparam Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Refactor postgres commands to follow ObjectVerb naming pattern Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Fix whitespace formatting in PostgresSetup.cs Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Fix command naming consistency - rename ParamSetCommand to ServerParamSetCommand Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Add 'get' verb to postgres server param command for consistency Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Ensure all postgres commands end with verbs (get, set, list, etc.) Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Fix whitespace formatting issues in PostgresSetup.cs Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Update CHANGELOG.md with postgres command refactoring entry for issue #865 Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> * Add PR link to CHANGELOG.md entry for postgres command refactoring Co-authored-by: jongio <2163001+jongio@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: jongio <2163001+jongio@users.noreply.github.com>
diff --git a/.vscode/cspell.json b/.vscode/cspell.json
@@ -300,6 +300,7 @@
         "servicebus",
         "sessionhost",
         "setparam",
+        "setpermission",
         "skillset",
         "staticwebapp",
         "syslib",
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -51,6 +51,7 @@ The Azure MCP Server updates automatically by default whenever a new release com
 ### Other Changes
 
 - Added caching for Cosmos DB databases and containers. [[813](https://github.com/Azure/azure-mcp/pull/813)]
+- Refactored PostgreSQL commands to follow ObjectVerb naming pattern, fix command hierarchy, and ensure all commands end with verbs. This improves consistency and discoverability across all postgres commands. [[#865](https://github.com/Azure/azure-mcp/issues/865)] [[#866](https://github.com/Azure/azure-mcp/pull/866)]
 
 #### Dependency Updates
 
diff --git a/areas/postgres/src/AzureMcp.Postgres/Commands/PostgresJsonContext.cs b/areas/postgres/src/AzureMcp.Postgres/Commands/PostgresJsonContext.cs
@@ -10,11 +10,11 @@ namespace AzureMcp.Postgres.Commands;
 
 [JsonSerializable(typeof(DatabaseListCommand.DatabaseListCommandResult))]
 [JsonSerializable(typeof(DatabaseQueryCommand.DatabaseQueryCommandResult))]
-[JsonSerializable(typeof(GetConfigCommand.GetConfigCommandResult))]
-[JsonSerializable(typeof(GetParamCommand.GetParamCommandResult))]
-[JsonSerializable(typeof(SetParamCommand.SetParamCommandResult))]
+[JsonSerializable(typeof(ServerConfigGetCommand.ServerConfigGetCommandResult))]
+[JsonSerializable(typeof(ServerParamGetCommand.ServerParamGetCommandResult))]
+[JsonSerializable(typeof(ServerParamSetCommand.ServerParamSetCommandResult))]
 [JsonSerializable(typeof(ServerListCommand.ServerListCommandResult))]
-[JsonSerializable(typeof(GetSchemaCommand.GetSchemaCommandResult))]
+[JsonSerializable(typeof(TableSchemaGetCommand.TableSchemaGetCommandResult))]
 [JsonSerializable(typeof(TableListCommand.TableListCommandResult))]
 
 internal sealed partial class PostgresJsonContext : JsonSerializerContext
diff --git a/areas/postgres/src/AzureMcp.Postgres/Commands/Server/ServerConfigGetCommand.cs b/areas/postgres/src/AzureMcp.Postgres/Commands/Server/ServerConfigGetCommand.cs
@@ -10,7 +10,7 @@
 
 namespace AzureMcp.Postgres.Commands.Server;
 
-public sealed class GetConfigCommand(ILogger<GetConfigCommand> logger) : BaseServerCommand<GetConfigOptions>(logger)
+public sealed class ServerConfigGetCommand(ILogger<ServerConfigGetCommand> logger) : BaseServerCommand<ServerConfigGetOptions>(logger)
 {
     private const string CommandTitle = "Get PostgreSQL Server Configuration";
     public override string Name => "config";
@@ -37,8 +37,8 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
             var config = await pgService.GetServerConfigAsync(options.Subscription!, options.ResourceGroup!, options.User!, options.Server!);
             context.Response.Results = config?.Length > 0 ?
                 ResponseResult.Create(
-                    new GetConfigCommandResult(config),
-                    PostgresJsonContext.Default.GetConfigCommandResult) :
+                    new ServerConfigGetCommandResult(config),
+                    PostgresJsonContext.Default.ServerConfigGetCommandResult) :
                 null;
         }
         catch (Exception ex)
@@ -48,5 +48,5 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
         }
         return context.Response;
     }
-    internal record GetConfigCommandResult(string Configuration);
+    internal record ServerConfigGetCommandResult(string Configuration);
 }
diff --git a/areas/postgres/src/AzureMcp.Postgres/Commands/Server/ServerParamGetCommand.cs b/areas/postgres/src/AzureMcp.Postgres/Commands/Server/ServerParamGetCommand.cs
@@ -11,7 +11,7 @@
 
 namespace AzureMcp.Postgres.Commands.Server;
 
-public sealed class GetParamCommand(ILogger<GetParamCommand> logger) : BaseServerCommand<GetParamOptions>(logger)
+public sealed class ServerParamGetCommand(ILogger<ServerParamGetCommand> logger) : BaseServerCommand<ServerParamGetOptions>(logger)
 {
     private const string CommandTitle = "Get PostgreSQL Server Parameter";
     private readonly Option<string> _paramOption = PostgresOptionDefinitions.Param;
@@ -30,7 +30,7 @@ protected override void RegisterOptions(Command command)
         command.AddOption(_paramOption);
     }
 
-    protected override GetParamOptions BindOptions(ParseResult parseResult)
+    protected override ServerParamGetOptions BindOptions(ParseResult parseResult)
     {
         var options = base.BindOptions(parseResult);
         options.Param = parseResult.GetValueForOption(_paramOption);
@@ -54,8 +54,8 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
             var parameterValue = await pgService.GetServerParameterAsync(options.Subscription!, options.ResourceGroup!, options.User!, options.Server!, options.Param!);
             context.Response.Results = parameterValue?.Length > 0 ?
                 ResponseResult.Create(
-                    new GetParamCommandResult(parameterValue),
-                    PostgresJsonContext.Default.GetParamCommandResult) :
+                    new ServerParamGetCommandResult(parameterValue),
+                    PostgresJsonContext.Default.ServerParamGetCommandResult) :
                 null;
         }
         catch (Exception ex)
@@ -66,5 +66,5 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
         return context.Response;
     }
 
-    internal record GetParamCommandResult(string ParameterValue);
+    internal record ServerParamGetCommandResult(string ParameterValue);
 }
diff --git a/areas/postgres/src/AzureMcp.Postgres/Commands/Server/ServerParamSetCommand.cs b/areas/postgres/src/AzureMcp.Postgres/Commands/Server/ServerParamSetCommand.cs
@@ -11,12 +11,12 @@
 
 namespace AzureMcp.Postgres.Commands.Server;
 
-public sealed class SetParamCommand(ILogger<SetParamCommand> logger) : BaseServerCommand<SetParamOptions>(logger)
+public sealed class ServerParamSetCommand(ILogger<ServerParamSetCommand> logger) : BaseServerCommand<ServerParamSetOptions>(logger)
 {
     private const string CommandTitle = "Set PostgreSQL Server Parameter";
     private readonly Option<string> _paramOption = PostgresOptionDefinitions.Param;
     private readonly Option<string> _valueOption = PostgresOptionDefinitions.Value;
-    public override string Name => "setparam";
+    public override string Name => "set";
 
     public override string Description =>
         "Sets a specific parameter of a PostgreSQL server to a certain value.";
@@ -32,7 +32,7 @@ protected override void RegisterOptions(Command command)
         command.AddOption(_valueOption);
     }
 
-    protected override SetParamOptions BindOptions(ParseResult parseResult)
+    protected override ServerParamSetOptions BindOptions(ParseResult parseResult)
     {
         var options = base.BindOptions(parseResult);
         options.Param = parseResult.GetValueForOption(_paramOption);
@@ -57,8 +57,8 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
             var result = await pgService.SetServerParameterAsync(options.Subscription!, options.ResourceGroup!, options.User!, options.Server!, options.Param!, options.Value!);
             context.Response.Results = !string.IsNullOrEmpty(result) ?
                 ResponseResult.Create(
-                    new SetParamCommandResult(result, options.Param!, options.Value!),
-                    PostgresJsonContext.Default.SetParamCommandResult) :
+                    new ServerParamSetCommandResult(result, options.Param!, options.Value!),
+                    PostgresJsonContext.Default.ServerParamSetCommandResult) :
                 null;
         }
         catch (Exception ex)
@@ -69,5 +69,5 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
         return context.Response;
     }
 
-    internal record SetParamCommandResult(string Message, string Parameter, string Value);
+    internal record ServerParamSetCommandResult(string Message, string Parameter, string Value);
 }
diff --git a/areas/postgres/src/AzureMcp.Postgres/Commands/Table/TableSchemaGetCommand.cs b/areas/postgres/src/AzureMcp.Postgres/Commands/Table/TableSchemaGetCommand.cs
@@ -11,7 +11,7 @@
 
 namespace AzureMcp.Postgres.Commands.Table;
 
-public sealed class GetSchemaCommand(ILogger<GetSchemaCommand> logger) : BaseDatabaseCommand<GetSchemaOptions>(logger)
+public sealed class TableSchemaGetCommand(ILogger<TableSchemaGetCommand> logger) : BaseDatabaseCommand<TableSchemaGetOptions>(logger)
 {
     private const string CommandTitle = "Get PostgreSQL Table Schema";
     private readonly Option<string> _tableOption = PostgresOptionDefinitions.Table;
@@ -28,7 +28,7 @@ protected override void RegisterOptions(Command command)
         command.AddOption(_tableOption);
     }
 
-    protected override GetSchemaOptions BindOptions(ParseResult parseResult)
+    protected override TableSchemaGetOptions BindOptions(ParseResult parseResult)
     {
         var options = base.BindOptions(parseResult);
         options.Table = parseResult.GetValueForOption(_tableOption);
@@ -52,8 +52,8 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
             List<string> schema = await pgService.GetTableSchemaAsync(options.Subscription!, options.ResourceGroup!, options.User!, options.Server!, options.Database!, options.Table!);
             context.Response.Results = schema?.Count > 0 ?
                 ResponseResult.Create(
-                    new GetSchemaCommandResult(schema),
-                    PostgresJsonContext.Default.GetSchemaCommandResult) :
+                    new TableSchemaGetCommandResult(schema),
+                    PostgresJsonContext.Default.TableSchemaGetCommandResult) :
                 null;
         }
         catch (Exception ex)
@@ -65,5 +65,5 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,
         return context.Response;
     }
 
-    internal record GetSchemaCommandResult(List<string> Schema);
+    internal record TableSchemaGetCommandResult(List<string> Schema);
 }
diff --git a/areas/postgres/src/AzureMcp.Postgres/Options/Server/ServerConfigGetOptions.cs b/areas/postgres/src/AzureMcp.Postgres/Options/Server/ServerConfigGetOptions.cs
@@ -3,4 +3,4 @@
 
 namespace AzureMcp.Postgres.Options.Server;
 
-public class GetConfigOptions : BasePostgresOptions;
+public class ServerConfigGetOptions : BasePostgresOptions;
diff --git a/areas/postgres/src/AzureMcp.Postgres/Options/Server/ServerParamGetOptions.cs b/areas/postgres/src/AzureMcp.Postgres/Options/Server/ServerParamGetOptions.cs
@@ -5,7 +5,7 @@
 
 namespace AzureMcp.Postgres.Options.Server;
 
-public class GetParamOptions : BasePostgresOptions
+public class ServerParamGetOptions : BasePostgresOptions
 {
     [JsonPropertyName(PostgresOptionDefinitions.ParamName)]
     public string? Param { get; set; }
diff --git a/areas/postgres/src/AzureMcp.Postgres/Options/Server/ServerParamSetOptions.cs b/areas/postgres/src/AzureMcp.Postgres/Options/Server/ServerParamSetOptions.cs
@@ -5,7 +5,7 @@
 
 namespace AzureMcp.Postgres.Options.Server;
 
-public class SetParamOptions : BasePostgresOptions
+public class ServerParamSetOptions : BasePostgresOptions
 {
     [JsonPropertyName(PostgresOptionDefinitions.ParamName)]
     public string? Param { get; set; }
diff --git a/areas/postgres/src/AzureMcp.Postgres/Options/Table/TableSchemaGetOptions.cs b/areas/postgres/src/AzureMcp.Postgres/Options/Table/TableSchemaGetOptions.cs
@@ -5,7 +5,7 @@
 
 namespace AzureMcp.Postgres.Options.Table;
 
-public class GetSchemaOptions : BasePostgresOptions
+public class TableSchemaGetOptions : BasePostgresOptions
 {
     [JsonPropertyName(PostgresOptionDefinitions.TableName)]
     public string? Table { get; set; }
diff --git a/areas/postgres/src/AzureMcp.Postgres/PostgresSetup.cs b/areas/postgres/src/AzureMcp.Postgres/PostgresSetup.cs
@@ -32,13 +32,22 @@ public void RegisterCommands(CommandGroup rootGroup, ILoggerFactory loggerFactor
         var table = new CommandGroup("table", "PostgreSQL table operations");
         pg.AddSubGroup(table);
         table.AddCommand("list", new TableListCommand(loggerFactory.CreateLogger<TableListCommand>()));
-        table.AddCommand("schema", new GetSchemaCommand(loggerFactory.CreateLogger<GetSchemaCommand>()));
+
+        var schema = new CommandGroup("schema", "PostgreSQL table schema operations");
+        table.AddSubGroup(schema);
+        schema.AddCommand("get", new TableSchemaGetCommand(loggerFactory.CreateLogger<TableSchemaGetCommand>()));
 
         var server = new CommandGroup("server", "PostgreSQL server operations");
         pg.AddSubGroup(server);
         server.AddCommand("list", new ServerListCommand(loggerFactory.CreateLogger<ServerListCommand>()));
-        server.AddCommand("config", new GetConfigCommand(loggerFactory.CreateLogger<GetConfigCommand>()));
-        server.AddCommand("param", new GetParamCommand(loggerFactory.CreateLogger<GetParamCommand>()));
-        server.AddCommand("setparam", new SetParamCommand(loggerFactory.CreateLogger<SetParamCommand>()));
+
+        var config = new CommandGroup("config", "PostgreSQL server configuration operations");
+        server.AddSubGroup(config);
+        config.AddCommand("get", new ServerConfigGetCommand(loggerFactory.CreateLogger<ServerConfigGetCommand>()));
+
+        var param = new CommandGroup("param", "PostgreSQL server parameter operations");
+        server.AddSubGroup(param);
+        param.AddCommand("get", new ServerParamGetCommand(loggerFactory.CreateLogger<ServerParamGetCommand>()));
+        param.AddCommand("set", new ServerParamSetCommand(loggerFactory.CreateLogger<ServerParamSetCommand>()));
     }
 }
diff --git a/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Server/ServerConfigGetCommandTests.cs b/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Server/ServerConfigGetCommandTests.cs
@@ -16,16 +16,16 @@
 
 namespace AzureMcp.Postgres.UnitTests.Server;
 
-public class GetConfigCommandTests
+public class ServerConfigGetCommandTests
 {
     private readonly IServiceProvider _serviceProvider;
     private readonly IPostgresService _postgresService;
-    private readonly ILogger<GetConfigCommand> _logger;
+    private readonly ILogger<ServerConfigGetCommand> _logger;
 
-    public GetConfigCommandTests()
+    public ServerConfigGetCommandTests()
     {
         _postgresService = Substitute.For<IPostgresService>();
-        _logger = Substitute.For<ILogger<GetConfigCommand>>();
+        _logger = Substitute.For<ILogger<ServerConfigGetCommand>>();
 
         var collection = new ServiceCollection();
         collection.AddSingleton(_postgresService);
@@ -39,7 +39,7 @@ public async Task ExecuteAsync_ReturnsConfig_WhenConfigExists()
         var expectedConfig = "config123";
         _postgresService.GetServerConfigAsync("sub123", "rg1", "user1", "server123").Returns(expectedConfig);
 
-        var command = new GetConfigCommand(_logger);
+        var command = new ServerConfigGetCommand(_logger);
         var args = command.GetCommand().Parse(["--subscription", "sub123", "--resource-group", "rg1", "--user", "user1", "--server", "server123"]);
         var context = new CommandContext(_serviceProvider);
 
@@ -60,7 +60,7 @@ public async Task ExecuteAsync_ReturnsNull_WhenConfigDoesNotExist()
     {
         _postgresService.GetServerConfigAsync("sub123", "rg1", "user1", "server123").Returns("");
 
-        var command = new GetConfigCommand(_logger);
+        var command = new ServerConfigGetCommand(_logger);
         var args = command.GetCommand().Parse(["--subscription", "sub123", "--resource-group", "rg1", "--user", "user1", "--server", "server123"]);
         var context = new CommandContext(_serviceProvider);
         var response = await command.ExecuteAsync(context, args);
@@ -78,7 +78,7 @@ public async Task ExecuteAsync_ReturnsNull_WhenConfigDoesNotExist()
     [InlineData("--server")]
     public async Task ExecuteAsync_ReturnsError_WhenParameterIsMissing(string missingParameter)
     {
-        var command = new GetConfigCommand(_logger);
+        var command = new ServerConfigGetCommand(_logger);
         var args = command.GetCommand().Parse(new string[]
         {
             missingParameter == "--subscription" ? "" : "--subscription", "sub123",
diff --git a/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Server/ServerParamGetCommandTests.cs b/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Server/ServerParamGetCommandTests.cs
@@ -16,16 +16,16 @@
 
 namespace AzureMcp.Postgres.UnitTests.Server;
 
-public class GetParamCommandTests
+public class ServerParamGetCommandTests
 {
     private readonly IServiceProvider _serviceProvider;
     private readonly IPostgresService _postgresService;
-    private readonly ILogger<GetParamCommand> _logger;
+    private readonly ILogger<ServerParamGetCommand> _logger;
 
-    public GetParamCommandTests()
+    public ServerParamGetCommandTests()
     {
         _postgresService = Substitute.For<IPostgresService>();
-        _logger = Substitute.For<ILogger<GetParamCommand>>();
+        _logger = Substitute.For<ILogger<ServerParamGetCommand>>();
 
         var collection = new ServiceCollection();
         collection.AddSingleton(_postgresService);
@@ -39,7 +39,7 @@ public async Task ExecuteAsync_ReturnsParamValue_WhenParamExists()
         var expectedValue = "value123";
         _postgresService.GetServerParameterAsync("sub123", "rg1", "user1", "server123", "param123").Returns(expectedValue);
 
-        var command = new GetParamCommand(_logger);
+        var command = new ServerParamGetCommand(_logger);
         var args = command.GetCommand().Parse(["--subscription", "sub123", "--resource-group", "rg1", "--user", "user1", "--server", "server123", "--param", "param123"]);
         var context = new CommandContext(_serviceProvider);
         var response = await command.ExecuteAsync(context, args);
@@ -60,7 +60,7 @@ public async Task ExecuteAsync_ReturnsParamValue_WhenParamExists()
     public async Task ExecuteAsync_ReturnsNull_WhenParamDoesNotExist()
     {
         _postgresService.GetServerParameterAsync("sub123", "rg1", "user1", "server123", "param123").Returns("");
-        var command = new GetParamCommand(_logger);
+        var command = new ServerParamGetCommand(_logger);
         var args = command.GetCommand().Parse(["--subscription", "sub123", "--resource-group", "rg1", "--user", "user1", "--server", "server123", "--param", "param123"]);
         var context = new CommandContext(_serviceProvider);
         var response = await command.ExecuteAsync(context, args);
@@ -79,7 +79,7 @@ public async Task ExecuteAsync_ReturnsNull_WhenParamDoesNotExist()
     [InlineData("--param")]
     public async Task ExecuteAsync_ReturnsError_WhenParameterIsMissing(string missingParameter)
     {
-        var command = new GetParamCommand(_logger);
+        var command = new ServerParamGetCommand(_logger);
         var args = command.GetCommand().Parse(new string[]
         {
             missingParameter == "--subscription" ? "" : "--subscription", "sub123",
diff --git a/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Server/ServerParamSetCommandTests.cs b/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Server/ServerParamSetCommandTests.cs
diff --git a/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Table/TableSchemaGetCommandTests.cs b/areas/postgres/tests/AzureMcp.Postgres.UnitTests/Table/TableSchemaGetCommandTests.cs
diff --git a/docs/azmcp-commands.md b/docs/azmcp-commands.md
diff --git a/docs/new-command.md b/docs/new-command.md
diff --git a/e2eTests/e2eTestPrompts.md b/e2eTests/e2eTestPrompts.md

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`
`11`	`11`	`namespace AzureMcp.Postgres.Commands.Server;`
`12`	`12`
`13`		`-public sealed class GetConfigCommand(ILogger<GetConfigCommand> logger) : BaseServerCommand<GetConfigOptions>(logger)`
	`13`	`+public sealed class ServerConfigGetCommand(ILogger<ServerConfigGetCommand> logger) : BaseServerCommand<ServerConfigGetOptions>(logger)`
`14`	`14`	`{`
`15`	`15`	`private const string CommandTitle = "Get PostgreSQL Server Configuration";`
`16`	`16`	`public override string Name => "config";`
`@@ -37,8 +37,8 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,`
`37`	`37`	`var config = await pgService.GetServerConfigAsync(options.Subscription!, options.ResourceGroup!, options.User!, options.Server!);`
`38`	`38`	`context.Response.Results = config?.Length > 0 ?`
`39`	`39`	`ResponseResult.Create(`
`40`		`- new GetConfigCommandResult(config),`
`41`		`- PostgresJsonContext.Default.GetConfigCommandResult) :`
	`40`	`+ new ServerConfigGetCommandResult(config),`
	`41`	`+ PostgresJsonContext.Default.ServerConfigGetCommandResult) :`
`42`	`42`	`null;`
`43`	`43`	`}`
`44`	`44`	`catch (Exception ex)`
`@@ -48,5 +48,5 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,`
`48`	`48`	`}`
`49`	`49`	`return context.Response;`
`50`	`50`	`}`
`51`		`- internal record GetConfigCommandResult(string Configuration);`
	`51`	`+ internal record ServerConfigGetCommandResult(string Configuration);`
`52`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`
`12`	`12`	`namespace AzureMcp.Postgres.Commands.Server;`
`13`	`13`
`14`		`-public sealed class GetParamCommand(ILogger<GetParamCommand> logger) : BaseServerCommand<GetParamOptions>(logger)`
	`14`	`+public sealed class ServerParamGetCommand(ILogger<ServerParamGetCommand> logger) : BaseServerCommand<ServerParamGetOptions>(logger)`
`15`	`15`	`{`
`16`	`16`	`private const string CommandTitle = "Get PostgreSQL Server Parameter";`
`17`	`17`	`private readonly Option<string> _paramOption = PostgresOptionDefinitions.Param;`
`@@ -30,7 +30,7 @@ protected override void RegisterOptions(Command command)`
`30`	`30`	`command.AddOption(_paramOption);`
`31`	`31`	`}`
`32`	`32`
`33`		`- protected override GetParamOptions BindOptions(ParseResult parseResult)`
	`33`	`+ protected override ServerParamGetOptions BindOptions(ParseResult parseResult)`
`34`	`34`	`{`
`35`	`35`	`var options = base.BindOptions(parseResult);`
`36`	`36`	`options.Param = parseResult.GetValueForOption(_paramOption);`
`@@ -54,8 +54,8 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,`
`54`	`54`	`var parameterValue = await pgService.GetServerParameterAsync(options.Subscription!, options.ResourceGroup!, options.User!, options.Server!, options.Param!);`
`55`	`55`	`context.Response.Results = parameterValue?.Length > 0 ?`
`56`	`56`	`ResponseResult.Create(`
`57`		`- new GetParamCommandResult(parameterValue),`
`58`		`- PostgresJsonContext.Default.GetParamCommandResult) :`
	`57`	`+ new ServerParamGetCommandResult(parameterValue),`
	`58`	`+ PostgresJsonContext.Default.ServerParamGetCommandResult) :`
`59`	`59`	`null;`
`60`	`60`	`}`
`61`	`61`	`catch (Exception ex)`
`@@ -66,5 +66,5 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,`
`66`	`66`	`return context.Response;`
`67`	`67`	`}`
`68`	`68`
`69`		`- internal record GetParamCommandResult(string ParameterValue);`
	`69`	`+ internal record ServerParamGetCommandResult(string ParameterValue);`
`70`	`70`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,12 +11,12 @@`
`11`	`11`
`12`	`12`	`namespace AzureMcp.Postgres.Commands.Server;`
`13`	`13`
`14`		`-public sealed class SetParamCommand(ILogger<SetParamCommand> logger) : BaseServerCommand<SetParamOptions>(logger)`
	`14`	`+public sealed class ServerParamSetCommand(ILogger<ServerParamSetCommand> logger) : BaseServerCommand<ServerParamSetOptions>(logger)`
`15`	`15`	`{`
`16`	`16`	`private const string CommandTitle = "Set PostgreSQL Server Parameter";`
`17`	`17`	`private readonly Option<string> _paramOption = PostgresOptionDefinitions.Param;`
`18`	`18`	`private readonly Option<string> _valueOption = PostgresOptionDefinitions.Value;`
`19`		`- public override string Name => "setparam";`
	`19`	`+ public override string Name => "set";`
`20`	`20`
`21`	`21`	`public override string Description =>`
`22`	`22`	`"Sets a specific parameter of a PostgreSQL server to a certain value.";`
`@@ -32,7 +32,7 @@ protected override void RegisterOptions(Command command)`
`32`	`32`	`command.AddOption(_valueOption);`
`33`	`33`	`}`
`34`	`34`
`35`		`- protected override SetParamOptions BindOptions(ParseResult parseResult)`
	`35`	`+ protected override ServerParamSetOptions BindOptions(ParseResult parseResult)`
`36`	`36`	`{`
`37`	`37`	`var options = base.BindOptions(parseResult);`
`38`	`38`	`options.Param = parseResult.GetValueForOption(_paramOption);`
`@@ -57,8 +57,8 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,`
`57`	`57`	`var result = await pgService.SetServerParameterAsync(options.Subscription!, options.ResourceGroup!, options.User!, options.Server!, options.Param!, options.Value!);`
`58`	`58`	`context.Response.Results = !string.IsNullOrEmpty(result) ?`
`59`	`59`	`ResponseResult.Create(`
`60`		`- new SetParamCommandResult(result, options.Param!, options.Value!),`
`61`		`- PostgresJsonContext.Default.SetParamCommandResult) :`
	`60`	`+ new ServerParamSetCommandResult(result, options.Param!, options.Value!),`
	`61`	`+ PostgresJsonContext.Default.ServerParamSetCommandResult) :`
`62`	`62`	`null;`
`63`	`63`	`}`
`64`	`64`	`catch (Exception ex)`
`@@ -69,5 +69,5 @@ public override async Task<CommandResponse> ExecuteAsync(CommandContext context,`
`69`	`69`	`return context.Response;`
`70`	`70`	`}`
`71`	`71`
`72`		`- internal record SetParamCommandResult(string Message, string Parameter, string Value);`
	`72`	`+ internal record ServerParamSetCommandResult(string Message, string Parameter, string Value);`
`73`	`73`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,4 +3,4 @@`
`3`	`3`
`4`	`4`	`namespace AzureMcp.Postgres.Options.Server;`
`5`	`5`
`6`		`-public class GetConfigOptions : BasePostgresOptions;`
	`6`	`+public class ServerConfigGetOptions : BasePostgresOptions;`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`
`6`	`6`	`namespace AzureMcp.Postgres.Options.Server;`
`7`	`7`
`8`		`-public class GetParamOptions : BasePostgresOptions`
	`8`	`+public class ServerParamGetOptions : BasePostgresOptions`
`9`	`9`	`{`
`10`	`10`	`[JsonPropertyName(PostgresOptionDefinitions.ParamName)]`
`11`	`11`	`public string? Param { get; set; }`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`
`6`	`6`	`namespace AzureMcp.Postgres.Options.Server;`
`7`	`7`
`8`		`-public class SetParamOptions : BasePostgresOptions`
	`8`	`+public class ServerParamSetOptions : BasePostgresOptions`
`9`	`9`	`{`
`10`	`10`	`[JsonPropertyName(PostgresOptionDefinitions.ParamName)]`
`11`	`11`	`public string? Param { get; set; }`