- Improved boardify SDK (API)

- Removed old plugin system that was using interfaces and used classes instead
- Removed tests for import and render registries (Need to be added later)

Improvements still must be made in terms of error catching as wrongly loaded plugin can make the entire application go dark.

Author: BluePixelDev

packages/app/src/AppLayout.tsx

@@ -9,7 +9,7 @@ import CSSSnippets from "./features/snippets/Snippets";
 import { Titlebar, ToolbarProvider } from "./features/titlebar";
 import { ModalContainer } from "./ui/modal";
 import { ThemeProvider } from "./features/theme";
-import { PluginProvider } from "./core/plugins";
+import { PluginProvider } from "./services/plugins";
 import { useAppDispatch, useAppSelector } from "./redux";
 
 interface AppLayoutProps {

packages/app/src/core/plugins/SdkInitializer.tsx

@@ -1,5 +1,5 @@
 import { useAppSelector } from "@/redux";
-import { rendererRegistry } from "@/core/renderers";
+import { rendererRegistry } from "@/services/renderer";
 import { ErrorBoundary } from "react-error-boundary";
 import { FallbackNode } from "./FallbackNode";
 import { NodeData, selectVisibleNodes } from "../..";
@@ -10,7 +10,7 @@ export function NodesRenderer() {
   return (
     <>
       {nodes.map((node: NodeData<unknown>) => {
-        const Renderer = rendererRegistry.getRenderer(node.type);
+        const Renderer = rendererRegistry.get(node.type);
         return (
           <ErrorBoundary
             key={node.id}

packages/app/src/core/renderers/renderRegistry.test.ts

@@ -2,7 +2,7 @@ import { configureStore } from "@reduxjs/toolkit";
 import { settingsReducer } from "@/features/settings/store";
 import appReducer from "@/redux/slices/appSlice";
 import { combineReducers } from "@reduxjs/toolkit";
-import pluginsSlice from "@/core/plugins/store";
+import pluginsSlice from "@/services/plugins/store";
 import notificationsReducer from "./slices/notificationsSlice";
 import snippetSlice from "@/features/snippets/store/snippetSlice";
 import keybindsSlice from "@/features/keybinds/redux/keybindsSlice";

packages/app/src/core/renderers/rendererRegistry.ts

@@ -1,35 +1,30 @@
-import {
-  ImporterEntry,
-  ImporterHandler,
-  ImporterRegistryInterface,
-  ImportEvent,
-  NoSuitableImporterError,
-} from "@boardify/sdk";
+import { ImporterEntry, ImporterHandler, ImportEvent } from "@boardify/sdk";
+import { NoSuitableImporterError } from "./types";
 
 /**
  * A registry to manage and prioritize data importers.
  */
-class ImporterRegistry implements ImporterRegistryInterface {
+class ImporterRegistry {
   private importers: ImporterEntry[] = [];
 
-  registerImporter(handler: ImporterHandler, priority = 0): void {
+  register(handler: ImporterHandler, priority = 0): void {
     if (!this.importers.some((entry) => entry.handler === handler)) {
       this.importers.push({ handler, priority });
       this.importers.sort((a, b) => b.priority - a.priority);
     }
   }
 
-  unregisterImporter(handler: ImporterHandler): void {
+  unregister(handler: ImporterHandler): void {
     this.importers = this.importers.filter(
       (entry) => entry.handler !== handler
     );
   }
 
-  clearImporters(): void {
+  clear(): void {
     this.importers = [];
   }
 
-  getImporters(): ImporterEntry[] {
+  get(): ImporterEntry[] {
     return this.importers;
   }
 

packages/app/src/features/board/components/nodes/NodesRenderer.tsx

@@ -1,8 +1,9 @@
 import { AppThunk } from "@/redux";
-import { importerRegistry } from "./ImporterRegistry";
 import { error } from "@tauri-apps/plugin-log";
 import { showErrorToast } from "@/features/notifications";
-import { ImportEvent, NoSuitableImporterError } from "@boardify/sdk";
+import { ImportEvent } from "@boardify/sdk";
+import { importerRegistry } from "./ImporterRegistry";
+import { NoSuitableImporterError } from "./types";
 
 export const importFile =
   (path: string, position: { x: number; y: number }, file: File): AppThunk =>

packages/app/src/redux/store.ts

@@ -0,0 +1,6 @@
+export class NoSuitableImporterError extends Error {
+  constructor(fileType: string, fileName: string) {
+    super(`No importer found for file type: ${fileType} (${fileName})`);
+    this.name = "NoSuitableImporterError";
+  }
+}