The properties-building block (line 703) used split(":", 1) which
incorrectly splits HTTPS URLs like https://github.com/org/repo.git:/workspace.
The early validation block (line 637) already correctly used rsplit(":", 1).
This fix makes both blocks consistent.
ISSUES CLOSED: #904
Add missing CLI flags per specification to resource and LSP commands:
- resource add --update: Re-register an existing resource by removing the
old entry and creating a new one. When the resource does not yet exist,
--update behaves as a normal add.
- resource add --clone-into REPO_URL:CONTAINER_PATH: Clone a remote
repository into a container at a specified path. Only valid for
container-instance and devcontainer-instance types. Stored in resource
properties as a JSON descriptor.
- resource add --mount: Expand container type validation to accept both
container-instance and devcontainer-instance (was restricted to only
container-instance).
- resource list --all: Include auto-discovered child resources in the
listing. Default behavior now excludes auto-discovered resources
(auto_discovered=True) by filtering at the service layer via a new
include_auto_discovered parameter on list_resources().
- resource tree --depth: Change default from -1 (unlimited) to 3, matching
the specification.
- resource type list [REGEX]: Add optional positional regex argument to
filter type names.
- lsp list [REGEX]: Add optional positional regex argument to filter
server names.
All existing test step definitions updated to pass the new parameters.
New Behave feature file (resource_cli_flags_904.feature) with 12 scenarios
covering each new flag.
ISSUES CLOSED: #904
freemo