output: fix runtime-import-check errors

The commit fixes `runtime-import-check` errors in `@theia/output`, but
properly organizing code that should be browser-specific outside of
common.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>

Author: vince-fugnitto

CHANGELOG.md

@@ -13,6 +13,8 @@
 - [task] `TaskDefinition.properties.required` is now optional to align with the specification [#10015](https://github.com/eclipse-theia/theia/pull/10015)
 - [core] `setTopPanelVisibily` renamed to `setTopPanelVisibility` [#10020](https://github.com/eclipse-theia/theia/pull/10020)
 - [application-manager] break `rebuild` API: second argument is now an optional object instead of an optional array.
+- [output] moved `output-channel` from `common` to `browser` []()
+- [output] moved `output-preferences` from `common` to `core` []()
 
 ## v1.17.2 - 9/1/2021
 

examples/api-samples/src/browser/output/sample-output-channel-with-severity.ts

@@ -15,7 +15,7 @@
  ********************************************************************************/
 import { FrontendApplicationContribution } from '@theia/core/lib/browser';
 import { inject, injectable, interfaces } from '@theia/core/shared/inversify';
-import { OutputChannelManager, OutputChannelSeverity } from '@theia/output/lib/common/output-channel';
+import { OutputChannelManager, OutputChannelSeverity } from '@theia/output/lib/browser/output-channel';
 
 @injectable()
 export class SampleOutputChannelWithSeverity

packages/debug/src/browser/debug-session-connection.ts

@@ -19,7 +19,7 @@
 import { DebugProtocol } from 'vscode-debugprotocol';
 import { Deferred } from '@theia/core/lib/common/promise-util';
 import { Event, Emitter, DisposableCollection, Disposable, MaybePromise } from '@theia/core';
-import { OutputChannel } from '@theia/output/lib/common/output-channel';
+import { OutputChannel } from '@theia/output/lib/browser/output-channel';
 import { IWebSocket } from '@theia/core/shared/vscode-ws-jsonrpc';
 
 export interface DebugExitEvent {

packages/debug/src/browser/debug-session-contribution.ts

@@ -23,7 +23,7 @@ import { WebSocketConnectionProvider } from '@theia/core/lib/browser/messaging/w
 import { DebugSession } from './debug-session';
 import { BreakpointManager } from './breakpoint/breakpoint-manager';
 import { DebugSessionOptions } from './debug-session-options';
-import { OutputChannelManager, OutputChannel } from '@theia/output/lib/common/output-channel';
+import { OutputChannelManager, OutputChannel } from '@theia/output/lib/browser/output-channel';
 import { DebugPreferences } from './debug-preferences';
 import { DebugSessionConnection } from './debug-session-connection';
 import { IWebSocket } from '@theia/core/shared/vscode-ws-jsonrpc';

packages/output/src/common/output-channel.ts packages/output/src/browser/output-channel.ts

@@ -20,12 +20,9 @@ import URI from '@theia/core/lib/common/uri';
 import { Deferred } from '@theia/core/lib/common/promise-util';
 import { Resource, ResourceResolver } from '@theia/core/lib/common/resource';
 import { Emitter, Event, Disposable, DisposableCollection } from '@theia/core';
-// eslint-disable-next-line @theia/runtime-import-check
 import { MonacoEditorModel } from '@theia/monaco/lib/browser/monaco-editor-model';
-// eslint-disable-next-line @theia/runtime-import-check
 import { MonacoTextModelService, IReference } from '@theia/monaco/lib/browser/monaco-text-model-service';
-import { OutputUri } from './output-uri';
-// eslint-disable-next-line @theia/runtime-import-check
+import { OutputUri } from '../common/output-uri';
 import { OutputResource } from '../browser/output-resource';
 import { OutputPreferences } from './output-preferences';
 

packages/output/src/browser/output-contribution.ts

@@ -25,7 +25,7 @@ import { OutputWidget } from './output-widget';
 import { OutputContextMenu } from './output-context-menu';
 import { OutputUri } from '../common/output-uri';
 import { ClipboardService } from '@theia/core/lib/browser/clipboard-service';
-import { OutputChannelManager, OutputChannel } from '../common/output-channel';
+import { OutputChannelManager, OutputChannel } from './output-channel';
 import { OutputCommands } from './output-commands';
 import { QuickPickService } from '@theia/core/lib/common/quick-pick-service';
 

packages/output/src/browser/output-frontend-module.ts

@@ -19,8 +19,8 @@ import { OutputWidget, OUTPUT_WIDGET_KIND } from './output-widget';
 import { TabBarToolbarContribution } from '@theia/core/lib/browser/shell/tab-bar-toolbar';
 import { ResourceResolver } from '@theia/core/lib/common';
 import { WidgetFactory, bindViewContribution, OpenHandler } from '@theia/core/lib/browser';
-import { OutputChannelManager } from '../common/output-channel';
-import { bindOutputPreferences } from '../common/output-preferences';
+import { OutputChannelManager } from './output-channel';
+import { bindOutputPreferences } from './output-preferences';
 import { OutputToolbarContribution } from './output-toolbar-contribution';
 import { OutputContribution } from './output-contribution';
 import { MonacoEditorFactory } from '@theia/monaco/lib/browser/monaco-editor-provider';

packages/output/src/common/output-preferences.ts packages/output/src/browser/output-preferences.ts

@@ -15,14 +15,12 @@
  ********************************************************************************/
 
 import { interfaces } from '@theia/core/shared/inversify';
-
 import {
-    createPreferenceProxy,
+    PreferenceContribution,
     PreferenceProxy,
+    PreferenceSchema,
     PreferenceService,
-    PreferenceContribution,
-    PreferenceSchema
-    // eslint-disable-next-line @theia/runtime-import-check
+    createPreferenceProxy
 } from '@theia/core/lib/browser/preferences';
 
 export const OutputConfigSchema: PreferenceSchema = {

packages/output/src/browser/output-toolbar-contribution.tsx

@@ -21,7 +21,7 @@ import { TabBarToolbarContribution, TabBarToolbarRegistry } from '@theia/core/li
 import { OutputWidget } from './output-widget';
 import { OutputCommands } from './output-commands';
 import { OutputContribution } from './output-contribution';
-import { OutputChannelManager } from '../common/output-channel';
+import { OutputChannelManager } from './output-channel';
 
 @injectable()
 export class OutputToolbarContribution implements TabBarToolbarContribution {

packages/output/src/browser/output-widget.ts

@@ -25,7 +25,7 @@ import { MonacoEditorProvider } from '@theia/monaco/lib/browser/monaco-editor-pr
 import { Disposable, DisposableCollection } from '@theia/core/lib/common/disposable';
 import { Message, BaseWidget, DockPanel, Widget, MessageLoop, StatefulWidget, codicon } from '@theia/core/lib/browser';
 import { OutputUri } from '../common/output-uri';
-import { OutputChannelManager, OutputChannel } from '../common/output-channel';
+import { OutputChannelManager, OutputChannel } from './output-channel';
 import { Emitter, Event, deepClone } from '@theia/core';
 
 @injectable()

packages/plugin-dev/src/browser/hosted-plugin-log-viewer.ts

@@ -15,7 +15,7 @@
  ********************************************************************************/
 
 import { injectable, inject, postConstruct } from '@theia/core/shared/inversify';
-import { OutputChannel, OutputChannelManager } from '@theia/output/lib/common/output-channel';
+import { OutputChannel, OutputChannelManager } from '@theia/output/lib/browser/output-channel';
 import { OutputContribution } from '@theia/output/lib/browser/output-contribution';
 import { LogPart } from '@theia/plugin-ext/lib/common/types';
 import { HostedPluginWatcher } from '@theia/plugin-ext/lib/hosted/browser/hosted-plugin-watcher';

packages/plugin-ext/src/main/browser/debug/debug-main.ts

@@ -38,7 +38,7 @@ import { DebugProtocol } from 'vscode-debugprotocol';
 import { DebugConfigurationManager } from '@theia/debug/lib/browser/debug-configuration-manager';
 import { TerminalService } from '@theia/terminal/lib/browser/base/terminal-service';
 import { MessageClient } from '@theia/core/lib/common/message-service-protocol';
-import { OutputChannelManager } from '@theia/output/lib/common/output-channel';
+import { OutputChannelManager } from '@theia/output/lib/browser/output-channel';
 import { DebugPreferences } from '@theia/debug/lib/browser/debug-preferences';
 import { PluginDebugAdapterContribution } from './plugin-debug-adapter-contribution';
 import { PluginDebugSessionContributionRegistrator, PluginDebugSessionContributionRegistry } from './plugin-debug-session-contribution-registry';

packages/plugin-ext/src/main/browser/debug/plugin-debug-session-factory.ts

@@ -20,7 +20,7 @@ import { EditorManager } from '@theia/editor/lib/browser/editor-manager';
 import { BreakpointManager } from '@theia/debug/lib/browser/breakpoint/breakpoint-manager';
 import { LabelProvider } from '@theia/core/lib/browser/label-provider';
 import { MessageClient } from '@theia/core/lib/common/message-service-protocol';
-import { OutputChannelManager } from '@theia/output/lib/common/output-channel';
+import { OutputChannelManager } from '@theia/output/lib/browser/output-channel';
 import { DebugPreferences } from '@theia/debug/lib/browser/debug-preferences';
 import { DebugSessionOptions } from '@theia/debug/lib/browser/debug-session-options';
 import { DebugSession } from '@theia/debug/lib/browser/debug-session';

packages/plugin-ext/src/main/browser/webview/webview.ts

@@ -40,7 +40,7 @@ import { Schemes } from '../../../common/uri-components';
 import { PluginSharedStyle } from '../plugin-shared-style';
 import { WebviewThemeDataProvider } from './webview-theme-data-provider';
 import { ExternalUriService } from '@theia/core/lib/browser/external-uri-service';
-import { OutputChannelManager } from '@theia/output/lib/common/output-channel';
+import { OutputChannelManager } from '@theia/output/lib/browser/output-channel';
 import { WebviewPreferences } from './webview-preferences';
 import { WebviewResourceCache } from './webview-resource-cache';
 import { Endpoint } from '@theia/core/lib/browser/endpoint';