test

Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>

Author: RomanNikitenko

code/src/vs/workbench/contrib/extensions/browser/extensionsWorkbenchService.ts

@@ -2405,15 +2405,13 @@ export class ExtensionsWorkbenchService extends Disposable implements IExtension
 			return new MarkdownString().appendText(nls.localize('disallowed', "This extension is disallowed to be installed."));
 		}
 
-		// Check if extension is allowed first (before checking platform compatibility)
 		if (extension.gallery) {
+			// Check if extension is allowed first (before checking platform compatibility or signing)
 			const allowedResult = this.allowedExtensionsService.isAllowed({ id: extension.gallery.identifier.id, publisherDisplayName: extension.gallery.publisherDisplayName });
 			if (allowedResult !== true) {
 				return new MarkdownString(nls.localize('extension not allowed to install', "This extension cannot be installed because it is not in the allowed list."));
 			}
-		}
 
-		if (extension.gallery) {
 			if (!extension.gallery.isSigned && shouldRequireRepositorySignatureFor(extension.private, await this.extensionGalleryManifestService.getExtensionGalleryManifest())) {
 				return new MarkdownString().appendText(nls.localize('not signed', "This extension is not signed."));
 			}
@@ -2433,7 +2431,32 @@ export class ExtensionsWorkbenchService extends Disposable implements IExtension
 				return true;
 			}
 
-			return localResult ?? remoteResult ?? webResult ?? new MarkdownString().appendText(nls.localize('cannot be installed', "Cannot install the '{0}' extension because it is not available in this setup.", extension.displayName ?? extension.identifier.id));
+			// If any of the canInstall checks returned a message about not being allowed, prioritize that
+			// Otherwise return platform compatibility or other messages
+			const allResults: IMarkdownString[] = [];
+			const isMarkdownString = (r: true | IMarkdownString | undefined): r is IMarkdownString => {
+				return r !== undefined && r !== true;
+			};
+			if (isMarkdownString(localResult)) {
+				allResults.push(localResult);
+			}
+			if (isMarkdownString(remoteResult)) {
+				allResults.push(remoteResult);
+			}
+			if (isMarkdownString(webResult)) {
+				allResults.push(webResult);
+			}
+			
+			const notAllowedResult = allResults.find(r => r.value.includes('not in the allowed list') || r.value.includes('not allowed'));
+			if (notAllowedResult) {
+				return new MarkdownString(nls.localize('extension not allowed to install', "This extension cannot be installed because it is not in the allowed list."));
+			}
+
+			// Return the first error message, or a generic message if none
+			if (allResults.length > 0) {
+				return allResults[0];
+			}
+			return new MarkdownString().appendText(nls.localize('cannot be installed', "Cannot install the '{0}' extension because it is not available in this setup.", extension.displayName ?? extension.identifier.id));
 		}
 
 		if (extension.resourceExtension && await this.extensionManagementService.canInstall(extension.resourceExtension) === true) {