[maintainability] Cycles found in extension's code

[marcdumais-work]

After updating to a more recent version of Eslint, the linter started to report an issue with ProgressMessages in vscode-trace-extension/src/trace-explorer/trace-tree.ts having already been defined. After scratching my head for a while, I thought it could be a "cycle" issue. That turned-out to apparently be correct: see yellow lines in this graph, generated using "dependency-cruiser" [1]. The worst of it seems centered around extension.ts

Dependency graph (look at the yellow lines):

For info about the configuration I used and how to run dependency-cruiser, see this commit on my fork

[marcdumais-work]

Text version of the "internal circular dependencies":

  warn no-circular: vscode-trace-extension/src/trace-explorer/trace-tree.ts → 
      vscode-trace-extension/src/trace-viewer-panel/trace-viewer-webview-panel.ts
      →
      vscode-trace-extension/src/extension.ts →
      vscode-trace-extension/src/trace-explorer/trace-tree.ts
  warn no-circular: vscode-trace-extension/src/trace-explorer/trace-tree.ts → 
      vscode-trace-extension/src/trace-explorer/analysis-tree.ts →
      vscode-trace-extension/src/trace-viewer-panel/trace-viewer-webview-panel.ts
      →
      vscode-trace-extension/src/extension.ts →
      vscode-trace-extension/src/trace-explorer/trace-tree.ts
  warn no-circular: vscode-trace-extension/src/trace-explorer/opened-traces/trace-explorer-opened-traces-webview-provider.ts → 
      vscode-trace-extension/src/trace-viewer-panel/trace-viewer-webview-panel.ts
      →
      vscode-trace-extension/src/extension.ts →
      vscode-trace-extension/src/trace-explorer/opened-traces/trace-explorer-opened-traces-webview-provider.ts
  warn no-circular: vscode-trace-extension/src/trace-explorer/available-views/trace-explorer-available-views-webview-provider.ts → 
      vscode-trace-extension/src/trace-viewer-panel/trace-viewer-webview-panel.ts
      →
      vscode-trace-extension/src/extension.ts →
      vscode-trace-extension/src/trace-explorer/available-views/trace-explorer-available-views-webview-provider.ts
  warn no-circular: vscode-trace-extension/src/external-api/external-api.ts → 
      vscode-trace-extension/src/trace-viewer-panel/trace-viewer-webview-panel.ts
      →
      vscode-trace-extension/src/extension.ts →
      vscode-trace-extension/src/external-api/external-api.ts
  warn no-circular: vscode-trace-extension/src/extension.ts → 
      vscode-trace-extension/src/trace-viewer-panel/trace-viewer-webview-panel.ts
      →
      vscode-trace-extension/src/extension.ts
  warn no-circular: vscode-trace-extension/src/extension.ts → 
      vscode-trace-extension/src/trace-explorer/trace-tree.ts →
      vscode-trace-extension/src/extension.ts
  warn no-circular: vscode-trace-extension/src/extension.ts → 
      vscode-trace-extension/src/trace-explorer/properties/trace-explorer-properties-view-webview-provider.ts
      →
      vscode-trace-extension/src/extension.ts
  warn no-circular: vscode-trace-extension/src/extension.ts → 
      vscode-trace-extension/src/trace-explorer/opened-traces/trace-explorer-opened-traces-webview-provider.ts
      →
      vscode-trace-extension/src/extension.ts
  warn no-circular: vscode-trace-extension/src/extension.ts → 
      vscode-trace-extension/src/trace-explorer/available-views/trace-explorer-available-views-webview-provider.ts
      →
      vscode-trace-extension/src/extension.ts
  warn no-circular: vscode-trace-extension/src/extension.ts → 
      vscode-trace-extension/src/trace-explorer/analysis-tree.ts →
      vscode-trace-extension/src/trace-viewer-panel/trace-viewer-webview-panel.ts
      →
      vscode-trace-extension/src/extension.ts
  warn no-circular: vscode-trace-extension/src/extension.ts → 
      vscode-trace-extension/src/external-api/external-api.ts →
      vscode-trace-extension/src/extension.ts