Use ICodeMirror, more type imports, order plugin for fixer

.github/workflows/job.test.yml

@@ -59,9 +59,8 @@ jobs:
             ${{ env.CACHE_EPOCH }}-${{ matrix.os }}-lint-
 
       - name: Set up Python and mamba
-        uses: conda-incubator/setup-miniconda@master
+        uses: conda-incubator/setup-miniconda@v2
         with:
-          condarc-file: .github/.condarc
           environment-file: requirements/github-actions.yml
           miniforge-variant: Mambaforge
           use-mamba: true
@@ -108,6 +107,9 @@ jobs:
       - name: Lint frontend
         run: jlpm lint:check
 
+      - name: Check distributions
+        run: python scripts/distcheck.py
+
   build:
     name: build
     runs-on: ${{ matrix.os }}
@@ -138,9 +140,8 @@ jobs:
             ${{ env.CACHE_EPOCH }}-${{ matrix.os }}-build-
 
       - name: Set up Python and mamba
-        uses: conda-incubator/setup-miniconda@master
+        uses: conda-incubator/setup-miniconda@v2
         with:
-          condarc-file: .github/.condarc
           miniforge-variant: Mambaforge
 
       - name: Install minimal build deps
@@ -253,9 +254,8 @@ jobs:
             ${{ env.CACHE_EPOCH }}-${{ matrix.os }}-${{ matrix.python }}-atest-
 
       - name: Set up Python and mamba
-        uses: conda-incubator/setup-miniconda@master
+        uses: conda-incubator/setup-miniconda@v2
         with:
-          condarc-file: .github/.condarc
           environment-file: requirements/github-actions.yml
           miniforge-variant: Mambaforge
           use-mamba: true

package.json

@@ -25,8 +25,8 @@
     "vscode-css-languageserver-bin": "^1.4.0",
     "vscode-html-languageserver-bin": "^1.4.0",
     "vscode-json-languageserver-bin": "^1.0.1",
-    "yaml-language-server": "^0.12.0",
-    "vscode-json-languageservice": "^3.9.1 <3.10.0"
+    "vscode-json-languageservice": "^3.9.1 <3.10.0",
+    "yaml-language-server": "^0.12.0"
   },
   "husky": {
     "hooks": {}
@@ -38,7 +38,7 @@
   },
   "private": true,
   "scripts": {
-    "bootstrap": "jlpm --no-optional --prefer-offline && lerna bootstrap && jlpm lint && jlpm clean && jlpm build",
+    "bootstrap": "jlpm --no-optional --prefer-offline && lerna bootstrap && jlpm clean && jlpm build && jlpm lint",
     "build": "jlpm build:schema && jlpm build:meta && jlpm build:ws && jlpm build:labextension",
     "build:schema": "lerna run build:schema --stream",
     "build:meta": "lerna run build --stream --scope @krassowski/jupyterlab-lsp-metapackage",

packages/.eslintrc.js

@@ -10,6 +10,9 @@ module.exports = {
   root: true,
   extends: [
     'eslint:recommended',
+    'plugin:import/errors',
+    'plugin:import/warnings',
+    'plugin:import/typescript',
     'plugin:@typescript-eslint/eslint-recommended',
     'plugin:@typescript-eslint/recommended',
     'prettier/@typescript-eslint',
@@ -55,12 +58,32 @@ module.exports = {
     'no-undef': 'warn',
     'no-useless-escape': 'off',
     'prefer-const': 'off',
-    // deviations from jupyterlab, should not be changed
-    // a pitfall of enums is that they do not work correctly
-    // when circular dependencies are present
-    // (see https://stackoverflow.com/a/59665223/)
-    'import/no-cycle': 'error',
+    // the default, but for reference...
+    'import/order': [
+      'warn',
+      {
+        groups: [
+          'builtin',
+          'external',
+          'parent',
+          'sibling',
+          'index',
+          'object',
+          'unknown'
+        ],
+        pathGroups: [
+          { pattern: 'react/**', group: 'builtin', order: 'after' },
+          { pattern: 'codemirror/**', group: 'external', order: 'before' },
+          { pattern: '@lumino/**', group: 'external', order: 'before' },
+          { pattern: '@jupyterlab/**', group: 'external', order: 'after' }
+        ],
+        'newlines-between': 'always',
+        alphabetize: { order: 'asc' }
+      }
+    ],
     // deviations from jupyterlab, should probably be fixed
+    'import/no-cycle': 'off', // somehow we lapsed here... ~200 cycles now
+    'import/export': 'off', // we do class/interface + NS pun exports _all over_
     '@typescript-eslint/triple-slash-reference': 'off',
     'jest/no-test-callback': 'off',
     'jest/valid-expect': 'off',

packages/_example-extractor/src/api.spec.ts

@@ -1,9 +1,8 @@
+import { IExtractedCode } from '@krassowski/jupyterlab-lsp';
 import { expect } from 'chai';
 
 import { extractor } from '.';
 
-import { IExtractedCode } from '@krassowski/jupyterlab-lsp';
-
 const EXAMPLE = `%%foo
 bar
 `;

packages/_example-extractor/src/index.ts

@@ -2,7 +2,6 @@ import {
   JupyterFrontEnd,
   JupyterFrontEndPlugin
 } from '@jupyterlab/application';
-
 import {
   ILSPCodeExtractorsManager,
   RegExpForeignCodeExtractor

packages/code-jumpers/src/history.ts

@@ -1,7 +1,8 @@
-import { IGlobalPosition } from './positions';
 import { IModelDB, IObservableUndoableList } from '@jupyterlab/observables';
 import { JSONValue } from '@lumino/coreutils';
 
+import { IGlobalPosition } from './positions';
+
 const DB_ENTRY = 'jumpy_history';
 
 export class JumpHistory {

packages/code-jumpers/src/index.ts

@@ -1,5 +1,5 @@
+import { FileEditorJumper } from './jumpers/fileeditor';
 import { CodeJumper } from './jumpers/jumper';
 import { NotebookJumper } from './jumpers/notebook';
-import { FileEditorJumper } from './jumpers/fileeditor';
 
 export { CodeJumper, NotebookJumper, FileEditorJumper };

packages/code-jumpers/src/jumpers/fileeditor.ts

@@ -1,10 +1,12 @@
+import { CodeEditor } from '@jupyterlab/codeeditor';
+import { IDocumentManager } from '@jupyterlab/docmanager';
+import { IDocumentWidget } from '@jupyterlab/docregistry';
 import { FileEditor } from '@jupyterlab/fileeditor';
+
+import { JumpHistory } from '../history';
 import { IGlobalPosition, ILocalPosition } from '../positions';
+
 import { CodeJumper, jumpers } from './jumper';
-import { JumpHistory } from '../history';
-import { IDocumentManager } from '@jupyterlab/docmanager';
-import { IDocumentWidget } from '@jupyterlab/docregistry';
-import { CodeEditor } from '@jupyterlab/codeeditor';
 
 export class FileEditorJumper extends CodeJumper {
   editor: FileEditor;

packages/code-jumpers/src/jumpers/jumper.ts

@@ -1,12 +1,13 @@
+import { Dialog, showDialog } from '@jupyterlab/apputils';
 import { CodeEditor } from '@jupyterlab/codeeditor';
-
-import { IGlobalPosition, ILocalPosition } from '../positions';
 import { IDocumentManager } from '@jupyterlab/docmanager';
 import { IDocumentWidget } from '@jupyterlab/docregistry';
-import { JumpHistory } from '../history';
 import { FileEditor } from '@jupyterlab/fileeditor';
+
+import { JumpHistory } from '../history';
+import { IGlobalPosition, ILocalPosition } from '../positions';
+
 import IEditor = CodeEditor.IEditor;
-import { Dialog, showDialog } from '@jupyterlab/apputils';
 
 const movement_keys = [
   'ArrowRight',

packages/code-jumpers/src/jumpers/notebook.ts

@@ -1,11 +1,12 @@
-import { Notebook, NotebookPanel } from '@jupyterlab/notebook';
+import { CodeEditor } from '@jupyterlab/codeeditor';
 import { IDocumentManager } from '@jupyterlab/docmanager';
+import { Notebook, NotebookPanel } from '@jupyterlab/notebook';
 
-import { CodeJumper, jumpers } from './jumper';
-import { IGlobalPosition, ILocalPosition } from '../positions';
-import { _ensureFocus } from '../notebook_private';
 import { JumpHistory } from '../history';
-import { CodeEditor } from '@jupyterlab/codeeditor';
+import { _ensureFocus } from '../notebook_private';
+import { IGlobalPosition, ILocalPosition } from '../positions';
+
+import { CodeJumper, jumpers } from './jumper';
 
 export class NotebookJumper extends CodeJumper {
   notebook: Notebook;

packages/completion-theme/src/about.tsx

@@ -1,10 +1,11 @@
+import { LabIcon } from '@jupyterlab/ui-components';
 import React, { ReactElement } from 'react';
+
 import {
+  COMPLETER_THEME_PREFIX,
   ICompletionTheme,
-  ILicenseInfo,
-  COMPLETER_THEME_PREFIX
+  ILicenseInfo
 } from './types';
-import { LabIcon } from '@jupyterlab/ui-components';
 
 function render_licence(licence: ILicenseInfo): ReactElement {
   return (

packages/completion-theme/src/index.ts

@@ -1,22 +1,23 @@
-import { kernelIcon, LabIcon } from '@jupyterlab/ui-components';
+import '../style/index.css';
+import { JupyterFrontEndPlugin } from '@jupyterlab/application';
 import {
   Dialog,
   ICommandPalette,
   IThemeManager,
   showDialog
 } from '@jupyterlab/apputils';
-import { JupyterFrontEndPlugin } from '@jupyterlab/application';
+import { LabIcon, kernelIcon } from '@jupyterlab/ui-components';
+
+import { render_themes_list } from './about';
 import {
+  COMPLETER_THEME_PREFIX,
+  CompletionItemKindStrings,
   ICompletionIconSet,
   ICompletionTheme,
   ILSPCompletionThemeManager,
-  PLUGIN_ID,
-  COMPLETER_THEME_PREFIX,
   KernelKind,
-  CompletionItemKindStrings
+  PLUGIN_ID
 } from './types';
-import { render_themes_list } from './about';
-import '../style/index.css';
 
 export class CompletionThemeManager implements ILSPCompletionThemeManager {
   protected current_icons: Map<string, LabIcon>;

packages/completion-theme/src/types.ts

@@ -1,5 +1,5 @@
-import { Token } from '@lumino/coreutils';
 import { LabIcon } from '@jupyterlab/ui-components';
+import { Token } from '@lumino/coreutils';
 
 export const COMPLETER_THEME_PREFIX = 'lsp-completer-theme-';
 

packages/jupyterlab-lsp/src/adapter_manager.ts

@@ -1,15 +1,17 @@
+import type { IClassicShell } from '@jupyterlab-classic/application';
+import { ILabShell, JupyterFrontEndPlugin } from '@jupyterlab/application';
+import { IDocumentWidget } from '@jupyterlab/docregistry';
+import { Signal } from '@lumino/signaling';
+
+import { WidgetAdapter } from './adapters/adapter';
 import {
   IAdapterRegistration,
   IAdapterTypeOptions,
   ILSPAdapterManager,
   PLUGIN_ID
 } from './tokens';
-import { Signal } from '@lumino/signaling';
-import { IDocumentWidget } from '@jupyterlab/docregistry';
-import { WidgetAdapter } from './adapters/adapter';
-import { ILabShell, JupyterFrontEndPlugin } from '@jupyterlab/application';
+
 import { LSPExtension } from './index';
-import type { IClassicShell } from '@jupyterlab-classic/application';
 
 export class WidgetAdapterManager implements ILSPAdapterManager {
   adapterTypeAdded: Signal<

packages/jupyterlab-lsp/src/adapters/adapter.ts

@@ -1,23 +1,25 @@
 import { JupyterFrontEnd } from '@jupyterlab/application';
 import { CodeEditor } from '@jupyterlab/codeeditor';
 import { DocumentRegistry, IDocumentWidget } from '@jupyterlab/docregistry';
-import { IVirtualEditor } from '../virtual/editor';
-import { IForeignContext, VirtualDocument } from '../virtual/document';
+import { JSONObject } from '@lumino/coreutils';
 import { Signal } from '@lumino/signaling';
-import { IRootPosition, IVirtualPosition } from '../positioning';
-import { LSPConnection } from '../connection';
+
 import { ICommandContext } from '../command_manager';
-import { JSONObject } from '@lumino/coreutils';
+import { LSPConnection } from '../connection';
 import {
   DocumentConnectionManager,
   IDocumentConnectionData,
   ISocketConnectionOptions
 } from '../connection_manager';
-import { ILSPExtension, ILSPLogConsole } from '../index';
-import { IFeatureEditorIntegration, IFeature } from '../feature';
 import { EditorAdapter } from '../editor_integration/editor_adapter';
-import IEditor = CodeEditor.IEditor;
+import { IFeature, IFeatureEditorIntegration } from '../feature';
+import { ILSPExtension, ILSPLogConsole } from '../index';
 import { LanguageIdentifier } from '../lsp';
+import { IRootPosition, IVirtualPosition } from '../positioning';
+import { IForeignContext, VirtualDocument } from '../virtual/document';
+import { IVirtualEditor } from '../virtual/editor';
+
+import IEditor = CodeEditor.IEditor;
 
 export class StatusMessage {
   /**

packages/jupyterlab-lsp/src/adapters/file_editor/file_editor.ts

@@ -1,15 +1,17 @@
-import { WidgetAdapter } from '../adapter';
-import { FileEditor } from '@jupyterlab/fileeditor';
-import { IDocumentWidget } from '@jupyterlab/docregistry';
-import { CodeMirrorEditor } from '@jupyterlab/codemirror';
 import { CodeEditor } from '@jupyterlab/codeeditor';
-import { LSPExtension } from '../../index';
+import { CodeMirrorEditor } from '@jupyterlab/codemirror';
+import { IDocumentWidget } from '@jupyterlab/docregistry';
+import { FileEditor } from '@jupyterlab/fileeditor';
+
+import { ICommandContext } from '../../command_manager';
 import { PositionConverter } from '../../converter';
+import { LSPExtension } from '../../index';
 import { IRootPosition, IVirtualPosition } from '../../positioning';
-import { ICommandContext } from '../../command_manager';
+import { VirtualDocument } from '../../virtual/document';
 import { IVirtualEditor } from '../../virtual/editor';
+import { WidgetAdapter } from '../adapter';
+
 import IEditor = CodeEditor.IEditor;
-import { VirtualDocument } from '../../virtual/document';
 
 export class FileEditorAdapter extends WidgetAdapter<
   IDocumentWidget<FileEditor>

packages/jupyterlab-lsp/src/adapters/file_editor/index.ts

@@ -1,9 +1,11 @@
 import { JupyterFrontEndPlugin } from '@jupyterlab/application';
-import { ILSPAdapterManager, PLUGIN_ID } from '../../tokens';
-import { IEditorTracker } from '@jupyterlab/fileeditor';
-import { FileEditorAdapter } from './file_editor';
 import { IDocumentWidget } from '@jupyterlab/docregistry';
+import { IEditorTracker } from '@jupyterlab/fileeditor';
+
 import { CommandEntryPoint } from '../../command_manager';
+import { ILSPAdapterManager, PLUGIN_ID } from '../../tokens';
+
+import { FileEditorAdapter } from './file_editor';
 
 export const FileEditorContextMenuEntryPoint: CommandEntryPoint =
   'file-editor-context-menu';

packages/jupyterlab-lsp/src/adapters/notebook/index.ts

@@ -1,13 +1,15 @@
-import { CommandEntryPoint } from '../../command_manager';
 import { JupyterFrontEndPlugin } from '@jupyterlab/application';
+import { IDocumentWidget } from '@jupyterlab/docregistry';
+import { INotebookTracker } from '@jupyterlab/notebook';
+
+import { CommandEntryPoint } from '../../command_manager';
 import {
   IAdapterTypeOptions,
   ILSPAdapterManager,
   PLUGIN_ID
 } from '../../tokens';
-import { INotebookTracker } from '@jupyterlab/notebook';
+
 import { NotebookAdapter } from './notebook';
-import { IDocumentWidget } from '@jupyterlab/docregistry';
 
 export const CellContextMenuEntryPoint: CommandEntryPoint =
   'notebook-cell-context-menu';