@forge/manifest

12.1.0-next.612.1.0-next.7
out/validators/permissions-validator.js
~out/validators/permissions-validator.jsModified
+7−17
Index: package/out/validators/permissions-validator.js
===================================================================
--- package/out/validators/permissions-validator.js
+++ package/out/validators/permissions-validator.js
@@ -210,9 +210,9 @@
                 perms: manifest.typedContent.permissions.external?.scripts?.map((key) => typeof key === 'object' ? key.address : key)
             }
         ];
         mapping.forEach((item) => this.validateExternalPermissionURLs(errors, item.element, item.perms, manifest));
-        const { fetch: originalFetch, ...restOfExternalPermissions } = manifest.typedContent.permissions.external || {};
+        const { fetch: originalFetch, configurable: _configurable, ...restOfExternalPermissions } = manifest.typedContent.permissions.external || {};
         Object.entries(originalFetch || {}).forEach(([key, values]) => {
             if (values.some((value) => typeof value === 'string')) {
                 errors.push({
                     message: text_1.errors.permissions.deprecatedEgressPermission(`fetch.${key}`),
@@ -239,26 +239,16 @@
             }
         });
         const remoteMap = manifest.typedContent.remotes?.reduce((prev, item) => {
             if ((0, features_1.configurableRemotesEnabled)()) {
-                if (typeof item.configurable !== 'undefined') {
-                    if (typeof item.configurable.default !== 'undefined') {
-                        const baseUrl = item.configurable.default;
-                        return prev.set(item.key, baseUrl);
-                    }
-                    else {
-                        return prev.set(item.key, null);
-                    }
+                if (item.baseUrl) {
+                    const baseUrl = typeof item.baseUrl === 'string' ? item.baseUrl : item.baseUrl.default;
+                    return prev.set(item.key, baseUrl);
                 }
-                else {
-                    if (typeof item.baseUrl !== 'undefined') {
-                        const baseUrl = typeof item.baseUrl === 'string' ? item.baseUrl : item.baseUrl.default;
-                        return prev.set(item.key, baseUrl);
-                    }
-                    else {
-                        return prev;
-                    }
+                if (!item.baseUrl && item.configurable) {
+                    return prev.set(item.key, null);
                 }
+                return prev;
             }
             else {
                 if (item.baseUrl !== undefined) {
                     const baseUrl = typeof item.baseUrl === 'string' ? item.baseUrl : item.baseUrl.default;