🧰: update partsbin when component browser gets opened the first time

LivelyKernel · Aug 14, 2023 · eab3beb · eab3beb
1 parent f87d64c
commit eab3beb
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 7 deletions.
diff --git a/lively.ide/studio/component-browser.cp.js b/lively.ide/studio/component-browser.cp.js
@@ -1,7 +1,7 @@
 import { pt, Color, rect } from 'lively.graphics';
 import { TilingLayout, morph, config, easings, MorphicDB, Icon, Morph, Label, ShadowObject, ViewModel, add, part, component } from 'lively.morphic';
 import { Project } from 'lively.project';
-import { PackageRegistry, isModuleLoaded, module, importPackage } from 'lively.modules';
+import { isModuleLoaded, module } from 'lively.modules';
 import { InputLineDefault } from 'lively.components/inputs.cp.js';
 import { MullerColumnView, ColumnListDark, ColumnListDefault } from 'lively.components/muller-columns.cp.js';
 import { TreeData } from 'lively.components';
@@ -15,13 +15,14 @@ import { DropDownList, DarkDropDownList } from 'lively.components/list.cp.js';
 import { withAllViewModelsDo } from 'lively.morphic/components/policy.js';
 import { ButtonDarkDefault, SystemButton } from 'lively.components/buttons.cp.js';
 import { Text } from 'lively.morphic/text/morph.js';
-
 import { COLORS } from '../js/browser/index.js';
 import { Spinner, CheckboxInactive, CheckboxActive, LabeledCheckbox, DarkPopupWindow } from './shared.cp.js';
 import { InteractiveComponentDescriptor } from '../components/editor.js';
 import { PopupWindow, SystemList } from '../styling/shared.cp.js';
-
 import { joinPath } from 'lively.lang/string.js';
+import { runCommand } from 'lively.shell/client-command.js';
+import ShellClientResource from 'lively.shell/client-resource.js';
+import { StatusMessageError, StatusMessageConfirm } from 'lively.halos/components/messages.cp.js';
 
 class MasterComponentTreeData extends TreeData {
   /**
@@ -760,8 +761,8 @@ export class ComponentBrowserModel extends ViewModel {
       }],
       [[...this.importAlive ? checked : unchecked, ' Enable behavior'], () => this.importAlive = !this.importAlive],
       ['Group Components by ', [
-        [[...this.groupBy == 'module' ? checked : unchecked, ' Modules'], () => { this.groupBy = 'module'; }],
-        [[...this.groupBy == 'name' ? checked : unchecked, ' Names'], () => { this.groupBy = 'name'; }]
+        [[...this.groupBy === 'module' ? checked : unchecked, ' Modules'], () => { this.groupBy = 'module'; }],
+        [[...this.groupBy === 'name' ? checked : unchecked, ' Names'], () => { this.groupBy = 'name'; }]
       ]]
     ];
   }
@@ -836,12 +837,26 @@ export class ComponentBrowserModel extends ViewModel {
     }
   }
 
-  viewDidLoad () {
+  async viewDidLoad () {
     if (!this.view.isComponent) {
       this.view.withMetaDo({ metaInteraction: true }, () => {
         this.ui.componentFilesView.setTreeData(new MasterComponentTreeData({ browser: this }));
       });
     }
+    const openedProject = $world.openedProject;
+    if (!(openedProject.owner === 'LivelyKernel' && openedProject.name === 'partsbin')) {
+      const li = $world.showLoadingIndicatorFor(null, 'Updating `partsbin`');
+      // This relies on the assumption, that the default directory the shell command gets dropped in is `lively.server`.
+      // `install.sh` ensures that the partsbin repository exists.
+      const cmd = runCommand('cd ../local_projects/LivelyKernel--partsbin && git stash && git checkout main && git pull', { l2lClient: ShellClientResource.defaultL2lClient });
+      await cmd.whenDone();
+      if (cmd.exitCode !== 0) {
+        $world.setStatusMessage('`partsbin` could not be updated.', StatusMessageError);
+        return;
+      }
+      $world.setStatusMessage('`partsbin` updated!', StatusMessageConfirm);
+      li.remove();
+    }
   }
 
   async refresh () {

diff --git a/lively.ide/world.js b/lively.ide/world.js
@@ -1207,7 +1207,7 @@ export class LivelyWorld extends World {
    */
   showLoadingIndicatorFor (requester, label) {
     const li = LoadingIndicator.open(label, { target: requester });
-    if (requester.hasFixedPosition) li.hasFixedPosition = true;
+    if (requester?.hasFixedPosition) li.hasFixedPosition = true;
     return li;
   }