Fixes microsoft#143923: Show a warning when extensionKind is defined without a main field.

Author: alexdima

src/vs/workbench/services/extensions/common/abstractExtensionService.ts

@@ -945,7 +945,7 @@ export abstract class AbstractExtensionService extends Disposable implements IEx
 
 	protected createLogger(): Logger {
 		return new Logger((severity, source, message) => {
-			if (this._isDev && source) {
+			if (source) {
 				this._logOrShowMessage(severity, `[${source}]: ${message}`);
 			} else {
 				this._logOrShowMessage(severity, message);
@@ -956,9 +956,8 @@ export abstract class AbstractExtensionService extends Disposable implements IEx
 	protected _logOrShowMessage(severity: Severity, msg: string): void {
 		if (this._isDev) {
 			this._showMessageToUser(severity, msg);
-		} else {
-			this._logMessageInConsole(severity, msg);
 		}
+		this._logMessageInConsole(severity, msg);
 	}
 
 	private _acquireInternalAPI(): IInternalExtensionService {

src/vs/workbench/services/extensions/common/extensionPoints.ts

@@ -468,6 +468,12 @@ class ExtensionManifestValidator extends ExtensionManifestHandler {
 				return false;
 			}
 		}
+		if (typeof extensionDescription.extensionKind !== 'undefined') {
+			if (typeof extensionDescription.main === 'undefined') {
+				notices.push(nls.localize('extensionDescription.extensionKind', "property `{0}` can be defined only if property `main` is also defined.", 'extensionKind'));
+				// not a failure case
+			}
+		}
 		if (typeof extensionDescription.main !== 'undefined') {
 			if (typeof extensionDescription.main !== 'string') {
 				notices.push(nls.localize('extensionDescription.main1', "property `{0}` can be omitted or must be of type `string`", 'main'));