Fix for incorrect error diagnostic on MSGraph existing resource usage (#18160)

Quick fix for #18158

I think the root cause of the issue is that
[ResourceTypeComponents](https://github.com/Azure/bicep/blob/2b1a61d6d6dbba88fb6f9dd798be7d2af5cfd90a/src/Bicep.Core/TypeSystem/Types/ResourceType.cs#L11)
has a property named "ValidParentScopes", which is intended to capture
both writable and readable scopes, but the type loading code was updated
to be more precise about differentiating readable + writable scopes, and
misinterpreted this field to just mean "writable scopes" (see
[here](https://github.com/Azure/bicep/blob/2b1a61d6d6dbba88fb6f9dd798be7d2af5cfd90a/src/Bicep.Core/TypeSystem/Providers/Extensibility/ExtensionResourceTypeFactory.cs#L42))

Author: anthony-c-martin

src/Bicep.Core.IntegrationTests/ExtensionRegistryTests.cs

@@ -946,4 +946,28 @@ extension bar
 }
 """);
     }
+
+    [TestMethod]
+    public async Task MSGraph_extension_1_0_0_is_valid()
+    {
+        // https://github.com/Azure/bicep/issues/18158
+        var registry = "mcr.microsoft.com";
+        var repository = "bicep/extensions/microsoftgraph/v1.0";
+
+        var services = ExtensionTestHelper.GetServiceBuilder(new MockFileSystem(), registry, repository, AllFeaturesEnabled);
+        var typesTgz = new EmbeddedFile(typeof(ExtensionRegistryTests).Assembly, "Files/ExtensionTypes/msgraph-1.0.0-types.tgz");
+        await RegistryHelper.PublishExtensionToRegistryAsync(services.Build(), $"br:{registry}/{repository}:1.0.0", typesTgz.BinaryData);
+
+        var result = await CompilationHelper.RestoreAndCompile(services, """
+            param application object
+
+            extension 'br:mcr.microsoft.com/bicep/extensions/microsoftgraph/v1.0:1.0.0' as microsoftGraphV1_0
+
+            resource ownerUsers 'Microsoft.Graph/[email protected]' existing = [for (owner, i) in application.owners: {
+              userPrincipalName: owner.upn
+            }]
+            """);
+
+        result.Should().NotHaveAnyCompilationBlockingDiagnostics();
+    }
 }

src/Bicep.Core.IntegrationTests/Files/ExtensionTypes/msgraph-1.0.0-types.tgz

@@ -11,15 +11,13 @@ public record EmbeddedFile(
         Assembly Assembly,
         string StreamPath)
     {
-        private readonly Lazy<string> contentsLazy = new(() =>
-        {
-            var stream = Assembly.GetManifestResourceStream(StreamPath);
-
-            return new StreamReader(stream!).ReadToEnd();
-        });
+        private readonly Lazy<BinaryData> binaryDataLazy = new(() => BinaryData.FromStream(Assembly.GetManifestResourceStream(StreamPath)!));
+        private readonly Lazy<string> contentsLazy = new(() => new StreamReader(Assembly.GetManifestResourceStream(StreamPath)!).ReadToEnd());
 
         public string Contents => contentsLazy.Value;
 
+        public BinaryData BinaryData => binaryDataLazy.Value;
+
         public string FileName => Path.GetFileName(StreamPath);
 
         public string RelativeSourcePath => Path.Combine("src", Assembly.GetName().Name!, StreamPath);

src/Bicep.Core.UnitTests/Baselines/EmbeddedFile.cs

@@ -39,7 +39,10 @@ public ResourceTypeComponents GetResourceType(Azure.Bicep.Types.Concrete.Resourc
             var (readableScopes, writableScopes) = GetScopeInfo(resourceType);
             var readOnlyScopes = readableScopes & ~writableScopes;
 
-            return new ResourceTypeComponents(resourceTypeReference, writableScopes, readOnlyScopes, ToResourceFlags(resourceType), bodyType);
+            // Temporary fix for https://github.com/Azure/bicep/issues/18158
+            var validParentScopes = readableScopes | writableScopes;
+
+            return new ResourceTypeComponents(resourceTypeReference, validParentScopes, readOnlyScopes, ToResourceFlags(resourceType), bodyType);
         }
 
         public TypeSymbol GetConfigurationType(Azure.Bicep.Types.Concrete.TypeBase configurationType)