feat: [FME-19823]: wire --comment/--title into CLI delete for feature_flag, environment, and segment:definition

[apetruccelli]

Description

FME-19742 (PR #12932) added comment/title audit fields to the v4 delete endpoints for feature flags, environments, and segment definitions — all as a JSON body (ArchiveUnarchiveRequest / SegmentDefinitionDeleteRequest, no @QueryParam).

• delete feature_flag: had no --comment/--title flags at all. Added them, sent as a JSON body, omitted (not sent as null/empty) when unset.
• delete fme_environment: same gap, same fix.
• delete segment:definition: already had --comment/--title, but wired as query params from before FME-19742. The merged backend now rejects that (400 Missing required title) since it expects a body. Moved to body_params to match.

Verified live end-to-end against qa0: created a feature flag, segment, and segment definition, then deleted each with --comment/--title — confirmed via --debug that the request body carries both fields and the server returns 200. delete fme_environment was confirmed to send the correct body (request accepted past validation) but full delete success wasn't observed in this qa0 project due to an unrelated 409 hasDependents (API tokens scoped to the environment) — a pre-existing backend guard, not caused by this change.

Commands

• harness delete feature_flag <name> [--comment <text>] [--title <text>]
• harness delete fme_environment <id> [--comment <text>] [--title <text>]
• harness delete segment:definition <segment-name> --env <env-id> [--comment <text>] [--title <text>]

Acceptance Criteria

• --comment/--title added to delete feature_flag, sent as a JSON body
• --comment/--title added to delete fme_environment, sent as a JSON body
• delete segment:definition moved from query params to a JSON body for --comment/--title
• All three omit comment/title from the body entirely when unset (not sent as null/empty)
• Unit tests for all three commands (comment/title present + omitted-when-unset), go test ./pkg/... passes
• Verified live against qa0 for feature_flag and segment:definition delete
• go build ./..., gofmt, and go vet clean

🤖 Generated with Claude Code