@malloy-publisher/sdk 0.0.87 → 0.0.88

package/src/components/Workbook/WorkbookList.tsx

@@ -8,7 +8,6 @@ import {
 } from "@mui/material";
 import React from "react";
 import { useWorkbookStorage } from "./WorkbookStorageProvider";
-import { usePackage } from "../Package";
 import { WorkbookLocator } from "./WorkbookStorage";
 
 interface WorkbookListProps {
@@ -20,7 +19,6 @@ interface WorkbookListProps {
 
 export function WorkbookList({ onWorkbookClick }: WorkbookListProps) {
    const { workbookStorage } = useWorkbookStorage();
-   const packageContext = usePackage();
    const [workbooks, setWorkbooks] = React.useState<WorkbookLocator[]>([]);
    const [lastError, setLastError] = React.useState<string | undefined>(
       undefined,
@@ -28,30 +26,28 @@ export function WorkbookList({ onWorkbookClick }: WorkbookListProps) {
 
    React.useEffect(() => {
       if (workbookStorage) {
-         workbookStorage
-            .listWorkspaces(packageContext, false)
-            .then((workspaces) => {
-               const allWorkbooks: WorkbookLocator[] = [];
-               Promise.all(
-                  workspaces.map(async (workspace) => {
-                     await workbookStorage
-                        .listWorkbooks(workspace, packageContext)
-                        .then((newWorkbooks) => {
-                           allWorkbooks.push(...newWorkbooks);
-                        })
-                        .catch((error) => {
-                           setLastError(
-                              `Error listing workbooks: ${error.message}`,
-                           );
-                        });
-                  }),
-               ).then(() => {
-                  setWorkbooks(allWorkbooks);
-                  setLastError(undefined);
-               });
+         workbookStorage.listWorkspaces(false).then((workspaces) => {
+            const allWorkbooks: WorkbookLocator[] = [];
+            Promise.all(
+               workspaces.map(async (workspace) => {
+                  await workbookStorage
+                     .listWorkbooks(workspace)
+                     .then((newWorkbooks) => {
+                        allWorkbooks.push(...newWorkbooks);
+                     })
+                     .catch((error) => {
+                        setLastError(
+                           `Error listing workbooks: ${error.message}`,
+                        );
+                     });
+               }),
+            ).then(() => {
+               setWorkbooks(allWorkbooks);
+               setLastError(undefined);
             });
+         });
       }
-   }, [workbookStorage, packageContext]);
+   }, [workbookStorage]);
 
    return (
       <>

package/src/components/Workbook/WorkbookManager.ts

@@ -1,4 +1,3 @@
-import { PackageContextProps } from "../Package";
 import type { WorkbookLocator, WorkbookStorage } from "./WorkbookStorage";
 
 /**
@@ -40,21 +39,17 @@ export interface WorkbookCellValue {
 export class WorkbookManager {
    private isSaved: boolean;
    private workbookStorage: WorkbookStorage;
-   private packageContext: PackageContextProps;
 
    /**
     * Creates a new WorkbookManager instance
     * @param {WorkbookStorage} workbookStorage - Storage implementation
-    * @param {PackageContextProps} packageContext - Package context for storage
     * @param {WorkbookData} workbookData - Initial workbook data
     */
    constructor(
       workbookStorage: WorkbookStorage,
-      packageContext: PackageContextProps,
       private workbookData: WorkbookData,
    ) {
       this.workbookStorage = workbookStorage;
-      this.packageContext = packageContext;
       if (this.workbookData) {
          this.isSaved = true;
       } else {
@@ -92,7 +87,6 @@ export class WorkbookManager {
       if (this.workbookData.workbookPath.path !== workbookPath) {
          try {
             await this.workbookStorage.moveWorkbook(
-               this.packageContext,
                this.workbookData.workbookPath,
                {
                   path: workbookPath,
@@ -154,17 +148,12 @@ export class WorkbookManager {
             throw new Error("Workbook path is not set");
          }
          await this.workbookStorage.saveWorkbook(
-            this.packageContext,
             this.workbookData.workbookPath,
             JSON.stringify(this.workbookData),
          );
          this.isSaved = true;
       }
-      return new WorkbookManager(
-         this.workbookStorage,
-         this.packageContext,
-         this.workbookData,
-      );
+      return new WorkbookManager(this.workbookStorage, this.workbookData);
    }
 
    /**
@@ -188,11 +177,8 @@ export class WorkbookManager {
          .join("\n");
    }
 
-   static newWorkbook(
-      workbookStorage: WorkbookStorage,
-      packageContext: PackageContextProps,
-   ): WorkbookManager {
-      return new WorkbookManager(workbookStorage, packageContext, undefined);
+   static newWorkbook(workbookStorage: WorkbookStorage): WorkbookManager {
+      return new WorkbookManager(workbookStorage, undefined);
    }
 
    /**
@@ -204,16 +190,11 @@ export class WorkbookManager {
     */
    static async loadWorkbook(
       workbookStorage: WorkbookStorage,
-      packageContext: PackageContextProps,
       workbookPath: WorkbookLocator,
    ): Promise<WorkbookManager> {
       let workbookData: WorkbookData | undefined = undefined;
-      console.log("loadWorkbook", workbookPath);
       try {
-         const saved = await workbookStorage.getWorkbook(
-            packageContext,
-            workbookPath,
-         );
+         const saved = await workbookStorage.getWorkbook(workbookPath);
          if (saved) {
             workbookData = JSON.parse(saved);
          }
@@ -225,6 +206,6 @@ export class WorkbookManager {
             workbookPath: workbookPath,
          };
       }
-      return new WorkbookManager(workbookStorage, packageContext, workbookData);
+      return new WorkbookManager(workbookStorage, workbookData);
    }
 }

package/src/components/Workbook/WorkbookStorage.ts

@@ -13,42 +13,22 @@ export interface WorkbookLocator {
 
 export interface WorkbookStorage {
    // Lists all available workspaces for the context.
-   listWorkspaces(
-      context: PackageContextProps,
-      writeableOnly: boolean,
-   ): Promise<Workspace[]>;
+   listWorkspaces(writeableOnly: boolean): Promise<Workspace[]>;
 
    // Lists all available workbooks for the context.
    // Workbooks names are like S3 paths- / denote hierarchical
    // folders, but otherwise folders are not "real" objects
-   listWorkbooks(
-      workspace: Workspace,
-      context: PackageContextProps,
-   ): Promise<WorkbookLocator[]>;
+   listWorkbooks(workspace: Workspace): Promise<WorkbookLocator[]>;
 
    // Returns the workbook at the specific path, throws an exception if no such workbook exists (or cannot be accessed)
-   getWorkbook(
-      context: PackageContextProps,
-      path: WorkbookLocator,
-   ): Promise<string>;
+   getWorkbook(path: WorkbookLocator): Promise<string>;
 
    // Deletes the workbook at the specified path, or throws an
    // Exception on failure
-   deleteWorkbook(
-      context: PackageContextProps,
-      path: WorkbookLocator,
-   ): Promise<void>;
+   deleteWorkbook(path: WorkbookLocator): Promise<void>;
 
-   saveWorkbook(
-      context: PackageContextProps,
-      path: WorkbookLocator,
-      workbook: string,
-   ): Promise<void>;
+   saveWorkbook(path: WorkbookLocator, workbook: string): Promise<void>;
 
    // Moves workbook from the "from" path to the "to" path
-   moveWorkbook(
-      context: PackageContextProps,
-      from: WorkbookLocator,
-      to: WorkbookLocator,
-   ): Promise<void>;
+   moveWorkbook(from: WorkbookLocator, to: WorkbookLocator): Promise<void>;
 }