@powerhousedao/vetra 5.1.0 → 5.2.0-staging.1

package/dist/subgraphs/vetra-package/resolvers.js

@@ -1,148 +1,191 @@
-import { addFile, childLogger } from "document-drive";
-import { createVetraPackageDocument } from "../../document-models/vetra-package/gen/ph-factories.js";
-import { actions } from "../../document-models/vetra-package/index.js";
-const DEFAULT_DRIVE_ID = "powerhouse";
-const logger = childLogger(["VetraPackageResolvers"]);
+import { addFile } from "document-drive";
+import { setName } from "document-model";
+import { actions, vetraPackageDocumentType, } from "@powerhousedao/vetra/document-models/vetra-package";
 export const getResolvers = (subgraph) => {
     const reactor = subgraph.reactor;
     return {
         Query: {
-            VetraPackage: async (_, args, ctx) => {
+            VetraPackage: async () => {
                 return {
                     getDocument: async (args) => {
-                        const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                        const docId = args.docId || "";
-                        const doc = await reactor.getDocument(driveId, docId);
+                        const { docId, driveId } = args;
+                        if (!docId) {
+                            throw new Error("Document id is required");
+                        }
+                        if (driveId) {
+                            const docIds = await reactor.getDocuments(driveId);
+                            if (!docIds.includes(docId)) {
+                                throw new Error(`Document with id ${docId} is not part of ${driveId}`);
+                            }
+                        }
+                        const doc = await reactor.getDocument(docId);
                         return {
                             driveId: driveId,
                             ...doc,
                             ...doc.header,
-                            // these will be ripped out in the future, but for now all doc models have global state
-                            // TODO (thegoldenmule): once the gql interface is updated for arbitrary state, we can remove this
-                            state: doc.state.global ?? {},
-                            stateJSON: doc.state.global ?? "{}",
-                            revision: doc.header.revision.global ?? 0,
+                            created: doc.header.createdAtUtcIso,
+                            lastModified: doc.header.lastModifiedAtUtcIso,
+                            state: doc.state.global,
+                            stateJSON: doc.state.global,
+                            revision: doc.header?.revision?.global ?? 0,
                         };
                     },
                     getDocuments: async (args) => {
-                        const driveId = args.driveId || DEFAULT_DRIVE_ID;
+                        const { driveId } = args;
                         const docsIds = await reactor.getDocuments(driveId);
                         const docs = await Promise.all(docsIds.map(async (docId) => {
-                            const doc = await reactor.getDocument(driveId, docId);
+                            const doc = await reactor.getDocument(docId);
                             return {
                                 driveId: driveId,
                                 ...doc,
                                 ...doc.header,
-                                // these will be ripped out in the future, but for now all doc models have global state
-                                // TODO (thegoldenmule): once the gql interface is updated for arbitrary state, we can remove this
-                                state: doc.state.global ?? {},
-                                stateJSON: doc.state.global ?? "{}",
-                                revision: doc.header.revision.global ?? 0,
+                                created: doc.header.createdAtUtcIso,
+                                lastModified: doc.header.lastModifiedAtUtcIso,
+                                state: doc.state.global,
+                                stateJSON: doc.state.global,
+                                revision: doc.header?.revision?.global ?? 0,
                             };
                         }));
-                        return docs.filter((doc) => doc.header.documentType === "powerhouse/package");
+                        return docs.filter((doc) => doc.header.documentType === vetraPackageDocumentType);
                     },
                 };
             },
         },
         Mutation: {
             VetraPackage_createDocument: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const document = createVetraPackageDocument();
-                const documentId = document.header.id;
-                await reactor.addDocument(document);
-                let revert = false;
-                try {
-                    await reactor.addDriveAction(driveId, addFile({
-                        id: documentId,
-                        name: args.name,
-                        documentType: document.header.documentType,
+                const { driveId, name } = args;
+                const document = await reactor.addDocument(vetraPackageDocumentType);
+                if (driveId) {
+                    await reactor.addAction(driveId, addFile({
+                        name,
+                        id: document.header.id,
+                        documentType: vetraPackageDocumentType,
                     }));
                 }
-                catch (error) {
-                    logger.error(`Created document but failed to add file to drive. Reverting: ${error.message}`);
-                    revert = true;
-                }
-                if (revert) {
-                    try {
-                        await reactor.deleteDocument(documentId);
-                    }
-                    catch (error) {
-                        logger.error(`Failed to revert document creation! This means there was a document created but not added to a drive. DocumentId: ${documentId}, Error: ${error.message}`);
-                        throw error;
-                    }
+                if (name) {
+                    await reactor.addAction(document.header.id, setName(name));
                 }
                 return document.header.id;
             },
             VetraPackage_setPackageName: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageName({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageName(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageName");
+                }
+                return true;
             },
             VetraPackage_setPackageDescription: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageDescription({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageDescription(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageDescription");
+                }
+                return true;
             },
             VetraPackage_setPackageCategory: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageCategory({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageCategory(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageCategory");
+                }
+                return true;
             },
             VetraPackage_setPackageAuthor: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageAuthor({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageAuthor(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageAuthor");
+                }
+                return true;
             },
             VetraPackage_setPackageAuthorName: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageAuthorName({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageAuthorName(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageAuthorName");
+                }
+                return true;
             },
             VetraPackage_setPackageAuthorWebsite: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageAuthorWebsite({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageAuthorWebsite(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageAuthorWebsite");
+                }
+                return true;
             },
             VetraPackage_addPackageKeyword: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.addPackageKeyword({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.addPackageKeyword(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to addPackageKeyword");
+                }
+                return true;
             },
             VetraPackage_removePackageKeyword: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.removePackageKeyword({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.removePackageKeyword(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to removePackageKeyword");
+                }
+                return true;
             },
             VetraPackage_setPackageGithubUrl: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageGithubUrl({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageGithubUrl(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageGithubUrl");
+                }
+                return true;
             },
             VetraPackage_setPackageNpmUrl: async (_, args) => {
-                const driveId = args.driveId || DEFAULT_DRIVE_ID;
-                const docId = args.docId || "";
-                const doc = await reactor.getDocument(driveId, docId);
-                await reactor.addAction(driveId, docId, actions.setPackageNpmUrl({ ...args.input }));
-                return (doc.header.revision.global ?? 0) + 1;
+                const { docId, input } = args;
+                const doc = await reactor.getDocument(docId);
+                if (!doc) {
+                    throw new Error("Document not found");
+                }
+                const result = await reactor.addAction(docId, actions.setPackageNpmUrl(input));
+                if (result.status !== "SUCCESS") {
+                    throw new Error(result.error?.message ?? "Failed to setPackageNpmUrl");
+                }
+                return true;
             },
         },
     };

package/dist/subgraphs/vetra-package/schema.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"schema.d.ts","sourceRoot":"","sources":["../../../subgraphs/vetra-package/schema.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAG5C,eAAO,MAAM,MAAM,EAAE,YAiIpB,CAAC"}
+{"version":3,"file":"schema.d.ts","sourceRoot":"","sources":["../../../subgraphs/vetra-package/schema.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAE5C,eAAO,MAAM,MAAM,EAAE,YA0GpB,CAAC"}

package/dist/subgraphs/vetra-package/schema.js

@@ -1,33 +1,10 @@
 import { gql } from "graphql-tag";
 export const schema = gql `
   """
-  Subgraph definition for VetraPackage (powerhouse/package)
-  """
-  type VetraPackageState {
-    name: String
-    description: String
-    category: String
-    author: Author!
-    keywords: [Keyword!]!
-    githubUrl: URL
-    npmUrl: URL
-  }
-
-  type Author {
-    name: String
-    website: URL
-  }
-
-  type Keyword {
-    id: OID!
-    label: String!
-  }
-
-  """
-  Queries: VetraPackage
+  Queries: VetraPackage Document
   """
   type VetraPackageQueries {
-    getDocument(driveId: String, docId: PHID): VetraPackage
+    getDocument(docId: PHID!, driveId: PHID): VetraPackage
     getDocuments(driveId: String!): [VetraPackage!]
   }
 
@@ -39,7 +16,7 @@ export const schema = gql `
   Mutations: VetraPackage
   """
   type Mutation {
-    VetraPackage_createDocument(driveId: String, name: String): String
+    VetraPackage_createDocument(name: String!, driveId: String): String
 
     VetraPackage_setPackageName(
       driveId: String