Skip to content

fix: deprecate --*.disabled CLI flags, wire up --mcp.disabled #3482

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
souvikghosh04 wants to merge 3 commits into
base: main
Choose a base branch
from
+34 −23
Open

fix: deprecate --*.disabled CLI flags, wire up --mcp.disabled #3482

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4b9e0f1
fix: deprecate --*.disabled CLI flags, wire up --mcp.disabled (#3376)
souvikghosh04 Apr 29, 2026
867439a
address review: update XML docs and add explicit MCP assertion branch
souvikghosh04 May 7, 2026
b5deead
Merge branch 'main' into Usr/sogh/deprecate-disabled-flags
souvikghosh04 May 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 16 additions & 1 deletion src/Cli.Tests/EndToEndTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1288,6 +1288,11 @@ public void TestBaseRouteIsConfigurableForSWA(string authProvider, bool isExcept
[DataRow(ApiType.GraphQL, true, false, true, false, DisplayName = "Validate that GraphQL endpoint is disabled when enabled option is omitted and disabled option is included in the init command.")]
[DataRow(ApiType.GraphQL, true, true, false, false, DisplayName = "Validate that GraphQL endpoint is disabled when enabled option is set to false and disabled option is included in the init command.")]
[DataRow(ApiType.GraphQL, true, true, true, true, true, DisplayName = "Validate that config generation fails when enabled and disabled options provide conflicting values for GraphQL endpoint.")]
[DataRow(ApiType.MCP, false, false, true, true, DisplayName = "Validate that MCP endpoint is enabled when both enabled and disabled options are omitted from the init command.")]
[DataRow(ApiType.MCP, false, true, true, true, DisplayName = "Validate that MCP endpoint is enabled when enabled option is set to true and disabled option is omitted from the init command.")]
[DataRow(ApiType.MCP, true, false, true, false, DisplayName = "Validate that MCP endpoint is disabled when enabled option is omitted and disabled option is included in the init command.")]
[DataRow(ApiType.MCP, true, true, false, false, DisplayName = "Validate that MCP endpoint is disabled when enabled option is set to false and disabled option is included in the init command.")]
[DataRow(ApiType.MCP, true, true, true, true, true, DisplayName = "Validate that config generation fails when enabled and disabled options provide conflicting values for MCP endpoint.")]
public void TestEnabledDisabledFlagsForApis(
ApiType apiType,
bool includeDisabledFlag,
Expand Down Expand Up @@ -1333,12 +1338,22 @@ public void TestEnabledDisabledFlagsForApis(
Assert.IsNotNull(runtimeConfig.Runtime.Rest);
Assert.AreEqual(expectedEnabledFlagValueInConfig, runtimeConfig.Runtime.Rest.Enabled);
}
else
else if (apiType is ApiType.GraphQL)
{
Assert.IsNotNull(runtimeConfig.Runtime);
Assert.IsNotNull(runtimeConfig.Runtime.GraphQL);
Assert.AreEqual(expectedEnabledFlagValueInConfig, runtimeConfig.Runtime.GraphQL.Enabled);
}
else if (apiType is ApiType.MCP)
{
Assert.IsNotNull(runtimeConfig.Runtime);
Assert.IsNotNull(runtimeConfig.Runtime.Mcp);
Assert.AreEqual(expectedEnabledFlagValueInConfig, runtimeConfig.Runtime.Mcp.Enabled);
}
else
{
Assert.Fail($"Unexpected ApiType value '{apiType}' in test.");
}
}
}

Expand Down
6 changes: 3 additions & 3 deletions src/Cli/Commands/InitOptions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,19 +111,19 @@ public InitOptions(
[Option("runtime.base-route", Default = null, Required = false, HelpText = "Specifies the base route for API requests.")]
public string? RuntimeBaseRoute { get; }

[Option("rest.disabled", Default = false, Required = false, HelpText = "Disables REST endpoint for all entities.")]
[Option("rest.disabled", Default = false, Required = false, Hidden = true, HelpText = "[Deprecated] Use --rest.enabled false instead. Disables REST endpoint for all entities.")]
public bool RestDisabled { get; }

[Option("graphql.path", Default = GraphQLRuntimeOptions.DEFAULT_PATH, Required = false, HelpText = "Specify the GraphQL endpoint's default prefix.")]
public string GraphQLPath { get; }

[Option("graphql.disabled", Default = false, Required = false, HelpText = "Disables GraphQL endpoint for all entities.")]
[Option("graphql.disabled", Default = false, Required = false, Hidden = true, HelpText = "[Deprecated] Use --graphql.enabled false instead. Disables GraphQL endpoint for all entities.")]
public bool GraphQLDisabled { get; }

[Option("mcp.path", Default = McpRuntimeOptions.DEFAULT_PATH, Required = false, HelpText = "Specify the MCP endpoint's default prefix.")]
public string McpPath { get; }

[Option("mcp.disabled", Default = false, Required = false, HelpText = "Disables MCP endpoint for all entities.")]
[Option("mcp.disabled", Default = false, Required = false, Hidden = true, HelpText = "[Deprecated] Use --mcp.enabled false instead. Disables MCP endpoint for all entities.")]
public bool McpDisabled { get; }

[Option("rest.enabled", Required = false, HelpText = "(Default: true) Enables REST endpoint for all entities. Supported values: true, false.")]
Expand Down
34 changes: 15 additions & 19 deletions src/Cli/ConfigGenerator.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,21 +102,28 @@ public static bool TryCreateRuntimeConfig(InitOptions options, FileSystemRuntime
// If --rest.disabled flag is included in the init command, we log a warning to not use this flag as it will be deprecated in future versions of DAB.
Copy link
Copy Markdown
Contributor

@RubenCerna2079 RubenCerna2079 May 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Change the comment so instead of saying the flag will be deprecated in the future, it will say that it is deprecated.

if (options.RestDisabled is true)
{
_logger.LogWarning("The option --rest.disabled will be deprecated and support for the option will be removed in future versions of Data API builder." +
" We recommend that you use the --rest.enabled option instead.");
_logger.LogWarning("The option --rest.disabled is deprecated and support for the option will be removed in future versions of Data API builder." +
" Use the --rest.enabled option instead.");
}

// If --graphql.disabled flag is included in the init command, we log a warning to not use this flag as it will be deprecated in future versions of DAB.
Copy link
Copy Markdown
Contributor

@RubenCerna2079 RubenCerna2079 May 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Change the comment so instead of saying the flag will be deprecated in the future, it will say that it is deprecated.

if (options.GraphQLDisabled is true)
{
_logger.LogWarning("The option --graphql.disabled will be deprecated and support for the option will be removed in future versions of Data API builder." +
" We recommend that you use the --graphql.enabled option instead.");
_logger.LogWarning("The option --graphql.disabled is deprecated and support for the option will be removed in future versions of Data API builder." +
" Use the --graphql.enabled option instead.");
}

// If --mcp.disabled flag is included in the init command, we log a warning to not use this flag as it will be deprecated in future versions of DAB.
Copy link
Copy Markdown
Contributor

@RubenCerna2079 RubenCerna2079 May 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Change the comment so instead of saying the flag will be deprecated in the future, it will say that it is deprecated.

if (options.McpDisabled is true)
{
_logger.LogWarning("The option --mcp.disabled is deprecated and support for the option will be removed in future versions of Data API builder." +
" Use the --mcp.enabled option instead.");
}

bool restEnabled, graphQLEnabled, mcpEnabled;
if (!TryDetermineIfApiIsEnabled(options.RestDisabled, options.RestEnabled, ApiType.REST, out restEnabled) ||
!TryDetermineIfApiIsEnabled(options.GraphQLDisabled, options.GraphQLEnabled, ApiType.GraphQL, out graphQLEnabled) ||
!TryDetermineIfMcpIsEnabled(options.McpEnabled, out mcpEnabled))
!TryDetermineIfApiIsEnabled(options.McpDisabled, options.McpEnabled, ApiType.MCP, out mcpEnabled))
{
Comment thread
souvikghosh04 marked this conversation as resolved.
return false;
}
Expand Down Expand Up @@ -236,7 +243,7 @@ public static bool TryCreateRuntimeConfig(InitOptions options, FileSystemRuntime
}
}

if (options.RestDisabled && options.GraphQLDisabled)
if (!restEnabled && !graphQLEnabled)
Copy link
Copy Markdown
Contributor

@RubenCerna2079 RubenCerna2079 May 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be changed so that all three rest, graphQL, and mcp cannot be disabled together?

Copy link
Copy Markdown
Contributor

@anushakolan anushakolan May 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more. 

if (!restEnabled && !graphQLEnabled && !mcpEnabled)
{
    _logger.LogError("At least one of REST, GraphQL, or MCP must be enabled.");
    return false;
}

This might be a better way to update the code.

{
_logger.LogError("Both Rest and GraphQL cannot be disabled together.");
return false;
Expand Down Expand Up @@ -296,12 +303,12 @@ public static bool TryCreateRuntimeConfig(InitOptions options, FileSystemRuntime

/// <summary>
/// Helper method to determine if the api is enabled or not based on the enabled/disabled options in the dab init command.
/// The method also validates that there is no mismatch in semantics of enabling/disabling the REST/GraphQL API(s)
/// The method also validates that there is no mismatch in semantics of enabling/disabling the REST/GraphQL/MCP API(s)
/// based on the values supplied in the enabled/disabled options for the API in the init command.
/// </summary>
/// <param name="apiDisabledOptionValue">Value of disabled option as in the init command. If the option is omitted in the command, default value is assigned.</param>
/// <param name="apiEnabledOptionValue">Value of enabled option as in the init command. If the option is omitted in the command, default value is assigned.</param>
/// <param name="apiType">ApiType - REST/GraphQL.</param>
/// <param name="apiType">ApiType - REST/GraphQL/MCP.</param>
/// <param name="isApiEnabled">Boolean value indicating whether the API endpoint is enabled or not.</param>
private static bool TryDetermineIfApiIsEnabled(bool apiDisabledOptionValue, CliBool apiEnabledOptionValue, ApiType apiType, out bool isApiEnabled)
{
Expand Down Expand Up @@ -333,17 +340,6 @@ private static bool TryDetermineIfApiIsEnabled(bool apiDisabledOptionValue, CliB
return true;
}

/// <summary>
/// Helper method to determine if the mcp api is enabled or not based on the enabled/disabled options in the dab init command.
/// </summary>
/// <param name="mcpEnabledOptionValue">True, if MCP is enabled</param>
/// <param name="isMcpEnabled">Out param isMcpEnabled</param>
/// <returns>True if MCP is enabled</returns>
private static bool TryDetermineIfMcpIsEnabled(CliBool mcpEnabledOptionValue, out bool isMcpEnabled)
{
return TryDetermineIfApiIsEnabled(false, mcpEnabledOptionValue, ApiType.MCP, out isMcpEnabled);
}

/// <summary>
/// Helper method to determine if the multiple create operation is enabled or not based on the inputs from dab init command.
/// </summary>
Expand Down
Loading