Fix addShortcutToLeftSidebar, use new API (Closes #509) (#511)

* Refactor addShortcutToLeftSidebar function to use Roam API for adding current page shortcuts. Update command label for clarity.

* 1.7.3

Author: mdroidian

package-lock.json

@@ -23,7 +23,7 @@
     "@types/mozilla-readability": "^0.2.0",
     "@types/turndown": "^5.0.1"
   },
-  "version": "1.7.2",
+  "version": "1.7.3",
   "samepage": {
     "extends": "node_modules/roamjs-components/package.json"
   }

package.json

@@ -478,22 +478,12 @@ const toggleQueries = () =>
     .forEach((element) => {
       element.click();
     });
-const addShortcutToLeftSidebar = () => {
-  const previousElement = document.activeElement as HTMLElement;
-  const emptyShortcuts = document.getElementsByClassName(
-    "bp3-button bp3-icon-star-empty"
-  ) as HTMLCollectionOf<HTMLSpanElement>;
-  const shortcuts = document.getElementsByClassName(
-    "bp3-button bp3-icon-star"
-  ) as HTMLCollectionOf<HTMLSpanElement>;
-  if (emptyShortcuts.length) {
-    emptyShortcuts[0].click();
-    previousElement?.focus();
-  } else if (shortcuts.length) {
-    shortcuts[0]?.click();
-    previousElement?.focus();
-  }
-};
+const addShortcutToLeftSidebar = () =>
+  getCurrentPageUid().then((uid) =>
+    window.roamAlphaAPI.data.page
+      // @ts-expect-error - addShortcut is not yet in roamjs-components types
+      .addShortcut(uid, 0)
+  );
 const pasteBlockWithChildrenAsReferences = () => {
   const uid = window.roamAlphaAPI.ui.getFocusedBlock()?.["block-uid"];
   if (uid) {
@@ -630,7 +620,7 @@ const commands = [
   { label: "Collapse all blocks on page", callback: collapseAllBlocksOnPage },
   { label: "Open this page in sidebar", callback: openPageInSidebar },
   {
-    label: "Add shortcut to page to left sidebar",
+    label: "Add current page to shortcuts",
     callback: addShortcutToLeftSidebar,
   },
   { label: "Toggle Linked Refs", callback: toggleLinkedRefs },