[Matthew Christopher (matthchr)]

• Improve regex parsing readability with named capture groups.
• Add comments to VMSizeType with examples to showcase what each field does.
• Remove unused (never set) H100/MI300X fields. Use the AcceleratorType field instead.

[Maximiliano Uribe Falcon (MaximilianoUribe)]

(AI-generated)

Could we add a Test_GetVMSize case for the confidential GPU size NCC40ads_H100_v5? It's the only GPU size that uses subfamilyExtra, and right now nothing checks that it parses to Subfamily: "CC". Verified against this branch in both modules:

{
	name: "Standard_NCC40ads_H100_v5",
	size: "NCC40ads_H100_v5",
	expectedVM: &VMSizeType{
		Family:                      "N",
		Subfamily:                   to.Ptr("CC"),
		Cpus:                        "40",
		AdditiveFeatures:            []rune{'a', 'd', 's'},
		AcceleratorType:             to.Ptr("H100"),
		ConfidentialChildCapability: false,
		Version:                     "v5",
		Series:                      "NCCads_v5",
	},
},

[Maximiliano Uribe Falcon (MaximilianoUribe)]

lgtm