Snapshot

[linglingye001]

No description provided.

[Copilot]

Pull Request Overview

This PR adds snapshot support to Azure App Configuration, allowing users to reference immutable snapshots of configuration data instead of using key/label filters. The implementation includes validation to ensure snapshots cannot be used with filters, proper API calls to retrieve snapshot data, and comprehensive test coverage.

• Adds SnapshotName field to Selector struct for referencing snapshots
• Implements validation logic to prevent mixing snapshots with key/label filters
• Adds snapshot retrieval logic using Azure SDK's snapshot APIs

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 16 comments.

Show a summary per file

File	Description
azureappconfiguration/options.go	Adds SnapshotName field to Selector struct with documentation
azureappconfiguration/utils.go	Updates selector validation to handle snapshot selectors and enforce mutual exclusivity with filters
azureappconfiguration/utils_test.go	Adds comprehensive test cases for snapshot selector validation scenarios
azureappconfiguration/settings_client.go	Implements snapshot retrieval logic using Azure SDK APIs
azureappconfiguration/azureappconfiguration.go	Updates feature flag loading to filter non-feature flag content and handle snapshots
azureappconfiguration/snapshot_test.go	Comprehensive test suite covering snapshot loading scenarios and mixed content handling

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The error message should specify what composition types are supported or provide more context about why only 'key' type is allowed.

Suggested change

	if snapshot.CompositionType == nil || *snapshot.CompositionType != azappconfig.CompositionTypeKey {
	return nil, fmt.Errorf("composition type for the selected snapshot '%s' must be 'key'", filter.SnapshotName)
	return nil, fmt.Errorf(
	"unsupported composition type '%v' for snapshot '%s': only 'key' is supported by this client. "+
	"Other composition types (e.g., 'set') are not supported.",
	snapshot.CompositionType, filter.SnapshotName)

[RichardChen820]

LGTM