Fix native MDM plugin marketplace parsing

[Joel Thiessen (jatjat)]

Fixes #333816

Summary

Native MDM delivers structured managed settings as JSON inside an OS string. After the policy layer parses a documented object-form extraKnownMarketplaces value, readConfiguredMarketplaces() silently discarded each marketplace because it accepted only string values.

This change preserves object-form values, adds the marketplace dictionary key as their name, and forwards them to the existing object parser. Existing string-map behavior remains unchanged. A focused regression test covers the native-MDM object shape, including autoUpdate.

Evidence

• The documented schema uses named marketplace objects.
• Native MDM uses the same logical values, encoding objects as JSON strings.
• The policy layer parses structured string values as JSON.
• The marketplace reader currently drops non-string values, although the existing downstream parser supports the documented object shape.

Why This Layer

parseMarketplaceObjectEntry() already supports the documented nested shape. Letting object values reach that parser fixes native MDM without changing policy ingestion or the canonical string-map path used by other delivery channels.

Testing

• npm run compile-client
• ./scripts/test.sh --run src/vs/workbench/contrib/chat/test/common/plugins/pluginMarketplaceService.test.ts (94 passing)
• npm run eslint -- src/vs/workbench/contrib/chat/common/plugins/marketplaceReference.ts src/vs/workbench/contrib/chat/test/common/plugins/pluginMarketplaceService.test.ts
• Manual clean-profile A/B with native MDM:
  • Unpatched Code OSS showed no managed agent plugins.
  • Patched Code OSS displayed the managed plugin, installed it, and loaded its skill in Copilot Chat.

[Copilot]

Pull request overview

Fixes native MDM object-form plugin marketplaces being discarded during configuration parsing.

Changes:

• Preserves and names object-form managed marketplace entries.
• Adds regression coverage for Git sources and autoUpdate.

Reviewed changes

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

File	Description
marketplaceReference.ts	Supports object-form policy entries.
pluginMarketplaceService.test.ts	Tests native MDM marketplace parsing.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Joel Thiessen (jatjat)]

@microsoft-github-policy-service agree company="Canadian Tire Corporation, Limited"