[aniket866]

Changes Implemented

• Added DeleteService(ref string) error to MicrocksClient, supporting both direct service IDs and name:version references.
• Updated resolveServiceID to scan all matching services:
• Returns KindNotFound when no service matches.
• Returns a clear error with KindUsage listing the matched service IDs and types when multiple services match the reference.
• Returns the single resolved ID on exact match.
• Handled HTTP 204 / 200 as successful responses and mapped HTTP 404 to errors.KindNotFound.
• Propagated the effective InsecureTLS setting to Keycloak client initialization during token refresh.

Command Layer (cmd/service.go & cmd/cmd.go)

• Added newServiceDeleteCommand under microcks service delete (and registered a top-level microcks - delete alias for backwards compatibility).
• Reused newCommandClient for authentication and client options.
• Uses RunE returning wrapped internal errors (errors.Wrapf(errors.KindUsage, ...)).
• Interactive terminal check via term.IsTerminal:
• Prompts user [y/N] before deletion if in an interactive terminal.
• Requires --force / -f flag in non-interactive/automated environments (e.g., CI/CD).
• Skips prompt if --force is supplied.

Closes #294

Ai disclosure: Chatgpt is used to analyze related files and do required changes.

[Caesarsage]

Thanks @aniket866, the delete command itself looks close to what we asked for. But @DivyanshuVortex still has #295 open for the same issue, you can reach out to work with her or if she is still working on ti

please feel free to reopen this after an agreement

[aniket866]

hi @Caesarsage we agreed to work on this: #294 (comment)

Please reopen and review

[Caesarsage]

Thank you for the clarification

[Caesarsage]

See my review here @aniket866
Could you take a look

#295 (comment)

[aniket866]

hi @Caesarsage this PR is ready

[Caesarsage]

hi @Caesarsage this PR is ready

Thanks @aniket866! There are no new commits since my last review, so the TLS changes (tracked in #287/#605) and the hidden delete alias are still in. Could you drop those so this stays about service delete?