Put Arduino libs on top of the Library Manager

arduino-ide-extension/src/browser/library/library-list-widget.ts

@@ -189,6 +189,32 @@ export class LibraryListWidget extends ListWidget<
       { timeout: 3000 }
     );
   }
+
+  protected override filterableListSort(
+    items: LibraryPackage[]
+  ): LibraryPackage[] {
+    const isArduinoMaintainedComparator = (
+      left: LibraryPackage,
+      right: LibraryPackage
+    ) => {
+      if (left.isArduinoMaintained && !right.isArduinoMaintained) {
+        return -1;
+      }
+
+      if (!left.isArduinoMaintained && right.isArduinoMaintained) {
+        return 1;
+      }
+
+      return 0;
+    };
+
+    return items.sort((left, right) => {
+      return (
+        isArduinoMaintainedComparator(left, right) ||
+        this.defaultSortComparator(left, right)
+      );
+    });
+  }
 }
 
 class MessageBoxDialog extends AbstractDialog<MessageBoxDialog.Result> {

arduino-ide-extension/src/browser/widgets/component-list/filterable-list-container.tsx

@@ -111,19 +111,7 @@ export class FilterableListContainer<
     const { searchable } = this.props;
     searchable
       .search(searchOptions)
-      .then((items) => this.setState({ items: this.sort(items) }));
-  }
-
-  protected sort(items: T[]): T[] {
-    const { itemLabel, itemDeprecated } = this.props;
-    return items.sort((left, right) => {
-      // always put deprecated items at the bottom of the list
-      if (itemDeprecated(left)) {
-        return 1;
-      }
-
-      return itemLabel(left).localeCompare(itemLabel(right));
-    });
+      .then((items) => this.setState({ items: this.props.sort(items) }));
   }
 
   protected async install(
@@ -139,7 +127,7 @@ export class FilterableListContainer<
       run: ({ progressId }) => install({ item, progressId, version }),
     });
     const items = await searchable.search(this.state.searchOptions);
-    this.setState({ items: this.sort(items) });
+    this.setState({ items: this.props.sort(items) });
   }
 
   protected async uninstall(item: T): Promise<void> {
@@ -167,7 +155,7 @@ export class FilterableListContainer<
       run: ({ progressId }) => uninstall({ item, progressId }),
     });
     const items = await searchable.search(this.state.searchOptions);
-    this.setState({ items: this.sort(items) });
+    this.setState({ items: this.props.sort(items) });
   }
 }
 
@@ -204,6 +192,7 @@ export namespace FilterableListContainer {
       progressId: string;
     }) => Promise<void>;
     readonly commandService: CommandService;
+    readonly sort: (items: T[]) => T[];
   }
 
   export interface State<T, S extends Searchable.Options> {

arduino-ide-extension/src/browser/widgets/component-list/list-widget.tsx

@@ -51,9 +51,11 @@ export abstract class ListWidget<
    */
   protected firstActivate = true;
 
+  protected readonly defaultSortComparator: (left: T, right: T) => number;
+
   constructor(protected options: ListWidget.Options<T, S>) {
     super();
-    const { id, label, iconClass } = options;
+    const { id, label, iconClass, itemDeprecated, itemLabel } = options;
     this.id = id;
     this.title.label = label;
     this.title.caption = label;
@@ -63,6 +65,17 @@ export abstract class ListWidget<
     this.node.tabIndex = 0; // To be able to set the focus on the widget.
     this.scrollOptions = undefined;
     this.toDispose.push(this.searchOptionsChangeEmitter);
+
+    this.defaultSortComparator = (left, right): number => {
+      // always put deprecated items at the bottom of the list
+      if (itemDeprecated(left)) {
+        return 1;
+      }
+
+      return itemLabel(left).localeCompare(itemLabel(right));
+    };
+
+    this.filterableListSort = this.filterableListSort.bind(this);
   }
 
   @postConstruct()
@@ -128,6 +141,10 @@ export abstract class ListWidget<
     return this.options.installable.uninstall({ item, progressId });
   }
 
+  protected filterableListSort(items: T[]): T[] {
+    return items.sort(this.defaultSortComparator);
+  }
+
   render(): React.ReactNode {
     return (
       <FilterableListContainer<T, S>
@@ -145,6 +162,7 @@ export abstract class ListWidget<
         messageService={this.messageService}
         commandService={this.commandService}
         responseService={this.responseService}
+        sort={this.filterableListSort}
       />
     );
   }

arduino-ide-extension/src/common/protocol/arduino-component.ts

@@ -10,6 +10,7 @@ export interface ArduinoComponent {
   readonly availableVersions: Installable.Version[];
   readonly installable: boolean;
   readonly installedVersion?: Installable.Version;
+  readonly isArduinoMaintained?: boolean;
   /**
    * This is the `Type` in IDE (1.x) UI.
    */

arduino-ide-extension/src/node/library-service-impl.ts

@@ -456,5 +456,6 @@ function toLibrary(
     summary: lib.getParagraph(),
     category: lib.getCategory(),
     types: lib.getTypesList(),
+    isArduinoMaintained: lib.getAuthor() === 'Arduino', // TODO check if .getMaintainer is more appropriate
   };
 }