Fix parseDockerLikeLabels corrupting label values containing commas

[Copilot]

docker ... ls --format '{{json .}}' returns Labels as a single key=value,key=value string with no escaping of commas inside values. parseDockerLikeLabels split the whole string on ,, so any comma-bearing value was truncated to its first segment and the remainder was written under an empty key. The common victim is com.docker.compose.project.config_files for compose projects started with multiple -f files.

parseDockerLikeLabels('com.docker.compose.project.config_files=/a/base.yml,/a/local.yml,com.docker.compose.project=demo')
// before: { 'com.docker.compose.project.config_files': '/a/base.yml', '': '/a/local.yml', 'com.docker.compose.project': 'demo' }
// after:  { 'com.docker.compose.project.config_files': '/a/base.yml,/a/local.yml', 'com.docker.compose.project': 'demo' }

Changes

• parseDockerLikeLabels.ts: Replaced split(',').reduce(...) with a loop that treats a fragment lacking = as a continuation of the previous label's value, re-joining the comma that split removed.
• parseDockerLikeLabels.test.ts: Added unit tests for single/multiple labels, empty values, empty input, and comma-bearing values.

Known limitation

This is a heuristic. It fully recovers values containing commas (the common case), but a value containing both a comma and an = (e.g. key=a,b=c) remains ambiguous in the flattened ls output—b=c is indistinguishable from a new label. Consumers needing exactness for such values should read docker inspect, whose Config.Labels is a proper JSON map.

[Copilot]

Pull request overview

This PR fixes parseDockerLikeLabels so label values containing commas (as emitted by docker ... ls --format '{{json .}}') are not corrupted by naïvely splitting on ,. It improves correctness for common compose labels like com.docker.compose.project.config_files when multiple -f files are used.

Changes:

• Updated parseDockerLikeLabels to treat comma-split fragments without = as continuations of the previous label’s value.
• Added unit tests covering empty input, multiple labels, empty values, and comma-bearing values.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/vscode-container-client/src/clients/DockerClientBase/parseDockerLikeLabels.ts	Reworks parsing to stitch comma-bearing label values back together instead of creating empty keys / truncating values.
packages/vscode-container-client/src/test/parseDockerLikeLabels.test.ts	Adds unit tests that reproduce the comma-in-value corruption and validate the new heuristic parsing behavior.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.