…roup 2) (#3417) ## Summary - Closes on #3418 Fixes 14 issues where CLI option `--help` text showed incorrect default values, wrong descriptions, or enum mismatches compared to the JSON schema (`dab.draft.schema.json`) and C# runtime defaults. ## Issues Addressed ### 2a. Default Value Mismatches (8 issues) | # | Property | Before | After | Resolution | |---|----------|--------|-------|------------| | #3347 | `runtime.mcp.dml-tools.read-records` | CLI: true, Schema: false | Both: true | Schema changed false?true to match C# runtime | | #3348 | `runtime.mcp.dml-tools.create-record` | CLI: true, Schema: false | Both: true | Schema changed false?true to match C# runtime | | #3349 | `runtime.mcp.dml-tools.describe-entities` | CLI: true, Schema: false | Both: true | Schema changed false?true to match C# runtime | | #3350 | `runtime.graphql.multiple-mutations.create.enabled` | CLI: true, Schema: false | CLI: false | HelpText corrected to match schema | | #3351 | `runtime.host.mode` | CLI: Development, Schema: production | CLI: production | HelpText corrected to match schema | | #3352 | `data-source.options.set-session-context` | CLI: "(default)" text | No default | Removed misleading text | | #3353 | `runtime.rest.request-body-strict` | init: false, Schema: true | init: false | HelpText corrected to match schema | | #3354 | `runtime.mcp.dml-tools.aggregate-records` | Schema: .enabled=true, top-level=false | Both: true | Schema changed false?true to match C# runtime | ### 2b. Description/Display Bugs (3 issues) | # | Property | Fix | |---|----------|-----| | #3355 | `--otel-service-name` | Changed description from "Headers for Open Telemetry" to "Service name for Open Telemetry" | | #3356 | `dab add-telemetry` options | Removed duplicate `(Default: X)` from HelpText where `Default=` attribute already displays the value | | #3357 | `runtime.graphql.depth-limit` | Changed description from "infinity"/"-1" to "Default: null (no limit)" | ### 2c. Enum Mismatches (3 issues) | # | Property | Fix | |---|----------|-----| | #3359 | `--database-type` (configure) | Already fixed in PR #3409 | | #3360 | `--database-type` (configure) | Changed to lowercase enum values | ### Cross-group overlaps resolved - #3378 (`rest.request-body-strict`) ? same fix as #3353 - #3379 (`host.mode`) ? same fix as #3351 - #3380 (`set-session-context`) ? same fix as #3352 - #3381 (`multiple-mutations.create.enabled`) ? same fix as #3350 - #3382 (DML tool defaults) ? same fix as #3347-#3349; schema aligned to `true` to match C# runtime ## Files Changed | File | Changes | |------|---------| | `src/Cli/Commands/ConfigureOptions.cs` | 5 HelpText corrections (host.mode, multiple-mutations, set-session-context, depth-limit, database-type) | | `src/Cli/Commands/InitOptions.cs` | 2 HelpText corrections (request-body-strict, database-type) | | `src/Cli/Commands/AddTelemetryOptions.cs` | 4 HelpText/description corrections | | `schemas/dab.draft.schema.json` | DML tool defaults changed false?true (8 locations) to match C# runtime (`DmlToolsConfig.DEFAULT_ENABLED = true`) | ## Testing - ConfigureOptionsTests: 78 passed, 0 failed - EndToEndTests: 138 passed, 0 failed, 3 skipped - Build: 0 warnings, 0 errors

[Bug]: CLI/Schema ENUM mismatch #3360

Description

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions