Cherry-pick for release/2.0: REST/OpenAPI/CLI/config-validation bug fixes

[aaronburtle]

Why make this change?

Closes #3515

Ports a set of REST, OpenAPI, CLI, and configuration-validation bug fixes from main to release/2.0 so the 2.0 release line includes:

• Correct HTTP semantics for invalid If-Match headers on REST PUT/PATCH (now returns 400 instead of mishandling).
• Cleaner OpenAPI documents — descriptive 404 message when filtering by a missing role, and removal of redundant _NoAutoPK / _NoPK schemas when request-body-strict is false.
• Accurate config validation: corrected JSON-type mapping for int system types, alignment of parent/child config behavior during deserialization and startup, and improved entity validation logic.
• Schema completeness: level-2 property on runtime.cache in dab.draft.schema.json.
• CLI quality-of-life: consistent log labels matching ASP.NET Core's default console formatter, and clearer error messaging from dab validate when a logger is not yet available.

What is this change?

Cherry-picked PRs (chronological order on main):

• Add level-2 property to runtime.cache in dab.draft.schema.json Add level-2 property to runtime.cache in dab.draft.schema.json #3317
• Make CLI Log Labels consistent with ASP.NET Core and use appropriate label on startup Make CLI Log Labels consistent with ASP.NET Core and use appropriate label on startup #3307
• Align behavior between Parent and Child configs during deserialization and startup Align behavior between Parent and Child configs during deserialization and startup #3321
• Add descriptive message when OpenAPI role filter returns 404 Add descriptive message when OpenAPI role filter returns 404 #3305
• Return 400 for invalid If-Match values Return 400 for invalid If-Match values #3415
• Fix dab validate error messaging when logger is not available Fix dab validate error messaging when logger is not available #3311
• Omit redundant _NoAutoPK and _NoPK OpenAPI schemas when request-body-strict is false Omit redundant _NoAutoPK and _NoPK OpenAPI schemas when request-body-strict is false #3325
• Map int types to integer in System Type to Json Type Map Map int types to integer in System Type to Json Type Map #3327
• Update config validation logic for entities Update config validation logic for entities #3306

How was this tested?

Existing and newly added unit/integration tests from each source PR.

[Copilot]

Pull request overview

This cherry-pick brings a set of REST/OpenAPI/CLI/config-validation fixes from main into release/2.0, improving HTTP semantics, generated OpenAPI output, and configuration parsing/validation behavior (including multi-config “root + data-source-files” scenarios).

Changes:

• REST/OpenAPI: return correct 400 for invalid If-Match on PUT/PATCH; add descriptive 404 ProblemDetails for missing/invalid OpenAPI role requests; reduce OpenAPI schema verbosity when request-body-strict is false.
• Config parsing/validation: align parent/child config handling, improve entity/autoentity validation after autoentity resolution, and refine error surfacing (clean parse errors when logger isn’t available).
• CLI/schema: align CLI log labels with ASP.NET Core console formatter and add missing runtime.cache.level-2 + validation constraints to dab.draft.schema.json.

Reviewed changes

Copilot reviewed 37 out of 37 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/Service/Startup.cs	Downgrades response-compression “enabled” log to Debug.
src/Service/HealthCheck/HealthCheckHelper.cs	Guards datasource health checks when DataSource is null.
src/Service/Controllers/RestController.cs	Moves If-Match semantics inside try/catch; returns descriptive 404 ProblemDetails for OpenAPI role lookups.
src/Service.Tests/UnitTests/RuntimeConfigLoaderJsonDeserializerTests.cs	Updates parsing tests to assert parseError rather than logger invocations; clarifies child-file behavior docs.
src/Service.Tests/SqlTests/RestApiTests/Put/PutApiTestBase.cs	Adds integration test ensuring invalid If-Match returns 400.
src/Service.Tests/SqlTests/RestApiTests/Patch/PatchApiTestBase.cs	Adds integration test ensuring invalid If-Match returns 400.
src/Service.Tests/OpenApiDocumentor/StoredProcedureGeneration.cs	Updates expected JSON types from number → integer for int-like params.
src/Service.Tests/OpenApiDocumentor/RequestBodyStrictTests.cs	Updates strict=false assertions to ensure redundant schemas are omitted and request bodies reference base schema.
src/Service.Tests/OpenApiDocumentor/ParameterValidationTests.cs	Updates stored-proc parameter JSON type assertions to integer.
src/Service.Tests/OpenApiDocumentor/MissingRoleNotFoundTests.cs	Adds integration test validating descriptive 404 ProblemDetails for missing/invalid OpenAPI roles.
src/Service.Tests/ModuleInitializer.cs	Updates Verify settings to ignore newly added runtime-only RuntimeConfig members.
src/Service.Tests/Configuration/RuntimeConfigLoaderTests.cs	Adds tests covering child config env-var ignore behavior and “existing but invalid child halts load”.
src/Service.Tests/Configuration/ConfigurationTests.cs	Adapts tests to updated parsing/validation flows and nullable DataSource handling.
src/Core/Services/TypeHelper.cs	Maps integral CLR types to JSON integer type.
src/Core/Services/OpenAPI/OpenApiDocumentor.cs	Threads isRequestBodyStrict through path/op/schema generation and gates redundant schemas.
src/Core/Services/OpenAPI/JsonDataType.cs	Introduces Integer JSON data type.
src/Core/Services/MetadataProviders/SqlMetadataProvider.cs	Wraps non-DAB exceptions during DB connection validation as initialization errors.
src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs	Tracks per-autoentity resolution counts for downstream validation.
src/Core/Services/MetadataProviders/CosmosSqlMetadataProvider.cs	Adjusts for nullable DataSource (but still needs consistent null-safety).
src/Core/Configurations/RuntimeConfigValidator.cs	Adds root/non-root entity presence validation after autoentity resolution; avoids GraphQL validation when datasource absent.
src/Core/Configurations/RuntimeConfigProvider.cs	Updates parsing to new TryParseConfig(..., out parseError) overload (but still needs nullable-safe dereferences).
src/Config/RuntimeConfigLoader.cs	Reworks parsing to return a clean parseError message instead of logging/printing stack traces.
src/Config/ObjectModel/RuntimeConfig.cs	Makes DataSource nullable; adds root/child tracking and autoentity-resolution metadata; improves child-config load behavior.
src/Config/ObjectModel/MultipleCreateOptions.cs	Removes trailing whitespace.
src/Config/ObjectModel/ChildConfigMetadata.cs	Adds a record type for child-config metadata (validation support).
src/Config/FileSystemRuntimeConfigLoader.cs	Emits clean parse errors to stderr and tracks whether they were already emitted.
src/Cli/CustomLoggerProvider.cs	Aligns CLI log level labels to ASP.NET Core abbreviations and hardens color lookups.
src/Cli/ConfigGenerator.cs	Adds clear CLI errors for root configs without datasources; suppresses duplicate parse-failure output (but needs nullable deref fixes).
src/Cli/Commands/ValidateOptions.cs	Simplifies invalid-config message.
src/Cli.Tests/ValidateConfigTests.cs	Adds extensive config-validation truth-table tests and verifies clean error paths.
src/Cli.Tests/UserDelegatedAuthRuntimeParsingTests.cs	Adjusts for nullable DataSource with null-forgiveness where appropriate.
src/Cli.Tests/ModuleInitializer.cs	Updates Verify ignore list for new runtime-only RuntimeConfig members.
src/Cli.Tests/EnvironmentTests.cs	Updates assertions to match clean parse-error stderr output.
src/Cli.Tests/EndToEndTests.cs	Updates end-to-end expectations to match new stderr/stdout behavior and nullable DataSource.
src/Cli.Tests/CustomLoggerTests.cs	Adds test coverage for abbreviated log labels.
src/Cli.Tests/ConfigureOptionsTests.cs	Adjusts for nullable DataSource with null-forgiveness where appropriate.
schemas/dab.draft.schema.json	Adds runtime.cache.level-2 and enforces ttl-seconds minimum of 1.

Comments suppressed due to low confidence (1)

src/Core/Configurations/RuntimeConfigProvider.cs:200

• RuntimeConfig.DataSource is now nullable, but this Initialize path still dereferences runtimeConfig.DataSource later (e.g., DatabaseType / ConnectionString). With nullable enabled + TreatWarningsAsErrors, this will produce nullable warnings and can also NRE if a config without data-source is passed. Add an explicit runtimeConfig.DataSource is null guard (with a clearer exception message) and/or use null-forgiveness after the guard so the code is both safe and compiles cleanly.

            if (string.IsNullOrEmpty(runtimeConfig.DataSource?.ConnectionString))
            {
                throw new ArgumentException($"'{nameof(runtimeConfig.DataSource.ConnectionString)}' cannot be null or empty.", nameof(runtimeConfig.DataSource.ConnectionString));
            }

[Aniruddh25]

I have approved the PR since its mostly cherry picks, but the comments from Copilot are pretty high importance. Did we resolve those comments when merged in main?

[aaronburtle]

I have approved the PR since its mostly cherry picks, but the comments from Copilot are pretty high importance. Did we resolve those comments when merged in main?

Comments #2 and #3 are not valid, but comment #1 was, its a 1 line fix, will send to main separately.